no-gold-plating
Installation
SKILL.md
No Gold-Plating
At higher effort, Fable 5's thoroughness can spill over into the diff: speculative abstractions, defensive checks for impossible states, cleanup nobody requested. Each of these enlarges review surface and risk. The discipline:
Rules
- The diff should map one-to-one to the request. A bug fix touches the bug; it does not reorganize the neighborhood.
- No helpers, layers, or abstractions for a single call site. Inline it.
- Do not design for requirements that don't exist yet. The simplest correct implementation wins; generality is a cost paid today for a benefit that may never arrive.
- Validate only at system boundaries — user input, external APIs, file contents. Inside the system, trust the framework and your own invariants; do not add error handling for states that cannot occur.
- Prefer changing code over compatibility shims, flags, or deprecation layers, unless the code is a published interface someone else depends on.
- If you notice genuinely valuable adjacent work, note it in one sentence at the end. Do not do it.
Self-check before finalizing a diff
- Could a reviewer trace every hunk back to the user's request in one step?
- Did I add any function, parameter, or branch "just in case"?
- Is any new error-handling path actually reachable?