Repository navigation
🛡️ Stop unplanned machinery at Implementor checkpoints - #808
Merged
Merged
Conversation
PR #808: 🛡️ Stop unplanned machinery at Implementor checkpoints1 files, +40 / -0 Scope✅ PR scope looks good. Structural✅ No structural bloat detected. Slop✅ Slop indicators look low. Static Analysis✅ Oxlint found no issues. CorrectnessNo extraneous code patterns detected. |
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
PR #807 reached 6,447 changed lines because the implementation introduced a
diagnostic language and an authority protocol that its accepted plan had not
named, then completed every downstream migration and test before returning for
feedback. The existing Implementor role says to avoid speculative abstractions,
but it does not say when unexpected machinery or scope growth must stop work.
What changes
Before:
An Implementor stops for a material product or architecture choice, but can
still interpret a broad outcome as permission to invent a generic mechanism and
prove that mechanism exhaustively before the first feedback commit.
After:
The Implementor stops before adding an unplanned language, rule engine,
protocol, framework, package boundary, generic extension point, or compatibility
mechanism. It also stops when implementation crosses a planned PR slice,
materially exceeds the plan, expands the evidence matrix, or review feedback
would harden an unaccepted abstraction.
When a plan defines reviewable slices, each slice produces its own focused
feedback commit before the next begins. One issue is no longer treated as a
reason to combine independently mergeable work.
What must stay true
checkpoint applies to unplanned architecture and material scope growth.
without implementing the proposed framework while awaiting a decision.
How to verify it
Read
.agents/implementor.mdfrom “Do not decide around the plan” through“Feedback commits.” The stop conditions precede working discipline, and the
slice checkpoint precedes execution of the complete frozen evidence matrix.
Scope
Included
Intentionally unchanged
Risks and limitations
intended outcome when the alternative is silently adding an unplanned
framework.
Scope confirmation