glimmervoid 0.38.2 → 0.39.0

package/dist/chunks/{config-store-BUNZs4j7.js → config-store-n23JDU4E.js}

@@ -2,10 +2,8 @@ import { n as errorLabel, r as errorMessage } from "./text-DoirglWV.js";
 import { n as equalsIgnoringCaseOnWindows } from "./paths-Bu4eH1Yd.js";
 import { n as glimmervoidHomeDir$1, t as decideConfigPath } from "./config-path-core-DFt-TemS.js";
 import { n as isRecord } from "./coerce-NSst8oRP.js";
-import { A as FLYING_ANIMALS_DURATION_RANGE, B as POSTHOG_TRAFFIC_MIN_USERS_RANGE, D as CHANGE_MAP_NARRATOR_TIMEOUT_RANGE, E as normalizeLegacyConfigNumbers, F as NON_NEGATIVE_NUMBER_RANGE, G as USAGE_INTEGER_RANGES, H as REPLAY_BUFFER_KB_RANGE, I as PORT_RANGE, J as WORKFLOWS_MAX_ACTIONS_PER_POLL_RANGE, K as USAGE_RETAIN_DAYS_RANGE, L as POSITIVE_NUMBER_RANGE, M as FLYING_ANIMALS_SCALE_RANGE, N as KEEP_MERGEABLE_TIMEOUT_MINUTES_RANGE, O as FACTORY_MAX_LIVE_WORKERS_RANGE, P as MEMORY_RESERVE_MB_RANGE, R as POSTHOG_FIX_TIMEOUT_RANGE, U as TASK_TITLE_TIMEOUT_RANGE, V as POSTHOG_TRAFFIC_MULTIPLIER_RANGE, W as USAGE_BUDGET_RANGE, Y as WORKFLOWS_MAX_CONCURRENT_SESSIONS_RANGE, a as DEFAULT_FACTORY_CHECKS, j as FLYING_ANIMALS_GAP_RANGE, k as FLYING_ANIMALS_DEFAULTS, m as UPDATE_CHANNELS, n as BranchGcFileSettings, o as DEFAULT_FACTORY_PROTECTED_PATHS, q as USAGE_SCAN_INTERVAL_RANGE, r as CHANGE_MAP_NARRATOR_ENGINES, u as POST_TURN_CHECK_MODES, w as USAGE_COST_MODES, z as POSTHOG_TRAFFIC_BASELINE_RANGE } from "./browser-config-BrBT8269.js";
-import { b as DEFAULT_CHECK_MEMORY_RESERVE_MB, d as writeJsonAtomicSync, f as writeTextAtomic, p as writeTextAtomicSync } from "./json-file-CIJf0Cel.js";
-import { c as RUNTIME_CONFIG_SCALAR_KEYS, i as Config, n as BUILTIN_AGENT_IDS, t as AGENT_ID_SHAPE_MESSAGE, u as configIssueMessage } from "./config-CjawSOBk.js";
-import { K as TELEMETRY_KILL_SWITCH_FLAG, U as FeatureFlagsResponse } from "./contracts-HVCEJfxs.js";
+import { d as writeJsonAtomicSync, f as writeTextAtomic, p as writeTextAtomicSync } from "./json-file-DFCTvPjj.js";
+import { $ as Config, At as POSTHOG_TRAFFIC_MULTIPLIER_RANGE, Ct as MEMORY_RESERVE_MB_RANGE, Dt as POSTHOG_FIX_TIMEOUT_RANGE, Et as POSITIVE_NUMBER_RANGE, Ft as USAGE_RETAIN_DAYS_RANGE, G as FeatureFlagsResponse, It as USAGE_SCAN_INTERVAL_RANGE, J as TELEMETRY_KILL_SWITCH_FLAG, Lt as WORKFLOWS_MAX_ACTIONS_PER_POLL_RANGE, Mt as TASK_TITLE_TIMEOUT_RANGE, Nt as USAGE_BUDGET_RANGE, Ot as POSTHOG_TRAFFIC_BASELINE_RANGE, Pt as USAGE_INTEGER_RANGES, Rt as WORKFLOWS_MAX_CONCURRENT_SESSIONS_RANGE, St as KEEP_MERGEABLE_TIMEOUT_MINUTES_RANGE, Tt as PORT_RANGE, X as AGENT_ID_SHAPE_MESSAGE, Z as BUILTIN_AGENT_IDS, _t as CHANGE_MAP_NARRATOR_TIMEOUT_RANGE, at as BranchGcFileSettings, bt as FLYING_ANIMALS_GAP_RANGE, ct as UPDATE_CHANNELS, gt as normalizeLegacyConfigNumbers, it as configIssueMessage, jt as REPLAY_BUFFER_KB_RANGE, kt as POSTHOG_TRAFFIC_MIN_USERS_RANGE, mt as USAGE_COST_MODES, nt as RUNTIME_CONFIG_SCALAR_KEYS, ot as CHANGE_MAP_NARRATOR_ENGINES, st as POST_TURN_CHECK_MODES, vt as FLYING_ANIMALS_DEFAULTS, wt as NON_NEGATIVE_NUMBER_RANGE, xt as FLYING_ANIMALS_SCALE_RANGE, yt as FLYING_ANIMALS_DURATION_RANGE } from "./contracts-DYQW11de.js";
 import fs from "node:fs";
 import path from "node:path";
 import os from "node:os";
@@ -574,15 +572,6 @@ function decideIsDevInstall({ installFlavor, isBundled, homeDir, defaultHomeDir
 	if (installFlavor === "clone" || !isBundled) return true;
 	return comparablePath(homeDir) !== comparablePath(defaultHomeDir);
 }
-function buildFactoryDecisionEvent(decision) {
-	return {
-		kind: decision.kind,
-		outcome: decision.outcome,
-		failure_class: decision.failureClass,
-		duration_ms: decision.durationMs,
-		attempt: decision.attempt
-	};
-}
 function nodeMajorVersion(nodeVersion) {
 	const major = Number.parseInt(nodeVersion.replace(/^v/, ""), 10);
 	return Number.isFinite(major) ? major : 0;
@@ -632,10 +621,6 @@ var SESSIONS = {
 	label: "sessions",
 	storedPerDisplay: 1
 };
-var WORKERS = {
-	label: "workers",
-	storedPerDisplay: 1
-};
 var RECURRENCES = {
 	label: "recurrences",
 	storedPerDisplay: 1
@@ -711,7 +696,6 @@ var DIAGNOSTIC = "A diagnostic switch kept out of the Settings view; change it i
 var PROCESS_SETTING = "Changes how the server process or its worktrees behave, so it is set in config.json with a restart or new sessions in mind.";
 var STRUCTURED_VALUE = "A structured value with no dashboard control; edit it in config.json.";
 var BY_RULE = "Never editable from the dashboard by rule; edit it in config.json.";
-var LANES_SECTION_ID = "lanes";
 function capitalizedOptions(values) {
 	return values.map((value) => ({
 		value,
@@ -755,20 +739,6 @@ var SECTIONS = Object.freeze([
 				defaultValue: "phyrexian",
 				...BROWSER_NOW
 			},
-			{
-				id: "compact-status",
-				path: "pref:compactStatusLabels",
-				title: "Hide status words in sidebar",
-				description: "Hide status words in the sidebar and on the phone Board while keeping the colored status mark and the phone Plan ready badge.",
-				control: "toggle",
-				keywords: [
-					"sidebar",
-					"status",
-					"compact"
-				],
-				defaultValue: false,
-				...BROWSER_NOW
-			},
 			{
 				id: "session-usage-chips",
 				path: "pref:sessionUsageChips",
@@ -785,6 +755,23 @@ var SECTIONS = Object.freeze([
 				defaultValue: false,
 				...BROWSER_NOW
 			},
+			{
+				id: "phone-arrow-pad",
+				path: "pref:phoneArrowPad",
+				title: "Swipe arrow pad on phone (experimental)",
+				description: "Replace the four arrow keys above the phone keyboard with one pad: swipe on it to send arrow keys, one per short drag.",
+				control: "toggle",
+				keywords: [
+					"experimental",
+					"mobile",
+					"keys",
+					"arrows",
+					"swipe",
+					"gesture"
+				],
+				defaultValue: false,
+				...BROWSER_NOW
+			},
 			{
 				id: "flying-animals",
 				path: "pref:flyingAnimalsEnabled",
@@ -955,7 +942,7 @@ var SECTIONS = Object.freeze([
 				id: "telegram-bot-token",
 				path: "telegram.botToken",
 				title: "Bot token",
-				description: "Credential used by the shared Telegram bot. When GLIMMERVOID_TELEGRAM_BOT_TOKEN supplies it, this field is read-only.",
+				description: "Credential used by the shared Telegram bot. Create a bot by messaging @BotFather in Telegram and sending /newbot; it replies with this token. When GLIMMERVOID_TELEGRAM_BOT_TOKEN supplies it, this field is read-only.",
 				control: "password",
 				keywords: ["credential", "api"],
 				defaultValue: "",
@@ -965,7 +952,7 @@ var SECTIONS = Object.freeze([
 				id: "telegram-chat-id",
 				path: "telegram.chatId",
 				title: "Chat id",
-				description: "Destination chat for enabled Telegram lanes.",
+				description: "Destination chat for enabled Telegram lanes. Send your bot any message, then open https://api.telegram.org/bot<token>/getUpdates in a browser and copy the number under message.chat.id.",
 				control: "text",
 				keywords: ["destination", "channel"],
 				defaultValue: "",
@@ -977,8 +964,13 @@ var SECTIONS = Object.freeze([
 				title: "Send session notifications",
 				description: "Send session completion, input and failure alerts when no dashboard is open. Unacknowledged alerts also escalate after the phone escalation delay while a dashboard remains open.",
 				control: "toggle",
-				keywords: ["phone", "off dashboard"],
+				keywords: [
+					"phone",
+					"off dashboard",
+					"test"
+				],
 				defaultValue: false,
+				status: "telegram-delivery",
 				...MACHINE_NOW
 			}
 		]
@@ -1312,20 +1304,13 @@ var SECTIONS = Object.freeze([
 			}
 		]
 	},
-	{
-		id: LANES_SECTION_ID,
-		level: "lanes",
-		title: "Lanes",
-		description: "The background work Glimmervoid runs for you, one sub-section per lane.",
-		settings: []
-	},
 	{
 		id: "lanes-visions",
 		level: "lanes",
-		parent: LANES_SECTION_ID,
 		title: "Visions",
 		description: "Editor-buffer findings, bounded model comments, and the machine-context ingest behind them.",
 		legacyIds: [
+			"lanes",
 			"visions",
 			"lanes-ingest",
 			"ingest"
@@ -1418,7 +1403,6 @@ var SECTIONS = Object.freeze([
 	{
 		id: "lanes-posthog",
 		level: "lanes",
-		parent: LANES_SECTION_ID,
 		title: "PostHog",
 		description: "Error monitoring, investigations and traffic-spike alerts.",
 		legacyIds: ["posthog"],
@@ -1531,7 +1515,6 @@ var SECTIONS = Object.freeze([
 	{
 		id: "lanes-team-review",
 		level: "lanes",
-		parent: LANES_SECTION_ID,
 		title: "Team review",
 		description: "Draft reviews of teammates' pull requests for you to post from the Reviews tab.",
 		legacyIds: ["team-review", "reviews"],
@@ -1666,7 +1649,6 @@ var SECTIONS = Object.freeze([
 	{
 		id: "lanes-workflows",
 		level: "lanes",
-		parent: LANES_SECTION_ID,
 		title: "Workflows",
 		description: "Rules that act on pull request events in the repositories you choose.",
 		legacyIds: ["workflows"],
@@ -1714,156 +1696,9 @@ var SECTIONS = Object.freeze([
 			}
 		]
 	},
-	{
-		id: "lanes-factory",
-		level: "lanes",
-		parent: LANES_SECTION_ID,
-		title: "Factory",
-		description: "Read Coherence reports from repositories whose integration branch holds a coherence.config.json and show their status in the Factory tab.",
-		legacyIds: ["factory"],
-		settings: [
-			{
-				id: "factory-enabled",
-				path: "factory.enabled",
-				title: "Enable the factory",
-				description: "Enable Coherence status polling and show the Factory tab. Off by default.",
-				control: "toggle",
-				keywords: [
-					"orchestrator",
-					"intents",
-					"queue",
-					"coherence",
-					"guardrails"
-				],
-				defaultValue: false,
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-max-risk",
-				path: "factory.maxRisk",
-				title: "Worker risk ceiling",
-				description: "Highest risk admitted for a ticket.",
-				control: "select",
-				defaultValue: "medium",
-				options: [
-					"low",
-					"medium",
-					"high",
-					"critical"
-				].map((value) => ({
-					value,
-					label: value
-				})),
-				keywords: ["risk", "admission"],
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-max-live-workers",
-				path: "factory.maxLiveWorkers",
-				title: "Live worker cap",
-				description: "Most live workers per factory project.",
-				control: "number",
-				range: FACTORY_MAX_LIVE_WORKERS_RANGE,
-				unit: WORKERS,
-				defaultValue: 2,
-				keywords: ["workers", "parallel"],
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-checks",
-				path: "factory.checks",
-				title: "Worker checks",
-				description: "Commands glimmervoid runs before merging worker commits.",
-				control: "list",
-				defaultValue: [
-					"npm run typecheck",
-					"npm run lint",
-					"npm test"
-				],
-				keywords: ["tests", "checks"],
-				listPlaceholder: "npm test",
-				listEmptyText: "No checks run before a merge.",
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-protected-paths",
-				path: "factory.protectedPaths",
-				title: "Protected paths",
-				description: "Repo-relative files, directory prefixes ending in /, or **/NAME basenames workers cannot change.",
-				control: "list",
-				defaultValue: [
-					"coherence.config.json",
-					".coherence/",
-					"package.json",
-					"package-lock.json",
-					"**/AGENTS.md",
-					"**/CLAUDE.md",
-					"**/CLAUDE.local.md",
-					".claude/",
-					".github/"
-				],
-				keywords: ["fence", "scope"],
-				listPlaceholder: "path, prefix/ or **/NAME",
-				listEmptyText: "Workers may change any path.",
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-reviewer-model",
-				path: "factory.reviewerModel",
-				title: "Reviewer model",
-				description: "Claude model for independent worker review. Blank uses the CLI default.",
-				control: "text",
-				defaultValue: null,
-				nullable: true,
-				keywords: ["claude", "review"],
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-daily-budget",
-				path: "factory.dailyBudgetUsd",
-				title: "Factory daily budget",
-				description: "Factory lane spend ceiling. Blank means no ceiling. Needs usage tracking: with it off, dispatch refuses.",
-				control: "number",
-				range: NON_NEGATIVE_NUMBER_RANGE,
-				unit: USD,
-				integer: false,
-				nullable: true,
-				step: .01,
-				defaultValue: null,
-				keywords: ["spend", "budget"],
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-verifier-model",
-				path: "factory.verifierModel",
-				title: "Verifier model",
-				description: "Claude model for independent intent verification. Blank uses the CLI default.",
-				control: "text",
-				nullable: true,
-				defaultValue: null,
-				keywords: ["claude", "verification"],
-				...MACHINE_NOW
-			},
-			{
-				id: "factory-mirror-harness-runs",
-				path: "factory.mirrorHarnessRuns",
-				title: "Mirror harness runs",
-				description: "Show the latest factory harness run as a read-only project in the Factory tab while it runs and for a day after. Needs factory.enabled on. The harness turns it off on the instance it boots.",
-				control: "toggle",
-				defaultValue: true,
-				keywords: [
-					"harness",
-					"test",
-					"read-only"
-				],
-				...MACHINE_NOW
-			}
-		]
-	},
 	{
 		id: "lanes-benchmarks",
 		level: "lanes",
-		parent: LANES_SECTION_ID,
 		title: "Benchmarks",
 		description: "Compare agent setups on frozen cases from the suites in the benchmarks folder of the Glimmervoid home.",
 		legacyIds: ["benchmarks"],
@@ -1884,25 +1719,21 @@ var SECTIONS = Object.freeze([
 		}]
 	},
 	{
-		id: "lanes-knowledge-graph",
+		id: "lanes-coherence",
 		level: "lanes",
-		parent: LANES_SECTION_ID,
-		title: "Knowledge graph (experimental)",
-		description: "A typed local graph of projects, tasks, notes, references and questions, driven from the glimmervoid kg command. Its SQLite database defaults to the Glimmervoid home; --db selects another path.",
-		legacyIds: ["knowledge-graph"],
+		title: "Coherence (experimental)",
+		description: "Per-session Coherence hook status and decision journal for agent sessions.",
 		settings: [{
-			id: "knowledge-graph-enabled",
-			path: "knowledgeGraph.enabled",
-			title: "Enable the knowledge graph",
-			description: "Allow the glimmervoid kg command. It reads Coherence ledgers to report where your tasks and the agents' tickets disagree, without modifying those ledgers. Graph writes go to the selected SQLite database, which can be inside a repository when --db names that path. Off by default.",
+			id: "coherence-enabled",
+			path: "coherence.enabled",
+			title: "Show session Coherence",
+			description: "Shows each session's Coherence hook status and decision journal when the repository installed @posthog/coherence 1.8 or newer. Off by default.",
 			control: "toggle",
 			keywords: [
 				"experimental",
-				"kg",
-				"tasks",
-				"notes",
-				"references",
-				"coherence"
+				"coherence",
+				"journal",
+				"decisions"
 			],
 			defaultValue: false,
 			...MACHINE_NOW
@@ -1911,7 +1742,6 @@ var SECTIONS = Object.freeze([
 	{
 		id: "lanes-change-map",
 		level: "lanes",
-		parent: LANES_SECTION_ID,
 		title: "Change map",
 		description: "Optional model narration for deterministic change facts.",
 		legacyIds: ["change-map"],
@@ -1963,7 +1793,7 @@ var SECTIONS = Object.freeze([
 			id: "telemetry-enabled",
 			path: "telemetry.enabled",
 			title: "Send anonymous usage and error data",
-			description: "Version, platform, app starts, daily activity, session starts and ends with agent kind, exit kind and duration, errors as their type, error code and scrubbed stack frames, per session token counts, models and estimated costs from the usage scan, and factory outcomes as the kind of decision, how it came out, its duration and attempt number, under a random install id. Never error messages, paths, repository or branch names, prompts, objectives or terminal output. GLIMMERVOID_TELEMETRY=0, DO_NOT_TRACK=1 or CI=true turn it off regardless.",
+			description: "Version, platform, app starts, daily activity, session starts and ends with agent kind, exit kind and duration, errors as their type, error code and scrubbed stack frames, per session token counts, models and estimated costs from the usage scan, under a random install id. Never error messages, paths, repository or branch names, prompts, objectives or terminal output. GLIMMERVOID_TELEMETRY=0, DO_NOT_TRACK=1 or CI=true turn it off regardless.",
 			control: "toggle",
 			keywords: [
 				"telemetry",
@@ -2034,24 +1864,6 @@ var FILE_ONLY_SETTINGS = Object.freeze([
 		fileOnlyReason: TUNING_KNOB,
 		...FILE_ONLY
 	},
-	{
-		id: "file-check-memory-reserve",
-		path: "checkMemoryReserveMB",
-		title: "Check run memory reserve",
-		group: "machine-sessions",
-		description: "Free memory a factory check run waits for; 0 runs checks without checking.",
-		control: "number",
-		range: MEMORY_RESERVE_MB_RANGE,
-		unit: {
-			label: "MB",
-			storedPerDisplay: 1
-		},
-		keywords: ["memory", "factory"],
-		defaultValue: 1500,
-		takesEffect: "now",
-		fileOnlyReason: TUNING_KNOB,
-		...FILE_ONLY
-	},
 	{
 		id: "file-cursor-blink",
 		path: "cursorBlink",
@@ -3544,7 +3356,6 @@ var DEFAULT_CONFIG = {
 	detectScheduledWakeups: true,
 	replayBufferKB: 512,
 	sessionMemoryReserveMB: 600,
-	checkMemoryReserveMB: DEFAULT_CHECK_MEMORY_RESERVE_MB,
 	recordSignals: true,
 	trace: { enabled: true },
 	planReview: { enabled: true },
@@ -3560,17 +3371,6 @@ var DEFAULT_CONFIG = {
 		model: "",
 		timeoutSeconds: 90
 	} },
-	factory: {
-		enabled: false,
-		maxRisk: "medium",
-		maxLiveWorkers: 2,
-		checks: DEFAULT_FACTORY_CHECKS,
-		protectedPaths: DEFAULT_FACTORY_PROTECTED_PATHS,
-		reviewerModel: null,
-		dailyBudgetUsd: null,
-		verifierModel: null,
-		mirrorHarnessRuns: true
-	},
 	guardrails: {
 		unguardedRepos: [],
 		blockedPaths: [],
@@ -3649,7 +3449,7 @@ function resolveConfigPath() {
 	}, (candidate) => fs.existsSync(candidate));
 	if (decided.path) return decided.path;
 	if (decided.source === "env") {
-		console.error(`Config file not found: ${decided.envPath}`);
+		console.error(`Config file not found: ${decided.envPath}, the path GLIMMERVOID_CONFIG names. Create that file, or unset GLIMMERVOID_CONFIG to use ${decided.homePath}.`);
 		process.exit(1);
 	}
 	const homeConfig = decided.homePath;
@@ -3795,11 +3595,14 @@ function isSuspectedExternalWipe(candidate, currentConfig) {
 		branchGc: resolveBranchGc(candidate.branchGc)
 	}) * 2 < currentKeyCount;
 }
-function warnInvalidConfig(action, validation) {
-	console.warn(`[config] Refusing to ${action}; validation failed: ${validation.errors.join("; ")}. Recovery sources: config.json.bak and config.json.boot.bak.`);
+function invalidConfigWarning(action, validation) {
+	return `[config] Refusing to ${action}; validation failed: ${validation.errors.join("; ")}. Recovery sources: config.json.bak and config.json.boot.bak.`;
+}
+function suspectedWipeWarning(action) {
+	return `[config] Refusing to ${action}; config.json has fewer than half the top-level keys of the in-memory config. This looks like an external wipe. Recovery sources: config.json.bak and config.json.boot.bak.`;
 }
 function warnSuspectedWipe(action) {
-	console.warn(`[config] Refusing to ${action}; config.json has fewer than half the top-level keys of the in-memory config. This looks like an external wipe. Recovery sources: config.json.bak and config.json.boot.bak.`);
+	console.warn(suspectedWipeWarning(action));
 }
 function ensureProjectIds(projects) {
 	let changed = false;
@@ -3879,36 +3682,30 @@ function createConfigStore({ settingsDefaults } = {}) {
 	let synchronousWriteRevision = 0;
 	let lastSynchronousContent = null;
 	let pendingSaves = Promise.resolve();
+	let lastSaveFailureReason = null;
+	function refuseSave(warning, operatorReason = warning.replace(/^\[config\] /, "")) {
+		console.warn(warning);
+		lastSaveFailureReason = operatorReason;
+		return null;
+	}
 	function prepareSave(freshConfig, mutatorFn) {
 		const freshValidation = validateConfig(freshConfig);
-		if (!freshValidation.ok) {
-			warnInvalidConfig("save config.json", freshValidation);
-			return null;
-		}
-		if (isSuspectedExternalWipe(freshConfig, config)) {
-			warnSuspectedWipe("save config.json");
-			return null;
-		}
+		if (!freshValidation.ok) return refuseSave(invalidConfigWarning("save config.json", freshValidation));
+		if (isSuspectedExternalWipe(freshConfig, config)) return refuseSave(suspectedWipeWarning("save config.json"));
 		if (mutatorFn(freshConfig) === "abort-config-save") return null;
 		const mutatedValidation = validateConfig(withEnvSecrets(freshConfig, envSecrets));
-		if (!mutatedValidation.ok) {
-			warnInvalidConfig("save config.json", mutatedValidation);
-			return null;
-		}
+		if (!mutatedValidation.ok) return refuseSave(invalidConfigWarning("save config.json", mutatedValidation));
 		return mutatedValidation.config;
 	}
 	function save(mutatorFn) {
+		lastSaveFailureReason = null;
 		let loaded;
 		try {
 			loaded = loadConfigFile(configPath, { exitOnError: false });
 		} catch (err) {
-			console.warn("[config] Failed to read config.json for save:", errorLabel(err));
-			return null;
-		}
-		if ("error" in loaded) {
-			console.warn(loaded.message);
-			return null;
+			return refuseSave(`[config] Failed to read config.json for save: ${errorLabel(err)}`);
 		}
+		if ("error" in loaded) return refuseSave(loaded.message, `${configPath} could not be read as a config file. The broken file was copied to ${loaded.invalidBackupPath} when possible. Fix it, or restore from ${configPath}.bak, then save again.`);
 		const freshConfig = loaded.config;
 		const freshContent = loaded.loadedContent;
 		const effectiveConfig = prepareSave(freshConfig, mutatorFn);
@@ -3922,8 +3719,7 @@ function createConfigStore({ settingsDefaults } = {}) {
 			lastSynchronousContent = nextContent;
 			synchronousWriteRevision++;
 		} catch (err) {
-			console.warn("[config] Failed to write config.json:", errorLabel(err));
-			return null;
+			return refuseSave(`[config] Failed to write config.json: ${errorLabel(err)}`);
 		}
 		return effectiveConfig;
 	}
@@ -4028,8 +3824,7 @@ function createConfigStore({ settingsDefaults } = {}) {
 			github: pickBlock("github", config.github),
 			teamReview: pickBlock("teamReview", config.teamReview),
 			benchmarks: pickBlock("benchmarks", config.benchmarks),
-			factory: pickBlock("factory", config.factory),
-			knowledgeGraph: pickBlock("knowledgeGraph", config.knowledgeGraph),
+			coherence: pickBlock("coherence", config.coherence),
 			workflows: config.workflows ?? null,
 			posthog: pickBlock("posthog", config.posthog),
 			secretSources: secretSourcesOf(process.env),
@@ -4076,8 +3871,7 @@ function createConfigStore({ settingsDefaults } = {}) {
 		if (newConfig.github != null) config.github = newConfig.github;
 		if (newConfig.teamReview != null) config.teamReview = newConfig.teamReview;
 		if (newConfig.benchmarks != null) config.benchmarks = newConfig.benchmarks;
-		if (newConfig.factory != null) config.factory = newConfig.factory;
-		if (newConfig.knowledgeGraph != null) config.knowledgeGraph = newConfig.knowledgeGraph;
+		if (newConfig.coherence != null) config.coherence = newConfig.coherence;
 		if (newConfig.posthog != null) config.posthog = newConfig.posthog;
 		if (newConfig.usage != null) config.usage = newConfig.usage;
 		if (newConfig.telegram != null) config.telegram = newConfig.telegram;
@@ -4147,6 +3941,7 @@ function createConfigStore({ settingsDefaults } = {}) {
 		configPath,
 		save,
 		saveAsync,
+		lastSaveFailureReason: () => lastSaveFailureReason,
 		idle: () => pendingSaves,
 		getSettings,
 		applySettings,
@@ -4156,4 +3951,4 @@ function createConfigStore({ settingsDefaults } = {}) {
 	};
 }
 //#endregion
-export { withEnvSecrets as $, settingsOf as A, buildExceptionProperties as B, deleteValueAtPath as C, scoreSettingsSearch as D, parsePosthogProjectSelection as E, TELEMETRY_BATCH_URL as F, decideTelemetryConsent as G, classifySessionExit as H, TELEMETRY_FLAGS_URL as I, parsePosthogReleaseId as J, exceptionFingerprint as K, adapterBucket as L, FILE_ONLY_GROUPS as M, SETTINGS_REGISTRY as N, setValueAtPath as O, FIRST_RUN_NOTICE as P, stableConfigKey as Q, buildAiGenerationEvents as R, cloneValue as S, hasScaledUnit as T, decideIsDevInstall as U, buildFactoryDecisionEvent as V, decideRemoteTelemetryState as W, ENV_SECRET_BINDINGS as X, resolveProjectToken as Y, readEnvSecrets as Z, SECRET_PRESENCE_SUFFIX as _, generateProjectId as a, usablePrefixes as at, validateSettingsBlock as b, resolveConfigPath as c, comparableDirectoryPath as ct, assetForPlatform as d, DEFAULT_BRANCH_GC_PREFIXES as et, decideRtkInstall as f, verifyDigest as g, isRtkBinaryName as h, ensureProjectIds as i, sessionIdFromBranch as it, valueAtPath as j, settingsGroupOf as k, saveConfigFileOnce as l, isSameDirectoryPath as lt, installTargetPath as m, DEFAULT_CONFIG as n, planBranchGc as nt, glimmervoidHomeDir as o, worktreeIntegrationTips as ot, findEscapingArchiveMember as p, nodeMajorVersion as q, createConfigStore as r, planWorktreeGc as rt, loadConfigFile as s, canonicalizePath as st, ABORT_CONFIG_SAVE as t, matchedPrefix as tt, validateConfig as u, mergeSettingsBlock as v, displayText as w, SETTINGS_INDEX as x, settingsBlockSpecFor as y, buildBrowserExceptionProperties as z };
+export { DEFAULT_BRANCH_GC_PREFIXES as $, settingsOf as A, buildExceptionProperties as B, deleteValueAtPath as C, scoreSettingsSearch as D, parsePosthogProjectSelection as E, TELEMETRY_BATCH_URL as F, exceptionFingerprint as G, decideIsDevInstall as H, TELEMETRY_FLAGS_URL as I, resolveProjectToken as J, nodeMajorVersion as K, adapterBucket as L, FILE_ONLY_GROUPS as M, SETTINGS_REGISTRY as N, setValueAtPath as O, FIRST_RUN_NOTICE as P, withEnvSecrets as Q, buildAiGenerationEvents as R, cloneValue as S, hasScaledUnit as T, decideRemoteTelemetryState as U, classifySessionExit as V, decideTelemetryConsent as W, readEnvSecrets as X, ENV_SECRET_BINDINGS as Y, stableConfigKey as Z, SECRET_PRESENCE_SUFFIX as _, generateProjectId as a, worktreeIntegrationTips as at, validateSettingsBlock as b, resolveConfigPath as c, isSameDirectoryPath as ct, assetForPlatform as d, matchedPrefix as et, decideRtkInstall as f, verifyDigest as g, isRtkBinaryName as h, ensureProjectIds as i, usablePrefixes as it, valueAtPath as j, settingsGroupOf as k, saveConfigFileOnce as l, installTargetPath as m, DEFAULT_CONFIG as n, planWorktreeGc as nt, glimmervoidHomeDir as o, canonicalizePath as ot, findEscapingArchiveMember as p, parsePosthogReleaseId as q, createConfigStore as r, sessionIdFromBranch as rt, loadConfigFile as s, comparableDirectoryPath as st, ABORT_CONFIG_SAVE as t, planBranchGc as tt, validateConfig as u, mergeSettingsBlock as v, displayText as w, SETTINGS_INDEX as x, settingsBlockSpecFor as y, buildBrowserExceptionProperties as z };