Skip to content

fix(implementation): remove integration context mechanism#81

Merged
leeovery merged 1 commit intomainfrom
imple-v4
Feb 7, 2026
Merged

fix(implementation): remove integration context mechanism#81
leeovery merged 1 commit intomainfrom
imple-v4

Conversation

@leeovery
Copy link
Copy Markdown
Owner

@leeovery leeovery commented Feb 6, 2026

Summary

What was removed

  • Integration context file accumulation ({topic}-context.md) and all references
  • Plan file access for per-task executor (kept for polish agent which legitimately needs it)
  • Expanded exploration instructions (7 bullets → V2's 3 bullets)
  • Codebase Cohesion review dimension (feat: add technical-implementation skill #6) — reviewer now has 5 dimensions
  • INTEGRATION_NOTES, COHESION_NOTES, CODEBASE_COHESION output fields
  • Convention Consistency section from code-quality.md

Files changed (9)

  • agents/implementation-task-executor.md
  • agents/implementation-task-reviewer.md
  • agents/implementation-polish.md
  • skills/technical-implementation/SKILL.md
  • skills/technical-implementation/references/code-quality.md
  • skills/technical-implementation/references/steps/invoke-executor.md
  • skills/technical-implementation/references/steps/invoke-reviewer.md
  • skills/technical-implementation/references/steps/invoke-polish.md
  • skills/technical-implementation/references/steps/task-loop.md

Test plan

  • Grep for orphaned references: integration context, INTEGRATION_NOTES, COHESION_NOTES, CODEBASE_COHESION, Convention Consistency — all clear
  • Verify reviewer says "five" not "six" in process step and hard rules
  • Verify executor has 5 standard + 2 re-attempt inputs
  • Verify reviewer has 3 inputs
  • Verify polish agent has 6 inputs
  • Read each file end-to-end for orphaned sentences

🤖 Generated with Claude Code

…itions)

V3's integration context created a "convention gravity well" where early
design choices got documented as established patterns and later tasks
faithfully propagated them. V2 without this mechanism won 21/23 tasks.

Removes: integration context file accumulation, plan file access for
executor, expanded exploration instructions, codebase cohesion review
dimension, Convention Consistency quality section.

Keeps: fix recommendations (PR #78), fix_gate_mode (PR #78), full
re-attempt context (PR #77), polish agent (PR #80).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@leeovery leeovery merged commit 815fa4f into main Feb 7, 2026
@leeovery leeovery deleted the imple-v4 branch February 7, 2026 11:57
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