Skip to content

fix: add self-check checklist to plan expert for format enforcement#783

Merged
FL4TLiN3 merged 5 commits intomainfrom
fix/plan-self-check
Mar 13, 2026
Merged

fix: add self-check checklist to plan expert for format enforcement#783
FL4TLiN3 merged 5 commits intomainfrom
fix/plan-self-check

Conversation

@FL4TLiN3
Copy link
Contributor

Summary

  • Plan expert was generating old-format sections (3 Test Queries, Success Criteria, Failure Conditions) instead of the new Hard Signal format
  • Add explicit binary self-check rules that plan must verify before finalizing plan.md:
    • Exact section name matching with explicit reject list for old names
    • Single test query enforcement
    • Every signal must be a concrete command with expected result
    • No soft language ("verify that", "should be", "looks correct")
    • Domain constraint coverage mapping (which signal covers which constraint)
    • Architecture contains names only, no implementation details

Test plan

  • Re-run create-expert and verify plan.md uses new section names
  • Verify Verification Signals contain commands, not observations
  • Confirm single Test Query instead of 3

🤖 Generated with Claude Code

FL4TLiN3 and others added 5 commits March 13, 2026 21:47
Plan was generating old-format sections (3 Test Queries, Success Criteria,
Failure Conditions) instead of the new format. Add explicit binary self-check
rules that must pass before plan.md is finalized:
- Exact section name matching (reject old names)
- Single test query enforcement
- Every signal must be a concrete command, not observation
- No soft language in signals
- Domain constraint coverage mapping

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The model was appending "-coordinator" to the user's team name (e.g.,
"bash-gaming" became "bash-gaming-coordinator"). Add explicit instruction
that the coordinator name must be the exact team name with no suffix.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…elegates

- Coordinator: use user-specified team name if given, no suffixes
- Delegates: name by function (@name/test, /verify, /build), not persona
  (/tester, /coder, /designer)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@FL4TLiN3 FL4TLiN3 enabled auto-merge (squash) March 13, 2026 21:58
@FL4TLiN3 FL4TLiN3 merged commit e91f5a3 into main Mar 13, 2026
11 checks passed
@FL4TLiN3 FL4TLiN3 deleted the fix/plan-self-check branch March 13, 2026 21:59
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