claude-dev-env 8.26.4 → 8.26.6

package/.agents/skills/{plugin-eval-standalone-skill → build-eval}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: plugin-eval-standalone-skill
+name: build-eval
 description: >-
   Evaluate skills with a bounded direct Codex review suite or `claude plugin eval`.
   Use labeled inputs and deterministic grading for output correctness; use a plugin
@@ -7,7 +7,7 @@ description: >-
   Use when the user asks to eval or test a skill.
 ---
 
-# Plugin eval for a standalone skill
+# Build eval
 
 For building an evaluation of an AI workflow, follow [the design process](reference/build-evaluation.md).
 For a direct Codex output-quality evaluation, use [the review suite](evals/review/README.md).
@@ -100,7 +100,7 @@ Otherwise work the steps in order.
   and the `claude -p` recipe for flag-gated features.
 
 ```text
-plugin-eval-standalone-skill/
+build-eval/
 ├── SKILL.md
 └── reference/
     └── graders-and-commands.md

package/.agents/skills/{plugin-eval-standalone-skill → build-eval}/evals/review/README.md

@@ -21,9 +21,9 @@ Report finding precision and recall, clean-case specificity and false-positive r
 From the repository root:
 
 ```powershell
-$suite = 'packages/claude-dev-env/.agents/skills/plugin-eval-standalone-skill/evals/review/run.py'
+$suite = 'packages/claude-dev-env/.agents/skills/build-eval/evals/review/run.py'
 python $suite validate
-python -m pytest packages/claude-dev-env/.agents/skills/plugin-eval-standalone-skill/evals/review/test_run.py -q
+python -m pytest packages/claude-dev-env/.agents/skills/build-eval/evals/review/test_run.py -q
 python $suite live --limit 2 --timeout 90 --model gpt-6.1-sol --effort low --output review-smoke
 python $suite live --split heldout --limit 6 --timeout 90 --model gpt-6.1-sol --effort low --output review-heldout
 ```

package/bin/ever-shipped-skills.mjs

@@ -7,6 +7,7 @@ export const EVER_SHIPPED_SKILL_NAMES = new Set([
     'bdd-protocol',
     'beat-sheet',
     'bg-agent',
+    'build-eval',
     'build-goal-prompt',
     'bugteam',
     'caveman',

package/bin/ever-shipped-skills.test.mjs

@@ -1,6 +1,6 @@
 import { strict as assert } from 'node:assert';
 import { test } from 'node:test';
-import { existsSync, readdirSync } from 'node:fs';
+import { existsSync, readFileSync, readdirSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { EVER_SHIPPED_SKILL_NAMES } from './ever-shipped-skills.mjs';
@@ -41,7 +41,21 @@ test('EVER_SHIPPED_SKILL_NAMES includes the windows scheduled task skill', () =>
     assert.equal(EVER_SHIPPED_SKILL_NAMES.has('windows-scheduled-task'), true);
 });
 
-test('EVER_SHIPPED_SKILL_NAMES includes the plugin eval standalone skill', () => {
+test('build-eval ships under its matching name without the retired active directory', () => {
+    const sourceSkillsDirectory = join(
+        PACKAGE_DIRECTORY,
+        PACKAGE_AGENTS_HOME_DIRECTORY_NAME,
+        MANAGED_SKILLS_DIRECTORY_NAME,
+    );
+    assert.ok(EVER_SHIPPED_SKILL_NAMES.has('build-eval'));
+    assert.match(
+        readFileSync(join(sourceSkillsDirectory, 'build-eval', 'SKILL.md'), 'utf8'),
+        /^---\r?\nname: build-eval\r?\n/,
+    );
+    assert.equal(existsSync(join(sourceSkillsDirectory, 'plugin-eval-standalone-skill')), false);
+});
+
+test('EVER_SHIPPED_SKILL_NAMES retains the retired plugin eval standalone skill', () => {
     assert.equal(EVER_SHIPPED_SKILL_NAMES.has('plugin-eval-standalone-skill'), true);
 });
 

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 = [
@@ -24,6 +24,7 @@ const RETIRED_SKILL_DIRECTORIES = [
     'fixbugs',
     'pr-scope-resolve',
     'post-audit-findings',
+    'plugin-eval-standalone-skill',
     'pr-consistency-audit',
     'bdd-protocol',
     'hitl',
@@ -607,6 +608,28 @@ test('a full reinstall over a pre-manifest dirty tree prunes retired skills and
     }
 });
 
+test('a full install replaces the retired plugin eval name with build-eval', () => {
+    const sandbox = createSandbox();
+    const retiredSkillName = 'plugin-eval-standalone-skill';
+    try {
+        plantSkillDirectory(sandbox.skillsDirectory, retiredSkillName, true);
+
+        runInstaller(sandbox.homeDirectory, []);
+
+        assert.equal(existsSync(join(sandbox.skillsDirectory, retiredSkillName)), false);
+        assert.ok(prunedSkillBackupContains(sandbox.claudeDirectory, retiredSkillName));
+        assert.match(
+            readFileSync(join(sandbox.skillsDirectory, 'build-eval', 'SKILL.md'), 'utf8'),
+            /^---\r?\nname: build-eval\r?\n/,
+        );
+        const manifest = readManifest(sandbox.manifestPath);
+        assert.ok(manifest.skills.includes('build-eval'));
+        assert.equal(manifest.skills.includes(retiredSkillName), false);
+    } finally {
+        rmSync(sandbox.homeDirectory, { recursive: true, force: true });
+    }
+});
+
 test('a full reinstall over an old-format manifest without a skills key still prunes via the ever-shipped fallback', () => {
     const sandbox = createSandbox();
     try {

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.4",
+    "version": "8.26.6",
     "description": "Claude Code development standards — rules, hooks, agents, commands, and skills",
     "type": "module",
     "bin": {