oh-my-codex 0.16.3 → 0.17.0

package/dist/planning/__tests__/artifacts.test.js

@@ -1,10 +1,11 @@
-import { afterEach, beforeEach, describe, it } from 'node:test';
+import { afterEach, beforeEach, describe, it, mock } from 'node:test';
 import assert from 'node:assert/strict';
 import { mkdir, mkdtemp, readFile, rm, writeFile } from 'node:fs/promises';
-import { existsSync } from 'node:fs';
+import fs, { existsSync } from 'node:fs';
 import { createHash } from 'node:crypto';
+import { syncBuiltinESMExports } from 'node:module';
 import { tmpdir } from 'node:os';
-import { join, relative } from 'node:path';
+import { basename, join, relative } from 'node:path';
 import { decodeApprovedExecutionQuotedValue, isPlanningComplete, readApprovedExecutionLaunchHint, readApprovedExecutionLaunchHintOutcome, readLatestPlanningArtifacts, readPlanningArtifacts, readTeamDagArtifactResolution, } from '../artifacts.js';
 import { readTeamDagHandoffForLatestPlan } from '../../team/dag-schema.js';
 let tempDir;
@@ -56,6 +57,28 @@ async function writeContextPack(slug, prdPath, testSpecPath, roles) {
     }, null, 2));
     return packPath;
 }
+async function writeContextPackWithEntries(slug, prdPath, testSpecPath, entries) {
+    const contextDir = join(tempDir, '.omx', 'context');
+    await mkdir(contextDir, { recursive: true });
+    const packPath = join(tempDir, canonicalContextPackRelativePath(slug));
+    const prdContent = await readFile(prdPath, 'utf-8');
+    const testSpecContent = await readFile(testSpecPath, 'utf-8');
+    await writeFile(packPath, JSON.stringify({
+        slug,
+        basis: {
+            prd: {
+                path: relativeToRepo(prdPath),
+                sha1: computeGitBlobSha1(prdContent),
+            },
+            testSpecs: [{
+                    path: relativeToRepo(testSpecPath),
+                    sha1: computeGitBlobSha1(testSpecContent),
+                }],
+        },
+        entries,
+    }, null, 2));
+    return packPath;
+}
 async function setup() {
     tempDir = await mkdtemp(join(tmpdir(), 'omx-planning-artifacts-'));
 }
@@ -66,7 +89,11 @@ async function cleanup() {
 }
 describe('planning artifacts', () => {
     beforeEach(async () => { await setup(); });
-    afterEach(async () => { await cleanup(); });
+    afterEach(async () => {
+        mock.restoreAll();
+        syncBuiltinESMExports();
+        await cleanup();
+    });
     it('round-trips single-quoted approved execution tasks with only escaped apostrophes normalized', () => {
         const task = String.raw `Fix Bob's regression in C:\\tmp`;
         assert.equal(decodeApprovedExecutionQuotedValue(encodeApprovedExecutionTask(task, 'single')), task);
@@ -273,6 +300,112 @@ describe('planning artifacts', () => {
         assert.deepEqual(hint?.missingRequiredContextPackRoles, []);
         assert.deepEqual(hint?.contextPackIssues, []);
     });
+    it('keeps approved hint role refs unchanged when ready packs carry private entry metadata', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        await mkdir(plansDir, { recursive: true });
+        const prdPath = join(plansDir, 'prd-context-ready-private.md');
+        const testSpecPath = join(plansDir, 'test-spec-context-ready-private.md');
+        await writeFile(prdPath, [
+            '# PRD',
+            '',
+            buildContextPackOutcome(canonicalContextPackRelativePath('context-ready-private')),
+            '',
+            'Launch via omx ralph "Execute context-ready private handoff"',
+        ].join('\n'));
+        await writeFile(testSpecPath, '# Test Spec\n');
+        const packPath = await writeContextPackWithEntries('context-ready-private', prdPath, testSpecPath, [
+            {
+                path: 'src/scope-ready.ts',
+                roles: ['scope'],
+            },
+            {
+                path: 'src/build-ready.ts',
+                roles: ['build'],
+                label: 'Build Focus',
+                tags: ['runtime', 'build'],
+                selector: { type: 'heading', value: ' ## Build Focus ', maxWords: 120 },
+                relationPath: [
+                    { tag: 'Plan', target: ' context-ready-private ' },
+                    { tag: 'Implements', target: ' src/build-ready.ts#build-focus ' },
+                ],
+            },
+            {
+                path: 'src/verify-ready.ts',
+                roles: ['verify'],
+                relationPath: [
+                    { tag: 'Plan', target: ' context-ready-private ' },
+                    { tag: 'Verifies', target: ' src/verify-ready.ts ' },
+                ],
+            },
+        ]);
+        const selection = readLatestPlanningArtifacts(tempDir);
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'ralph');
+        assert.deepEqual(selection.contextPack, { path: packPath });
+        assert.equal(selection.contextPackStatus, 'ready');
+        assert.deepEqual(selection.contextPackRoleRefs, {
+            scope: ['src/scope-ready.ts'],
+            build: ['src/build-ready.ts'],
+            verify: ['src/verify-ready.ts'],
+        });
+        assert.ok(hint);
+        assert.deepEqual(hint?.contextPack, { path: packPath });
+        assert.equal(hint?.contextPackStatus, 'ready');
+        assert.deepEqual(hint?.contextPackRoleRefs, {
+            scope: ['src/scope-ready.ts'],
+            build: ['src/build-ready.ts'],
+            verify: ['src/verify-ready.ts'],
+        });
+        assert.deepEqual(hint?.missingRequiredContextPackRoles, []);
+        assert.deepEqual(hint?.contextPackIssues, []);
+    });
+    it('keeps approved hint role refs unchanged when ready packs carry malformed private metadata', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        await mkdir(plansDir, { recursive: true });
+        const prdPath = join(plansDir, 'prd-context-ready-private-counterfactual.md');
+        const testSpecPath = join(plansDir, 'test-spec-context-ready-private-counterfactual.md');
+        await writeFile(prdPath, [
+            '# PRD',
+            '',
+            buildContextPackOutcome(canonicalContextPackRelativePath('context-ready-private-counterfactual')),
+            '',
+            'Launch via omx ralph "Execute context-ready private counterfactual handoff"',
+        ].join('\n'));
+        await writeFile(testSpecPath, '# Test Spec\n');
+        const packPath = await writeContextPackWithEntries('context-ready-private-counterfactual', prdPath, testSpecPath, [
+            {
+                path: 'src/scope-counterfactual.ts',
+                roles: ['scope'],
+            },
+            {
+                path: 'src/build-counterfactual.ts',
+                roles: ['build'],
+                selector: { type: 'heading', value: 'Build Focus', maxWords: 20 },
+            },
+            {
+                path: 'src/verify-counterfactual.ts',
+                roles: ['verify'],
+            },
+        ]);
+        const selection = readLatestPlanningArtifacts(tempDir);
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'ralph');
+        assert.deepEqual(selection.contextPack, { path: packPath });
+        assert.equal(selection.contextPackStatus, 'ready');
+        assert.deepEqual(selection.contextPackRoleRefs, {
+            scope: ['src/scope-counterfactual.ts'],
+            build: ['src/build-counterfactual.ts'],
+            verify: ['src/verify-counterfactual.ts'],
+        });
+        assert.ok(hint);
+        assert.deepEqual(hint?.contextPack, { path: packPath });
+        assert.equal(hint?.contextPackStatus, 'ready');
+        assert.deepEqual(hint?.contextPackRoleRefs, {
+            scope: ['src/scope-counterfactual.ts'],
+            build: ['src/build-counterfactual.ts'],
+            verify: ['src/verify-counterfactual.ts'],
+        });
+        assert.deepEqual(hint?.missingRequiredContextPackRoles, []);
+        assert.deepEqual(hint?.contextPackIssues, []);
+    });
     it('preserves invalid context-pack issues on approved hints without widening them into missing roles', async () => {
         const plansDir = join(tempDir, '.omx', 'plans');
         await mkdir(plansDir, { recursive: true });
@@ -437,6 +570,49 @@ describe('planning artifacts', () => {
         assert.equal(hint?.sourcePath, alphaPrdPath);
         assert.deepEqual(hint?.testSpecPaths, [join(plansDir, 'test-spec-alpha.md')]);
     });
+    it('binds approved launch hints through canonical-equivalent requested PRD aliases', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        await mkdir(plansDir, { recursive: true });
+        const alphaPrdPath = join(plansDir, 'prd-alpha.md');
+        await writeFile(alphaPrdPath, '# Alpha\n\nLaunch via omx ralph "Execute alpha"\n');
+        await writeFile(join(plansDir, 'test-spec-alpha.md'), '# Alpha Test Spec\n');
+        await writeFile(join(plansDir, 'prd-zeta.md'), '# Zeta\n\nLaunch via omx ralph "Execute zeta"\n');
+        await writeFile(join(plansDir, 'test-spec-zeta.md'), '# Zeta Test Spec\n');
+        const aliases = [
+            '.omx/plans/prd-alpha.md',
+            'prd-alpha.md',
+            alphaPrdPath,
+            '.omx/plans/../plans/prd-alpha.md',
+            '../plans/prd-alpha.md',
+            join(tempDir, '.omx', 'plans', '..', 'plans', 'prd-alpha.md'),
+        ];
+        for (const prdPath of aliases) {
+            const outcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { prdPath });
+            assert.equal(outcome.status, 'resolved');
+            if (outcome.status !== 'resolved') {
+                throw new Error(`expected resolved hint for alias ${prdPath}`);
+            }
+            assert.equal(outcome.hint.task, 'Execute alpha');
+            assert.equal(outcome.hint.sourcePath, alphaPrdPath);
+            assert.deepEqual(outcome.hint.testSpecPaths, [join(plansDir, 'test-spec-alpha.md')]);
+        }
+    });
+    it('does not bind requested PRD aliases that do not resolve to a discovered canonical PRD', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-alpha.md'), '# Alpha\n\nLaunch via omx ralph "Execute alpha"\n');
+        await writeFile(join(plansDir, 'test-spec-alpha.md'), '# Alpha Test Spec\n');
+        const rejectedAliases = [
+            '.omx/plans/prd-missing.md',
+            '../prd-alpha.md',
+            join('..', basename(tempDir), '.omx', 'plans', 'prd-alpha.md'),
+            join(tempDir, '.omx', 'prd-alpha.md'),
+            relative(process.cwd(), join(plansDir, 'prd-alpha.md')),
+        ];
+        for (const prdPath of rejectedAliases) {
+            assert.equal(readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { prdPath }).status, 'absent');
+        }
+    });
     it('honors the requested Ralph task when a single plan lists multiple Ralph launch hints', async () => {
         const plansDir = join(tempDir, '.omx', 'plans');
         await mkdir(plansDir, { recursive: true });
@@ -452,6 +628,30 @@ describe('planning artifacts', () => {
         assert.equal(hint?.task, 'Execute alpha');
         assert.equal(hint?.command, 'omx ralph "Execute alpha"');
     });
+    it('reuses one planning artifact scan while task lookup checks older same-lineage PRDs', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute cached planning artifact lineage lookup';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-alpha.md'), `# Alpha\n\nLaunch via omx ralph ${JSON.stringify(task)}\n`);
+        await writeFile(join(plansDir, 'test-spec-alpha.md'), '# Alpha Test Spec\n');
+        await writeFile(join(plansDir, 'prd-beta.md'), `# Beta\n\nLaunch via omx ralph ${JSON.stringify(task)}\n`);
+        await writeFile(join(plansDir, 'prd-gamma.md'), `# Gamma\n\nLaunch via omx ralph ${JSON.stringify(task)}\n`);
+        await writeFile(join(plansDir, 'prd-zeta.md'), `# Zeta\n\nLaunch via omx ralph ${JSON.stringify(task)}\n`);
+        let readdirCount = 0;
+        const originalReaddirSync = fs.readdirSync;
+        mock.method(fs, 'readdirSync', ((...args) => {
+            readdirCount += 1;
+            return Reflect.apply(originalReaddirSync, fs, args);
+        }));
+        syncBuiltinESMExports();
+        const outcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { task });
+        assert.equal(outcome.status, 'resolved');
+        if (outcome.status !== 'resolved') {
+            return;
+        }
+        assert.equal(outcome.hint.sourcePath, join(plansDir, 'prd-alpha.md'));
+        assert.equal(readdirCount, 2);
+    });
     it('fails closed for bare Ralph lookups when a single plan lists multiple Ralph launch hints', async () => {
         const plansDir = join(tempDir, '.omx', 'plans');
         await mkdir(plansDir, { recursive: true });
@@ -465,6 +665,335 @@ describe('planning artifacts', () => {
         const hint = readApprovedExecutionLaunchHint(tempDir, 'ralph');
         assert.equal(hint, null);
     });
+    it('ignores Ralph launch hints that appear only inside indented code blocks', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute hidden indented ralph plan';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-hidden-indented-ralph.md'), [
+            '# PRD',
+            '',
+            `    omx ralph ${JSON.stringify(task)}`,
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-hidden-indented-ralph.md'), '# Test Spec\n');
+        const outcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { task });
+        assert.equal(outcome.status, 'absent');
+    });
+    it('resolves Ralph launch hints wrapped across visible markdown lines', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute wrapped visible ralph plan';
+        const command = `omx ralph ${JSON.stringify(task)}`;
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-wrapped-visible-ralph.md'), [
+            '# PRD',
+            '',
+            'Launch via omx ralph',
+            JSON.stringify(task),
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-wrapped-visible-ralph.md'), '# Test Spec\n');
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'ralph', { task });
+        assert.ok(hint);
+        assert.equal(hint?.task, task);
+        assert.equal(hint?.command, command);
+        assert.equal(hint?.contextPackStatus, 'plan-only');
+        const exactHint = readApprovedExecutionLaunchHint(tempDir, 'ralph', { command });
+        assert.ok(exactHint);
+        assert.equal(exactHint?.command, command);
+    });
+    it('does not let Ralph launch hints span hidden markdown gaps', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        await mkdir(plansDir, { recursive: true });
+        const scenarios = [
+            {
+                name: 'fenced',
+                buildHiddenLines: (task) => [
+                    '```md',
+                    JSON.stringify(task),
+                    '```',
+                ],
+            },
+            {
+                name: 'commented',
+                buildHiddenLines: (task) => [
+                    '<!--',
+                    JSON.stringify(task),
+                    '-->',
+                ],
+            },
+            {
+                name: 'indented',
+                buildHiddenLines: (task) => [
+                    `    ${JSON.stringify(task)}`,
+                ],
+            },
+        ];
+        for (const scenario of scenarios) {
+            const task = `Execute hidden-gap ${scenario.name} ralph plan`;
+            await writeFile(join(plansDir, `prd-hidden-gap-${scenario.name}-ralph.md`), [
+                '# PRD',
+                '',
+                'Launch via omx ralph',
+                ...scenario.buildHiddenLines(task),
+            ].join('\n'));
+            await writeFile(join(plansDir, `test-spec-hidden-gap-${scenario.name}-ralph.md`), '# Test Spec\n');
+            const outcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { task });
+            assert.equal(outcome.status, 'absent', scenario.name);
+        }
+    });
+    it('resolves Team launch hints wrapped across visible markdown lines', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute wrapped visible team plan';
+        const command = `omx team 2:executor ${JSON.stringify(task)}`;
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-wrapped-visible-team.md'), [
+            '# PRD',
+            '',
+            'Launch via omx team',
+            '2:executor',
+            JSON.stringify(task),
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-wrapped-visible-team.md'), '# Test Spec\n');
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'team', { task });
+        assert.ok(hint);
+        assert.equal(hint?.task, task);
+        assert.equal(hint?.workerCount, 2);
+        assert.equal(hint?.agentType, 'executor');
+        assert.equal(hint?.command, command);
+        assert.equal(hint?.contextPackStatus, 'plan-only');
+        const exactHint = readApprovedExecutionLaunchHint(tempDir, 'team', { command });
+        assert.ok(exactHint);
+        assert.equal(exactHint?.command, command);
+    });
+    it('normalizes wrapped linked-Ralph team launch hints for exact command matching', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute wrapped linked ralph team plan';
+        const command = `$team ralph 5:debugger ${JSON.stringify(task)}`;
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-wrapped-visible-team-linked-ralph.md'), [
+            '# PRD',
+            '',
+            'Launch via $team',
+            'ralph',
+            '5:debugger',
+            JSON.stringify(task),
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-wrapped-visible-team-linked-ralph.md'), '# Test Spec\n');
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'team', { command });
+        assert.ok(hint);
+        assert.equal(hint?.command, command);
+        assert.equal(hint?.workerCount, 5);
+        assert.equal(hint?.agentType, 'debugger');
+        assert.equal(hint?.linkedRalph, true);
+    });
+    it('keeps exact-command normalization bounded to visible whitespace-only variants', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        await mkdir(plansDir, { recursive: true });
+        const task = 'Execute exact-command normalization state table plan';
+        const command = `omx team 2:executor ${JSON.stringify(task)}`;
+        const cases = [
+            {
+                name: 'visible-whitespace-only-variant',
+                prdPath: 'prd-exact-command-visible.md',
+                content: [
+                    '# PRD',
+                    '',
+                    'Launch via omx team',
+                    '2:executor',
+                    JSON.stringify(task),
+                ].join('\n'),
+                expectedStatus: 'resolved',
+            },
+            {
+                name: 'hidden-gap-counterfactual',
+                prdPath: 'prd-exact-command-hidden-gap.md',
+                content: [
+                    '# PRD',
+                    '',
+                    'Launch via omx team',
+                    '```md',
+                    'hidden',
+                    '```',
+                    '2:executor',
+                    JSON.stringify(task),
+                ].join('\n'),
+                expectedStatus: 'absent',
+            },
+            {
+                name: 'formatting-only-duplicate',
+                prdPath: 'prd-exact-command-duplicate.md',
+                content: [
+                    '# PRD',
+                    '',
+                    `Launch via ${command}`,
+                    'Launch via omx team',
+                    '2:executor',
+                    JSON.stringify(task),
+                ].join('\n'),
+                expectedStatus: 'ambiguous',
+            },
+        ];
+        for (const scenario of cases) {
+            await writeFile(join(plansDir, scenario.prdPath), scenario.content);
+            await writeFile(join(plansDir, scenario.prdPath.replace(/^prd-/, 'test-spec-')), '# Test Spec\n');
+            const outcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'team', { command });
+            assert.equal(outcome.status, scenario.expectedStatus, scenario.name);
+            await rm(join(plansDir, scenario.prdPath), { force: true });
+            await rm(join(plansDir, scenario.prdPath.replace(/^prd-/, 'test-spec-')), { force: true });
+        }
+    });
+    it('does not normalize whitespace that changes the quoted task payload', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        await mkdir(plansDir, { recursive: true });
+        const task = 'Execute embedded\nnewline task';
+        const exactCommand = [
+            'omx ralph "Execute embedded',
+            'newline task"',
+        ].join('\n');
+        const collapsedCommand = 'omx ralph "Execute embedded newline task"';
+        await writeFile(join(plansDir, 'prd-exact-command-embedded-newline-task.md'), [
+            '# PRD',
+            '',
+            'Launch via omx ralph "Execute embedded',
+            'newline task"',
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-exact-command-embedded-newline-task.md'), '# Test Spec\n');
+        const exactOutcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { command: exactCommand });
+        assert.equal(exactOutcome.status, 'resolved');
+        if (exactOutcome.status !== 'resolved') {
+            return;
+        }
+        assert.equal(exactOutcome.hint.command, exactCommand);
+        assert.equal(exactOutcome.hint.task, task);
+        assert.equal(readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { command: collapsedCommand }).status, 'absent');
+    });
+    it('ignores Team launch hints inside fenced code blocks', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute approved issue 1314 fenced team plan';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-issue-1314-fenced-team.md'), [
+            '# PRD',
+            '',
+            '```sh',
+            `omx team 5:reviewer ${JSON.stringify(task)}`,
+            '```',
+            '',
+            `Launch via omx team 2:executor ${JSON.stringify(task)}`,
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-issue-1314-fenced-team.md'), '# Test Spec\n');
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'team');
+        assert.ok(hint);
+        assert.equal(hint?.task, task);
+        assert.equal(hint?.workerCount, 2);
+        assert.equal(hint?.agentType, 'executor');
+        assert.equal(hint?.command, `omx team 2:executor ${JSON.stringify(task)}`);
+        assert.equal(hint?.contextPackStatus, 'plan-only');
+    });
+    it('ignores Team launch hints that appear only inside fenced code blocks', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute hidden fenced team plan';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-hidden-fenced-team.md'), [
+            '# PRD',
+            '',
+            '```sh',
+            `omx team 2:executor ${JSON.stringify(task)}`,
+            '```',
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-hidden-fenced-team.md'), '# Test Spec\n');
+        const outcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'team', { task });
+        assert.equal(outcome.status, 'absent');
+    });
+    it('ignores adversarial hidden Team launch hints that try to break out with deceptive fence lines', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute adversarial fenced team plan';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-adversarial-fenced-team.md'), [
+            '# PRD',
+            '',
+            '```md',
+            `omx team 9:reviewer ${JSON.stringify(task)}`,
+            '    ```',
+            '```still-open',
+            '~~~',
+            `omx team 7:critic ${JSON.stringify(task)}`,
+            '```',
+            '',
+            `Launch via omx team 2:executor ${JSON.stringify(task)}`,
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-adversarial-fenced-team.md'), '# Test Spec\n');
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'team');
+        assert.ok(hint);
+        assert.equal(hint?.task, task);
+        assert.equal(hint?.workerCount, 2);
+        assert.equal(hint?.agentType, 'executor');
+        assert.equal(hint?.command, `omx team 2:executor ${JSON.stringify(task)}`);
+        assert.equal(hint?.contextPackStatus, 'plan-only');
+    });
+    it('ignores Team launch hints inside nested commented blocks with inner fence lookalikes', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute commented nested team plan';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-commented-nested-team.md'), [
+            '# PRD',
+            '',
+            '<!--',
+            `Launch via omx team 9:reviewer ${JSON.stringify(task)}`,
+            '<!--',
+            `omx team 7:critic ${JSON.stringify(task)}`,
+            '```md',
+            `Launch via omx team 6:debugger ${JSON.stringify(task)}`,
+            '-->',
+            '~~~',
+            '-->',
+            '',
+            `Launch via omx team 2:executor ${JSON.stringify(task)}`,
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-commented-nested-team.md'), '# Test Spec\n');
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'team');
+        assert.ok(hint);
+        assert.equal(hint?.task, task);
+        assert.equal(hint?.workerCount, 2);
+        assert.equal(hint?.agentType, 'executor');
+        assert.equal(hint?.command, `omx team 2:executor ${JSON.stringify(task)}`);
+        assert.equal(hint?.contextPackStatus, 'plan-only');
+    });
+    it('ignores Ralph launch hints inside indented code blocks', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute approved issue 1314 indented ralph plan';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-issue-1314-indented-ralph.md'), [
+            '# PRD',
+            '',
+            `    omx ralph ${JSON.stringify(task)}`,
+            '',
+            `Launch via omx ralph ${JSON.stringify(task)}`,
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-issue-1314-indented-ralph.md'), '# Test Spec\n');
+        const hint = readApprovedExecutionLaunchHint(tempDir, 'ralph');
+        assert.ok(hint);
+        assert.equal(hint?.task, task);
+        assert.equal(hint?.command, `omx ralph ${JSON.stringify(task)}`);
+        assert.equal(hint?.contextPackStatus, 'plan-only');
+    });
+    it('ignores Ralph launch hints that appear only inside nested commented blocks', async () => {
+        const plansDir = join(tempDir, '.omx', 'plans');
+        const task = 'Execute hidden commented ralph plan';
+        await mkdir(plansDir, { recursive: true });
+        await writeFile(join(plansDir, 'prd-hidden-commented-ralph.md'), [
+            '# PRD',
+            '',
+            '<!--',
+            `Launch via omx ralph ${JSON.stringify(task)}`,
+            '<!--',
+            `omx ralph ${JSON.stringify(task)}`,
+            '```md',
+            `Launch via omx ralph ${JSON.stringify(task)}`,
+            '-->',
+            '-->',
+        ].join('\n'));
+        await writeFile(join(plansDir, 'test-spec-hidden-commented-ralph.md'), '# Test Spec\n');
+        const outcome = readApprovedExecutionLaunchHintOutcome(tempDir, 'ralph', { task });
+        assert.equal(outcome.status, 'absent');
+    });
     it('honors the requested team task when a single plan lists multiple team launch hints', async () => {
         const plansDir = join(tempDir, '.omx', 'plans');
         await mkdir(plansDir, { recursive: true });