oh-my-codex 0.10.2 → 0.10.3

package/dist/team/__tests__/worker-bootstrap.test.js

@@ -1,22 +1,22 @@
-import { describe, it } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtemp, readFile, writeFile, rm, mkdir } from 'fs/promises';
-import { join } from 'path';
-import { tmpdir } from 'os';
-import { generateWorkerOverlay, applyWorkerOverlay, stripWorkerOverlay, writeTeamWorkerInstructionsFile, writeWorkerRoleInstructionsFile, removeTeamWorkerInstructionsFile, generateInitialInbox, generateTaskAssignmentInbox, generateShutdownInbox, generateTriggerMessage, generateMailboxTriggerMessage, generateLeaderMailboxTriggerMessage, } from '../worker-bootstrap.js';
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { mkdtemp, readFile, writeFile, rm, mkdir } from "fs/promises";
+import { join } from "path";
+import { tmpdir } from "os";
+import { generateWorkerOverlay, applyWorkerOverlay, stripWorkerOverlay, writeTeamWorkerInstructionsFile, writeWorkerRoleInstructionsFile, removeTeamWorkerInstructionsFile, generateInitialInbox, generateTaskAssignmentInbox, generateShutdownInbox, generateTriggerMessage, generateMailboxTriggerMessage, generateLeaderMailboxTriggerMessage, } from "../worker-bootstrap.js";
 function setMockCodexHome(codexHomePath) {
     const previous = process.env.CODEX_HOME;
     process.env.CODEX_HOME = codexHomePath;
     return () => {
-        if (typeof previous === 'string')
+        if (typeof previous === "string")
             process.env.CODEX_HOME = previous;
         else
             delete process.env.CODEX_HOME;
     };
 }
-describe('worker bootstrap', () => {
-    it('worker skill lifecycle instructions are claim-safe (issue #448)', async () => {
-        const workerSkill = await readFile(join(process.cwd(), 'skills', 'worker', 'SKILL.md'), 'utf8');
+describe("worker bootstrap", () => {
+    it("worker skill lifecycle instructions are claim-safe (issue #448)", async () => {
+        const workerSkill = await readFile(join(process.cwd(), "skills", "worker", "SKILL.md"), "utf8");
         assert.match(workerSkill, /omx team api claim-task/);
         assert.match(workerSkill, /omx team api transition-task-status/);
         assert.match(workerSkill, /omx team api release-task-claim/);
@@ -25,16 +25,16 @@ describe('worker bootstrap', () => {
         assert.doesNotMatch(workerSkill, /`?\{"status":"completed","result":"\.\.\."\}`?/);
         assert.doesNotMatch(workerSkill, /`?\{"status":"failed","error":"\.\.\."\}`?/);
     });
-    it('generateWorkerOverlay produces markdown with correct start/end markers', () => {
-        const overlay = generateWorkerOverlay('alpha-team');
+    it("generateWorkerOverlay produces markdown with correct start/end markers", () => {
+        const overlay = generateWorkerOverlay("alpha-team");
         assert.match(overlay, /<!-- OMX:TEAM:WORKER:START -->/);
         assert.match(overlay, /<!-- OMX:TEAM:WORKER:END -->/);
     });
-    it('generateWorkerOverlay includes the team name', () => {
-        const overlay = generateWorkerOverlay('my-team');
+    it("generateWorkerOverlay includes the team name", () => {
+        const overlay = generateWorkerOverlay("my-team");
         assert.match(overlay, /team "my-team"/);
         assert.match(overlay, /\$\{CODEX_HOME:-~\/\.codex\}\/skills\/worker\/SKILL\.md/);
-        assert.match(overlay, /~\/\.agents\/skills\/worker\/SKILL\.md/);
+        assert.match(overlay, /<leader_cwd>\/\.codex\/skills\/worker\/SKILL\.md/);
         assert.match(overlay, /Resolve canonical team state root/i);
         assert.match(overlay, /<team_state_root>\/team\/my-team\/tasks/);
         assert.match(overlay, /tasks\/task-<id>\.json/);
@@ -43,18 +43,19 @@ describe('worker bootstrap', () => {
         assert.match(overlay, /omx team api transition-task-status/);
         assert.match(overlay, /omx team api release-task-claim/);
         assert.doesNotMatch(overlay, /On completion: write \{"status": "completed"/);
-        assert.match(overlay, /Do NOT spawn sub-agents/);
+        assert.match(overlay, /You may spawn Codex native subagents when parallel execution improves throughput/);
+        assert.match(overlay, /Use subagents only for independent, bounded subtasks/);
         assert.match(overlay, /do not pass workingDirectory unless the lead explicitly tells you to/);
         assert.doesNotMatch(overlay, /tasks\/\{id\}\.json/);
     });
-    it('applyWorkerOverlay appends to existing AGENTS.md content', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("applyWorkerOverlay appends to existing AGENTS.md content", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const agentsMdPath = join(cwd, 'AGENTS.md');
-            await writeFile(agentsMdPath, '# Base AGENTS\n\nBase content.\n', 'utf8');
-            const overlay = generateWorkerOverlay('team-a');
+            const agentsMdPath = join(cwd, "AGENTS.md");
+            await writeFile(agentsMdPath, "# Base AGENTS\n\nBase content.\n", "utf8");
+            const overlay = generateWorkerOverlay("team-a");
             await applyWorkerOverlay(agentsMdPath, overlay);
-            const content = await readFile(agentsMdPath, 'utf8');
+            const content = await readFile(agentsMdPath, "utf8");
             assert.match(content, /# Base AGENTS/);
             assert.match(content, /Base content\./);
             assert.match(content, /<!-- OMX:TEAM:WORKER:START -->/);
@@ -64,15 +65,15 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('applyWorkerOverlay is idempotent (calling twice doesn\'t duplicate)', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("applyWorkerOverlay is idempotent (calling twice doesn't duplicate)", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const agentsMdPath = join(cwd, 'AGENTS.md');
-            await writeFile(agentsMdPath, '# Base\n', 'utf8');
-            const overlay = generateWorkerOverlay('team-idempotent');
+            const agentsMdPath = join(cwd, "AGENTS.md");
+            await writeFile(agentsMdPath, "# Base\n", "utf8");
+            const overlay = generateWorkerOverlay("team-idempotent");
             await applyWorkerOverlay(agentsMdPath, overlay);
             await applyWorkerOverlay(agentsMdPath, overlay);
-            const content = await readFile(agentsMdPath, 'utf8');
+            const content = await readFile(agentsMdPath, "utf8");
             const starts = content.match(/<!-- OMX:TEAM:WORKER:START -->/g) ?? [];
             const ends = content.match(/<!-- OMX:TEAM:WORKER:END -->/g) ?? [];
             assert.equal(starts.length, 1);
@@ -82,15 +83,15 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('stripWorkerOverlay removes the overlay section', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("stripWorkerOverlay removes the overlay section", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const agentsMdPath = join(cwd, 'AGENTS.md');
-            const base = '# Base\n\nKeep me.\n';
-            const overlay = generateWorkerOverlay('team-strip');
-            await writeFile(agentsMdPath, `${base}\n${overlay}\n`, 'utf8');
+            const agentsMdPath = join(cwd, "AGENTS.md");
+            const base = "# Base\n\nKeep me.\n";
+            const overlay = generateWorkerOverlay("team-strip");
+            await writeFile(agentsMdPath, `${base}\n${overlay}\n`, "utf8");
             await stripWorkerOverlay(agentsMdPath);
-            const content = await readFile(agentsMdPath, 'utf8');
+            const content = await readFile(agentsMdPath, "utf8");
             assert.match(content, /# Base/);
             assert.match(content, /Keep me\./);
             assert.doesNotMatch(content, /<!-- OMX:TEAM:WORKER:START -->/);
@@ -100,16 +101,16 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('stripWorkerOverlay is idempotent (calling on already-stripped is no-op)', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("stripWorkerOverlay is idempotent (calling on already-stripped is no-op)", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const agentsMdPath = join(cwd, 'AGENTS.md');
-            await writeFile(agentsMdPath, '# Base only\n', 'utf8');
-            const before = await readFile(agentsMdPath, 'utf8');
+            const agentsMdPath = join(cwd, "AGENTS.md");
+            await writeFile(agentsMdPath, "# Base only\n", "utf8");
+            const before = await readFile(agentsMdPath, "utf8");
             await stripWorkerOverlay(agentsMdPath);
-            const afterFirst = await readFile(agentsMdPath, 'utf8');
+            const afterFirst = await readFile(agentsMdPath, "utf8");
             await stripWorkerOverlay(agentsMdPath);
-            const afterSecond = await readFile(agentsMdPath, 'utf8');
+            const afterSecond = await readFile(agentsMdPath, "utf8");
             assert.equal(afterFirst, before);
             assert.equal(afterSecond, before);
         }
@@ -117,13 +118,13 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('applyWorkerOverlay works on non-existent file (creates it)', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("applyWorkerOverlay works on non-existent file (creates it)", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const agentsMdPath = join(cwd, 'AGENTS.md');
-            const overlay = generateWorkerOverlay('new-team');
+            const agentsMdPath = join(cwd, "AGENTS.md");
+            const overlay = generateWorkerOverlay("new-team");
             await applyWorkerOverlay(agentsMdPath, overlay);
-            const content = await readFile(agentsMdPath, 'utf8');
+            const content = await readFile(agentsMdPath, "utf8");
             assert.match(content, /<!-- OMX:TEAM:WORKER:START -->/);
             assert.match(content, /team "new-team"/);
         }
@@ -131,42 +132,42 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('applyWorkerOverlay reaps stale AGENTS lock directory', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("applyWorkerOverlay reaps stale AGENTS lock directory", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const agentsMdPath = join(cwd, 'AGENTS.md');
-            const lockPath = join(cwd, '.omx', 'state', 'agents-md.lock');
+            const agentsMdPath = join(cwd, "AGENTS.md");
+            const lockPath = join(cwd, ".omx", "state", "agents-md.lock");
             await mkdir(lockPath, { recursive: true });
-            await writeFile(join(lockPath, 'owner.json'), JSON.stringify({ pid: 999_999_999, ts: Date.now() - 60_000 }), 'utf8');
-            await writeFile(agentsMdPath, '# Base\n', 'utf8');
-            const overlay = generateWorkerOverlay('team-stale-lock');
+            await writeFile(join(lockPath, "owner.json"), JSON.stringify({ pid: 999_999_999, ts: Date.now() - 60_000 }), "utf8");
+            await writeFile(agentsMdPath, "# Base\n", "utf8");
+            const overlay = generateWorkerOverlay("team-stale-lock");
             await applyWorkerOverlay(agentsMdPath, overlay);
-            const content = await readFile(agentsMdPath, 'utf8');
+            const content = await readFile(agentsMdPath, "utf8");
             assert.match(content, /team "team-stale-lock"/);
-            await assert.rejects(readFile(join(lockPath, 'owner.json'), 'utf8'));
+            await assert.rejects(readFile(join(lockPath, "owner.json"), "utf8"));
         }
         finally {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('generateInitialInbox includes worker name, team name, and all tasks', () => {
+    it("generateInitialInbox includes worker name, team name, and all tasks", () => {
         const tasks = [
             {
-                id: '1',
-                subject: 'First task',
-                description: 'Do first thing',
-                status: 'pending',
+                id: "1",
+                subject: "First task",
+                description: "Do first thing",
+                status: "pending",
                 created_at: new Date().toISOString(),
             },
             {
-                id: '2',
-                subject: 'Second task',
-                description: 'Do second thing',
-                status: 'in_progress',
+                id: "2",
+                subject: "Second task",
+                description: "Do second thing",
+                status: "in_progress",
                 created_at: new Date().toISOString(),
             },
         ];
-        const inbox = generateInitialInbox('worker-1', 'team-inbox', 'executor', tasks);
+        const inbox = generateInitialInbox("worker-1", "team-inbox", "executor", tasks);
         assert.match(inbox, /# Worker Assignment: worker-1/);
         assert.match(inbox, /\*\*Team:\*\* team-inbox/);
         assert.match(inbox, /\*\*Role:\*\* executor/);
@@ -178,7 +179,7 @@ describe('worker bootstrap', () => {
         assert.match(inbox, /omx team api transition-task-status/);
         assert.match(inbox, /omx team api release-task-claim/);
         assert.match(inbox, /\$\{CODEX_HOME:-~\/\.codex\}\/skills\/worker\/SKILL\.md/);
-        assert.match(inbox, /~\/\.agents\/skills\/worker\/SKILL\.md/);
+        assert.match(inbox, /\/\.codex\/skills\/worker\/SKILL\.md/);
         assert.match(inbox, /ACK: worker-1 initialized/);
         assert.match(inbox, /Mailbox Delivery Protocol \(Required\)/);
         assert.match(inbox, /mailbox-mark-delivered/);
@@ -187,60 +188,85 @@ describe('worker bootstrap', () => {
         assert.match(inbox, /Verification Requirements/);
         assert.match(inbox, /Fix-Verify Loop/);
     });
-    it('generateInitialInbox shows blocked_by info for blocked tasks', () => {
+    it("generateInitialInbox shows blocked_by info for blocked tasks", () => {
         const tasks = [
             {
-                id: '3',
-                subject: 'Blocked task',
-                description: 'Wait on dependencies',
-                status: 'pending',
-                blocked_by: ['1', '2'],
+                id: "3",
+                subject: "Blocked task",
+                description: "Wait on dependencies",
+                status: "pending",
+                blocked_by: ["1", "2"],
                 created_at: new Date().toISOString(),
             },
         ];
-        const inbox = generateInitialInbox('worker-2', 'team-blocked', 'executor', tasks);
+        const inbox = generateInitialInbox("worker-2", "team-blocked", "executor", tasks);
         assert.match(inbox, /Blocked by: 1, 2/);
     });
-    it('generateInitialInbox uses workerRole when provided', () => {
+    it("generateInitialInbox uses workerRole when provided", () => {
         const tasks = [
-            { id: '1', subject: 'Test task', description: 'Write tests', status: 'pending', created_at: new Date().toISOString() },
+            {
+                id: "1",
+                subject: "Test task",
+                description: "Write tests",
+                status: "pending",
+                created_at: new Date().toISOString(),
+            },
         ];
-        const inbox = generateInitialInbox('worker-1', 'team-role', 'executor', tasks, {
-            workerRole: 'test-engineer',
+        const inbox = generateInitialInbox("worker-1", "team-role", "executor", tasks, {
+            workerRole: "test-engineer",
         });
         assert.match(inbox, /\*\*Role:\*\* test-engineer/);
         assert.doesNotMatch(inbox, /\*\*Role:\*\* executor/);
     });
-    it('generateInitialInbox includes specialization section when rolePromptContent provided', () => {
+    it("generateInitialInbox includes specialization section when rolePromptContent provided", () => {
         const tasks = [
-            { id: '1', subject: 'Design UI', description: 'Build component', status: 'pending', created_at: new Date().toISOString() },
+            {
+                id: "1",
+                subject: "Design UI",
+                description: "Build component",
+                status: "pending",
+                created_at: new Date().toISOString(),
+            },
         ];
-        const inbox = generateInitialInbox('worker-2', 'team-spec', 'executor', tasks, {
-            workerRole: 'designer',
-            rolePromptContent: 'You focus on UI/UX design and component architecture.',
+        const inbox = generateInitialInbox("worker-2", "team-spec", "executor", tasks, {
+            workerRole: "designer",
+            rolePromptContent: "You focus on UI/UX design and component architecture.",
         });
         assert.match(inbox, /## Your Specialization/);
         assert.match(inbox, /\*\*designer\*\* agent/);
         assert.match(inbox, /UI\/UX design and component architecture/);
     });
-    it('generateInitialInbox omits specialization section when no rolePromptContent', () => {
+    it("generateInitialInbox omits specialization section when no rolePromptContent", () => {
         const tasks = [
-            { id: '1', subject: 'Task', description: 'Do work', status: 'pending', created_at: new Date().toISOString() },
+            {
+                id: "1",
+                subject: "Task",
+                description: "Do work",
+                status: "pending",
+                created_at: new Date().toISOString(),
+            },
         ];
-        const inbox = generateInitialInbox('worker-1', 'team-no-spec', 'executor', tasks, {
-            workerRole: 'executor',
+        const inbox = generateInitialInbox("worker-1", "team-no-spec", "executor", tasks, {
+            workerRole: "executor",
         });
         assert.doesNotMatch(inbox, /## Your Specialization/);
     });
-    it('generateInitialInbox shows task role in task list', () => {
+    it("generateInitialInbox shows task role in task list", () => {
         const tasks = [
-            { id: '1', subject: 'Test task', description: 'Write tests', status: 'pending', role: 'test-engineer', created_at: new Date().toISOString() },
+            {
+                id: "1",
+                subject: "Test task",
+                description: "Write tests",
+                status: "pending",
+                role: "test-engineer",
+                created_at: new Date().toISOString(),
+            },
         ];
-        const inbox = generateInitialInbox('worker-1', 'team-task-role', 'executor', tasks);
+        const inbox = generateInitialInbox("worker-1", "team-task-role", "executor", tasks);
         assert.match(inbox, /Role: test-engineer/);
     });
-    it('generateTaskAssignmentInbox includes task ID and description', () => {
-        const inbox = generateTaskAssignmentInbox('worker-3', 'team-followup', '42', 'Implement parser update');
+    it("generateTaskAssignmentInbox includes task ID and description", () => {
+        const inbox = generateTaskAssignmentInbox("worker-3", "team-followup", "42", "Implement parser update");
         assert.match(inbox, /\*\*Task ID:\*\* 42/);
         assert.match(inbox, /Implement parser update/);
         assert.match(inbox, /team_state_root/);
@@ -252,31 +278,31 @@ describe('worker bootstrap', () => {
         assert.match(inbox, /Verification Requirements/);
         assert.match(inbox, /PASS\/FAIL/);
     });
-    it('generateShutdownInbox contains exit instruction and concrete ack path', () => {
-        const inbox = generateShutdownInbox('team-x', 'worker-1');
+    it("generateShutdownInbox contains exit instruction and concrete ack path", () => {
+        const inbox = generateShutdownInbox("team-x", "worker-1");
         assert.match(inbox, /Shutdown Request/);
         assert.match(inbox, /team_state_root/);
         assert.match(inbox, /team\/team-x\/workers\/worker-1\/shutdown-ack\.json/);
         assert.match(inbox, /Type `exit` or press Ctrl\+C/);
     });
-    it('generateTriggerMessage is always < 200 characters', () => {
-        const message = generateTriggerMessage('worker-very-long-name', 'team-with-a-reasonably-long-name');
+    it("generateTriggerMessage is always < 200 characters", () => {
+        const message = generateTriggerMessage("worker-very-long-name", "team-with-a-reasonably-long-name");
         assert.ok(message.length < 200);
     });
-    it('generateTriggerMessage does not contain [OMX_TMUX_INJECT]', () => {
-        const message = generateTriggerMessage('worker-1', 'team-safe');
-        assert.equal(message.includes('[OMX_TMUX_INJECT]'), false);
+    it("generateTriggerMessage does not contain [OMX_TMUX_INJECT]", () => {
+        const message = generateTriggerMessage("worker-1", "team-safe");
+        assert.equal(message.includes("[OMX_TMUX_INJECT]"), false);
     });
-    it('generateTriggerMessage contains the inbox path', () => {
-        const message = generateTriggerMessage('worker-9', 'team-path');
+    it("generateTriggerMessage contains the inbox path", () => {
+        const message = generateTriggerMessage("worker-9", "team-path");
         assert.match(message, /\.omx\/state\/team\/team-path\/workers\/worker-9\/inbox\.md/);
         assert.match(message, /start work now/i);
         assert.match(message, /concrete progress/i);
         assert.match(message, /continue assigned work/i);
         assert.match(message, /next feasible task/i);
     });
-    it('generateTriggerMessage uses provided state-root reference for worktree workers', () => {
-        const message = generateTriggerMessage('worker-9', 'team-path', '$OMX_TEAM_STATE_ROOT');
+    it("generateTriggerMessage uses provided state-root reference for worktree workers", () => {
+        const message = generateTriggerMessage("worker-9", "team-path", "$OMX_TEAM_STATE_ROOT");
         assert.match(message, /\$OMX_TEAM_STATE_ROOT\/team\/team-path\/workers\/worker-9\/inbox\.md/);
         assert.match(message, /work now/i);
         assert.match(message, /report progress/i);
@@ -284,12 +310,12 @@ describe('worker bootstrap', () => {
         assert.match(message, /next feasible task/i);
         assert.ok(message.length < 200);
     });
-    it('generateMailboxTriggerMessage is always < 200 characters', () => {
-        const message = generateMailboxTriggerMessage('worker-long-name', 'team-with-long-name', 42);
+    it("generateMailboxTriggerMessage is always < 200 characters", () => {
+        const message = generateMailboxTriggerMessage("worker-long-name", "team-with-long-name", 42);
         assert.ok(message.length < 200);
     });
-    it('generateMailboxTriggerMessage contains mailbox path and count', () => {
-        const message = generateMailboxTriggerMessage('worker-2', 'team-mail', 3);
+    it("generateMailboxTriggerMessage contains mailbox path and count", () => {
+        const message = generateMailboxTriggerMessage("worker-2", "team-mail", 3);
         assert.match(message, /3 new message/);
         assert.match(message, /Read .*\.omx\/state\/team\/team-mail\/mailbox\/worker-2\.json/);
         assert.match(message, /act now/i);
@@ -297,8 +323,8 @@ describe('worker bootstrap', () => {
         assert.match(message, /continue assigned work/i);
         assert.match(message, /next feasible task/i);
     });
-    it('generateMailboxTriggerMessage uses provided state-root reference for worktree workers', () => {
-        const message = generateMailboxTriggerMessage('worker-2', 'team-mail', 3, '$OMX_TEAM_STATE_ROOT');
+    it("generateMailboxTriggerMessage uses provided state-root reference for worktree workers", () => {
+        const message = generateMailboxTriggerMessage("worker-2", "team-mail", 3, "$OMX_TEAM_STATE_ROOT");
         assert.match(message, /3 new msg/);
         assert.match(message, /read .*\$OMX_TEAM_STATE_ROOT\/team\/team-mail\/mailbox\/worker-2\.json/i);
         assert.match(message, /act/i);
@@ -307,42 +333,42 @@ describe('worker bootstrap', () => {
         assert.match(message, /next feasible task/i);
         assert.ok(message.length < 200);
     });
-    it('generateLeaderMailboxTriggerMessage is always < 200 characters', () => {
-        const message = generateLeaderMailboxTriggerMessage('team-with-long-name', 'worker-long-name');
+    it("generateLeaderMailboxTriggerMessage is always < 200 characters", () => {
+        const message = generateLeaderMailboxTriggerMessage("team-with-long-name", "worker-long-name");
         assert.ok(message.length < 200);
     });
-    it('generateLeaderMailboxTriggerMessage tells the leader to read the mailbox and reply', () => {
-        const message = generateLeaderMailboxTriggerMessage('team-mail', 'worker-2');
+    it("generateLeaderMailboxTriggerMessage tells the leader to read the mailbox and reply", () => {
+        const message = generateLeaderMailboxTriggerMessage("team-mail", "worker-2");
         assert.match(message, /Read .*\.omx\/state\/team\/team-mail\/mailbox\/leader-fixed\.json/);
         assert.match(message, /worker-2 sent a new message/);
         assert.match(message, /Reply with the next concrete step/);
     });
-    it('generateLeaderMailboxTriggerMessage uses provided state-root reference for worktree leaders', () => {
-        const message = generateLeaderMailboxTriggerMessage('team-mail', 'worker-2', '$OMX_TEAM_STATE_ROOT');
+    it("generateLeaderMailboxTriggerMessage uses provided state-root reference for worktree leaders", () => {
+        const message = generateLeaderMailboxTriggerMessage("team-mail", "worker-2", "$OMX_TEAM_STATE_ROOT");
         assert.match(message, /read .*\$OMX_TEAM_STATE_ROOT\/team\/team-mail\/mailbox\/leader-fixed\.json/i);
         assert.match(message, /new msg from worker-2/i);
         assert.match(message, /reply next step/i);
         assert.ok(message.length < 200);
     });
-    it('writeTeamWorkerInstructionsFile composes user + project AGENTS.md with overlay', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
-        const restoreCodexHome = setMockCodexHome(join(cwd, 'home', '.codex'));
+    it("writeTeamWorkerInstructionsFile composes user + project AGENTS.md with overlay", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
+        const restoreCodexHome = setMockCodexHome(join(cwd, "home", ".codex"));
         try {
-            await mkdir(join(cwd, 'home', '.codex'), { recursive: true });
-            await writeFile(join(cwd, 'home', '.codex', 'AGENTS.md'), '# User Instructions\n\nStart globally.\n', 'utf8');
-            await writeFile(join(cwd, 'AGENTS.md'), '# Project Instructions\n\nDo good work.\n', 'utf8');
-            const overlay = generateWorkerOverlay('compose-team');
-            const outPath = await writeTeamWorkerInstructionsFile('compose-team', cwd, overlay);
-            const content = await readFile(outPath, 'utf8');
+            await mkdir(join(cwd, "home", ".codex"), { recursive: true });
+            await writeFile(join(cwd, "home", ".codex", "AGENTS.md"), "# User Instructions\n\nStart globally.\n", "utf8");
+            await writeFile(join(cwd, "AGENTS.md"), "# Project Instructions\n\nDo good work.\n", "utf8");
+            const overlay = generateWorkerOverlay("compose-team");
+            const outPath = await writeTeamWorkerInstructionsFile("compose-team", cwd, overlay);
+            const content = await readFile(outPath, "utf8");
             assert.match(content, /# User Instructions/);
             assert.match(content, /# Project Instructions/);
-            assert.ok(content.indexOf('# User Instructions') <
-                content.indexOf('# Project Instructions'));
+            assert.ok(content.indexOf("# User Instructions") <
+                content.indexOf("# Project Instructions"));
             assert.match(content, /Do good work/);
             assert.match(content, /<!-- OMX:TEAM:WORKER:START -->/);
             assert.match(content, /<!-- OMX:TEAM:WORKER:END -->/);
             // Verify project AGENTS.md was NOT modified
-            const projectContent = await readFile(join(cwd, 'AGENTS.md'), 'utf8');
+            const projectContent = await readFile(join(cwd, "AGENTS.md"), "utf8");
             assert.doesNotMatch(projectContent, /<!-- OMX:TEAM:WORKER:START -->/);
         }
         finally {
@@ -350,24 +376,24 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('writeTeamWorkerInstructionsFile deduplicates duplicate skill references in favor of project scope', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
-        const restoreCodexHome = setMockCodexHome(join(cwd, 'home', '.codex'));
+    it("writeTeamWorkerInstructionsFile deduplicates duplicate skill references in favor of project scope", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
+        const restoreCodexHome = setMockCodexHome(join(cwd, "home", ".codex"));
         try {
-            const userAgentsPath = join(cwd, 'home', '.codex', 'AGENTS.md');
-            const projectAgentsPath = join(cwd, 'AGENTS.md');
-            const userSkillDir = join(cwd, 'home', '.codex', 'skills', 'help');
-            const projectSkillDir = join(cwd, '.agents', 'skills', 'help');
-            await mkdir(join(cwd, 'home', '.codex'), { recursive: true });
+            const userAgentsPath = join(cwd, "home", ".codex", "AGENTS.md");
+            const projectAgentsPath = join(cwd, "AGENTS.md");
+            const userSkillDir = join(cwd, "home", ".codex", "skills", "help");
+            const projectSkillDir = join(cwd, ".codex", "skills", "help");
+            await mkdir(join(cwd, "home", ".codex"), { recursive: true });
             await mkdir(userSkillDir, { recursive: true });
             await mkdir(projectSkillDir, { recursive: true });
-            await writeFile(join(userSkillDir, 'SKILL.md'), '# user help\n', 'utf8');
-            await writeFile(join(projectSkillDir, 'SKILL.md'), '# project help\n', 'utf8');
-            await writeFile(userAgentsPath, '- help user (file: /tmp/home/.codex/skills/help/SKILL.md)\n', 'utf8');
-            await writeFile(projectAgentsPath, '- help project (file: /tmp/project/.agents/skills/help/SKILL.md)\n', 'utf8');
-            const overlay = generateWorkerOverlay('dedupe-team');
-            const outPath = await writeTeamWorkerInstructionsFile('dedupe-team', cwd, overlay);
-            const content = await readFile(outPath, 'utf8');
+            await writeFile(join(userSkillDir, "SKILL.md"), "# user help\n", "utf8");
+            await writeFile(join(projectSkillDir, "SKILL.md"), "# project help\n", "utf8");
+            await writeFile(userAgentsPath, "- help user (file: /tmp/home/.codex/skills/help/SKILL.md)\n", "utf8");
+            await writeFile(projectAgentsPath, "- help project (file: /tmp/project/.codex/skills/help/SKILL.md)\n", "utf8");
+            const overlay = generateWorkerOverlay("dedupe-team");
+            const outPath = await writeTeamWorkerInstructionsFile("dedupe-team", cwd, overlay);
+            const content = await readFile(outPath, "utf8");
             assert.equal((content.match(/skills\/help\/SKILL\.md/g) || []).length, 1);
             assert.doesNotMatch(content, /help user/);
             assert.match(content, /help project/);
@@ -377,13 +403,13 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('writeWorkerRoleInstructionsFile layers role prompt on top of team worker instructions', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("writeWorkerRoleInstructionsFile layers role prompt on top of team worker instructions", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const overlay = generateWorkerOverlay('role-team');
-            const basePath = await writeTeamWorkerInstructionsFile('role-team', cwd, overlay);
-            const outPath = await writeWorkerRoleInstructionsFile('role-team', 'worker-2', cwd, basePath, 'writer', '<identity>Writer role prompt</identity>');
-            const content = await readFile(outPath, 'utf8');
+            const overlay = generateWorkerOverlay("role-team");
+            const basePath = await writeTeamWorkerInstructionsFile("role-team", cwd, overlay);
+            const outPath = await writeWorkerRoleInstructionsFile("role-team", "worker-2", cwd, basePath, "writer", "<identity>Writer role prompt</identity>");
+            const content = await readFile(outPath, "utf8");
             assert.match(content, /team "role-team"/);
             assert.match(content, /<!-- OMX:TEAM:ROLE:START -->/);
             assert.match(content, /\*\*writer\*\* role/);
@@ -393,12 +419,12 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('writeTeamWorkerInstructionsFile works without project AGENTS.md', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("writeTeamWorkerInstructionsFile works without project AGENTS.md", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const overlay = generateWorkerOverlay('no-agents-team');
-            const outPath = await writeTeamWorkerInstructionsFile('no-agents-team', cwd, overlay);
-            const content = await readFile(outPath, 'utf8');
+            const overlay = generateWorkerOverlay("no-agents-team");
+            const outPath = await writeTeamWorkerInstructionsFile("no-agents-team", cwd, overlay);
+            const content = await readFile(outPath, "utf8");
             assert.match(content, /<!-- OMX:TEAM:WORKER:START -->/);
             assert.match(content, /team "no-agents-team"/);
         }
@@ -406,25 +432,25 @@ describe('worker bootstrap', () => {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('removeTeamWorkerInstructionsFile cleans up the file', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("removeTeamWorkerInstructionsFile cleans up the file", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
-            const overlay = generateWorkerOverlay('cleanup-team');
-            await writeTeamWorkerInstructionsFile('cleanup-team', cwd, overlay);
-            await removeTeamWorkerInstructionsFile('cleanup-team', cwd);
-            const { existsSync } = await import('fs');
-            const outPath = join(cwd, '.omx', 'state', 'team', 'cleanup-team', 'worker-agents.md');
+            const overlay = generateWorkerOverlay("cleanup-team");
+            await writeTeamWorkerInstructionsFile("cleanup-team", cwd, overlay);
+            await removeTeamWorkerInstructionsFile("cleanup-team", cwd);
+            const { existsSync } = await import("fs");
+            const outPath = join(cwd, ".omx", "state", "team", "cleanup-team", "worker-agents.md");
             assert.equal(existsSync(outPath), false);
         }
         finally {
             await rm(cwd, { recursive: true, force: true });
         }
     });
-    it('removeTeamWorkerInstructionsFile is safe to call when file does not exist', async () => {
-        const cwd = await mkdtemp(join(tmpdir(), 'omx-worker-bootstrap-'));
+    it("removeTeamWorkerInstructionsFile is safe to call when file does not exist", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-worker-bootstrap-"));
         try {
             // Should not throw
-            await removeTeamWorkerInstructionsFile('nonexistent-team', cwd);
+            await removeTeamWorkerInstructionsFile("nonexistent-team", cwd);
         }
         finally {
             await rm(cwd, { recursive: true, force: true });