oh-my-opencode 4.16.1 → 4.16.3

package/packages/omo-codex/plugin/test/aggregate-agents.test.mjs

@@ -16,9 +16,50 @@ const agentSchemaKeys = new Set([
 ]);
 
 const lazycodexAgentInvariants = new Map([
+	[
+		"explorer.toml",
+		{
+			model: "gpt-5.6-terra",
+			effort: "medium",
+			includes: [/Read-only/, /working tree/, /rg/],
+		},
+	],
+	[
+		"librarian.toml",
+		{
+			model: "gpt-5.6-terra",
+			effort: "medium",
+			includes: [/Read-only/, /SHA-pinned GitHub permalink/, /external/],
+		},
+	],
+	[
+		"metis.toml",
+		{
+			model: "gpt-5.6-sol",
+			effort: "high",
+			includes: [/pre-planning analyst/i, /contradictions/, /Read-only/],
+		},
+	],
+	[
+		"momus.toml",
+		{
+			model: "gpt-5.6-sol",
+			effort: "ultra",
+			includes: [/plan reviewer/i, /OKAY, ITERATE, or REJECT/, /Read-only/],
+		},
+	],
+	[
+		"plan.toml",
+		{
+			model: "gpt-5.6-sol",
+			effort: "xhigh",
+			includes: [/strategic planning consultant/i, /\.omo\/plans\/<slug>\.md/, /never implements/i],
+		},
+	],
 	[
 		"lazycodex-executor.toml",
 		{
+			model: "gpt-5.6-sol",
 			effort: "high",
 			includes: [/EVIDENCE_RECORDED: <path>/, /scenario/i, /artifact/i],
 		},
@@ -26,6 +67,7 @@ const lazycodexAgentInvariants = new Map([
 	[
 		"lazycodex-clone-fidelity-reviewer.toml",
 		{
+			model: "gpt-5.6-sol",
 			effort: "xhigh",
 			includes: [/recommendation/, /blockers/, /\.omo\/evidence\/<goal>-clone-fidelity\.md/],
 		},
@@ -33,6 +75,7 @@ const lazycodexAgentInvariants = new Map([
 	[
 		"lazycodex-code-reviewer.toml",
 		{
+			model: "gpt-5.6-sol",
 			effort: "xhigh",
 			includes: [/codeQualityStatus/, /recommendation/, /\.omo\/evidence\/<goal>-code-review\.md/],
 		},
@@ -40,6 +83,7 @@ const lazycodexAgentInvariants = new Map([
 	[
 		"lazycodex-qa-executor.toml",
 		{
+			model: "gpt-5.6-terra",
 			effort: "medium",
 			includes: [/not_applicable/, /surfaceEvidence/, /adversarialCases/],
 		},
@@ -47,6 +91,7 @@ const lazycodexAgentInvariants = new Map([
 	[
 		"lazycodex-gate-reviewer.toml",
 		{
+			model: "gpt-5.6-sol",
 			effort: "xhigh",
 			includes: [/APPROVE\/REJECT/, /blockers/, /\.omo\/evidence\/<goal>-gate-review\.md/],
 		},
@@ -109,7 +154,8 @@ test("#given lazycodex agent prompts #when inspected #then each role pins model
 	for (const [fileName, invariant] of lazycodexAgentInvariants) {
 		const prompt = await readFile(join(agentsDir, fileName), "utf8");
 
-		assert.match(prompt, /^model\s*=\s*"gpt-5\.5"$/m);
+		const escapedModel = invariant.model.replace(/\./g, "\\.");
+		assert.match(prompt, new RegExp(`^model\\s*=\\s*"${escapedModel}"$`, "m"));
 		assert.match(prompt, new RegExp(`^model_reasoning_effort\\s*=\\s*"${invariant.effort}"$`, "m"));
 		assert.doesNotMatch(prompt, /^tools\s*=/m);
 		assert.doesNotMatch(prompt, /^blocking\s*=/m);

package/packages/omo-codex/plugin/test/aggregate-plugin-fixture.mjs

@@ -88,7 +88,7 @@ export function findSpawnAgentCallsWithoutForkContextFalse(content) {
 	const regex = /spawn_agent\(([^)]*)\)/g;
 	for (const match of content.matchAll(regex)) {
 		const call = match[0];
-		if (!/"fork_context"\s*:\s*false|fork_context:\s*false|fork_context=false/.test(call)) {
+		if (!/"fork_context"\s*:\s*false|fork_context:\s*false|fork_context=false|"fork_turns"\s*:\s*"none"|fork_turns:\s*"none"|fork_turns="none"/.test(call)) {
 			missingForkContext.push(call);
 		}
 	}

package/packages/omo-codex/plugin/test/aggregate-skills.test.mjs

@@ -34,7 +34,8 @@ test('#given synced skills and bundled rules #when role-specific agents are spaw
 	const promptFiles = skillEntries
 		.filter((entry) => entry.isDirectory())
 		.map((entry) => join(skillsDir, entry.name, "SKILL.md"));
-	promptFiles.push(join(root, "components", "rules", "bundled-rules", "hephaestus.md"));
+	promptFiles.push(join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.5.md"));
+	promptFiles.push(join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.6.md"));
 
 	const missingForkContext = [];
 	for (const promptPath of promptFiles) {
@@ -53,7 +54,8 @@ test("#given long-running orchestration prompts #when waiting on child agents #t
 		join(root, "skills", "ulw-loop", "references", "full-workflow.md"),
 		join(root, "skills", "review-work", "SKILL.md"),
 		join(root, "skills", "start-work", "SKILL.md"),
-		join(root, "components", "rules", "bundled-rules", "hephaestus.md"),
+		join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.5.md"),
+		join(root, "components", "rules", "bundled-rules", "hephaestus", "gpt-5.6.md"),
 	];
 
 	const missingLivenessGuidance = [];

package/packages/omo-codex/plugin/test/auto-update-restart-notice.test.mjs

@@ -224,3 +224,36 @@ test("#given completed pending update #when hook session-start runs as CLI #then
 	assert.equal(repeat.status, 0);
 	assert.equal(repeat.stdout, "");
 });
+
+test("#given the script reached through a symlinked directory #when hook session-start runs as CLI #then the entry guard still fires", async () => {
+	const root = await mkdtemp(join(tmpdir(), "lazycodex-restart-notice-symlink-"));
+	const env = autoUpdateEnv(root, {
+		LAZYCODEX_CURRENT_VERSION: "1.0.1",
+		LAZYCODEX_LATEST_VERSION: "1.0.1",
+		LAZYCODEX_CONFIG_MIGRATION_DISABLED: "1",
+	});
+	await writeFile(env.LAZYCODEX_AUTO_UPDATE_STATE_PATH, JSON.stringify({
+		lastCheckedAt: Date.now() - 1_000,
+		lastStatus: "success",
+		pendingNotice: { fromVersion: "1.0.0", toVersion: "1.0.1", startedAt: 1 },
+	}));
+	// Simulate a symlinked plugin cache dir: argv[1] spells the symlink, while
+	// import.meta.url resolves to the real path. Seen in the wild; the old
+	// `pathToFileURL(process.argv[1])` guard never matched and the hook was a
+	// silent no-op.
+	const { symlink } = await import("node:fs/promises");
+	const { dirname } = await import("node:path");
+	const linkDir = join(root, "linked-scripts");
+	await symlink(dirname(SCRIPT_PATH), linkDir, "dir");
+	const linkedScript = join(linkDir, "auto-update.mjs");
+
+	const result = spawnSync(process.execPath, [linkedScript, "hook", "session-start"], {
+		encoding: "utf8",
+		env: { ...process.env, ...env },
+	});
+
+	assert.equal(result.status, 0);
+	const lines = result.stdout.split("\n").filter((line) => line.length > 0);
+	assert.equal(lines.length, 1, `expected one SessionStart JSON line, got stdout=${JSON.stringify(result.stdout)} stderr=${JSON.stringify(result.stderr)}`);
+	assert.equal(JSON.parse(lines[0]).hookSpecificOutput.hookEventName, "SessionStart");
+});

package/packages/omo-codex/plugin/test/auto-update.test.mjs

@@ -2,9 +2,10 @@ import assert from "node:assert/strict";
 import { mkdir, mkdtemp, readFile, symlink, utimes, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
+import { PassThrough } from "node:stream";
 import test from "node:test";
 
-import { resolveAutoUpdatePlan, resolveLazyCodexUpdatePlan, runAutoUpdateCheck } from "../scripts/auto-update.mjs";
+import { readSessionModelFromStdin, resolveAutoUpdatePlan, resolveLazyCodexUpdatePlan, runAutoUpdateCheck } from "../scripts/auto-update.mjs";
 import { detectInstallFlow } from "../scripts/install-flow.mjs";
 import { resolveSpawnInvocation } from "../scripts/spawn-command.mjs";
 
@@ -20,6 +21,12 @@ function autoUpdateEnv(root, extra = {}) {
 	};
 }
 
+function stdinFrom(payload) {
+	const stream = new PassThrough();
+	stream.end(payload);
+	return stream;
+}
+
 test("#given auto update is disabled #when resolving plan #then no command is scheduled", () => {
 	const plan = resolveAutoUpdatePlan({
 		env: { LAZYCODEX_AUTO_UPDATE_DISABLED: "1" },
@@ -31,6 +38,33 @@ test("#given auto update is disabled #when resolving plan #then no command is sc
 	assert.equal(plan.reason, "disabled");
 });
 
+test("#given SessionStart payload with model #when reading stdin #then returns the model", async () => {
+	const payload = JSON.stringify({
+		hook_event_name: "SessionStart",
+		session_id: "s-1",
+		cwd: "/tmp",
+		model: "gpt-5.6-terra",
+		permission_mode: "default",
+		source: "startup",
+	});
+
+	assert.equal(await readSessionModelFromStdin(stdinFrom(payload)), "gpt-5.6-terra");
+});
+
+test("#given empty or malformed stdin #when reading session model #then returns null", async () => {
+	assert.equal(await readSessionModelFromStdin(stdinFrom("")), null);
+	assert.equal(await readSessionModelFromStdin(stdinFrom("not json")), null);
+	assert.equal(await readSessionModelFromStdin(stdinFrom(JSON.stringify({ model: "  " }))), null);
+	assert.equal(await readSessionModelFromStdin(stdinFrom(JSON.stringify({ session_id: "s-1" }))), null);
+	assert.equal(await readSessionModelFromStdin(null), null);
+});
+
+test("#given a TTY stdin #when reading session model #then returns null without waiting", async () => {
+	const stream = stdinFrom("{}");
+	stream.isTTY = true;
+	assert.equal(await readSessionModelFromStdin(stream), null);
+});
+
 test("#given stale state #when resolving plan #then installer update command is scheduled", () => {
 	const plan = resolveAutoUpdatePlan({
 		env: { LAZYCODEX_CURRENT_VERSION: "1.0.0", LAZYCODEX_LATEST_VERSION: "1.0.1" },