feat(unic-archon-dlc): port /pr-review to a generic fan-out Archon workflow (redesign step 09)

apps/claude-code/unic-archon-dlc/.archon/commands/unic-dlc-pr-review.md

@@ -0,0 +1,96 @@
+---
+description: Run the unic-archon-dlc PR review workflow — seven intent-grounded aspects fanned out, then a summary comment + inline comments posted/updated on the current PR, with iteration-aware re-review.
+---
+
+# /unic-dlc-pr-review
+
+Runs the `pr-review` box: composes a shared **Intent Brief** (from the linked work items, Confluence/MD
+docs, the PR description, and `PRD.md`), fans out **seven review aspects** as parallel fresh nodes
+(code-quality, test-coverage, silent-failure, type-design, comment-rot, code-simplification, and an
+intent/AC-coverage check), synthesises the findings, **reconciles them against the prior iteration**
+(new / still-present / fixed / regressed), and — after a config-gated human confirm — posts or updates a
+single structured **summary comment** plus **inline comments** on the current PR.
+
+Generic and **self-contained**: it harvests `unic-pr-review`'s review learnings (confidence rubric,
+structured summary, hidden-marker idempotency, conditional spawn table, two-surface posting) **without
+its ADO code and without any runtime dependency** ([ADR-0016](../../docs/adr/0016-dlc-thin-process-layer.md)/
+[ADR-0017](../../docs/adr/0017-container-follows-structural-need.md)). Ported to the key-discriminated
+Archon node schema ([ADR-0011](../../docs/adr/0011-archon-schema-target.md)); design in
+[ADR-0026](../../docs/adr/0026-pr-review-generic-archon-harvest.md).
+
+## Usage
+
+```
+/unic-dlc-pr-review <slug>
+```
+
+`<slug>` is the session identifier from `/unic-archon-dlc:specs` / `tickets` / `build`. It scopes the
+review artefacts (`<artifacts_dir>/<slug>/pr-review/`) and locates `PRD.md`. There is **no PRD
+precondition** — intent is composed from whatever sources resolve.
+
+## What this workflow does
+
+1. **bootstrap** — parse the slug from `$ARGUMENTS`, read `.archon/unic-dlc.config.yaml`
+   (`artifacts_dir`, `gates.pr-review`, `pr-review.confidence_threshold`, `pr-review.inline_comments`,
+   `tracker.type`, `docs.*`, `project.branching`). Missing slug/config cancels cleanly.
+
+2. **prep** — identify the open PR + its description; compute the diff and **categorise** the changed
+   files (for the spawn gates); compose **one Intent Brief** from the linked work items, Confluence/MD
+   docs, the PR body, and `PRD.md` (recording any **contradictions across sources**); and detect the
+   **prior review iteration** by its hidden marker. Writes everything to `<artifacts_dir>/<slug>/pr-review/`.
+
+3. **7 aspect nodes** (parallel, fresh) — each reads the shared Intent Brief (**every aspect is
+   intent-grounded**) + the diff and emits findings scored on the **confidence rubric** (90–100 Critical
+   / 80–89 Important / 60–79 Minor / below the threshold dropped). **Spawn gates** run each aspect only
+   when meaningful: code-quality + intent-check always; tests/type-design/comment-rot/simplifier/
+   silent-failure gated on the changed-file categories.
+
+4. **synthesize** — merge + dedupe this run's findings, bucket by severity, assemble the summary
+   sections + Intent Check + "What's good".
+
+5. **reconcile** — the re-review coordinator: classify each finding against the prior iteration
+   (new / still-present / fixed / regressed), compute the "since iteration N−1" delta, and finalise the
+   summary comment (with the `Iteration N` marker + footer) and the inline-comment plan.
+
+6. **review-gate** — **HITL by default** (`gates.pr-review`); skipped in AFK. Shows the composed summary,
+   the finding counts, and any contradiction warnings; APPROVE to post, REJECT to halt without posting.
+
+7. **post** — post/update the **summary comment** (matched by the `<!-- unic-dlc-pr-review:iteration= -->`
+   marker, never author identity) and, when `inline_comments` and the tracker supports inline threads,
+   reconcile **inline comments** per finding (still-present → update, fixed → resolve, regressed →
+   reopen, new → create). Trackers without inline threads (jira / local-markdown) degrade to
+   summary-only.
+
+## Gates & AFK
+
+`gates.pr-review` (default `hitl`) governs the review-gate. Set it to `afk` to run unattended — the gate
+is skipped and both surfaces post directly. Posting is **advisory / non-blocking**: unlike `/qa`,
+`/pr-review` never merges anything, so there is no fail-closed merge guard here — the real merge
+checkpoint is `/qa`.
+
+## Prerequisites
+
+- The current branch has an open PR (for the summary + inline comments).
+- `.archon/unic-dlc.config.yaml` is present (from `/unic-archon-dlc:setup`).
+- The configured tracker CLI/MCP is reachable (`gh` / `az` / `jira`, or the `azure-devops-cli` skill).
+- Archon ≥ 0.5.0.
+
+## Configuration reference
+
+Read from `.archon/unic-dlc.config.yaml`:
+
+| Field                            | Type         | Default     | Description                                                                      |
+| -------------------------------- | ------------ | ----------- | -------------------------------------------------------------------------------- |
+| `gates.pr-review`                | `hitl`/`afk` | `hitl`      | HITL pauses at the review-gate before posting; AFK posts directly                |
+| `pr-review.confidence_threshold` | number       | `60`        | Findings below this confidence are dropped before posting                        |
+| `pr-review.inline_comments`      | boolean      | `true`      | Post inline per-finding comments in addition to the summary (where supported)    |
+| `artifacts_dir`                  | string       | `workflows` | Session artefact home (`<artifacts_dir>/<slug>/pr-review/`)                      |
+| `tracker.*`                      | object       | —           | Composed to read the PR, work items, and post comments (MCP-first, CLI-fallback) |
+| `docs.*`                         | object       | —           | Composed to fetch Confluence/MD intent sources                                   |
+| `project.branching`              | string       | `gitflow`   | `gitflow` → base `develop`; else `main` (for the merge-base diff)                |
+
+## Runs
+
+```
+archon workflow run unic-dlc-pr-review --input <slug>
+```