Skip to content

refactor: merge design-roles into plan, fix Built-in Verification principle#768

Merged
FL4TLiN3 merged 1 commit intomainfrom
refactor/merge-design-roles-into-plan
Mar 13, 2026
Merged

refactor: merge design-roles into plan, fix Built-in Verification principle#768
FL4TLiN3 merged 1 commit intomainfrom
refactor/merge-design-roles-into-plan

Conversation

@FL4TLiN3
Copy link
Contributor

Summary

  • Abolish design-roles and find-skill as separate pipeline stages — plan now handles requirements analysis + architecture design directly (601→481 lines)
  • Fix Built-in Verification principle: verifiers must execute artifacts to test whether they work, not just review code. Old wording ("inspects without re-executing") produced verifiers with only readTextFile — unable to actually verify anything works
  • Enforce the principle consistently across 3 stages: plan (design time), write-definition (self-check Fix: Handle individual close failures in closeSkillManagers #5), verify-test (Step 3 semantic review)

Test plan

  • bun run test passes (all 20 tasks successful)
  • Typecheck failures are pre-existing (@perstack/log module not found)
  • No remaining references to design-roles/find-skill in definitions
  • README tree diagram updated

🤖 Generated with Claude Code

…erification principle (#768)

Abolish design-roles and find-skill as separate pipeline stages. Plan now
handles both requirements analysis and architecture design directly.

Fix the Built-in Verification principle: verifiers must actually execute
artifacts to test whether they work, not just review code. The principle
is now enforced consistently across plan (design time), write-definition
(self-check), and verify-test (semantic review).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@FL4TLiN3 FL4TLiN3 enabled auto-merge (squash) March 13, 2026 15:12
@FL4TLiN3 FL4TLiN3 merged commit 54916b6 into main Mar 13, 2026
11 checks passed
@FL4TLiN3 FL4TLiN3 deleted the refactor/merge-design-roles-into-plan branch March 13, 2026 15:13
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