Skip to content

🛡️ Stop unplanned machinery at Implementor checkpoints - #808

Merged
taras merged 1 commit into
mainfrom
agent/implementor-complexity-stop
Sep 12, 2026
Merged

taras merged 1 commit into
mainfrom
agent/implementor-complexity-stop

Conversation

@taras

@taras taras commented Sep 12, 2026

Copy link
Copy Markdown
Owner

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

  • Ordinary local implementation choices remain with the Implementor; the new
    checkpoint applies to unplanned architecture and material scope growth.
  • The Implementor reports the concrete conflict and considered primitives
    without implementing the proposed framework while awaiting a decision.
  • The existing focused-evidence and feedback-commit contract remains unchanged.

How to verify it

git diff --check

Read .agents/implementor.md from “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

  • Implementor complexity stop conditions.
  • Early feedback commits for planned reviewable slices.

Intentionally unchanged

Risks and limitations

  • Implementors may return for an architecture decision more often. That is the
    intended outcome when the alternative is silently adding an unplanned
    framework.

Scope confirmation

  • Every changed file supports the purpose described above.
  • Unrelated cleanup and formatting changes are excluded.
  • Generated or mechanical changes are clearly identified.
  • The description matches the final diff and test results.

@github-actions

Copy link
Copy Markdown

PR #808: 🛡️ Stop unplanned machinery at Implementor checkpoints

1 files, +40 / -0

Scope

✅ PR scope looks good.

Structural

✅ No structural bloat detected.

Slop

✅ Slop indicators look low.

Static Analysis

✅ Oxlint found no issues.

Correctness

No extraneous code patterns detected.

@taras
taras merged commit 4739fa4 into main Sep 12, 2026
37 checks passed
@taras
taras deleted the agent/implementor-complexity-stop branch September 12, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant