Parent
docs/issues/unic-archon-dlc/PRD.md
What to build
Quality and consolidation phase of build: a verification node that catches low-quality implementations, a goals-check coverage matrix that ties code back to the PRD, a consolidated report, and the human PR gate that follows.
In scope:
verification node:
- Runs the full test suite for the slug's touched packages.
- Checks coverage thresholds (from
.archon/unic-dlc.config.json).
- Detects stub patterns:
TODO / FIXME / empty function bodies that only return or pass / hardcoded sentinel values returned where logic was expected.
- Audits wiring across layers: component → API → DB; form → handler. Mismatches are reported with file:line references so the report is actionable.
goals-check node: reads docs/workflow/<slug>/PRD.md and produces a coverage matrix mapping each acceptance criterion (from every user story / issue) to implementation evidence (file paths, test names, or "MISSING"). The matrix is the artifact reviewers look at before merging.
report node: writes docs/workflow/<slug>/report.md consolidating:
- What was built (high-level summary of merged issues)
- Goals-check matrix (verbatim from the previous node)
- Test outcomes (pass/fail counts, coverage numbers)
- Decisions made during implementation (links to any new ADRs)
- Tech debt flagged for cleanup (consumed by slice 13)
- Human PR gate (
interactive: true) after report: opens a PR with the implementation plus report.md. Workflow pauses until approved. Rejected approvals return control to verification for another pass.
Out of scope: the /unic-dlc-review command itself (slice 11) — this slice's PR gate is the human review; slice 11 adds the automated AFK review.
Acceptance criteria
Blocked by
PRD: docs/issues/unic-archon-dlc/PRD.md
Migrated from: docs/issues/unic-archon-dlc/10-build-verification-goals-check-and-report.md (source removed after migration)
Touched by PRs: #33
Parent
docs/issues/unic-archon-dlc/PRD.mdWhat to build
Quality and consolidation phase of
build: a verification node that catches low-quality implementations, a goals-check coverage matrix that ties code back to the PRD, a consolidated report, and the human PR gate that follows.In scope:
verificationnode:.archon/unic-dlc.config.json).TODO/FIXME/ empty function bodies that onlyreturnorpass/ hardcoded sentinel values returned where logic was expected.goals-checknode: readsdocs/workflow/<slug>/PRD.mdand produces a coverage matrix mapping each acceptance criterion (from every user story / issue) to implementation evidence (file paths, test names, or "MISSING"). The matrix is the artifact reviewers look at before merging.reportnode: writesdocs/workflow/<slug>/report.mdconsolidating:interactive: true) afterreport: opens a PR with the implementation plusreport.md. Workflow pauses until approved. Rejected approvals return control toverificationfor another pass.Out of scope: the
/unic-dlc-reviewcommand itself (slice 11) — this slice's PR gate is the human review; slice 11 adds the automated AFK review.Acceptance criteria
verificationfails when coverage is below threshold, when stub patterns are present, or when a wiring mismatch is detected.goals-checkproduces a coverage matrix where each row maps an acceptance criterion to evidence or "MISSING"; matrices with any "MISSING" rows fail the node.report.mdcontains all five sections and is committed todocs/workflow/<slug>/.verification.Blocked by
PRD:
docs/issues/unic-archon-dlc/PRD.mdMigrated from:
docs/issues/unic-archon-dlc/10-build-verification-goals-check-and-report.md(source removed after migration)Touched by PRs: #33