oh-my-opencode 4.19.4 → 5.0.0-beta.10

package/packages/omo-codex/scripts/install-bin-links.test.mjs

@@ -26,14 +26,61 @@ async function writeRuntimeWrapperFixture({ withNodeCli = false } = {}) {
 	}
 	await mkdir(homeDir, { recursive: true });
 	const link = await linkRootRuntimeBin({ binDir, codexHome, repoRoot, platform: "linux" });
-	return { homeDir, link, repoRoot };
+	return { binDir, codexHome, homeDir, link, repoRoot };
 }
 
 function runtimeWrapperEnv(homeDir) {
 	return { PATH: `${dirname(process.execPath)}:/usr/bin:/bin`, HOME: homeDir };
 }
 
-test("#given bun absent from PATH but present in ~/.bun/bin #when running the omo runtime wrapper #then resolves the bun fallback", async (t) => {
+test("#given the generated installer #when linking the root runtime #then writes only the canonical wrapper with marker and edition exports", async () => {
+	const { binDir, link } = await writeRuntimeWrapperFixture();
+
+	assert.equal(link.name, "omo-agent-toolkit");
+	assert.equal(link.path, join(binDir, "omo-agent-toolkit"));
+	const wrapper = await readFile(link.path, "utf8");
+	assert.match(wrapper, /OMO_GENERATED_RUNTIME_WRAPPER/);
+	assert.match(wrapper, /export OMO_INVOCATION_NAME=omo-agent-toolkit/);
+	assert.match(wrapper, /export OMO_EDITION=codex/);
+	await assert.rejects(lstat(join(binDir, "omo")));
+});
+
+test("#given a marker-bearing legacy omo #when linking the root runtime #then deletes it", async () => {
+	const root = await makeTempDir();
+	const repoRoot = join(root, "repo");
+	const binDir = join(root, "bin");
+	await mkdir(join(repoRoot, "dist", "cli"), { recursive: true });
+	await mkdir(binDir, { recursive: true });
+	await writeFile(join(repoRoot, "dist", "cli", "index.js"), "");
+	await writeFile(join(binDir, "omo"), "#!/bin/sh\n# OMO_GENERATED_RUNTIME_WRAPPER\n");
+
+	await linkRootRuntimeBin({ binDir, codexHome: join(root, "codex"), repoRoot, platform: "linux" });
+
+	await assert.rejects(lstat(join(binDir, "omo")));
+	assert.match(await readFile(join(binDir, "omo-agent-toolkit"), "utf8"), /OMO_GENERATED_RUNTIME_WRAPPER/);
+});
+
+test("#given an unmarked user-owned omo #when linking twice #then preserves it byte-identical and remains idempotent", async () => {
+	const root = await makeTempDir();
+	const repoRoot = join(root, "repo");
+	const binDir = join(root, "bin");
+	const legacyPath = join(binDir, "omo");
+	const canonicalPath = join(binDir, "omo-agent-toolkit");
+	const userContent = Buffer.from("#!/bin/sh\necho user-owned\n");
+	await mkdir(join(repoRoot, "dist", "cli"), { recursive: true });
+	await mkdir(binDir, { recursive: true });
+	await writeFile(join(repoRoot, "dist", "cli", "index.js"), "");
+	await writeFile(legacyPath, userContent);
+
+	await linkRootRuntimeBin({ binDir, codexHome: join(root, "codex"), repoRoot, platform: "linux" });
+	const firstCanonical = await readFile(canonicalPath);
+	await linkRootRuntimeBin({ binDir, codexHome: join(root, "codex"), repoRoot, platform: "linux" });
+
+	assert.deepEqual(await readFile(legacyPath), userContent);
+	assert.deepEqual(await readFile(canonicalPath), firstCanonical);
+});
+
+test("#given bun absent from PATH but present in ~/.bun/bin #when running the omo-agent-toolkit runtime wrapper #then resolves the bun fallback", async (t) => {
 	if (process.platform === "win32") return t.skip("posix wrapper execution");
 	const { homeDir, link } = await writeRuntimeWrapperFixture();
 	await mkdir(join(homeDir, ".bun", "bin"), { recursive: true });
@@ -114,7 +161,12 @@ test("#given Windows platform #when writing the omo runtime wrapper #then embeds
 
 	const link = await linkRootRuntimeBin({ binDir, codexHome: join(root, "codex"), repoRoot, platform: "win32" });
 
+	assert.equal(link.name, "omo-agent-toolkit");
+	assert.equal(link.path, join(binDir, "omo-agent-toolkit.cmd"));
 	const wrapper = await readFile(link.path, "utf8");
+	assert.match(wrapper, /OMO_GENERATED_RUNTIME_WRAPPER/);
+	assert.match(wrapper, /set "OMO_INVOCATION_NAME=omo-agent-toolkit"/);
+	assert.match(wrapper, /set "OMO_EDITION=codex"/);
 	assert.match(wrapper, /OMO_RUNTIME/);
 	assert.match(wrapper, /dist[\\/]cli-node[\\/]index\.js/);
 	assert.ok(wrapper.indexOf("OMO_RUNTIME") < wrapper.indexOf("where bun"), "node override must precede bun discovery");
@@ -246,6 +298,7 @@ test("#given nested component declares reserved omo bin #when linking bins #then
 		name: "@example/ulw-loop",
 		bin: {
 			omo: "./dist/cli.js",
+			"omo-agent-toolkit": "./dist/cli.js",
 			"omo-ulw-loop": "./dist/cli.js",
 			ulw: "./dist/cli.js",
 			"ulw-loop": "./dist/cli.js",
@@ -261,6 +314,7 @@ test("#given nested component declares reserved omo bin #when linking bins #then
 		{ name: "ulw-loop", path: join(binDir, "ulw-loop"), target: join(componentRoot, "dist", "cli.js") },
 	]);
 	await assert.rejects(readlink(join(binDir, "omo")));
+	await assert.rejects(readlink(join(binDir, "omo-agent-toolkit")));
 	assert.equal(await readlink(join(binDir, "omo-ulw-loop")), join(componentRoot, "dist", "cli.js"));
 	assert.equal(await readlink(join(binDir, "ulw")), join(componentRoot, "dist", "cli.js"));
 	assert.equal(await readlink(join(binDir, "ulw-loop")), join(componentRoot, "dist", "cli.js"));

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

@@ -3,7 +3,7 @@ import test from "node:test";
 
 import { buildDelegatedOmoInvocation, runDelegatedOmoCommand } from "./install-local.mjs";
 
-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 () => {
+test("#given a lazycodex passthrough command #when delegating to omo-agent-toolkit #then resets OMO_INVOCATION_NAME so the delegate does not re-enter the lazycodex path", async () => {
 	// given
 	const parsed = { kind: "command", command: "boulder", args: [] };
 	let received;
@@ -25,12 +25,12 @@ test("#given a lazycodex passthrough command #when delegating to omo #then reset
 	assert.equal(received.runOptions.cwd, "/tmp/project");
 	assert.equal(
 		received.runOptions.env?.OMO_INVOCATION_NAME,
-		"omo",
-		"delegated omo command must run with OMO_INVOCATION_NAME=omo to avoid infinite recursion",
+		"omo-agent-toolkit",
+		"delegated command must run with OMO_INVOCATION_NAME=omo-agent-toolkit to avoid infinite recursion",
 	);
 });
 
-test("#given OMO_INVOCATION_NAME=lazycodex in the parent env #when delegating a cleanup passthrough #then the child env overrides it to omo", async () => {
+test("#given OMO_INVOCATION_NAME=lazycodex in the parent env #when delegating a cleanup passthrough #then the child env overrides it to omo-agent-toolkit", async () => {
 	// given
 	const previous = process.env.OMO_INVOCATION_NAME;
 	process.env.OMO_INVOCATION_NAME = "lazycodex";
@@ -52,7 +52,7 @@ test("#given OMO_INVOCATION_NAME=lazycodex in the parent env #when delegating a
 	}
 
 	// then
-	assert.equal(received.env.OMO_INVOCATION_NAME, "omo");
+	assert.equal(received.env.OMO_INVOCATION_NAME, "omo-agent-toolkit");
 });
 
 test("#given a dry-run doctor #when delegating #then routes to the Codex LazyCodex doctor workflow", async () => {
@@ -239,5 +239,5 @@ test("#given dry-run args with shell metacharacters #when delegating #then logs
 	});
 
 	// then
-	assert.equal(logged, "npx --yes --package oh-my-openagent omo cleanup --platform=codex --project '/tmp/lazy codex'\\''s qa'");
+	assert.equal(logged, "npx --yes --package oh-my-openagent omo-agent-toolkit cleanup --platform=codex --project '/tmp/lazy codex'\\''s qa'");
 });