@mmerterden/multi-agent-pipeline 8.6.2 → 10.0.6

package/pipeline/schemas/figma-project-config.schema.json

@@ -53,7 +53,7 @@
     "github": {
       "type": "object",
       "additionalProperties": false,
-      "description": "GitHub configuration — repo handle + Projects V2 board metadata. Required for automatic issue + Jira triad.",
+      "description": "GitHub configuration  -  repo handle + Projects V2 board metadata. Required for automatic issue + Jira triad.",
       "properties": {
         "repo": {
           "type": "string",
@@ -170,7 +170,7 @@
         },
         "artifactsRepo": {
           "type": ["string", "null"],
-          "description": "Relative path (inside packagesContainer or repo root) to the implementation-artifacts submodule/folder — holds IMPLEMENTATION_HISTORY.md and Figma/Confluence per-component trees. Null = feature disabled.",
+          "description": "Relative path (inside packagesContainer or repo root) to the implementation-artifacts submodule/folder  -  holds IMPLEMENTATION_HISTORY.md and Figma/Confluence per-component trees. Null = feature disabled.",
           "examples": ["Packages/my-ios-implementation-artifacts", "artifacts"]
         },
         "packagesContainer": {
@@ -334,7 +334,7 @@
         },
         "url": {
           "type": ["string", "null"],
-          "description": "Used by `separate-repo` mode — clone URL."
+          "description": "Used by `separate-repo` mode  -  clone URL."
         },
         "branch": {
           "type": ["string", "null"],

package/pipeline/schemas/learnings-ledger.schema.json

@@ -0,0 +1,39 @@
+{
+  "$schema": "https://json-schema.org/draft/2020-12/schema",
+  "$id": "https://github.com/mmerterden/multi-agent-pipeline/pipeline/schemas/learnings-ledger.schema.json",
+  "version": "1.0.0",
+  "title": "Multi-Agent Pipeline  -  persistent learnings ledger row",
+  "description": "One durable knowledge entry, append-only, one file per repo at $HOME/.claude/memory/multi-agent/<repo-slug>/learnings-ledger.jsonl. Distinct from triage-corpus (per-task findings): the ledger holds knowledge replayed into EVERY future run of the same repo - architectural facts, conventions, and explicitly rejected review preferences - so agents stop re-discovering structure and reviewers stop re-flagging rejected feedback.",
+  "type": "object",
+  "additionalProperties": false,
+  "required": ["v", "ts", "repo_slug", "kind", "scope", "statement"],
+  "properties": {
+    "v": { "type": "string", "description": "Row schema version." },
+    "ts": { "type": "string", "format": "date-time", "description": "When the entry was recorded." },
+    "repo_slug": { "type": "string", "minLength": 1, "description": "Repo the knowledge belongs to." },
+    "kind": {
+      "type": "string",
+      "enum": ["fact", "convention", "rejected-preference"],
+      "description": "fact = architectural truth ('auth tokens go through KeychainStore'); convention = a project rule; rejected-preference = a review suggestion the team explicitly rejected, so reviewers should not re-raise it."
+    },
+    "scope": {
+      "type": "string",
+      "minLength": 1,
+      "description": "Path glob the entry applies to (e.g. 'Auth/*'), or 'global'."
+    },
+    "statement": {
+      "type": "string",
+      "minLength": 1,
+      "description": "The knowledge itself, in one line."
+    },
+    "source_task": {
+      "type": ["string", "null"],
+      "description": "Task id that produced this entry, or null when added manually."
+    },
+    "confidence": {
+      "type": "string",
+      "enum": ["low", "medium", "high"],
+      "description": "How strongly to weight the entry when replaying it."
+    }
+  }
+}

package/pipeline/schemas/migrations/README.md

@@ -49,7 +49,7 @@ node pipeline/scripts/migrate-state.mjs --type=prefs \
 
 Migrations are **always applied in order**. If a state is at 2.0.0 and the
 current schema is 4.0.0, the runner applies 2→3 then 3→4 sequentially.
-Partial chains are not allowed — all versions in between must have
+Partial chains are not allowed  -  all versions in between must have
 migrations committed.
 
 ## Current State
@@ -63,7 +63,7 @@ target schema's `const`, done.
 
 ## Guidelines for Authors
 
-- Keep each migration pure — no fs / network / random. Same input must produce
+- Keep each migration pure  -  no fs / network / random. Same input must produce
   same output so replaying is idempotent.
 - Always write the `schemaVersion` in the returned object.
 - Never delete user data silently. If a field is removed, either copy it

package/pipeline/schemas/migrations/figma-config-1.0.0-to-2.0.0.mjs

@@ -19,7 +19,7 @@ export function migrate(prev) {
 
   if (prev.figma) next.figma = { ...prev.figma };
 
-  // github — new top-level; may already be partially present if the user hand-edited.
+  // github  -  new top-level; may already be partially present if the user hand-edited.
   if (prev.github) {
     next.github = { ...prev.github };
   }
@@ -40,7 +40,7 @@ export function migrate(prev) {
   // Retain `board` for backward-compat (labels-only / none users unaffected).
   if (prev.board) next.board = { ...prev.board };
 
-  // jira, confluence — pass through if present.
+  // jira, confluence  -  pass through if present.
   if (prev.jira) next.jira = { ...prev.jira };
   if (prev.confluence) next.confluence = { ...prev.confluence };
 
@@ -74,15 +74,15 @@ export function migrate(prev) {
     next.repos = { ...(next.repos ?? {}), ...prev.repos };
   }
 
-  // build — new in v2; pass through if hand-added.
+  // build  -  new in v2; pass through if hand-added.
   if (prev.build) next.build = { ...prev.build };
 
-  // artifactBasePath, tokens, teams — pass through.
+  // artifactBasePath, tokens, teams  -  pass through.
   if (prev.artifactBasePath) next.artifactBasePath = prev.artifactBasePath;
   if (prev.tokens) next.tokens = { ...prev.tokens };
   if (prev.teams) next.teams = [...prev.teams];
 
-  // registry, review, iteration, scripts, models — pass through unchanged.
+  // registry, review, iteration, scripts, models  -  pass through unchanged.
   for (const key of ["registry", "review", "iteration", "scripts", "models"]) {
     if (prev[key]) next[key] = JSON.parse(JSON.stringify(prev[key]));
   }

package/pipeline/schemas/migrations/prefs-2.0.0-to-2.1.0.mjs

@@ -1,12 +1,12 @@
 /**
- * prefs-2.0.0-to-2.1.0.mjs — preference migration (delegates to migrate-prefs.mjs)
+ * prefs-2.0.0-to-2.1.0.mjs  -  preference migration (delegates to migrate-prefs.mjs)
  *
  * This stub satisfies the generic migrate-state.mjs runner's migration-chain
  * discovery. The actual migration logic lives in scripts/migrate-prefs.mjs
  * (which is richer, handles firebase consolidation, and is idempotent).
  *
- * @param {object} data — parsed JSON content of the preferences file
- * @returns {object}    — migrated data with schemaVersion set to "2.1.0"
+ * @param {object} data  -  parsed JSON content of the preferences file
+ * @returns {object}     -  migrated data with schemaVersion set to "2.1.0"
  */
 export default function migrate(data) {
   // Delegate to the same logic as migrate-prefs.mjs inline:

package/pipeline/schemas/migrations/prefs-2.1.0-to-2.2.0.mjs

@@ -1,5 +1,5 @@
 /**
- * prefs-2.1.0-to-2.2.0.mjs — preference schema upgrade 2.1.0 → 2.2.0
+ * prefs-2.1.0-to-2.2.0.mjs  -  preference schema upgrade 2.1.0 → 2.2.0
  *
  * Formalizes the v5.7 / v5.8 additions that had been running as sub-migrations
  * inside the 2.1.0 migrator without a proper schemaVersion bump. v6.0.0 gives
@@ -20,8 +20,8 @@
  * Idempotent: safe on already-2.2.0 input. Non-destructive: preserves any
  * user-set field values.
  *
- * @param {object} data — parsed JSON content of the preferences file
- * @returns {object}    — same data with schemaVersion = "2.2.0"
+ * @param {object} data  -  parsed JSON content of the preferences file
+ * @returns {object}     -  same data with schemaVersion = "2.2.0"
  */
 export default function migrate(data) {
   const out = JSON.parse(JSON.stringify(data));
@@ -30,7 +30,7 @@ export default function migrate(data) {
 
   if (!out.global) out.global = {};
 
-  // Defensive backfill — these MUST exist at v2.2.0 even if the upstream
+  // Defensive backfill  -  these MUST exist at v2.2.0 even if the upstream
   // v2.1.0 sub-migration somehow missed them (e.g. manual state edits).
   // Never overwrite existing user-set values.
   if (!out.global.reportChannels) {

package/pipeline/schemas/migrations/prefs-2.2.0-to-2.3.0.mjs

@@ -1,10 +1,10 @@
 /**
- * prefs-2.2.0-to-2.3.0.mjs — preference schema upgrade 2.2.0 → 2.3.0
+ * prefs-2.2.0-to-2.3.0.mjs  -  preference schema upgrade 2.2.0 → 2.3.0
  *
  * v7.4.0 picker UX series adds two optional fields to the global block:
- *   - global.recentAccounts   (v7.3.1) — LRU of {id, prefix, lastUsed, count}
+ *   - global.recentAccounts   (v7.3.1)  -  LRU of {id, prefix, lastUsed, count}
  *                                        used by the autopilot account picker.
- *   - global.accounts         (v7.4.0) — per-account host overrides;
+ *   - global.accounts         (v7.4.0)  -  per-account host overrides;
  *                                        wins over global.hosts.{jira,bitbucket}.
  *
  * Both default to empty arrays. Existing v2.2.0 prefs continue to validate
@@ -14,8 +14,8 @@
  * Idempotent: safe on already-2.3.0 input. Non-destructive: preserves any
  * user-set entries.
  *
- * @param {object} data — parsed JSON content of the preferences file
- * @returns {object}    — same data with schemaVersion = "2.3.0"
+ * @param {object} data  -  parsed JSON content of the preferences file
+ * @returns {object}     -  same data with schemaVersion = "2.3.0"
  */
 export default function migrate(data) {
   const out = JSON.parse(JSON.stringify(data));

package/pipeline/schemas/migrations/state-2.0.0-to-2.1.0.mjs

@@ -1,11 +1,11 @@
 /**
- * state-2.0.0-to-2.1.0.mjs — agent-state migration
+ * state-2.0.0-to-2.1.0.mjs  -  agent-state migration
  *
  * Adds the `projects[]` array for multi-repo support.
  * Single-repo scalar fields remain as backward-compat mirrors.
  *
- * @param {object} data — parsed agent-state.json
- * @returns {object}    — migrated data with schemaVersion "2.1.0"
+ * @param {object} data  -  parsed agent-state.json
+ * @returns {object}     -  migrated data with schemaVersion "2.1.0"
  */
 export default function migrate(data) {
   const out = JSON.parse(JSON.stringify(data));

package/pipeline/schemas/plan-todos.schema.json

@@ -19,7 +19,7 @@
     "todos": {
       "type": "array",
       "minItems": 1,
-      "description": "Ordered list of steps. Phase 3 picks them in `deps`-respecting order — see plan-todos.sh next.",
+      "description": "Ordered list of steps. Phase 3 picks them in `deps`-respecting order  -  see plan-todos.sh next.",
       "items": {
         "type": "object",
         "additionalProperties": false,
@@ -28,11 +28,11 @@
           "id": {
             "type": "string",
             "pattern": "^t[0-9]+$",
-            "description": "Stable identifier (t1, t2, …). Used in dependency edges + progress lines."
+            "description": "Stable identifier (t1, t2, ...). Used in dependency edges + progress lines."
           },
           "task": {
             "type": "string",
-            "description": "Imperative one-liner — what gets done. Avoid 'investigate X' or 'consider Y' — Phase 3 needs concrete actions."
+            "description": "Imperative one-liner  -  what gets done. Avoid 'investigate X' or 'consider Y'  -  Phase 3 needs concrete actions."
           },
           "status": {
             "type": "string",
@@ -47,7 +47,7 @@
           },
           "startedAt":   { "type": "string", "format": "date-time" },
           "completedAt": { "type": "string", "format": "date-time" },
-          "notes":       { "type": "string", "description": "Free-text outcome — commit hash, test count, anything Phase 7 should echo." },
+          "notes":       { "type": "string", "description": "Free-text outcome  -  commit hash, test count, anything Phase 7 should echo." },
           "failureReason": { "type": "string" },
           "skipReason":    { "type": "string" },
           "estimatedMinutes": {

package/pipeline/schemas/planning-output.schema.json

@@ -2,15 +2,15 @@
   "$schema": "https://json-schema.org/draft/2020-12/schema",
   "$id": "https://github.com/mmerterden/multi-agent-pipeline/pipeline/schemas/planning-output.schema.json",
   "version": "1.0.0",
-  "title": "Multi-Agent Pipeline — Phase 2 planning output",
-  "description": "Contract for Phase 2 task decomposition. Consumed by Phase 3 TDD cycles — each task becomes one red→green→refactor iteration.",
+  "title": "Multi-Agent Pipeline  -  Phase 2 planning output",
+  "description": "Contract for Phase 2 task decomposition. Consumed by Phase 3 TDD cycles  -  each task becomes one red→green→refactor iteration.",
   "type": "object",
   "additionalProperties": false,
   "required": ["tasks", "approved"],
   "properties": {
     "approach": {
       "type": "string",
-      "description": "One paragraph explaining the overall strategy — why this decomposition, what's deferred, what's in scope."
+      "description": "One paragraph explaining the overall strategy  -  why this decomposition, what's deferred, what's in scope."
     },
     "tasks": {
       "type": "array",

package/pipeline/schemas/prefs.schema.json

@@ -704,19 +704,19 @@
         "review": {
           "type": "object",
           "additionalProperties": false,
-          "description": "v8.6+ — Standalone /multi-agent:review knobs. Only affects the post-pr-review.sh path (PR-mode comment posting); does not change Phase 4 reviewer dispatch inside the full pipeline.",
+          "description": "v8.6+  -  Standalone /multi-agent:review knobs. Only affects the post-pr-review.sh path (PR-mode comment posting); does not change Phase 4 reviewer dispatch inside the full pipeline.",
           "properties": {
             "dedupeInlineComments": {
               "type": "boolean",
               "default": true,
-              "description": "Bugbot/Devin Review behaviour: before posting an inline comment, scan existing PR comments for a stable fingerprint marker (sha-16 of path|line|issue). If a comment with the same marker exists, skip — preserves audit trail without flooding the PR on re-runs. Provider-agnostic: GitHub /pulls/{n}/comments + /issues/{n}/comments and Bitbucket Server /pull-requests/{id}/activities?fromType=COMMENT are both checked. Set to false to restore the pre-v8.6 behavior (every run posts fresh comments, original spec)."
+              "description": "Bugbot/Devin Review behaviour: before posting an inline comment, scan existing PR comments for a stable fingerprint marker (sha-16 of path|line|issue). If a comment with the same marker exists, skip  -  preserves audit trail without flooding the PR on re-runs. Provider-agnostic: GitHub /pulls/{n}/comments + /issues/{n}/comments and Bitbucket Server /pull-requests/{id}/activities?fromType=COMMENT are both checked. Set to false to restore the pre-v8.6 behavior (every run posts fresh comments, original spec)."
             }
           }
         },
         "shadowGit": {
           "type": "object",
           "additionalProperties": false,
-          "description": "v8.6+ — Cline-style per-tool-call shadow Git checkpoints. A separate git repo under ~/.claude/state/shadow-git/<task-id>/.git/ snapshots the worktree at each Phase 3 step boundary (per Plan Todo) and optionally after each Edit/Write/MultiEdit/Bash mutation. The shadow lives outside the project's real .git so semantic commits stay clean. Pattern source: Cline checkpoints (https://docs.cline.bot/features/checkpoints). Off by default — adds a per-step git commit on top of the existing TDD loop; flip on for risky refactors where sub-phase rollback is worth ~50ms per snapshot.",
+          "description": "v8.6+  -  Cline-style per-tool-call shadow Git checkpoints. A separate git repo under ~/.claude/state/shadow-git/<task-id>/.git/ snapshots the worktree at each Phase 3 step boundary (per Plan Todo) and optionally after each Edit/Write/MultiEdit/Bash mutation. The shadow lives outside the project's real .git so semantic commits stay clean. Pattern source: Cline checkpoints (https://docs.cline.bot/features/checkpoints). Off by default  -  adds a per-step git commit on top of the existing TDD loop; flip on for risky refactors where sub-phase rollback is worth ~50ms per snapshot.",
           "properties": {
             "enabled": {
               "type": "boolean",
@@ -741,7 +741,7 @@
         "planTodos": {
           "type": "object",
           "additionalProperties": false,
-          "description": "v8.6+ — Plan-as-live-Todo-list. Phase 2 emits agent-state.plan.todos[] conforming to pipeline/schemas/plan-todos.schema.json; Phase 3 iterates step-by-step via pipeline/lib/plan-todos.sh next/start/complete; Phase 7 renders the rollup into agent-log.md and the PR body. Pattern source: Windsurf Cascade (https://docs.windsurf.com/windsurf/cascade) Todo list inside the conversation; Cursor Plan Mode (https://cursor.com/docs/agent/planning) reviewable plan. Off by default — opt in to add status-transition writes per Phase 3 step in exchange for sub-step visibility + per-step notes.",
+          "description": "v8.6+  -  Plan-as-live-Todo-list. Phase 2 emits agent-state.plan.todos[] conforming to pipeline/schemas/plan-todos.schema.json; Phase 3 iterates step-by-step via pipeline/lib/plan-todos.sh next/start/complete; Phase 7 renders the rollup into agent-log.md and the PR body. Pattern source: Windsurf Cascade (https://docs.windsurf.com/windsurf/cascade) Todo list inside the conversation; Cursor Plan Mode (https://cursor.com/docs/agent/planning) reviewable plan. Off by default  -  opt in to add status-transition writes per Phase 3 step in exchange for sub-step visibility + per-step notes.",
           "properties": {
             "enabled": {
               "type": "boolean",
@@ -753,7 +753,7 @@
         "clarifyAmbiguous": {
           "type": "object",
           "additionalProperties": false,
-          "description": "v8.6+ — Phase 0 Step 9 clarifying-question loop. Before Phase 1 starts, a cheap Haiku classifier scores task ambiguity (0-10) and emits up to N questions if score < threshold. Pattern source: Devin Knowledge / Ask Devin (https://docs.devin.ai/work-with-devin/devin-review) and Cursor Plan Mode clarifying questions (https://cursor.com/docs/agent/planning). Cost: ~$0.0025 per Haiku call. Off by default — flip on for teams burned by ambiguity-driven rework or when working on cross-team issues where the spec lives in someone else's head.",
+          "description": "v8.6+  -  Phase 0 Step 9 clarifying-question loop. Before Phase 1 starts, a cheap Haiku classifier scores task ambiguity (0-10) and emits up to N questions if score < threshold. Pattern source: Devin Knowledge / Ask Devin (https://docs.devin.ai/work-with-devin/devin-review) and Cursor Plan Mode clarifying questions (https://cursor.com/docs/agent/planning). Cost: ~$0.0025 per Haiku call. Off by default  -  flip on for teams burned by ambiguity-driven rework or when working on cross-team issues where the spec lives in someone else's head.",
           "properties": {
             "enabled": {
               "type": "boolean",
@@ -788,10 +788,59 @@
             }
           }
         },
+        "intentGuard": {
+          "type": "object",
+          "additionalProperties": false,
+          "description": "v9.3+  -  Phase 0 conceptual-vs-edit guard for free-text input. A deterministic local classifier (pipeline/lib/classify-intent.sh, no model call) detects when the input is a question rather than a task and answers it in place instead of creating a branch/worktree. Targets the most-cited daily annoyance with coding agents (the agent edits code when asked a conceptual question). On by default because the check is free and only fires on free-text (Jira/issue inputs are always explicit tasks).",
+          "properties": {
+            "enabled": {
+              "type": "boolean",
+              "default": true,
+              "description": "When true, free-text classified as a question is answered in chat and the dev chain is not started. Set false to always treat free-text as a task."
+            }
+          }
+        },
+        "costBudget": {
+          "type": "object",
+          "additionalProperties": false,
+          "description": "v9.2+  -  Proactive per-task cost ceiling. Complements the end-of-run cost summary: pipeline/scripts/cost-budget-check.mjs reads the phase-tracker token accumulators after each phase, prices them via cost-table.json, and warns or halts before spend runs away. Top user-cited reason to abandon an AI coding tool is opaque/spiralling cost; this surfaces it live. Disabled by default.",
+          "properties": {
+            "enabled": {
+              "type": "boolean",
+              "default": false,
+              "description": "Master switch. When off, cost-budget-check.mjs exits 0 silently and the pipeline behaves exactly as before."
+            },
+            "maxUsd": {
+              "type": "number",
+              "minimum": 0,
+              "default": 5.0,
+              "description": "Hard ceiling on estimated USD spend for a single task across all phases. Exceeding it returns exit 11."
+            },
+            "warnPct": {
+              "type": "integer",
+              "minimum": 1,
+              "maximum": 99,
+              "default": 80,
+              "description": "Emit a warning (exit 10) once estimated spend reaches this percentage of maxUsd, so the user can intervene before the ceiling."
+            },
+            "onExceed": {
+              "type": "string",
+              "enum": ["warn", "halt"],
+              "default": "warn",
+              "description": "warn = surface the overage and continue (autopilot keeps going). halt = pause the run and require multi-agent:resume once the user accepts the cost. Interactive runs always surface the line regardless."
+            },
+            "pricingModel": {
+              "type": "string",
+              "enum": ["opus", "sonnet", "haiku"],
+              "default": "opus",
+              "description": "Which cost-table.json rate to price accumulated tokens at. Defaults to opus for a deliberately conservative (upper-bound) estimate, so the ceiling trips early rather than late."
+            }
+          }
+        },
         "reviewWatch": {
           "type": "object",
           "additionalProperties": false,
-          "description": "v8.6+ — Auto-review incoming PRs via gh CLI polling. Inspired by Cursor Bugbot (https://cursor.com/docs/bugbot) and Devin Review (https://docs.devin.ai/work-with-devin/devin-review): trigger /multi-agent:review on PRs the user did NOT author. Disabled by default. Configure repos via .repos[] then either run pipeline/lib/review-watch.sh --watch as a background process or schedule it via cron.",
+          "description": "v8.6+  -  Auto-review incoming PRs via gh CLI polling. Inspired by Cursor Bugbot (https://cursor.com/docs/bugbot) and Devin Review (https://docs.devin.ai/work-with-devin/devin-review): trigger /multi-agent:review on PRs the user did NOT author. Disabled by default. Configure repos via .repos[] then either run pipeline/lib/review-watch.sh --watch as a background process or schedule it via cron.",
           "properties": {
             "enabled": {
               "type": "boolean",
@@ -820,7 +869,7 @@
         "repoMap": {
           "type": "object",
           "additionalProperties": false,
-          "description": "v8.6+ — Aider-style token-budgeted repo map injection. When enabled, the orchestrator runs pipeline/scripts/repo-map.mjs BEFORE Phase 1 (Analysis) and BEFORE Phase 4 (Review) and injects the resulting markdown as ${REPO_MAP} into the reviewer/analyser prompts. Deterministic — no embeddings, no network. Pattern source: https://aider.chat/docs/repomap.html. Off by default (introduces a ~150-300ms repo scan + repo-shaped prompt growth); flip on to lower per-call token cost on follow-up exploration phases.",
+          "description": "v8.6+  -  Aider-style token-budgeted repo map injection. When enabled, the orchestrator runs pipeline/scripts/repo-map.mjs BEFORE Phase 1 (Analysis) and BEFORE Phase 4 (Review) and injects the resulting markdown as ${REPO_MAP} into the reviewer/analyser prompts. Deterministic  -  no embeddings, no network. Pattern source: https://aider.chat/docs/repomap.html. Off by default (introduces a ~150-300ms repo scan + repo-shaped prompt growth); flip on to lower per-call token cost on follow-up exploration phases.",
           "properties": {
             "enabled": {
               "type": "boolean",
@@ -839,15 +888,15 @@
               "minimum": 5,
               "maximum": 100,
               "default": 25,
-              "description": "Maximum number of ranked files to consider for inclusion. The token budget further trims this list — topFiles is an upper bound, not a guarantee."
+              "description": "Maximum number of ranked files to consider for inclusion. The token budget further trims this list  -  topFiles is an upper bound, not a guarantee."
             },
             "include": {
               "type": "string",
-              "description": "Comma-separated path substrings — files NOT containing any of these are skipped. Empty/unset = include all supported sources. Use to scope the map to a sub-tree on monorepos (e.g. 'Sources/,Tests/')."
+              "description": "Comma-separated path substrings  -  files NOT containing any of these are skipped. Empty/unset = include all supported sources. Use to scope the map to a sub-tree on monorepos (e.g. 'Sources/,Tests/')."
             },
             "exclude": {
               "type": "string",
-              "description": "Comma-separated path substrings — files containing ANY of these are skipped. Stacks with their own caches (e.g. 'Generated/,.pb.swift,_mocks.kt') benefit from this."
+              "description": "Comma-separated path substrings  -  files containing ANY of these are skipped. Stacks with their own caches (e.g. 'Generated/,.pb.swift,_mocks.kt') benefit from this."
             }
           }
         },
@@ -890,7 +939,7 @@
         "priorArtEnrichment": {
           "type": "object",
           "additionalProperties": false,
-          "description": "v8.3+ — Per-repo triage memory layer. When enabled, Phase 7 ingests every triage output's accepted/deferred/rejected rows into ~/.claude/memory/multi-agent/<repo-slug>/triage-corpus.jsonl (idempotent), and Phase 4 triage queries the corpus for similar past findings to inject as context. Per-repo isolation — never cross-leaks between projects.",
+          "description": "v8.3+  -  Per-repo triage memory layer. When enabled, Phase 7 ingests every triage output's accepted/deferred/rejected rows into ~/.claude/memory/multi-agent/<repo-slug>/triage-corpus.jsonl (idempotent), and Phase 4 triage queries the corpus for similar past findings to inject as context. Per-repo isolation  -  never cross-leaks between projects.",
           "properties": {
             "enabled": {
               "type": "boolean",
@@ -916,6 +965,41 @@
             "topN": 3
           }
         },
+        "learningsLedger": {
+          "type": "object",
+          "additionalProperties": false,
+          "description": "v9.3+  -  Per-repo persistent LEARNINGS ledger (pipeline/scripts/learnings-ledger.mjs), stored next to triage-corpus at ~/.claude/memory/multi-agent/<repo-slug>/learnings-ledger.jsonl. Holds durable architectural facts, conventions, and explicitly rejected review preferences. A compact brief is injected into Phase 1 analysis and Phase 4 triage so agents stop re-discovering structure and reviewers stop re-flagging rejected feedback (the most-cited cold-boot-amnesia complaint). Phase 7 distills each run's rejected findings into the ledger. Per-repo isolated; never cross-leaks.",
+          "properties": {
+            "enabled": {
+              "type": "boolean",
+              "default": true,
+              "description": "Master switch. When false, no brief is injected and Phase 7 does not append to the ledger."
+            },
+            "injectIntoAnalysis": {
+              "type": "boolean",
+              "default": true,
+              "description": "Inject the durable-knowledge brief into Phase 1 analysis context."
+            },
+            "injectIntoTriage": {
+              "type": "boolean",
+              "default": true,
+              "description": "Inject the rejected-preference brief into Phase 4 triage so known-rejected suggestions are not re-accepted."
+            },
+            "maxBriefEntries": {
+              "type": "integer",
+              "minimum": 1,
+              "maximum": 100,
+              "default": 20,
+              "description": "Cap on entries in an injected brief (most-recent first)."
+            }
+          },
+          "default": {
+            "enabled": true,
+            "injectIntoAnalysis": true,
+            "injectIntoTriage": true,
+            "maxBriefEntries": 20
+          }
+        },
         "testGap": {
           "type": "object",
           "additionalProperties": false,

package/pipeline/schemas/reviewer-output.schema.json

@@ -2,7 +2,7 @@
   "$schema": "https://json-schema.org/draft/2020-12/schema",
   "$id": "https://github.com/mmerterden/multi-agent-pipeline/pipeline/schemas/reviewer-output.schema.json",
   "version": "1.0.0",
-  "title": "Multi-Agent Pipeline — Phase 4 reviewer output",
+  "title": "Multi-Agent Pipeline  -  Phase 4 reviewer output",
   "description": "Contract for a single code-reviewer subagent's JSON output in Phase 4 Step 2. Reviewer set is CLI-aware: Claude Code dispatches 2 parallel reviewers (Opus, Sonnet); Copilot CLI dispatches 3 (GPT-5.4, Opus, Sonnet). Every reviewer must return an object matching this shape before Opus triage merges them.",
   "type": "object",
   "additionalProperties": false,
@@ -10,12 +10,16 @@
   "properties": {
     "findings": {
       "type": "array",
-      "description": "Raw findings. May be empty. Duplicates and out-of-scope items are acceptable — Opus triage will deduplicate and filter.",
+      "description": "Raw findings. May be empty. Duplicates and out-of-scope items are acceptable  -  Opus triage will deduplicate and filter.",
       "items": { "$ref": "#/$defs/finding" }
     },
     "approved": {
       "type": "boolean",
       "description": "Reviewer's own verdict. MUST be false if any finding has severity=blocking, true otherwise. Triage may override."
+    },
+    "reviewer": {
+      "type": "string",
+      "description": "Model label for this output (e.g. 'opus', 'sonnet', 'gpt'). Present once the parallel reviewer outputs are merged into the Phase 4 array so triage/consensus can attribute each finding to its source. Optional on a single reviewer's raw pre-merge output."
     }
   },
   "$defs": {
@@ -47,7 +51,7 @@
         "fix": {
           "type": "string",
           "minLength": 4,
-          "description": "Concrete remediation. Not 'refactor this' — actionable guidance."
+          "description": "Concrete remediation. Not 'refactor this'  -  actionable guidance."
         }
       }
     }

package/pipeline/schemas/test-gap.schema.json

@@ -2,7 +2,7 @@
   "$schema": "https://json-schema.org/draft/2020-12/schema",
   "$id": "https://github.com/mmerterden/multi-agent-pipeline/pipeline/schemas/test-gap.schema.json",
   "version": "1.0.0",
-  "title": "Multi-Agent Pipeline — Phase 5 test gap report",
+  "title": "Multi-Agent Pipeline  -  Phase 5 test gap report",
   "description": "Output of test-gap-scan.mjs. Lists symbols added/changed in the diff that have no paired test file or no matching test method. Advisory in default mode; opt-in blocking via prefs.testGap.blockingThreshold.",
   "type": "object",
   "additionalProperties": false,

package/pipeline/schemas/token-budget.json

@@ -3,15 +3,15 @@
   "$id": "https://github.com/mmerterden/multi-agent-pipeline/pipeline/schemas/token-budget.json",
   "description": "Per-phase token budget for lazy-loaded pipeline docs. Enforced by smoke-token-budget.sh.",
   "phases": {
-    "phase-0-init":      { "max_tokens": 8900, "warn_tokens": 7850 },
-    "phase-1-analysis":  { "max_tokens": 2550, "warn_tokens": 2250 },
-    "phase-2-planning":  { "max_tokens": 4200, "warn_tokens": 3700 },
-    "phase-3-dev":       { "max_tokens": 4950, "warn_tokens": 4350 },
-    "phase-4-review":    { "max_tokens": 6900, "warn_tokens": 6050 },
+    "phase-0-init":      { "max_tokens": 11250, "warn_tokens": 9900 },
+    "phase-1-analysis":  { "max_tokens": 3750, "warn_tokens": 3300 },
+    "phase-2-planning":  { "max_tokens": 6500, "warn_tokens": 5750 },
+    "phase-3-dev":       { "max_tokens": 7650, "warn_tokens": 6750 },
+    "phase-4-review":    { "max_tokens": 11100, "warn_tokens": 9750 },
     "phase-5-test":      { "max_tokens": 2300, "warn_tokens": 2000 },
     "phase-6-commit":    { "max_tokens": 5550, "warn_tokens": 4900 },
-    "phase-7-report":    { "max_tokens": 4400, "warn_tokens": 3850 }
+    "phase-7-report":    { "max_tokens": 5600, "warn_tokens": 4950 }
   },
-  "total_max_tokens": 39750,
-  "note": "Token estimate = ceil(chars / 4). Per-phase budget rule: warn = current+10% (rounded to nearest 50), max = current+25%. Gives ~6 edit cycles of headroom before warn trips — intentionally quiet under normal maintenance, loud when a phase grows unusually. Only the active phase is loaded (lazy)."
+  "total_max_tokens": 46500,
+  "note": "Token estimate = ceil(chars / 4). Per-phase budget rule: warn = current+10% (rounded to nearest 50), max = current+25%. Gives ~6 edit cycles of headroom before warn trips  -  intentionally quiet under normal maintenance, loud when a phase grows unusually. Only the active phase is loaded (lazy). Recalibrated at v10.0.0 after the validator/consistency/simplifier/lesson gate contracts landed in phases 1-4 (prose already compressed; the residual growth is the gate contracts themselves)."
 }

package/pipeline/schemas/triage-corpus.schema.json

@@ -2,7 +2,7 @@
   "$schema": "https://json-schema.org/draft/2020-12/schema",
   "$id": "https://github.com/mmerterden/multi-agent-pipeline/pipeline/schemas/triage-corpus.schema.json",
   "version": "1.0.0",
-  "title": "Multi-Agent Pipeline — Triage corpus row",
+  "title": "Multi-Agent Pipeline  -  Triage corpus row",
   "description": "Shape of one row in the JSONL triage corpus at $HOME/.claude/memory/multi-agent/<repo-slug>/triage-corpus.jsonl. Append-only. Each row records a single accepted/deferred/rejected finding from a past triage run plus minimal task context. Used by Phase 1 (similar-task enrichment) and Phase 4 (prior-art lookup).",
   "type": "object",
   "additionalProperties": false,

package/pipeline/schemas/triage-output.schema.json

@@ -1,9 +1,9 @@
 {
   "$schema": "https://json-schema.org/draft/2020-12/schema",
   "$id": "https://github.com/mmerterden/multi-agent-pipeline/pipeline/schemas/triage-output.schema.json",
-  "version": "3.0.0",
-  "title": "Multi-Agent Pipeline — Phase 4 triage output",
-  "description": "Contract for the Opus triage agent's JSON output in Phase 4 Step 3. Triage consumes merged reviewer findings and splits them into accepted/deferred/rejected. Only `accepted` blocking/important items trigger Phase 3 rework.",
+  "version": "3.1.0",
+  "title": "Multi-Agent Pipeline  -  Phase 4 triage output",
+  "description": "Contract for the Opus triage agent's JSON output in Phase 4 Step 3. Triage consumes merged reviewer findings and splits them into accepted/deferred/rejected. Only `accepted` blocking/important items trigger Phase 3 rework. v3.1.0 adds the optional `consensus` block so triage can surface reviewer-agreement risk (false consensus among same-base-model reviewers) instead of silently merging.",
   "type": "object",
   "additionalProperties": false,
   "required": ["accepted", "deferred", "rejected", "approved"],
@@ -25,7 +25,7 @@
           "reason": {
             "type": "string",
             "minLength": 4,
-            "description": "Why deferred — e.g. 'out of scope: touches payment module, task is auth-only'."
+            "description": "Why deferred  -  e.g. 'out of scope: touches payment module, task is auth-only'."
           }
         }
       }
@@ -42,7 +42,7 @@
           "reason": {
             "type": "string",
             "minLength": 4,
-            "description": "Why rejected — e.g. 'false positive: the null check the reviewer wants already happens in the caller'."
+            "description": "Why rejected  -  e.g. 'false positive: the null check the reviewer wants already happens in the caller'."
           }
         }
       }
@@ -50,7 +50,8 @@
     "approved": {
       "type": "boolean",
       "description": "True if no accepted BLOCKING items remain. The pipeline uses this to gate Phase 5 / Phase 6 entry."
-    }
+    },
+    "consensus": { "$ref": "#/$defs/consensus" }
   },
   "if": {
     "properties": {
@@ -76,6 +77,43 @@
       "enum": ["opus", "sonnet"],
       "description": "Which reviewer produced the raw finding. Haiku was removed in v2.1.0."
     },
+    "consensus": {
+      "type": "object",
+      "additionalProperties": false,
+      "description": "Optional v3.1.0 reviewer-agreement metadata. Unanimous agreement among reviewers that share a base model is NOT proof on judgment calls (correlated failure); triage marks those `unverified` so the user can confirm rather than the pipeline treating agreement as a pass.",
+      "required": ["reviewerCount", "verdict"],
+      "properties": {
+        "reviewerCount": {
+          "type": "integer",
+          "minimum": 1,
+          "description": "Reviewers dispatched this iteration (Claude Code = 2, Copilot CLI = 3)."
+        },
+        "verdict": {
+          "type": "string",
+          "enum": ["unanimous-pass", "unanimous-block", "split", "unverified"],
+          "description": "unanimous-pass/block = clear-cut agreement; split = reviewers disagreed and triage resolved it; unverified = reviewers agreed on a judgment call but the agreement may be correlated, so it is surfaced to the user rather than trusted."
+        },
+        "disagreements": {
+          "type": "array",
+          "description": "Findings where reviewers split (existence or severity). Surfaced verbatim in the Phase 7 report and at the Step 4 user checkpoint.",
+          "items": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["file", "issue", "note"],
+            "properties": {
+              "file": { "type": "string", "minLength": 1 },
+              "line": { "type": "integer", "minimum": 0 },
+              "issue": { "type": "string", "minLength": 4 },
+              "note": {
+                "type": "string",
+                "minLength": 4,
+                "description": "How the reviewers split  -  e.g. 'Opus blocking, Sonnet approved'."
+              }
+            }
+          }
+        }
+      }
+    },
     "rawFinding": {
       "type": "object",
       "additionalProperties": false,