devrites 5.13.1 → 5.14.1

package/pack/generated/omp/skills/rite-polish/reference/anti-ai-slop.md

@@ -10,50 +10,72 @@ Tells of generic LLM-generated work — in **UI** and in **code**. Applies at tw
 Avoid these unless the project's existing system explicitly uses them. When in doubt,
 match the neighbors.
 
+## UI admission rule (tells are leads, not findings)
+Build avoids every UI entry; review and polish admit a hit only by this rule. Classes:
+**ENG** (engineering/realism), **A11Y** (WCAG 2.2 SC or platform requirement), **DS**
+(binding only where project tokens, components or `design-brief.md` declare it), **AES**
+(optional aesthetic). A hit is a finding only with cited mismatch evidence: it contradicts
+a declared token, brief line or nearest neighbour (cite both locations), or it has a
+measured user consequence (contrast ratio, overflow, CLS, lost or hidden focus). Otherwise
+it is an AES proposal under "Proposals (non-blocking)": never a Visual Verdict FAIL, never
+scored, capped at Suggestion/Minor, never blocking unless `design-brief.md` adopts it (then
+DS). Act on clusters per [`prose-style.md`](../../devrites-lib/reference/standards/prose-style.md).
+A broken task, data loss or misleading state is reported and fixed before any AES item.
+**Failing case:** `tokens.css` declares Inter and a 20 px card radius, yet review files
+"reflex font" and "balloon radius" as Important; or three eyebrows on six sections become
+a FAIL row with no brief conflict.
+
+**UI-copy boundary.** Craft owns functional copy: action-named controls, errors with cause
+and recovery, next-step empty states, product vocabulary, fit under expansion, and an
+accessible name containing the visible label (SC 2.5.3, A11Y). Long-form product text
+routes to `devrites-prose-craft`; translation catalogs stay untouched without their owner.
+
 ## UI anti-slop (banned defaults)
-- Default **purple/blue gradients** as the brand look. Special case: any
+- [AES] Default **purple/blue gradients** as the brand look. Special case: any
   hero hex in the `#6366f1 → #a855f7 → #ec4899` family applied as a default.
-- **Gradient text** (`background-clip: text`) used decoratively on headings.
-- **Glassmorphism** (`backdrop-filter: blur(...)` on translucent panels) as
+- [AES] **Gradient text** (`background-clip: text`) used decoratively on headings.
+- [AES] **Glassmorphism** (`backdrop-filter: blur(...)` on translucent panels) as
   a default surface.
-- **Side-stripe colored borders** on cards/sections — the
+- [AES] **Side-stripe colored borders** on cards/sections — the
   "tiny-bar-of-meaningful-accent-color-on-the-left" pattern. Distinctive
   templating tell.
-- **Pure `#000` / `#fff`** as raw text or background — too clinical; use
+- [AES] **Pure `#000` / `#fff`** as raw text or background — too clinical; use
   near-black/near-white tokens (`oklch(0.18 0 0)` / `oklch(0.98 0 0)` or
   the project's surface tokens).
-- **All-CAPS body text** for paragraphs/labels. Reserve uppercase for short
+- [AES] **All-CAPS body text** for paragraphs/labels. Reserve uppercase for short
   micro-copy (badges, eyebrows); never for sentences.
-- **Em-dashes in UI copy — zero.** Soft limits ("use sparingly") get ignored; the rule
-  is binary and greppable: no `—` in any visible string. Use a comma, period, or colon.
-- **Cards inside cards** — nested bordered/elevated containers.
-- **Identical card grids** for everything, regardless of content.
-- A **generic rounded-square icon tile** above every heading/section.
-- **Gray text on colored backgrounds** (fails contrast, looks templated).
-- The **hero-metric cliché** — three big numbers in a row with no real meaning.
-- **Decorative bounce / elastic easing** on everything; motion without purpose.
-- **Reflex fonts** picked because they're the default in a tutorial:
+- [AES] **Em dashes in UI copy** are a lead, not a ban: DS only where the project's
+  style guide bans them; otherwise no rule applies to product copy, because
+  [`prose-style.md`](../../devrites-lib/reference/standards/prose-style.md) governs DevRites
+  prose, not product UI.
+- [AES] **Cards inside cards** — nested bordered/elevated containers.
+- [AES] **Identical card grids** for everything, regardless of content.
+- [AES] A **generic rounded-square icon tile** above every heading/section.
+- [A11Y] **Gray text on colored backgrounds** — a finding with the measured ratio (SC 1.4.3).
+- [AES] The **hero-metric cliché** — three big numbers in a row with no real meaning.
+- [AES] **Decorative bounce / elastic easing** on everything; motion without purpose.
+- [AES] **Reflex fonts** picked because they're the default in a tutorial:
   - Inter for every product when the project has its own choice.
   - **DM Sans**, **Plus Jakarta Sans**, **Fraunces**, **Newsreader** when
     they're not the project's actual type system.
   Match the project; don't reach for the "tasteful default" of 2024.
-- **Modal-first thinking** — reaching for a modal as the answer to every interaction.
-- **Ghost-card** — a `1px` border *and* a soft (blur ≥16px) shadow on the same element.
+- [AES] **Modal-first thinking** — reaching for a modal as the answer to every interaction.
+- [AES] **Ghost-card** — a `1px` border *and* a soft (blur ≥16px) shadow on the same element.
   Borders separate; shadows lift — pick one
   ([`quality-standards.md`](../../devrites-frontend-craft/reference/quality-standards.md) — Materiality).
-- **Balloon radius** — `border-radius` above ~16px on cards, inputs, panels. Cards top
+- [AES] **Balloon radius** — `border-radius` above ~16px on cards, inputs, panels. Cards top
   out around 12–16px; full pills are for tags and buttons only.
-- **Uppercase tracked eyebrow above every section** — one named kicker is voice; one per
+- [AES] **Uppercase tracked eyebrow above every section** — one named kicker is voice; one per
   section is template grammar (countable cap below).
-- **Numbered-section scaffolding** (`01 / 02 / 03`) when the sections aren't a sequence.
-- **Fake UI-in-a-div** — a "product screenshot" assembled from nested divs, or hand-drawn/
+- [AES] **Numbered-section scaffolding** (`01 / 02 / 03`) when the sections aren't a sequence.
+- [ENG] **Fake UI-in-a-div** — a "product screenshot" assembled from nested divs, or hand-drawn/
   sketchy SVG scenery. Ship a real capture/asset or nothing (quality-standards — Materiality).
-- **Hero prop badges** — version labels (`V0.6`, `BETA`) and decorative pulsing status
+- [AES] **Hero prop badges** — version labels (`V0.6`, `BETA`) and decorative pulsing status
   dots as set dressing.
-- **Unstyled UA chrome** — shipping the browser's default blue selection, caret,
-  scrollbar, or focus outline as the look (quality-standards § Browser chrome).
-- **Invented metrics** — precise-looking numbers with no source (`+247% faster`,
-  `99.99% uptime`). **Off-token color** — a hex used inline that the token set doesn't
+- [AES] **UA chrome** — default selection, caret or scrollbar; the UA focus ring is a
+  finding only when SC 2.4.7/1.4.11 fails (quality-standards § Browser chrome).
+- [ENG] **Invented metrics** — precise-looking numbers with no source (`+247% faster`,
+  `99.99% uptime`). [DS] **Off-token color** — a hex used inline that the token set doesn't
   define. Both are realism failures, not styling choices.
 
 ### Required remediations (fix the hit, don't just flag it)
@@ -65,11 +87,11 @@ an incomplete finding.
 | Default purple/blue gradient; gradient text; glassmorphism default | Re-derive from the scene sentence and colour commitment, then re-run both category-reflex tests |
 | Invented metric | Replace with the state lattice's missing-data placeholder (quality-standards § Focus & states) plus a "metric to confirm" question, or delete the proof slot |
 | Off-token color | Lift into the token set as a named color; replace every inline use |
-| Wrapping CTA / nav overflow | Shorter label, `white-space: nowrap`, or a collapsed nav — never a shrunken tap target |
-| Horizontal scroll in 320–1920 | `minmax(0, 1fr)` on content tracks + `overflow-x: clip` on `html`/`body` (quality-standards § Responsive) |
+| Wrapping CTA / nav overflow | Shorter label or a collapsed nav, verified at 200% text zoom — never `white-space: nowrap` or a shrunken tap target |
+| Horizontal scroll in 320–1920 | Fix intrinsic sizing (`minmax(0, 1fr)`, wrapping, media constraints) with the bounded exception of quality-standards § Responsive — never `overflow-x: clip` on `html`/`body` |
 | Fake UI-in-a-div screenshot | Ship a real capture/asset or remove the block |
 | Sticky sub-nav hidden by a banner | Offset by the banner's token height; split z-index roles instead of one raised value |
-| Unstyled UA selection / focus ring / caret / scrollbar | Theme from tokens per quality-standards § Browser chrome, or decline in `design-brief.md` |
+| UA focus ring failing SC 2.4.7/1.4.11 | Apply the DS focus-ring token per quality-standards § Browser chrome |
 
 ### Copy & data realism
 Placeholder content is a tell even when the layout is clean: fake-perfect numbers
@@ -101,14 +123,15 @@ second one catches what the first one misses.
 Both pass = the surface looks like *this product*, not "an app in this category".
 
 ### Mechanical pre-flight (countable — run, don't vibe)
-Each is pass/fail by counting or grep, not judgment:
-- **Em-dashes** in visible UI strings: exactly **0**.
-- **Eyebrows** (uppercase-tracked kickers): ≤ `ceil(sections / 3)`.
-- **Layout families** (hero, image+text split, card grid, bento, table…): no family more
+Each is counted or grepped, not judged. ENG and DS rows pass or fail; an AES count past
+its cap is a lead for the admission rule, not a failure:
+- [AES] **Em dashes** in visible UI strings: counted; DS only where the style guide bans them.
+- [AES] **Eyebrows** (uppercase-tracked kickers): ≤ `ceil(sections / 3)`.
+- [AES] **Layout families** (hero, image+text split, card grid, bento, table…): no family more
   than twice per page; never 3 consecutive image+text zigzags.
-- **Icons**: 0 emoji-as-icon; exactly one icon set imported.
-- **State presence in code**: default+hover+focus-visible+active+disabled for every interactive element; loading/error/empty where data delays or fails — an unreachable state's screenshot proves nothing.
-- **Form inputs**: no border-width shifts between states; focus ring from outline/ring (not border swap); consistent control height; reserved helper slot; disabled beyond opacity alone.
+- [DS] **Icons**: 0 emoji-as-icon; exactly one icon set imported.
+- [ENG] **State presence in code**: every state the role lattice requires (quality-standards § Focus & states) — an unreachable state's screenshot proves nothing.
+- [ENG] **Form inputs**: no border-width shifts between states; focus ring from outline/ring (not border swap); consistent control height; reserved helper slot; disabled beyond opacity alone.
 
 ## Code anti-slop (UI **and** backend)
 - **Over-defensive checks** — `if (x && x.length > 0)` repeated, layered null guards,

package/pack/generated/omp/skills/rite-polish/reference/harden-checklist.md

@@ -55,7 +55,8 @@ might miss.
 - **Screen reader:** every control has a name; live regions for async
   feedback; reading order matches visual order.
 - **Reduced motion:** `prefers-reduced-motion` removes non-essential motion
-  (no auto-carousel, no spring entrances).
+  (no auto-carousel, no spring entrances); auto-moving content over 5 s pauses under any
+  preference ([`quality-standards.md`](../../devrites-frontend-craft/reference/quality-standards.md) § Motion).
 - **High zoom:** 200% text zoom; layout doesn't break, nothing overflows.
 
 ## 4. Localisation / i18n

package/pack/generated/omp/skills/rite-polish/reference/ui.md

@@ -50,6 +50,13 @@ Goal: this feature looks and behaves like it belongs in *this* product.
    components, align flow shape with neighbors, consolidate duplication, remove
    obsolete styles created by this feature. Ask the user when a design-system
    principle is genuinely ambiguous: don't guess the system's intent.
+4. **Protected contracts:** normalization never changes URL slugs or anchors, form
+   field `name`/`autocomplete`/order, analytics or test IDs, or legal/consent copy
+   unless `spec.md` or `decisions.md` names the change. The wright contract lists
+   these identifiers, the returned diff is checked against them, and visible-label
+   renames are recorded in `polish-report.md`. **Failing case:** polish renames
+   `name="email"` to `emailAddress`; autofill and analytics break while role-query
+   tests pass.
 
 ## Phase 4: UI polish
 
@@ -74,7 +81,7 @@ path isn't polish.
 - [ ] **Alignment** perfect at every breakpoint
 - [ ] **Typography** hierarchy consistent; line length sane (65-75ch prose)
 - [ ] **Color & contrast**: semantic roles; text ≥ 4.5:1 (≥ 3:1 large/UI), WCAG 2.2 AA
-- [ ] **All interaction states**: hover, active, focus, disabled, selected
+- [ ] **All states** the role lattice requires (quality-standards § Focus & states)
 - [ ] **Focus states visible** and **keyboard navigation** works
 - [ ] **Loading** states clear; **empty** states welcoming with a next action
 - [ ] **Error** states helpful (what happened + how to recover); **success** states clear
@@ -82,7 +89,7 @@ path isn't polish.
 - [ ] **Copy & terminology** consistent with the product's vocabulary
 - [ ] **Icons** consistent set, properly sized and aligned
 - [ ] **Responsive** behavior correct across target viewports
-- [ ] **Target size** ≥ 24×24px (WCAG 2.2 AA); ≥ 44×44px for primary touch; **no drag-only** actions
+- [ ] **Target size** ≥ 24×24px (WCAG 2.2 AA); 44×44px for primary touch is a DS preference; **no drag-only** actions
 - [ ] **Motion** smooth (~60fps), purposeful; **reduced-motion** respected
 - [ ] **Performance basics:** no obvious jank, oversized images, or blocking work
 - [ ] **No console errors or warnings**
@@ -105,7 +112,9 @@ Cite the evidence in `browser-evidence.md`.
 ### Triage
 
 If it ships in 30 minutes, fix what users will notice (hierarchy,
-states, broken layout, bad copy) before micro-spacing. Don't introduce bugs
+states, broken layout, bad copy) before micro-spacing. A broken task, data loss or
+misleading state outranks every AES proposal
+([anti-ai-slop.md](anti-ai-slop.md) admission rule); AES work waits while one is open. Don't introduce bugs
 while polishing: re-prove after changes.
 
 ### Refinement modes (optional dials within Phase 4)

package/pack/generated/omp/skills/rite-pov/SKILL.md

@@ -4,7 +4,7 @@ description: Project-grounded verdict for adopting, switching, rejecting, or rev
 argument-hint: "[candidate/link/question]"
 user-invocable: true
 ---
-<!-- loads: {"always":["devrites-lib/reference/standards/core.md"],"workspace":["decisions.md"]} -->
+<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/standards/tooling.md"],"workspace":["decisions.md"]} -->
 > Read-set manifest: `devrites-engine context [slug] --skill rite-pov` bundles every file named below into one deduplicated read.
 
 # /rite-pov: project-grounded external verdict
@@ -22,7 +22,12 @@ named outside candidate.
    incumbent dependency/call site, integration seam, relevant ADR/decision, or
    confirmed absence. No profile cache.
 3. Verify the external claim from current primary documentation, source,
-   advisory, or release notes.
+   advisory, or release notes, cited with URL, version or advisory id, and
+   retrieval date ([`tooling.md`](../devrites-lib/reference/standards/tooling.md) citation contract). A CVE verdict also needs the
+   installed version from the lockfile (`path:line`) checked against the
+   advisory's affected range, and a call-site search for the affected API
+   (`reachable` / `unreachable` / `unknown`). Known exploitation (a KEV listing)
+   raises priority, never severity.
 4. Compare fit, existing alternatives, migration cost, reversibility, project
    principles, security, licensing, and deprecation risk.
 5. Return exactly one verdict: `Adopt`, `Trial`, `Hold`, `Reject`, or
@@ -37,7 +42,8 @@ named outside candidate.
 Verdict: <grade> — <one sentence>
 Tier: <two-way | bounded-one-way | high-stakes>
 Project evidence: <file:line or local doc>
-External evidence: <primary source>
+External evidence: <URL — version/advisory id — retrieved YYYY-MM-DD>
+CVE applicability: <n/a | range in/out (lockfile path:line); reachability reachable/unreachable/unknown>
 Why: <three bullets max>
 Next: <one action | done>
 Record: <path | not written>
@@ -57,4 +63,9 @@ before any `Adopt`/`Trial`.
 - Missing project evidence or missing live primary source → `Hold`. **Failing
   case:** "Adopt Redis" with no incumbent call site and no opened docs URL.
 - High-stakes / irreversible without a named rollback → cannot be `Adopt`.
+- A citation missing version or retrieval date, or a CVE verdict missing range
+  evidence → `Hold`. `Not-our-problem` for a CVE needs out-of-range or
+  `unreachable` evidence. **Failing case:** "Not-our-problem: we don't call that
+  function" with no call-site search, or "Adopt upgrade" while the lockfile already
+  pins a fixed version.
 - Persist only when asked; a chat-only verdict is not a decision record.

package/pack/generated/omp/skills/rite-pr-feedback/SKILL.md

@@ -22,7 +22,11 @@ Step 0: Read [`core.md`](../devrites-lib/reference/standards/core.md), plus [`gi
 - Never resolve a thread without a reply that names what happened.
 
 ## Workflow
-1. **Locate PR/thread.** Argument URL = targeted thread; PR number = all unresolved threads on that PR; blank = current branch PR via `gh pr view`. Stop if `gh` is unavailable.
+1. **Locate PR/thread.** Argument URL = targeted thread; PR number = all unresolved threads on that PR; blank = current branch PR via `gh pr view`. Stop if `gh` is unavailable. Each stop below quotes the failing command's output and names one recovery command; never substitute another branch's or the most recent PR:
+   - `gh auth status` fails → unauthenticated; recovery `gh auth login`.
+   - `git symbolic-ref -q HEAD` fails → detached HEAD (step 5 cannot push); recovery `gh pr checkout <number>` or `git switch <branch>`.
+   - blank argument and `gh pr view` finds no PR for the branch → recovery `/rite-pr-feedback <number>` for an existing PR, or `gh pr create`.
+   **Completion:** exactly one PR (and thread, for a URL) located, or one named stop reported.
 2. **Fetch.** Use GitHub GraphQL/CLI to collect unresolved review threads with file, line, author, body, and thread id. Completion: every unresolved thread is represented once, or the fetch error is reported.
 3. **Legitimacy gate.** For each item, read the surrounding code and classify: `fix`, `not-addressing`, `declined`, `reply-only`, or `needs-human`. Deduplicate overlapping items.
 4. **Fix approved items.** Apply contained fixes, add/update tests when behavior changes, and run targeted checks. Larger product/API/security calls become `needs-human`.

package/pack/generated/omp/skills/rite-pressure-test/SKILL.md

@@ -4,7 +4,7 @@ description: "Pressure-test a rough/vague idea: ideate, explore 3-5 approaches,
 argument-hint: "[rough idea or plan to stress-test]"
 user-invocable: true
 ---
-<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/intent-map.md"],"workspace":["decisions.md"]} -->
+<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/intent-map.md"],"workspace":["decisions.md","questions.md"]} -->
 > Read-set manifest: `devrites-engine context [slug] --skill rite-pressure-test` bundles every file named below into one deduplicated read.
 
 # /rite-pressure-test: diverge then converge
@@ -37,7 +37,9 @@ If the hinge is an uncertain material fact, freeze at most three questions for
 `unavailable`, never support. Record claim, support, strongest contrary evidence, and
 `supported | assumption | refuted`. Refutation changes the recommendation; an assumption
 advances only when non-decisive with bounded downside. Weak/conflicting decisive evidence
-returns **Hold** with the resolving evidence, never `/rite-spec`. Tie-breaker:
+returns **Hold** with the resolving evidence, never `/rite-spec`. Persist a Hold as an open
+`questions.md` entry (`gate: blocking`, the premise and resolving evidence) in the active or
+provisional-slug workspace before stopping; a reply-only Hold is lost. Tie-breaker:
 see [`intent-map.md`](../devrites-lib/reference/intent-map.md) (`rite-pressure-test`
 vs `rite-spec`). Do not research preferences
 or reversible implementation choices.
@@ -64,6 +66,6 @@ Changed: workspace only
 Evidence: options compared <n>; recommendation <option>; hinge <condition>; premises <supported/assumption/refuted + sources>
 Open: <none | unresolved premise and resolving evidence>
 Next: <when supported: /rite-spec <feature>; when Hold: none — requires <resolving evidence>>
-Record: <investigation-map.md path | not applicable>
+Record: <investigation-map.md path | questions.md id (Hold) | not applicable>
 ↻ Hygiene: /clear before starting the lifecycle
 ```

package/pack/generated/omp/skills/rite-prove/SKILL.md

@@ -5,7 +5,7 @@ argument-hint: "[feature-slug]"
 user-invocable: true
 ---
 
-<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/candidate-integrity.md","devrites-lib/reference/standards/testing.md","devrites-lib/reference/standards/spec-grammar.md","devrites-lib/reference/standards/test-proof-checklist.md","devrites-lib/reference/standards/agents.md","devrites-lib/reference/standards/gates.md","devrites-lib/reference/standards/verification-methods.md","rite-prove/reference/acceptance-proof.md","rite-prove/reference/proof-ladder.md","rite-prove/reference/failure-triage.md","rite-prove/reference/test-command-discovery.md","rite-prove/reference/anti-patterns.md","rite-build/reference/checkpoint.md"],"triggers":{"applicability":["devrites-lib/reference/standards/repository-topology.md","devrites-lib/reference/standards/data-integrity.md","devrites-lib/reference/standards/integration-reliability.md"],"browser":["devrites-lib/reference/standards/browser-proof-checklist.md","rite-prove/reference/browser-proof.md"],"devex":["devrites-lib/reference/standards/developer-experience.md"],"dod":["devrites-lib/reference/standards/definition-of-done.md"],"observability":["devrites-lib/reference/standards/observability.md"],"one-shot":["devrites-lib/reference/standards/one-shot-actions.md"],"performance":["devrites-lib/reference/standards/performance.md"],"workflow-artifacts":["devrites-lib/reference/standards/workflow-artifacts.md"]},"workspace":["brief.md","spec.md","state.md","decisions.md","assumptions.md","questions.md","decision-coverage.md","architecture.md","plan.md","tasks.md","traceability.md","eng-review.md","test-plan.md","gates.md","evidence.md","touched-files.md","browser-evidence.md"],"workspaceByRole":{"proof-runner":["spec.md","test-plan.md","tasks.md","evidence.md","touched-files.md","browser-evidence.md","state.md"],"spec-reviewer":["brief.md","spec.md","decision-coverage.md","questions.md","decisions.md","assumptions.md","state.md"]}} -->
+<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/candidate-integrity.md","devrites-lib/reference/standards/testing.md","devrites-lib/reference/standards/spec-grammar.md","devrites-lib/reference/standards/test-proof-checklist.md","devrites-lib/reference/standards/agents.md","devrites-lib/reference/standards/gates.md","devrites-lib/reference/standards/verification-methods.md","rite-prove/reference/acceptance-proof.md","rite-prove/reference/proof-ladder.md","rite-prove/reference/failure-triage.md","rite-prove/reference/test-command-discovery.md","rite-prove/reference/anti-patterns.md","rite-build/reference/checkpoint.md"],"triggers":{"applicability":["devrites-lib/reference/standards/repository-topology.md","devrites-lib/reference/standards/data-integrity.md","devrites-lib/reference/standards/integration-reliability.md"],"browser":["devrites-lib/reference/standards/browser-proof-checklist.md","rite-prove/reference/browser-proof.md"],"devex":["devrites-lib/reference/standards/developer-experience.md"],"dod":["devrites-lib/reference/standards/definition-of-done.md"],"observability":["devrites-lib/reference/standards/observability.md"],"one-shot":["devrites-lib/reference/standards/one-shot-actions.md"],"performance":["devrites-lib/reference/standards/performance.md"],"workflow-artifacts":["devrites-lib/reference/standards/workflow-artifacts.md"]},"workspace":["brief.md","spec.md","state.md","decisions.md","assumptions.md","questions.md","decision-coverage.md","architecture.md","plan.md","tasks.md","traceability.md","eng-review.md","test-plan.md","gates.md","evidence.md","touched-files.md","browser-evidence.md"],"workspaceByRole":{"proof-runner":["spec.md","test-plan.md","tasks.md","gates.md","evidence.md","touched-files.md","browser-evidence.md","state.md"],"spec-reviewer":["brief.md","spec.md","decision-coverage.md","questions.md","decisions.md","assumptions.md","state.md"]}} -->
 > Read-set manifest: `devrites-engine context <slug> --phase prove` bundles every file named below into one deduplicated read. Trigger names map to the conditional rules in the sections that follow.
 
 
@@ -72,16 +72,24 @@ test changes.
 4. **Execute a frozen candidate.** Run `devrites-engine check candidate <slug>` and retain its digest.
    The root runs only commands declared by `test-plan.md`, with exact approved
    command, cwd, prerequisites, exit, and sanitized
-   decisive output. Run relevant suite plus build/typecheck/lint. Recheck the
+   decisive output, plus the test selector, runner counts
+   (collected/ran/passed/failed/skipped), and attempt number with first-attempt
+   outcome; output and exit come from one execution. Run relevant suite plus build/typecheck/lint. Recheck the
    candidate and require identical digest/no source mutation. Reject substituted
    commands, malformed manifests, zero-test/skipped/filtered behavioral claims,
-   exit-status-only claims, and source drift. Static gates prove only their named
+   exit-status-only claims, status-masking forms, and source drift. A retry-only
+   pass is `flaky` per [`testing.md`](../devrites-lib/reference/standards/testing.md#determinism-no-flaky-tests),
+   never green. Static gates prove only their named
    static criterion.
    Run the acceptance ledger: `devrites-engine gates run <slug>` executes every
    unmet runnable gate against the same approved preflight rows and records
    definition-bound evidence; `gates reverify <slug>` re-executes all of them
    when any prior pass is in doubt. Attest manual gates with `gates attest
-   <slug> <id> <note>` only against observed judgment; an impossible outcome
+   <slug> <id> <note>` only against observed judgment whose note cites the
+   human's own words or source; placeholder notes are unmet and an AFK run never
+   self-attests. Only gates manual at Vet are attested; runnable-to-manual
+   conversion, EXPECT loosening, or CHECK/CWD/row change here is Plan repair plus
+   re-Vet ([`gates.md`](../devrites-lib/reference/standards/gates.md)). An impossible outcome
    is `gates abandon` plus a handoff, never a deleted row or a false pass.
    `gates status <slug>` must reduce to `all-met` before Prove records green.
 5. **Gate consumptive actions.** Immediately before execution, apply

package/pack/generated/omp/skills/rite-prove/reference/anti-patterns.md

@@ -13,7 +13,7 @@ see [standards/anti-patterns.md](../../devrites-lib/reference/standards/anti-pat
 |---|---|
 | "Tests passed during build; that's enough." | Build = per-slice sanity check. Prove = whole-feature suite + build/typecheck/lint + browser. |
 | "Browser proof is overkill for this UI change." | If it's UI and a browser can run, the ladder applies. If you can't, document which rung and why. |
-| "Failure is in pre-existing code, not in this feature." | Still surfaces as a blocker. Feature scope = fix in scope or record + halt. |
+| "Failure is in pre-existing code, not in this feature." | Still surfaces as a blocker, never green. `pre-existing` needs recorded same-command baseline evidence ([failure triage](failure-triage.md)); without it the result is unresolved. Feature scope = fix in scope or record + halt. |
 | "Some slices are still pending but the rest can be proved." | No: `/rite-prove` runs once, when every slice is built. Otherwise you're proving a partial system. |
 
 ## Red Flags

package/pack/generated/omp/skills/rite-prove/reference/failure-triage.md

@@ -14,7 +14,14 @@ loop to `devrites-debug-recovery`.
    - environment/setup (missing dep, wrong node/ruby) → fix it when agent-owned;
      record a blocker only for human/access ownership, scope overflow, or exhausted
      fingerprint recovery.
-   - flaky (passes on re-run, timing/order) → note it; don't paper over with retries.
+   - pre-existing (fails without this candidate) → only with baseline evidence: the
+     same command/cwd/environment run on the baseline candidate (an earlier
+     `EVID-###`, or a base-ref run in a separate worktree), recorded. It is a named
+     blocker citing that baseline, never green; without the baseline it is unresolved.
+   - flaky (at least one fail and one pass on the unchanged candidate digest;
+     timing/order) → record every attempt and apply
+     [the Determinism standard](../../devrites-lib/reference/standards/testing.md#determinism-no-flaky-tests):
+     the green attempt proves nothing; fix it or record a blocker. Don't paper over with retries.
    - external dependency down → record blocker; don't fake the result.
 3. **Isolate:** minimize to the smallest failing case; bisect the diff if needed.
    If consumptive evidence maps to multiple causal seams, this is a
@@ -23,8 +30,10 @@ loop to `devrites-debug-recovery`.
    the cleaned-away past state terminal.
 4. **Fix within scope:** the current slice/feature only. If the fix reaches outside
    scope, stop and record a blocker in `state.md` (then `/rite-plan unblock`).
-5. **Re-run** the same command when it is repeatable; confirm green and record both
-   attempts in `evidence.md`. A consumptive action needs re-vetted evidence
+5. **Re-run** the same command when it is repeatable, after an admitted
+   correction; confirm green and record both attempts in `evidence.md`. A green
+   rerun with no correction on an unchanged digest is `flaky` (step 2), never a
+   close. A consumptive action needs re-vetted evidence
    completeness and any fresh authorization before a new attempt. Its spent
    action authorization does not stop offline recovery: a retained new fingerprint
    enters caller-owned diagnosis, correction, fixtures, and narrow Vet immediately.

package/pack/generated/omp/skills/rite-quick/SKILL.md

@@ -51,6 +51,8 @@ Load this lane's conditional standards when needed:
    `state.md` directly; `/rite-quick` does **not** require
    a workspace and does **not** create the full tree.
 1. **Significance gate** (above). Fail → STOP, tell the user to run `/rite-spec <feature>`.
+   If the only failing item is size (more than one slice) and no risk item holds, name
+   `/rite-fast <task>` as the lighter option.
 2. **One-line contract.** Restate in 1-3 lines: the change, its **acceptance** (how you'll
    know it works), and the **scope boundary** (what you will NOT touch). This is the entire
    "spec + plan" for a quick change: keep it in the chat; optionally drop a `brief.md` +
@@ -66,6 +68,8 @@ Load this lane's conditional standards when needed:
 4. **Prove (scoped).** Run the **targeted** tests + typecheck / lint for what changed (not
    the whole suite) → green. Record the command + output — resolve the project's real
    typecheck/lint commands with `devrites-engine detect commands` rather than guessing.
+   Engine missing or erroring, or zero detected commands for the changed surface:
+   stop with `unproven: <what could not run>`, never green and never a guessed command.
    A tautological test that can't
    fail is not proof.
 5. **Review-lite + ship.** Self-review the diff for correctness, scope, and idiom. If

package/pack/generated/omp/skills/rite-resolve/SKILL.md

@@ -13,7 +13,7 @@ disable-model-invocation: true
 `/rite-resolve` resumes an **async** human gate: a checkpoint that already paused and
 **stopped the session** (an AFK blocking/escalating/irreversible queue, or a HITL pause
 left unanswered), plus `--batch`. When `/rite-build` asks a question **inline**
-via `AskUserQuestion` and the human is present, that pick resolves the gate **in place** through
+via `ask` and the human is present, that pick resolves the gate **in place** through
 the `devrites-engine state resolve` writer. You don't type `/rite-resolve` for it. For the async case this
 skill takes the human's answer (or `--drop` / `--batch`), writes it to `questions.md`, updates
 `state.md` (clears `Awaiting human`, sets `Status: running`), and recommends the next command.
@@ -80,7 +80,13 @@ Pull these via `Read` when shaping the resolve:
      that block and sets `Status: running`;
    - appends a `Log` line to `state.md`.
 
-   On resume, also clear or update the unblocked slice's `Slice mode` in `state.md`: if
+   **Writer failure:** command not found or nonzero exit → STOP. Do not hand-edit
+   `Slice mode`, `questions.md`, or `state.md`; report the exact command, its exit code,
+   and the decisive stderr line, and recommend `/rite-doctor`. Both files stay as the
+   writer left them.
+   **Completion:** zero exit, or the failure reported with no hand edit.
+
+   On resume, and only after a zero exit, also clear or update the unblocked slice's `Slice mode` in `state.md`: if
    the answer lets the slice proceed, drop the pause-time `Slice mode` so `/rite-build`
    re-derives it on the next selection; if the answer re-shapes how the slice should be
    built, set `Slice mode` to match.

package/pack/generated/omp/skills/rite-resolve/reference/answer-protocol.md

@@ -51,7 +51,7 @@ q-2026-05-28-002: clinician sign-off required at runtime, not at plan time
 - Order matters only for `state.md` clearance: the script processes top-to-bottom and
   flushes `Awaiting human` after each match.
 - Errors abort the batch at the first failing line so partial state is small and
-  diagnosable. (Re-run after fixing the offending line; already-applied lines refuse to
+  diagnosable. Report the failing line, exit code, and the qids already applied. (Re-run after fixing the offending line; already-applied lines refuse to
   re-apply because their `status` is no longer `open`.)
 
 ## Never overwrite

package/pack/generated/omp/skills/rite-review/SKILL.md

@@ -87,7 +87,8 @@ Pull these via `Read` when the diff demands them:
    what the spec asked but breaks project conventions (Spec pass, Code-review fail).
    Separate contexts prevent one axis from masking the other. This is the initial
    full pass; a correction uses step 7's bounded recheck:
-   - Freeze that closed digest and give the same digest to **two** read-only reviewers in
+   - Freeze that closed digest and give the same digest to the **two** axis reviewers (three when
+     [engineering lanes](../devrites-lib/reference/parallel-dispatch.md) split the code axis) in
      parallel, each with its own narrow brief and no
      cross-pollination — and evaluate the frontend (step 4), security (step 5),
      and performance (step 6) dispatch conditions from this same diff now,
@@ -104,6 +105,8 @@ Pull these via `Read` when the diff demands them:
        checks. Apply canonical anti-slop and silent-failure lenses; inspect each hunk
        for unrequested deletion. Unexcepted `.devrites/principles.md` violations are
        Critical. Distinguish binding standards from judgment-only baseline smells."
+       Route it through [parallel-dispatch.md § Engineering lanes](../devrites-lib/reference/parallel-dispatch.md#engineering-lanes):
+       a frontend+backend diff gets two lane reviewers in this cohort.
    - **Do NOT merge or re-rank** their findings. Present them under separate
      `## Spec` and `## Code review` sub-sections in `review.md`. Surface contradictions
      between the axes explicitly (e.g. "Spec axis says complete, Code-review axis says

package/pack/generated/omp/skills/rite-review/reference/performance-checklist.md

@@ -37,6 +37,11 @@ Identify the framework first; apply only its idioms.
   handlers, but preserve sensitive-data cache policy: `Cache-Control: no-store`
   eligibility varies by browser and conditions. Never remove a security control
   for a performance score; test account-switch and restored-page behavior.
+- **CLS attribution:** name the initiator (content inserted above, late
+  validation message, font swap), not only the shifted element, and exercise the
+  post-load state; a navigation-only run misses it. A CLS change is layout stability,
+  never reported as speed or latency. **Failing case:** space reserved on the shifted
+  card while the banner inserted above still causes the shift.
 
 ## Backend (every feature, UI or not)
 
@@ -62,7 +67,10 @@ Identify the framework first; apply only its idioms.
 ## AI-codegen perf smells (fold into the area above, not a separate finding category)
 
 - State duplicated instead of lifted; effects with over-broad deps that re-run needlessly.
-- Sequential `await`s where `Promise.all` / parallel fetch fits.
+- Sequential `await`s where `Promise.all` / parallel fetch fits, only after
+  authorization: a reorder never starts a private fetch before the auth check.
+  **Failing case:** after a "waterfall fix" an unauthenticated request reaches the
+  private fetch → Critical.
 - Over-fetching "just in case"; redundant calls a dedup would collapse.
 - Defensive memoization wrapping cheap components: cost with no benefit.
 

package/pack/generated/omp/skills/rite-seal/SKILL.md

@@ -5,7 +5,7 @@ argument-hint: "[feature-slug] [--full]"
 user-invocable: true
 ---
 
-<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/standards/code-navigation.md","devrites-lib/reference/candidate-integrity.md","devrites-lib/reference/orchestration-profiles.md","devrites-lib/reference/parallel-dispatch.md","devrites-lib/reference/standards/verification-methods.md","rite-seal/reference/phase-contract.md","rite-seal/reference/final-evidence.md","rite-seal/reference/go-no-go.md","rite-seal/reference/risk-and-rollback.md","rite-seal/reference/seal-template.md","rite-seal/reference/output.md","rite-seal/reference/anti-patterns.md"],"triggers":{"agents":["devrites-lib/reference/standards/agents.md"],"browser":["devrites-lib/reference/standards/browser-proof-checklist.md"],"code-review":["devrites-lib/reference/standards/code-review.md"],"deprecation":["devrites-lib/reference/standards/deprecation.md"],"documentation":["devrites-lib/reference/standards/documentation.md"],"dod":["devrites-lib/reference/standards/definition-of-done.md"],"observability":["devrites-lib/reference/standards/observability.md"],"principles":["devrites-lib/reference/standards/principles.md"],"security":["devrites-lib/reference/standards/security.md"],"testing":["devrites-lib/reference/standards/testing.md"]},"workspace":["brief.md","spec.md","state.md","decisions.md","assumptions.md","questions.md","decision-coverage.md","architecture.md","plan.md","tasks.md","traceability.md","eng-review.md","test-plan.md","gates.md","evidence.md","touched-files.md","review.md","seal.md"],"workspaceByRole":{"code-reviewer":["spec.md","plan.md","tasks.md","test-plan.md","architecture.md","decisions.md","state.md","evidence.md","touched-files.md","review.md"],"devex-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"doubt-reviewer":["spec.md","plan.md","tasks.md","state.md","decisions.md","review.md"],"frontend-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"performance-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"proof-runner":["spec.md","test-plan.md","tasks.md","evidence.md","state.md"],"security-auditor":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"simplifier-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","review.md"],"spec-reviewer":["brief.md","spec.md","decision-coverage.md","questions.md","decisions.md","assumptions.md","state.md","review.md"],"test-analyst":["spec.md","tasks.md","test-plan.md","state.md","evidence.md","review.md"]}} -->
+<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/standards/code-navigation.md","devrites-lib/reference/candidate-integrity.md","devrites-lib/reference/orchestration-profiles.md","devrites-lib/reference/parallel-dispatch.md","devrites-lib/reference/standards/verification-methods.md","rite-seal/reference/phase-contract.md","rite-seal/reference/final-evidence.md","rite-seal/reference/go-no-go.md","rite-seal/reference/risk-and-rollback.md","rite-seal/reference/seal-template.md","rite-seal/reference/output.md","rite-seal/reference/anti-patterns.md"],"triggers":{"agents":["devrites-lib/reference/standards/agents.md"],"browser":["devrites-lib/reference/standards/browser-proof-checklist.md"],"code-review":["devrites-lib/reference/standards/code-review.md"],"deprecation":["devrites-lib/reference/standards/deprecation.md"],"documentation":["devrites-lib/reference/standards/documentation.md"],"dod":["devrites-lib/reference/standards/definition-of-done.md"],"observability":["devrites-lib/reference/standards/observability.md"],"principles":["devrites-lib/reference/standards/principles.md"],"security":["devrites-lib/reference/standards/security.md"],"testing":["devrites-lib/reference/standards/testing.md"]},"workspace":["brief.md","spec.md","state.md","decisions.md","assumptions.md","questions.md","decision-coverage.md","architecture.md","plan.md","tasks.md","traceability.md","eng-review.md","test-plan.md","gates.md","evidence.md","touched-files.md","review.md","seal.md"],"workspaceByRole":{"code-reviewer":["spec.md","plan.md","tasks.md","test-plan.md","architecture.md","decisions.md","state.md","evidence.md","touched-files.md","review.md"],"devex-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"doubt-reviewer":["spec.md","plan.md","tasks.md","state.md","decisions.md","review.md"],"frontend-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"performance-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"proof-runner":["spec.md","test-plan.md","tasks.md","gates.md","evidence.md","state.md"],"security-auditor":["spec.md","plan.md","tasks.md","state.md","touched-files.md","evidence.md","review.md"],"simplifier-reviewer":["spec.md","plan.md","tasks.md","state.md","touched-files.md","review.md"],"spec-reviewer":["brief.md","spec.md","decision-coverage.md","questions.md","decisions.md","assumptions.md","state.md","review.md"],"test-analyst":["spec.md","tasks.md","test-plan.md","state.md","evidence.md","review.md"]}} -->
 > Read-set manifest: `devrites-engine context <slug> --phase seal` bundles every file named below into one deduplicated read. Trigger names map to the conditional rules in the sections that follow.
 
 
@@ -62,9 +62,11 @@ Follow [`reference/phase-contract.md`](reference/phase-contract.md)
 1. read workspace and final diff;
 2. recheck the canonical candidate/bindings and approved repository/CI proof;
 3. admit valid Spec/Code accounts; dispatch remaining required exact roles under
-   `../devrites-lib/reference/parallel-dispatch.md` on the same frozen candidate;
+   `../devrites-lib/reference/parallel-dispatch.md` on the same frozen candidate,
+   code review through its § Engineering lanes;
 4. reconcile their read-only verdicts against source and immutable evidence;
-5. draft GO/NO-GO `seal.md` with acceptance, tests, decisions, and seven accounts;
+5. draft GO/NO-GO `seal.md` with acceptance, tests, decisions, and seven accounts
+   (each engineering lane its own account);
 6. write exactly one candidate binding, then run
    `devrites-engine check seal <slug>` for structure and identity, never semantics,
    and, when that check reports a stale readiness-input binding, run

package/pack/generated/omp/skills/rite-seal/reference/anti-patterns.md

@@ -12,7 +12,7 @@ see [standards/anti-patterns.md](../../devrites-lib/reference/standards/anti-pat
 | Excuse | Rebuttal |
 |---|---|
 | "All Important findings are minor; just GO." | `Important > 0` triggers an interactive y/N: do not bypass the prompt, present it. |
-| "User clearly wants this to ship." | Ship is evidence-driven, not consent-driven. A Critical finding outranks "the user said go". |
+| "User clearly wants this to ship." | Ship is evidence-driven, not consent-driven. With a Critical open the verdict stays NO-GO and `seal.md` records the user's override request; the user may accept named Important risks at the y/N prompt, but no request turns an open Critical or failed gate into GO. |
 | "Drift is small; resolve it inline and seal." | Unresolved drift = NO-GO, no exceptions. Route through `/rite-plan` to repair, then seal. |
 | "Reviewer findings overlap; average them away." | Surface disagreements explicitly. Averaging hides where the reviewers disagree. |
 | "Carry the old reviewer verdicts though the diff changed." | If the diff changed since `/rite-review`, re-run the Spec + Code axes: don't carry stale verdicts forward. |

package/pack/generated/omp/skills/rite-seal/reference/phase-contract.md

@@ -28,7 +28,7 @@
    and the [`review roster`](../../devrites-lib/reference/parallel-dispatch.md) on
    the candidate. Admit each result through
    [`result admission`](../../devrites-lib/reference/standards/agents.md#result-admission).
-   `## Reviewer Accounts` preserves all seven; gap/missing/malformed blocks;
+   `## Reviewer Accounts` preserves all seven roles, each engineering lane as its own account; gap/missing/malformed blocks;
    conditional `Not-applicable` names why.
 7. **Route corrections.** Reconcile cited findings/disagreements, but do not
    edit the candidate or manifest in Seal. Route an accepted correction through

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

@@ -51,7 +51,12 @@ load a routed standard to discover required behavior, not to prescribe implement
    else echo "greenfield or already onboarded → continue spec"; fi
    ```
 1. **Understand the request** (`$ARGUMENTS`). State outcome and underlying
-   problem in one sentence. **Completion:** it includes both.
+   problem in one sentence. List each distinct explicit ask under `brief.md` `## Asks`,
+   quoted from the user's own words or a cited issue/PRD, never the invocation (`spec this`).
+   Readiness maps every ask to REQ/AC, a reasoned Non-goal, or a blocking question; an ask
+   with two readings that change acceptance is a question, never a silent pick.
+   **Completion:** it includes both, and every ask is listed. **Failing case:** "CSV export
+   and an email digest" yields a CSV-only spec; the digest is neither Non-goal nor question.
 1a. **Dedupe.** Search local issues/PRDs, work, and archive. On a close match ask
    extend/adopt/new, then record the choice; otherwise continue silently.
    A recorded [feature-sequence continuation](../rite-plan/reference/slicing.md#continuation-workspaces)
@@ -138,15 +143,18 @@ load a routed standard to discover required behavior, not to prescribe implement
    complete-block preservation rule.
    `tasks.md` deliberately does not exist yet; cross-artifact traceability starts in
    `/rite-define`. Any grammar or delta failure blocks. The interruption forecast must be resolved, owned, or a justified
-   action-time gate. Then write `Spec gate: passed <iso>`.
+   action-time gate. Run the [`spec-grammar.md`](../devrites-lib/reference/standards/spec-grammar.md) marker sweep; a kept hit blocks. Then write
+   `Spec gate: passed <iso>` with its `marker sweep:` line.
 6a. **Existing-spec delta.** For a pre-existing slug/workspace, compare old/new
    spec and questions, then show `Acceptance delta` (added/removed/reworded IDs)
    and `Open-question delta` (opened/resolved/reworded IDs), using `none` for
    empty categories. Show before proceeding; new workspaces skip.
 6b. **Review-before-code digest.** Before planning, render the compact human review:
    `Intent` (one sentence), `Done means` (top acceptance/scenario IDs), `Scope/risk` (what is in/out
-   plus the hard gates), and `Build exactly this?` (yes → next phase; no → revise now). The digest
-   is a view over `spec.md`, not a new artifact. **Stop** after the digest.
+   plus the hard gates), `Assumed:` (each `assumptions.md` id, one line, or `none`), and
+   `Build exactly this?` (yes → next phase; no → revise now). A "no" on one `Assumed:` line
+   reopens only that line as a human decision under step 4. The digest
+   is a view over `spec.md` and `assumptions.md`, not a new artifact. **Stop** after the digest.
 
 > **Mid-flight discipline.** Do not skip investigation, gap resolution, or placement
 > decisions. See [`anti-patterns`](reference/anti-patterns.md).

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

@@ -58,7 +58,7 @@ delta are gaps. Resolve them before `/rite-define`. Mark each gap inline with
 For each material gap or issue that changes scope, placement, data model, UX, security,
 migration risk, or acceptance, **ask the human** one gap at a time with a ranked
 option set with the recommended option **first and marked `(Recommended)`** plus an escape
-hatch (via `AskUserQuestion` in HITL):
+hatch (via `ask` in HITL):
 ```
 <gap/issue stated in one line>. Why it matters: <...>
 1. <recommended option> (Recommended) — <implication / where it places the work; cite any web/docs finding>