From b33be1e6e5d3dbc420861b31c22f69078f264b51 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Thu, 23 Jul 2026 19:47:53 +0000
Subject: [PATCH 1/3] Graph-engineering upgrades: model tiering, budget
 hard-stops, diagrams
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Distilled the actionable parts of a "graph engineering" writeup (our Workflow
tool already is a graph runtime) into three concrete improvements:

- Model tiering across all 6 workflows: high-volume fan-out nodes (critics,
  per-story evidence, per-dimension gaps, per-category E2E cases, meta-critics)
  run model:'sonnet'; the high-stakes gates/judgment (adversarial verifiers,
  status verifier, synthesis) run model:'opus'. Tiers are FANOUT_MODEL/
  JUDGE_MODEL constants overridable via args.models. The loop-iteration
  implementer is the deliberate exception — it writes real code, so it inherits
  the session model rather than being silently downgraded.
- Budget as a hard stop for the expensive fan-outs, guarded on budget.total
  (null => no cap): improve-skills skips the verify phase below a token floor
  and routes findings to needsDesign (unverified, never auto-applied);
  loop-iteration escalates all items below its floor instead of failing
  mid-fix. Fail-safe direction preserved — a budget stop never approves
  unverified work.
- docs/workflow-graphs.md: Mermaid node/edge diagram for every workflow with
  the per-node model tier (the "see the graph before you run" view).

Filed the tiering, soft/hard-budget, and diagram-drift conventions in the
memory wiki. Syntax validated for all six scripts.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014xUCrJdREvUMjoVzSX5T8G
---
 .claude/memory/log.md               |  20 ++++
 .claude/memory/topics/workflow.md   |  20 ++++
 .claude/workflows/critic-panel.js   |   9 +-
 .claude/workflows/e2e-design.js     |   7 +-
 .claude/workflows/gap-analysis.js   |   9 +-
 .claude/workflows/improve-skills.js |  53 ++++++++---
 .claude/workflows/loop-iteration.js |  21 ++++-
 .claude/workflows/scan-docs.js      |   9 +-
 docs/workflow-graphs.md             | 139 ++++++++++++++++++++++++++++
 9 files changed, 264 insertions(+), 23 deletions(-)
 create mode 100644 docs/workflow-graphs.md

diff --git a/.claude/memory/log.md b/.claude/memory/log.md
index b67e856..cae839f 100644
--- a/.claude/memory/log.md
+++ b/.claude/memory/log.md
@@ -151,3 +151,23 @@ change), `rotate` (archived old entries). Format: `## [YYYY-MM-DD] <op> | <summa
 - Lesson: a self-improving system MUST add its own prompt surface (`.claude/`,
   `CLAUDE.md`) to every autonomous denylist — otherwise the loop can edit the rules
   that constrain it. The meta-critic caught this in its own machinery on the first run.
+
+## [2026-06-29] ingest | Graph-engineering upgrades (model tiering, budget, diagrams)
+- Distilled from a "graph engineering" post (Slate/Random Labs). Our `Workflow` tool
+  already IS a graph runtime (nodes=agents, edges=hand-offs, parallel/pipeline, scoped
+  failure); three real gaps were closed:
+  - **Model tiering** across all 6 workflows: fan-out nodes (critics, evidence, gaps,
+    E2E cases, meta-critics) → `model: 'sonnet'`; gates/judgment (verifiers, status
+    verifier, synthesis) → `model: 'opus'`; overridable via `args.models`. The
+    `loop-iteration` implementer stays on the session model (never downgrade a
+    code-writing node). Cuts fan-out cost, keeps quality at the decision points.
+  - **Budget hard-stops**: `improve-skills` skips the verify fan-out below a token
+    floor (findings → `needsDesign`, unverified, never auto-applied); `loop-iteration`
+    escalates all items below its floor instead of failing mid-fix. Guarded on
+    `budget.total` (null ⇒ no cap).
+  - **Diagram-first**: `docs/workflow-graphs.md` draws every workflow's node/edge graph
+    (Mermaid) with the per-node model tier — the "see the graph before you run" view.
+- Filed the tiering + soft/hard-budget + diagram-drift conventions in topics/workflow.md.
+- Not adopted (out of scope): the post's quant/hedge-fund framing and the standalone
+  Slate runtime — we already have the graph runtime; the trading claims overstate what
+  orchestration solves (data quality, costs, overfitting remain the hard part).
diff --git a/.claude/memory/topics/workflow.md b/.claude/memory/topics/workflow.md
index 3c76db5..a8d5c06 100644
--- a/.claude/memory/topics/workflow.md
+++ b/.claude/memory/topics/workflow.md
@@ -57,6 +57,26 @@
   missing), and give a maker/checker checker the **real artifact** (the diff), not
   the maker's self-report. The DSL auto-caps concurrency (~min(16, cores-2)), so a
   large fan-out won't spawn unbounded agents — no manual limit needed.
+- **Model tiering (graph-engineering): fan-out → fast tier, gate → strong tier.**
+  High-volume, individually-low-stakes fan-out nodes (critics, per-story evidence,
+  per-dimension gaps, per-category E2E cases) run `model: 'sonnet'`; the high-stakes
+  gates/judgment (adversarial verifiers, status verifier, synthesis) run
+  `model: 'opus'`. Define `FANOUT_MODEL`/`JUDGE_MODEL` constants (default `sonnet`/
+  `opus`, overridable via `args.models = { fanout, judge }`) and pass `model:` per
+  `agent()`. **Exception:** a node that writes real code (the `loop-iteration`
+  implementer) inherits the session model — never silently downgrade a code-writing
+  node. This cuts cost on the wide fan-out and keeps quality where the decision is made.
+- **Budget as soft + hard control.** For big fan-outs, guard on `budget.total` (it is
+  `null` when no target was set — without the guard `budget.remaining()` is `Infinity`
+  and a loop never stops). Hard-stop pattern: if `budget.total && budget.remaining()
+  < FLOOR`, skip the expensive stage instead of failing mid-run — `loop-iteration`
+  escalates all items (`budgetStopped`), `improve-skills` skips the verify fan-out and
+  routes findings to `needsDesign` (unverified ⇒ never auto-applied). Keep the fail-safe
+  direction: a budget stop must never *approve* unverified work.
+- **Diagram-first:** every workflow's node/edge graph is drawn in
+  `docs/workflow-graphs.md` (Mermaid), with the model tier per node. Update the diagram
+  when you add a node, edge, or change a tier — `/improve-skills` and `/dream lint`
+  flag drift between a diagram and its script.
 - A command's `description` also lets Claude **auto-invoke** it. Add
   `disable-model-invocation: true` to heavy/code-changing commands that should be
   manual-only.
diff --git a/.claude/workflows/critic-panel.js b/.claude/workflows/critic-panel.js
index 522ea73..00f337a 100644
--- a/.claude/workflows/critic-panel.js
+++ b/.claude/workflows/critic-panel.js
@@ -19,6 +19,11 @@ const priorEvidence = (args && args.priorEvidence) || null
 // Defaults to including them (safe). Callers pass `uiInScope: false` to save fan-out.
 const uiInScope = !(args && args.uiInScope === false)
 
+// Model tiering (graph-engineering): high-volume fan-out on the fast tier, the
+// high-stakes adversarial gate on the strong tier. Override via args.models.
+const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
+const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
+
 // The standard critic roster. `ui: true` marks browser-facing critics.
 const ALL_CRITICS = [
   { key: 'first-time-user', ui: true, label: 'First-Time User Critic', lens: 'Can a new user understand the first screen and complete the flow without help? Labels, errors, hidden assumptions.' },
@@ -73,7 +78,7 @@ const reviews = await parallel(
   CRITICS.map((c) => () =>
     agent(
       `You are the **${c.label}** for this project. Read CLAUDE.md and the relevant memory topic page(s), inspect ${focus} (read the code; if a localhost flow is in scope, drive it via Playwright/Chrome MCP and check console + network), then review strictly through your lens:\n${c.lens}\n${priorEvidence ? `\nEvidence already gathered by the Lead — verify against it instead of re-deriving from scratch:\n${typeof priorEvidence === 'string' ? priorEvidence : JSON.stringify(priorEvidence)}\n` : ''}\nReturn only findings you can back with concrete evidence. An empty list is a valid, honest answer.`,
-      { label: `critic:${c.key}`, phase: 'Review', schema: FINDINGS_SCHEMA },
+      { label: `critic:${c.key}`, phase: 'Review', schema: FINDINGS_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ({ critic: c.key, label: c.label, findings: (r && r.findings) || [] })),
   ),
 )
@@ -107,7 +112,7 @@ const verified = await parallel(
   toVerify.map((f, i) => () =>
     agent(
       `Adversarially verify this ${f.severity} finding from the ${f.criticLabel}. Try to REFUTE it by inspecting the actual code/flow — default to real:false if the evidence does not hold up.\n\nTitle: ${f.title}\nEvidence: ${f.evidence}\nRecommendation: ${f.recommendation}`,
-      { label: `verify:${f.critic}:${i}`, phase: 'Verify', schema: VERDICT_SCHEMA },
+      { label: `verify:${f.critic}:${i}`, phase: 'Verify', schema: VERDICT_SCHEMA, model: JUDGE_MODEL },
     ).then((v) => ({
       ...f,
       // Fail safe: a missing/failed verdict must NOT confirm the finding.
diff --git a/.claude/workflows/e2e-design.js b/.claude/workflows/e2e-design.js
index 8d8f539..5087459 100644
--- a/.claude/workflows/e2e-design.js
+++ b/.claude/workflows/e2e-design.js
@@ -7,6 +7,11 @@ export const meta = {
 
 const flow = (args && (args.flow || (typeof args === 'string' ? args : null))) || 'the target flow'
 
+// Model tiering (graph-engineering): this workflow is pure fan-out enumeration
+// (the dedup is plain code, not an agent), so every node runs on the fast tier.
+// Override via args.models.fanout.
+const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
+
 const CATEGORIES = [
   { key: 'happy', prompt: 'the happy path(s) a user follows when everything works' },
   { key: 'failure', prompt: 'validation/failure paths and error states' },
@@ -37,7 +42,7 @@ const results = await parallel(
   CATEGORIES.map((c) => () =>
     agent(
       `Enumerate concrete Playwright E2E test cases for "${flow}" covering ${c.prompt}. Inspect the routes/components and existing specs first. Each case needs: title, steps, and the visible expected outcome to assert.`,
-      { label: `e2e:${c.key}`, phase: 'Design', schema: CASES_SCHEMA },
+      { label: `e2e:${c.key}`, phase: 'Design', schema: CASES_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ((r && r.cases) || []).map((x) => ({ ...x, category: c.key }))),
   ),
 )
diff --git a/.claude/workflows/gap-analysis.js b/.claude/workflows/gap-analysis.js
index fe8ff8f..afbf3e2 100644
--- a/.claude/workflows/gap-analysis.js
+++ b/.claude/workflows/gap-analysis.js
@@ -10,6 +10,11 @@ export const meta = {
 
 const scope = (args && (args.scope || (typeof args === 'string' ? args : null))) || 'the whole project'
 
+// Model tiering (graph-engineering): per-dimension gap discovery on the fast tier,
+// the synthesis/prioritization judgment on the strong tier. Override via args.models.
+const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
+const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
+
 const DIMENSIONS = [
   { key: 'docs-not-impl', prompt: 'Requirements in docs (docs/prd.md, docs/stories/*, README) with NO implementation evidence in the codebase.' },
   { key: 'impl-not-docs', prompt: 'Implemented features/behaviors NOT described in any product doc.' },
@@ -55,7 +60,7 @@ const results = await parallel(
   DIMENSIONS.map((d) => () =>
     agent(
       `Analyze the gap for "${scope}" along this dimension by reading docs/, src/, e2e/, and the story map:\n${d.prompt}\nReturn only gaps backed by concrete references; an empty list is a valid, honest answer.`,
-      { label: `gap:${d.key}`, phase: 'Dimensions', schema: GAP_SCHEMA },
+      { label: `gap:${d.key}`, phase: 'Dimensions', schema: GAP_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ({ dimension: d.key, gaps: (r && r.gaps) || [] })),
   ),
 )
@@ -71,7 +76,7 @@ log(`${total} gaps across ${DIMENSIONS.length} dimensions`)
 phase('Synthesize')
 const synth = await agent(
   `Given these gaps grouped by dimension, propose the recommended NEXT-implementation order (highest-leverage first), each with a one-line "why". Be concrete.\n\n${JSON.stringify(byDimension)}`,
-  { label: 'synthesize', phase: 'Synthesize', schema: ORDER_SCHEMA },
+  { label: 'synthesize', phase: 'Synthesize', schema: ORDER_SCHEMA, model: JUDGE_MODEL },
 )
 
 return { scope, totalGaps: total, byDimension, recommendedOrder: (synth && synth.order) || [] }
diff --git a/.claude/workflows/improve-skills.js b/.claude/workflows/improve-skills.js
index 949e679..bbdcafc 100644
--- a/.claude/workflows/improve-skills.js
+++ b/.claude/workflows/improve-skills.js
@@ -36,6 +36,11 @@ const DEFAULT_TARGETS = [
 const targets = (a && Array.isArray(a.targets) && a.targets.length ? a.targets : DEFAULT_TARGETS)
 const targetList = targets.join(', ')
 
+// Model tiering (graph-engineering): the fan-out of meta-critics on the fast tier,
+// the two-axis adversarial verifier on the strong tier. Override via args.models.
+const FANOUT_MODEL = (a && a.models && a.models.fanout) || 'sonnet'
+const JUDGE_MODEL = (a && a.models && a.models.judge) || 'opus'
+
 // Meta-critic lenses — each reviews the SYSTEM'S OWN instructions, not product code.
 // `code: true` lenses focus on the workflow .js orchestration; the rest on prose prompts.
 const LENSES = [
@@ -88,7 +93,7 @@ const reviews = await parallel(
   LENSES.map((c) => () =>
     agent(
       `You are the **${c.label}** auditing this project's OWN Claude Code configuration — its custom prompts, skills, and multi-agent orchestration. You are NOT reviewing product/application code; you are reviewing the instructions that drive the agent.\n\nRead these files (the prompt surface): ${targetList}\n${c.code ? 'Focus on the workflow orchestration scripts (.claude/workflows/*.js).' : 'Focus on the prose prompts (commands, CLAUDE.md, skills-index, loop files).'}\n\nReview strictly through your lens:\n${c.lens}\n\nScope: ${scope}\n\nFor each issue return concrete evidence (a quote or file:line) AND a specific proposedEdit (what to change to what). Propose improvements to the prompts/workflows themselves — do not propose product-code changes. An empty list is a valid, honest answer.`,
-      { label: `meta:${c.key}`, phase: 'Review', schema: FINDINGS_SCHEMA },
+      { label: `meta:${c.key}`, phase: 'Review', schema: FINDINGS_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ({ critic: c.key, label: c.label, findings: (r && r.findings) || [] })),
   ),
 )
@@ -118,20 +123,37 @@ log(`${deduped.length} findings from ${LENSES.length} meta-critics (${all.length
 // a safe improvement (so we never "improve" a prompt into being weaker or less safe).
 phase('Verify')
 const toVerify = deduped.filter((f) => f.severity === 'blocker' || f.severity === 'important')
-const verified = await parallel(
-  toVerify.map((f, i) => () =>
-    agent(
-      `You are a SEPARATE skeptic (not the critic who raised this). Read the actual file and try to REFUTE this ${f.severity} finding about the project's own prompts/workflows.\n\nJudge two things independently:\n1. real — is the problem genuine? Default real:false if the evidence does not hold up.\n2. editSafe — would the proposed edit be a real improvement that does NOT weaken instruction-following, remove a safety/confirmation gate, or break a workflow? Default editSafe:false if you cannot confirm it is safe.\n\nFile: ${f.file}\nTitle: ${f.title}\nEvidence: ${f.evidence}\nProposed edit: ${f.proposedEdit}`,
-      { label: `verify:${f.critic}:${i}`, phase: 'Verify', schema: VERDICT_SCHEMA },
-    ).then((v) => ({
-      ...f,
-      // Fail safe: a missing/failed verdict must NOT confirm the finding or its edit.
-      real: v ? v.real : false,
-      editSafe: v ? v.editSafe : false,
-      verifyReason: (v && v.reason) || 'verifier returned no verdict — not confirmed',
-    })),
-  ),
-)
+
+// Budget as a HARD stop (soft + hard controls, graph-engineering): if the turn's token
+// target is nearly spent, do NOT spawn the verifier fan-out. Route findings to
+// needsDesign (real:true, editSafe:false) so nothing is auto-applied unverified.
+const VERIFY_FLOOR = 60_000 // tokens to leave for the verify phase
+const budgetStop = !!(budget.total && budget.remaining() < VERIFY_FLOOR)
+let verified
+if (budgetStop) {
+  log(`budget floor reached (${Math.round(budget.remaining() / 1000)}k left) — skipping adversarial verify; ${toVerify.length} findings routed to needsDesign (unverified, never auto-applied)`)
+  verified = toVerify.map((f) => ({
+    ...f,
+    real: true,
+    editSafe: false, // unverified ⇒ not apply-ready; surfaced for human review only
+    verifyReason: 'unverified — token budget floor reached before the verify phase',
+  }))
+} else {
+  verified = await parallel(
+    toVerify.map((f, i) => () =>
+      agent(
+        `You are a SEPARATE skeptic (not the critic who raised this). Read the actual file and try to REFUTE this ${f.severity} finding about the project's own prompts/workflows.\n\nJudge two things independently:\n1. real — is the problem genuine? Default real:false if the evidence does not hold up.\n2. editSafe — would the proposed edit be a real improvement that does NOT weaken instruction-following, remove a safety/confirmation gate, or break a workflow? Default editSafe:false if you cannot confirm it is safe.\n\nFile: ${f.file}\nTitle: ${f.title}\nEvidence: ${f.evidence}\nProposed edit: ${f.proposedEdit}`,
+        { label: `verify:${f.critic}:${i}`, phase: 'Verify', schema: VERDICT_SCHEMA, model: JUDGE_MODEL },
+      ).then((v) => ({
+        ...f,
+        // Fail safe: a missing/failed verdict must NOT confirm the finding or its edit.
+        real: v ? v.real : false,
+        editSafe: v ? v.editSafe : false,
+        verifyReason: (v && v.reason) || 'verifier returned no verdict — not confirmed',
+      })),
+    ),
+  )
+}
 
 const niceToHaves = deduped.filter((f) => f.severity === 'nice-to-have')
 const settled = verified.filter(Boolean)
@@ -146,6 +168,7 @@ const refuted = settled.filter((f) => f.real === false)
 return {
   scope,
   targets,
+  budgetStop, // true ⇒ verify was skipped for budget; applyReady is empty by design
   counts: {
     lenses: LENSES.length,
     confirmedBlockers: confirmed.filter((f) => f.severity === 'blocker').length,
diff --git a/.claude/workflows/loop-iteration.js b/.claude/workflows/loop-iteration.js
index ee235b6..596dd9d 100644
--- a/.claude/workflows/loop-iteration.js
+++ b/.claude/workflows/loop-iteration.js
@@ -11,10 +11,29 @@ export const meta = {
 // args.items: [{ id, description }] — already triaged and denylist-cleared by the loop.
 const items = (args && args.items) || []
 const level = (args && args.level) || 'L2'
+
+// Model tiering (graph-engineering): the adversarial verifier (the safety gate) runs
+// on the strong tier. The IMPLEMENTER intentionally has NO override — it writes real
+// code in a worktree, so it inherits the session model rather than being silently
+// downgraded (correctness of the change matters more than fan-out throughput here).
+// Override the verifier tier via args.models.judge.
+const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
 if (!items.length) {
   return { error: 'No work items. Triage + denylist-check first, then pass args.items = [{id, description}].', applied: [], rejected: [] }
 }
 
+// Budget as a HARD stop (soft + hard controls, graph-engineering): worktree
+// implementers are expensive, so if the turn's token target is nearly spent, do NOT
+// start any of them — escalate every item for a later run instead of failing mid-fix.
+const ITER_FLOOR = 80_000 // tokens to leave before starting the implement/verify fan-out
+if (budget.total && budget.remaining() < ITER_FLOOR) {
+  return {
+    level, applied: [], rejected: [], escalate: items.map((it) => it.id),
+    budgetStopped: true,
+    note: `budget floor reached (${Math.round(budget.remaining() / 1000)}k left < ${ITER_FLOOR / 1000}k) — no items processed; all ${items.length} escalated for a later run`,
+  }
+}
+
 const IMPL_SCHEMA = {
   type: 'object', additionalProperties: false, required: ['changedFiles', 'diff', 'checksPassed', 'notes'],
   properties: {
@@ -68,7 +87,7 @@ const outcomes = await pipeline(
     }
     return agent(
       `You are a SEPARATE verifier (not the implementer). Adversarially review the ACTUAL diff below for correctness, scope creep, and denylist violations, and judge whether the change is safe to apply. Default to pass:false if you cannot confirm it from the diff.\n\nItem: ${JSON.stringify(item)}\nChanged files: ${JSON.stringify(im.changedFiles || [])}\nchecksPassed (self-reported): ${!!im.checksPassed}\nDiff:\n${im.diff || '(no diff returned)'}`,
-      { label: `verify:${item.id || 'item'}-${i}`, phase: 'Verify', schema: VERDICT_SCHEMA },
+      { label: `verify:${item.id || 'item'}-${i}`, phase: 'Verify', schema: VERDICT_SCHEMA, model: JUDGE_MODEL },
     ).then((v) => ({
       id: item.id,
       // Fail safe: no verdict / verifier died → not approved.
diff --git a/.claude/workflows/scan-docs.js b/.claude/workflows/scan-docs.js
index 529576f..8eeecea 100644
--- a/.claude/workflows/scan-docs.js
+++ b/.claude/workflows/scan-docs.js
@@ -23,6 +23,11 @@ const REPO_STATUSES = [
   'Unit tested', 'E2E tested', 'Blocked', 'Deprecated',
 ]
 
+// Model tiering (graph-engineering): per-story evidence gathering on the fast tier,
+// the strict status verifier (the gate) on the strong tier. Override via args.models.
+const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
+const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
+
 const EVIDENCE_SCHEMA = {
   type: 'object',
   additionalProperties: false,
@@ -60,7 +65,7 @@ const records = await pipeline(
   (story, i) =>
     agent(
       `Gather IMPLEMENTATION and TEST evidence for this user story by searching the codebase and tests (do not write any files):\n${JSON.stringify(story)}\n\nReport which files implement it, whether unit and E2E tests cover it, and the most defensible status. "Not started" is correct when no implementation is found.`,
-      { label: `evidence:${story.id || 'story'}-${i}`, phase: 'Evidence', schema: EVIDENCE_SCHEMA },
+      { label: `evidence:${story.id || 'story'}-${i}`, phase: 'Evidence', schema: EVIDENCE_SCHEMA, model: FANOUT_MODEL },
     ),
   (evidence, story, i) => {
     // Guard the first-stage result like every sibling workflow does — a null/failed
@@ -68,7 +73,7 @@ const records = await pipeline(
     const ev = evidence || {}
     return agent(
       `You are a strict status verifier for story ${story.id} (${story.title}). Confirm a status that is justified by REPO evidence only (code + tests). From repo evidence you can confirm at most "E2E tested"; "Browser verified" and "Done" require a separate browser pass and are OUT of scope here — never assign them. Downgrade if the proposed status overclaims.\n\nProposed: ${JSON.stringify(ev)}`,
-      { label: `verify:${story.id || 'story'}-${i}`, phase: 'Verify', schema: VERDICT_SCHEMA },
+      { label: `verify:${story.id || 'story'}-${i}`, phase: 'Verify', schema: VERDICT_SCHEMA, model: JUDGE_MODEL },
     ).then((v) => ({
       id: story.id,
       title: story.title,
diff --git a/docs/workflow-graphs.md b/docs/workflow-graphs.md
new file mode 100644
index 0000000..db4fc16
--- /dev/null
+++ b/docs/workflow-graphs.md
@@ -0,0 +1,139 @@
+# Workflow graphs
+
+Visual map of every deterministic multi-agent Workflow under `.claude/workflows/`.
+Each is a **graph**: nodes = subagents (own context), edges = data hand-offs. This is
+the "see the graph before you run it" view (the diagram-first idea from graph
+engineering), kept in-repo so it version-controls alongside the scripts.
+
+**Model tiering** (graph-engineering convention): high-volume fan-out runs on the
+**fast tier** (Sonnet); gates, judgment, and synthesis run on the **strong tier**
+(Opus). The loop implementer is the deliberate exception — it writes real code, so it
+inherits the session model instead of being downgraded. All tiers are overridable via
+`args.models = { fanout, judge }`.
+
+> Legend: 🟢 fast tier (Sonnet) · 🔵 strong tier (Opus) · ⚪ inherits session model ·
+> ▫️ plain code (no agent). Barriers (`parallel`) collect all nodes before the next
+> stage; pipelines (`pipeline`) flow each item through independently.
+
+---
+
+## `critic-panel` — critic round (`/critic-round`, `/multi-agent-dev`, `/qa-pass`, `/loop`)
+
+```mermaid
+flowchart LR
+  focus([focus / changed files]) --> R
+  subgraph R["Review — parallel barrier · 🟢 Sonnet"]
+    c1[First-Time User]
+    c2[UX Flow]
+    c3[Designer]
+    c4[Artistic Direction]
+    c5[Frontend Arch]
+    c6[QA / E2E]
+    c7[Accessibility]
+    c8[Performance]
+    c9[Security]
+    c10[Regression]
+  end
+  R --> dedup[▫️ merge-dedup<br/>keep corroborating critics]
+  dedup --> V
+  subgraph V["Verify — one skeptic per blocker/important · 🔵 Opus"]
+    v1[refute finding #1]
+    v2[refute finding #2]
+    vn[refute finding #N]
+  end
+  V --> out([confirmed matrix + niceToHaves + refuted])
+```
+
+## `improve-skills` — meta-critic over our own prompts (`/improve-skills`)
+
+```mermaid
+flowchart LR
+  surface([.claude prompt + workflow surface]) --> R
+  subgraph R["Review — per lens, parallel · 🟢 Sonnet"]
+    l1[Clarity]
+    l2[Instruction-following risk]
+    l3[Safety / gating]
+    l4[Consistency / DRY]
+    l5[Workflow DSL / schema]
+    l6[Routing / discoverability]
+  end
+  R --> dedup[▫️ merge-dedup]
+  dedup --> gate{budget floor?}
+  gate -- "under floor" --> skip[▫️ skip verify →<br/>needsDesign, unverified]
+  gate -- ok --> V
+  subgraph V["Verify — 2-axis skeptic per finding · 🔵 Opus"]
+    v1[real? + editSafe?]
+  end
+  V --> out([applyReady + needsDesign + refuted])
+  skip --> out
+```
+
+## `scan-docs` — evidence-based story status (`/scan-project-docs`)
+
+```mermaid
+flowchart LR
+  stories([stories]) --> P
+  subgraph P["pipeline — per story, independent"]
+    e[🟢 gather code+test evidence] --> v[🔵 strict status verifier<br/>fail-closed cap]
+  end
+  P --> out([per-story status records])
+```
+
+## `gap-analysis` — docs↔code↔tests gaps (`/story-gap-analysis`)
+
+```mermaid
+flowchart LR
+  scope([scope]) --> D
+  subgraph D["Dimensions — parallel barrier · 🟢 Sonnet"]
+    d1[docs-not-impl]
+    d2[impl-not-docs]
+    d3[impl-not-unit]
+    d4[impl-not-e2e]
+    d5[e2e-missing-edges]
+    d6[ux-design]
+    d7[ambiguous]
+  end
+  D --> s[🔵 synthesize prioritized order]
+  s --> out([recommendedOrder])
+```
+
+## `e2e-design` — Playwright case enumeration (`/multi-agent-e2e`)
+
+```mermaid
+flowchart LR
+  flow([target flow]) --> D
+  subgraph D["Design — per path category, parallel · 🟢 Sonnet"]
+    d1[happy]
+    d2[failure]
+    d3[edge]
+    d4[accessibility]
+    d5[regression]
+  end
+  D --> dedup[▫️ dedup on category+title]
+  dedup --> out([concrete test cases])
+```
+
+## `loop-iteration` — maker/checker for `/loop` L2/L3
+
+```mermaid
+flowchart LR
+  items([triaged, denylist-cleared items]) --> budget{budget floor?}
+  budget -- "under floor" --> esc0[▫️ escalate all]
+  budget -- ok --> P
+  subgraph P["pipeline — per item, independent"]
+    impl[⚪ implementer<br/>isolated worktree, real git diff] --> dl{▫️ denylist gate<br/>.claude / CLAUDE.md / auth / secrets…}
+    dl -- hit --> escd[escalated-denylist]
+    dl -- clear --> ver[🔵 verifier reviews real diff]
+  end
+  P --> out([applied + rejected + escalate])
+  esc0 --> out
+```
+
+---
+
+## How to regenerate
+
+These diagrams are hand-maintained to mirror `.claude/workflows/*.js`. When you add a
+node, change a tier, or add an edge, update the matching diagram here. `/improve-skills`
+(Consistency lens) and `/dream lint` should flag a diagram that has drifted from its
+script.

From 5c85e09a392d3397b387f29dbe21fe9bcc964358 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Thu, 23 Jul 2026 20:15:38 +0000
Subject: [PATCH 2/3] Apply gate findings on the tiering change: fix denylist
 bypass + wire drift checks
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Ran /improve-skills over the graph-engineering diff (on the new tiering itself);
it confirmed 1 blocker + 7 important. Applied all:

- BLOCKER (loop-iteration): the deterministic denylist gate trusted the
  implementer's self-reported changedFiles, but the caller applies the real
  diff. An under-reported file list touching a denylisted path (.claude/, .env)
  could bypass the gate. Now parse paths from the actual diff (parseDiffPaths)
  and gate on the UNION of diff-paths + changedFiles. Unit-tested: a diff that
  edits .claude/loop.md while changedFiles omits it now escalates. Same
  "give the checker the real artifact, not the self-report" rule — it applies
  to the deterministic gate, not just the LLM verifier.
- Ported the JSON-string args normalization (previously only in improve-skills)
  to critic-panel, gap-analysis, e2e-design, scan-docs, loop-iteration, rewiring
  every args.X read to the normalized `a` so the args.models tiering override
  no longer silently no-ops when args arrive as a JSON string.
- workflow-dsl lens now also checks model-tiering + budget-guard fail-safety;
  consistency-dry lens and /dream lint now check docs/workflow-graphs.md for
  drift vs the scripts; added docs/workflow-graphs.md to DEFAULT_TARGETS (the
  drift check was documented but not wired).
- improve-skills.md now documents budgetStop and instructs disclosing a
  skipped-verification run (needsDesign items are UNVERIFIED, not design-pending).

All six workflow scripts validated; the denylist union fix unit-tested.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014xUCrJdREvUMjoVzSX5T8G
---
 .claude/commands/dream.md           |  5 ++++
 .claude/commands/improve-skills.md  |  8 +++++-
 .claude/memory/log.md               | 24 ++++++++++++++++++
 .claude/workflows/critic-panel.js   | 21 ++++++++++++----
 .claude/workflows/e2e-design.js     | 14 +++++++++--
 .claude/workflows/gap-analysis.js   | 16 +++++++++---
 .claude/workflows/improve-skills.js |  5 ++--
 .claude/workflows/loop-iteration.js | 38 ++++++++++++++++++++++++-----
 .claude/workflows/scan-docs.js      | 16 +++++++++---
 9 files changed, 125 insertions(+), 22 deletions(-)

diff --git a/.claude/commands/dream.md b/.claude/commands/dream.md
index 97b99ce..b81b1cf 100644
--- a/.claude/commands/dream.md
+++ b/.claude/commands/dream.md
@@ -65,6 +65,11 @@ $ARGUMENTS
    `.claude/commands/*.md` has a `description` and a row in the index; **flag**
    (report, don't auto-remove) any index rows for commands that no longer exist and
    any near-duplicate commands.
+2b. Health-check the **workflow diagrams** (`docs/workflow-graphs.md`) against
+   `.claude/workflows/*.js`: for each workflow, confirm the diagram's node labels,
+   model tier (🟢/🔵/⚪/▫️), and edges (parallel/pipeline/gates) match the script's
+   `CRITICS`/`LENSES`/`DIMENSIONS`/`CATEGORIES` arrays and `FANOUT_MODEL`/`JUDGE_MODEL`
+   usage; **report (don't auto-fix)** any diagram that has drifted from its script.
 3. Auto-fix only additive/clerical issues (broken links, a missing index row, a
    missing `description`). Everything else — removing a dead index row, deleting a
    command, merging duplicates, or removing/rewriting a fact — is **reported only**
diff --git a/.claude/commands/improve-skills.md b/.claude/commands/improve-skills.md
index c919438..39e3d14 100644
--- a/.claude/commands/improve-skills.md
+++ b/.claude/commands/improve-skills.md
@@ -21,12 +21,18 @@ $ARGUMENTS
    instruction-following risk, safety/gating, consistency/DRY, workflow DSL, routing)
    runs in its own context, in parallel; a *separate* skeptic adversarially verifies
    every blocker/important finding. Invoking Workflow here is expected.
-2. The workflow returns `{ scope, targets, counts, applyReady, needsDesign, niceToHaves, refuted }`.
+2. The workflow returns `{ scope, targets, budgetStop, counts, applyReady, needsDesign, niceToHaves, refuted }`.
    Render `applyReady` and `needsDesign` as a matrix and list `niceToHaves` separately:
 
    | Lens | Severity | File | Finding | Evidence | Proposed edit | Edit verified safe? |
    |---|---|---|---|---|---|---|
 
+   **If `budgetStop` is true**, tell the user up front that the adversarial verify phase
+   was skipped due to the token-budget floor: every `needsDesign` item in this run is
+   **UNVERIFIED** (not merely design-pending — neither its reality nor its edit-safety
+   was checked), `applyReady` is empty by design, and a follow-up `/improve-skills` pass
+   should re-run verification before anything is applied.
+
 3. **Confirmation gate (hard rule).** This command may propose edits to core prompts
    but must **never apply them without explicit confirmation** — the same rule
    `/dream` follows for creating/rewriting skills. Present the proposed edits and ask
diff --git a/.claude/memory/log.md b/.claude/memory/log.md
index cae839f..6a0e280 100644
--- a/.claude/memory/log.md
+++ b/.claude/memory/log.md
@@ -171,3 +171,27 @@ change), `rotate` (archived old entries). Format: `## [YYYY-MM-DD] <op> | <summa
 - Not adopted (out of scope): the post's quant/hedge-fund framing and the standalone
   Slate runtime — we already have the graph runtime; the trading claims overstate what
   orchestration solves (data quality, costs, overfitting remain the hard part).
+
+## [2026-06-29] ingest | Gate on the tiering change caught a denylist bypass (blocker)
+- Ran `/improve-skills` (21 agents, on the NEW tiering — Sonnet fan-out + Opus verify)
+  over the graph-engineering diff: 1 blocker + 7 important confirmed, 7 refuted. Fixes:
+  - **Blocker — `loop-iteration` denylist bypass:** the "hard, deterministic" gate
+    checked the implementer's SELF-REPORTED `changedFiles`, but the caller applies the
+    real `diff`. An under-reported file list touching a denylisted path (`.claude/`,
+    `.env`) would slip past. Fixed by parsing paths from the actual diff
+    (`parseDiffPaths`) and gating on the UNION of diff-paths + changedFiles. Unit-tested:
+    the attack (changedFiles omits `.claude/loop.md` while the diff edits it) now
+    escalates. This is the same "give the checker the real artifact, not the self-report"
+    lesson — it applies to the deterministic gate too, not just the LLM verifier.
+  - **args normalization** ported to the other 5 workflows (the JSON-string parse lived
+    only in `improve-skills`), rewiring every `args.X` read to the normalized `a` so the
+    `args.models` tiering override doesn't silently no-op.
+  - **`workflow-dsl` lens** now also checks model-tiering + budget-guard fail-safety;
+    **`consistency-dry` lens** + `/dream lint` now check `docs/workflow-graphs.md` for
+    drift vs the scripts; `docs/workflow-graphs.md` added to `improve-skills`
+    DEFAULT_TARGETS (the drift check was promised but not wired).
+  - **`improve-skills.md`** now documents `budgetStop` and instructs disclosing a
+    skipped-verification run (needsDesign items are UNVERIFIED, not design-pending).
+- Lesson: a deterministic gate is only as trustworthy as the input it reads — derive the
+  checked paths from the artifact that is actually applied (the diff), never from a
+  parallel self-report. The maker/checker "real artifact" rule extends to code gates.
diff --git a/.claude/workflows/critic-panel.js b/.claude/workflows/critic-panel.js
index 00f337a..24e2a57 100644
--- a/.claude/workflows/critic-panel.js
+++ b/.claude/workflows/critic-panel.js
@@ -8,21 +8,32 @@ export const meta = {
   ],
 }
 
+// Normalize args: the Workflow tool may hand args as an object, a plain string, or a
+// JSON-encoded string. Parse the JSON-string case so structured fields (models,
+// priorEvidence, uiInScope) resolve instead of silently no-opping.
+let a = args
+if (typeof a === 'string') {
+  const s = a.trim()
+  if (s.startsWith('{') || s.startsWith('[')) {
+    try { a = JSON.parse(s) } catch { /* keep the string as a plain focus note */ }
+  }
+}
+
 // What to review (focus) + how deep, passed via the Workflow `args`.
 const focus =
-  (args && (args.focus || (typeof args === 'string' ? args : null))) ||
+  (a && (a.focus || (typeof a === 'string' ? a : null))) ||
   'the current app/flow and the changed files'
 // Optional: evidence the Lead already gathered (test/lint/build output, browser
 // findings, git diff). Critics verify against it instead of re-deriving from scratch.
-const priorEvidence = (args && args.priorEvidence) || null
+const priorEvidence = (a && a.priorEvidence) || null
 // Optional: skip the 5 UI/browser critics for non-UI (backend/docs/config) changes.
 // Defaults to including them (safe). Callers pass `uiInScope: false` to save fan-out.
-const uiInScope = !(args && args.uiInScope === false)
+const uiInScope = !(a && a.uiInScope === false)
 
 // Model tiering (graph-engineering): high-volume fan-out on the fast tier, the
 // high-stakes adversarial gate on the strong tier. Override via args.models.
-const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
-const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
+const FANOUT_MODEL = (a && a.models && a.models.fanout) || 'sonnet'
+const JUDGE_MODEL = (a && a.models && a.models.judge) || 'opus'
 
 // The standard critic roster. `ui: true` marks browser-facing critics.
 const ALL_CRITICS = [
diff --git a/.claude/workflows/e2e-design.js b/.claude/workflows/e2e-design.js
index 5087459..bb75893 100644
--- a/.claude/workflows/e2e-design.js
+++ b/.claude/workflows/e2e-design.js
@@ -5,12 +5,22 @@ export const meta = {
   phases: [{ title: 'Design', detail: 'one agent per path category enumerates test cases in parallel' }],
 }
 
-const flow = (args && (args.flow || (typeof args === 'string' ? args : null))) || 'the target flow'
+// Normalize args: object, plain string, or JSON-encoded string. Parse the JSON case
+// so structured fields (models) resolve instead of silently no-opping.
+let a = args
+if (typeof a === 'string') {
+  const s = a.trim()
+  if (s.startsWith('{') || s.startsWith('[')) {
+    try { a = JSON.parse(s) } catch { /* keep the string as a plain flow note */ }
+  }
+}
+
+const flow = (a && (a.flow || (typeof a === 'string' ? a : null))) || 'the target flow'
 
 // Model tiering (graph-engineering): this workflow is pure fan-out enumeration
 // (the dedup is plain code, not an agent), so every node runs on the fast tier.
 // Override via args.models.fanout.
-const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
+const FANOUT_MODEL = (a && a.models && a.models.fanout) || 'sonnet'
 
 const CATEGORIES = [
   { key: 'happy', prompt: 'the happy path(s) a user follows when everything works' },
diff --git a/.claude/workflows/gap-analysis.js b/.claude/workflows/gap-analysis.js
index afbf3e2..c4b5018 100644
--- a/.claude/workflows/gap-analysis.js
+++ b/.claude/workflows/gap-analysis.js
@@ -8,12 +8,22 @@ export const meta = {
   ],
 }
 
-const scope = (args && (args.scope || (typeof args === 'string' ? args : null))) || 'the whole project'
+// Normalize args: object, plain string, or JSON-encoded string. Parse the JSON case
+// so structured fields (models) resolve instead of silently no-opping.
+let a = args
+if (typeof a === 'string') {
+  const s = a.trim()
+  if (s.startsWith('{') || s.startsWith('[')) {
+    try { a = JSON.parse(s) } catch { /* keep the string as a plain scope note */ }
+  }
+}
+
+const scope = (a && (a.scope || (typeof a === 'string' ? a : null))) || 'the whole project'
 
 // Model tiering (graph-engineering): per-dimension gap discovery on the fast tier,
 // the synthesis/prioritization judgment on the strong tier. Override via args.models.
-const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
-const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
+const FANOUT_MODEL = (a && a.models && a.models.fanout) || 'sonnet'
+const JUDGE_MODEL = (a && a.models && a.models.judge) || 'opus'
 
 const DIMENSIONS = [
   { key: 'docs-not-impl', prompt: 'Requirements in docs (docs/prd.md, docs/stories/*, README) with NO implementation evidence in the codebase.' },
diff --git a/.claude/workflows/improve-skills.js b/.claude/workflows/improve-skills.js
index bbdcafc..6b5d648 100644
--- a/.claude/workflows/improve-skills.js
+++ b/.claude/workflows/improve-skills.js
@@ -27,6 +27,7 @@ const scope =
 const DEFAULT_TARGETS = [
   '.claude/commands/*.md',
   '.claude/workflows/*.js',
+  'docs/workflow-graphs.md', // the Mermaid graphs — reviewed for drift vs the scripts
   'CLAUDE.md',
   '.claude/skills-index.md',
   '.claude/memory/index.md',
@@ -47,8 +48,8 @@ const LENSES = [
   { key: 'clarity', code: false, label: 'Clarity & Actionability Critic', lens: 'Is every instruction unambiguous and written as an imperative the agent will actually follow? Flag vague/hedged steps, undefined terms, missing acceptance criteria, and steps with no checkable outcome.' },
   { key: 'instruction-following', code: false, label: 'Instruction-Following Risk Critic', lens: 'Where would an LLM plausibly misread, skip, or over-comply? Conflicting directives, a critical rule buried mid-paragraph, ambiguous ordering, "should" where "must" is meant, or prompts so long the key constraint is lost.' },
   { key: 'safety-gating', code: false, label: 'Safety & Gating Critic', lens: 'Are destructive or auto-applied actions properly confirmation-gated and denylist-aware? Flag any prompt that could auto-create/rewrite/delete files, commit/push, or self-modify core prompts WITHOUT an explicit confirmation gate. This is the most important lens for a self-improving system.' },
-  { key: 'consistency-dry', code: false, label: 'Consistency & DRY Critic', lens: 'Contradictions or duplication ACROSS CLAUDE.md, the commands, skills-index.md, and the memory wiki. Flag facts/lists stated in multiple places that can drift apart (single-source-of-truth risk), and any command whose body contradicts CLAUDE.md or the wiki.' },
-  { key: 'workflow-dsl', code: true, label: 'Workflow DSL & Schema Critic', lens: 'For .claude/workflows/*.js only: fail-safe verifiers (a null/failed verdict must NOT confirm), unique agent labels that include the map index, guarded first pipeline stage before deref, fail-closed defaults, maker/checker given the REAL artifact (not a self-report), valid JSON Schemas, and a meta block that is a pure literal. Flag deviations.' },
+  { key: 'consistency-dry', code: false, label: 'Consistency & DRY Critic', lens: 'Contradictions or duplication ACROSS CLAUDE.md, the commands, skills-index.md, and the memory wiki. Flag facts/lists stated in multiple places that can drift apart (single-source-of-truth risk), and any command whose body contradicts CLAUDE.md or the wiki. Also: for each diagram in docs/workflow-graphs.md, verify its node list, model-tier annotations (🟢/🔵/⚪/▫️), and edges match the actual agent() calls (CRITICS/LENSES/DIMENSIONS/CATEGORIES arrays and FANOUT_MODEL/JUDGE_MODEL usage) in the corresponding .claude/workflows/*.js — flag any diagram that has drifted from its script.' },
+  { key: 'workflow-dsl', code: true, label: 'Workflow DSL & Schema Critic', lens: 'For .claude/workflows/*.js only: fail-safe verifiers (a null/failed verdict must NOT confirm), unique agent labels that include the map index, guarded first pipeline stage before deref, fail-closed defaults, maker/checker given the REAL artifact (not a self-report), valid JSON Schemas, and a meta block that is a pure literal. Also check the graph-engineering conventions: correct FANOUT_MODEL/JUDGE_MODEL tiering (fan-out nodes on the fast tier; gates/judgment/synthesis on the strong tier; any code-writing node explicitly exempted rather than silently downgraded); and, where a budget guard exists, that it fails safe (skips or escalates on a floor breach and never auto-approves unverified work). Flag deviations.' },
   { key: 'routing', code: false, label: 'Routing & Discoverability Critic', lens: 'Does each command have a one-line description (for model routing) and a matching skills-index row? Flag near-duplicate commands, dead index rows, missing argument-hints, and heavy/code-changing commands that lack disable-model-invocation where they should be manual-only.' },
 ]
 
diff --git a/.claude/workflows/loop-iteration.js b/.claude/workflows/loop-iteration.js
index 596dd9d..07808a4 100644
--- a/.claude/workflows/loop-iteration.js
+++ b/.claude/workflows/loop-iteration.js
@@ -8,16 +8,26 @@ export const meta = {
   ],
 }
 
+// Normalize args: object, plain string, or JSON-encoded string. Parse the JSON case
+// so structured fields (items, level, models) resolve instead of silently no-opping.
+let a = args
+if (typeof a === 'string') {
+  const s = a.trim()
+  if (s.startsWith('{') || s.startsWith('[')) {
+    try { a = JSON.parse(s) } catch { /* no structured fields available */ }
+  }
+}
+
 // args.items: [{ id, description }] — already triaged and denylist-cleared by the loop.
-const items = (args && args.items) || []
-const level = (args && args.level) || 'L2'
+const items = (a && a.items) || []
+const level = (a && a.level) || 'L2'
 
 // Model tiering (graph-engineering): the adversarial verifier (the safety gate) runs
 // on the strong tier. The IMPLEMENTER intentionally has NO override — it writes real
 // code in a worktree, so it inherits the session model rather than being silently
 // downgraded (correctness of the change matters more than fan-out throughput here).
 // Override the verifier tier via args.models.judge.
-const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
+const JUDGE_MODEL = (a && a.models && a.models.judge) || 'opus'
 if (!items.length) {
   return { error: 'No work items. Triage + denylist-check first, then pass args.items = [{id, description}].', applied: [], rejected: [] }
 }
@@ -66,6 +76,19 @@ const DENYLIST = [
 ]
 const hitsDenylist = (files) => (files || []).some((f) => DENYLIST.some((re) => re.test(f)))
 
+// Parse the file paths out of the ACTUAL unified diff (the artifact the caller
+// applies). The gate must not trust the implementer's self-reported `changedFiles`
+// alone: a maker that under-reports its changed files while its diff still touches a
+// denylisted path would otherwise slip past. We check the UNION of both.
+function parseDiffPaths(diff) {
+  const paths = []
+  const s = String(diff || '')
+  for (const m of s.matchAll(/^diff --git a\/(.+?) b\/(.+)$/gm)) { paths.push(m[1], m[2]) }
+  for (const m of s.matchAll(/^\+\+\+ b\/(.+)$/gm)) { if (m[1] !== '/dev/null') paths.push(m[1]) }
+  for (const m of s.matchAll(/^--- a\/(.+)$/gm)) { if (m[1] !== '/dev/null') paths.push(m[1]) }
+  return paths
+}
+
 phase('Implement')
 // Pipeline: each item flows implement → verify independently.
 const outcomes = await pipeline(
@@ -77,12 +100,15 @@ const outcomes = await pipeline(
     ),
   (impl, item, i) => {
     const im = impl || {}
-    // Hard denylist gate — independent of the verifier.
-    if (hitsDenylist(im.changedFiles)) {
+    // Hard denylist gate — independent of the verifier. Check the UNION of the
+    // self-reported changedFiles AND the paths parsed from the real diff (the artifact
+    // the caller applies), so an under-reported file list can't bypass the gate.
+    const gatedFiles = [...(im.changedFiles || []), ...parseDiffPaths(im.diff)]
+    if (hitsDenylist(gatedFiles)) {
       return {
         id: item.id, applied: false, status: 'escalated-denylist',
         changedFiles: im.changedFiles || [], diff: im.diff || '', checksPassed: !!im.checksPassed,
-        verdict: 'changed files match the denylist — forced escalation regardless of verdict',
+        verdict: 'changed files or diff paths match the denylist — forced escalation regardless of verdict',
       }
     }
     return agent(
diff --git a/.claude/workflows/scan-docs.js b/.claude/workflows/scan-docs.js
index 8eeecea..c9b09dc 100644
--- a/.claude/workflows/scan-docs.js
+++ b/.claude/workflows/scan-docs.js
@@ -8,9 +8,19 @@ export const meta = {
   ],
 }
 
+// Normalize args: object, plain string, or JSON-encoded string. Parse the JSON case
+// so structured fields (stories, models) resolve instead of silently no-opping.
+let a = args
+if (typeof a === 'string') {
+  const s = a.trim()
+  if (s.startsWith('{') || s.startsWith('[')) {
+    try { a = JSON.parse(s) } catch { /* no structured fields available */ }
+  }
+}
+
 // args.stories: [{ id, title, source, acceptanceCriteria? }]
 // The caller extracts the story list from the PRD/docs first, then runs this.
-const stories = (args && args.stories) || []
+const stories = (a && a.stories) || []
 if (!stories.length) {
   return { error: 'No stories provided. Extract stories from docs first, then pass args.stories = [{id,title,source}].', records: [] }
 }
@@ -25,8 +35,8 @@ const REPO_STATUSES = [
 
 // Model tiering (graph-engineering): per-story evidence gathering on the fast tier,
 // the strict status verifier (the gate) on the strong tier. Override via args.models.
-const FANOUT_MODEL = (args && args.models && args.models.fanout) || 'sonnet'
-const JUDGE_MODEL = (args && args.models && args.models.judge) || 'opus'
+const FANOUT_MODEL = (a && a.models && a.models.fanout) || 'sonnet'
+const JUDGE_MODEL = (a && a.models && a.models.judge) || 'opus'
 
 const EVIDENCE_SCHEMA = {
   type: 'object',

From e163307906639862cd5c0adc25f0c6d61483b05d Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Thu, 23 Jul 2026 20:47:07 +0000
Subject: [PATCH 3/3] Apply nice-to-haves from the tiering gate run

Low-severity polish confirmed (recorded) by the /improve-skills gate:

- Fan-out agent labels now include the map index for consistency with the
  project's own documented DSL rule ("always include the map index"): critic:,
  meta:, gap:, e2e: labels switch from ${key} to ${key}-${i} in critic-panel,
  improve-skills, gap-analysis, e2e-design. Harmless today (keys are unique),
  but aligns every fan-out with the Verify-phase pattern in the same files.
- docs/workflow-graphs.md: note under the critic-panel diagram that
  uiInScope:false drops the 5 UI-facing critics (the 10-node graph is the
  default/full case).
- skills-index.md: document the args.models = { fanout, judge } tiering
  override once, in the shared workflows note, so it's discoverable without
  reading the .js source.

Syntax validated for all six scripts.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014xUCrJdREvUMjoVzSX5T8G
---
 .claude/skills-index.md             | 2 ++
 .claude/workflows/critic-panel.js   | 4 ++--
 .claude/workflows/e2e-design.js     | 4 ++--
 .claude/workflows/gap-analysis.js   | 4 ++--
 .claude/workflows/improve-skills.js | 4 ++--
 docs/workflow-graphs.md             | 4 ++++
 6 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/.claude/skills-index.md b/.claude/skills-index.md
index 3d2559b..339c755 100644
--- a/.claude/skills-index.md
+++ b/.claude/skills-index.md
@@ -35,6 +35,8 @@ catalog so both you and Claude can route to the right command.
 > `scan-docs` (`/scan-project-docs`), `gap-analysis` (`/story-gap-analysis`),
 > `e2e-design` (`/multi-agent-e2e`), `loop-iteration` (`/loop` L2/L3),
 > `improve-skills` (`/improve-skills` — meta-critic over our own prompts/workflows).
+> Pass `args.models = { fanout, judge }` when invoking any of these to override the
+> default `sonnet`/`opus` model tiering (e.g. for cost control on a large fan-out).
 
 | Command | When to use | Args |
 |---|---|---|
diff --git a/.claude/workflows/critic-panel.js b/.claude/workflows/critic-panel.js
index 24e2a57..3e95bbb 100644
--- a/.claude/workflows/critic-panel.js
+++ b/.claude/workflows/critic-panel.js
@@ -86,10 +86,10 @@ const VERDICT_SCHEMA = {
 // the full set before deduping and verifying).
 phase('Review')
 const reviews = await parallel(
-  CRITICS.map((c) => () =>
+  CRITICS.map((c, i) => () =>
     agent(
       `You are the **${c.label}** for this project. Read CLAUDE.md and the relevant memory topic page(s), inspect ${focus} (read the code; if a localhost flow is in scope, drive it via Playwright/Chrome MCP and check console + network), then review strictly through your lens:\n${c.lens}\n${priorEvidence ? `\nEvidence already gathered by the Lead — verify against it instead of re-deriving from scratch:\n${typeof priorEvidence === 'string' ? priorEvidence : JSON.stringify(priorEvidence)}\n` : ''}\nReturn only findings you can back with concrete evidence. An empty list is a valid, honest answer.`,
-      { label: `critic:${c.key}`, phase: 'Review', schema: FINDINGS_SCHEMA, model: FANOUT_MODEL },
+      { label: `critic:${c.key}-${i}`, phase: 'Review', schema: FINDINGS_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ({ critic: c.key, label: c.label, findings: (r && r.findings) || [] })),
   ),
 )
diff --git a/.claude/workflows/e2e-design.js b/.claude/workflows/e2e-design.js
index bb75893..a13b4f4 100644
--- a/.claude/workflows/e2e-design.js
+++ b/.claude/workflows/e2e-design.js
@@ -49,10 +49,10 @@ const CASES_SCHEMA = {
 
 phase('Design')
 const results = await parallel(
-  CATEGORIES.map((c) => () =>
+  CATEGORIES.map((c, i) => () =>
     agent(
       `Enumerate concrete Playwright E2E test cases for "${flow}" covering ${c.prompt}. Inspect the routes/components and existing specs first. Each case needs: title, steps, and the visible expected outcome to assert.`,
-      { label: `e2e:${c.key}`, phase: 'Design', schema: CASES_SCHEMA, model: FANOUT_MODEL },
+      { label: `e2e:${c.key}-${i}`, phase: 'Design', schema: CASES_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ((r && r.cases) || []).map((x) => ({ ...x, category: c.key }))),
   ),
 )
diff --git a/.claude/workflows/gap-analysis.js b/.claude/workflows/gap-analysis.js
index c4b5018..63c4191 100644
--- a/.claude/workflows/gap-analysis.js
+++ b/.claude/workflows/gap-analysis.js
@@ -67,10 +67,10 @@ const ORDER_SCHEMA = {
 
 phase('Dimensions')
 const results = await parallel(
-  DIMENSIONS.map((d) => () =>
+  DIMENSIONS.map((d, i) => () =>
     agent(
       `Analyze the gap for "${scope}" along this dimension by reading docs/, src/, e2e/, and the story map:\n${d.prompt}\nReturn only gaps backed by concrete references; an empty list is a valid, honest answer.`,
-      { label: `gap:${d.key}`, phase: 'Dimensions', schema: GAP_SCHEMA, model: FANOUT_MODEL },
+      { label: `gap:${d.key}-${i}`, phase: 'Dimensions', schema: GAP_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ({ dimension: d.key, gaps: (r && r.gaps) || [] })),
   ),
 )
diff --git a/.claude/workflows/improve-skills.js b/.claude/workflows/improve-skills.js
index 6b5d648..e0c603a 100644
--- a/.claude/workflows/improve-skills.js
+++ b/.claude/workflows/improve-skills.js
@@ -91,10 +91,10 @@ const VERDICT_SCHEMA = {
 // (a barrier: we want the full set before deduping and verifying).
 phase('Review')
 const reviews = await parallel(
-  LENSES.map((c) => () =>
+  LENSES.map((c, i) => () =>
     agent(
       `You are the **${c.label}** auditing this project's OWN Claude Code configuration — its custom prompts, skills, and multi-agent orchestration. You are NOT reviewing product/application code; you are reviewing the instructions that drive the agent.\n\nRead these files (the prompt surface): ${targetList}\n${c.code ? 'Focus on the workflow orchestration scripts (.claude/workflows/*.js).' : 'Focus on the prose prompts (commands, CLAUDE.md, skills-index, loop files).'}\n\nReview strictly through your lens:\n${c.lens}\n\nScope: ${scope}\n\nFor each issue return concrete evidence (a quote or file:line) AND a specific proposedEdit (what to change to what). Propose improvements to the prompts/workflows themselves — do not propose product-code changes. An empty list is a valid, honest answer.`,
-      { label: `meta:${c.key}`, phase: 'Review', schema: FINDINGS_SCHEMA, model: FANOUT_MODEL },
+      { label: `meta:${c.key}-${i}`, phase: 'Review', schema: FINDINGS_SCHEMA, model: FANOUT_MODEL },
     ).then((r) => ({ critic: c.key, label: c.label, findings: (r && r.findings) || [] })),
   ),
 )
diff --git a/docs/workflow-graphs.md b/docs/workflow-graphs.md
index db4fc16..ae677e4 100644
--- a/docs/workflow-graphs.md
+++ b/docs/workflow-graphs.md
@@ -44,6 +44,10 @@ flowchart LR
   V --> out([confirmed matrix + niceToHaves + refuted])
 ```
 
+> `uiInScope: false` drops the 5 UI-facing critics (First-Time User, UX Flow, Designer,
+> Artistic Direction, Accessibility) from the Review fan-out for non-UI (backend / docs /
+> config) changes — the 10-node graph above is the default/full case.
+
 ## `improve-skills` — meta-critic over our own prompts (`/improve-skills`)
 
 ```mermaid