substrate-ai 0.20.32 → 0.20.34

package/dist/adapter-registry-Cu2eLuau.js

@@ -0,0 +1,4 @@
+import { AdapterRegistry } from "./dist-BnMsd9hC.js";
+import "./adapter-registry-DXLMTmfD.js";
+
+export { AdapterRegistry };

package/dist/cli/index.js

@@ -1,15 +1,15 @@
 #!/usr/bin/env node
-import { FileStateStore, RunManifest, SUBSTRATE_OWNED_SETTINGS_KEYS, SupervisorLock, VALID_PHASES, WorkGraphRepository, ZERO_FINDING_COUNTS, buildPipelineStatusOutput, createDatabaseAdapter, createStateStore, findPackageRoot, formatOutput, formatPipelineStatusHuman, formatPipelineSummary, formatTokenTelemetry, getAllDescendantPids, getAutoHealthData, getSubstrateDefaultSettings, inspectProcessTree, parseDbTimestampAsUtc, registerHealthCommand, resolveBmadMethodSrcPath, resolveBmadMethodVersion, resolveMainRepoRoot, resolveRunManifest, rollupFindingCounts } from "../health-DJ4z2uWN.js";
+import { FileStateStore, RunManifest, SUBSTRATE_OWNED_SETTINGS_KEYS, SupervisorLock, VALID_PHASES, WorkGraphRepository, ZERO_FINDING_COUNTS, buildPipelineStatusOutput, createDatabaseAdapter, createStateStore, findPackageRoot, formatOutput, formatPipelineStatusHuman, formatPipelineSummary, formatTokenTelemetry, getAllDescendantPids, getAutoHealthData, getSubstrateDefaultSettings, inspectProcessTree, parseDbTimestampAsUtc, registerHealthCommand, resolveBmadMethodSrcPath, resolveBmadMethodVersion, resolveMainRepoRoot, resolveRunManifest, rollupFindingCounts } from "../health-CnV6ndAb.js";
 import { createLogger } from "../logger-KeHncl-f.js";
 import { createEventBus } from "../helpers-CElYrONe.js";
-import { AdapterRegistry, BudgetConfigSchema, CURRENT_CONFIG_FORMAT_VERSION, CURRENT_TASK_GRAPH_VERSION, ConfigError, CostTrackerConfigSchema, DEFAULT_CONFIG, DoltClient, DoltNotInstalled, GlobalSettingsSchema, IngestionServer, MonitorDatabaseImpl, OPERATIONAL_FINDING, PartialGlobalSettingsSchema, PartialProviderConfigSchema, ProvidersSchema, RoutingRecommender, STORY_METRICS, TelemetryConfigSchema, addTokenUsage, aggregateTokenUsageForRun, checkDoltInstalled, compareRunMetrics, createAmendmentRun, createConfigSystem, createDecision, createDoltClient, createPipelineRun, getActiveDecisions, getAllCostEntriesFiltered, getBaselineRunMetrics, getDecisionsByCategory, getDecisionsByPhaseForRun, getLatestCompletedRun, getLatestRun, getPipelineRunById, getPlanningCostTotal, getRetryableEscalations, getRunMetrics, getRunningPipelineRuns, getSessionCostSummary, getSessionCostSummaryFiltered, getStoryMetricsForRun, getTokenUsageSummary, incrementRunRestarts, initSchema, initializeDolt, listRunMetrics, loadParentRunDecisions, supersedeDecision, tagRunAsBaseline, updatePipelineRun } from "../dist-CqtWS9wF.js";
+import { AdapterRegistry, BudgetConfigSchema, CURRENT_CONFIG_FORMAT_VERSION, CURRENT_TASK_GRAPH_VERSION, ConfigError, CostTrackerConfigSchema, DEFAULT_CONFIG, DoltClient, DoltNotInstalled, GlobalSettingsSchema, IngestionServer, MonitorDatabaseImpl, OPERATIONAL_FINDING, PartialGlobalSettingsSchema, PartialProviderConfigSchema, ProvidersSchema, RoutingRecommender, STORY_METRICS, TelemetryConfigSchema, addTokenUsage, aggregateTokenUsageForRun, checkDoltInstalled, compareRunMetrics, createAmendmentRun, createConfigSystem, createDecision, createDoltClient, createPipelineRun, getActiveDecisions, getAllCostEntriesFiltered, getBaselineRunMetrics, getDecisionsByCategory, getDecisionsByPhaseForRun, getLatestCompletedRun, getLatestRun, getPipelineRunById, getPlanningCostTotal, getRetryableEscalations, getRunMetrics, getRunningPipelineRuns, getSessionCostSummary, getSessionCostSummaryFiltered, getStoryMetricsForRun, getTokenUsageSummary, incrementRunRestarts, initSchema, initializeDolt, listRunMetrics, loadParentRunDecisions, supersedeDecision, tagRunAsBaseline, updatePipelineRun } from "../dist-BnMsd9hC.js";
 import "../adapter-registry-DXLMTmfD.js";
-import { AdapterTelemetryPersistence, AppError, DoltRepoMapMetaRepository, DoltSymbolRepository, ERR_REPO_MAP_STORAGE_WRITE, EpicIngester, GitClient, GrammarLoader, RepoMapInjector, RepoMapModule, RepoMapQueryEngine, RepoMapStorage, SymbolParser, createContextCompiler, createDispatcher, createEventEmitter, createImplementationOrchestrator, createPackLoader, createPhaseOrchestrator, createStopAfterGate, createTelemetryAdvisor, formatPhaseCompletionSummary, getFactoryRunSummaries, getScenarioResultsForRun, getTwinRunsForRun, listGraphRuns, registerExportCommand, registerFactoryCommand, registerRunCommand, registerScenariosCommand, resolveStoryKeys, runAnalysisPhase, runPlanningPhase, runSolutioningPhase, validateStopAfterFromConflict } from "../run-DGyWCmcu.js";
-import "../errors-1uLGqnvr.js";
+import { AdapterTelemetryPersistence, AppError, DoltRepoMapMetaRepository, DoltSymbolRepository, ERR_REPO_MAP_STORAGE_WRITE, EpicIngester, GitClient, GrammarLoader, RepoMapInjector, RepoMapModule, RepoMapQueryEngine, RepoMapStorage, SymbolParser, createContextCompiler, createDispatcher, createEventEmitter, createImplementationOrchestrator, createPackLoader, createPhaseOrchestrator, createStopAfterGate, createTelemetryAdvisor, formatPhaseCompletionSummary, getFactoryRunSummaries, getScenarioResultsForRun, getTwinRunsForRun, listGraphRuns, registerExportCommand, registerFactoryCommand, registerRunCommand, registerScenariosCommand, resolveStoryKeys, runAnalysisPhase, runPlanningPhase, runSolutioningPhase, validateStopAfterFromConflict } from "../run-BQxm1Qix.js";
+import "../errors-DSGhhrgv.js";
 import "../routing-CcBOCuC9.js";
 import "../decisions-C0pz9Clx.js";
 import "../version-manager-impl-BmOWu8ml.js";
-import { registerUpgradeCommand } from "../upgrade-k15NGTno.js";
+import { registerUpgradeCommand } from "../upgrade-D3NuTkSL.js";
 import { Command } from "commander";
 import { fileURLToPath } from "url";
 import { dirname, join, resolve } from "path";
@@ -3667,7 +3667,7 @@ async function runStatusAction(options) {
 			logger$12.debug({ err }, "Work graph query failed, continuing without work graph data");
 		}
 		if (run === void 0) {
-			const { inspectProcessTree: inspectProcessTree$1 } = await import("../health-DLqMd9uN.js");
+			const { inspectProcessTree: inspectProcessTree$1 } = await import("../health-Bqzqu6zC.js");
 			const substrateDirPath = join(projectRoot, ".substrate");
 			const processInfo = inspectProcessTree$1({
 				projectRoot,
@@ -4617,7 +4617,7 @@ function defaultSupervisorDeps() {
 				if (cached === null) {
 					const { AdapterRegistry: AR } = await import(
 						/* @vite-ignore */
-						"../adapter-registry-N0Klnk7Y.js"
+						"../adapter-registry-Cu2eLuau.js"
 );
 					cached = new AR();
 					await cached.discoverAndRegister();
@@ -5184,11 +5184,11 @@ async function runSupervisorAction(options, deps = {}) {
 						try {
 							const { createExperimenter } = await import(
 								/* @vite-ignore */
-								"../experimenter-VEJHtfLh.js"
+								"../experimenter-CN8PhRbv.js"
 );
 							const { getLatestRun: getLatest } = await import(
 								/* @vite-ignore */
-								"../decisions-7UJcRETq.js"
+								"../decisions-DlPFWRDm.js"
 );
 							const expAdapter = createDatabaseAdapter({
 								backend: "auto",
@@ -5198,7 +5198,7 @@ async function runSupervisorAction(options, deps = {}) {
 								await initSchema(expAdapter);
 								const { runRunAction: runPipeline } = await import(
 									/* @vite-ignore */
-									"../run-D-OcJNtZ.js"
+									"../run-B7jP4apv.js"
 );
 								const runStoryFn = async (opts) => {
 									const exitCode = await runPipeline({
@@ -5730,7 +5730,7 @@ async function runMetricsAction(options) {
 			const routingConfigPath = join(dbDir, "routing.yml");
 			let routingConfig = null;
 			if (existsSync$1(routingConfigPath)) try {
-				const { loadModelRoutingConfig } = await import("../routing-Ccpb_syA.js");
+				const { loadModelRoutingConfig } = await import("../routing-sauIk945.js");
 				routingConfig = loadModelRoutingConfig(routingConfigPath);
 			} catch {}
 			if (routingConfig === null) routingConfig = {
@@ -9003,8 +9003,8 @@ async function createProgram() {
 /** Fire-and-forget startup version check (story 8.3, AC3/AC5) */
 function checkForUpdatesInBackground(currentVersion) {
 	if (process.env.SUBSTRATE_NO_UPDATE_CHECK === "1") return;
-	import("../upgrade-CVaD7iUR.js").then(async () => {
-		const { createVersionManager } = await import("../version-manager-impl-CEiZEsmi.js");
+	import("../upgrade-BXfinu5K.js").then(async () => {
+		const { createVersionManager } = await import("../version-manager-impl-ChzfLw90.js");
 		const vm = createVersionManager();
 		const result = await vm.checkForUpdates();
 		if (result.updateAvailable) {

package/dist/{decisions-7UJcRETq.js → decisions-DlPFWRDm.js}

@@ -1,4 +1,4 @@
-import { addTokenUsage, createDecision, createPipelineRun, createRequirement, getArtifactByTypeForRun, getArtifactsByRun, getDecisionsByCategory, getDecisionsByPhase, getDecisionsByPhaseForRun, getLatestRun, getPipelineRunById, getRunningPipelineRuns, getTokenUsageSummary, listRequirements, registerArtifact, updateDecision, updatePipelineRun, updatePipelineRunConfig, upsertDecision } from "./dist-CqtWS9wF.js";
+import { addTokenUsage, createDecision, createPipelineRun, createRequirement, getArtifactByTypeForRun, getArtifactsByRun, getDecisionsByCategory, getDecisionsByPhase, getDecisionsByPhaseForRun, getLatestRun, getPipelineRunById, getRunningPipelineRuns, getTokenUsageSummary, listRequirements, registerArtifact, updateDecision, updatePipelineRun, updatePipelineRunConfig, upsertDecision } from "./dist-BnMsd9hC.js";
 import "./decisions-C0pz9Clx.js";
 
 export { getLatestRun };

package/dist/{dist-CqtWS9wF.js → dist-BnMsd9hC.js}

@@ -283,7 +283,21 @@ function parseYamlResult(yamlText, schema) {
 				error: `YAML parse error: ${message}`
 			};
 		}
-		else return {
+		else if (message.includes("bad indentation of a mapping entry")) {
+			const recovered = attemptBlockScalarRecovery(yamlText, message);
+			if (recovered !== null) try {
+				raw = yaml.load(sanitizeYamlEscapes(recovered));
+			} catch {
+				return {
+					parsed: null,
+					error: `YAML parse error: ${message}`
+				};
+			}
+			else return {
+				parsed: null,
+				error: `YAML parse error: ${message}`
+			};
+		} else return {
 			parsed: null,
 			error: `YAML parse error: ${message}`
 		};
@@ -307,6 +321,67 @@ function parseYamlResult(yamlText, schema) {
 	};
 }
 /**
+* Field names whose values are known free-form strings and may safely be
+* rewritten as block scalars when the parser fails on an embedded colon.
+* Other fields (numeric, enum, structural) stay strict.
+*
+* Allowlist intent: we only rescue the failure mode that obs_015 documents
+* (reviewer-style agents quoting colons in description-shaped fields).
+* Adding fields here loosens the recovery surface; do so only when an
+* empirical case shows a new field needs the same protection.
+*/
+const RECOVERABLE_BLOCK_SCALAR_FIELDS = new Set([
+	"description",
+	"message",
+	"error",
+	"notes",
+	"comment",
+	"finding",
+	"command",
+	"details",
+	"rationale",
+	"reason"
+]);
+/**
+* Attempt to rewrite a single `<field>: <value-with-colons>` line as a
+* literal block scalar (`<field>: |-\n  <value>`) so the YAML parser
+* doesn't reinterpret the inner colons as mapping separators.
+*
+* Returns the rewritten YAML on success, or `null` when:
+*  - the error message doesn't carry a parseable position
+*  - no allowlisted field within a 4-line backward window matches
+*  - the matched value doesn't contain a colon (the trigger condition)
+*/
+function attemptBlockScalarRecovery(yamlText, errorMessage) {
+	const positionMatch = errorMessage.match(/\((\d+):(\d+)\)/);
+	if (positionMatch === null) return null;
+	const lineNum = Number.parseInt(positionMatch[1] ?? "", 10);
+	if (!Number.isFinite(lineNum) || lineNum <= 0) return null;
+	const lineIdx = lineNum - 1;
+	const lines = yamlText.split("\n");
+	if (lineIdx >= lines.length) return null;
+	for (let i = lineIdx; i >= Math.max(0, lineIdx - 4); i -= 1) {
+		const line = lines[i] ?? "";
+		const m = line.match(/^(\s*-?\s*)([A-Za-z_][A-Za-z0-9_]*):\s+(\S.*?)\s*$/);
+		if (m === null) continue;
+		const fieldName = (m[2] ?? "").toLowerCase();
+		if (!RECOVERABLE_BLOCK_SCALAR_FIELDS.has(fieldName)) continue;
+		const value = m[3] ?? "";
+		if (!value.includes(":")) continue;
+		const prefix = m[1] ?? "";
+		const valueIndent = " ".repeat(prefix.length + 2);
+		const rewrittenKeyLine = `${prefix}${m[2]}: |-`;
+		const valueLines = value.split("\n").map((v) => `${valueIndent}${v}`);
+		return [
+			...lines.slice(0, i),
+			rewrittenKeyLine,
+			...valueLines,
+			...lines.slice(i + 1)
+		].join("\n");
+	}
+	return null;
+}
+/**
 * Valid YAML escape sequences in double-quoted strings (YAML 1.2 spec).
 * Any backslash followed by a character NOT in this set is invalid.
 */
@@ -10940,4 +11015,4 @@ async function callLLM(params) {
 
 //#endregion
 export { ADVISORY_NOTES, AdapterRegistry, AdtError, BudgetConfigSchema, CURRENT_CONFIG_FORMAT_VERSION, CURRENT_TASK_GRAPH_VERSION, Categorizer, ClaudeCodeAdapter, CodexCLIAdapter, ConfigError, ConfigIncompatibleFormatError, ConsumerAnalyzer, CostTrackerConfigSchema, DEFAULT_CONFIG, DEFAULT_GLOBAL_SETTINGS, DispatcherImpl, DoltClient, DoltNotInstalled, DoltQueryError, ESCALATION_DIAGNOSIS, EXPERIMENT_RESULT, EfficiencyScorer, GeminiCLIAdapter, GlobalSettingsSchema, IngestionServer, LEARNING_FINDING, LogTurnAnalyzer, ModelRoutingConfigSchema, MonitorDatabaseImpl, OPERATIONAL_FINDING, PartialGlobalSettingsSchema, PartialProviderConfigSchema, ProviderPolicySchema, ProvidersSchema, Recommender, RoutingConfigError, RoutingRecommender, RoutingResolver, RoutingTelemetry, RoutingTokenAccumulator, RoutingTuner, STORY_METRICS, STORY_OUTCOME, SubstrateConfigSchema, TASK_TYPE_PHASE_MAP, TEST_EXPANSION_FINDING, TEST_PLAN, TelemetryConfigSchema, TelemetryNormalizer, TelemetryPipeline, TurnAnalyzer, VersionManagerImpl, addTokenUsage, aggregateTokenUsageForRun, aggregateTokenUsageForStory, buildAuditLogEntry, buildBranchName, buildModificationDirective, buildPRBody, buildWorktreePath, callLLM, checkDoltInstalled, compareRunMetrics, createAmendmentRun, createConfigSystem, createDatabaseAdapter as createDatabaseAdapter$1, createDecision, createDoltClient, createExperimenter, createPipelineRun, createRequirement, createVersionManager, detectInterfaceChanges, determineVerdict, getActiveDecisions, getAllCostEntriesFiltered, getArtifactByTypeForRun, getArtifactsByRun, getBaselineRunMetrics, getDecisionsByCategory, getDecisionsByPhase, getDecisionsByPhaseForRun, getLatestCompletedRun, getLatestRun, getModelTier, getPipelineRunById, getPlanningCostTotal, getRetryableEscalations, getRunMetrics, getRunningPipelineRuns, getSessionCostSummary, getSessionCostSummaryFiltered, getStoryMetricsForRun, getTokenUsageSummary, incrementRunRestarts, initSchema, initializeDolt, listRequirements, listRunMetrics, loadModelRoutingConfig, loadParentRunDecisions, registerArtifact, resolvePromptFile, supersedeDecision, tagRunAsBaseline, updateDecision, updatePipelineRun, updatePipelineRunConfig, upsertDecision, writeRunMetrics, writeStoryMetrics };
-//# sourceMappingURL=dist-CqtWS9wF.js.map
+//# sourceMappingURL=dist-BnMsd9hC.js.map

package/dist/{errors-1uLGqnvr.js → errors-DSGhhrgv.js}

@@ -1,4 +1,4 @@
-import { AdtError } from "./dist-CqtWS9wF.js";
+import { AdtError } from "./dist-BnMsd9hC.js";
 
 //#region src/core/errors.ts
 /** Error thrown when task configuration is invalid */
@@ -71,4 +71,4 @@ var TaskGraphIncompatibleFormatError = class extends AdtError {
 
 //#endregion
 export { BudgetExceededError, GitError, RecoveryError, TaskConfigError, TaskGraphCycleError, TaskGraphError, TaskGraphIncompatibleFormatError, WorkerError, WorkerNotFoundError };
-//# sourceMappingURL=errors-1uLGqnvr.js.map
+//# sourceMappingURL=errors-DSGhhrgv.js.map

package/dist/{experimenter-VEJHtfLh.js → experimenter-CN8PhRbv.js}

@@ -1,3 +1,3 @@
-import { buildAuditLogEntry, buildBranchName, buildModificationDirective, buildPRBody, buildWorktreePath, createExperimenter, determineVerdict, resolvePromptFile } from "./dist-CqtWS9wF.js";
+import { buildAuditLogEntry, buildBranchName, buildModificationDirective, buildPRBody, buildWorktreePath, createExperimenter, determineVerdict, resolvePromptFile } from "./dist-BnMsd9hC.js";
 
 export { createExperimenter };

package/dist/{health-DLqMd9uN.js → health-Bqzqu6zC.js}

@@ -1,6 +1,6 @@
-import { DEFAULT_STALL_THRESHOLD_SECONDS, getAllDescendantPids, getAutoHealthData, inspectProcessTree, isOrchestratorProcessLine, registerHealthCommand, runHealthAction } from "./health-DJ4z2uWN.js";
+import { DEFAULT_STALL_THRESHOLD_SECONDS, getAllDescendantPids, getAutoHealthData, inspectProcessTree, isOrchestratorProcessLine, registerHealthCommand, runHealthAction } from "./health-CnV6ndAb.js";
 import "./logger-KeHncl-f.js";
-import "./dist-CqtWS9wF.js";
+import "./dist-BnMsd9hC.js";
 import "./decisions-C0pz9Clx.js";
 
 export { inspectProcessTree };

package/dist/{health-DJ4z2uWN.js → health-CnV6ndAb.js}

@@ -1,5 +1,5 @@
 import { createLogger } from "./logger-KeHncl-f.js";
-import { DoltClient, DoltQueryError, LEARNING_FINDING, createDatabaseAdapter$1 as createDatabaseAdapter, createDecision, getDecisionsByCategory, getLatestRun, getPipelineRunById, initSchema } from "./dist-CqtWS9wF.js";
+import { DoltClient, DoltQueryError, LEARNING_FINDING, createDatabaseAdapter$1 as createDatabaseAdapter, createDecision, getDecisionsByCategory, getLatestRun, getPipelineRunById, initSchema } from "./dist-BnMsd9hC.js";
 import { createRequire } from "module";
 import { dirname, join } from "path";
 import { readFile } from "fs/promises";
@@ -3831,6 +3831,40 @@ function tail(text, bytes = PROBE_TAIL_BYTES) {
 * way a typo in one author's probe surfaces as a deterministic finding,
 * not a pipeline crash that masks the rest of the run.
 */
+/**
+* Story 63-2: canonical error-envelope shapes that signal the tool
+* responded with a structured error despite exiting 0. Detected
+* regardless of whether the author declared an assertion — defense-in-depth
+* against under-test probes that count tool advertisement but don't check
+* response shape (the obs_012 REOPENED class).
+*
+* Patterns are matched against the FULL captured stdout (not the tailed
+* excerpt) so error envelopes deeper than `PROBE_TAIL_BYTES` aren't
+* missed when a probe runs multiple sub-commands and the error surfaces
+* partway through.
+*
+* Match keys are JSON-shape only (case-sensitive); we do not try to
+* detect prose-form errors ("ERROR:" log lines) because those produce
+* far too many false positives — many tools log informational warnings
+* to stdout that contain the word "error".
+*/
+const ERROR_SHAPE_PATTERNS = [{
+	pattern: /"isError"\s*:\s*true\b/,
+	description: "\"isError\": true (MCP / Anthropic tool error envelope)"
+}, {
+	pattern: /"status"\s*:\s*"error"/,
+	description: "\"status\": \"error\" (REST / RPC error envelope)"
+}];
+/**
+* Story 63-2: scan stdout for canonical error-envelope shapes. Returns
+* an array of human-readable descriptions of detected patterns; empty
+* array when the response shape looks clean.
+*/
+function detectErrorShapeIndicators(stdout) {
+	const indicators = [];
+	for (const { pattern, description } of ERROR_SHAPE_PATTERNS) if (pattern.test(stdout)) indicators.push(description);
+	return indicators;
+}
 function evaluateStdoutAssertions(probe, stdout) {
 	const failures = [];
 	for (const pattern of probe.expect_stdout_no_regex ?? []) {
@@ -3931,11 +3965,18 @@ function executeProbeOnHost(probe, options = {}) {
 			const duration = Date.now() - start;
 			let outcome = code === 0 ? "pass" : "fail";
 			let assertionFailures;
+			let errorShapeIndicators;
 			if (outcome === "pass") {
 				const failures = evaluateStdoutAssertions(probe, stdout);
 				if (failures.length > 0) {
 					outcome = "fail";
 					assertionFailures = failures;
+				} else {
+					const indicators = detectErrorShapeIndicators(stdout);
+					if (indicators.length > 0) {
+						outcome = "fail";
+						errorShapeIndicators = indicators;
+					}
 				}
 			}
 			finalize({
@@ -3945,7 +3986,8 @@ function executeProbeOnHost(probe, options = {}) {
 				stdoutTail: tail(stdout),
 				stderrTail: tail(stderr),
 				durationMs: duration,
-				...assertionFailures !== void 0 ? { assertionFailures } : {}
+				...assertionFailures !== void 0 ? { assertionFailures } : {},
+				...errorShapeIndicators !== void 0 ? { errorShapeIndicators } : {}
 			});
 		});
 	});
@@ -3966,6 +4008,16 @@ const CATEGORY_TIMEOUT = "runtime-probe-timeout";
 */
 const CATEGORY_ASSERTION_FAIL = "runtime-probe-assertion-fail";
 /**
+* Story 63-2: command exited 0 and no author-declared assertion tripped,
+* but the captured stdout contains a canonical error-envelope shape
+* (`"isError": true`, `"status": "error"`). Defense-in-depth against
+* under-test probes that count tool advertisement without checking
+* response shape — the structural fix for obs_2026-04-25_012 (REOPENED).
+* Distinct from CATEGORY_ASSERTION_FAIL because the author DIDN'T declare
+* an assertion; the executor caught the error envelope automatically.
+*/
+const CATEGORY_ERROR_RESPONSE = "runtime-probe-error-response";
+/**
 * Story 60-11: source AC describes an event-driven mechanism (hook, timer,
 * signal, webhook) but no probe's command invokes a known production-trigger
 * pattern. Strata Run 13 (Story 1-12, 2026-04-26): vault conflict hook
@@ -4103,11 +4155,12 @@ var RuntimeProbeCheck = class {
 			}
 			const result = await this._executors.host(probe);
 			if (result.outcome === "pass") continue;
-			const category = result.outcome === "timeout" ? CATEGORY_TIMEOUT : result.assertionFailures !== void 0 ? CATEGORY_ASSERTION_FAIL : CATEGORY_FAIL;
+			const category = result.outcome === "timeout" ? CATEGORY_TIMEOUT : result.assertionFailures !== void 0 ? CATEGORY_ASSERTION_FAIL : result.errorShapeIndicators !== void 0 ? CATEGORY_ERROR_RESPONSE : CATEGORY_FAIL;
 			const descriptor = probe.description ? ` (${probe.description})` : "";
 			let message;
 			if (result.outcome === "timeout") message = `probe "${probe.name}"${descriptor} timed out after ${result.durationMs}ms`;
 			else if (result.assertionFailures !== void 0) message = `probe "${probe.name}"${descriptor} exit 0 but stdout assertion failed: ` + result.assertionFailures.join("; ");
+			else if (result.errorShapeIndicators !== void 0) message = `probe "${probe.name}"${descriptor} exit 0 but response contained error envelope: ` + result.errorShapeIndicators.join("; ") + " — the tool returned an error-shaped JSON response despite a clean exit code. This is structural evidence the implementation didn't actually work; add an explicit `expect_stdout_no_regex` assertion to make the failure surface earlier in author-controlled form.";
 			else message = `probe "${probe.name}"${descriptor} failed with exit ${result.exitCode ?? "unknown"}`;
 			findings.push({
 				category,
@@ -6291,4 +6344,4 @@ function registerHealthCommand(program, _version = "0.0.0", projectRoot = proces
 
 //#endregion
 export { BMAD_BASELINE_TOKENS_FULL, DEFAULT_STALL_THRESHOLD_SECONDS, DoltMergeConflict, FileStateStore, FindingsInjector, RunManifest, RuntimeProbeListSchema, STOP_AFTER_VALID_PHASES, STORY_KEY_PATTERN$1 as STORY_KEY_PATTERN, SUBSTRATE_OWNED_SETTINGS_KEYS, SupervisorLock, VALID_PHASES, WorkGraphRepository, ZERO_FINDING_COUNTS, __commonJS, __require, __toESM, applyConfigToGraph, buildPipelineStatusOutput, createDatabaseAdapter$1 as createDatabaseAdapter, createDefaultVerificationPipeline, createGraphOrchestrator, createSdlcCodeReviewHandler, createSdlcCreateStoryHandler, createSdlcDevStoryHandler, createSdlcPhaseHandler, createStateStore, detectCycles, detectsEventDrivenAC, extractTargetFilesFromStoryContent, findPackageRoot, formatOutput, formatPipelineStatusHuman, formatPipelineSummary, formatTokenTelemetry, getAllDescendantPids, getAutoHealthData, getSubstrateDefaultSettings, inspectProcessTree, isOrchestratorProcessLine, parseDbTimestampAsUtc, registerHealthCommand, renderFindings, resolveBmadMethodSrcPath, resolveBmadMethodVersion, resolveGraphPath, resolveMainRepoRoot, resolveRunManifest, rollupFindingCounts, runHealthAction, validateStoryKey };
-//# sourceMappingURL=health-DJ4z2uWN.js.map
+//# sourceMappingURL=health-CnV6ndAb.js.map

package/dist/index.d.ts

@@ -2106,6 +2106,24 @@ interface OrchestratorEvents {
     storyKey: string;
     reason: string;
   };
+  /**
+   * Story 62-3: code-review agent emitted YAML output that failed schema
+   * validation (typically a parse error from unquoted-colon-in-value or
+   * unbalanced quotes — see obs_2026-04-27_015). Distinct from the generic
+   * phantom-review path so operators can debug agent emission shape rather
+   * than chase a crash/timeout phantom. Fires on each malformed-output
+   * cycle; final escalation (after MAX_SCHEMA_VALIDATION_RETRIES exhausted)
+   * surfaces via the standard `orchestrator:story-escalated` event with
+   * `lastVerdict: 'code-review-output-malformed-budget-exceeded'`.
+   */
+  'orchestrator:code-review-output-malformed': {
+    storyKey: string;
+    reviewCycles: number;
+    attempt: number;
+    maxAttempts: number;
+    error: string;
+    details?: string;
+  };
   /** Implementation orchestrator has finished all stories */
   'orchestrator:complete': {
     totalStories: number;

package/dist/index.js

@@ -1,8 +1,8 @@
 import { childLogger, createLogger, logger } from "./logger-KeHncl-f.js";
 import { assertDefined, createEventBus, createTuiApp, deepClone, formatDuration, generateId, isPlainObject, isTuiCapable, printNonTtyWarning, sleep, withRetry } from "./helpers-CElYrONe.js";
-import { AdapterRegistry, AdtError, ClaudeCodeAdapter, CodexCLIAdapter, ConfigError, ConfigIncompatibleFormatError, GeminiCLIAdapter } from "./dist-CqtWS9wF.js";
+import { AdapterRegistry, AdtError, ClaudeCodeAdapter, CodexCLIAdapter, ConfigError, ConfigIncompatibleFormatError, GeminiCLIAdapter } from "./dist-BnMsd9hC.js";
 import "./adapter-registry-DXLMTmfD.js";
-import { BudgetExceededError, GitError, RecoveryError, TaskConfigError, TaskGraphCycleError, TaskGraphError, TaskGraphIncompatibleFormatError, WorkerError, WorkerNotFoundError } from "./errors-1uLGqnvr.js";
+import { BudgetExceededError, GitError, RecoveryError, TaskConfigError, TaskGraphCycleError, TaskGraphError, TaskGraphIncompatibleFormatError, WorkerError, WorkerNotFoundError } from "./errors-DSGhhrgv.js";
 
 //#region src/core/di.ts
 /**

package/dist/{routing-Ccpb_syA.js → routing-sauIk945.js}

@@ -1,4 +1,4 @@
-import { ModelRoutingConfigSchema, ProviderPolicySchema, RoutingConfigError, RoutingRecommender, RoutingResolver, RoutingTelemetry, RoutingTokenAccumulator, RoutingTuner, TASK_TYPE_PHASE_MAP, getModelTier, loadModelRoutingConfig } from "./dist-CqtWS9wF.js";
+import { ModelRoutingConfigSchema, ProviderPolicySchema, RoutingConfigError, RoutingRecommender, RoutingResolver, RoutingTelemetry, RoutingTokenAccumulator, RoutingTuner, TASK_TYPE_PHASE_MAP, getModelTier, loadModelRoutingConfig } from "./dist-BnMsd9hC.js";
 import "./routing-CcBOCuC9.js";
 
 export { loadModelRoutingConfig };

package/dist/{run-D-OcJNtZ.js → run-B7jP4apv.js}

@@ -1,8 +1,8 @@
-import "./health-DJ4z2uWN.js";
+import "./health-CnV6ndAb.js";
 import "./logger-KeHncl-f.js";
 import "./helpers-CElYrONe.js";
-import "./dist-CqtWS9wF.js";
-import { normalizeGraphSummaryToStatus, registerRunCommand, resolveMaxReviewCycles, runRunAction, wireNdjsonEmitter } from "./run-DGyWCmcu.js";
+import "./dist-BnMsd9hC.js";
+import { normalizeGraphSummaryToStatus, registerRunCommand, resolveMaxReviewCycles, runRunAction, wireNdjsonEmitter } from "./run-BQxm1Qix.js";
 import "./routing-CcBOCuC9.js";
 import "./decisions-C0pz9Clx.js";
 

package/dist/{run-DGyWCmcu.js → run-BQxm1Qix.js}

@@ -1,7 +1,7 @@
-import { BMAD_BASELINE_TOKENS_FULL, DoltMergeConflict, FileStateStore, FindingsInjector, RunManifest, RuntimeProbeListSchema, STOP_AFTER_VALID_PHASES, STORY_KEY_PATTERN, VALID_PHASES, WorkGraphRepository, __commonJS, __require, __toESM, applyConfigToGraph, buildPipelineStatusOutput, createDatabaseAdapter, createDefaultVerificationPipeline, createGraphOrchestrator, createSdlcCodeReviewHandler, createSdlcCreateStoryHandler, createSdlcDevStoryHandler, createSdlcPhaseHandler, detectCycles, detectsEventDrivenAC, extractTargetFilesFromStoryContent, formatOutput, formatPipelineSummary, formatTokenTelemetry, inspectProcessTree, parseDbTimestampAsUtc, renderFindings, resolveGraphPath, resolveMainRepoRoot, validateStoryKey } from "./health-DJ4z2uWN.js";
+import { BMAD_BASELINE_TOKENS_FULL, DoltMergeConflict, FileStateStore, FindingsInjector, RunManifest, RuntimeProbeListSchema, STOP_AFTER_VALID_PHASES, STORY_KEY_PATTERN, VALID_PHASES, WorkGraphRepository, __commonJS, __require, __toESM, applyConfigToGraph, buildPipelineStatusOutput, createDatabaseAdapter, createDefaultVerificationPipeline, createGraphOrchestrator, createSdlcCodeReviewHandler, createSdlcCreateStoryHandler, createSdlcDevStoryHandler, createSdlcPhaseHandler, detectCycles, detectsEventDrivenAC, extractTargetFilesFromStoryContent, formatOutput, formatPipelineSummary, formatTokenTelemetry, inspectProcessTree, parseDbTimestampAsUtc, renderFindings, resolveGraphPath, resolveMainRepoRoot, validateStoryKey } from "./health-CnV6ndAb.js";
 import { createLogger } from "./logger-KeHncl-f.js";
 import { TypedEventBusImpl, createEventBus, createTuiApp, isTuiCapable, printNonTtyWarning, sleep } from "./helpers-CElYrONe.js";
-import { ADVISORY_NOTES, Categorizer, ConsumerAnalyzer, DEFAULT_GLOBAL_SETTINGS, DispatcherImpl, DoltClient, ESCALATION_DIAGNOSIS, EXPERIMENT_RESULT, EfficiencyScorer, IngestionServer, LogTurnAnalyzer, OPERATIONAL_FINDING, Recommender, RoutingRecommender, RoutingResolver, RoutingTelemetry, RoutingTokenAccumulator, RoutingTuner, STORY_METRICS, STORY_OUTCOME, SubstrateConfigSchema, TEST_EXPANSION_FINDING, TEST_PLAN, TelemetryNormalizer, TelemetryPipeline, TurnAnalyzer, addTokenUsage, aggregateTokenUsageForRun, aggregateTokenUsageForStory, callLLM, createConfigSystem, createDatabaseAdapter$1, createDecision, createPipelineRun, createRequirement, detectInterfaceChanges, getArtifactByTypeForRun, getArtifactsByRun, getDecisionsByCategory, getDecisionsByPhase, getDecisionsByPhaseForRun, getLatestRun, getPipelineRunById, getRunMetrics, getRunningPipelineRuns, getStoryMetricsForRun, getTokenUsageSummary, initSchema, listRequirements, loadModelRoutingConfig, registerArtifact, updatePipelineRun, updatePipelineRunConfig, upsertDecision, writeRunMetrics, writeStoryMetrics } from "./dist-CqtWS9wF.js";
+import { ADVISORY_NOTES, Categorizer, ConsumerAnalyzer, DEFAULT_GLOBAL_SETTINGS, DispatcherImpl, DoltClient, ESCALATION_DIAGNOSIS, EXPERIMENT_RESULT, EfficiencyScorer, IngestionServer, LogTurnAnalyzer, OPERATIONAL_FINDING, Recommender, RoutingRecommender, RoutingResolver, RoutingTelemetry, RoutingTokenAccumulator, RoutingTuner, STORY_METRICS, STORY_OUTCOME, SubstrateConfigSchema, TEST_EXPANSION_FINDING, TEST_PLAN, TelemetryNormalizer, TelemetryPipeline, TurnAnalyzer, addTokenUsage, aggregateTokenUsageForRun, aggregateTokenUsageForStory, callLLM, createConfigSystem, createDatabaseAdapter$1, createDecision, createPipelineRun, createRequirement, detectInterfaceChanges, getArtifactByTypeForRun, getArtifactsByRun, getDecisionsByCategory, getDecisionsByPhase, getDecisionsByPhaseForRun, getLatestRun, getPipelineRunById, getRunMetrics, getRunningPipelineRuns, getStoryMetricsForRun, getTokenUsageSummary, initSchema, listRequirements, loadModelRoutingConfig, registerArtifact, updatePipelineRun, updatePipelineRunConfig, upsertDecision, writeRunMetrics, writeStoryMetrics } from "./dist-BnMsd9hC.js";
 import { basename, dirname, extname, join } from "path";
 import { access, readFile, readdir, stat } from "fs/promises";
 import { EventEmitter } from "node:events";
@@ -14168,11 +14168,14 @@ function createImplementationOrchestrator(deps) {
 		let keepReviewing = true;
 		let timeoutRetried = false;
 		let previousIssueList = [];
+		let schemaValidationRetries = 0;
+		let previousIterationWasMalformed = false;
+		const MAX_SCHEMA_VALIDATION_RETRIES = 3;
 		while (keepReviewing) {
 			await waitIfPaused();
 			if (_state !== "RUNNING") return;
 			if (reviewCycles === 0) startPhase(storyKey, "code-review");
-			if (reviewCycles > 0) {
+			if (reviewCycles > 0 && !previousIterationWasMalformed) {
 				const currentRetries = _storyRetryCount.get(storyKey) ?? 0;
 				const budget = config.retryBudget ?? 2;
 				if (currentRetries >= budget) {
@@ -14197,6 +14200,7 @@ function createImplementationOrchestrator(deps) {
 				}
 				incrementRetryCount(storyKey);
 			}
+			previousIterationWasMalformed = false;
 			updateStory(storyKey, {
 				phase: "IN_REVIEW",
 				reviewCycles
@@ -14316,6 +14320,48 @@ function createImplementationOrchestrator(deps) {
 					}, "Failed to record code-review token usage"));
 				}
 				const isPhantomReview = reviewResult.dispatchFailed === true || reviewResult.verdict !== "SHIP_IT" && reviewResult.verdict !== "LGTM_WITH_NOTES" && (reviewResult.issue_list === void 0 || reviewResult.issue_list.length === 0) && reviewResult.error !== void 0;
+				const isMalformedOutput = isPhantomReview && reviewResult.error === "schema_validation_failed";
+				if (isMalformedOutput) {
+					schemaValidationRetries++;
+					eventBus.emit("orchestrator:code-review-output-malformed", {
+						storyKey,
+						reviewCycles,
+						attempt: schemaValidationRetries,
+						maxAttempts: MAX_SCHEMA_VALIDATION_RETRIES,
+						error: reviewResult.error ?? "schema_validation_failed",
+						...reviewResult.details !== void 0 ? { details: reviewResult.details } : {}
+					});
+					if (schemaValidationRetries <= MAX_SCHEMA_VALIDATION_RETRIES) {
+						logger$26.warn({
+							storyKey,
+							reviewCycles,
+							attempt: schemaValidationRetries,
+							error: reviewResult.error
+						}, "Code-review output malformed (schema validation failed) — retrying review");
+						previousIterationWasMalformed = true;
+						continue;
+					}
+					logger$26.warn({
+						storyKey,
+						reviewCycles,
+						schemaValidationRetries
+					}, "Code-review output malformed across MAX_SCHEMA_VALIDATION_RETRIES attempts — escalating");
+					endPhase(storyKey, "code-review");
+					updateStory(storyKey, {
+						phase: "ESCALATED",
+						error: "code-review-output-malformed-budget-exceeded",
+						completedAt: new Date().toISOString()
+					});
+					await writeStoryMetricsBestEffort(storyKey, "escalated", reviewCycles + 1);
+					await emitEscalation({
+						storyKey,
+						lastVerdict: "code-review-output-malformed-budget-exceeded",
+						reviewCycles: reviewCycles + 1,
+						issues: [`Code-review agent output failed schema validation ${schemaValidationRetries} consecutive times. The output was malformed (likely YAML parse error from unquoted colons, unbalanced quotes, or invalid escapes), not absent. Inspect the rawOutput on the manifest entry for the agent's emission shape and update the prompt or yaml-parser fallbacks.`]
+					});
+					await persistState();
+					return;
+				}
 				if (isPhantomReview && !timeoutRetried) {
 					timeoutRetried = true;
 					logger$26.warn({
@@ -45155,4 +45201,4 @@ function registerRunCommand(program, _version = "0.0.0", projectRoot = process.c
 
 //#endregion
 export { AdapterTelemetryPersistence, AppError, DoltRepoMapMetaRepository, DoltSymbolRepository, ERR_REPO_MAP_STORAGE_WRITE, EpicIngester, GitClient, GrammarLoader, RepoMapInjector, RepoMapModule, RepoMapQueryEngine, RepoMapStorage, SymbolParser, createContextCompiler, createDispatcher, createEventEmitter, createImplementationOrchestrator, createPackLoader, createPhaseOrchestrator, createStopAfterGate, createTelemetryAdvisor, formatPhaseCompletionSummary, getFactoryRunSummaries, getScenarioResultsForRun, getTwinRunsForRun, listGraphRuns, normalizeGraphSummaryToStatus, registerExportCommand, registerFactoryCommand, registerRunCommand, registerScenariosCommand, resolveMaxReviewCycles, resolveStoryKeys, runAnalysisPhase, runPlanningPhase, runRunAction, runSolutioningPhase, validateStopAfterFromConflict, wireNdjsonEmitter };
-//# sourceMappingURL=run-DGyWCmcu.js.map
+//# sourceMappingURL=run-BQxm1Qix.js.map

package/dist/{upgrade-CVaD7iUR.js → upgrade-BXfinu5K.js}

@@ -1,5 +1,5 @@
-import "./dist-CqtWS9wF.js";
+import "./dist-BnMsd9hC.js";
 import "./version-manager-impl-BmOWu8ml.js";
-import { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand } from "./upgrade-k15NGTno.js";
+import { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand } from "./upgrade-D3NuTkSL.js";
 
 export { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand };

package/dist/{upgrade-k15NGTno.js → upgrade-D3NuTkSL.js}

@@ -1,4 +1,4 @@
-import { createVersionManager } from "./dist-CqtWS9wF.js";
+import { createVersionManager } from "./dist-BnMsd9hC.js";
 import { execSync, spawn } from "child_process";
 import * as readline from "readline";
 
@@ -123,4 +123,4 @@ function registerUpgradeCommand(program) {
 
 //#endregion
 export { isGlobalInstall, registerUpgradeCommand, runUpgradeCommand };
-//# sourceMappingURL=upgrade-k15NGTno.js.map
+//# sourceMappingURL=upgrade-D3NuTkSL.js.map

package/dist/{version-manager-impl-CEiZEsmi.js → version-manager-impl-ChzfLw90.js}

@@ -1,4 +1,4 @@
-import { VersionManagerImpl, createVersionManager } from "./dist-CqtWS9wF.js";
+import { VersionManagerImpl, createVersionManager } from "./dist-BnMsd9hC.js";
 import "./version-manager-impl-BmOWu8ml.js";
 
 export { createVersionManager };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "substrate-ai",
-  "version": "0.20.32",
+  "version": "0.20.34",
   "description": "Substrate — multi-agent orchestration daemon for AI coding agents",
   "type": "module",
   "license": "MIT",

package/packs/bmad/prompts/code-review.md

@@ -57,7 +57,40 @@ Adversarial code review. Find what's wrong. Validate story claims against actual
 
 ## Output Contract
 
-After completing the review, emit ONLY raw YAML — no markdown fences, no ``` wrappers, no other text:
+After completing the review, emit ONLY raw YAML — no markdown fences, no ``` wrappers, no other text.
+
+### CRITICAL: Block-scalar form for free-form string fields
+
+When a finding's `description`, `message`, `command`, `notes`, or any other free-form string field may contain a colon (`:`), a quoted shell snippet, a file path with extension (`scripts/install.sh:`, `src/foo.ts:`), or any other text that could be interpreted as a YAML key-value separator, you MUST emit it as a YAML block scalar (`field: |`) followed by indented content. Block scalars don't interpret `:` specially, so the value's content can't be misparsed as a nested mapping.
+
+**Wrong** — unquoted scalar with internal colons (the YAML parser sees a key `description`, then sees `scripts/install.sh: copies ...` as a NEW key, fails with "bad indentation of a mapping entry"):
+
+```text
+issue_list:
+  - severity: minor
+    description: scripts/install.sh: copies Quadlet via 'cp ${QUADLET_SRC} ${QUADLET_DEST}': copies file
+    file: scripts/install.sh
+```
+
+**Right** — block scalar (`|`) preserves the value verbatim, colons and all:
+
+```yaml
+issue_list:
+  - severity: minor
+    description: |
+      scripts/install.sh: copies Quadlet via 'cp ${QUADLET_SRC} ${QUADLET_DEST}': copies file
+    file: scripts/install.sh
+```
+
+The `|` indicator is a literal block scalar (preserves newlines). Use `|-` if you want trailing newlines stripped. Either is safe.
+
+Quoted scalars (`"..."` and `'...'`) ARE technically able to contain colons — but they're brittle: any embedded quote of the matching kind closes the scalar early, leaving the rest of the value exposed to the parser. Block scalars have no closing delimiter to worry about, so they're the safer default whenever a string might contain code.
+
+**When in doubt — when a description quotes any code from the reviewed source — default to block-scalar form.** The cost is two extra characters; the benefit is that substrate's parser won't reject your output and trigger a false-positive escalation.
+
+### YAML format
+
+After the rules above, emit raw YAML:
 
 ```yaml
 verdict: SHIP_IT

package/dist/adapter-registry-N0Klnk7Y.js

@@ -1,4 +0,0 @@
-import { AdapterRegistry } from "./dist-CqtWS9wF.js";
-import "./adapter-registry-DXLMTmfD.js";
-
-export { AdapterRegistry };