Add scope-discipline directives to the always-on working-style standard - #1
Merged
assert-not-singularity merged 3 commits intoAug 29, 2026
Conversation
…ndard Encode a standing principle against over-engineering for backward compatibility that nothing depends on. The default is the simplest design; compatibility is preserved only where a real consumer relies on it (a versioned/published API, an SLA, a production pipeline, or a persisted data format), and genuine uncertainty is resolved by asking rather than keeping both code paths. Update the README summary of working-style to match, and bump the plugin version so /preflight:update surfaces the change to already-scaffolded projects. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TsB9nGcmkk7WtAqfQ5Mq61
The always-on standard had one bullet on preferring simplicity but nothing on scope minimalism during implementation, while scalable-architecture nudges toward adding structure. Add four directives so the counterweight is in context on every task: solve the specific ask (not its generalization until a second caller exists), don't re-derive what the input already states, bound the fix by a one-sentence statement of the defect, and justify each touched file against the requirement rather than architectural symmetry. Refocus the existing "prefer the simplest solution" bullet on backward compatibility so it doesn't overlap the new generalization directive, and update the README summary. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TsB9nGcmkk7WtAqfQ5Mq61
The "solve the specific ask" bullet's absolute rule-of-two forbade the single-implementation interface that scalable-architecture intentionally permits for test fakes and swappable I/O boundaries. Soften MUST NOT to a heuristic and carve out I/O seams as a present need governed by that rule, keeping the anti-over-build teeth for feature generality. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TsB9nGcmkk7WtAqfQ5Mq61
There was a problem hiding this comment.
🟢 Approval recommended
Changes are limited to documentation/standards text plus a straightforward manifest version bump with no apparent behavioral risk.
Pull request overview
This PR updates the always-on working-style standard to add explicit “scope discipline” directives that steer agents toward minimal, non-overgeneralized implementations, and surfaces that update via documentation and a plugin version bump.
Changes:
- Added new working-style bullets covering simplest-solution bias, avoiding premature generalization, not re-deriving stated inputs, bounding fixes with a one-sentence defect, and justifying touched files by requirement.
- Updated the README’s summary line for
working-styleto reflect the new scope-discipline emphasis. - Bumped the plugin manifest version from
0.1.0to0.2.0so updates are discoverable via/preflight:update.
File summaries
| File | Description |
|---|---|
| README.md | Updates the working-style row summary to mention scope discipline / simplest-solution guidance. |
| plugin/templates/standards/working-style.md | Adds new always-on bullets that constrain scope, avoid unnecessary compatibility/generalization, and bound fixes. |
| plugin/.claude-plugin/plugin.json | Increments plugin version to surface the updated always-on standard to existing installs. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Agents frequently overcomplicate solutions — preserving backward compatibility nothing depends on, generalizing beyond the ask, re-deriving values the input already states, or extending every layer a pattern touches. The always-on
working-style.mdstandard covered honesty and propose-before-acting but had nothing on scope minimalism, whilescalable-architecture.mdnudges the other way (ports/adapters, boundary validation). This adds the missing always-in-context counterweight.What changed
All in
plugin/templates/standards/working-style.md(the always-on standard, so it applies to every task in every repo that installs preflight):scalable-architecture's single-implementation interfaces for test fakes / swappable I/O boundaries.Also:
working-style.0.1.0 → 0.2.0so/preflight:updatesurfaces the change to already-scaffolded projects.Notes
scalable-architecture.md— the new always-on bullets already supply the counterweight it was missing, and the generalization bullet explicitly defers seam decisions back to that rule where a project follows it.🤖 Generated with Claude Code
Generated by Claude Code