Skip to content

fix: Phase 0 consistency, incremental QA, and user handoff#37

Closed
mythkiven wants to merge 1 commit into
revfactory:mainfrom
mythkiven:fix/phase-consistency-handoff
Closed

fix: Phase 0 consistency, incremental QA, and user handoff#37
mythkiven wants to merge 1 commit into
revfactory:mainfrom
mythkiven:fix/phase-consistency-handoff

Conversation

@mythkiven

Copy link
Copy Markdown

Summary

Fixes three cross-review issues: Phase number mismatch (Phase 1 vs Phase 0 context check), missing incremental QA guidance in orchestrator templates, and missing post-build user handoff step.

Motivation

Same fixes as open PR #6 — Phase 5-5 said "Phase 1" while templates use Phase 0; templates regress to waterfall QA; users don't know how to trigger the harness after build.

Scope of change

  • Skill / meta-skill logic — SKILL.md (Phase 5-5, 6-7, checklist)
  • Agent template(s) — orchestrator-template.md incremental QA section
  • Plugin manifest
  • Documentation (README/docs)
  • CHANGELOG.md
  • CI / GitHub Actions
  • Tests

Tests

  • Manual: grep confirms Phase 0 wording is consistent between SKILL.md and orchestrator-template.md.

CHANGELOG update

Y

SemVer impact

patch — documentation/consistency fixes, no breaking behavior.

Made with Cursor

Align SKILL.md Phase 5-5/checklist with orchestrator Template A Phase 0,
add incremental QA rules to orchestrator-template, and Phase 6-7 user
handoff guidance after harness build completes.

Co-authored-by: Cursor <cursoragent@cursor.com>
@mythkiven mythkiven closed this by deleting the head repository Jun 15, 2026
@mythkiven

Copy link
Copy Markdown
Author

Superseded by #44 — branch restored, PR re-created for genuine contribution. Please review #44 instead.

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