claude-dev-env 8.26.5 → 8.26.6

package/bin/install.cursor-rules.test.mjs

@@ -239,6 +239,66 @@ test('reinstall moves the old routing hook and keeps user hooks', () => {
     }
 });
 
+test('Codex reinstall removes omitted package hooks and preserves user configuration', () => {
+    const homeDirectory = mkdtempSync(join(tmpdir(), 'cdev-codex-migration-'));
+    try {
+        const codexHome = join(homeDirectory, 'custom-codex');
+        const codexHooksPath = join(codexHome, 'hooks.json');
+        const configPath = join(codexHome, 'config.toml');
+        const disabledConfig = '[features]\nhooks = false\n';
+        const allLegacyRegistrations = [
+            ['PreToolUse', 'Edit', '.claude', 'blocking/pre_tool_use_dispatcher.py'],
+            ['PreToolUse', 'Bash', '.agents', 'blocking/bash_pre_tool_use_dispatcher.py'],
+            ['SessionStart', '*', 'custom-codex', 'session/issue_tracker_session_starter.py'],
+            ['Stop', '*', '.claude', 'session/skill_loaded_reminder.py'],
+            ['SessionEnd', '*', 'custom-codex', 'lifecycle/session_end_cleanup.py'],
+        ];
+        const hooksByEvent = {};
+        for (const [event, matcher, root, script] of allLegacyRegistrations) {
+            hooksByEvent[event] ??= [];
+            hooksByEvent[event].push({
+                matcher,
+                metadata: 'keep-group',
+                hooks: [
+                    { type: 'command', command: `python "${join(homeDirectory, root, 'hooks', script)}"` },
+                    { type: 'command', command: `user-${event}-${matcher}`, timeout: 7 },
+                ],
+            });
+        }
+        const foreignCommand = `python "${join(homeDirectory, 'foreign', '.claude', 'hooks', 'blocking', 'pre_tool_use_dispatcher.py')}"`;
+        hooksByEvent.PreToolUse.push({ matcher: 'foreign', hooks: [{ type: 'command', command: foreignCommand }] });
+        const suffixCommand = `python "${join(codexHome, 'hooks', 'blocking', 'pre_tool_use_dispatcher.py.backup')}"`;
+        hooksByEvent.PreToolUse.push({ matcher: 'suffix', hooks: [{ type: 'command', command: suffixCommand }] });
+        hooksByEvent.CustomEvent = { metadata: 'keep-event' };
+        const expectedUserHooks = JSON.parse(JSON.stringify(hooksByEvent));
+        for (const eachGroup of Object.values(expectedUserHooks).flat()) {
+            if (eachGroup.hooks && eachGroup.metadata) eachGroup.hooks.shift();
+        }
+        hooksByEvent.LegacyValidator = [{ hooks: [{ type: 'command', command: 'python -c "sys.path.insert(0, __claude_dev_env_managed_hooks__); from validators.run_all_validators import main; main()"' }] }];
+        mkdirSync(codexHome, { recursive: true });
+        writeFileSync(configPath, disabledConfig);
+        writeFileSync(codexHooksPath, JSON.stringify({ enabled: false, metadata: 'keep-file', hooks: hooksByEvent }));
+
+        runInstaller(homeDirectory, ['--only', 'core'], { CODEX_HOME: codexHome });
+
+        const migratedHooks = JSON.parse(readFileSync(codexHooksPath, 'utf8'));
+        const allSpawnGroups = migratedHooks.hooks.PreToolUse.filter(eachGroup =>
+            ['Agent|Task', 'multi_agent_v1__spawn_agent'].includes(eachGroup.matcher));
+        assert.equal(allSpawnGroups.length, 2);
+        migratedHooks.hooks.PreToolUse = migratedHooks.hooks.PreToolUse.filter(eachGroup => !allSpawnGroups.includes(eachGroup));
+        assert.deepEqual(migratedHooks, { enabled: false, metadata: 'keep-file', hooks: expectedUserHooks });
+        assert.equal(readFileSync(configPath, 'utf8'), disabledConfig);
+        const firstInstall = readFileSync(codexHooksPath, 'utf8');
+        runInstaller(homeDirectory, ['--only', 'core'], { CODEX_HOME: codexHome });
+        assert.equal(readFileSync(codexHooksPath, 'utf8'), firstInstall);
+        const claudeSettings = JSON.parse(readFileSync(join(homeDirectory, '.claude', 'settings.json'), 'utf8'));
+        assert.ok(claudeSettings.hooks.SessionStart.length > 0);
+        assert.ok(claudeSettings.hooks.PreToolUse.some(eachGroup => eachGroup.matcher === 'Bash'));
+    } finally {
+        rmSync(homeDirectory, { recursive: true, force: true });
+    }
+});
+
 test('uninstall removes the routing hook and keeps a Codex user hook', () => {
     const homeDirectory = mkdtempSync(join(tmpdir(), 'cdev-routing-uninstall-'));
     try {

package/bin/install.mjs

@@ -2038,6 +2038,7 @@ function mergeHooksAtPath(
     pythonCommand,
     selectHooksConfig = hooksConfig => hooksConfig,
     pluginRootDir = CLAUDE_HOME,
+    shouldPruneSourceHooks = false,
 ) {
     const hooksSource = resolvePackageManagedDirectory(
         hooksSourceRoot,
@@ -2045,8 +2046,17 @@ function mergeHooksAtPath(
     );
     const hooksJsonPath = join(hooksSource, 'hooks.json');
     if (!existsSync(hooksJsonPath)) return 0;
-    const hooksConfig = selectHooksConfig(JSON.parse(readFileSync(hooksJsonPath, 'utf8')));
+    const sourceHooksConfig = JSON.parse(readFileSync(hooksJsonPath, 'utf8'));
     const settings = loadClaudeSettingsObject(settingsPath);
+    if (shouldPruneSourceHooks && settings.hooks && typeof settings.hooks === 'object') {
+        stripRetiredHookEntries(
+            settings,
+            managedHookScriptRelativePaths(sourceHooksConfig),
+            retiredHookOwnership(),
+        );
+        pruneManagedHooksFromSettings(settings, new Set());
+    }
+    const hooksConfig = selectHooksConfig(sourceHooksConfig);
     const groupCount = mergeHooksIntoSettings(settings, hooksConfig, pluginRootDir, pythonCommand);
     writeFileSync(settingsPath, JSON.stringify(settings, null, indentation) + '\n');
     return groupCount;
@@ -2081,6 +2091,7 @@ function mergeCodexHooks(hooksSourceRoot, pythonCommand) {
         pythonCommand,
         selectCodexNativeHooksConfig,
         dirname(CODEX_HOOKS_CONFIGURATION_PATH),
+        true,
     );
 }
 

package/bin/install.prune.test.mjs

@@ -16,7 +16,7 @@ import { fileURLToPath } from 'node:url';
 import { detectPython } from './install.mjs';
 const THIS_DIRECTORY = dirname(fileURLToPath(import.meta.url));
 const INSTALLER_PATH = join(THIS_DIRECTORY, 'install.mjs');
-const INSTALLER_PROCESS_TIMEOUT_MS = 60_000;
+const INSTALLER_PROCESS_TIMEOUT_MS = 120_000;
 const EXCLUDED_PACKAGE_COPY_DIRECTORY = 'node_modules';
 
 const RETIRED_SKILL_DIRECTORIES = [

package/bin/install.transaction.test.mjs

@@ -373,7 +373,13 @@ test('installer fault after_settings_write restores prior settings and files', (
         const { claudeDirectory, rulesFile } = seedPriorInstall(homeDirectory);
         const priorSettings = readFileSync(join(claudeDirectory, 'settings.json'), 'utf8');
         const codexHooksPath = join(homeDirectory, '.codex', 'hooks.json');
-        writeFileWithParents(codexHooksPath, '{"hooks":{"PreToolUse":[{"matcher":"custom"}]}}\n');
+        writeFileWithParents(codexHooksPath, JSON.stringify({
+            enabled: false,
+            hooks: {
+                PreToolUse: [{ matcher: 'custom' }],
+                SessionEnd: [{ hooks: [{ type: 'command', command: `python "${join(homeDirectory, '.codex', 'hooks', 'lifecycle', 'session_end_cleanup.py')}"` }] }],
+            },
+        }) + '\n');
         const priorCodexHooks = readFileSync(codexHooksPath, 'utf8');
 
         const failedRun = runInstaller(homeDirectory, [], {

package/docs/codex-compatibility.md

@@ -26,8 +26,17 @@ Run `codex-compat bridge --surface <name> --payload '<json-object>'`. The bridge
 
 The package installer seeds `subagent-model-policy.json` in the agents-home
 `rules/` directory when the file is absent. It registers one `PreToolUse` group
-for `multi_agent_v1__spawn_agent` in `$CODEX_HOME/hooks.json`. Existing Codex
-hook groups stay in place. Claude-only hook groups do not enter the Codex file.
+for `multi_agent_v1__spawn_agent` and one for `Agent|Task` in `$CODEX_HOME/hooks.json`.
+Before adding these groups, the installer removes obsolete package-owned hook
+registrations from every event. Custom hook entries and Codex's enabled or
+disabled hook settings stay unchanged. Claude-only hook groups do not enter the
+Codex file.
+
+A detached Windows Codex daemon can open a console for each background command.
+If this happens, run local interactive sessions with `codex --no-daemon` and keep
+`codex exec` commands unchanged. [Codex issue #44768](https://github.com/openai/codex/issues/44768)
+describes the symptom. [PR #49164](https://github.com/openai/codex/pull/49164)
+adds the upstream console fix.
 
 Edit the installed policy file to change the next routing decision. The resolver
 loads that path for each invocation.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-dev-env",
-    "version": "8.26.5",
+    "version": "8.26.6",
     "description": "Claude Code development standards — rules, hooks, agents, commands, and skills",
     "type": "module",
     "bin": {