@mmerterden/multi-agent-pipeline 20.2.1 → 20.3.1

package/pipeline/schemas/agent-state.schema.json

@@ -34,6 +34,10 @@
       "minimum": 1,
       "description": "Auto-incremented local short id from .worktrees/.multi-agent-counter  -  unique per machine, not per project."
     },
+    "sessionId": {
+      "type": ["string", "null"],
+      "description": "The Claude Code session id this run was launched under, copied by Phase 0 from MULTI_AGENT_SESSION_ID when the launcher set it. Continuous mode sets it to the id it passed as --session-id, and matches this field to find ITS run's state rather than another run in the same repo. Absent or null on an attended run: nothing sets the variable there."
+    },
     "jiraId": {
       "type": ["string", "null"],
       "description": "Jira issue key (PROJ-12345) if the task originated from Jira, else null."
@@ -193,7 +197,7 @@
     "maturity": {
       "type": "object",
       "additionalProperties": false,
-      "description": "How ready the fetched item was to be developed. Produced by lib/issue-fetcher.sh, which owns the gap codes and their localized wording. Declared as of v17.6.0 - it had been written by every issue-shaped run since long before and read by Phase 0, Phase 3 and the clarifier, while this schema forbade it under additionalProperties: false.",
+      "description": "How ready the fetched item was to be developed. Produced by lib/issue-fetcher.sh, which owns the gap codes and their localized wording. Read by Phase 0, Phase 3 and the clarifier.",
       "properties": {
         "score": {
           "type": ["integer", "null"],
@@ -268,18 +272,212 @@
     },
     "waitingFor": {
       "type": "string",
-      "enum": ["maturity", "user-channels-choice"],
-      "description": "The step a paused run must RE-ENTER, as opposed to the phase it should continue past. Declared as of v17.6.0: Phase 5's channels pause has written it since long before, and resume/SKILL.md never read it, so that pause was resumable only in prose. resume reads this first and falls back to currentPhase + 1 when it is absent. maturity: Phase 0's maturity step, with the item re-fetched (refs/features/maturity-followup.md). user-channels-choice: Phase 5's channels menu, with the stored channelsInput."
+      "enum": ["maturity", "user-channels-choice", "question"],
+      "description": "The step a paused run must RE-ENTER, as opposed to the phase it should continue past. resume reads this first and falls back to currentPhase + 1 when it is absent. maturity: Phase 0's maturity step, with the item re-fetched (refs/features/maturity-followup.md). user-channels-choice: Phase 5's channels menu, with the stored channelsInput. question: the step named by pendingQuestion.stepId, answered by `resume <id> --answer <optionId>`; open-questions-gate.mjs writes it as the last step of Phase 1 with stepId phase-1/open-questions."
     },
     "workspaceSource": {
       "type": "string",
-      "enum": ["asked", "command", "autopilot"],
-      "description": "Who decided where the branch lives. asked = the user answered the Step 5b workspace picker; command = a flow that only ever builds worktrees stated it up front; autopilot = resolved to a worktree without asking, because an unattended commit in the user's own checkout is what worktrees prevent. localMode alone cannot say: false is both a chosen worktree and one nothing asked about."
+      "enum": ["asked", "command", "autopilot", "request"],
+      "description": "Who decided where the branch lives. asked = the user answered the Step 5b workspace picker; command = a flow that only ever builds worktrees stated it up front; autopilot = resolved to a worktree without asking, because an unattended commit in the user's own checkout is what worktrees prevent; request = the launch request (MA_LAUNCH_REQUEST) answered the picker before the run started. localMode alone cannot say: false is both a chosen worktree and one nothing asked about."
+    },
+    "launchRequest": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["contractVersion", "mode", "answered", "resolvedAt"],
+      "description": "Present only when Phase 0 Step 0.1 applied a launch request (launch-request.mjs resolve). Records which pickers the request answered, so a reader can tell a pre-filled answer from one the user gave in the session. Absent on every run started without MA_LAUNCH_REQUEST.",
+      "properties": {
+        "contractVersion": { "type": "string" },
+        "mode": { "enum": ["interactive", "autopilot"] },
+        "answered": {
+          "type": "array",
+          "items": { "type": "string" },
+          "description": "Question ids from run-questions.json."
+        },
+        "resolvedAt": { "type": "string", "format": "date-time" }
+      }
+    },
+    "pendingQuestion": {
+      "type": ["object", "null"],
+      "additionalProperties": false,
+      "required": ["id", "stepId", "text", "kind", "options"],
+      "description": "A question a run stopped on, with waitingFor = question. Written by the step that needed an answer; answered by `/multi-agent:resume <id> --answer <optionId>` through answer-question.mjs, which accepts only one of options[].id and clears this record in the same write. The answer selects an option; it is never read as an instruction.",
+      "properties": {
+        "id": {
+          "type": "string",
+          "pattern": "^[a-z][a-z0-9-]*$",
+          "description": "Stable question id, a run-questions.json id when the step has one."
+        },
+        "stepId": {
+          "type": "string",
+          "minLength": 1,
+          "description": "The step to re-enter, e.g. phase-0/maturity or phase-3/user-test."
+        },
+        "text": {
+          "type": "string",
+          "minLength": 1,
+          "maxLength": 2000,
+          "description": "The question as the step would have asked it, in outputLanguage."
+        },
+        "kind": { "enum": ["single-select", "multi-select"] },
+        "options": {
+          "type": "array",
+          "minItems": 2,
+          "items": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["id", "label"],
+            "properties": {
+              "id": { "type": "string", "pattern": "^[a-z0-9][a-z0-9+_-]*$" },
+              "label": { "type": "string", "minLength": 1 }
+            }
+          }
+        },
+        "askedAt": { "type": "string", "format": "date-time" }
+      }
+    },
+    "research": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["round", "kind", "decision", "dir", "json", "md", "closed", "at"],
+      "description": "The latest research pass over this run's gaps (refs/features/research.md), written by research-gate.mjs and by nothing else: the gate reads research/research.json, verifies every cited source, re-runs the check and records the result here. The files stay in the run's state directory; these are pointers plus the verdict. decision proceed: the check passed with the verified findings applied, and the run re-enters the step it was parked on (the maturity step, or Phase 2 for open questions). decision ask: the run is parked on a pendingQuestion that lists the gaps still open and the unverified candidates.",
+      "properties": {
+        "round": {
+          "type": "integer",
+          "minimum": 1,
+          "description": "Passes applied to this run so far. The runner's own ceiling across runs of one item is autopilot config maxAskRounds, counted from attempted.jsonl."
+        },
+        "kind": { "enum": ["maturity", "open-questions"] },
+        "decision": { "enum": ["proceed", "ask"] },
+        "reason": { "type": "string" },
+        "dir": { "type": "string", "minLength": 1 },
+        "json": {
+          "type": "string",
+          "minLength": 1,
+          "description": "research.json, schemas/research-output.schema.json."
+        },
+        "context": {
+          "type": "string",
+          "description": "context.json: the fetched descriptor and documents the evidence was checked against."
+        },
+        "md": {
+          "type": "string",
+          "minLength": 1,
+          "description": "research.md, the self-contained story the dev run reads."
+        },
+        "gaps": {
+          "type": "array",
+          "items": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["id", "category", "status"],
+            "properties": {
+              "id": { "type": "string" },
+              "category": { "type": "string" },
+              "status": { "enum": ["closed", "candidate", "open"] },
+              "sources": { "type": "array", "items": { "type": "object" } }
+            }
+          }
+        },
+        "closed": { "type": "array", "items": { "type": "string" } },
+        "candidates": { "type": "array", "items": { "type": "string" } },
+        "open": { "type": "array", "items": { "type": "string" } },
+        "ignored": {
+          "type": "array",
+          "items": { "type": "string" },
+          "description": "Finding gap ids the run does not have."
+        },
+        "resolved": {
+          "type": "array",
+          "description": "Open questions closed by a verified source, for Phase 1 Step 5 to write into the analysis document with their source labels.",
+          "items": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["id", "question", "answer", "sources"],
+            "properties": {
+              "id": { "type": "string" },
+              "question": { "type": "string" },
+              "answer": { "type": "string" },
+              "sources": { "type": "array", "items": { "type": "object" } }
+            }
+          }
+        },
+        "maturityBefore": { "type": ["object", "null"] },
+        "maturityAfter": { "type": ["object", "null"] },
+        "at": { "type": "string", "format": "date-time" }
+      }
+    },
+    "lastAnswer": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["questionId", "stepId", "optionIds", "answeredAt"],
+      "description": "The answer answer-question.mjs recorded when it cleared pendingQuestion. The re-entered step reads it instead of asking again, then leaves it as a trace; the next answer overwrites it.",
+      "properties": {
+        "questionId": { "type": "string" },
+        "stepId": { "type": "string" },
+        "optionIds": { "type": "array", "minItems": 1, "items": { "type": "string" } },
+        "answeredAt": { "type": "string", "format": "date-time" }
+      }
     },
     "remoteType": {
       "type": "string",
-      "enum": ["github", "bitbucket"],
-      "description": "Determines which PR API and reviewer strategy is used."
+      "enum": ["github", "bitbucket", "gitlab", "generic-git", "local"],
+      "description": "Determines which PR API and reviewer strategy is used. generic-git has no PR API; local means the run has no remote at all."
+    },
+    "inputType": {
+      "type": "string",
+      "enum": ["github-issue-url", "github-issue-number", "jira-url", "jira-id", "free-text"],
+      "description": "The form the task input arrived in, as Phase 0 classified it. Commands that apply to one tracker only (the issue/Jira triad) branch on it."
+    },
+    "offlineOnly": {
+      "type": "boolean",
+      "default": false,
+      "description": "True when every entry in projects[] is local. Phases 4 and 5 read it to skip the PR prompt and remote reporting."
+    },
+    "figmaUrl": {
+      "type": ["string", "null"],
+      "description": "The Figma URL found in the task body at intake, or null. Component dispatch passes it to the stack plugin's create-component skill."
+    },
+    "instructionFiles": {
+      "type": "object",
+      "additionalProperties": { "type": "string" },
+      "description": "Instruction files steering an instruction-driven run, keyed by the step they replace (start, validate, dev, commit). Empty when instructionDriven is false."
+    },
+    "contextLinks": {
+      "type": "array",
+      "description": "Typed external links extracted from the task text by lib/context-link-extractor.sh at Phase 0 Step 1b. Phase 1 dispatches each entry to its fetcher by type.",
+      "items": {
+        "type": "object",
+        "additionalProperties": false,
+        "required": ["type", "url", "metadata"],
+        "properties": {
+          "type": {
+            "enum": [
+              "swagger",
+              "confluence",
+              "crashlytics",
+              "fortify",
+              "document",
+              "graylog",
+              "figma",
+              "generic-doc"
+            ]
+          },
+          "url": { "type": ["string", "null"] },
+          "metadata": { "type": "object" }
+        }
+      }
+    },
+    "crashContext": {
+      "type": ["object", "null"],
+      "description": "Crashlytics issue detail fetched at Phase 0 Step 1b.1 (features/url-enrichment.md). Phase 1 treats it as ground truth for the crash."
+    },
+    "fortifyFinding": {
+      "type": ["object", "null"],
+      "description": "Fortify SSC finding fetched at Phase 0 Step 1b.2 (features/url-enrichment.md), or {skipped: <reason>} when the fetch was skipped."
+    },
+    "graylogContext": {
+      "type": ["object", "array", "null"],
+      "description": "Graylog messages fetched at Phase 0 Step 1b.3 (features/url-enrichment.md): one object, an array when several ids were extracted, or {skipped: <reason>}."
     },
     "currentPhase": {
       "type": "integer",
@@ -289,8 +487,13 @@
     },
     "status": {
       "type": "string",
-      "enum": ["in_progress", "awaiting_input", "paused", "complete", "failed"],
-      "description": "Overall task status. `awaiting_input` is deliberately distinct from both `in_progress` and `paused`: the run is not broken and not resumable by a retry - it is finished with what it can do alone and is holding for a human answer, which is what Phase 5 does by design after the PR is open. Without a word for it, a run whose work had landed read exactly like one that crashed mid-development, and a state file in the wild had already invented `awaiting-user-test-main-checkout` to say it. `paused` stays what it was: stopped deliberately, resumable, nothing expected from anyone."
+      "enum": ["in_progress", "awaiting_input", "paused", "complete", "failed", "abandoned"],
+      "description": "Overall task status. `abandoned` is written only by the autopilot runner, through the locked state writer, for a run its session launched that ended with neither a PR nor a parked question; `abandonedBy` names the writer. `awaiting_input` is deliberately distinct from both `in_progress` and `paused`: the run is not broken and not resumable by a retry - it is finished with what it can do alone and is holding for a human answer, which is what Phase 5 does by design after the PR is open. Without a word for it, a run whose work had landed read exactly like one that crashed mid-development, and a state file in the wild had already invented `awaiting-user-test-main-checkout` to say it. `paused` stays what it was: stopped deliberately, resumable, nothing expected from anyone."
+    },
+    "abandonedBy": {
+      "type": "string",
+      "enum": ["autopilot-runner"],
+      "description": "Who marked the run `abandoned`. Only the autopilot runner does, and only for a run whose state carries the session id it launched the run with."
     },
     "startedAt": {
       "type": "string",
@@ -407,11 +610,11 @@
     "telemetry": {
       "type": "object",
       "additionalProperties": true,
-      "description": "Run telemetry. mcpCalls[] is consumed by smoke-no-mcp-in-dev-phases.sh to enforce the 'no MCP outside analysis' contract  -  every mcp__* invocation MUST append an entry tagged with the phase it ran in. skillCalls[] is Phase 3's self-report of which skills, plugin skills and guides it consulted; Phase 3 Step 1.78 reads it as CORROBORATING evidence only and never computes coverage from it.",
+      "description": "Run telemetry. mcpCalls[] records every mcp__* invocation tagged with the phase it ran in; smoke-no-mcp-in-dev-phases.sh audits it for Figma calls outside analysis as a maintainer regression check. skillCalls[] is Phase 3's self-report of which skills, plugin skills and guides it consulted; Phase 3 Step 1.78 reads it as CORROBORATING evidence only and never computes coverage from it.",
       "properties": {
         "mcpCalls": {
           "type": "array",
-          "description": "One entry per MCP tool invocation during the run. The gate fails if any entry has phase >= 2.",
+          "description": "One entry per MCP tool invocation during the run. A Figma tool entry with phase >= 2 is a violation of the analysis-only design source.",
           "items": {
             "type": "object",
             "required": ["tool", "phase"],
@@ -424,7 +627,7 @@
                 "type": "integer",
                 "minimum": 0,
                 "maximum": 5,
-                "description": "Pipeline phase the call ran in. Only 0 (init) and 1 (analysis) are permitted."
+                "description": "Pipeline phase the call ran in. Figma tools are permitted only in 0 (init) and 1 (analysis)."
               },
               "timestamp": {
                 "type": "string",
@@ -484,11 +687,20 @@
     "instructionDriven": {
       "type": "boolean",
       "default": false,
-      "description": "A Figma / skill instruction file is steering phases 3-6."
+      "description": "A Figma / skill instruction file is steering phases 2-4."
     },
     "mode": {
       "type": ["string", "null"],
-      "description": "Pipeline mode for this run (e.g. 'dev', 'local', 'design-check'). Absent = full pipeline."
+      "description": "Pipeline mode for this run (e.g. 'autopilot', 'analysis', 'design-check'). Absent = full pipeline."
+    },
+    "taskType": {
+      "type": "string",
+      "enum": ["component", "bugfix", "feature", "refactor", "chore"],
+      "description": "Phase 0 Step 7 classification. Phase 2 branches on it, and phase0-exit-gate.mjs fails a run that reaches Phase 1 without it. A Figma reference forces component."
+    },
+    "plan": {
+      "type": ["object", "null"],
+      "description": "The Phase 1 plan as a Todo list, written by plan-todos.sh when prefs.global.planTodos.enabled is true. Shape: plan-todos.schema.json, which plan-todos.sh validates on write; absent when the opt-in is off."
     },
     "analysis": {
       "type": ["object", "null"],
@@ -511,6 +723,25 @@
           "type": ["object", "null"],
           "additionalProperties": true,
           "description": "Parsed YAML header of the active platform's document."
+        },
+        "openQuestions": {
+          "type": "array",
+          "description": "Section 20 rows Phase 1 Step 5 deferred. open-questions-gate.mjs parks a gated run on a non-empty list and passes on an empty one; absent means no analysis recorded any.",
+          "items": { "type": ["string", "object"] }
+        },
+        "assumptions": {
+          "type": "array",
+          "description": "Open questions the run continues on as stated assumptions, moved here by open-questions-gate.mjs when the answer to its parked question is `assume`.",
+          "items": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["question", "source", "assumedAt"],
+            "properties": {
+              "question": { "type": "string" },
+              "source": { "type": "string" },
+              "assumedAt": { "type": "string", "format": "date-time" }
+            }
+          }
         }
       }
     },
@@ -782,7 +1013,7 @@
               "type": "integer",
               "minimum": 0,
               "maximum": 2,
-              "description": "v5.3.0 Phase 1 Plan Approval Gate  -  number of clarification question/answer rounds the orchestrator ran before producing the first plan. Cap 2; hitting the cap produces a 'best-effort' plan instead of asking more questions. Only emitted on a Full interactive run (a Short run has no plan, autopilot may not ask)."
+              "description": "v5.3.0 Phase 1 Plan Approval Gate  -  number of clarification question/answer rounds the orchestrator ran before producing the first plan. Cap 2; hitting the cap produces a 'best-effort' plan instead of asking more questions. Only emitted on an interactive run (autopilot may not ask)."
             },
             "clarificationQuestions": {
               "type": "array",
@@ -806,7 +1037,7 @@
             "planApprovedAt": {
               "type": ["string", "null"],
               "format": "date-time",
-              "description": "v5.3.0 Phase 1 Plan Approval Gate  -  timestamp when the user approved the plan. Null when the gate is not applicable (Short run, or autopilot) or the task was aborted at the gate."
+              "description": "v5.3.0 Phase 1 Plan Approval Gate  -  timestamp when the user approved the plan. Null when the gate is not applicable (autopilot) or the task was aborted at the gate."
             },
             "planEditRequests": {
               "type": "array",
@@ -854,6 +1085,10 @@
               "description": "The exact test command that produced this baseline. Phase 3 compares against its own Gate 3 command and treats a mismatch as unknown."
             }
           }
+        },
+        "sha": {
+          "type": "string",
+          "description": "The base commit the baseline was captured at. Pre-existing claims in unattended runs are checked against it (verify-citations.mjs)."
         }
       }
     },
@@ -1089,7 +1324,27 @@
           "accepted": {
             "type": "array"
           },
-          "validatorResult": {}
+          "validatorResult": {},
+          "reviewDecision": {
+            "type": "object",
+            "additionalProperties": true,
+            "description": "review-decision-gate.mjs --json output for this iteration, merged by Phase 3 after Step 3.7 while the quality gates are active (features/review-decision.md). kept[] lists the blocking findings that stayed blocking with their basis (corroborated | failing-test | test-integrity); downgraded[] lists each blocking finding lowered to important with its fingerprint and reason. Absent on an attended run.",
+            "properties": {
+              "verdict": {
+                "type": "string",
+                "enum": ["pass", "rebuttal-round", "unreadable"]
+              },
+              "approved": {
+                "type": "boolean"
+              },
+              "kept": {
+                "type": "array"
+              },
+              "downgraded": {
+                "type": "array"
+              }
+            }
+          }
         }
       }
     },
@@ -1253,7 +1508,7 @@
           },
           "remoteType": {
             "type": "string",
-            "enum": ["github", "bitbucket", "gitlab", "generic-git"]
+            "enum": ["github", "bitbucket", "gitlab", "generic-git", "local"]
           },
           "identity": {
             "type": "object",
@@ -1617,6 +1872,213 @@
           }
         }
       }
+    },
+    "gates": {
+      "type": "array",
+      "description": "Gate ledger, written while the quality gates are active (lib/unattended.mjs gatesActive: MULTI_AGENT_UNATTENDED=1, or state.autopilot true): one entry per verdict, appended by gate-ledger.mjs under the write-state lock. With MULTI_AGENT_UNATTENDED=1 in its environment, pre-commit-check.sh blocks a commit unless the latest entry of every mandatory gate for HEAD is pass or not-applicable. Absent in a run with neither.",
+      "items": {
+        "type": "object",
+        "additionalProperties": false,
+        "required": ["gate", "verdict", "at", "sha", "detail"],
+        "properties": {
+          "gate": {
+            "type": "string",
+            "minLength": 1,
+            "description": "Gate id, e.g. evidence-gate/test, test-summary, symbol-existence."
+          },
+          "verdict": {
+            "type": "string",
+            "enum": ["pass", "fail", "not-applicable"]
+          },
+          "at": {
+            "type": "string",
+            "format": "date-time"
+          },
+          "sha": {
+            "type": ["string", "null"],
+            "description": "Repository HEAD the gate ran against."
+          },
+          "detail": {
+            "type": "object",
+            "additionalProperties": true
+          }
+        }
+      }
+    },
+    "planCritique": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["round", "digest", "verdict", "at"],
+      "description": "The one plan-critique round of this run (refs/features/plan-critic.md), written by plan-critique-gate.mjs and by nothing else while the quality gates are active. digest is the sha256 of the judged critique with its replies; a later invocation with a different digest is refused as a second round. advisory is what Phase 1 carries into the plan and Phase 4 into the PR summary.",
+      "properties": {
+        "round": { "type": "integer", "const": 1 },
+        "path": { "type": "string" },
+        "digest": { "type": "string", "pattern": "^[0-9a-f]{64}$" },
+        "objectionsDigest": { "type": "string", "pattern": "^[0-9a-f]{64}$" },
+        "verdict": { "enum": ["pass", "fail"] },
+        "objections": { "type": "integer", "minimum": 0 },
+        "blocking": {
+          "type": "array",
+          "items": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["objection", "rule", "reason"],
+            "properties": {
+              "objection": { "type": "string" },
+              "rule": { "type": "string" },
+              "reason": { "type": "string" }
+            }
+          }
+        },
+        "advisory": {
+          "type": "array",
+          "items": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["objection", "lens", "claim", "label", "reply"],
+            "properties": {
+              "objection": { "type": "string" },
+              "lens": { "enum": ["scope", "feasibility", "security", "alternative"] },
+              "claim": { "type": "string" },
+              "label": { "enum": ["anchored", "inference"] },
+              "rule": { "type": "string" },
+              "ruleStatus": { "enum": ["binding", "proposed", "unknown", "unchecked"] },
+              "reply": { "enum": ["accept", "rebut", null] }
+            }
+          }
+        },
+        "at": { "type": "string", "format": "date-time" }
+      }
+    },
+    "publish": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["outcome", "at"],
+      "description": "Written by the autopilot runner's publish step (autopilot-publish.mjs) when it pushed a verified branch to a host with no draft PR. The runner reports it as pushed-awaiting-pr. A GitHub publish writes `pr` instead; a refused one writes `verificationFailed` with a `publish/<gate>` gate.",
+      "properties": {
+        "outcome": { "const": "pushed-awaiting-pr" },
+        "at": { "type": "string", "format": "date-time" },
+        "host": { "type": "string" },
+        "branch": { "type": "string" },
+        "base": { "type": "string" },
+        "sha": { "type": "string" }
+      }
+    },
+    "verificationFailed": {
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["gate", "reason", "at"],
+      "description": "Set by gate-ledger.mjs park when an unattended run's own gate rejected its work. The runner reports the attempt as verification-failed (parked, not retried).",
+      "properties": {
+        "gate": {
+          "type": "string",
+          "minLength": 1
+        },
+        "reason": {
+          "type": "string"
+        },
+        "at": {
+          "type": "string",
+          "format": "date-time"
+        }
+      }
+    },
+    "review": {
+      "type": ["object", "null"],
+      "additionalProperties": true,
+      "description": "State for a standalone /multi-agent:review run. post-pr-review.sh reads input, findings, iterationNumber and headCommitSha to post the verdict.",
+      "properties": {
+        "input": {
+          "type": "object",
+          "additionalProperties": false,
+          "required": ["kind", "provider", "raw"],
+          "description": "Parsed review target.",
+          "properties": {
+            "kind": {
+              "type": "string",
+              "enum": ["pr", "branch", "pick"]
+            },
+            "provider": {
+              "type": "string",
+              "enum": ["github", "bitbucket-server", "local"]
+            },
+            "ref": {
+              "type": "string"
+            },
+            "orgRepo": {
+              "type": "string"
+            },
+            "host": {
+              "type": "string"
+            },
+            "bbServerHost": {
+              "type": "string"
+            },
+            "bbProjectKey": {
+              "type": "string"
+            },
+            "bbRepoSlug": {
+              "type": "string"
+            },
+            "prNumber": {
+              "type": ["integer", "string"]
+            },
+            "raw": {
+              "type": "string"
+            }
+          }
+        },
+        "headCommitSha": {
+          "type": "string",
+          "description": "PR head commit; GitHub inline comments require it as commit_id."
+        },
+        "moduleGuides": {
+          "type": "array",
+          "items": {
+            "type": "string"
+          },
+          "description": "Repo-relative module guide paths injected into every reviewer prompt."
+        },
+        "findings": {
+          "type": "array",
+          "items": {
+            "type": "object",
+            "additionalProperties": true,
+            "properties": {
+              "severity": {
+                "type": "string"
+              },
+              "status": {
+                "type": "string"
+              },
+              "file": {
+                "type": "string"
+              },
+              "line": {
+                "type": ["integer", "null"]
+              },
+              "issue": {
+                "type": "string"
+              },
+              "fix": {
+                "type": "string"
+              },
+              "ruleId": {
+                "type": "string"
+              }
+            }
+          }
+        },
+        "iterationNumber": {
+          "type": "integer",
+          "minimum": 1
+        }
+      }
+    },
+    "complaintSpec": {
+      "type": ["object", "null"],
+      "additionalProperties": true,
+      "description": "Answers and progress of a /multi-agent:complaint-analysis run. Shape: complaint-analysis-spec.schema.json."
     }
   }
 }

package/pipeline/schemas/analysis-spec.schema.json

@@ -3,7 +3,7 @@
   "$id": "https://github.com/{owner}/multi-agent-pipeline/pipeline/schemas/analysis-spec.schema.json",
   "version": "1.3.0",
   "title": "Multi-Agent Pipeline - /multi-agent:analysis spec output (v3 template)",
-  "description": "Contract for the feature-spec analysis document generated by /multi-agent:analysis. Platform-agnostic concept layer + Pass B per-platform render with repo-driven conventions. 23 sections in Full mode; 8 of them in Lite mode. Sections may be absent - omission is the policy when no evidence exists.",
+  "description": "Contract for the feature-spec analysis document generated by /multi-agent:analysis. Platform-agnostic concept layer + Pass B per-platform render with repo-driven conventions. 23 sections. Sections may be absent - omission is the policy when no evidence exists.",
   "type": "object",
   "additionalProperties": false,
   "required": ["featureName", "language", "platforms", "repos", "evidence", "sections"],
@@ -38,12 +38,12 @@
       "type": "string",
       "enum": ["full", "lite"],
       "default": "full",
-      "description": "Output mode. Full renders 23 sections. Lite renders 1, 2, 4, 9, 13, 14, 21 + optional 23. Auto-resolved when both --lite and --full flags are absent."
+      "description": "Always full: sections are chosen by the omission table, not by a mode. `lite` is accepted only so documents written before v19.0.0 still validate; validate-analysis-doc.mjs reports it as a legacy value."
     },
     "liteModeAuto": {
       "type": "boolean",
       "default": false,
-      "description": "True when Phase 1c detected small-feature signals (spec < 100 lines AND figma frames <= 1 AND repo direct-match >= 8). User flags override this."
+      "description": "Never written. Accepted only so documents written before v19.0.0 still validate."
     },
     "templateVersion": {
       "type": "string",
@@ -898,7 +898,7 @@
         },
         "glossary": {
           "type": ["object", "null"],
-          "description": "Section 22 - Sözlük (footer). Optional in Lite mode."
+          "description": "Section 22 - Sözlük (footer). Default drop; see the omission table."
         },
         "changelog": {
           "type": ["object", "null"],