@oisincoveney/pipeline 3.24.2 → 3.24.3

package/dist/bench/eval-report.js

@@ -1,12 +1,14 @@
+import { booleanValue, mutableArray, numberValue, parseJsonWithSchema, requiredString, struct } from "../schema-boundary.js";
 //#region src/bench/eval-report.ts
-function buildEvalReport(results) {
-	const variants = [...new Set(results.map((r) => r.variant))].sort();
-	return {
-		tasks: new Set(results.map((r) => r.task)).size,
-		variants: variants.map((variant) => summarizeVariant(variant, results.filter((r) => r.variant === variant)))
-	};
-}
-function summarizeVariant(variant, runs) {
+const evalRunResultsSchema = mutableArray(struct({
+	costTokens: numberValue(),
+	resolved: booleanValue(),
+	task: requiredString,
+	variant: requiredString,
+	wallMs: numberValue()
+}));
+const parseEvalRunResultsJson = (source) => parseJsonWithSchema(evalRunResultsSchema, source);
+const summarizeVariant = (variant, runs) => {
 	const resolved = runs.filter((r) => r.resolved).length;
 	const totalWall = runs.reduce((sum, r) => sum + r.wallMs, 0);
 	return {
@@ -17,11 +19,18 @@ function summarizeVariant(variant, runs) {
 		totalCostTokens: runs.reduce((sum, r) => sum + r.costTokens, 0),
 		variant
 	};
-}
-function renderEvalReport(report) {
+};
+const buildEvalReport = (results) => {
+	const variants = [...new Set(results.map((r) => r.variant))].toSorted();
+	return {
+		tasks: new Set(results.map((r) => r.task)).size,
+		variants: variants.map((variant) => summarizeVariant(variant, results.filter((r) => r.variant === variant)))
+	};
+};
+const renderEvalReport = (report) => {
 	const lines = [`Eval over ${report.tasks} task(s):`, "variant | resolved | rate | tokens | avg ms"];
 	for (const v of report.variants) lines.push(`${v.variant} | ${v.resolved}/${v.count} | ${(v.resolutionRate * 100).toFixed(0)}% | ${v.totalCostTokens} | ${v.avgWallMs}`);
 	return lines.join("\n");
-}
+};
 //#endregion
-export { buildEvalReport, renderEvalReport };
+export { buildEvalReport, parseEvalRunResultsJson, renderEvalReport };

package/dist/claude-settings-config.js

@@ -1,7 +1,27 @@
 import { applyJsonEdit, ensureTrailingNewline, formatJson, isRecord, parseJsonRecord, setIfMissing } from "./json-config-merge.js";
 //#region src/claude-settings-config.ts
-function mergeClaudeSettings(currentText, projection) {
-	if (currentText === void 0) return {
+const applyMcpServersProjection = (content, parsed, projection) => Object.entries(projection.mcpServers ?? {}).reduce((nextContent, [name, server]) => setIfMissing(nextContent, parsed, ["mcpServers", name], server), content);
+const projectedAllowList = (projection) => projection.permissions?.allow ?? [];
+const unionPreservingOrder = (existing, extra) => {
+	const merged = [...existing];
+	for (const entry of extra) if (!merged.includes(entry)) merged.push(entry);
+	return merged;
+};
+const sameOrderedList = (a, b) => a.length === b.length && a.every((value, index) => value === b[index]);
+const permissionAllowList = (parsed) => {
+	const { permissions } = parsed;
+	if (!isRecord(permissions)) return [];
+	const { allow } = permissions;
+	return Array.isArray(allow) ? allow.filter((entry) => typeof entry === "string") : [];
+};
+const applyPermissionsAllow = (content, parsed, projection) => {
+	const existing = permissionAllowList(parsed);
+	const merged = unionPreservingOrder(existing, projectedAllowList(projection));
+	return sameOrderedList(existing, merged) ? content : applyJsonEdit(content, ["permissions", "allow"], merged);
+};
+const applyClaudeProjection = (currentText, parsed, projection) => applyPermissionsAllow(applyMcpServersProjection(currentText, parsed, projection), parsed, projection);
+const mergeClaudeSettings = (currentText = "", projection) => {
+	if (currentText === "") return {
 		content: formatJson(projection),
 		ok: true
 	};
@@ -11,34 +31,6 @@ function mergeClaudeSettings(currentText, projection) {
 		content: ensureTrailingNewline(applyClaudeProjection(currentText, parsed.value, projection)),
 		ok: true
 	};
-}
-function applyClaudeProjection(currentText, parsed, projection) {
-	return applyPermissionsAllow(applyMcpServersProjection(currentText, parsed, projection), parsed, projection);
-}
-function applyMcpServersProjection(content, parsed, projection) {
-	return Object.entries(projection.mcpServers ?? {}).reduce((nextContent, [name, server]) => setIfMissing(nextContent, parsed, ["mcpServers", name], server), content);
-}
-function applyPermissionsAllow(content, parsed, projection) {
-	const existing = permissionAllowList(parsed);
-	const merged = unionPreservingOrder(existing, projectedAllowList(projection));
-	return sameOrderedList(existing, merged) ? content : applyJsonEdit(content, ["permissions", "allow"], merged);
-}
-function projectedAllowList(projection) {
-	return projection.permissions?.allow ?? [];
-}
-function unionPreservingOrder(existing, extra) {
-	const merged = [...existing];
-	for (const entry of extra) if (!merged.includes(entry)) merged.push(entry);
-	return merged;
-}
-function sameOrderedList(a, b) {
-	return a.length === b.length && a.every((value, index) => value === b[index]);
-}
-function permissionAllowList(parsed) {
-	const permissions = parsed.permissions;
-	if (!isRecord(permissions)) return [];
-	const allow = permissions.allow;
-	return Array.isArray(allow) ? allow.filter((entry) => typeof entry === "string") : [];
-}
+};
 //#endregion
 export { mergeClaudeSettings };

package/dist/claude-user-config.js

@@ -1,28 +1,18 @@
 import { applyJsonEdit, ensureTrailingNewline, formatJson, parseJsonRecord, setIfMissing } from "./json-config-merge.js";
 //#region src/claude-user-config.ts
-function mergeClaudeUserConfig(currentText, projection) {
-	if (currentText === void 0) return {
+const mergeClaudeUserConfigWith = (projection, project, currentText = "") => {
+	if (currentText === "") return {
 		content: formatJson(projection),
 		ok: true
 	};
 	const parsed = parseJsonRecord(currentText);
 	if (!parsed.ok) return parsed;
 	return {
-		content: ensureTrailingNewline(Object.entries(projection.mcpServers ?? {}).reduce((nextContent, [name, server]) => setIfMissing(nextContent, parsed.value, ["mcpServers", name], server), currentText)),
+		content: ensureTrailingNewline(project(currentText, parsed.value, projection)),
 		ok: true
 	};
-}
-function replaceClaudeUserMcpServers(currentText, projection) {
-	if (currentText === void 0) return {
-		content: formatJson(projection),
-		ok: true
-	};
-	const parsed = parseJsonRecord(currentText);
-	if (!parsed.ok) return parsed;
-	return {
-		content: ensureTrailingNewline(applyJsonEdit(currentText, ["mcpServers"], projection.mcpServers ?? {})),
-		ok: true
-	};
-}
+};
+const mergeClaudeUserConfig = (projection, currentText = "") => mergeClaudeUserConfigWith(projection, (text, parsed, value) => Object.entries(value.mcpServers ?? {}).reduce((nextContent, [name, server]) => setIfMissing(nextContent, parsed, ["mcpServers", name], server), text), currentText);
+const replaceClaudeUserMcpServers = (projection, currentText = "") => mergeClaudeUserConfigWith(projection, (text, _parsed, value) => applyJsonEdit(text, ["mcpServers"], value.mcpServers ?? {}), currentText);
 //#endregion
 export { mergeClaudeUserConfig, replaceClaudeUserMcpServers };

package/dist/cli/bootstrap-commands.js

@@ -4,10 +4,10 @@ import { runDoctor } from "./doctor.js";
 import { formatDoctorResult } from "./format.js";
 import { resolve } from "node:path";
 //#region src/cli/bootstrap-commands.ts
-function registerBootstrapCommands(program) {
+const registerBootstrapCommands = (program) => {
 	program.command("doctor").description("Check local prerequisites for pipeline init and execution").option("--cluster [namespace]", "also check runner-job Kubernetes prerequisites").option("--json", "print machine-readable readiness results").option("--kube-context <context>", "kubectl context for cluster checks").option("--kubeconfig <path>", "kubeconfig path for cluster checks").action(async (flags) => {
 		const result = await runDoctor(process.env.PIPELINE_TARGET_PATH ?? process.cwd(), flags);
-		console.log(flags.json ? JSON.stringify(result) : formatDoctorResult(result));
+		console.log(flags.json === true ? JSON.stringify(result) : formatDoctorResult(result));
 		if (!result.passed) throw new Error("Doctor checks failed.");
 	});
 	program.command("init").description([
@@ -34,6 +34,6 @@ function registerBootstrapCommands(program) {
 		console.log(formatCodexAuthSyncResult(result));
 		if (!result.ok) process.exitCode = 1;
 	});
-}
+};
 //#endregion
 export { registerBootstrapCommands };

package/dist/cli/doctor.d.ts

@@ -16,6 +16,6 @@ interface DoctorFlags {
   kubeContext?: string;
   kubeconfig?: string;
 }
-declare function runDoctor(cwd: string, options?: DoctorFlags): Promise<DoctorResult>;
+declare const runDoctor: (cwd: string, options?: DoctorFlags) => Promise<DoctorResult>;
 //#endregion
 export { DoctorCheck, DoctorFlags, DoctorResult, runDoctor };

package/dist/cli/doctor.js

@@ -2,6 +2,7 @@ import { PipelineConfigError } from "../config/schemas.js";
 import { loadPipelineConfig } from "../config/load.js";
 import "../config.js";
 import { loadMokaGlobalConfig } from "../moka-global-config.js";
+import { isRecord } from "../safe-json.js";
 import { opencodeAgentName } from "../runtime/opencode-agent-name.js";
 import { defaultClusterDoctorNamespace, runClusterDoctor } from "../cluster-doctor.js";
 import { existsSync, readFileSync, readdirSync, statSync } from "node:fs";
@@ -23,50 +24,123 @@ const OPENCODE_AGENT_LIST_ARGS = [
 	"list",
 	"--json"
 ];
-const BULLET_PREFIX_RE = /^[-*]\s+/;
-const LINE_RE = /\r?\n/;
-async function runDoctor(cwd, options = {}) {
-	const commandChecks = await Promise.all([
-		checkCommand("npx", ["--version"], cwd),
-		checkCommand("opencode", ["--version"], cwd),
-		checkCommand("fallow", ["--version"], cwd)
-	]);
-	const configCheck = checkPipelineConfig(cwd);
-	const config = configCheck.passed ? loadPipelineConfig(cwd) : null;
-	const [sdkCheck, agentVisibility] = await Promise.all([checkOpenCodeSdk(), config ? checkMokaAgents(cwd, config) : Promise.resolve({ check: {
-		detail: "skipped because pipeline config is invalid",
+const BULLET_PREFIX_RE = /^[-*]\s+/u;
+const LINE_RE = /\r?\n/u;
+const clusterNamespace = (value, configuredNamespace) => typeof value === "string" && value.length > 0 ? value : configuredNamespace ?? defaultClusterDoctorNamespace();
+const checkPipelineConfig = (cwd) => {
+	try {
+		loadPipelineConfig(cwd);
+		return {
+			detail: "valid",
+			name: "pipeline-config",
+			passed: true
+		};
+	} catch (error) {
+		let message = "invalid";
+		if (error instanceof PipelineConfigError) message = error.issues.map((issue) => issue.message).join("; ");
+		else if (error instanceof Error) message = error.message;
+		return {
+			detail: message || "missing or invalid",
+			name: "pipeline-config",
+			passed: false
+		};
+	}
+};
+const skippedAgentVisibility = (detail) => ({
+	check: {
+		detail: "skipped because OpenCode agent listing is unavailable",
 		name: "moka-agents",
 		passed: true
-	} })]);
-	const globalConfig = loadMokaGlobalConfig();
-	const clusterResult = options.cluster ? await runClusterDoctor({
-		kubeContext: options.kubeContext,
-		kubeconfigPath: options.kubeconfig ?? globalConfig?.momokaya.kubernetes.kubeconfig,
-		namespace: clusterNamespace(options.cluster, globalConfig?.momokaya.kubernetes.namespace)
-	}) : { checks: [] };
-	const checks = [
-		...commandChecks,
-		configCheck,
-		sdkCheck,
-		agentVisibility.check,
-		...clusterResult.checks
-	];
-	const warnings = [...agentVisibility.warning ? [agentVisibility.warning] : [], ...headlessPermissionWarnings(cwd)];
-	const blockers = checks.filter((check) => !check.passed);
+	},
+	warning: {
+		detail,
+		name: "moka-agents",
+		passed: true
+	}
+});
+const expectedRunAgentNames = (config) => {
+	const profiles = /* @__PURE__ */ new Set();
+	for (const catalog of Object.values(config.scheduler.node_catalogs)) for (const node of Object.values(catalog.nodes)) if (RUN_READINESS_CATEGORIES.has(node.category)) profiles.add(node.profile);
+	return [...profiles].filter((profileId) => config.profiles[profileId]?.runner === "opencode").map(opencodeAgentName).toSorted((a, b) => a.localeCompare(b));
+};
+const collectAgentNames = (value, names, inAgentList) => {
+	if (typeof value === "string") {
+		if (inAgentList) names.add(value);
+		return inAgentList;
+	}
+	if (Array.isArray(value)) {
+		let recognized = inAgentList;
+		for (const item of value) recognized = collectAgentNames(item, names, inAgentList) || recognized;
+		return recognized;
+	}
+	if (!isRecord(value)) return false;
+	let recognized = false;
+	for (const key of [
+		"agent",
+		"id",
+		"name",
+		"subagent_type",
+		"title"
+	]) if (typeof value[key] === "string") {
+		names.add(value[key]);
+		recognized = true;
+	}
+	for (const key of [
+		"agents",
+		"data",
+		"items",
+		"result"
+	]) {
+		const item = value[key];
+		if (Array.isArray(item) || typeof item === "object" && item !== null) recognized = collectAgentNames(item, names, true) || recognized;
+	}
+	return recognized;
+};
+const visibleAgentNames = (stdout) => {
+	const names = /* @__PURE__ */ new Set();
+	try {
+		const parsed = JSON.parse(stdout);
+		return {
+			ambiguous: false,
+			names,
+			recognized: collectAgentNames(parsed, names, Array.isArray(parsed))
+		};
+	} catch {
+		for (const line of stdout.split(LINE_RE)) {
+			const name = line.trim().replace(BULLET_PREFIX_RE, "");
+			if (name !== "") names.add(name);
+		}
+	}
 	return {
-		blockers,
-		checks,
-		passed: blockers.length === 0,
-		warnings
+		ambiguous: true,
+		names,
+		recognized: names.size > 0
 	};
-}
-function clusterNamespace(value, configuredNamespace) {
-	return typeof value === "string" && value.length > 0 ? value : configuredNamespace ?? defaultClusterDoctorNamespace();
-}
-function checkCommand(name, args, cwd) {
-	return checkCommandWithRunner(name, name, args, cwd);
-}
-async function checkCommandWithRunner(name, command, args, cwd) {
+};
+const interactivePermissionPaths = (value, path = ["permission"]) => {
+	if (typeof value === "string") return HEADLESS_AGENT_PERMISSION_VALUES.has(value.toLowerCase()) ? [path.join(".")] : [];
+	if (Array.isArray(value)) return value.flatMap((item, index) => interactivePermissionPaths(item, [...path, String(index)]));
+	if (!isRecord(value)) return [];
+	return Object.entries(value).flatMap(([key, item]) => interactivePermissionPaths(item, [...path, key]));
+};
+const isHeadless = () => {
+	const ci = process.env.CI?.toLowerCase();
+	return ci !== void 0 && ci !== "" && ci !== "0" && ci !== "false" || !process.stdin.isTTY;
+};
+const stringField = (value, field) => {
+	if (typeof value !== "object" || value === null) return "";
+	const candidate = Reflect.get(value, field);
+	return typeof candidate === "string" ? candidate : "";
+};
+const commandErrorDetail = (err) => {
+	return ([
+		stringField(err, "shortMessage"),
+		stringField(err, "stderr"),
+		stringField(err, "message"),
+		String(err)
+	].find((candidate) => candidate.trim() !== "") ?? "not available").trim();
+};
+const checkCommandWithRunner = async (name, command, args, cwd) => {
 	try {
 		await execa(command, args, {
 			cwd,
@@ -77,34 +151,16 @@ async function checkCommandWithRunner(name, command, args, cwd) {
 			name,
 			passed: true
 		};
-	} catch (err) {
+	} catch (error) {
 		return {
-			detail: commandErrorDetail(err),
+			detail: commandErrorDetail(error),
 			name,
 			passed: false
 		};
 	}
-}
-function checkPipelineConfig(cwd) {
-	try {
-		loadPipelineConfig(cwd);
-		return {
-			detail: "valid",
-			name: "pipeline-config",
-			passed: true
-		};
-	} catch (err) {
-		let message = "invalid";
-		if (err instanceof PipelineConfigError) message = err.issues.map((issue) => issue.message).join("; ");
-		else if (err instanceof Error) message = err.message;
-		return {
-			detail: message || "missing or invalid",
-			name: "pipeline-config",
-			passed: false
-		};
-	}
-}
-async function checkOpenCodeSdk() {
+};
+const checkCommand = async (name, args, cwd) => await checkCommandWithRunner(name, name, args, cwd);
+const checkOpenCodeSdk = async () => {
 	try {
 		if (typeof (await import("@opencode-ai/sdk")).createOpencodeClient !== "function") return {
 			detail: "@opencode-ai/sdk does not export createOpencodeClient",
@@ -116,15 +172,15 @@ async function checkOpenCodeSdk() {
 			name: "opencode-sdk",
 			passed: true
 		};
-	} catch (err) {
+	} catch (error) {
 		return {
-			detail: commandErrorDetail(err),
+			detail: commandErrorDetail(error),
 			name: "opencode-sdk",
 			passed: false
 		};
 	}
-}
-async function checkMokaAgents(cwd, config) {
+};
+const checkMokaAgents = async (cwd, config) => {
 	const expected = expectedRunAgentNames(config);
 	if (expected.length === 0) return { check: {
 		detail: "no configured MoKa run agents",
@@ -148,121 +204,68 @@ async function checkMokaAgents(cwd, config) {
 			name: "moka-agents",
 			passed: true
 		} };
-	} catch (err) {
-		return skippedAgentVisibility(`Could not cheaply list OpenCode agents: ${commandErrorDetail(err)}`);
+	} catch (error) {
+		return skippedAgentVisibility(`Could not cheaply list OpenCode agents: ${commandErrorDetail(error)}`);
 	}
-}
-function skippedAgentVisibility(detail) {
-	return {
-		check: {
-			detail: "skipped because OpenCode agent listing is unavailable",
-			name: "moka-agents",
-			passed: true
-		},
-		warning: {
-			detail,
-			name: "moka-agents",
-			passed: true
-		}
-	};
-}
-function expectedRunAgentNames(config) {
-	const profiles = /* @__PURE__ */ new Set();
-	for (const catalog of Object.values(config.scheduler.node_catalogs)) for (const node of Object.values(catalog.nodes)) if (RUN_READINESS_CATEGORIES.has(node.category)) profiles.add(node.profile);
-	return [...profiles].filter((profileId) => config.profiles[profileId]?.runner === "opencode").map(opencodeAgentName).sort((a, b) => a.localeCompare(b));
-}
-function visibleAgentNames(stdout) {
-	const names = /* @__PURE__ */ new Set();
-	try {
-		const parsed = JSON.parse(stdout);
-		return {
-			ambiguous: false,
-			names,
-			recognized: collectAgentNames(parsed, names, Array.isArray(parsed))
-		};
-	} catch {
-		for (const line of stdout.split(LINE_RE)) {
-			const name = line.trim().replace(BULLET_PREFIX_RE, "");
-			if (name) names.add(name);
-		}
-	}
-	return {
-		ambiguous: true,
-		names,
-		recognized: names.size > 0
-	};
-}
-function collectAgentNames(value, names, inAgentList) {
-	if (typeof value === "string") {
-		if (inAgentList) names.add(value);
-		return inAgentList;
-	}
-	if (Array.isArray(value)) {
-		let recognized = inAgentList;
-		for (const item of value) recognized = collectAgentNames(item, names, inAgentList) || recognized;
-		return recognized;
-	}
-	if (!(value && typeof value === "object")) return false;
-	const record = value;
-	let recognized = false;
-	for (const key of [
-		"agent",
-		"id",
-		"name",
-		"subagent_type",
-		"title"
-	]) if (typeof record[key] === "string") {
-		names.add(record[key]);
-		recognized = true;
-	}
-	for (const key of [
-		"agents",
-		"data",
-		"items",
-		"result"
-	]) {
-		const item = record[key];
-		if (Array.isArray(item) || item && typeof item === "object") recognized = collectAgentNames(item, names, true) || recognized;
-	}
-	return recognized;
-}
-function headlessPermissionWarnings(cwd) {
-	if (!isHeadless()) return [];
-	const agentDir = join(cwd, ".opencode", "agents");
-	if (!existsSync(agentDir)) return [];
-	return readdirSync(agentDir).filter((entry) => entry.endsWith(".md")).flatMap((entry) => headlessPermissionWarning(join(agentDir, entry), entry));
-}
-function headlessPermissionWarning(path, entry) {
+};
+const headlessPermissionWarning = (path, entry) => {
 	try {
 		if (!statSync(path).isFile()) return [];
-		const risky = interactivePermissionPaths(matter(readFileSync(path, "utf8")).data.permission);
+		const risky = interactivePermissionPaths(matter(readFileSync(path, "utf-8")).data.permission);
 		if (risky.length === 0) return [];
 		return [{
 			detail: `${entry} requires interactive permission prompts at ${risky.join(", ")}; headless MoKa runs may block.`,
 			name: "headless-permissions",
 			passed: true
 		}];
-	} catch (err) {
+	} catch (error) {
 		return [{
-			detail: `Could not inspect ${entry} for headless permission risks: ${commandErrorDetail(err)}`,
+			detail: `Could not inspect ${entry} for headless permission risks: ${commandErrorDetail(error)}`,
 			name: "headless-permissions",
 			passed: true
 		}];
 	}
-}
-function interactivePermissionPaths(value, path = ["permission"]) {
-	if (typeof value === "string") return HEADLESS_AGENT_PERMISSION_VALUES.has(value.toLowerCase()) ? [path.join(".")] : [];
-	if (Array.isArray(value)) return value.flatMap((item, index) => interactivePermissionPaths(item, [...path, String(index)]));
-	if (!(value && typeof value === "object")) return [];
-	return Object.entries(value).flatMap(([key, item]) => interactivePermissionPaths(item, [...path, key]));
-}
-function isHeadless() {
-	const ci = process.env.CI?.toLowerCase();
-	return ci !== void 0 && ci !== "" && ci !== "0" && ci !== "false" || !process.stdin.isTTY;
-}
-function commandErrorDetail(err) {
-	const error = err;
-	return (error.shortMessage || error.stderr || error.message || String(err)).trim() || "not available";
-}
+};
+const headlessPermissionWarnings = (cwd) => {
+	if (!isHeadless()) return [];
+	const agentDir = join(cwd, ".opencode", "agents");
+	if (!existsSync(agentDir)) return [];
+	return readdirSync(agentDir).filter((entry) => entry.endsWith(".md")).flatMap((entry) => headlessPermissionWarning(join(agentDir, entry), entry));
+};
+const runDoctor = async (cwd, options = {}) => {
+	const commandChecks = await Promise.all([
+		checkCommand("npx", ["--version"], cwd),
+		checkCommand("opencode", ["--version"], cwd),
+		checkCommand("fallow", ["--version"], cwd)
+	]);
+	const configCheck = checkPipelineConfig(cwd);
+	const config = configCheck.passed ? loadPipelineConfig(cwd) : null;
+	const [sdkCheck, agentVisibility] = await Promise.all([checkOpenCodeSdk(), config ? checkMokaAgents(cwd, config) : Promise.resolve({ check: {
+		detail: "skipped because pipeline config is invalid",
+		name: "moka-agents",
+		passed: true
+	} })]);
+	const globalConfig = loadMokaGlobalConfig();
+	const clusterResult = options.cluster !== void 0 && options.cluster !== false ? await runClusterDoctor({
+		kubeContext: options.kubeContext,
+		kubeconfigPath: options.kubeconfig ?? (globalConfig === null ? void 0 : globalConfig.momokaya.kubernetes.kubeconfig),
+		namespace: clusterNamespace(options.cluster, globalConfig === null ? void 0 : globalConfig.momokaya.kubernetes.namespace)
+	}) : { checks: [] };
+	const checks = [
+		...commandChecks,
+		configCheck,
+		sdkCheck,
+		agentVisibility.check,
+		...clusterResult.checks
+	];
+	const warnings = [...agentVisibility.warning ? [agentVisibility.warning] : [], ...headlessPermissionWarnings(cwd)];
+	const blockers = checks.filter((check) => !check.passed);
+	return {
+		blockers,
+		checks,
+		passed: blockers.length === 0,
+		warnings
+	};
+};
 //#endregion
 export { runDoctor };

package/dist/cli/factory-commands.js

@@ -2,32 +2,33 @@ import { runCreateExperiment } from "../factory/create-experiment.js";
 import { runTemplateUpdate, summarizeTemplateUpdate } from "../factory/template-update.js";
 import { Option } from "commander";
 //#region src/cli/factory-commands.ts
-function registerFactoryCommands(program) {
+const hasText = (value) => typeof value === "string" && value !== "";
+const registerFactoryCommands = (program) => {
 	program.command("create-experiment").description("Birth a fleet experiment: copier-stamp momokaya-template, create+push the org repo, register it in infra's fleet registry").requiredOption("--name <name>", "app name (kebab-case)").addOption(new Option("--flavor <flavor>", "app flavor").choices(["web", "expo-web"]).default("web")).option("--no-db", "skip the database surface").option("--no-previews", "skip per-PR preview environments").option("--org <org>", "GitHub org for the new repo").option("--template-src <source>", "copier template source").option("--template-ref <ref>", "template tag/ref (default: latest tag)").option("--infra-repo-url <url>", "infra repo the registry entry lands in").action(async (flags) => {
 		const result = await runCreateExperiment({
 			db: flags.db,
 			flavor: flags.flavor,
-			...flags.infraRepoUrl ? { infraRepoUrl: flags.infraRepoUrl } : {},
+			...hasText(flags.infraRepoUrl) ? { infraRepoUrl: flags.infraRepoUrl } : {},
 			name: flags.name,
-			...flags.org ? { org: flags.org } : {},
+			...hasText(flags.org) ? { org: flags.org } : {},
 			previews: flags.previews,
-			...flags.templateRef ? { templateRef: flags.templateRef } : {},
-			...flags.templateSrc ? { templateSource: flags.templateSrc } : {}
+			...hasText(flags.templateRef) ? { templateRef: flags.templateRef } : {},
+			...hasText(flags.templateSrc) ? { templateSource: flags.templateSrc } : {}
 		});
 		console.log(`Experiment born: ${result.repoUrl} (registry ${result.registryPath} @ infra ${result.infraCommitSha})`);
 	});
 	program.command("template-update").description("Fan copier-update PRs out across repos stamped from momokaya-template").option("--repos <repos>", "comma-separated repo list (skips fleet-registry discovery)").option("--org <org>", "GitHub org the stamped repos live in").option("--template-match <substring>", "answers-file _src_path filter for stamp detection").option("--template-ref <ref>", "template tag/ref (default: latest tag)").option("--infra-repo-url <url>", "infra repo used for discovery").action(async (flags) => {
 		const { results } = await runTemplateUpdate({
-			...flags.infraRepoUrl ? { infraRepoUrl: flags.infraRepoUrl } : {},
-			...flags.org ? { org: flags.org } : {},
-			...flags.repos ? { repos: flags.repos.split(",").map((repo) => repo.trim()).filter((repo) => repo.length > 0) } : {},
-			...flags.templateMatch ? { templateMatch: flags.templateMatch } : {},
-			...flags.templateRef ? { templateRef: flags.templateRef } : {}
+			...hasText(flags.infraRepoUrl) ? { infraRepoUrl: flags.infraRepoUrl } : {},
+			...hasText(flags.org) ? { org: flags.org } : {},
+			...hasText(flags.repos) ? { repos: flags.repos.split(",").map((repo) => repo.trim()).filter((repo) => repo.length > 0) } : {},
+			...hasText(flags.templateMatch) ? { templateMatch: flags.templateMatch } : {},
+			...hasText(flags.templateRef) ? { templateRef: flags.templateRef } : {}
 		});
 		const { failed, opened } = summarizeTemplateUpdate(results);
 		console.log(`template-update: ${opened} PR(s) opened, ${failed} error(s)`);
 		if (failed > 0) process.exitCode = 1;
 	});
-}
+};
 //#endregion
 export { registerFactoryCommands };