devrites 4.7.0 → 5.1.0

package/pack/generated/codex/skills/rite-seal/reference/final-evidence.md

@@ -9,10 +9,9 @@ results and exact `devrites-spec-reviewer` checks implementation and proof
 against its meaning. Bug fixes require original-symptom proof; self-report,
 inspection alone, and generic "tests pass" do not count.
 
-Behavioral claims require positive, discriminating evidence under `testing.md`.
-Skipped/focused/filtered/pending, zero-test, assertion-free, tautological, unexecuted, or
-exit-only results are unproven. Static gates prove only their named static criterion;
-discriminating shell/golden/text assertions remain valid for textual or CLI acceptance.
+Apply [`testing.md`'s positive, discriminating proof](../../devrites-lib/reference/standards/testing.md#positive-discriminating-proof),
+including invalid-result exclusions and static/textual criterion boundaries.
+Unsupported behavioral claims remain unproven: NO-GO.
 
 ## Cross-check
 
@@ -37,5 +36,5 @@ devrites-engine check seal "<slug>"
 ```
 
 This checks structure and exact candidate bindings, never acceptance, test
-quality, doubt, or review. Mutation stays optional unless project policy or
-`test-plan.md` requires it.
+quality, doubt, or review. Apply [`testing.md`'s safe perturbation](../../devrites-lib/reference/standards/testing.md#safe-perturbation)
+for its mandatory critical/regression cases and any additional project/test-plan requirement.

package/pack/generated/codex/skills/rite-seal/reference/risk-and-rollback.md

@@ -38,7 +38,7 @@ For each risky step, state how to back it out:
   or no outage/backlog recovery and monitoring is **NO-GO**.
 - Unsafe intermediate deployment order, unvalidated required configuration, or no watched
   rollout signal/owner is **NO-GO** for live exposure.
-- Record the chosen rollback path in `seal.md` → "Rollback / Recovery".
+- Record the chosen rollback path in `seal.md` → "Risks / Rollback".
 
 ## Fresh-context availability
 

package/pack/generated/codex/skills/rite-seal/reference/seal-template.md

@@ -18,6 +18,8 @@ Candidate SHA-256: <64 lowercase hex>
 - devrites-devex-reviewer: <outcome>
 ## Risks / Rollback
 <risks/recovery>
+## Stood Decisions
+<doubt verdict per stood decision>
 ## Blockers / Follow-ups
 <blockers/follow-ups>
 ## Final Decision

package/pack/generated/codex/skills/rite-ship/SKILL.md

@@ -13,7 +13,7 @@ if absent, route to `$rite-spec <feature>`. Ship only with current **GO** in `se
 ## Rules consulted (read on demand from `.agents/skills/devrites-lib/reference/standards/`)
 **Step 0:** Read `core.md`, then pull as needed:
 - `git-workflow.md`: Conventional Commits, atomicity, never-commit list.
-- `afk-hitl.md`: type-GO irreversible-action gate.
+- `afk-hitl.md`: AFK pause rules for irreversible actions (the type-GO gate itself: [`git-ship.md`](reference/git-ship.md), [`rollout.md`](reference/rollout.md)).
 - `definition-of-done.md`: acceptance, evidence, drift, rollback, docs.
 - [`release/ship-checklist.md`](../devrites-lib/reference/standards/release/ship-checklist.md): final pass/fail sweep.
 
@@ -77,7 +77,8 @@ if absent, route to `$rite-spec <feature>`. Ship only with current **GO** in `se
    unchanged pre-GO reconciliation.
 6. **Close.** Set `state.md` phase `done`; follow
    [`close-out.md`](reference/close-out.md) and run `devrites-engine state close <slug>`
-   to move `.devrites/work/<slug>` → `.devrites/archive/<slug>` and clear `ACTIVE`.
+   to move `.devrites/work/<slug>` → `.devrites/archive/<slug>` and clear `ACTIVE`
+   only if it names this slug; preserve another active feature's cursor.
    Preserve every `.md`; stable hierarchical instructions need no active-workspace sync.
 
 > **Mid-flight discipline.** No seal GO, skipped type-GO, out-of-manifest stage, or

package/pack/generated/codex/skills/rite-ship/reference/close-out.md

@@ -14,8 +14,16 @@ deleted. DevRites keeps the audit trail; it just moves out of the live path.
    It moves `.devrites/work/<slug>/` → `.devrites/archive/<slug>/` (every `.md`
    intact) and clears `.devrites/ACTIVE` **only if** ACTIVE still points at `<slug>`.
    It refuses to clobber an existing `.devrites/archive/<slug>/` (exit 5).
-3. **Confirm.** ACTIVE is now empty, so the next `$rite-spec` starts a clean feature
-   and `$rite-status` reports "no active feature".
+3. **Confirm the observed post-state.** The archive exists with its audit records
+   intact and the live workspace is absent. If ACTIVE named this slug, verify it is
+   empty; if it named another feature, verify that value remains unchanged. Only an
+   empty cursor permits the claim "no active feature".
+
+Any nonzero exit stops close-out. Archive collision (exit 5) never authorizes overwrite.
+If clearing ACTIVE fails, the engine attempts to move the archive back; that rollback
+can also fail. Inspect the actual live path, archive and cursor before reporting or
+retrying. Record the command/error and observed locations; never claim restored state
+from an attempted rollback or repeat a move against an unverified destination.
 
 ## Why archive, not delete
 
@@ -26,9 +34,21 @@ deleted. DevRites keeps the audit trail; it just moves out of the live path.
 
 ## Re-opening an archived feature
 
-To resume archived work, move it back:
-`mv .devrites/archive/<slug> .devrites/work/<slug>` and write `<slug>` into
-`.devrites/ACTIVE`. Nothing is lost: close-out is reversible by design.
+Before restoring an archive, verify its identity, released cursor form and declared
+workspace schema, that the live destination is absent, and that ACTIVE is empty or
+already names this slug. A collision or another active feature stops restoration;
+do not replace its workspace or cursor. Supported released cursor forms remain valid:
+age or encoding alone never requires normalization (see
+[workspace-artifact-schema.md](../../devrites-lib/reference/workspace-artifact-schema.md)).
+An unsupported declared schema stops restoration: record the observed version,
+installed engine's recovery diagnostic and pending normalization owner. Do not guess
+a repair or claim `$rite-upgrade` migrates archives; archived/done work is its no-op.
+
+After an authorized restoration, verify the actual paths and cursor and preserve the
+audit records. Reconcile the resumed scope through `$rite-converge`; changes return to
+the applicable earlier lifecycle stage. Historical proof/GO remains historical: a new
+candidate requires fresh Prove → Polish → Review → Seal bindings before Ship. Restoring files
+does not reactivate an old GO.
 
 Cross-feature review is user-invoked through `$rite-learn`; close-out does not
 mine, score, nudge, or write a retrospective ledger.

package/pack/generated/codex/skills/rite-ship/reference/rollout.md

@@ -6,7 +6,7 @@ runbook. A Seal GO or AFK setting never authorizes production action.
 
 ## Preconditions before exposure
 
-Record one rollout sheet in `ship.md`/`evidence.md`:
+Record one rollout sheet in `ship.md`:
 
 | Item | Required decision/evidence |
 | --- | --- |

package/pack/generated/codex/skills/rite-spec/SKILL.md

@@ -106,7 +106,8 @@ load a routed standard to discover required behavior, not to prescribe implement
    [state-workspace](reference/state-workspace.md). If the slug already exists,
    update the existing workspace rather than overwrite it; preserve history and
    unrelated settled content. Write every required artifact and conditional
-   annex exactly from [spec-template](reference/spec-template.md), including its
+   annex exactly from [spec-template](reference/spec-template.md)
+   ([ai-spec-template](reference/ai-spec-template.md)), including its
    capability impact, existing-behavior preservation, grammar/delta,
    stakeholder/constraint/invariant, failure/recovery, applicability, coverage-seed,
    qualified backstop, edge/prohibition, UI, and AI rules. Native

package/pack/generated/codex/skills/rite-spec/reference/anti-patterns.md

@@ -25,4 +25,3 @@ Pack-wide rationalizations + red flags: see [standards/anti-patterns.md](../../d
 - A `[NEEDS CLARIFICATION]` marker remains on a blocking item.
 - Design references were attached but never opened, saved, or indexed in `references.md`.
 - The investigation didn't read the module/component that currently owns this area.
-- A material decision landed in `spec.md` / `assumptions.md` that the human never picked from an option set: decided on the agent's confidence alone.

package/pack/generated/codex/skills/rite-spec/reference/investigation.md

@@ -47,9 +47,7 @@ Use native file search over `.devrites/archive/*/{spec,decisions}.md`.
    action-time decisions the acceptance path requires. Separate these from agent-owned
    implementation and diagnostic work.
 
-Gather any design or reference materials the human supplies, including screenshots,
-Figma, links, or video. See [references-intake](references-intake.md). Record whether
-each is a target, constraint, or inspiration for later phases.
+Design/reference materials the human supplies: see [references-intake](references-intake.md).
 
 ## Gap analysis (present → desired)
 State the present and desired states. Their delta defines the work; unknowns in that

package/pack/generated/codex/skills/rite-spec/reference/spec-checklists.md

@@ -76,7 +76,7 @@ Each question checks one requirement-prose failure mode:
 
 The spec **Readiness gate** at the bottom of
 [`spec-template.md`](spec-template.md) requires every
-emitted `checklists/<domain>.md` must reach `Verdict: pass` (zero CRITICAL fails) before the gate
+emitted `checklists/<domain>.md` to reach `Verdict: pass` (zero CRITICAL fails) before the gate
 passes. Minor fails are logged, not blocking. A single open CRITICAL keeps the spec `Status: Draft`.
 `$rite-define` reads the checklists at step 0 and **hard-blocks while any CRITICAL is
 unchecked**. A spec without checklists is not yet checked, so define stops and routes

package/pack/generated/codex/skills/rite-spec/reference/spec-template.md

@@ -138,7 +138,7 @@ status routes Define/Vet/Build/Prove to the named standard without copying it he
 | Q-001 | blocking | [NEEDS CLARIFICATION: <question>] | AC-001 |
 
 ## Readiness gate
-- [ ] No blocking clarification; REQ/AC IDs are valid and ACs independently provable.
+- [ ] No blocking clarification or open `blocking`/`escalating` question; REQ/AC IDs are valid and ACs independently provable.
 - [ ] Stakeholder conflicts/priority rules, constraints, and invariants are explicit;
       implementation preferences are not disguised as requirements.
 - [ ] Existing affected behavior maps to preserving REQ/AC + current evidence, or uses the exact justified greenfield `none`.

package/pack/generated/codex/skills/rite-spec/reference/state-workspace.md

@@ -14,6 +14,7 @@ Load that schema before creating or updating workspace artifacts.
       README.md                  # compact workspace map
       brief.md
       spec.md
+      checklists/                # from $rite-spec step 5a
       architecture.md            # from $rite-define
       flows.md                   # optional Mermaid-first; only when useful
       visual/                    # optional HTML+outline companions (never readiness)
@@ -21,6 +22,7 @@ Load that schema before creating or updating workspace artifacts.
       assumptions.md
       questions.md
       decision-coverage.md       # from $rite-clarify
+      strategy.md                # from $rite-temper
       plan.md                    # from $rite-define
       tasks.md                   # from $rite-define
       traceability.md            # from $rite-define
@@ -96,11 +98,12 @@ last_updated: <date>
 | brief.md | Objective and bounds |
 | spec.md | Product contract |
 | decision-coverage.md | Clarification verdict; absent until $rite-clarify |
+| strategy.md | Temper record; absent until $rite-temper |
 | architecture.md | Technical map; absent until $rite-define |
 | plan.md | Technical approach; absent until $rite-define |
 | tasks.md | Vertical slices; absent until $rite-define |
 | traceability.md | Coverage dashboard; absent until $rite-define |
-| evidence.md | Proof log; absent until $rite-prove |
+| evidence.md | Proof log; absent until $rite-build or $rite-prove |
 
 ## Read next
 | Phase / role | Read |
@@ -151,8 +154,7 @@ work preserves that valid cursor. Once the prerequisite chain is green, the
 controlling caller restores the saved phase/action and removes both rows in one
 rewrite. A real HITL or exhausted-recovery stop retains the cursor for cold
 resume. Never overwrite an existing valid return cursor, and preserve every
-unrelated Markdown byte. `$rite-clarify` applies its stricter native cursor
-protocol below.
+unrelated Markdown byte. `$rite-clarify` applies its stricter native cursor protocol in `$rite-clarify`.
 
 On cold resume, a terminal `next_action` is a claim to verify, not a counter.
 Reconcile the exact fingerprint and its attempts in `drift.md` / `evidence.md`.
@@ -177,6 +179,7 @@ slices remaining`; preserve that presentation rather than migrating it.
 status: open | answered | dropped
 slice: spec | plan | SLICE-001
 gate: advisory | validating | blocking | escalating
+route: <specialist tag> (escalating only; omit otherwise)
 question: <one crisp sentence>
 options:
   1. <recommended> (Recommended) - <trade-off>

package/pack/generated/codex/skills/rite-status/SKILL.md

@@ -28,7 +28,8 @@ If `state.md` is unreadable/malformed: report a gap with the defect and stop —
 4. the single `Next step` recorded in `state.md`;
 5. proven versus unproven evidence;
 6. open questions by gate, including the exact resolving command when awaiting
-   a human;
+   a human; list `gate: escalating` entries separately under **Escalating:**
+   with their `route:` specialist tag (do not mix with synchronous blockers);
 7. unresolved drift and material risks;
 8. handoff readiness.
 
@@ -47,6 +48,7 @@ Feature: <slug> — <objective>
 Phase: <phase>; slice: <slice|n/a>; mode: <HITL|AFK>; status: <status>
 Evidence: <fresh/proven summary | gaps>
 Open: <questions/drift/blockers | none>
+Escalating: <qid route:tag … | none>
 Handoff: <ready | missing durable context>
 Next: <single persisted command>
 ```

package/pack/generated/codex/skills/rite-temper/SKILL.md

@@ -53,7 +53,7 @@ the spec applicability map triggers that risk family.
    repair` (record in `drift.md`), not a blind spec edit.
 1. **Significance test:** [`reference/significance.md`](reference/significance.md). Low-stakes
    / shape-not-meaning work → write the exact one-line `skipped — low stakes (<trigger>)` verdict to `strategy.md`,
-   set `state.md` `Next step: $rite-define`, and recommend it. Otherwise fire the full pass below.
+   set `state.md` `Phase: temper` + `Next step: $rite-define`, and recommend it. Otherwise fire the full pass below.
 2. **FORWARD pass + mode selection:** [`reference/scope-modes.md`](reference/scope-modes.md).
    First, the **one-sentence-intent test**: state the whole change's intent in a single sentence.
    If you can't without an "and" that joins two unrelated outcomes, it is **two features**: the
@@ -85,21 +85,21 @@ the spec applicability map triggers that risk family.
    records the interactive review but does not replace it. Present each material scope decision through
    `AskUserQuestion`, one at a time, best-guess + **why**. Each material scope call ends as a
    **recorded decision**: a resolved `questions.md` qid (HITL) or a `decisions.md` ADR (AFK):
-   so the review leaves an auditable record outside chat. (AFK gate policy is single-sourced in
-   [`reference/significance.md`](reference/significance.md): `hold-rigor` + `reduce-to-MVP`
+   so the review leaves an auditable record outside chat. (AFK gate policy —
+   [`reference/significance.md`](reference/significance.md) — `hold-rigor` + `reduce-to-MVP`
    auto-apply, **any `expand` is a blocking pause**, irreversible-risk always pauses.)
    Apply objective clarity, mitigation, and assumption fixes directly.
 6. **Write `strategy.md` + fold back:** [`reference/strategy-template.md`](reference/strategy-template.md).
    **Choose the drift path based on whether a plan exists:** *no `plan.md` yet* (the normal pre-define
-   case) → update `spec.md` **through the Spec Drift Guard**. *Success criteria* **and**
-   *Acceptance criteria* for each opt-in expansion / cut, **Non-goals** for every deferred item,
-   *Constraints* **and** *Risks* the pre-mortem demands, and the gaps/decisions table; *`plan.md`
+   case) → update `spec.md` **through the Spec Drift Guard**. *Success metrics* (outcome-level, no AC id)
+   **and** *Acceptance criteria* for each opt-in expansion / cut, **Non-goals** for every deferred item,
+   *Constraints and invariants*, with top failure modes + mitigations folded into *Failure and recovery
+   behavior* rows, and the *Open questions* table; *`plan.md`
    already exists* → do **not** edit `spec.md` here: write the deltas to `drift.md` and hand off
    to `$rite-plan repair`. Either way, append `decisions.md` (one ADR per scope call: context ·
    decision · why-not · what-would-change-it) and `assumptions.md` (every "we'll probably need X" →
-   assumption-to-verify). **Every scope delta must carry a recorded human decision**: a resolved
-   `questions.md` qid (HITL) or a `decisions.md` ADR (AFK within the ceiling); a folded change with
-   no recorded decision is invalid. **Re-check the spec Readiness gate** (it fails if
+   assumption-to-verify). (every delta carries the step-5 recorded decision; an untraceable
+   fold is invalid). **Re-check the spec Readiness gate** (it fails if
    any folded scope delta lacks its decision or leaves a foreseeable human build choice).
    After any edit to `brief.md`, `spec.md`, `decisions.md`, `assumptions.md`, or
    `questions.md`, re-scan the affected coverage rows, assumption audit, residual uncertainty,

package/pack/generated/codex/skills/rite-temper/reference/review-dimensions.md

@@ -17,8 +17,8 @@ the gate uses the **lowest band**, not an average.
    human prerequisites and action-time approvals instead of deferring generic uncertainty to
    build. **Unmitigated top risks are gating.**
 5. **Over-engineering / YAGNI:** speculative capability, unused extension points, premature
-   abstraction, second-system bloat. Apply the pack's standard (`patterns.md`) + the "imagine
-   the later refactor" test. Defer unless now-cost is trivial AND deferred-cost is large.
+   abstraction, second-system bloat. Apply the pack's standard (`patterns.md`) and the YAGNI
+   ledger test (scope-modes.md Pass 2).
 6. **Acceptance testability and completion:** every acceptance criterion measurable, technology-
    agnostic, and comparable **down to a baseline** ("better than the current workaround"), not
    up to an unbounded ideal. Vague adjectives ("fast/robust/intuitive") and "handles X

package/pack/generated/codex/skills/rite-temper/reference/scope-modes.md

@@ -26,8 +26,8 @@ Invert. Two subtractive moves:
 - **YAGNI ledger.** Each candidate scope item gets the "imagine the later refactor" test: if
   adding it *later* isn't materially more expensive, **defer it** (with a revisit note).
   Bias hard toward defer: most presumed-needed features are never used. Reuse the
-  pack's existing `patterns.md` standards, including "no abstraction before two real callers"
-  and "speculative generality." Don't invent a new YAGNI rule.
+  pack's existing `patterns.md` standard ('no speculative abstraction or pattern
+  without a current need'). Don't invent a new YAGNI rule.
 
 ## The four modes: commit exactly one
 Mode selection is the first convergence move. Record the chosen mode + rationale + the

package/pack/generated/codex/skills/rite-temper/reference/significance.md

@@ -5,9 +5,6 @@ enough to earn it, a one-line skip otherwise. This is the shared definition used
 optional `$rite-temper` path and the always-run (significance-gated) `$rite-autocomplete` step,
 so they agree on when it fires.
 
-A strategic review of a five-line, reversible change is theatre; a strategic review of an
-auth rewrite is the cheapest insurance you'll buy. Gate on stakes and shape, not uniformly.
-
 ## Fire the FULL review when ANY trips
 - **Irreversible-risk contact:** the slice/spec touches anything on the `afk-hitl.md`
   irreversible-risk list: destructive data migration, auth/authz boundary, public-API break,
@@ -32,7 +29,10 @@ On skip, **don't run the passes**. Write only the one-line verdict to `strategy.
 
 ```markdown
 # Strategy: <slug>
-Significance: skipped — low stakes (<the trigger that was NOT met, e.g. "single-module, reversible, scope unambiguous">)
+Tempered: <iso>
+
+## 1. Significance
+skipped — low stakes (<trigger>)
 Next: $rite-define
 ```
 

package/pack/generated/codex/skills/rite-temper/reference/strategy-template.md

@@ -65,14 +65,14 @@ already exists, record in `drift.md` and route via `$rite-plan repair` instead o
 
 - **`spec.md`:** these are the spec template's actual section headings. Edit each named one;
   don't collapse pairs.
-  - Add to *Success criteria* **and** *Acceptance criteria* (separate sections) for
+  - Add to *Success metrics* **and** *Acceptance criteria* (separate sections) for
     every opt-in **expansion**; remove (via the Guard) from both for every **reduction**. Each
     must stay measurable + technology-agnostic.
   - *Non-goals*: append every deferred item (from the YAGNI ledger + the Deferred register), so
     it is neither silently dropped nor silently re-injected mid-build.
-  - *Constraints* **and** *Risks* (separate sections): tighten *Constraints* where the pre-mortem
-    demands; add the top failure modes + mitigations to *Risks*.
-  - *Gaps, issues & decisions* table: one row per scope call (options offered → decision → owner).
+  - *Constraints and invariants*: tighten where the pre-mortem demands; top failure modes +
+    mitigations go to *Failure and recovery behavior* rows.
+  - *Open questions* table: one row per scope call (options offered → decision → owner).
   - **Re-run the Readiness gate** at the bottom of the spec after edits; it must still pass, and
     **every folded scope delta must trace to a recorded decision** (a `questions.md` qid or a
     `decisions.md` ADR): an untraceable change is the batch-dump failure.
@@ -86,5 +86,5 @@ already exists, record in `drift.md` and route via `$rite-plan repair` instead o
   `devrites-strategy-reviewer` agent is **read-only** and only returns findings + bands.
 - **No silent scope.** An expansion that the human didn't opt into, or a reduction that drops an
   acceptance criterion without a recorded decision, is a defect, not a convenience.
-- Add `strategy.md` to the workspace between `spec.md` and `plan.md`; `$rite-define`,
-  `$rite-review`, and `$rite-seal` read it.
+- Add `strategy.md` to the workspace between `spec.md` and `plan.md`; `$rite-define` and
+  `$rite-ship` read it.

package/pack/generated/codex/skills/rite-upgrade/SKILL.md

@@ -8,7 +8,7 @@ disable-model-invocation: true
 
 # $rite-upgrade: reconcile a released workspace safely
 
-Use when an unfinished workspace from an older release cannot resume. This is a
+Use when an unfinished workspace from an older release cannot resume. This is an
 audit/orchestrator—not a pack update, cursor conversion, release replay,
 structural migration, or generic cleanup.
 
@@ -107,6 +107,6 @@ Changed: <admitted active-workspace paths | none>
 Evidence: assessment=current; candidate rc=<n|n/a>; seal rc=<n|n/a>; readiness rc=<n|n/a>; protected history/source unchanged
 Open: <none | exact unsupported shape, evidence gap, or human gate>
 Next: <state.md next action | one owning rite>
-Record: .devrites/work/<slug>/state.md
+Record: <owning rite's state.md — written by the owner, not Upgrade>
 ↻ Hygiene: /clear before the next lifecycle step
 ```

package/pack/generated/codex/skills/rite-vet/SKILL.md

@@ -10,8 +10,9 @@ user-invocable: true
 Vet every plan before code for scope, architecture, quality, proof, performance,
 failure modes, and writer safety. Cite findings; fold accepted technical
 hardening into planning artifacts; design Build tests. Temper owns product
-scope, Vet owns implementation; current `$ARGUMENTS` selects depth under
-[`orchestration-profiles.md`](../devrites-lib/reference/orchestration-profiles.md), never removing the exact plan-reviewer gate.
+scope, Vet owns implementation; current `$ARGUMENTS` (`--full`) feeds the depth triggers in
+`reference/depth.md`; profiles never remove the exact plan-reviewer gate
+([orchestration-profiles.md](../devrites-lib/reference/orchestration-profiles.md)).
 
 ## Rules
 
@@ -38,6 +39,8 @@ plan declares a root-authored executable workflow file, read
 - Root alone asks, decides, folds, writes, and sets readiness. Reviewers judge;
   they add no route policy. Cite every finding and confidence; suppress
   unverified or confidence ≤4 findings under `review-axes.md`.
+  Lens arrows in `eng-lenses.md` are heuristics; band findings only under the
+  four `review-axes.md` names.
 - Auth, migration, public API, and data-model changes use maximum caution and the
   irreversible-risk stop. Project principles never become trade-offs.
 - **Governance-protected paths** (`.devrites/**`, pack skill/agent trees,
@@ -79,9 +82,12 @@ plan declares a root-authored executable workflow file, read
    shared artifact/readiness gates and return to caller or next repair.
 2. **Challenge scope.** Apply review-axes §0 and search accepted decisions.
    Harden to the smallest contract-complete plan, using marked topology action.
-   Then verify bidirectional ID-and-meaning traceability across spec/tasks/
-   acceptance, terms, principles, anti-slop, and conventions. Critical gaps and
-   unexcepted principle breaches block; write `analysis.md` after recheck.
+   Then verify bidirectional ID-and-meaning traceability across spec/plan/
+   tasks/test-plan/traceability, acceptance, terms, principles, anti-slop,
+   and conventions; every slice and test-plan row maps to a live requirement
+   and vice versa. Critical gaps and
+   unexcepted principle breaches block; record the challenge result in `eng-review.md`
+   §2 (Scope challenge) after recheck.
 3. **Preflight Build entry.** Under `reference/artifacts.md`, verify exact
    command/cwd/tool/version/prerequisite; output filters must preserve upstream
    failure. Verify dependencies from authoritative source plus nearest manifest.
@@ -90,10 +96,8 @@ plan declares a root-authored executable workflow file, read
    Record complete SHA-256 provenance. Every behavioral mapping names a positive
    discriminating assertion and decisive signal, never only exit zero.
 
-   For each consumptive action, one-shot evidence completeness must bind durable
-   retention, trust-safe diagnostics, cleanup order, terminal coverage, finite
-   injective boundary map, per-boundary fault fixtures, and an executed collision mutant.
-   Every fingerprint identifies one actionable seam; aliasing multiple
+   For each consumptive action, bind every `reference/artifacts.md` Consumptive
+   action gates column; every fingerprint identifies one actionable seam; aliasing multiple
    emit sites is a gap. Preflight observes but need not make future behavior pass.
 4. **Audit readiness.** Goal-backward map every requirement, criterion, NFR,
    interaction, edge/prohibition, and decision row to one slice and executable
@@ -122,7 +126,7 @@ plan declares a root-authored executable workflow file, read
    fold again. A closed input plus a distinct Critical/Important invariant returns
    that new fingerprint as progress. Then close matrix and rerun ID/meaning audit.
 8. **Build readback and readiness.** Add a cited five-line readback to
-   `eng-review.md`: outcome/ACs; IN/OUT/must-NOT; UI direction and architecture/
+   `eng-review.md` (artifacts.md §7 rows 1–5): outcome/ACs; IN/OUT/must-NOT; UI direction and architecture/
    critical flow; slice order/first slice; decisive proof/action-time gates. A
    fresh implementer must need no product, architecture, or proof invention.
    Contradiction, ownerlessness, or material ambiguity blocks via Clarify or Plan.

package/pack/generated/codex/skills/rite-vet/reference/artifacts.md

@@ -12,12 +12,15 @@ Authority: `.agents/skills/devrites-lib/reference/standards/acceptance-preservin
 <!-- END RESLICE ROUTE-TO-ACTION -->
 
 ## `eng-review.md`: the record
+
 Write/update `.devrites/work/<slug>/eng-review.md`; never clobber.
 
 ```markdown
 # Eng review: <slug>
 Vetted: <iso>   Cross-model: ran (codex) | off
 Implementation readiness: <READY | NEEDS CLARIFICATION | NEEDS REPLAN>
+(`NEEDS REPLAN` is the eng-review field value; `NEEDS_REPLAN` with an underscore
+is the autocomplete/reply-contract routing token — do not conflate.)
 Readiness inputs SHA-256: <64 lowercase hex>
 
 ## 1. Depth
@@ -83,6 +86,7 @@ Suppressed (confidence ≤4, unverified): <count — one line each>
 ```
 
 ## `test-plan.md`: the build-readable coverage target
+
 Write/update `.devrites/work/<slug>/test-plan.md`. `$rite-build`'s slice-wright
 writes tests from it; `$rite-prove` walks it against `evidence.md`. Specify
 *what/where* to test, not implementation.
@@ -131,17 +135,19 @@ fails any row with no passing result — an unverified element is a NO-GO, like
 | email field | field | unit | <form.test> | invalid → error; valid → accepted |
 | login | flow ★★★ | E2E | <login.e2e> | submit → dashboard |
 ```
+
 Omit the whole section only for a slice with **no** interactive surface (pure backend/logic).
 
 ## Acceptance → test map
+
 - <spec acceptance criterion> → <T1, T3>   # every criterion maps to ≥1 test
-```
 
 For every row in `plan.md`'s `Shared contract proof`, the per-gap requirements name both
 provider- and consumer-side asserting tests and state how each consumes the same canonical
 artifact. Reference the plan row; do not reproduce its contract table here.
 
 ## Parallelization table (in `eng-review.md` §5 when there's an opportunity)
+
 ```markdown
 | Step / workstream | Modules touched (dir-level) | Depends on |
 |---|---|---|
@@ -154,6 +160,7 @@ Conflicts: Lanes A and B both touch models/ — sequential or coordinate.
 ```
 
 ## Fold-back: the part the build follows
+
 Vet hardens the plan. Use the marked action before fold-back.
 
 - **Write directly into `plan.md` / `tasks.md`** (single canonical writer: you, not the reviewer):
@@ -175,6 +182,7 @@ Vet hardens the plan. Use the marked action before fold-back.
   never smuggled into scope as a fact.
 
 ## Write discipline
+
 - **Single canonical writer.** The skill writes review/test-plan and edits plan/tasks;
   `devrites-plan-reviewer` is **read-only** and returns findings + bands.
 - **No silent scope.** Growing acceptance without a decision, or dropping a criterion

package/pack/generated/codex/skills/rite-vet/reference/review-axes.md

@@ -60,6 +60,7 @@ Fold verified technical findings into the plan; present each human-owned decisio
 uses `depth.md`. Never invent findings.
 
 ### 1. Architecture
+
 - Boundaries, coupling, data flow, single points of failure. Record invariants, not
   scaffolding; medium+ decisions state `Binds:`/`Prevents:`.
 - Repository/deployable roots, canonical contract and mutable-state ownership, shared
@@ -72,6 +73,7 @@ uses `depth.md`. Never invent findings.
   build will write? Name the files that should carry one.
 
 ### 2. Plan code-quality
+
 - Implied modules and planned cross-slice repetition.
 - Error-handling + edge cases the plan names, and the ones it doesn't (call those out explicitly).
 - Over-engineering (premature abstraction, an extension point with no second caller) vs
@@ -79,7 +81,9 @@ uses `depth.md`. Never invent findings.
 - Tech-debt hotspots and diagrams the change makes stale.
 
 ### 3. Test-coverage design
+
 Design tests before code.
+
 - **Framework:** match the existing runner/conventions; never add a runner for one change.
 - **Acceptance → tests.** Every AC maps to a planned surface assertion, not an internal proxy.
 - **Map applicable risk → tests.** Data and integration rows cover their relevant
@@ -95,9 +99,10 @@ Design tests before code.
 - Produce the **coverage diagram** (shape below) and add a specific test requirement per GAP.
 
 #### Coverage diagram (write to `test-plan.md`)
+
 Both code paths and user flows in one view; mark E2E-worthy `[→E2E]` and eval-worthy `[→EVAL]`:
 
-```
+```text
 CODE PATHS                                          USER FLOWS
 [+] services/billing                                [+] Checkout
   ├── processPayment()                                ├── [★★★ planned] complete purchase — checkout.e2e
@@ -106,13 +111,15 @@ CODE PATHS                                          USER FLOWS
   └── refundPayment()                               [+] Error states
       └── [★  planned] full refund only               └── [GAP]        network-timeout UX
 
-COVERAGE: 4/9 planned (44%)  |  GAPS: 5 (2 E2E)  |  REGRESSIONS: 1 (Critical)
+COVERAGE: 3/7 planned (43%)  |  GAPS: 4 (1 E2E)  |  REGRESSIONS: 0
 ```
+
 Legend: ★★★ behavior+edge+error · ★★ happy path · ★ smoke · [→E2E] integration · [→EVAL] LLM eval.
 **Fast path:** every acceptance criterion already maps to a planned test → "Test coverage: all
 acceptance criteria covered ✓" and continue.
 
 ### 4. Performance
+
 - N+1 / unbounded queries and DB access patterns; memory concerns; caching opportunities;
   high-complexity hot paths.
 - Per `performance.md`: name a number to measure or a budget: don't recommend speculative
@@ -121,7 +128,9 @@ acceptance criteria covered ✓" and continue.
 ---
 
 ## Confidence and verification gate
+
 Tag each finding `[severity] (confidence: N/10) <plan/task/spec ref> — finding`:
+
 - **9-10** verified against a quoted line · **7-8** strong pattern match → report normally.
 - **5-6** moderate → report with "verify this is real".
 - **≤4** speculative → **suppress from the walk-through**, appendix only.
@@ -132,7 +141,9 @@ Quote supporting lines. Without them confidence is ≤4 and suppressed; never in
 ---
 
 ## Present human-owned decisions
+
 Use standard `AskUserQuestion`:
+
 - **One decision = one call.** Never ask about agent work or batch unrelated choices.
 - Concrete: name the plan/task section + the quoted line.
 - 2-3 options, including "do nothing / proceed as-is" where reasonable.
@@ -148,6 +159,7 @@ Use standard `AskUserQuestion`:
 ---
 
 ## Required outputs (after the axes)
+
 1. **"NOT in scope":** considered/deferred work + rationale, folded into plan boundaries and
    spec Non-goals via the Guard.
 2. **"What exists":** solving code/flows and reuse/rebuild disposition; missed reuse is §0.

package/pack/generated/codex/skills/rite-watch-pr/SKILL.md

@@ -1,13 +1,13 @@
 ---
 name: rite-watch-pr
-description: Observe one GitHub PR and CI state without mutating code, Git, threads, checks, or shared state. Safe for admitted native host schedules and events.
+description: Observe one GitHub PR and CI state without mutating code, Git, threads, checks, or shared state. Safe for native host schedules and events.
 argument-hint: "[PR number|PR URL|blank for current branch]"
 user-invocable: true
 ---
 
 # $rite-watch-pr: read-only PR and CI observation
 
-Observe one PR once. Return a bounded snapshot and stop. A proven native host
+Observe one PR once. Return a bounded snapshot and stop. A native host
 schedule, routine, channel, automation, or GitHub event may invoke another fresh
 observation. When that activation is absent or uncertain, use an explicit turn or
 user-owned automation; never create a polling daemon, detached process, shell loop, or

package/pack/generated/codex/skills/rite-zoom-out/SKILL.md

@@ -12,8 +12,8 @@ When the agent (or the user) is staring at unfamiliar code without a working men
 model of how it fits the larger system. Stops the "open more files" reflex by returning
 a single, structured map instead.
 
-Read `.agents/skills/devrites-lib/reference/standards/core.md` first: chiefly its vocabulary / existing-conventions
-disciplines, which keep the map in the project's own language. The other rule files load
+Read `.agents/skills/devrites-lib/reference/standards/core.md` first: chiefly its existing-conventions
+discipline, which keeps the map in the project's own language. The other rule files load
 on demand.
 
 ## What this skill returns
@@ -58,11 +58,11 @@ end; don't try to fix it here.
 
 ```
 Done: mapped <area> in the project's vocabulary.
-Changed: workspace only
+Changed: none (read-only)
 Evidence: modules <n>; callers <n>; callees <n>; decisions <n>
 Open: <none | fuzzy term | suspected drift | open question>
 Next: <single recommended command>
-Record: <decision/ADR path | not applicable>
+Record: not applicable (paths printed only)
 ↻ Hygiene: /clear if this was only orientation; $rite-handoff if it informs active work
 ```
 

package/pack/generated/omp/.omp-plugin/plugin.json

@@ -0,0 +1,6 @@
+{
+  "name": "devrites",
+  "description": "DevRites: a disciplined senior-engineer workflow pack for omp",
+  "skills": "./skills",
+  "agents": "./agents"
+}