[nevridge]

Closes #400. Merges the API, web and MCP feature-factory pipelines into one. Sub-issue A (#436, rules, settings and hooks) already merged to main in #439; B, C and two follow-ups were built and reviewed on feature/pipeline-merge and land here together.

What changes

• Agents (chore: consolidate the feature-factory agents into seven in root .claude/agents #437, PR chore: consolidate the feature-factory agents into seven in root .claude/agents #445): seven project agents in root .claude/agents/, all model: sonnet: backend-builder, frontend-builder, assistant-builder, test-verifier, implementation-validator, claude-integration-advisor, ux-advisor. The assistant/.claude/ agents (six files) and its separate feature-factory skill are removed. ux-planner now points at ux-advisor.
• Orchestrator (chore: replace the two feature-factory skills with one orchestrator skill #438, PR chore: replace the two feature-factory skills with one orchestrator skill #446): one feature-factory skill in .claude/skills/feature-factory/ replaces the two. Stages: preflight, research, story (stop 1, skipped for groomed issues), brief (stop 2, with the UX and Claude-integration advisors when the Scope block calls for them), build (api, web, assistant; one commit each), verify, validate, correct (shared budget of 2, respec budget of 1), finish. Run state lives in .feature-factory/<issue-or-slug>/ (gitignored) and survives a restart. Finish posts an <!-- implementation-notes --> comment and opens the PR.
• fix: four project agents do not register (unquoted colon in description frontmatter) #448 (PR fix: quote description frontmatter in four agents (#448) #449): four agents did not register because an unquoted : in the description: frontmatter made the file invalid YAML. Descriptions are now double-quoted.
• fix: make test-verifier read the story ACs and the test bodies #451 (PR fix: make test-verifier read the story ACs and the test bodies (#451) #452): test-verifier takes the AC list from story.md, cites the test and name behind every covered AC, and may only report UNTESTABLE or NOT COVERED as unverified.
• assistant/README.md: one line updated for the new pipeline.

17 files, +302 / -1549. Prompt, agent and docs files only; no application code.

Verification

• chore: consolidate the feature-factory agents into seven in root .claude/agents #445, chore: replace the two feature-factory skills with one orchestrator skill #446, fix: quote description frontmatter in four agents (#448) #449 and fix: make test-verifier read the story ACs and the test bodies (#451) #452 were each reviewed against their issue's acceptance criteria and merged into this branch with CI green.
• End-to-end run on a throwaway issue ([Throwaway] Show open subtask count badge on task list rows #447, draft PR feat: open subtask count badge on task list rows (#447) #450, both closed unmerged): every stage ran, the stops occurred, the Scope block skipped assistant with a reason, both advisors behaved as designed, and the notes comment had its five sections. It surfaced fix: four project agents do not register (unquoted colon in description frontmatter) #448 and the fix: make test-verifier read the story ACs and the test bodies #451 gap, both fixed here. Result: chore: replace the two feature-factory skills with one orchestrator skill #438 (comment)
• All seven agents' frontmatter parses with js-yaml.

Not verified

• AC 12 on chore: replace the two feature-factory skills with one orchestrator skill #438 (the agent-teams environment variable in the builder session) was not confirmed.
• The manual 360px light and dark check from the throwaway run was not done.
• Agents load at session start; restart Claude Code after merging to pick up the changes.

Review notes

No tests apply: agent prompts, a skill and docs. CodeQL, the Docker scan and dependency review run only on PRs to main, so this is their first run for these changes. Review is required.

[nevridge]

Review: 💬 Comments

This PR cleanly consolidates the three pipelines into one orchestrator skill and seven project agents, with no application code touched. Two small gaps in the new feature-factory skill were found in review and fixed in the follow-up commit.

#	Severity	Location	Finding	Status
1	🟠 Important	.claude/skills/feature-factory/SKILL.md:77	Verifier code failures are routed to Correct but never forwarded to a builder, and Halt ignores a still-failing gate	✅ Fixed in 4c5ee84
2	⚪ Nit	.claude/skills/feature-factory/SKILL.md:73	Validator checklist names TaskFlow.Contracts, which does not exist	✅ Fixed in 4c5ee84

Fixes pushed

Commit 4c5ee84 addresses: [1], [2]

---

_{Reviewed at 4c5ee84 · findings from pr-review + code-review, verified and triaged by @nevridge with Claude Code}