oh-my-codex 0.15.2 → 0.16.0

package/src/scripts/__tests__/hook-derived-watcher.test.ts

@@ -2,7 +2,7 @@ import assert from 'node:assert/strict';
 import { spawn, spawnSync } from 'node:child_process';
 import { once } from 'node:events';
 import { existsSync } from 'node:fs';
-import { appendFile, mkdir, mkdtemp, readFile, rename, rm, writeFile } from 'node:fs/promises';
+import { appendFile, mkdir, mkdtemp, readFile, readdir, rename, rm, writeFile } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { describe, it } from 'node:test';
@@ -47,6 +47,50 @@ describe('hook-derived-watcher', () => {
     assert.doesNotMatch(source, /stat\(path\)\.catch\(\(\) => \(\{ size: 0 \}\)\)/);
   });
 
+  it('stores watcher state and logs under boxed runtime root when OMX_ROOT is set', async () => {
+    const base = await mkdtemp(join(tmpdir(), 'omx-hook-derived-boxed-'));
+    const homeDir = join(base, 'home');
+    const cwd = join(base, 'cwd');
+    const boxedRoot = join(base, 'boxed-runtime');
+
+    try {
+      await mkdir(todaySessionDir(homeDir), { recursive: true });
+      await mkdir(cwd, { recursive: true });
+      const watcherScript = new URL('../hook-derived-watcher.js', import.meta.url).pathname;
+      const result = spawnSync(
+        process.execPath,
+        [watcherScript, '--once', '--cwd', cwd, '--poll-ms', '250'],
+        {
+          cwd,
+          env: {
+            ...process.env,
+            HOME: homeDir,
+            OMX_ROOT: boxedRoot,
+            OMXBOX_ACTIVE: '1',
+            OMX_SOURCE_CWD: cwd,
+            OMX_HOOK_DERIVED_SIGNALS: '1',
+          },
+          encoding: 'utf8',
+        },
+      );
+
+      assert.equal(result.status, 0, result.stderr || result.stdout);
+      assert.equal(
+        existsSync(join(boxedRoot, '.omx', 'state', 'hook-derived-watcher-state.json')),
+        true,
+      );
+      assert.equal(
+        existsSync(join(cwd, '.omx', 'state', 'hook-derived-watcher-state.json')),
+        false,
+      );
+      const logDir = join(boxedRoot, '.omx', 'logs');
+      const logNames = await readdir(logDir);
+      assert.equal(logNames.some((name) => name.startsWith('hook-derived-watcher-')), true);
+    } finally {
+      await rm(base, { recursive: true, force: true });
+    }
+  });
+
   it('dispatches needs-input for assistant_message content arrays', async () => {
     const base = await mkdtemp(join(tmpdir(), 'omx-hook-derived-array-'));
     const homeDir = join(base, 'home');

package/src/scripts/__tests__/run-test-files.test.ts

@@ -69,4 +69,50 @@ describe('run-test-files diagnostics', () => {
       rmSync(wd, { recursive: true, force: true });
     }
   });
+
+  it('serializes test files by default in CI to avoid cross-file child-process leaks', () => {
+    const wd = mkdtempSync(join(tmpdir(), 'omx-run-test-files-'));
+    try {
+      const testsDir = join(wd, '__tests__');
+      mkdirSync(testsDir, { recursive: true });
+      writeFileSync(
+        join(testsDir, 'pass.test.js'),
+        [
+          "import { test } from 'node:test';",
+          "test('passes', () => {});",
+          '',
+        ].join('\n'),
+      );
+
+      const result = runCompiledRunner(wd, { CI: 'true' });
+
+      assert.equal(result.status, 0, result.stderr || result.stdout);
+      assert.match(result.stderr, /test concurrency 1/);
+    } finally {
+      rmSync(wd, { recursive: true, force: true });
+    }
+  });
+
+  it('honors an explicit test concurrency override in CI', () => {
+    const wd = mkdtempSync(join(tmpdir(), 'omx-run-test-files-'));
+    try {
+      const testsDir = join(wd, '__tests__');
+      mkdirSync(testsDir, { recursive: true });
+      writeFileSync(
+        join(testsDir, 'pass.test.js'),
+        [
+          "import { test } from 'node:test';",
+          "test('passes', () => {});",
+          '',
+        ].join('\n'),
+      );
+
+      const result = runCompiledRunner(wd, { CI: 'true', OMX_NODE_TEST_CONCURRENCY: '2' });
+
+      assert.equal(result.status, 0, result.stderr || result.stdout);
+      assert.match(result.stderr, /test concurrency 2/);
+    } finally {
+      rmSync(wd, { recursive: true, force: true });
+    }
+  });
 });