Skip to content

[Workflow Hardening] Harden PAW workflow control state#300

Draft
lossyrob wants to merge 14 commits intomainfrom
feature/workflow-hardening
Draft

[Workflow Hardening] Harden PAW workflow control state#300
lossyrob wants to merge 14 commits intomainfrom
feature/workflow-hardening

Conversation

@lossyrob
Copy link
Copy Markdown
Owner

@lossyrob lossyrob commented Apr 2, 2026

Closes #299

Summary

Hardens PAW and PAW Review around durable embedded control state so workflow progression, review sequencing, status, and resume paths can rely on explicit artifact state instead of prompt memory alone.

Changes

  • embed compact hardened-state contracts for workflow and review artifacts and mirror active required items into TODOs
  • gate PAW transitions, review procedures, git/PR entrypoints, and review terminal outcomes on explicit control-state progression
  • align VS Code status, handoff, and configuration surfaces with the shared workflow contract without moving CLI behavior into src/
  • document hardened-state behavior, legacy best-effort mode, and review terminal-state semantics
  • resolve final-review follow-ups around final PR gating, legacy VS Code fallback, PAW Review handoff identifiers, and review-posting idempotency

Testing

  • �ash -lc ""./scripts/lint-prompting.sh skills/paw-pr/SKILL.md skills/paw-final-review/SKILL.md skills/paw-planning-docs-review/SKILL.md skills/paw-review-github/SKILL.md skills/paw-status/SKILL.md""

pm run lint

pm run compile

pm test

  • cd tests/integration && npx tsx --test tests/skills/control-state-reconciliation-content.test.ts tests/workflows/final-review.test.ts tests/workflows/review-terminal-state-tracking.test.ts

Artifacts

🐾 Generated with PAW

Rob Emanuele and others added 14 commits March 31, 2026 17:40
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@lossyrob lossyrob changed the title [Workflow Hardening] Planning artifacts [Workflow Hardening] Harden PAW workflow control state Apr 3, 2026
@lossyrob lossyrob added the bug Something isn't working label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Workflow hardening: externalize mandatory state and side-effect ownership across PAW and PAW Review

1 participant