refactor(aidd-pm): migrate ticket-info, prd, and spec to router contract - #628
Merged
blafourcade merged 3 commits intoAug 14, 2026
Merged
Conversation
Moves tool/identifier lookup rules into references/tool-detection.md, standardizes the display output via assets/ticket-template.md, and replaces the old action table with the mermaid + | Action | Does | shape used by the other 7 aidd-pm skills. Part of #564; phase 1 of 3. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Splits the single prd action into draft/finalize (single responsibility each, matching 08-three-amigos), restores the 5+1 Transversal rules boilerplate shared by 6 other aidd-pm skills, drops the dead task-template.md asset, and inlines the save path into finalize instead of a one-fact references/ file. Part of #564; phase 2 of 3. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Moves the build-vs-refine dispatch into the mermaid as two entry nodes, gives TBD: one canonical spelling in references/tbd-marker.md (fixing the 2 drifted spellings in 02-refine.md), and keeps spec's own Transversal rules rather than the epic/task/prd boilerplate, which doesn't fit a skill with no approval gate. Also fixes two pre-existing gaps caught while auditing citations: spec-template.md was never actually linked from build.md, and the feature-folder resolve step was underspecified. Two follow-up issues filed for behavior gaps out of this issue's "no behaviour change" scope: #625 (SDLC never validates a spec before handing it to Deliver) and #626 (spec-template.md has no home for TBD markers, confirmed non-deterministic placement). Closes #564. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
blafourcade
marked this pull request as ready for review
August 14, 2026 07:55
blafourcade
deleted the
blafourcade/refactor-aidd-pm-migrate-ticket-info-prd-and-spe
branch
August 14, 2026 07:55
aidd-bot Bot
pushed a commit
that referenced
this pull request
Aug 14, 2026
…act (#628) * refactor(aidd-pm): migrate 01-ticket-info to router contract Moves tool/identifier lookup rules into references/tool-detection.md, standardizes the display output via assets/ticket-template.md, and replaces the old action table with the mermaid + | Action | Does | shape used by the other 7 aidd-pm skills. Part of #564; phase 1 of 3. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * refactor(aidd-pm): migrate 03-prd to router contract Splits the single prd action into draft/finalize (single responsibility each, matching 08-three-amigos), restores the 5+1 Transversal rules boilerplate shared by 6 other aidd-pm skills, drops the dead task-template.md asset, and inlines the save path into finalize instead of a one-fact references/ file. Part of #564; phase 2 of 3. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * refactor(aidd-pm): migrate 04-spec to router contract Moves the build-vs-refine dispatch into the mermaid as two entry nodes, gives TBD: one canonical spelling in references/tbd-marker.md (fixing the 2 drifted spellings in 02-refine.md), and keeps spec's own Transversal rules rather than the epic/task/prd boilerplate, which doesn't fit a skill with no approval gate. Also fixes two pre-existing gaps caught while auditing citations: spec-template.md was never actually linked from build.md, and the feature-folder resolve step was underspecified. Two follow-up issues filed for behavior gaps out of this issue's "no behaviour change" scope: #625 (SDLC never validates a spec before handing it to Deliver) and #626 (spec-template.md has no home for TBD markers, confirmed non-deterministic placement). Closes #564. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
blafourcade
added a commit
that referenced
this pull request
Aug 14, 2026
…act (#628) * refactor(aidd-pm): migrate 01-ticket-info to router contract Moves tool/identifier lookup rules into references/tool-detection.md, standardizes the display output via assets/ticket-template.md, and replaces the old action table with the mermaid + | Action | Does | shape used by the other 7 aidd-pm skills. Part of #564; phase 1 of 3. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * refactor(aidd-pm): migrate 03-prd to router contract Splits the single prd action into draft/finalize (single responsibility each, matching 08-three-amigos), restores the 5+1 Transversal rules boilerplate shared by 6 other aidd-pm skills, drops the dead task-template.md asset, and inlines the save path into finalize instead of a one-fact references/ file. Part of #564; phase 2 of 3. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * refactor(aidd-pm): migrate 04-spec to router contract Moves the build-vs-refine dispatch into the mermaid as two entry nodes, gives TBD: one canonical spelling in references/tbd-marker.md (fixing the 2 drifted spellings in 02-refine.md), and keeps spec's own Transversal rules rather than the epic/task/prd boilerplate, which doesn't fit a skill with no approval gate. Also fixes two pre-existing gaps caught while auditing citations: spec-template.md was never actually linked from build.md, and the feature-folder resolve step was underspecified. Two follow-up issues filed for behavior gaps out of this issue's "no behaviour change" scope: #625 (SDLC never validates a spec before handing it to Deliver) and #626 (spec-template.md has no home for TBD markers, confirmed non-deterministic placement). Closes #564. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 What & why
Migrates
01-ticket-info,03-prd,04-specto the router contract the other 7aidd-pmskills already follow — closes the last gap in issue #564.🛠️ How it works
Each skill: mermaid flow +
| Action | Does |table + canonical lead-in replace the old| # | Action | Role | Input |shape; every rule an action/reference could hold moved out of the router.TBD:unified to one spelling in04-spec/references/tbd-marker.md(fixes 2 drifted spellings).03-prdsplit intodraft/finalize(single-responsibility actions, matching08-three-amigos) — a deliberate scope extension beyond pure shape migration, called out since #564 says "no behaviour change"; net behavior is unchanged, only action count changed. Two pre-existing gaps caught and fixed mid-migration:04-spec/actions/01-build.mdnever linkedspec-template.md;03-prd/assets/task-template.md(dead, name-colliding with10-task's real template) is deleted. Plugin-wide verified: all 10aidd-pmskills share the same router shape/order, every asset cited by a named action. Issue's own "eleven routers" count is off by one (repo has 10, 7 already matched + 3 here) — noted in plan.md.🧪 How to verify
node scripts/check-markdown-links.js plugins/aidd-pm/→ 0 brokennode scripts/check-skill-argument-hints.mjs→ cleanclaude -p --plugin-dir ./plugins/aidd-pm "/aidd-pm:<skill> ..."— correct output shape, correct persistence.Two follow-up issues filed for real behavior gaps found while dogfooding, out of this PR's shape-only scope: #625 (SDLC's Frame stage never validates a spec against
spec-validator.ymlbefore Deliver), #626 (spec-template.mdhas no home for TBD markers — confirmed non-deterministic).🔗 Linked issue
Closes #564
✅ I certify