devrites 3.2.2 → 3.2.3

package/engine/tests/lib_parity_test.go

@@ -653,8 +653,8 @@ func TestCoverageGolden(t *testing.T) {
 // ---- doubt-coverage -------------------------------------------------------
 //
 // Golden: the decisions.md `doubt: MISSING` short-circuit (checked first), the
-// inline-build branch, the no-log pass, the wright/doubt count line, and the
-// 0/1/2/3 exit taxonomy are checked against golden snapshots.
+// no-log pass, the wright/doubt count line, and the 0/1/2/3 exit taxonomy are
+// checked against golden snapshots.
 
 func TestParityDoubtCoverage(t *testing.T) {
 	work := t.TempDir()
@@ -664,15 +664,7 @@ func TestParityDoubtCoverage(t *testing.T) {
 		"## Decisions stood\n- use flock — doubt: MISSING\n")
 	writeFeatureFile(t, work, "missing", "footprint.log", "1000 wright 01\n")
 
-	// MISSING is checked BEFORE the inline marker → still exit 3.
-	writeFeatureFile(t, work, "missing-inline", "decisions.md", "- x — doubt: MISSING\n")
-	writeFeatureFile(t, work, "missing-inline", ".reconcile-inline", "")
-
-	// inline build, no MISSING → exit 0.
-	writeFeatureFile(t, work, "inline", ".reconcile-inline", "")
-	writeFeatureFile(t, work, "inline", "decisions.md", "## Decisions stood\n- x — doubt: yes\n")
-
-	// dir exists, no log / dec / marker → "no footprint log" pass.
+	// dir exists, no log / decisions → "no footprint log" pass.
 	makeFeatureDir(t, work, "nolog")
 
 	// wright dispatch(es), zero doubt → exit 1.
@@ -687,7 +679,7 @@ func TestParityDoubtCoverage(t *testing.T) {
 		"1000 reviewer code-reviewer\n1000 doubt d1\n")
 
 	for _, arg := range []string{
-		"missing", "missing-inline", "inline", "nolog",
+		"missing", "nolog",
 		"nodoubt", "doubted", "nowright",
 		"ghost", // no workspace at all → "no footprint log" pass
 	} {
@@ -708,13 +700,12 @@ func TestParityDoubtCoverage(t *testing.T) {
 }
 
 // TestDoubtCoverageGolden asserts each verdict's exact stdout inline: the
-// SKIPPED / inline / no-log messages and the "no doubt ran" prompt.
+// SKIPPED / no-log messages and the "no doubt ran" prompt.
 func TestDoubtCoverageGolden(t *testing.T) {
 	work := t.TempDir()
 	const base = ".devrites/features/"
 
 	writeFile(t, work, base+"miss/decisions.md", "- x — doubt: MISSING\n")
-	writeFile(t, work, base+"in/.reconcile-inline", "")
 	mkdirAllT(t, work, base+"empty")
 	writeFile(t, work, base+"nd/footprint.log", "1000 wright 01\n1000 wright 02\n")
 	writeFile(t, work, base+"ok/footprint.log", "1000 wright 01\n1000 doubt d1\n")
@@ -723,9 +714,6 @@ func TestDoubtCoverageGolden(t *testing.T) {
 		{"skipped", []string{"doubt-coverage", "miss"},
 			"doubt-coverage: SKIPPED: a '## Decisions stood' entry in decisions.md has doubt: MISSING.\n" +
 				"A decision was stood and recorded but never doubted. Re-dispatch doubt or escalate.\n", 3},
-		{"inline", []string{"doubt-coverage", "in"},
-			"doubt-coverage: inline build (.reconcile-inline): footprint heuristic n/a; no MISSING\n" +
-				"verdict recorded. Verify by hand that each stood decision carries a devrites-doubt verdict.\n", 0},
 		{"no-log", []string{"doubt-coverage", "empty"},
 			"doubt-coverage: no footprint log: nothing to assess (pass)\n", 0},
 		{"no-doubt-ran", []string{"doubt-coverage", "nd"},

package/engine/tests/parity_reconcile_test.go

@@ -68,7 +68,7 @@ func TestParityReconcile(t *testing.T) {
 		assertGolden(t, out, code)
 	})
 
-	t.Run("inline-fallback", func(t *testing.T) {
+	t.Run("close-window", func(t *testing.T) {
 		work := newRepo(t)
 		snapshot(t, work)
 		out, code := runArgv(t, work, libRootEnv(work), "", binPath, "reconcile", "close", "feat")

package/engine/tests/parity_test.go

@@ -269,15 +269,12 @@ func TestParityRedwatch(t *testing.T) {
 
 // TestParityA1Guard checks `hook a1-guard` against golden snapshots for each
 // payload. It blocks a main-thread source edit only when the build window is open
-// and enforce is on, and allows subagent edits, .devrites bookkeeping edits, an
-// inline-fallback window, and a closed window.
+// and enforce is on, and allows subagent edits, .devrites bookkeeping edits, and
+// a closed window.
 func TestParityA1Guard(t *testing.T) {
 	work := t.TempDir()
 	makeFeatureDir(t, work, "win")
 	writeFeatureFile(t, work, "win", ".reconcile-base", "snapshot\n")
-	makeFeatureDir(t, work, "inline")
-	writeFeatureFile(t, work, "inline", ".reconcile-base", "snapshot\n")
-	writeFeatureFile(t, work, "inline", ".reconcile-inline", "")
 	makeFeatureDir(t, work, "nowin") // no .reconcile-base → window closed
 
 	enforce := []string{"DEVRITES_A1_HOOK=enforce"}
@@ -291,7 +288,6 @@ func TestParityA1Guard(t *testing.T) {
 		{"devrites-edit-allowed", "win", enforce, `{"tool_name":"Edit","tool_input":{"file_path":".devrites/features/win/state.md"}}`},
 		{"non-edit-tool", "win", enforce, `{"tool_name":"Bash","tool_input":{"command":"ls"}}`},
 		{"observe-mode-silent", "win", nil, `{"tool_name":"Edit","tool_input":{"file_path":"src/app.js"}}`},
-		{"inline-fallback-silent", "inline", enforce, `{"tool_name":"Edit","tool_input":{"file_path":"src/app.js"}}`},
 		{"no-window-silent", "nowin", enforce, `{"tool_name":"Edit","tool_input":{"file_path":"src/app.js"}}`},
 	} {
 		tc := tc

package/pack/.claude/skills/devrites-api-interface/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-api-interface
 description: Design stable API/interface contracts before implementation: REST/GraphQL endpoints, module boundaries, type contracts, FE/BE splits. Use when the user says "design the API" or a slice crosses a boundary. Not for helpers.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-api-interface: contract before implementation

package/pack/.claude/skills/devrites-audit/SKILL.md

@@ -3,6 +3,7 @@ name: devrites-audit
 description: Read-only feature audit on security/perf/simplify: auth trust boundary, injection, secrets, hot-path, payload-size, perf budget, Chesterton delete/cleanup. Use when auditing one axis. Not for writes.
 argument-hint: "<security | perf | simplify>"
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-audit: read-only audit dispatch
@@ -39,14 +40,12 @@ no cross-pollination and the shared maximum of three concurrent read-only roles.
 
 ## Fallback and scope
 
-Use the capability ladder; inline is allowed only when no safe fresh-context rung
-preserves the role's read-only boundary or policy rejects it, and must say
-`independence: fallback`. Use these role contracts:
+Use the capability ladder. If no safe fresh-context rung preserves the role's
+read-only boundary, stop for HITL. Use these role contracts:
 
 - `.claude/agents/devrites-security-auditor.md`
 - `.claude/agents/devrites-performance-reviewer.md`
 - `.claude/agents/devrites-simplifier-reviewer.md`
 
-Seal weights that fallback under
-[`risk-and-rollback.md`](../rite-seal/reference/risk-and-rollback.md). Stay inside the
-active feature. Critical findings block seal; simplification never changes behavior.
+Stay inside the active feature. Critical findings block seal; simplification never
+changes behavior.

package/pack/.claude/skills/devrites-browser-proof/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-browser-proof
 description: Observe and prove working UI behavior in a real browser with screenshots, interactions, network evidence, and Core Web Vitals. Use for browser proof or performance after the page works.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-browser-proof: runtime evidence for UI

package/pack/.claude/skills/devrites-debug-recovery/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-debug-recovery
 description: Debug application failures: tests, builds, CI, runtime exceptions, browser errors, app 500s. Reproduce, rank hypotheses, fix root cause, regression-test. Use when "debug", "build is red", "tests fail", or the app is broken. Not for DevRites install health.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-debug-recovery: fix the root cause, not the symptom

package/pack/.claude/skills/devrites-doubt/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-doubt
 description: Stress-test one non-trivial decision with an adversarial independent take. Use when the user says "are we sure", "double-check", "what could go wrong", or before boundary/data/auth/API/migration choices. Not for trivial choices.
 user-invocable: false
+required-agent-roles: devrites-doubt-reviewer
 ---
 
 # devrites-doubt: CLAIM → EXTRACT → DOUBT → RECONCILE → STOP
@@ -22,7 +23,7 @@ code · claiming "this is safe", "this scales", or "this matches the spec".
   `agent-packet/v1` and the adversarial objective *"find what's wrong; do not
   validate."* Follow
   [`agents.md`](../devrites-lib/reference/standards/agents.md). Inline does not satisfy
-  independence; it is a labeled fallback only at the final capability-ladder rung.
+  independence; if no fresh-agent rung is available, stop for HITL.
 - [ ] **4. RECONCILE**: classify EVERY finding: contract misread | valid & actionable |
   valid trade-off | noise. **Doubt-theater check:** if two or more cycles find substantive
   issues but classify **zero** as actionable, the review is too agreeable. Sharpen the

package/pack/.claude/skills/devrites-frontend-craft/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-frontend-craft
 description: Implement frontend at senior designer-engineer quality: every state covered, anti-AI-slop, WCAG 2.2 AA. Use when the user says "build the UI", "design system", "a11y", or `/rite-build` detects UI. Not for polishing a built feature or design exploration.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-frontend-craft: UI like a senior designer-engineer

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

@@ -2,6 +2,7 @@
 name: devrites-interview
 description: Interview the user one question at a time to extract intent. Use when the user says "interview me", "I am not sure what I want", or the ask is underspecified. Not for casual clarification.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-interview: extract intent

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

@@ -3,6 +3,7 @@ name: devrites-lib
 description: Internal shared DevRites helper library. Documents cross-cutting engine commands and references; not a user workflow. Do not invoke directly.
 user-invocable: false
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # devrites-lib: internal shared helpers (not a command)

package/pack/.claude/skills/devrites-lib/reference/model-tiers.md

@@ -21,13 +21,14 @@ one to a reviewer to save cost. The tier a scout runs on is where savings are sa
 ## The degradation rule
 
 Tiers are an optimization, never a correctness dependency. A skill written against tiers must run
-correctly on a harness that cannot select models per agent, or has no subagent primitive at all:
+correctly on a harness that cannot select models per agent:
 
 1. **Per-agent model override unavailable** → dispatch the scout on the **inherited** model and
    keep its **read budget and output cap**. The cost control falls back to structure, not model
    choice. This is why every scout dispatch also carries an explicit read budget.
-2. **No subagent primitive at all** → run the scout work **inline** in the orchestrator, with the
-   same budgets. The tier still tells you how much to spend, even with nothing to dispatch to.
+2. **No safe fresh-agent rung** → stop for HITL when the scout is required, or record the
+   explicitly optional scout as skipped. Never relabel root-context work as independent
+   specialist dispatch.
 
 A skill that only works when it can pick a cheap model is broken. The tier names the *intent*; the
 budget enforces it regardless of what the harness can do.

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

@@ -91,9 +91,8 @@ devrites-engine footprint roster <slug>
 ```
 
 `roster` rc=3 means an unaccounted reviewer: dispatch or record why it does not apply.
-rc=1 means an always-on skip; this is valid only for an unchanged-candidate carry-forward
-or the explicit independence fallback in
-[`../../rite-seal/reference/risk-and-rollback.md`](../../rite-seal/reference/risk-and-rollback.md).
+rc=1 means an always-on skip; this is valid only for an unchanged-candidate
+carry-forward. A required reviewer that cannot be dispatched blocks the phase.
 
 ## Reconciliation
 
@@ -106,5 +105,5 @@ or the explicit independence fallback in
 6. Apply the caller's gate; do not invent a composite score.
 
 Every panel reviewer runs at the ceiling or inherited tier. If the named role is
-unavailable, use the universal ladder. Record an inline fallback as non-independent
-under the seal risk rule, never as a fresh-context pass.
+unavailable, use the universal ladder. If no fresh-agent rung is available, stop for
+HITL; never execute the reviewer in the root context.

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

@@ -50,7 +50,7 @@ review fan-out. Phase skills own their phase-locked role choice.
 
 ## Fresh-context dispatch ladder
 
-Use the first available rung; capability failure never changes the contract:
+Use the first safe rung:
 
 1. Dispatch the named custom role with the host's spawn primitive.
 2. If spawning works but that role is unavailable, use a generic fresh agent only when
@@ -59,13 +59,19 @@ Use the first available rung; capability failure never changes the contract:
    allowlist (or an isolated/staged checkout) for a `worker`. Tell it to read the
    generated role contract, then give it the same packet. An instruction-only shared-tree
    writer is not an available rung.
-3. When no safe fresh-context rung exists or policy rejects spawning, run the discipline
-   inline under the same budgets and label it `independence: fallback`. It is not an
-   independent review; seal applies
-   [`risk-and-rollback.md`](../../../rite-seal/reference/risk-and-rollback.md).
-
-Host mapping: Claude Code uses `Agent` (`Task` is its legacy alias); Codex uses
-`spawn_agent`. Use the host-neutral term **fresh-context dispatch** in shared guidance.
+3. Otherwise stop for HITL; specialist work never runs in the root context.
+
+Every canonical skill declares `required-agent-roles` in its frontmatter. Use `none`
+when no fresh agent is unconditionally required for that invocation. Codex reads this
+field from the installed skill at `UserPromptSubmit` and arms a fail-closed completion
+receipt for every listed role; the engine derives roles from skill metadata.
+Conditional scouts and reviewers remain owned by their explicit phase triggers.
+
+Claude uses `Agent` (`Task` alias); Codex uses `spawn_agent`. V2 sends the exact named
+role, a unique `task_name`, and `fork_turns="none"`; Codex loads its TOML natively.
+Because V2 lifecycle calls bypass hooks, Stop/reconcile verify the durable rollout's
+role, instructions, wait, completion, and delivered result. V1 uses guarded
+`explorer`/`worker` with injected rules and a lifecycle receipt. Prose isn't evidence.
 
 ## File-backed dispatch contract
 

package/pack/.claude/skills/devrites-prose-craft/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-prose-craft
 description: Humanize prose without losing technical content. Use when asked to remove AI-writing tells, match a writing sample or project voice, edit artifacts/replies/commit or PR prose, or run `/rite-polish`'s prose pass. Not for code comments or UI craft.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-prose-craft: prose that reads human

package/pack/.claude/skills/devrites-refresh-indexes/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-refresh-indexes
 description: Refresh stale optional indexes. Use when structural lookup disagrees with live code or rite-doctor requests a synchronous reindex.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-refresh-indexes: keep the code-intelligence indexes fresh

package/pack/.claude/skills/devrites-source-driven/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-source-driven
 description: Verify uncertain framework/library behavior against official docs or installed source before relying on it. Use when the user says "check the docs", "verify this assumption", or hits an unfamiliar API. Not for internal code.
 user-invocable: false
+required-agent-roles: devrites-evidence-scout
 ---
 
 # devrites-source-driven: verify, don't guess

package/pack/.claude/skills/devrites-ux-shape/SKILL.md

@@ -2,6 +2,7 @@
 name: devrites-ux-shape
 description: Plan UX/UI before code: design direction, design brief, visual direction, key states, interaction model for screens/forms/dashboards/settings pages. Use when shaping UI. Not for building/polish.
 user-invocable: false
+required-agent-roles: none
 ---
 
 # devrites-ux-shape: plan the UX/UI before code

package/pack/.claude/skills/rite/SKILL.md

@@ -4,6 +4,7 @@ description: User-invoked DevRites menu and router; no args renders the menu, a
 argument-hint: "[verb [args...]]"
 user-invocable: true
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # /rite: DevRites menu + router

package/pack/.claude/skills/rite-adopt/SKILL.md

@@ -3,6 +3,7 @@ name: rite-adopt
 description: Adopt an existing or legacy codebase into DevRites by reverse-engineering current behavior, seeding conventions, and establishing the workflow. Use for inherited or live applications.
 argument-hint: "[path or area to adopt] [+ what you want to build next]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-adopt: onboard existing code

package/pack/.claude/skills/rite-autocomplete/SKILL.md

@@ -3,6 +3,7 @@ name: rite-autocomplete
 description: Run the DevRites lifecycle end-to-end unattended; --ship confirms the final gate. Use when the user says "autocomplete", "one-shot this feature", or "ship it autonomously". Not for a single phase.
 argument-hint: "[idea] [--ship|--yolo] [--max-slices N]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-autocomplete: full lifecycle, unattended

package/pack/.claude/skills/rite-build/SKILL.md

@@ -3,6 +3,7 @@ name: rite-build
 description: Build one approved vertical slice, then stop with evidence. Use when the next planned slice should be implemented. Not for multiple slices.
 argument-hint: "[slice number or name]"
 user-invocable: true
+required-agent-roles: devrites-slice-wright
 ---
 
 # /rite-build: one verified slice
@@ -19,7 +20,7 @@ doubt, fail-on-red, recording, and stop checks. See
 ## Rules consulted (read on demand from `.claude/skills/devrites-lib/reference/standards/`)
 Read `.claude/skills/devrites-lib/reference/standards/core.md` first (workflow step 0). The
 following load on demand: **the wright reads them** (they are named in its contract) while it
-writes; read them yourself for the doubt/record gates or in the inline fallback:
+writes; read them yourself for the doubt/record gates:
 - `coding-style.md`: naming, function shape, guard clauses, comments, reuse-first.
 - `error-handling.md`: fail fast, no silent catches, fail closed.
 - `testing.md`: pyramid, behaviour over implementation, see-it-fail-first.

package/pack/.claude/skills/rite-build/reference/phase-contract.md

@@ -55,9 +55,8 @@ See also [`one-slice-cycle.md`](one-slice-cycle.md).
    **Default serial branch.** Follow
    [`wright-dispatch.md`](wright-dispatch.md) end to end; it owns snapshot timing, stuck
    logging, the exact packet, conditional skill loading, and return checks. Dispatch through
-   named wright → safely enforced generic fresh worker → labelled inline fallback. Inline is
-   allowed only when no fresh worker preserves recognized wright identity plus exact allowlist
-   enforcement or isolated staging; it never skips reconciliation. Before any canonical
+   named wright → safely enforced generic fresh worker. If neither is available, stop for
+   HITL; the root never performs wright work. Before any canonical
    `.devrites/` mutation, validate the typed result and require a clean immediate
    `devrites-engine reconcile check`; exit `5` rejects the writer result.
 4. **Doubt every stood decision before accepting the slice.** The writer never grades its
@@ -176,4 +175,4 @@ See also [`one-slice-cycle.md`](one-slice-cycle.md).
    just built. **When every slice is built** (`✅ ALL BUILT`) say so explicitly (the build
    phase is complete, the next phase is `/rite-prove`) don't leave completion implicit.
 
-> **Mid-flight discipline.** The wright (or you, in the inline fallback) must resist doing two slices, skipping TDD, adding a defensive check, or wandering outside `touched-files.md`; you must resist skipping the post-return `devrites-doubt` because the wright "seems confident": see [`anti-patterns.md`](anti-patterns.md). Load it the moment you reach for the excuse.
+> **Mid-flight discipline.** The wright must resist doing two slices, skipping TDD, adding a defensive check, or wandering outside `touched-files.md`; you must resist skipping the post-return `devrites-doubt` because the wright "seems confident": see [`anti-patterns.md`](anti-patterns.md). Load it the moment you reach for the excuse.

package/pack/.claude/skills/rite-build/reference/wright-dispatch.md

@@ -121,15 +121,14 @@ Only then persist `state.md`, `evidence.md`, `touched-files.md`, decision/doubt
 and browser evidence. Keep the retained window across a genuine human wait that will
 resume the same slice. Close it before abandoning the slice for a scope/plan transition.
 
-## Host fallback
-
-Use named wright → safely enforced generic fresh worker → labelled inline fallback from
-`standards/agents.md`. A generic worker is eligible only when the host preserves
-recognized wright identity plus exact allowlist enforcement, or contains it in an
-isolated/staged checkout. Post-hoc reconciliation alone cannot authorize an
-instruction-only shared-tree writer.
-
-When no safe fresh-context writer rung is callable, the root may execute the same bounded
-wright discipline inline. Write `.reconcile-inline` after the initial snapshot so the legacy
-A1 pre-hook recognizes the authorized fallback; reconciliation does **not** skip. The
-same allowlist, baseline, integrity gates, and close command remain mandatory.
+## Host dispatch
+
+Use named wright → enforced V1 generic worker from `standards/agents.md`. On Codex
+MultiAgent V2, dispatch the exact named `devrites-slice-wright` with a unique
+`task_name` and `fork_turns="none"`; its durable child rollout must prove the native
+wright role, wait, completion, and non-empty result from the current reconcile window.
+Any generic worker still needs the exact allowlist or an isolated checkout;
+reconciliation alone is insufficient. If no safe fresh-agent rung is available, stop
+for HITL. On Codex, `reconcile check` and `reconcile close` require the completed
+wright receipt bound to the current reconcile snapshot.
+The root never performs wright work.

package/pack/.claude/skills/rite-clarify/SKILL.md

@@ -3,6 +3,7 @@ name: rite-clarify
 description: Run the topology-first decision-coverage scan after rite-spec and before strategy or architecture. Audit actors, journeys, states, data, integrations, operations, proof, assumptions, and must-NOT boundaries; close Partial/Missing rows and emit decision-coverage.md CLEAR. Use for missing/stale coverage or a missed pre-build product decision.
 argument-hint: "[feature-slug]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-clarify: resolve product decisions

package/pack/.claude/skills/rite-converge/SKILL.md

@@ -3,6 +3,7 @@ name: rite-converge
 description: Converge intent and live code. Use when resuming a half-built feature, after `/rite-adopt` drift, or the user asks "what's left to build". Not for initial planning.
 argument-hint: "[feature-slug]"
 user-invocable: true
+required-agent-roles: devrites-evidence-scout
 ---
 
 # /rite-converge: compare live code with intent

package/pack/.claude/skills/rite-customize/SKILL.md

@@ -4,6 +4,7 @@ description: User-invoked helper for authoring DevRites overrides or extensions.
 argument-hint: "[override <agent> | extension <name>]"
 user-invocable: true
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # /rite-customize: guided project customization

package/pack/.claude/skills/rite-define/SKILL.md

@@ -3,6 +3,7 @@ name: rite-define
 description: Define the first build plan from an approved spec: architecture, task slices, traceability, and state. Use when turning approved intent into its initial plan.
 argument-hint: "[feature-slug]"
 user-invocable: true
+required-agent-roles: devrites-plan-drafter
 ---
 
 # /rite-define: plan from the spec

package/pack/.claude/skills/rite-doctor/SKILL.md

@@ -3,6 +3,7 @@ name: rite-doctor
 description: Doctor DevRites setup or index health. Use when workflow wiring is broken, the user asks to check DevRites, or says "reindex". Not for application bugs.
 argument-hint: "[--code | --reindex]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-doctor: health check

package/pack/.claude/skills/rite-dogfood/SKILL.md

@@ -4,6 +4,7 @@ description: Explicit browser dogfood QA for the active feature or branch.
 argument-hint: "[feature-slug|branch] [--port N]"
 user-invocable: true
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # /rite-dogfood: diff-scoped browser QA

package/pack/.claude/skills/rite-explain/SKILL.md

@@ -4,6 +4,7 @@ description: User-invoked explainer that teaches one concept, diff, idea, or rec
 argument-hint: "[a concept | a diff ref | an idea | \"what did I do this week?\"], or bare to be asked"
 user-invocable: true
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # /rite-explain: the human half of the learning loop
@@ -42,8 +43,9 @@ Dispatch by task shape, per [`devrites-lib/reference/model-tiers.md`](../devrite
   orchestrator's own model; nothing is dispatched down. Teaching quality is the whole product: do
   not cheapen it.
 
-The degradation rule in that file applies: no per-agent model control → run the scout inline under
-the same budget.
+No per-agent model control means dispatch the scout as a same-model fresh agent under the
+same budget. If no fresh-agent rung is available, stop with the missing capability; never run
+the scout in the root context.
 
 ## Workflow
 

package/pack/.claude/skills/rite-frame/SKILL.md

@@ -3,6 +3,7 @@ name: rite-frame
 description: Frame an ad-hoc ask before coding, then audit the diff. Use for underspecified imperative asks, raw-diff self-audits, or `/rite-quick` setup. Not a lifecycle gate.
 argument-hint: "[task to frame | diff to audit]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-frame: frame the goal, audit the diff

package/pack/.claude/skills/rite-handoff/SKILL.md

@@ -4,6 +4,7 @@ description: User-invoked handoff writer: sync chat-only context into `.devrites
 user-invocable: true
 disable-model-invocation: true
 argument-hint: "[what the next session will focus on]"
+required-agent-roles: none
 ---
 
 # /rite-handoff: chat-only context, into a fresh-agent doc

package/pack/.claude/skills/rite-learn/SKILL.md

@@ -4,6 +4,7 @@ description: User-invoked review of the learning ledger; promotes recurring less
 argument-hint: "[--mine | \"<lesson to record>\"]"
 user-invocable: true
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # /rite-learn: the cross-feature learning loop

package/pack/.claude/skills/rite-plan/SKILL.md

@@ -3,6 +3,7 @@ name: rite-plan
 description: Re-plan existing work when reality invalidates the plan: reslice a slice that is too big, repair drift, reorder dependencies, split boundaries, or unblock work. Not first-pass decomposition.
 argument-hint: "[mode: decompose|reslice|repair|reorder|split|unblock|course-correct|revise]"
 user-invocable: true
+required-agent-roles: devrites-plan-drafter
 ---
 
 # /rite-plan: (re)plan an active feature

package/pack/.claude/skills/rite-polish/SKILL.md

@@ -3,6 +3,7 @@ name: rite-polish
 description: Polish the active feature before review: code always, UI when touched. Use when the user says "polish this", "finish before review", "normalize the UI", or "make it bolder/quieter". Not for repo-wide refactors.
 argument-hint: "[target | bolder | quieter | distill | harden | normalize-only]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-polish: finish before review

package/pack/.claude/skills/rite-pov/SKILL.md

@@ -3,6 +3,7 @@ name: rite-pov
 description: Project-grounded verdict for adopting, switching, rejecting, or revisiting a named external technology, library, platform, CVE, or pattern. Use when deciding whether this project should commit to an outside option.
 argument-hint: "[candidate/link/question]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-pov: project-grounded external verdict

package/pack/.claude/skills/rite-pr-feedback/SKILL.md

@@ -4,6 +4,7 @@ description: Explicit utility for resolving GitHub PR review feedback.
 argument-hint: "[PR number|thread URL|blank for current branch]"
 user-invocable: true
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # /rite-pr-feedback: resolve PR review threads

package/pack/.claude/skills/rite-pressure-test/SKILL.md

@@ -3,6 +3,7 @@ name: rite-pressure-test
 description: Pressure-test a rough/vague idea: ideate, explore 3-5 approaches, radically different shapes, diverge then converge on direction before spec. Not for writing spec.
 argument-hint: "[rough idea or plan to stress-test]"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-pressure-test: diverge then converge

package/pack/.claude/skills/rite-prototype/SKILL.md

@@ -4,6 +4,7 @@ description: User-invoked throwaway prototype for one logic or UI design questio
 user-invocable: true
 disable-model-invocation: true
 argument-hint: "[the question the prototype is answering]"
+required-agent-roles: none
 ---
 
 # /rite-prototype: answer one question with throwaway code

package/pack/.claude/skills/rite-prove/SKILL.md

@@ -3,6 +3,7 @@ name: rite-prove
 description: Prove a completed feature with tests and the full test suite, build/typecheck/lint, end-to-end/browser evidence, screenshots, commands, and outputs for seal. Not for single-slice proof.
 argument-hint: "[feature-slug]"
 user-invocable: true
+required-agent-roles: devrites-proof-runner
 ---
 
 # /rite-prove: prove the completed feature

package/pack/.claude/skills/rite-quick/SKILL.md

@@ -3,6 +3,7 @@ name: rite-quick
 description: Express lane for a small reversible change: typo/copy/config, rate-limit constant, error message, local variable rename, feature flag, README link, one-file/function fix.
 argument-hint: "<what to change>"
 user-invocable: true
+required-agent-roles: none
 ---
 
 # /rite-quick: express lane for small changes

package/pack/.claude/skills/rite-resolve/SKILL.md

@@ -4,6 +4,7 @@ description: User-invoked resume verb for answering, dropping, or batch-resolvin
 argument-hint: "<qid> \"<answer>\"  |  --drop <qid> [\"<reason>\"]  |  --batch <path-to-yaml>"
 user-invocable: true
 disable-model-invocation: true
+required-agent-roles: none
 ---
 
 # /rite-resolve: answer the human gate

package/pack/.claude/skills/rite-review/SKILL.md

@@ -3,6 +3,7 @@ name: rite-review
 description: Review polished feature diff for correctness, readability, architecture, security, tests proving acceptance, Critical/Important findings, and quality dimensions before seal.
 argument-hint: "[scope: slice N | feature] [--full]"
 user-invocable: true
+required-agent-roles: devrites-spec-reviewer, devrites-code-reviewer
 ---
 
 # /rite-review: feature-scoped review

package/pack/.claude/skills/rite-seal/SKILL.md

@@ -3,6 +3,7 @@ name: rite-seal
 description: Decide GO / NO-GO readiness on the active feature. Use when the user asks "seal this", "GO / NO-GO", "decide if we can ship", "can we ship", or "safe to merge". Not for commit/push/tag/close; use /rite-ship.
 argument-hint: "[feature-slug] [--full]"
 user-invocable: true
+required-agent-roles: devrites-test-analyst
 ---
 
 # /rite-seal: GO / NO-GO