oh-my-codex 0.11.13 → 0.12.0

package/dist/scripts/__tests__/codex-native-hook.test.js

@@ -0,0 +1,720 @@
+import assert from "node:assert/strict";
+import { execFileSync } from "node:child_process";
+import { existsSync } from "node:fs";
+import { mkdir, mkdtemp, readFile, rm, writeFile } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { dirname, join } from "node:path";
+import { describe, it } from "node:test";
+import { buildManagedCodexHooksConfig } from "../../config/codex-hooks.js";
+import { initTeamState } from "../../team/state.js";
+import { dispatchCodexNativeHook, mapCodexHookEventToOmxEvent, resolveSessionOwnerPidFromAncestry, } from "../codex-native-hook.js";
+async function writeJson(path, value) {
+    await mkdir(dirname(path), { recursive: true }).catch(() => { });
+    await writeFile(path, JSON.stringify(value, null, 2));
+}
+describe("codex native hook config", () => {
+    it("builds the expected managed hooks.json shape", () => {
+        const config = buildManagedCodexHooksConfig("/tmp/omx");
+        assert.deepEqual(Object.keys(config.hooks), [
+            "SessionStart",
+            "PreToolUse",
+            "PostToolUse",
+            "UserPromptSubmit",
+            "Stop",
+        ]);
+        const preToolUse = config.hooks.PreToolUse[0];
+        assert.equal(preToolUse.matcher, "Bash");
+        assert.match(String(preToolUse.hooks?.[0]?.command || ""), /codex-native-hook\.js"?$/);
+        const stop = config.hooks.Stop[0];
+        assert.equal(stop.hooks?.[0]?.timeout, 30);
+    });
+});
+describe("codex native hook dispatch", () => {
+    it("maps Codex events onto OMX logical surfaces", () => {
+        assert.equal(mapCodexHookEventToOmxEvent("SessionStart"), "session-start");
+        assert.equal(mapCodexHookEventToOmxEvent("UserPromptSubmit"), "keyword-detector");
+        assert.equal(mapCodexHookEventToOmxEvent("PreToolUse"), "pre-tool-use");
+        assert.equal(mapCodexHookEventToOmxEvent("PostToolUse"), "post-tool-use");
+        assert.equal(mapCodexHookEventToOmxEvent("Stop"), "stop");
+    });
+    it("writes SessionStart state against the long-lived session owner pid", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-session-start-"));
+        try {
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "SessionStart",
+                cwd,
+                session_id: "sess-start-1",
+            }, {
+                cwd,
+                sessionOwnerPid: 43210,
+            });
+            assert.equal(result.omxEventName, "session-start");
+            assert.deepEqual(result.outputJson, {
+                hookSpecificOutput: {
+                    hookEventName: "SessionStart",
+                    additionalContext: "OMX native SessionStart detected. Load workspace conventions from AGENTS.md, restore relevant .omx runtime/project memory context, and continue from existing mode state before making changes.",
+                },
+            });
+            const sessionState = JSON.parse(await readFile(join(cwd, ".omx", "state", "session.json"), "utf-8"));
+            assert.equal(sessionState.session_id, "sess-start-1");
+            assert.equal(sessionState.pid, 43210);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("appends .omx/ to repo-root .gitignore during SessionStart when missing", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-session-gitignore-"));
+        try {
+            await writeFile(join(cwd, ".gitignore"), "node_modules/\n");
+            execFileSync("git", ["init"], { cwd, stdio: "pipe" });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "SessionStart",
+                cwd,
+                session_id: "sess-gitignore-1",
+            }, { cwd, sessionOwnerPid: 43210 });
+            assert.equal(result.omxEventName, "session-start");
+            const gitignore = await readFile(join(cwd, ".gitignore"), "utf-8");
+            assert.match(gitignore, /^node_modules\/\n\.omx\/\n$/);
+            assert.match(JSON.stringify(result.outputJson), /Added \.omx\/ to .*\.gitignore/);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("includes persisted project-memory summary in SessionStart context", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-session-memory-"));
+        try {
+            await writeJson(join(cwd, ".omx", "project-memory.json"), {
+                techStack: "TypeScript + Node.js",
+                build: "npm test",
+                conventions: "small diffs, verify before claim",
+                directives: [
+                    { directive: "Keep native Stop bounded to real continuation decisions.", priority: "high" },
+                ],
+                notes: [
+                    { category: "env", content: "Requires LOCAL_API_BASE for smoke tests", timestamp: new Date().toISOString() },
+                ],
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "SessionStart",
+                cwd,
+                session_id: "sess-memory-1",
+            }, { cwd, sessionOwnerPid: 43210 });
+            const serialized = JSON.stringify(result.outputJson);
+            assert.match(serialized, /\[Project memory\]/);
+            assert.match(serialized, /TypeScript \+ Node\.js/);
+            assert.match(serialized, /small diffs, verify before claim/);
+            assert.match(serialized, /Keep native Stop bounded to real continuation decisions\./);
+            assert.match(serialized, /Requires LOCAL_API_BASE for smoke tests/);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("resolves the Codex owner from ancestry without mistaking codex-native-hook wrappers for Codex", () => {
+        const commands = new Map([
+            [2100, 'sh -c node "/repo/dist/scripts/codex-native-hook.js"'],
+            [1100, 'node /usr/local/bin/codex.js'],
+            [900, 'bash'],
+        ]);
+        const parents = new Map([
+            [2100, 1100],
+            [1100, 900],
+            [900, 1],
+        ]);
+        const resolved = resolveSessionOwnerPidFromAncestry(2100, {
+            readParentPid: (pid) => parents.get(pid) ?? null,
+            readProcessCommand: (pid) => commands.get(pid) ?? "",
+        });
+        assert.equal(resolved, 1100);
+    });
+    it("records keyword activation from UserPromptSubmit payloads", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-"));
+        try {
+            await mkdir(join(cwd, ".omx", "state"), { recursive: true });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "UserPromptSubmit",
+                cwd,
+                session_id: "sess-1",
+                thread_id: "thread-1",
+                turn_id: "turn-1",
+                prompt: "$ralplan implement issue #1307",
+            }, { cwd });
+            assert.equal(result.omxEventName, "keyword-detector");
+            assert.equal(result.skillState?.skill, "ralplan");
+            assert.ok(result.outputJson, "UserPromptSubmit should emit developer context");
+            const statePath = join(cwd, ".omx", "state", "skill-active-state.json");
+            assert.equal(existsSync(statePath), true);
+            const state = JSON.parse(await readFile(statePath, "utf-8"));
+            assert.equal(state.skill, "ralplan");
+            assert.equal(state.active, true);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns a destructive-command caution on PreToolUse for rm -rf dist", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-pretool-danger-"));
+        try {
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "PreToolUse",
+                cwd,
+                tool_name: "Bash",
+                tool_use_id: "tool-danger",
+                tool_input: { command: "rm -rf dist" },
+            }, { cwd });
+            assert.equal(result.omxEventName, "pre-tool-use");
+            assert.deepEqual(result.outputJson, {
+                hookSpecificOutput: {
+                    hookEventName: "PreToolUse",
+                },
+                systemMessage: "Destructive Bash command detected (`rm -rf dist`). Confirm the target and expected side effects before running it.",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("stays silent on PreToolUse for neutral pwd", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-pretool-neutral-"));
+        try {
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "PreToolUse",
+                cwd,
+                tool_name: "Bash",
+                tool_use_id: "tool-neutral",
+                tool_input: { command: "pwd" },
+            }, { cwd });
+            assert.equal(result.omxEventName, "pre-tool-use");
+            assert.equal(result.outputJson, null);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns PostToolUse remediation guidance for command-not-found output", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-posttool-failure-"));
+        try {
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "PostToolUse",
+                cwd,
+                tool_name: "Bash",
+                tool_use_id: "tool-fail",
+                tool_input: { command: "foo --version" },
+                tool_response: "{\"exit_code\":127,\"stdout\":\"\",\"stderr\":\"bash: foo: command not found\"}",
+            }, { cwd });
+            assert.equal(result.omxEventName, "post-tool-use");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "The Bash output indicates a command/setup failure that should be fixed before retrying.",
+                hookSpecificOutput: {
+                    hookEventName: "PostToolUse",
+                    additionalContext: "Bash reported `command not found`, `permission denied`, or a missing file/path. Verify the command, dependency installation, PATH, file permissions, and referenced paths before retrying.",
+                },
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("treats stderr-only informative non-zero output as reviewable instead of a generic failure", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-posttool-informative-stderr-"));
+        try {
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "PostToolUse",
+                cwd,
+                tool_name: "Bash",
+                tool_use_id: "tool-useful-stderr",
+                tool_input: { command: "gh pr checks" },
+                tool_response: "{\"exit_code\":8,\"stdout\":\"\",\"stderr\":\"build pending\\nlint pass\"}",
+            }, { cwd });
+            assert.equal(result.omxEventName, "post-tool-use");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "The Bash command returned a non-zero exit code but produced useful output that should be reviewed before retrying.",
+                hookSpecificOutput: {
+                    hookEventName: "PostToolUse",
+                    additionalContext: "The Bash output appears informative despite the non-zero exit code. Review and report the output before retrying instead of assuming the command simply failed.",
+                },
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("treats non-zero gh pr checks style output as informative instead of a generic failure", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-posttool-informative-"));
+        try {
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "PostToolUse",
+                cwd,
+                tool_name: "Bash",
+                tool_use_id: "tool-useful",
+                tool_input: { command: "gh pr checks" },
+                tool_response: "{\"exit_code\":8,\"stdout\":\"build\\tpending\\t2m\\nlint\\tpass\\t18s\",\"stderr\":\"\"}",
+            }, { cwd });
+            assert.equal(result.omxEventName, "post-tool-use");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "The Bash command returned a non-zero exit code but produced useful output that should be reviewed before retrying.",
+                hookSpecificOutput: {
+                    hookEventName: "PostToolUse",
+                    additionalContext: "The Bash output appears informative despite the non-zero exit code. Review and report the output before retrying instead of assuming the command simply failed.",
+                },
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("stays silent on neutral successful PostToolUse output", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-posttool-neutral-"));
+        try {
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "PostToolUse",
+                cwd,
+                tool_name: "Bash",
+                tool_use_id: "tool-ok",
+                tool_input: { command: "pwd" },
+                tool_response: "{\"exit_code\":0,\"stdout\":\"/repo\",\"stderr\":\"\"}",
+            }, { cwd });
+            assert.equal(result.omxEventName, "post-tool-use");
+            assert.equal(result.outputJson, null);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output while Autopilot is active", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-autopilot-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            await writeJson(join(stateDir, "autopilot-state.json"), {
+                active: true,
+                current_phase: "execution",
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-autopilot",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX autopilot is still active (phase: execution); continue the task and gather fresh verification evidence before stopping.",
+                stopReason: "autopilot_execution",
+                systemMessage: "OMX autopilot is still active (phase: execution).",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output while Ultrawork is active", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-ultrawork-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            await writeJson(join(stateDir, "ultrawork-state.json"), {
+                active: true,
+                current_phase: "executing",
+            });
+            const result = await dispatchCodexNativeHook({ hook_event_name: "Stop", cwd, session_id: "sess-stop-ultrawork" }, { cwd });
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX ultrawork is still active (phase: executing); continue the task and gather fresh verification evidence before stopping.",
+                stopReason: "ultrawork_executing",
+                systemMessage: "OMX ultrawork is still active (phase: executing).",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output while UltraQA is active", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-ultraqa-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            await writeJson(join(stateDir, "ultraqa-state.json"), {
+                active: true,
+                current_phase: "diagnose",
+            });
+            const result = await dispatchCodexNativeHook({ hook_event_name: "Stop", cwd, session_id: "sess-stop-ultraqa" }, { cwd });
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX ultraqa is still active (phase: diagnose); continue the task and gather fresh verification evidence before stopping.",
+                stopReason: "ultraqa_diagnose",
+                systemMessage: "OMX ultraqa is still active (phase: diagnose).",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output while team phase is non-terminal", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-team-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            await writeJson(join(stateDir, "team-state.json"), {
+                active: true,
+                current_phase: "team-exec",
+                team_name: "review-team",
+            });
+            await writeJson(join(stateDir, "team", "review-team", "phase.json"), {
+                current_phase: "team-verify",
+                max_fix_attempts: 3,
+                current_fix_attempt: 0,
+                transitions: [],
+                updated_at: new Date().toISOString(),
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-team",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX team pipeline is still active (review-team) at phase team-verify; continue coordinating until the team reaches a terminal phase.",
+                stopReason: "team_team-verify",
+                systemMessage: "OMX team pipeline is still active at phase team-verify.",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output from canonical team state when coarse mode state is missing", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-team-canonical-"));
+        try {
+            await initTeamState("canonical-team", "canonical stop fallback", "executor", 1, cwd, undefined, { ...process.env, OMX_SESSION_ID: "sess-stop-team-canonical" });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-team-canonical",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX team pipeline is still active (canonical-team) at phase team-exec; continue coordinating until the team reaches a terminal phase.",
+                stopReason: "team_team-exec",
+                systemMessage: "OMX team pipeline is still active at phase team-exec.",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("does not block Stop from canonical team state alone when the canonical phase is terminal", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-team-terminal-"));
+        try {
+            await initTeamState("terminal-team", "terminal stop fallback", "executor", 1, cwd, undefined, { ...process.env, OMX_SESSION_ID: "sess-stop-team-terminal" });
+            await writeJson(join(cwd, ".omx", "state", "team", "terminal-team", "phase.json"), {
+                current_phase: "complete",
+                max_fix_attempts: 3,
+                current_fix_attempt: 0,
+                transitions: [],
+                updated_at: new Date().toISOString(),
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-team-terminal",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.equal(result.outputJson, null);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output from canonical team state when manifest session ownership is missing", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-team-legacy-"));
+        try {
+            await initTeamState("legacy-team", "legacy stop fallback", "executor", 1, cwd, undefined, { ...process.env, OMX_SESSION_ID: "sess-stop-team-legacy" });
+            const manifestPath = join(cwd, ".omx", "state", "team", "legacy-team", "manifest.v2.json");
+            const manifest = JSON.parse(await readFile(manifestPath, "utf-8"));
+            await writeJson(manifestPath, {
+                ...manifest,
+                leader: {
+                    ...manifest.leader,
+                    session_id: "",
+                },
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-team-legacy",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX team pipeline is still active (legacy-team) at phase team-exec; continue coordinating until the team reaches a terminal phase.",
+                stopReason: "team_team-exec",
+                systemMessage: "OMX team pipeline is still active at phase team-exec.",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("reads canonical Stop fallback team state from OMX_TEAM_STATE_ROOT when configured", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-team-root-"));
+        const sharedRoot = join(cwd, "shared-root");
+        const priorTeamStateRoot = process.env.OMX_TEAM_STATE_ROOT;
+        try {
+            process.env.OMX_TEAM_STATE_ROOT = "shared-root";
+            await initTeamState("canonical-root-team", "canonical stop root fallback", "executor", 1, cwd, undefined, { ...process.env, OMX_SESSION_ID: "sess-stop-team-root", OMX_TEAM_STATE_ROOT: "shared-root" });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-team-root",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX team pipeline is still active (canonical-root-team) at phase team-exec; continue coordinating until the team reaches a terminal phase.",
+                stopReason: "team_team-exec",
+                systemMessage: "OMX team pipeline is still active at phase team-exec.",
+            });
+            assert.equal(existsSync(join(sharedRoot, "team", "canonical-root-team", "phase.json")), true);
+        }
+        finally {
+            if (typeof priorTeamStateRoot === "string")
+                process.env.OMX_TEAM_STATE_ROOT = priorTeamStateRoot;
+            else
+                delete process.env.OMX_TEAM_STATE_ROOT;
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output from canonical team state rooted via OMX_TEAM_STATE_ROOT", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-team-env-root-"));
+        const previousTeamStateRoot = process.env.OMX_TEAM_STATE_ROOT;
+        try {
+            process.env.OMX_TEAM_STATE_ROOT = "shared-team-state";
+            await initTeamState("env-root-team", "env root stop fallback", "executor", 1, cwd, undefined, {
+                ...process.env,
+                OMX_SESSION_ID: "sess-stop-team-env-root",
+                OMX_TEAM_STATE_ROOT: "shared-team-state",
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-team-env-root",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX team pipeline is still active (env-root-team) at phase team-exec; continue coordinating until the team reaches a terminal phase.",
+                stopReason: "team_team-exec",
+                systemMessage: "OMX team pipeline is still active at phase team-exec.",
+            });
+        }
+        finally {
+            if (typeof previousTeamStateRoot === "string")
+                process.env.OMX_TEAM_STATE_ROOT = previousTeamStateRoot;
+            else
+                delete process.env.OMX_TEAM_STATE_ROOT;
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output for active ralplan skill without active subagents", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-skill-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(join(stateDir, "sessions", "sess-stop-skill"), { recursive: true });
+            await writeJson(join(stateDir, "session.json"), { session_id: "sess-stop-skill" });
+            await writeJson(join(stateDir, "sessions", "sess-stop-skill", "skill-active-state.json"), {
+                active: true,
+                skill: "ralplan",
+                phase: "planning",
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-skill",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX skill ralplan is still active (phase: planning); continue until the current ralplan workflow reaches a terminal state.",
+                stopReason: "skill_ralplan_planning",
+                systemMessage: "OMX skill ralplan is still active (phase: planning).",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("does not block on active ralplan skill when subagents are still active", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-skill-subagent-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(join(stateDir, "sessions", "sess-stop-skill-subagent"), { recursive: true });
+            await writeJson(join(stateDir, "session.json"), { session_id: "sess-stop-skill-subagent" });
+            await writeJson(join(stateDir, "sessions", "sess-stop-skill-subagent", "skill-active-state.json"), {
+                active: true,
+                skill: "ralplan",
+                phase: "planning",
+            });
+            await writeJson(join(stateDir, "subagent-tracking.json"), {
+                schemaVersion: 1,
+                sessions: {
+                    "sess-stop-skill-subagent": {
+                        session_id: "sess-stop-skill-subagent",
+                        leader_thread_id: "leader-1",
+                        updated_at: new Date().toISOString(),
+                        threads: {
+                            "leader-1": {
+                                thread_id: "leader-1",
+                                kind: "leader",
+                                first_seen_at: new Date().toISOString(),
+                                last_seen_at: new Date().toISOString(),
+                                turn_count: 1,
+                            },
+                            "sub-1": {
+                                thread_id: "sub-1",
+                                kind: "subagent",
+                                first_seen_at: new Date().toISOString(),
+                                last_seen_at: new Date().toISOString(),
+                                turn_count: 1,
+                            },
+                        },
+                    },
+                },
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-skill-subagent",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.equal(result.outputJson, null);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output for active deep-interview skill without active subagents", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-deep-interview-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(join(stateDir, "sessions", "sess-stop-deep-interview"), { recursive: true });
+            await writeJson(join(stateDir, "session.json"), { session_id: "sess-stop-deep-interview" });
+            await writeJson(join(stateDir, "sessions", "sess-stop-deep-interview", "skill-active-state.json"), {
+                active: true,
+                skill: "deep-interview",
+                phase: "planning",
+            });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-deep-interview",
+            }, { cwd });
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX skill deep-interview is still active (phase: planning); continue until the current deep-interview workflow reaches a terminal state.",
+                stopReason: "skill_deep-interview_planning",
+                systemMessage: "OMX skill deep-interview is still active (phase: planning).",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output while Ralph is active", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            await writeFile(join(stateDir, "ralph-state.json"), JSON.stringify({
+                active: true,
+                current_phase: "executing",
+            }));
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "OMX Ralph is still active (phase: executing); continue the task and gather fresh verification evidence before stopping.",
+                stopReason: "ralph_executing",
+                systemMessage: "OMX Ralph is still active (phase: executing); continue the task and gather fresh verification evidence before stopping.",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("does not re-block Ralph when Stop already continued once", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-stop-ralph-once-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            await writeFile(join(stateDir, "ralph-state.json"), JSON.stringify({
+                active: true,
+                current_phase: "executing",
+            }));
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-ralph-once",
+                stop_hook_active: true,
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.equal(result.outputJson, null);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("returns Stop continuation output for native auto-nudge stall prompts", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-auto-nudge-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-auto",
+                last_assistant_message: "Would you like me to keep going and finish the cleanup?",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.deepEqual(result.outputJson, {
+                decision: "block",
+                reason: "yes, proceed",
+                stopReason: "auto_nudge",
+                systemMessage: "OMX native Stop detected a stall/permission-style handoff and continued the turn automatically.",
+            });
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+    it("does not auto-nudge again when Stop already continued once", async () => {
+        const cwd = await mkdtemp(join(tmpdir(), "omx-native-hook-auto-nudge-once-"));
+        try {
+            const stateDir = join(cwd, ".omx", "state");
+            await mkdir(stateDir, { recursive: true });
+            const result = await dispatchCodexNativeHook({
+                hook_event_name: "Stop",
+                cwd,
+                session_id: "sess-stop-auto-once",
+                stop_hook_active: true,
+                last_assistant_message: "Would you like me to continue?",
+            }, { cwd });
+            assert.equal(result.omxEventName, "stop");
+            assert.equal(result.outputJson, null);
+        }
+        finally {
+            await rm(cwd, { recursive: true, force: true });
+        }
+    });
+});
+//# sourceMappingURL=codex-native-hook.test.js.map