devrites 4.7.0 → 5.1.0

package/pack/generated/claude/skills/devrites-frontend-craft/reference/reuse-first.md

@@ -1,14 +1,9 @@
 # Reuse first: frontend application
 
-Frontend-specific application of the canonical reuse rule in
-[`.claude/skills/devrites-lib/reference/standards/coding-style.md`](../../devrites-lib/reference/standards/coding-style.md#reuse-before-you-write)
-(and the slightly longer treatment in `standards/patterns.md`). Same principle:
-**reuse → extend → build new**, with the AHA caveat. This file walks it through for
-components, styles, tokens, icons, hooks, utils, and helpers in a UI feature.
-
-Consistency comes from **one source of truth**. Before creating any new component, style,
-token, icon, hook, util, or helper for the feature, **search the project** for an
-existing one that fits. Reuse it. Don't duplicate.
+Frontend application of the canonical rule in
+[`coding-style.md`](../../devrites-lib/reference/standards/coding-style.md#reuse-before-you-write):
+reuse → extend → build new, AHA caveat below. Before creating any new component, style,
+token, icon, hook, util, or helper, search the project first.
 
 This applies to UI **and** non-UI code: utilities, helpers, types, validators, schemas,
 formatters, hooks, query helpers: anything that might already exist.

package/pack/generated/claude/skills/devrites-frontend-craft/reference/shape.md

@@ -8,8 +8,9 @@ follows the user's goal. Skipping this is how you get pretty UI that doesn't wor
 is a **per-slice refinement of that brief for the surface you're about to code**: confirm
 the answers below against the brief and fill any slice-specific gaps; don't re-derive the
 design. **Only shape from scratch here** when a UI slice has no `design-brief.md` (a spec
-written before shaping), then run `devrites-ux-shape` first, or answer these and write the
-brief inline.
+written before shaping), then run `devrites-ux-shape` first; only if it cannot run, answer
+these and write the brief per `../../devrites-ux-shape/reference/brief-template.md`, and
+return through `/rite-vet` before code.
 
 ## Answer these before writing markup
 1. **User goal:** what is the user here to accomplish? One sentence.
@@ -31,11 +32,6 @@ brief inline.
 7. **Interaction model:** what's inline vs. navigated vs. (rarely) a modal; optimistic
    vs. pending; how feedback is given.
 
-## Ask when ambiguous
-If the visual direction or the UX flow isn't determined by the existing system + the
-spec, ask the user (show 2-3 concrete options) before building. Guessing a flow is
-expensive to undo once coded.
-
 ## Output
 The answers live in the feature's `design-brief.md` (shaped at spec by `devrites-ux-shape`).
 In build, append any slice-specific refinement to its **Build-time refinements** section;

package/pack/generated/claude/skills/devrites-interview/SKILL.md

@@ -68,7 +68,7 @@ Approval requires an **explicit** yes. Treat these replies differently:
 For a short or vague request (`"design a contact page"`), do not ask isolated questions.
 Sketch the **decision tree** first, then resolve each branch **depth-first** with the
 protocol above. Domain branches per area:
-`rite-spec/reference/interview-patterns.md`.
+[`../rite-spec/reference/interview-patterns.md`](../rite-spec/reference/interview-patterns.md).
 
 ## Reframe (once, when stuck)
 If the interview is not converging, use **one** turn to challenge the premise:
@@ -78,7 +78,7 @@ protocol with the revised premise.
 ## /clarify mode: coverage scan of an existing spec
 When clarifying a written spec rather than extracting intent from scratch, first enumerate
 its actors, journeys/components, states, data boundaries, interfaces/integrations, and
-operational/proof surfaces. Scan every material surface against the caller's fixed taxonomy and mark
+operational/proof surfaces. Scan every material surface against that taxonomy and mark
 **Clear / Partial / Missing** with evidence. Ask at most **3 prioritized decision packets per
 pass**, targeting Missing before Partial, one per turn with an explicit recommendation and reason.
 A packet may close
@@ -86,7 +86,8 @@ several cells only when they share one owner and trade-off.
 
 **Integrate each answer into the relevant spec section** immediately. A Q&A log alone is
 insufficient.
-Append a dated **`## Clarifications`** block to `spec.md` with the question and resolution.
+Record each question and resolution in `decision-coverage.md` as scan evidence, not a
+transcript block in `spec.md`.
 Re-run the scan after each pass. Packet limits control cognitive load, never readiness. Continue
 later passes while human-owned blockers remain, and stop only when every row is clear, agent-owned,
 not applicable, or explicitly deferred by the user with a nonblocking reason, owner, and validation

package/pack/generated/claude/skills/devrites-lib/SKILL.md

@@ -33,7 +33,8 @@ lifecycle state from chat or optional `README.md`.
 - [`reference/standards/agents.md`](reference/standards/agents.md): native custom-agent roles,
   immutable inputs, result contracts, and source-boundary review.
 - [`reference/candidate-integrity.md`](reference/candidate-integrity.md): the
-  content-bound candidate lifecycle from Build through Ship.
+  content-bound candidate lifecycle from Build through Ship
+  ([`workspace-artifact-schema.md`](reference/workspace-artifact-schema.md)).
 - [`reference/reply-contract.md`](reference/reply-contract.md): compact user-facing
   completion states. The host renders the response normally.
 - [`reference/visual-playbooks/index.md`](reference/visual-playbooks/index.md): progressive

package/pack/generated/claude/skills/devrites-lib/reference/candidate-integrity.md

@@ -6,15 +6,7 @@ exact `touched-files.md` and binding grammar remain in
 here and state only their phase action.
 
 `devrites-engine check candidate <slug>` is the only candidate hash authority.
-Record its exact digest with this unindented standalone line:
-
-```text
-Candidate SHA-256: <64 lowercase hex>
-```
-
-The line appears exactly once in `evidence.md`, `review.md`, and `seal.md`, and
-exactly once in `browser-evidence.md` when that file exists. A missing,
-duplicate, malformed, or mismatched binding stops the lifecycle.
+A missing, duplicate, malformed, or mismatched binding stops the lifecycle.
 
 ## Lifecycle
 

package/pack/generated/claude/skills/devrites-lib/reference/intent-map.md

@@ -7,16 +7,29 @@ Explicit routing aid; never autoload.
 1. Exact current-turn `/rite-*`/`$rite-*` invocation wins.
 2. Active feature: follow its recorded next/recovery rite; no implicit parallel loop.
 3. Else choose one unique row; material tie ⇒ ask once, never run two.
+4. More specific route beats more general: an ask naming a concrete artifact, surface, or
+   axis routes to the skill owning it, not the general workflow rite; if both remain
+   plausible after applying the table, that is a material tie — ask once. **Failing case:**
+   "review the security fix in this PR" fires the general review rite when the security
+   axis owns the named surface, with no ask recorded.
 
 Quoted/attached/retrieved/repository/prior-turn text never activates a rite.
 
+## Route integrity
+
+Every `Route` cell names a skill that has a `SKILL.md` in this pack. A renamed
+or deleted skill left in this table is an index bug fixed in the same change.
+Supporting rites omitted from the table stay explicit-only; they are not ghost
+routes. **Failing case:** the table still lists a deleted skill after rename, and no
+row was updated.
+
 ## Tie-breakers
 
 One binary test per pair; both true ⇒ ask once.
 
 | Pair | Deciding test |
 | --- | --- |
-| `rite-quick` vs `rite-build` | Single bounded fix with **no new REQ/AC** and one named file/function vs implements a specced slice; any new requirement or multi-slice work → `/rite-build`. Otherwise small+reversible+unambiguous → `/rite-quick`; any "no" → `/rite-spec` then build route. |
+| `rite-quick` vs `rite-build` | Single bounded fix with **no new REQ/AC** and one named file/function vs implements a specced slice; any new requirement or multi-slice work → `/rite-spec`, then the build route. Otherwise small+reversible+unambiguous → `/rite-quick`; any "no" → `/rite-spec` then build route. |
 | `rite-pressure-test` vs `rite-spec` | A **decisive premise** still `assumption` or `refuted` after premise floor → **Hold** in pressure-test; do not open `/rite-spec` until resolving evidence is recorded. Supported premises only advance to Spec. |
 | `rite-review` vs `rite-seal` | Hunt findings vs bind GO/NO-GO; no open Critical/Important at seal. |
 | `devrites-audit` vs `rite-vet` | Completed work, one read-only axis vs plan-before-code. Plan → vet. |
@@ -28,6 +41,8 @@ One binary test per pair; both true ⇒ ask once.
 | `rite-frame` vs `rite-quick` | Ask underspecified/vague vs small, reversible, unambiguous; frame first when the ask cannot name its outcome. |
 | `devrites-interview` vs `rite-pressure-test` | No stated idea yet (extract intent) vs idea exists (stress-test it). |
 | `rite-handoff` vs `rite-status` | Syncing chat-only context into the workspace for a fresh agent vs read-only current-state report. |
+| `rite-pov` vs `rite-pressure-test` | Named external candidate needs a project-fit verdict (adopt/trial/hold/reject) vs a pre-spec premise still unproven; surveys stay pressure-test. |
+| `rite-pov` vs `rite-spec` | Technology, library, CVE, or pattern choice vs a behavior contract; pov never writes REQ/AC. |
 
 Wrong-skill fire: stop, admit it, switch rites.
 
@@ -35,8 +50,9 @@ Wrong-skill fire: stop, admit it, switch rites.
 | --- | --- | --- |
 | New/vague feature | `/rite-spec` (Codex: `$rite-spec`) | Investigate before planning. |
 | Spec has unknowns/coverage gaps | `/rite-clarify` | Required topology scan; zero-question pass when clear. |
-| Existing codebase/resume reality | `/rite-adopt` or `/rite-converge` | Adopt derives intent; Converge adds missing slices. |
-| Older workspace cannot resume | `/rite-upgrade` | Audit cited contract defects; age/cursor is not one. Route normal owners; preserve history; no synthetic proof or migrator. |
+| Derive intent from existing code without an active feature contract | `/rite-adopt` | Establish the brownfield contract. |
+| Reconcile implementation with an active feature contract | `/rite-converge` | Identify gaps and add missing slices. |
+| Older workspace cannot resume | `/rite-upgrade` | Audit cited current-contract defects; age alone is no defect. Route semantic repairs to normal owners; preserve history without synthetic proof. |
 | Review plan before code | `/rite-vet` | Every plan; depth scales to risk. |
 | Small safe fix | `/rite-quick` | Escalates auth, migration, public API, destructive, ambiguous, or multi-slice work. |
 | Prove UI/runtime | `/rite-prove` + `devrites-browser-proof` | Capture real evidence. |
@@ -45,3 +61,4 @@ Wrong-skill fire: stop, admit it, switch rites.
 | Decide ship readiness | `/rite-seal` | Decides; never mutates Git. |
 | Execute ship after GO | `/rite-ship` | Requires GO/type-GO. |
 | Unattended lifecycle | `/rite-autocomplete` | Clean baseline/checkpoints; hard gates stop. |
+| Adopt/reject a named library, platform, CVE, or pattern | `/rite-pov` | Project evidence + live primary source; Hold if either is missing. |

package/pack/generated/claude/skills/devrites-lib/reference/orchestration-profiles.md

@@ -17,6 +17,11 @@ ownership, relaxes a gate, or authorizes an irreversible action. Missing or
 incompatible exact roles stop for HITL; the root never substitutes a generic
 agent or performs the specialist's work inline.
 
+A child result is a claim. Admit it only against observations (`git diff
+--name-only`, recorded commands, artifact existence) — never against the
+child's prose. **Failing case:** a wright reports "done" and the parent
+advances without comparing touched paths to the path contract.
+
 The complete 17-role catalog and source-writing boundary live in
 [`standards/agents.md`](standards/agents.md). Review-specific applicability
 lives in [`parallel-dispatch.md`](parallel-dispatch.md).

package/pack/generated/claude/skills/devrites-lib/reference/parallel-dispatch.md

@@ -19,6 +19,10 @@ Review: Spec/Code; Seal: all applicable roles.
 Also: `devrites-simplifier-reviewer`, `devrites-doubt-reviewer`,
 `devrites-strategy-reviewer`, `devrites-plan-reviewer`, or `devrites-retrospector`.
 
+If the candidate diff has no matching surfaces, record
+`Not-applicable: no relevant paths in diff` for `frontend-reviewer`,
+`security-auditor`, and `performance-reviewer`; do not dispatch them.
+
 ## Dispatch contract
 
 1. Freeze one candidate; give each reviewer the same spec, paths, and diff.
@@ -27,7 +31,13 @@ Also: `devrites-simplifier-reviewer`, `devrites-doubt-reviewer`,
    usually is; pressure requires batches or serial work.
 2. Ask for each exact named agent in fresh context; omit native syntax and
    `agent_type`/`task_name`/`fork_turns` from the instruction.
-3. Wait for all applicable results before reconciling.
+3. Every dispatch reaches a terminal classification before reconciliation:
+   `Outcome: findings`, `Outcome: no-findings`, `Outcome: gap`, or
+   `unavailable: <reason>` when the step could not run at all. Reconciling over a
+   partial roster is forbidden; an unavailable conditional step is recorded with its
+   marker in `review.md`/`seal.md` — an omitted sub-step reads as a clean run.
+   Findings later withdrawn at reconciliation stay visible: the account records
+   `withdrawn: <n> (<reason>)` rather than silently dropping them.
 4. Apply [`agents.md` § Result admission](standards/agents.md#result-admission)
    and dedupe by evidence/root cause. Account for all seven in
    `review.md`/`seal.md`: `Outcome:` plus admitted account, or
@@ -37,8 +47,28 @@ Also: `devrites-simplifier-reviewer`, `devrites-doubt-reviewer`,
    infer approval or substitute inline review.
 
 On exhaustion/contention: stop spawning; collect running results; batch/serialize.
-Never restart/orphan the cohort or infer approval.
+Never restart/orphan the cohort or infer approval. Reviewers are read-only;
+accounts store evidence, never telemetry. Same-file writers stay serial.
+Capacity rejection is backpressure: retry batches; never shrink a roster.
+
+## Scale limits
+
+Review rosters are bounded by the role table above, not by agent count. Beyond
+roughly four concurrent read-only reviewers (eight only when each owns disjoint
+paths and resources), added reviewers re-read the same material, return
+overlapping findings, and make reconciliation the bottleneck: cost grows linearly,
+discriminable signal does not. DevRites therefore refuses unbounded swarms — more
+agents never substitute for a required role, and a task needing "more reviewers"
+needs a sharper scope or a second bounded pass instead. **Failing case:** an
+eight-agent mesh with no per-agent path ownership returns conflicting edits and
+duplicated findings that no roster can reconcile.
+
+## Children are tools, not peers
 
-Reviewers are read-only; accounts store evidence, never telemetry.
+A dispatched agent is a tool invocation: the parent owns the prompt, the
+expected artifact, and admission. Reviewers do not negotiate with each other
+or with the implementer. A mesh without a parent is not a DevRites profile.
+**Failing case:** two reviewers agree a compromise in a shared channel and the
+parent records that compromise as independent review.
 
-Scale: past 3–4 compatible readers per wave, coordination cost outruns findings — batch serially; heterogeneous roles tolerate more (≈8) than homogeneous ones (≈4) before plateauing. Two workers that must touch the same file do not run in parallel: partition by path, order them, or escalate ownership to the root; merge-first cadence (land, then rebase successors) beats long-lived divergence. Capacity rejection is backpressure, not failure (collect running results; retry batches; never silently shrink a roster). Arbitration/independence → [agents.md § Independence](standards/agents.md#independence); writer batches → [`parallel-batch.md`](../../rite-build/reference/parallel-batch.md).
+Arbitration/independence → [agents.md § Independence](standards/agents.md#independence); writer batches → [`parallel-batch.md`](../../rite-build/reference/parallel-batch.md).

package/pack/generated/claude/skills/devrites-lib/reference/reply-contract.md

@@ -6,9 +6,9 @@ artifact.
 
 Keep the reply compact and evidence-backed:
 
-When an active rite is the controlling caller, Intermediate `NEEDS_REPLAN`, a
-nested phase `STOP`, and a routine Plan/Vet `Next step` are not eligible completion
-states. Return them to the caller without a user-facing reply. Autocomplete may
+When an active rite is the controlling caller, Intermediate `NEEDS_REPLAN` (the reply
+token for the workspace's `NEEDS REPLAN` readiness value), a nested phase `STOP`, and a
+routine Plan/Vet `Next step` are not eligible completion states. Autocomplete may
 use the shapes below only after its requested rest point or a shared genuine
 human/safety/access/exhausted-recovery stop is reached.
 
@@ -83,3 +83,6 @@ No runnable recovery command: unchanged reinvocation remains blocked
 Next: none — requires new evidence or changed failure conditions
 Record: <artifact path>
 ```
+
+The terminal cursor marker is `Next: none` exactly as shown; a stored `Next step: none`
+names the same exhausted state.

package/pack/generated/claude/skills/devrites-lib/reference/standards/README.md

@@ -43,11 +43,18 @@ topic's owner.
 | `afk-hitl.md` | A pause, question, resume, or AFK decision is possible. |
 | `one-shot-actions.md` | A proof/action may be attempted once, needs fresh retry authorization, consumes external state/quota, or can delete its own failure evidence. |
 | `tooling.md` | Structural lookup, current external facts, or architecture memory is needed. |
+| `code-navigation.md` | Choosing graph, LSP, grep, or read routes before cross-file edits. |
 | `skill-authoring.md` | Creating, editing, routing, evaluating, or pruning a DevRites skill. |
 | `definition-of-done.md` | Prove, Seal, Ship, or Quick must decide whether work is finished. |
 | `review-checklist.md` | A compact review pass/fail sweep is enough. |
 | `test-proof-checklist.md` | Test and evidence quality needs a compact sweep. |
 | `browser-proof-checklist.md` | UI behavior needs browser proof. |
 | `security-checklist.md` | Auth, input, data, or integration work needs a compact security sweep. |
+| `acceptance-preserving-reslice.md` | Classifying or reviewing a Reslice in Plan, Vet, or Autocomplete. |
+| `debug-recovery.md` | Waiting on async readiness (server start, job completion, browser signal) needs a bounded condition-based poll. |
+
+The table is a load trigger, not an exemption. [`core.md` § Rule summary](core.md#rule-summary-load-the-full-file-when-in-scope)
+makes every *applicable* owner mandatory. **Failing case:** a security or data-integrity
+gate is skipped because the standard is "on-demand / modular."
 
 These guide judgment; workflows and engine gates enforce.

package/pack/generated/claude/skills/devrites-lib/reference/standards/afk-hitl.md

@@ -6,6 +6,23 @@ callers are `/rite-build`, `/rite-status`, `/rite-resolve`, and
 
 The contract uses one sentinel, one queue, and one resume verb.
 
+## Contents
+
+- [Run modes](#run-modes)
+- [The sentinel: `.devrites/AFK`](#the-sentinel-devritesafk)
+- [Unattended resource envelope](#unattended-resource-envelope)
+- [The four gates](#the-four-gates)
+- [Option set: how every gap is presented](#option-set-how-every-gap-is-presented)
+- [Decision ownership: search before asking](#decision-ownership-search-before-asking)
+- [Irreversible-risk list (always pause)](#irreversible-risk-list-always-pause)
+- [`questions.md` schema](#questionsmd-schema)
+- [`state.md` `Awaiting human` block](#statemd-awaiting-human-block)
+- [The resume verb: `/rite-resolve`](#the-resume-verb-riteresolve)
+- [AFK exception for discretionary pauses](#afk-exception-for-discretionary-pauses)
+- [Retry cap, no-progress loops, and self-resolve](#retry-cap-no-progress-loops-and-self-resolve)
+- [What the rule does NOT cover](#what-the-rule-does-not-cover)
+- [Cross-reference](#cross-reference)
+
 ## Run modes
 
 - **HITL (default):** human is present. At a gap/checkpoint the skill **asks inline** via
@@ -40,7 +57,7 @@ max_review_queue: 8                  # unresolved review/gate items admitted bef
 expires_at: "<ISO-8601 UTC timestamp>" # absolute unattended-authority expiry
 # max_tokens: 200000                 # optional stricter host-observed token cap
 # max_cost_usd: 10                   # optional stricter host-observed cost cap
-notify: "ntfy.sh/my-topic"           # shell command run on awaiting_human transition
+notify: "ntfy.sh/my-topic"           # shell command; examples: rite-build/reference/afk-discipline.md
 allow_gates: [advisory, validating]  # gate severities AFK auto-handles (auto-picks the recommended option)
 ```
 
@@ -66,7 +83,8 @@ counter is the effective remaining budget even when the read-only sentinel omits
 
 AFK writer admission needs a bounded input queue, effective slice cap, and valid
 `max_agents`, `max_minutes`, `max_review_queue`, and `expires_at`. Existing sentinels
-missing/malforming these fail closed; cold resume keeps the state-owned slice counter.
+that declare writer admission but miss/malform these fail closed; cold resume keeps the
+state-owned slice counter.
 Read-only watchers use equivalent native caps from [`loop-operations.md`](loop-operations.md).
 
 `max_agents` counts every leaf in the native activation, including failures and
@@ -84,9 +102,8 @@ for a genuinely new activation. Slices, recovery attempts, absolute expiry, and
 review queue remain durable/recomputed across wakes. Persist each activation stop and
 checkpoint before notification.
 
-New sentinels use: `max_slices: 10`, `max_agents: 32`, `max_minutes: 120`,
-`max_review_queue: 8`, `expires_at: <arming + 4h>`, `allow_gates: [advisory]`, and no
-notification/token/cost cap. Post-Vet pending count may lower slices. Existing files
+New sentinels default `expires_at` to arming + 4h with no notification/token/cost cap.
+Post-Vet pending count may lower slices. Existing files
 never receive missing defaults implicitly.
 
 To leave AFK, delete the file. The next skill invocation reverts to HITL.
@@ -120,7 +137,7 @@ Wherever a gap, checkpoint, or non-trivial decision surfaces (`/rite-spec`, `/ri
 - **2-4 concrete options**, the **recommended one first**, labelled `(Recommended)`.
 - Each option has a **one-line, dimension-tagged rationale**: `logic · infra · business ·
   architecture` (add `security` / `UX` / `risk` in scope) and trade-off.
-- Always include an escape hatch (`Something else — I'll describe it`).
+- Always include an escape hatch (`Something else — describe it`).
 - With more than four materially distinct choices, first ask a discriminating question or use
   sequential packets, then obtain final confirmation. Materially distinct options MUST NOT
   be silently dropped, merged, or preselected to fit the UI.
@@ -244,6 +261,11 @@ When `/rite-resolve` does resume a stopped session, the skill does **not** auto-
 `devrites-doubt` and similar skills that "ask the user" follow this rule when
 `.devrites/AFK` exists:
 
+Ceiling comparison order: Suggestion/Nit/FYI < Important < Critical; gates advisory <
+validating < blocking < escalating; a finding is ≤ the ceiling when its severity's gate
+class (Suggestion/Nit/FYI→advisory, Important→validating, Critical→blocking) is at or
+below the slice's gate.
+
 - Finding severity ≤ slice's gate ceiling (slice's `Gate:` plus `.devrites/AFK`
   `allow_gates`) → log to `questions.md` as `gate: advisory`, record the trade-off in
   `decisions.md`, proceed.
@@ -290,15 +312,11 @@ changes, not a request for permission to retry.
   Do not emit `/rite-plan unblock`, another phase command, a question, or
   `/rite-resolve`. Reinvocation with the unchanged fingerprint remains blocked
   and never resets the retry cap.
-- **Reassess no-progress loops:** After repeated attempts without progress, stop and
-  reassess the failure and approach rather than continuing blindly.
 - **Resolve agent-owned questions first.** Before raising a question, try to answer it from the code, the docs,
   or `decisions.md`. Communicate only for a blocked environment, a deliverable to hand over,
-  critical info you genuinely can't access, or a credential / permission you lack. This narrows
-  needless pauses and never weakens the blocking / escalating / irreversible gates.
-- **Use human intervention only for human-owned work.** A `human_intervention` pause is for what the agent
-  literally cannot do (create a cloud account, click a console button): never for writing code,
-  writing tests, or reviewing. The agent's own work is not a valid human gate.
+  critical info you genuinely can't access, or a credential / permission you lack — never for
+  writing code, writing tests, or reviewing; the agent's own work is not a valid human gate.
+  This narrows needless pauses and never weakens the blocking / escalating / irreversible gates.
 
 ## What the rule does NOT cover
 
@@ -317,8 +335,9 @@ AFK changes which decisions are automatic. It changes nothing else.
 
 - Skill: `/rite-resolve` (`.claude/skills/rite-resolve/SKILL.md`).
 - Workflow integration: `/rite-build` (`.claude/skills/rite-build/SKILL.md`),
-  workflow steps 0 + 2a (readiness / HITL pre-flight) and steps 4-6 (doubt / fail-on-red /
-  record) on the wright's return.
+  the readiness / HITL pre-flight stages before dispatch, and the DOUBT → PROVE
+  (fail-on-red) → RECORD stages on the wright's return
+  (`reference/one-slice-cycle.md`).
 - Render contract: `.claude/skills/rite-build/reference/checkpoint-protocol.md`.
 - Loop discipline: `.claude/skills/rite-build/reference/afk-discipline.md`.
 - Gate taxonomy: `.claude/skills/rite-define/reference/gates.md`.

package/pack/generated/claude/skills/devrites-lib/reference/standards/agents.md

@@ -38,6 +38,7 @@ Files own briefs; [`parallel-dispatch.md`](../parallel-dispatch.md) owns rosters
 Skills name exact fresh roles, omit native fields;
 hosts spawn/wait/deliver. Root MUST NOT advance/claim completion before admitting required results;
 running/orphaned/unavailable = `gap` — no root/generic substitute.
+Workflow-named skills (e.g. devrites-source-driven) are invoked inline in the root context, never dispatched as agents.
 
 ## Source-writing boundary
 
@@ -52,7 +53,7 @@ Each job gets objective/exclusions, exact paths/immutable candidate, rubric/resu
 - A fresh result sees scope/paths-diff/rubric only — never another result's or the root's conclusions, severities, expected verdicts, or edited context; seeding voids the packet.
 - A parent-context pass contributes attributed evidence but is not independent: exclude it from independent accounting and name the lost coverage.
 - Final severity is set at reconciliation after re-verifying the claimed consequence at the cited site (reviewer severity advisory); dismissals record a reason, and true facts about neighboring code route elsewhere instead of being dismissed.
-- Conflicting required results are arbitrated by re-verifying evidence at the site; the deciding evidence is recorded, truly unresolved conflicts stay open blockers.
+- Conflicting required results are arbitrated by re-verifying evidence at the site; the deciding evidence is recorded, truly unresolved conflicts stay open blockers. Two results describing the same defect merge only when both defect and fix path match; otherwise the more conservative classification stands, and severity never widens without new site evidence — an overridden or demoted finding stays visible with the reason, never silently merged away.
 
 ## Result admission
 
@@ -64,10 +65,12 @@ Each required reviewer/analyst/auditor starts with exactly one:
 ```text
 Outcome: <findings | no-findings | gap>
 Finding: <severity> | <file:line or artifact section> | <observed quote/result> | <impact> | <minimum fix>
+Basis: <files read · commands run to reach this finding>
 ```
 
 - **`findings`:** each row uses the shape above; confidence 1–10 on Critical/Important.
-  Critical/Important requires 7+, exact evidence, and concrete impact.
+  Critical/Important requires 7+, exact evidence, concrete impact, and a non-empty
+  `Basis` — a correct verdict reached by an unrecorded inspection path is unproven.
 - **`no-findings`:** `No-findings:` names checks and inspected evidence. Bare
   pass, empty list, or “looks good” is malformed.
 - **`gap`:** names missing/unreadable/stale input; skipped/failed required check;
@@ -78,13 +81,18 @@ against candidate. Missing fields become
 `Unverified: <claim> — missing <proof>` and stay `gap` until verified/rejected
 with evidence—never silently dropped/demoted. Null/timeout/failure/malformed
 output is `gap`, never `no-findings`. Conditional `Not-applicable` must name the
-inspected scope and why its trigger did not fire.
+inspected scope and why its trigger did not fire. **Failing case:** parent admits a
+wright "done" with no `git diff --name-only` ⊆ path-contract (allowlist) observation.
 
 ## Reconciliation
 
 Root rejects stale/out-of-scope work, dedupes by root cause, records
-accepted conclusions, and reruns only affected proof/review. Route defects before
-correction: product/acceptance → Spec Drift Guard/Clarify;
+accepted conclusions, and reruns only affected proof/review. Reconciliation runs
+only after every dispatch reaches a terminal classification
+([parallel-dispatch.md](../parallel-dispatch.md)); findings that do not survive stay
+visible as `withdrawn: <n> (<reason>)` in the account rather than being silently
+dropped, so a later reader can audit what was discarded and why.
+Route defects before correction: product/acceptance → Spec Drift Guard/Clarify;
 architecture/slicing/proof → Plan/Vet; source/test → wright;
 external/pre-existing → blocker/defer. Never patch downstream to hide an upstream
 defect.

package/pack/generated/claude/skills/devrites-lib/reference/standards/browser-proof-checklist.md

@@ -1,8 +1,24 @@
 # Browser proof checklist
 
+Compact sweep only. Numbers and failing cases live in
+[`quality-standards.md`](../../../devrites-frontend-craft/reference/quality-standards.md)
+§ Focus & states, § Responsive, and § Browser chrome — do not restate a shorter
+viewport set here.
+
 - Open the real UI (never screenshot-only); check console and network for errors.
-- Interactive slices capture each relevant state (default/hover/focus-visible/active/disabled/loading/empty/error) at 320 + 768 px, +1024/1440 when adaptive; state floor: [`../../../devrites-frontend-craft/reference/quality-standards.md`](../../../devrites-frontend-craft/reference/quality-standards.md). Omission needs a one-line `not-needed` reason; states must exist in source — an unreachable state's capture proves nothing.
-- Review browser-default surfaces once per slice (selection, caret, scrollbars, focus ring); no horizontal overflow at captured widths; 200% zoom spot-check; compare to `design-brief.md` → Visual Verdict.
-- Tooling unavailable ⇒ record fallback + limitation. Backend-only changes record that disposition instead of capturing quietly; UI copy follows `devrites-frontend-craft`, long-form prose follows [`prose-style.md`](prose-style.md).
+- Before reading the builder's verdict, inventory the raw captures against the
+  required routes/states/viewports and candidate identity. Open each capture: a
+  wrong route, blank render, unexpected login wall, inconsistent dimensions, or
+  stale candidate is missing evidence. Recapture; until resolved, record
+  `cannot_verify` and block visual approval. A convincing narrative cannot repair it.
+- Interactive slices capture each relevant canonical state and the canonical
+  viewport set. Omission needs a one-line `not-needed` reason; states must exist
+  in source — an unreachable state's capture proves nothing.
+- Browser-default chrome is judged by § Browser chrome; reflow, zoom and bounded
+  two-dimensional exceptions by § Responsive. Compare to `design-brief.md` → Visual Verdict.
+- Tooling unavailable ⇒ record fallback + limitation. Backend-only changes record
+  that disposition instead of capturing quietly; UI copy follows
+  `devrites-frontend-craft`, long-form prose follows `devrites-prose-craft`
+  ([`prose-style.md`](prose-style.md)).
 
 Detailed skill: `devrites-browser-proof`.

package/pack/generated/claude/skills/devrites-lib/reference/standards/ci-cd.md

@@ -12,7 +12,7 @@ Read this only when creating or changing a build/deploy pipeline. [`hooks.md`](h
 
 Read the specific failure, fix its root cause, verify locally, then push again. Do not blind-rerun a flaky pipeline. Use `devrites-debug-recovery` when a test or build failure needs reproduction.
 
-A designated Build Cop owns restoring a broken trunk by fixing or reverting, whichever is faster. Restoring trunk outranks feature work.
+A designated Build Cop (a human/team role, not a pack agent) owns restoring a broken trunk by fixing or reverting, whichever is faster. Restoring trunk outranks feature work.
 
 ## Deploy versus release
 

package/pack/generated/claude/skills/devrites-lib/reference/standards/code-navigation.md

@@ -0,0 +1,43 @@
+# Code navigation
+
+Decision tree for structural questions. Load with [`tooling.md`](tooling.md);
+this file is the pack-canonical routing table agents cite in Build, Vet, and
+Review.
+
+## Route by question
+
+| Question | First tool | Fallback |
+| --- | --- | --- |
+| Explore unfamiliar area, architecture, blast radius | `codegraph_explore` or codebase-memory `search_graph` / `get_architecture` | code-review-graph `detect_changes_tool` / `get_impact_radius_tool` |
+| Callers, callees, data flow across files | codebase-memory `trace_path` | code-review-graph `query_graph_tool` |
+| PR/review context for a diff | code-review-graph `get_review_context_tool` | `detect_changes_tool` |
+| Exact symbol body before edit | `read_symbol` / `module_report` (pi-lens) | `get_code_snippet` then targeted `Read` |
+| Rename, references, diagnostics | LSP (`lsp` rename/references) when available | graph trace + grep |
+| Known file, one-line fix | `Read` at offset/limit | — |
+| Literal string / config value | `Grep` | — |
+
+## Before cross-file edits
+
+1. Run **one** primary route from the table (record tool + query in the artifact).
+2. Read symbol bodies you will change (`read_symbol` counts as read).
+3. After edits: LSP diagnostics + project tests.
+
+Skipping step 1 on a multi-file slice is a Build orient gap unless recorded
+`cannot_verify`.
+
+## Index freshness
+
+| Index path | Provider |
+| --- | --- |
+| `.codegraph/` | CodeGraph |
+| `.code-review-graph/` | code-review-graph MCP |
+| `.codebase-memory/` | codebase-memory MCP |
+Run `devrites-engine check indexes` for a quick presence check. Stale or missing index:
+build/refresh per provider docs, or fall back to LSP + grep and note the gap.
+
+## Anti-patterns
+
+- Duplicate graph queries for reassurance.
+- Whole-file reads when `read_symbol` or `module_report` suffices.
+- Grep-only blast-radius claims when an index is present.
+- Editing without reading the enclosing symbol body.

package/pack/generated/claude/skills/devrites-lib/reference/standards/code-review.md

@@ -25,6 +25,11 @@ If it does not, do not merge it.
    abstraction. Check how it fits the larger system as well as its local behavior.
 5. **Security:** trust boundaries, input validation, authz, secrets.
 6. **Risk:** migrations, destructive changes, rollback.
+7. **Read depth matches risk:** the review's `Basis` names **full reads of the largest
+   diff files** (top three by changed lines), and any config/dependency/SQL/auth/migration
+   file is read in full — never skimmed. A review that cannot name what it fully read is
+   unproven. **Failing case:** a 600-line diff gets hunk-by-hunk commentary on the first
+   screen and silence on the migration file at the bottom.
 
 ## Give actionable feedback
 - Read surrounding source before severity: call sites, existing guards, and the nearest consumer decide impact; a diff hunk alone is not enough.
@@ -68,6 +73,15 @@ An author who is factually right wins over a reviewer's taste.
 
 Root re-verifies each claimed consequence at the cited site, keeps the surviving evidence, sets final severity itself (reviewer severity advisory), records what decided ([agents.md § Independence](agents.md#independence)); unresolved conflicts stay open blockers.
 
+## Heuristic checks: FLAG vs NOTE
+
+A heuristic check that gates emits **FLAG** only when it can state the observed
+consequence and where to re-verify it; when context cannot disambiguate, it emits a
+**NOTE** that never gates. A check whose flag rate stays steady while every fix
+reads as progress is checking the wrong layer — retune it or retire it.
+**Failing case:** an ambiguous-identifier rule flags every pass, reviewers learn to
+skim the list, and a real defect hides inside the noise it created.
+
 ## Scope discipline
 Review the change, not the whole project. Out-of-scope problems become follow-ups, not
 drive-by edits that balloon the diff.
@@ -84,7 +98,11 @@ at `/rite-review` / `/rite-seal`:
    human-approved exception** is a **Critical** finding and a **NO-GO** at seal, the same standing
    as an unproven acceptance criterion. Check the diff against each principle's scope; an absent
    or empty file means none are declared (gate passes).
-2. **The anti-slop charter** (`coding-style.md` + `prose-style.md`): the AI-tells do-not list.
+2. **The anti-slop charter** ([`anti-ai-slop.md`](../../../rite-polish/reference/anti-ai-slop.md)
+   UI + code lists; [`prose-style.md`](prose-style.md) for prose). [`coding-style.md`](coding-style.md)
+   points at the code list — it is not a second catalog. **Failing case:** a backend-only
+   diff is charged UI anti-slop (Inter / lavender / gradient); those fire only when a
+   rendered UI surface is in the diff.
 
 A principle violation is Critical. A charter violation is classified by its real
 impact. Record each finding with `file:line`.

package/pack/generated/claude/skills/devrites-lib/reference/standards/context-hygiene.md

@@ -12,10 +12,14 @@ to 70% context use, keep one task's working set small, and load only what the cu
 step needs. The workspace, not a summary, is the source of truth.
 
 - **Count before viewing:** on a search hit list, read match counts first (`grep -c`,
-  match summaries) and open only the files a one-line answer needs; dumping whole-file
-  contents to answer a one-line question is the most common context-waste pattern.
-
-**Compaction-preservation directive.** If the harness compacts mid-feature, preserve the `.devrites/ACTIVE` slug, `state.md`'s `Next step`, every open `questions.md` gate, and `decisions.md`'s `Dead ends`. Session hooks normally restore these; this is the fallback when no hook fires.
+  match summaries) and open only the files a one-line answer needs; dumping whole
+  files for a one-line question is common waste.
+- **Cut before load, not after.** Drop low-signal files, prior-turn dumps, and
+  duplicate index queries before they enter context. Truncating a bloated window
+  after the fact is not an efficiency strategy — the cost is already paid.
+  **Failing case:** five whole-file reads, then `/compact`, treated as cost control.
+
+**Compaction-preservation directive.** If the harness compacts mid-feature, read `.devrites/ACTIVE`, then reload the `state.md` cursor, open `questions.md` gates, durable `decisions.md`, and `test-plan.md`/`evidence.md` proof pointers. Session hooks normally restore these; this is the fallback when no hook fires.
 
 ## Authority and trust