oh-my-opencode 4.12.1 → 4.13.0

package/packages/omo-codex/scripts/install-cli-args.test.mjs

@@ -88,7 +88,7 @@ test("#given installer help #when formatting usage #then includes uninstall", ()
 	assert.equal(includesUninstall, true);
 });
 
-test("#given installer help #when formatting usage #then advertises every pass-through command it actually delegates", () => {
+test("#given installer help #when formatting usage #then advertises every supported command", () => {
 	// given
 	const help = formatLazyCodexInstallHelp();
 

package/packages/omo-codex/scripts/install-config-git-bash.test.mjs

@@ -89,3 +89,26 @@ test("#given non-windows platforms #when updating config #then disables git_bash
 		assert.match(content, /\[plugins\."omo@sisyphuslabs"\.mcp_servers\.git_bash\][\s\S]*?enabled = false/);
 	}
 });
+
+test("#given CodeGraph MCP disabled by installer policy #when updating config #then disables only codegraph plugin mcp policy", async () => {
+	// given
+	const root = await mkdtemp(join(tmpdir(), "omo-codex-config-codegraph-disabled-"));
+	const configPath = join(root, "config.toml");
+
+	// when
+	await updateCodexConfig({
+		configPath,
+		repoRoot: "/repo/packages/omo-codex",
+		marketplaceName: "sisyphuslabs",
+		marketplaceSource: { sourceType: "local", source: "/repo/packages/omo-codex/cache/sisyphuslabs" },
+		pluginNames: ["omo"],
+		platform: "darwin",
+		codegraphMcpEnabled: false,
+	});
+
+	// then
+	const content = await readFile(configPath, "utf8");
+	assert.match(content, /\[plugins\."omo@sisyphuslabs"\][\s\S]*?enabled = true/);
+	assert.match(content, /\[plugins\."omo@sisyphuslabs"\.mcp_servers\.codegraph\][\s\S]*?enabled = false/);
+	assert.match(content, /\[plugins\."omo@sisyphuslabs"\.mcp_servers\.context7\][\s\S]*?enabled = true/);
+});

package/packages/omo-codex/scripts/install-delegated-command.test.mjs

@@ -5,7 +5,7 @@ import { buildDelegatedOmoInvocation, runDelegatedOmoCommand } from "./install-l
 
 test("#given a lazycodex passthrough command #when delegating to omo #then resets OMO_INVOCATION_NAME so the delegate does not re-enter the lazycodex path", async () => {
 	// given
-	const parsed = { kind: "command", command: "doctor", args: [] };
+	const parsed = { kind: "command", command: "boulder", args: [] };
 	let received;
 	const options = {
 		cwd: "/tmp/project",
@@ -55,9 +55,9 @@ test("#given OMO_INVOCATION_NAME=lazycodex in the parent env #when delegating a
 	assert.equal(received.env.OMO_INVOCATION_NAME, "omo");
 });
 
-test("#given a dry-run passthrough #when delegating #then logs the invocation without invoking runCommand", async () => {
+test("#given a dry-run doctor #when delegating #then routes to the Codex LazyCodex doctor workflow", async () => {
 	// given
-	const parsed = { kind: "command", command: "doctor", dryRun: true, args: [] };
+	const parsed = { kind: "command", command: "doctor", dryRun: true, args: ["--json"] };
 	let logged;
 	let ran = false;
 
@@ -74,5 +74,77 @@ test("#given a dry-run passthrough #when delegating #then logs the invocation wi
 
 	// then
 	assert.equal(ran, false);
-	assert.match(logged, /npx --yes --package oh-my-openagent omo doctor/);
+	assert.match(logged, /^codex exec /);
+	assert.match(logged, /Use \$omo:lcx-doctor/);
+	assert.match(logged, /Requested doctor arguments: --json/);
+	assert.doesNotMatch(logged, /oh-my-openagent omo doctor/);
+});
+
+test("#given doctor recursion guard is active #when lazycodex doctor delegates #then rejects before launching Codex", async () => {
+	// given
+	const parsed = { kind: "command", command: "doctor", args: [] };
+	const previous = process.env.LAZYCODEX_DOCTOR_LCX_ACTIVE;
+	process.env.LAZYCODEX_DOCTOR_LCX_ACTIVE = "1";
+	let ran = false;
+
+	try {
+		// when/then
+		await assert.rejects(
+			runDelegatedOmoCommand(parsed, {
+				cwd: "/tmp/project",
+				log: () => {},
+				runCommand: async () => {
+					ran = true;
+				},
+			}),
+			/Refusing recursive lazycodex doctor invocation/,
+		);
+	} finally {
+		if (previous === undefined) delete process.env.LAZYCODEX_DOCTOR_LCX_ACTIVE;
+		else process.env.LAZYCODEX_DOCTOR_LCX_ACTIVE = previous;
+	}
+
+	// then
+	assert.equal(ran, false);
+});
+
+test("#given a dry-run doctor with JSON output #when delegating #then asks the Codex workflow to return JSON", async () => {
+	// given
+	const parsed = { kind: "command", command: "doctor", dryRun: true, args: ["--json"] };
+	let logged;
+
+	// when
+	await runDelegatedOmoCommand(parsed, {
+		cwd: "/tmp/project",
+		log: (line) => {
+			logged = line;
+		},
+		runCommand: async () => {},
+	});
+
+	// then
+	assert.match(logged, /Return exactly one JSON object/);
+});
+
+test("#given dry-run args with shell metacharacters #when delegating #then logs a shell-safe command", async () => {
+	// given
+	const parsed = {
+		kind: "command",
+		command: "cleanup",
+		dryRun: true,
+		args: ["--project", "/tmp/lazy codex's qa"],
+	};
+	let logged;
+
+	// when
+	await runDelegatedOmoCommand(parsed, {
+		cwd: "/tmp/project",
+		log: (line) => {
+			logged = line;
+		},
+		runCommand: async () => {},
+	});
+
+	// then
+	assert.equal(logged, "npx --yes --package oh-my-openagent omo cleanup --platform=codex --project '/tmp/lazy codex'\\''s qa'");
 });