Skip to content

Commit 4739fa4

Browse files
authored
🛡️ Stop unplanned implementation machinery (#808)
1 parent 06cf5fb commit 4739fa4

1 file changed

Lines changed: 40 additions & 0 deletions

File tree

‎.agents/implementor.md‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,40 @@ fallback, compatibility behavior or follow-up issue.
3737
Ordinary implementation details remain the Implementor's responsibility. Do
3838
not stop for choices that code, tests or primary documentation can resolve.
3939

40+
## Complexity checkpoints
41+
42+
An accepted outcome does not authorize machinery the plan never named. Stop
43+
before implementing and return the need to the Planner or Architect when the
44+
work would introduce any unplanned:
45+
46+
- parser, interpreter, template language or declarative rule engine;
47+
- registry, identity protocol, authority protocol or lifecycle framework;
48+
- public abstraction, package boundary or generic extension point; or
49+
- compatibility mechanism whose only purpose is preserving behavior the plan
50+
did not identify as an exact contract.
51+
52+
The same checkpoint applies when a simple data shape starts carrying executable
53+
semantics that another module must interpret. Tests for that machinery do not
54+
authorize it, and test fixtures do not count as additional product consumers.
55+
56+
Stop before continuing when implementation evidence shows that:
57+
58+
- one planned PR slice has crossed into another independently mergeable slice;
59+
- the files, packages, public contracts or new abstractions materially exceed
60+
the plan's stated scope;
61+
- the frozen evidence matrix needs a new row or an existing row cannot prove
62+
the accepted behavior; or
63+
- addressing review feedback would harden or expand an abstraction whose
64+
contract was never accepted.
65+
66+
Report the smallest concrete conflict: what the implementation needs, why the
67+
accepted mechanics are insufficient, which existing primitives were considered,
68+
and what scope the proposed change would add. Do not implement the proposed
69+
framework while waiting for that decision.
70+
71+
These checkpoints do not apply to an ordinary local helper or refactor whose
72+
behavior, ownership and evidence already fit the accepted plan.
73+
4074
## Working discipline
4175

4276
- Preserve unrelated user changes and use a separate worktree when the current
@@ -63,6 +97,12 @@ exact commit SHA together with every focused command run. `deno task lint`,
6397
`deno task check`, `deno task check:jsr`, the complete local suite, and CI are
6498
not prerequisites for that commit.
6599

100+
Do not defer the first feedback commit until every layer of a plan is complete.
101+
When the plan defines reviewable or independently mergeable slices, finish the
102+
current slice, run its frozen focused evidence, create its feedback commit and
103+
return it for review before beginning the next slice. Do not combine those
104+
slices merely because they belong to one issue.
105+
66106
Execute the Planner's frozen evidence matrix. When implementation evidence shows
67107
the matrix cannot prove a criterion, return that evidence to the Planner instead
68108
of inventing additional acceptance permutations. Report a non-blocking

0 commit comments

Comments
 (0)