claudecode-omc 4.8.4 → 4.9.4

package/dist/__tests__/setup-claude-md-script.test.js

@@ -0,0 +1,292 @@
+import { describe, it, expect, afterEach } from 'vitest';
+import { spawnSync } from 'node:child_process';
+import { copyFileSync, existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync, } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+const REPO_ROOT = join(__dirname, '..', '..');
+const SETUP_SCRIPT = join(REPO_ROOT, 'scripts', 'setup-claude-md.sh');
+const tempRoots = [];
+function createPluginFixture(claudeMdContent) {
+    const root = mkdtempSync(join(tmpdir(), 'omc-setup-claude-md-'));
+    tempRoots.push(root);
+    const pluginRoot = join(root, 'plugin');
+    const projectRoot = join(root, 'project');
+    const homeRoot = join(root, 'home');
+    mkdirSync(join(pluginRoot, 'scripts'), { recursive: true });
+    mkdirSync(join(pluginRoot, 'docs'), { recursive: true });
+    mkdirSync(join(pluginRoot, 'skills', 'omc-reference'), { recursive: true });
+    mkdirSync(projectRoot, { recursive: true });
+    mkdirSync(homeRoot, { recursive: true });
+    copyFileSync(SETUP_SCRIPT, join(pluginRoot, 'scripts', 'setup-claude-md.sh'));
+    writeFileSync(join(pluginRoot, 'docs', 'CLAUDE.md'), claudeMdContent);
+    writeFileSync(join(pluginRoot, 'skills', 'omc-reference', 'SKILL.md'), `---
+name: omc-reference
+description: Test fixture reference skill
+user-invocable: false
+---
+
+# Test OMC Reference
+`);
+    return {
+        pluginRoot,
+        projectRoot,
+        homeRoot,
+        scriptPath: join(pluginRoot, 'scripts', 'setup-claude-md.sh'),
+    };
+}
+afterEach(() => {
+    while (tempRoots.length > 0) {
+        const root = tempRoots.pop();
+        if (root) {
+            rmSync(root, { recursive: true, force: true });
+        }
+    }
+});
+describe('setup-claude-md.sh (issue #1572)', () => {
+    it('installs the canonical docs/CLAUDE.md content with OMC markers', () => {
+        const fixture = createPluginFixture(`<!-- OMC:START -->
+<!-- OMC:VERSION:9.9.9 -->
+
+# Canonical CLAUDE
+Use the real docs file.
+<!-- OMC:END -->
+`);
+        const result = spawnSync('bash', [fixture.scriptPath, 'local'], {
+            cwd: fixture.projectRoot,
+            env: {
+                ...process.env,
+                HOME: fixture.homeRoot,
+            },
+            encoding: 'utf-8',
+        });
+        expect(result.status).toBe(0);
+        const installedPath = join(fixture.projectRoot, '.claude', 'CLAUDE.md');
+        expect(existsSync(installedPath)).toBe(true);
+        const installed = readFileSync(installedPath, 'utf-8');
+        expect(installed).toContain('<!-- OMC:START -->');
+        expect(installed).toContain('<!-- OMC:END -->');
+        expect(installed).toContain('<!-- OMC:VERSION:9.9.9 -->');
+        expect(installed).toContain('# Canonical CLAUDE');
+        const installedSkillPath = join(fixture.projectRoot, '.claude', 'skills', 'omc-reference', 'SKILL.md');
+        expect(existsSync(installedSkillPath)).toBe(true);
+        expect(readFileSync(installedSkillPath, 'utf-8')).toContain('# Test OMC Reference');
+    });
+    it('refuses to install a canonical source that lacks OMC markers', () => {
+        const fixture = createPluginFixture(`# oh-my-claudecode (OMC) v9.9.9 Summary
+
+This is a summarized CLAUDE.md without markers.
+`);
+        const result = spawnSync('bash', [fixture.scriptPath, 'local'], {
+            cwd: fixture.projectRoot,
+            env: {
+                ...process.env,
+                HOME: fixture.homeRoot,
+            },
+            encoding: 'utf-8',
+        });
+        expect(result.status).not.toBe(0);
+        expect(`${result.stdout}\n${result.stderr}`).toContain('missing required OMC markers');
+        expect(existsSync(join(fixture.projectRoot, '.claude', 'CLAUDE.md'))).toBe(false);
+    });
+    it('adds a local git exclude block for .omc artifacts while preserving .omc/skills', () => {
+        const fixture = createPluginFixture(`<!-- OMC:START -->
+<!-- OMC:VERSION:9.9.9 -->
+
+# Canonical CLAUDE
+Use the real docs file.
+<!-- OMC:END -->
+`);
+        const gitInit = spawnSync('git', ['init'], {
+            cwd: fixture.projectRoot,
+            env: {
+                ...process.env,
+                HOME: fixture.homeRoot,
+            },
+            encoding: 'utf-8',
+        });
+        expect(gitInit.status).toBe(0);
+        const result = spawnSync('bash', [fixture.scriptPath, 'local'], {
+            cwd: fixture.projectRoot,
+            env: {
+                ...process.env,
+                HOME: fixture.homeRoot,
+            },
+            encoding: 'utf-8',
+        });
+        expect(result.status).toBe(0);
+        const excludePath = join(fixture.projectRoot, '.git', 'info', 'exclude');
+        expect(existsSync(excludePath)).toBe(true);
+        const excludeContents = readFileSync(excludePath, 'utf-8');
+        expect(excludeContents).toContain('# BEGIN OMC local artifacts');
+        expect(excludeContents).toContain('.omc/*');
+        expect(excludeContents).toContain('!.omc/skills/');
+        expect(excludeContents).toContain('!.omc/skills/**');
+        expect(excludeContents).toContain('# END OMC local artifacts');
+    });
+    it('does not duplicate the local git exclude block on repeated local setup runs', () => {
+        const fixture = createPluginFixture(`<!-- OMC:START -->
+<!-- OMC:VERSION:9.9.9 -->
+
+# Canonical CLAUDE
+Use the real docs file.
+<!-- OMC:END -->
+`);
+        const gitInit = spawnSync('git', ['init'], {
+            cwd: fixture.projectRoot,
+            env: {
+                ...process.env,
+                HOME: fixture.homeRoot,
+            },
+            encoding: 'utf-8',
+        });
+        expect(gitInit.status).toBe(0);
+        const firstRun = spawnSync('bash', [fixture.scriptPath, 'local'], {
+            cwd: fixture.projectRoot,
+            env: {
+                ...process.env,
+                HOME: fixture.homeRoot,
+            },
+            encoding: 'utf-8',
+        });
+        expect(firstRun.status).toBe(0);
+        const secondRun = spawnSync('bash', [fixture.scriptPath, 'local'], {
+            cwd: fixture.projectRoot,
+            env: {
+                ...process.env,
+                HOME: fixture.homeRoot,
+            },
+            encoding: 'utf-8',
+        });
+        expect(secondRun.status).toBe(0);
+        const excludeContents = readFileSync(join(fixture.projectRoot, '.git', 'info', 'exclude'), 'utf-8');
+        expect(excludeContents.match(/# BEGIN OMC local artifacts/g)).toHaveLength(1);
+    });
+});
+describe('setup-claude-md.sh stale CLAUDE_PLUGIN_ROOT resolution', () => {
+    it('uses docs/CLAUDE.md from the active version in installed_plugins.json, not the stale script location', () => {
+        // Simulate: script lives at old version (4.8.2), but installed_plugins.json points to new version (4.9.0)
+        const root = mkdtempSync(join(tmpdir(), 'omc-stale-root-'));
+        tempRoots.push(root);
+        const cacheBase = join(root, '.claude', 'plugins', 'cache', 'omc', 'oh-my-claudecode');
+        const oldVersion = join(cacheBase, '4.8.2');
+        const newVersion = join(cacheBase, '4.9.0');
+        const projectRoot = join(root, 'project');
+        const homeRoot = join(root, 'home');
+        // Create old version (where the script will be copied)
+        mkdirSync(join(oldVersion, 'scripts'), { recursive: true });
+        mkdirSync(join(oldVersion, 'docs'), { recursive: true });
+        copyFileSync(SETUP_SCRIPT, join(oldVersion, 'scripts', 'setup-claude-md.sh'));
+        writeFileSync(join(oldVersion, 'docs', 'CLAUDE.md'), `<!-- OMC:START -->\n<!-- OMC:VERSION:4.8.2 -->\n\n# Old Version\n<!-- OMC:END -->\n`);
+        // Create new version (the active one)
+        mkdirSync(join(newVersion, 'docs'), { recursive: true });
+        writeFileSync(join(newVersion, 'docs', 'CLAUDE.md'), `<!-- OMC:START -->\n<!-- OMC:VERSION:4.9.0 -->\n\n# New Version\n<!-- OMC:END -->\n`);
+        // Create installed_plugins.json pointing to the new version
+        mkdirSync(join(homeRoot, '.claude', 'plugins'), { recursive: true });
+        writeFileSync(join(homeRoot, '.claude', 'plugins', 'installed_plugins.json'), JSON.stringify({
+            'oh-my-claudecode@omc': [
+                {
+                    installPath: newVersion,
+                    version: '4.9.0',
+                },
+            ],
+        }));
+        // Create project dir and settings.json (needed for plugin verification)
+        mkdirSync(projectRoot, { recursive: true });
+        mkdirSync(join(homeRoot, '.claude'), { recursive: true });
+        writeFileSync(join(homeRoot, '.claude', 'settings.json'), JSON.stringify({ plugins: ['oh-my-claudecode'] }));
+        // Run the OLD version's script — it should resolve to the NEW version's docs/CLAUDE.md
+        const result = spawnSync('bash', [join(oldVersion, 'scripts', 'setup-claude-md.sh'), 'local'], {
+            cwd: projectRoot,
+            env: {
+                ...process.env,
+                HOME: homeRoot,
+                CLAUDE_CONFIG_DIR: join(homeRoot, '.claude'),
+            },
+            encoding: 'utf-8',
+        });
+        expect(result.status).toBe(0);
+        const installed = readFileSync(join(projectRoot, '.claude', 'CLAUDE.md'), 'utf-8');
+        // Should contain the NEW version, not the old one
+        expect(installed).toContain('<!-- OMC:VERSION:4.9.0 -->');
+        expect(installed).toContain('# New Version');
+        expect(installed).not.toContain('<!-- OMC:VERSION:4.8.2 -->');
+    });
+    it('uses docs/CLAUDE.md from the active version when installed_plugins.json wraps plugins under a plugins key', () => {
+        const root = mkdtempSync(join(tmpdir(), 'omc-stale-wrapped-root-'));
+        tempRoots.push(root);
+        const cacheBase = join(root, '.claude', 'plugins', 'cache', 'omc', 'oh-my-claudecode');
+        const oldVersion = join(cacheBase, '4.8.2');
+        const newVersion = join(cacheBase, '4.9.0');
+        const projectRoot = join(root, 'project');
+        const homeRoot = join(root, 'home');
+        mkdirSync(join(oldVersion, 'scripts'), { recursive: true });
+        mkdirSync(join(oldVersion, 'docs'), { recursive: true });
+        copyFileSync(SETUP_SCRIPT, join(oldVersion, 'scripts', 'setup-claude-md.sh'));
+        writeFileSync(join(oldVersion, 'docs', 'CLAUDE.md'), `<!-- OMC:START -->\n<!-- OMC:VERSION:4.8.2 -->\n\n# Old Version\n<!-- OMC:END -->\n`);
+        mkdirSync(join(newVersion, 'docs'), { recursive: true });
+        writeFileSync(join(newVersion, 'docs', 'CLAUDE.md'), `<!-- OMC:START -->\n<!-- OMC:VERSION:4.9.0 -->\n\n# New Version\n<!-- OMC:END -->\n`);
+        mkdirSync(join(homeRoot, '.claude', 'plugins'), { recursive: true });
+        writeFileSync(join(homeRoot, '.claude', 'plugins', 'installed_plugins.json'), JSON.stringify({
+            plugins: {
+                'oh-my-claudecode@omc': [
+                    {
+                        installPath: newVersion,
+                        version: '4.9.0',
+                    },
+                ],
+            },
+        }));
+        mkdirSync(projectRoot, { recursive: true });
+        mkdirSync(join(homeRoot, '.claude'), { recursive: true });
+        writeFileSync(join(homeRoot, '.claude', 'settings.json'), JSON.stringify({ plugins: ['oh-my-claudecode'] }));
+        const result = spawnSync('bash', [join(oldVersion, 'scripts', 'setup-claude-md.sh'), 'local'], {
+            cwd: projectRoot,
+            env: {
+                ...process.env,
+                HOME: homeRoot,
+                CLAUDE_CONFIG_DIR: join(homeRoot, '.claude'),
+            },
+            encoding: 'utf-8',
+        });
+        expect(result.status).toBe(0);
+        const installed = readFileSync(join(projectRoot, '.claude', 'CLAUDE.md'), 'utf-8');
+        expect(installed).toContain('<!-- OMC:VERSION:4.9.0 -->');
+        expect(installed).toContain('# New Version');
+        expect(installed).not.toContain('<!-- OMC:VERSION:4.8.2 -->');
+    });
+    it('falls back to scanning cache for latest version when installed_plugins.json is unavailable', () => {
+        const root = mkdtempSync(join(tmpdir(), 'omc-stale-fallback-'));
+        tempRoots.push(root);
+        const cacheBase = join(root, '.claude', 'plugins', 'cache', 'omc', 'oh-my-claudecode');
+        const oldVersion = join(cacheBase, '4.8.2');
+        const newVersion = join(cacheBase, '4.9.0');
+        const projectRoot = join(root, 'project');
+        const homeRoot = join(root, 'home');
+        // Create old version (where the script lives)
+        mkdirSync(join(oldVersion, 'scripts'), { recursive: true });
+        mkdirSync(join(oldVersion, 'docs'), { recursive: true });
+        copyFileSync(SETUP_SCRIPT, join(oldVersion, 'scripts', 'setup-claude-md.sh'));
+        writeFileSync(join(oldVersion, 'docs', 'CLAUDE.md'), `<!-- OMC:START -->\n<!-- OMC:VERSION:4.8.2 -->\n\n# Old\n<!-- OMC:END -->\n`);
+        // Create new version (no installed_plugins.json, relies on cache scan)
+        mkdirSync(join(newVersion, 'docs'), { recursive: true });
+        writeFileSync(join(newVersion, 'docs', 'CLAUDE.md'), `<!-- OMC:START -->\n<!-- OMC:VERSION:4.9.0 -->\n\n# New\n<!-- OMC:END -->\n`);
+        // No installed_plugins.json — fallback to cache scan
+        mkdirSync(join(homeRoot, '.claude'), { recursive: true });
+        mkdirSync(projectRoot, { recursive: true });
+        writeFileSync(join(homeRoot, '.claude', 'settings.json'), JSON.stringify({ plugins: ['oh-my-claudecode'] }));
+        const result = spawnSync('bash', [join(oldVersion, 'scripts', 'setup-claude-md.sh'), 'local'], {
+            cwd: projectRoot,
+            env: {
+                ...process.env,
+                HOME: homeRoot,
+                CLAUDE_CONFIG_DIR: join(homeRoot, '.claude'),
+            },
+            encoding: 'utf-8',
+        });
+        expect(result.status).toBe(0);
+        const installed = readFileSync(join(projectRoot, '.claude', 'CLAUDE.md'), 'utf-8');
+        expect(installed).toContain('<!-- OMC:VERSION:4.9.0 -->');
+        expect(installed).not.toContain('<!-- OMC:VERSION:4.8.2 -->');
+    });
+});
+//# sourceMappingURL=setup-claude-md-script.test.js.map

package/dist/__tests__/setup-claude-md-script.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"setup-claude-md-script.test.js","sourceRoot":"","sources":["../../src/__tests__/setup-claude-md-script.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACzD,OAAO,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/C,OAAO,EACL,YAAY,EACZ,UAAU,EACV,SAAS,EACT,WAAW,EACX,YAAY,EACZ,MAAM,EACN,aAAa,GACd,MAAM,SAAS,CAAC;AACjB,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACjC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAEjC,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC;AAC9C,MAAM,YAAY,GAAG,IAAI,CAAC,SAAS,EAAE,SAAS,EAAE,oBAAoB,CAAC,CAAC;AAEtE,MAAM,SAAS,GAAa,EAAE,CAAC;AAE/B,SAAS,mBAAmB,CAAC,eAAuB;IAClD,MAAM,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,sBAAsB,CAAC,CAAC,CAAC;IACjE,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAErB,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC;IACxC,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;IAC1C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;IAEpC,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC5D,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACzD,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,QAAQ,EAAE,eAAe,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC5E,SAAS,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC5C,SAAS,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAEzC,YAAY,CAAC,YAAY,EAAE,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC,CAAC,CAAC;IAC9E,aAAa,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,eAAe,CAAC,CAAC;IACtE,aAAa,CAAC,IAAI,CAAC,UAAU,EAAE,QAAQ,EAAE,eAAe,EAAE,UAAU,CAAC,EAAE;;;;;;;CAOxE,CAAC,CAAC;IAED,OAAO;QACL,UAAU;QACV,WAAW;QACX,QAAQ;QACR,UAAU,EAAE,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC;KAC9D,CAAC;AACJ,CAAC;AAED,SAAS,CAAC,GAAG,EAAE;IACb,OAAO,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC5B,MAAM,IAAI,GAAG,SAAS,CAAC,GAAG,EAAE,CAAC;QAC7B,IAAI,IAAI,EAAE,CAAC;YACT,MAAM,CAAC,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACjD,CAAC;IACH,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,kCAAkC,EAAE,GAAG,EAAE;IAChD,EAAE,CAAC,gEAAgE,EAAE,GAAG,EAAE;QACxE,MAAM,OAAO,GAAG,mBAAmB,CAAC;;;;;;CAMvC,CAAC,CAAC;QAEC,MAAM,MAAM,GAAG,SAAS,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,EAAE;YAC9D,GAAG,EAAE,OAAO,CAAC,WAAW;YACxB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,OAAO,CAAC,QAAQ;aACvB;YACD,QAAQ,EAAE,OAAO;SAClB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE9B,MAAM,aAAa,GAAG,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,SAAS,EAAE,WAAW,CAAC,CAAC;QACxE,MAAM,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAE7C,MAAM,SAAS,GAAG,YAAY,CAAC,aAAa,EAAE,OAAO,CAAC,CAAC;QACvD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QAClD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;QAChD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;QAC1D,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QAElD,MAAM,kBAAkB,GAAG,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,SAAS,EAAE,QAAQ,EAAE,eAAe,EAAE,UAAU,CAAC,CAAC;QACvG,MAAM,CAAC,UAAU,CAAC,kBAAkB,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClD,MAAM,CAAC,YAAY,CAAC,kBAAkB,EAAE,OAAO,CAAC,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,MAAM,OAAO,GAAG,mBAAmB,CAAC;;;CAGvC,CAAC,CAAC;QAEC,MAAM,MAAM,GAAG,SAAS,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,EAAE;YAC9D,GAAG,EAAE,OAAO,CAAC,WAAW;YACxB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,OAAO,CAAC,QAAQ;aACvB;YACD,QAAQ,EAAE,OAAO;SAClB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClC,MAAM,CAAC,GAAG,MAAM,CAAC,MAAM,KAAK,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;QACvF,MAAM,CAAC,UAAU,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,SAAS,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACpF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gFAAgF,EAAE,GAAG,EAAE;QACxF,MAAM,OAAO,GAAG,mBAAmB,CAAC;;;;;;CAMvC,CAAC,CAAC;QAEC,MAAM,OAAO,GAAG,SAAS,CAAC,KAAK,EAAE,CAAC,MAAM,CAAC,EAAE;YACzC,GAAG,EAAE,OAAO,CAAC,WAAW;YACxB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,OAAO,CAAC,QAAQ;aACvB;YACD,QAAQ,EAAE,OAAO;SAClB,CAAC,CAAC;QACH,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE/B,MAAM,MAAM,GAAG,SAAS,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,EAAE;YAC9D,GAAG,EAAE,OAAO,CAAC,WAAW;YACxB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,OAAO,CAAC,QAAQ;aACvB;YACD,QAAQ,EAAE,OAAO;SAClB,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE9B,MAAM,WAAW,GAAG,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,CAAC,CAAC;QACzE,MAAM,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAE3C,MAAM,eAAe,GAAG,YAAY,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;QAC3D,MAAM,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,6BAA6B,CAAC,CAAC;QACjE,MAAM,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;QAC5C,MAAM,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QACnD,MAAM,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;QACrD,MAAM,CAAC,eAAe,CAAC,CAAC,SAAS,CAAC,2BAA2B,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,GAAG,EAAE;QACrF,MAAM,OAAO,GAAG,mBAAmB,CAAC;;;;;;CAMvC,CAAC,CAAC;QAEC,MAAM,OAAO,GAAG,SAAS,CAAC,KAAK,EAAE,CAAC,MAAM,CAAC,EAAE;YACzC,GAAG,EAAE,OAAO,CAAC,WAAW;YACxB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,OAAO,CAAC,QAAQ;aACvB;YACD,QAAQ,EAAE,OAAO;SAClB,CAAC,CAAC;QACH,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE/B,MAAM,QAAQ,GAAG,SAAS,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,EAAE;YAChE,GAAG,EAAE,OAAO,CAAC,WAAW;YACxB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,OAAO,CAAC,QAAQ;aACvB;YACD,QAAQ,EAAE,OAAO;SAClB,CAAC,CAAC;QACH,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAEhC,MAAM,SAAS,GAAG,SAAS,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC,EAAE;YACjE,GAAG,EAAE,OAAO,CAAC,WAAW;YACxB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,OAAO,CAAC,QAAQ;aACvB;YACD,QAAQ,EAAE,OAAO;SAClB,CAAC,CAAC;QACH,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,YAAY,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,CAAC,EAAE,OAAO,CAAC,CAAC;QACpG,MAAM,CAAC,eAAe,CAAC,KAAK,CAAC,8BAA8B,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAChF,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,wDAAwD,EAAE,GAAG,EAAE;IACtE,EAAE,CAAC,sGAAsG,EAAE,GAAG,EAAE;QAC9G,0GAA0G;QAC1G,MAAM,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,iBAAiB,CAAC,CAAC,CAAC;QAC5D,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAErB,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,SAAS,EAAE,OAAO,EAAE,KAAK,EAAE,kBAAkB,CAAC,CAAC;QACvF,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAC1C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QAEpC,uDAAuD;QACvD,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5D,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,YAAY,CAAC,YAAY,EAAE,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC,CAAC,CAAC;QAC9E,aAAa,CACX,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,CAAC,EACrC,qFAAqF,CACtF,CAAC;QAEF,sCAAsC;QACtC,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,aAAa,CACX,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,CAAC,EACrC,qFAAqF,CACtF,CAAC;QAEF,4DAA4D;QAC5D,SAAS,CAAC,IAAI,CAAC,QAAQ,EAAE,SAAS,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACrE,aAAa,CACX,IAAI,CAAC,QAAQ,EAAE,SAAS,EAAE,SAAS,EAAE,wBAAwB,CAAC,EAC9D,IAAI,CAAC,SAAS,CAAC;YACb,sBAAsB,EAAE;gBACtB;oBACE,WAAW,EAAE,UAAU;oBACvB,OAAO,EAAE,OAAO;iBACjB;aACF;SACF,CAAC,CACH,CAAC;QAEF,wEAAwE;QACxE,SAAS,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5C,SAAS,CAAC,IAAI,CAAC,QAAQ,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC1D,aAAa,CACX,IAAI,CAAC,QAAQ,EAAE,SAAS,EAAE,eAAe,CAAC,EAC1C,IAAI,CAAC,SAAS,CAAC,EAAE,OAAO,EAAE,CAAC,kBAAkB,CAAC,EAAE,CAAC,CAClD,CAAC;QAEF,uFAAuF;QACvF,MAAM,MAAM,GAAG,SAAS,CACtB,MAAM,EACN,CAAC,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC,EAAE,OAAO,CAAC,EAC5D;YACE,GAAG,EAAE,WAAW;YAChB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,QAAQ;gBACd,iBAAiB,EAAE,IAAI,CAAC,QAAQ,EAAE,SAAS,CAAC;aAC7C;YACD,QAAQ,EAAE,OAAO;SAClB,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE9B,MAAM,SAAS,GAAG,YAAY,CAAC,IAAI,CAAC,WAAW,EAAE,SAAS,EAAE,WAAW,CAAC,EAAE,OAAO,CAAC,CAAC;QACnF,kDAAkD;QAClD,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;QAC1D,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC7C,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;IAChE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2GAA2G,EAAE,GAAG,EAAE;QACnH,MAAM,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,yBAAyB,CAAC,CAAC,CAAC;QACpE,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAErB,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,SAAS,EAAE,OAAO,EAAE,KAAK,EAAE,kBAAkB,CAAC,CAAC;QACvF,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAC1C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QAEpC,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5D,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,YAAY,CAAC,YAAY,EAAE,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC,CAAC,CAAC;QAC9E,aAAa,CACX,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,CAAC,EACrC,qFAAqF,CACtF,CAAC;QAEF,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,aAAa,CACX,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,CAAC,EACrC,qFAAqF,CACtF,CAAC;QAEF,SAAS,CAAC,IAAI,CAAC,QAAQ,EAAE,SAAS,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACrE,aAAa,CACX,IAAI,CAAC,QAAQ,EAAE,SAAS,EAAE,SAAS,EAAE,wBAAwB,CAAC,EAC9D,IAAI,CAAC,SAAS,CAAC;YACb,OAAO,EAAE;gBACP,sBAAsB,EAAE;oBACtB;wBACE,WAAW,EAAE,UAAU;wBACvB,OAAO,EAAE,OAAO;qBACjB;iBACF;aACF;SACF,CAAC,CACH,CAAC;QAEF,SAAS,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5C,SAAS,CAAC,IAAI,CAAC,QAAQ,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC1D,aAAa,CACX,IAAI,CAAC,QAAQ,EAAE,SAAS,EAAE,eAAe,CAAC,EAC1C,IAAI,CAAC,SAAS,CAAC,EAAE,OAAO,EAAE,CAAC,kBAAkB,CAAC,EAAE,CAAC,CAClD,CAAC;QAEF,MAAM,MAAM,GAAG,SAAS,CACtB,MAAM,EACN,CAAC,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC,EAAE,OAAO,CAAC,EAC5D;YACE,GAAG,EAAE,WAAW;YAChB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,QAAQ;gBACd,iBAAiB,EAAE,IAAI,CAAC,QAAQ,EAAE,SAAS,CAAC;aAC7C;YACD,QAAQ,EAAE,OAAO;SAClB,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE9B,MAAM,SAAS,GAAG,YAAY,CAAC,IAAI,CAAC,WAAW,EAAE,SAAS,EAAE,WAAW,CAAC,EAAE,OAAO,CAAC,CAAC;QACnF,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;QAC1D,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC7C,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;IAChE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4FAA4F,EAAE,GAAG,EAAE;QACpG,MAAM,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,qBAAqB,CAAC,CAAC,CAAC;QAChE,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAErB,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,EAAE,SAAS,EAAE,OAAO,EAAE,KAAK,EAAE,kBAAkB,CAAC,CAAC;QACvF,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;QAC5C,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QAC1C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QAEpC,8CAA8C;QAC9C,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5D,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,YAAY,CAAC,YAAY,EAAE,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC,CAAC,CAAC;QAC9E,aAAa,CACX,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,CAAC,EACrC,6EAA6E,CAC9E,CAAC;QAEF,uEAAuE;QACvE,SAAS,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,aAAa,CACX,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,CAAC,EACrC,6EAA6E,CAC9E,CAAC;QAEF,qDAAqD;QACrD,SAAS,CAAC,IAAI,CAAC,QAAQ,EAAE,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC1D,SAAS,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5C,aAAa,CACX,IAAI,CAAC,QAAQ,EAAE,SAAS,EAAE,eAAe,CAAC,EAC1C,IAAI,CAAC,SAAS,CAAC,EAAE,OAAO,EAAE,CAAC,kBAAkB,CAAC,EAAE,CAAC,CAClD,CAAC;QAEF,MAAM,MAAM,GAAG,SAAS,CACtB,MAAM,EACN,CAAC,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,oBAAoB,CAAC,EAAE,OAAO,CAAC,EAC5D;YACE,GAAG,EAAE,WAAW;YAChB,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,IAAI,EAAE,QAAQ;gBACd,iBAAiB,EAAE,IAAI,CAAC,QAAQ,EAAE,SAAS,CAAC;aAC7C;YACD,QAAQ,EAAE,OAAO;SAClB,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE9B,MAAM,SAAS,GAAG,YAAY,CAAC,IAAI,CAAC,WAAW,EAAE,SAAS,EAAE,WAAW,CAAC,EAAE,OAAO,CAAC,CAAC;QACnF,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;QAC1D,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;IAChE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/shared-state-locking.test.d.ts

@@ -0,0 +1,10 @@
+/**
+ * Regression tests for race condition bug fixes.
+ *
+ * BUG 1: shared-state updateSharedTask has no file locking
+ * BUG 2: git-worktree removeWorkerWorktree has unlocked metadata update
+ * BUG 3: team-ops teamCreateTask has race on task ID generation
+ * BUG 4: generateJobId not collision-safe
+ */
+export {};
+//# sourceMappingURL=shared-state-locking.test.d.ts.map

package/dist/__tests__/shared-state-locking.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"shared-state-locking.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/shared-state-locking.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG"}

package/dist/__tests__/shared-state-locking.test.js

@@ -0,0 +1,61 @@
+/**
+ * Regression tests for race condition bug fixes.
+ *
+ * BUG 1: shared-state updateSharedTask has no file locking
+ * BUG 2: git-worktree removeWorkerWorktree has unlocked metadata update
+ * BUG 3: team-ops teamCreateTask has race on task ID generation
+ * BUG 4: generateJobId not collision-safe
+ */
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { mkdtempSync, rmSync, readFileSync, existsSync } from 'fs';
+import { join } from 'path';
+import { tmpdir } from 'os';
+// ---------------------------------------------------------------------------
+// ---------------------------------------------------------------------------
+describe('shared-state updateSharedTask locking', () => {
+    let tempDir;
+    beforeEach(() => {
+        tempDir = mkdtempSync(join(tmpdir(), 'shared-state-lock-test-'));
+    });
+    afterEach(() => {
+        rmSync(tempDir, { recursive: true, force: true });
+    });
+    it('updateSharedTask uses withFileLockSync for read-modify-write', async () => {
+        // Verify the source code contains the locking pattern
+        const sourcePath = join(__dirname, '..', 'interop', 'shared-state.ts');
+        const source = readFileSync(sourcePath, 'utf-8');
+        // Must import withFileLockSync
+        expect(source).toContain("import { withFileLockSync } from '../lib/file-lock.js'");
+        // The updateSharedTask function must use withFileLockSync
+        const fnMatch = source.match(/export function updateSharedTask[\s\S]*?^}/m);
+        expect(fnMatch).toBeTruthy();
+        const fnBody = fnMatch[0];
+        expect(fnBody).toContain('withFileLockSync');
+        expect(fnBody).toContain("taskPath + '.lock'");
+    });
+    it('updateSharedTask functionally updates a task with locking', async () => {
+        const { addSharedTask, updateSharedTask, initInteropSession } = await import('../interop/shared-state.js');
+        initInteropSession('test-session', tempDir);
+        const task = addSharedTask(tempDir, {
+            source: 'omc',
+            target: 'omx',
+            type: 'analyze',
+            description: 'test task for locking',
+        });
+        const updated = updateSharedTask(tempDir, task.id, {
+            status: 'completed',
+            result: 'done',
+        });
+        expect(updated).not.toBeNull();
+        expect(updated.status).toBe('completed');
+        expect(updated.result).toBe('done');
+        expect(updated.completedAt).toBeTruthy();
+        // Verify lock file does not persist after operation
+        const lockPath = join(tempDir, '.omc', 'state', 'interop', 'tasks', `${task.id}.json.lock`);
+        expect(existsSync(lockPath)).toBe(false);
+    });
+});
+// ---------------------------------------------------------------------------
+// BUG 2: git-worktree removeWorkerWorktree must use file locking
+// ---------------------------------------------------------------------------
+//# sourceMappingURL=shared-state-locking.test.js.map

package/dist/__tests__/shared-state-locking.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"shared-state-locking.test.js","sourceRoot":"","sources":["../../src/__tests__/shared-state-locking.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,EAAE,WAAW,EAAa,MAAM,EAAE,YAAY,EAAiB,UAAU,EAAE,MAAM,IAAI,CAAC;AAC7F,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AAC5B,OAAO,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AAG5B,8EAA8E;AAC9E,8EAA8E;AAE9E,QAAQ,CAAC,uCAAuC,EAAE,GAAG,EAAE;IACrD,IAAI,OAAe,CAAC;IAEpB,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,yBAAyB,CAAC,CAAC,CAAC;IACnE,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,sDAAsD;QACtD,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,SAAS,EAAE,iBAAiB,CAAC,CAAC;QACvE,MAAM,MAAM,GAAG,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;QAEjD,+BAA+B;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,wDAAwD,CAAC,CAAC;QAEnF,0DAA0D;QAC1D,MAAM,OAAO,GAAG,MAAM,CAAC,KAAK,CAAC,6CAA6C,CAAC,CAAC;QAC5E,MAAM,CAAC,OAAO,CAAC,CAAC,UAAU,EAAE,CAAC;QAC7B,MAAM,MAAM,GAAG,OAAQ,CAAC,CAAC,CAAC,CAAC;QAC3B,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;QAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;QACzE,MAAM,EAAE,aAAa,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,GAAG,MAAM,MAAM,CAC1E,4BAA4B,CAC7B,CAAC;QAEF,kBAAkB,CAAC,cAAc,EAAE,OAAO,CAAC,CAAC;QAE5C,MAAM,IAAI,GAAG,aAAa,CAAC,OAAO,EAAE;YAClC,MAAM,EAAE,KAAK;YACb,MAAM,EAAE,KAAK;YACb,IAAI,EAAE,SAAS;YACf,WAAW,EAAE,uBAAuB;SACrC,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,gBAAgB,CAAC,OAAO,EAAE,IAAI,CAAC,EAAE,EAAE;YACjD,MAAM,EAAE,WAAW;YACnB,MAAM,EAAE,MAAM;SACf,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;QAC/B,MAAM,CAAC,OAAQ,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QAC1C,MAAM,CAAC,OAAQ,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACrC,MAAM,CAAC,OAAQ,CAAC,WAAW,CAAC,CAAC,UAAU,EAAE,CAAC;QAE1C,oDAAoD;QACpD,MAAM,QAAQ,GAAG,IAAI,CACnB,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC,EAAE,YAAY,CACrE,CAAC;QACF,MAAM,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,iEAAiE;AACjE,8EAA8E"}

package/dist/__tests__/skills.test.js

@@ -1,15 +1,44 @@
-import { describe, it, expect, beforeEach } from 'vitest';
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
 import { createBuiltinSkills, getBuiltinSkill, listBuiltinSkillNames, clearSkillsCache } from '../features/builtin-skills/skills.js';
 describe('Builtin Skills', () => {
+    const originalPluginRoot = process.env.CLAUDE_PLUGIN_ROOT;
+    const originalPath = process.env.PATH;
     // Clear cache before each test to ensure fresh loads
     beforeEach(() => {
+        if (originalPluginRoot === undefined) {
+            delete process.env.CLAUDE_PLUGIN_ROOT;
+        }
+        else {
+            process.env.CLAUDE_PLUGIN_ROOT = originalPluginRoot;
+        }
+        if (originalPath === undefined) {
+            delete process.env.PATH;
+        }
+        else {
+            process.env.PATH = originalPath;
+        }
+        clearSkillsCache();
+    });
+    afterEach(() => {
+        if (originalPluginRoot === undefined) {
+            delete process.env.CLAUDE_PLUGIN_ROOT;
+        }
+        else {
+            process.env.CLAUDE_PLUGIN_ROOT = originalPluginRoot;
+        }
+        if (originalPath === undefined) {
+            delete process.env.PATH;
+        }
+        else {
+            process.env.PATH = originalPath;
+        }
         clearSkillsCache();
     });
     describe('createBuiltinSkills()', () => {
-        it('should return correct number of skills (45 including aliases)', () => {
+        it('should return correct number of skills (31 canonical + 1 alias)', () => {
             const skills = createBuiltinSkills();
-            // 45 entries: 44 canonical skills + 1 deprecated alias (psm)
-            expect(skills).toHaveLength(45);
+            // 32 entries: 31 canonical skills + 1 deprecated alias (psm)
+            expect(skills).toHaveLength(32);
         });
         it('should return an array of BuiltinSkill objects', () => {
             const skills = createBuiltinSkills();
@@ -58,6 +87,7 @@ describe('Builtin Skills', () => {
                 'cancel',
                 'ccg',
                 'configure-notifications',
+                'deep-dive',
                 'deep-interview',
                 'deepinit',
                 'omc-doctor',
@@ -68,6 +98,7 @@ describe('Builtin Skills', () => {
                 'omc-setup',
                 'omc-teams',
                 'omc-plan',
+                'omc-reference',
                 'project-session-manager',
                 'psm',
                 'ralph',
@@ -77,13 +108,15 @@ describe('Builtin Skills', () => {
                 'setup',
                 'skill',
                 'team',
+                'trace',
                 'ultraqa',
                 'ultrawork',
+                'visual-verdict',
                 'writer-memory',
             ];
             const actualSkillNames = skills.map((s) => s.name);
             expect(actualSkillNames).toEqual(expect.arrayContaining(expectedSkills));
-            expect(actualSkillNames.length).toBeGreaterThanOrEqual(expectedSkills.length);
+            expect(actualSkillNames.length).toBe(expectedSkills.length);
         });
         it('should not have duplicate skill names', () => {
             const skills = createBuiltinSkills();
@@ -103,6 +136,76 @@ describe('Builtin Skills', () => {
             expect(skill).toBeDefined();
             expect(skill?.name).toBe('ai-slop-cleaner');
         });
+        it('should surface bundled skill resources for skills with additional files', () => {
+            const skill = getBuiltinSkill('project-session-manager');
+            expect(skill).toBeDefined();
+            expect(skill?.template).toContain('## Skill Resources');
+            expect(skill?.template).toContain('skills/project-session-manager');
+            expect(skill?.template).toContain('`lib/`');
+            expect(skill?.template).toContain('`psm.sh`');
+        });
+        it('should emphasize process-first install routing in the setup skill', () => {
+            const skill = getBuiltinSkill('setup');
+            expect(skill).toBeDefined();
+            expect(skill?.description).toContain('install/update routing');
+            expect(skill?.template).toContain('Process the request by the **first argument only**');
+            expect(skill?.template).toContain('/oh-my-claudecode:setup doctor --json');
+            expect(skill?.template).not.toContain('{{ARGUMENTS_AFTER_DOCTOR}}');
+        });
+        it('should emphasize worktree-first guidance in project session manager skill text', () => {
+            const skill = getBuiltinSkill('project-session-manager');
+            expect(skill).toBeDefined();
+            expect(skill?.description).toContain('Worktree-first');
+            expect(skill?.template).toContain('Quick Start (worktree-first)');
+            expect(skill?.template).toContain('`omc teleport`');
+        });
+        it('should keep ask as the canonical process-first advisor wrapper', () => {
+            const skill = getBuiltinSkill('ask');
+            expect(skill).toBeDefined();
+            expect(skill?.description).toContain('Process-first advisor routing');
+            expect(skill?.template).toContain('omc ask {{ARGUMENTS}}');
+            expect(skill?.template).toContain('Do NOT manually construct raw provider CLI commands');
+        });
+        it('should retrieve the trace skill by name', () => {
+            const skill = getBuiltinSkill('trace');
+            expect(skill).toBeDefined();
+            expect(skill?.name).toBe('trace');
+            expect(skill?.template).toContain('Claude built-in team mode');
+            expect(skill?.template).toContain('3 tracer lanes by default');
+            expect(skill?.template).toContain('Ranked Hypotheses');
+            expect(skill?.template).toContain('trace_timeline');
+            expect(skill?.template).toContain('trace_summary');
+        });
+        it('should retrieve the deep-dive skill with pipeline metadata and 3-point injection', () => {
+            const skill = getBuiltinSkill('deep-dive');
+            expect(skill).toBeDefined();
+            expect(skill?.name).toBe('deep-dive');
+            expect(skill?.pipeline).toEqual({
+                steps: ['deep-dive', 'omc-plan', 'autopilot'],
+                nextSkill: 'omc-plan',
+                nextSkillArgs: '--consensus --direct',
+                handoff: '.omc/specs/deep-dive-{slug}.md',
+            });
+            // Verify 3-point injection mechanism
+            expect(skill?.template).toContain('3-Point Injection');
+            expect(skill?.template).toContain('initial_idea enrichment');
+            expect(skill?.template).toContain('codebase_context replacement');
+            expect(skill?.template).toContain('initial question queue injection');
+            // Verify per-lane critical unknowns (B3 fix)
+            expect(skill?.template).toContain('Per-Lane Critical Unknowns');
+            // Verify pipeline handoff is fully wired (B1 fix)
+            expect(skill?.template).toContain('Skill("oh-my-claudecode:autopilot")');
+            expect(skill?.template).toContain('consensus plan as Phase 0+1 output');
+            // Verify untrusted data guard (NB1 fix)
+            expect(skill?.template).toContain('trace-context');
+            expect(skill?.template).toContain('untrusted data');
+            // Verify state schema compatibility (B2 fix)
+            expect(skill?.template).toContain('interview_id');
+            expect(skill?.template).toContain('challenge_modes_used');
+            expect(skill?.template).toContain('ontology_snapshots');
+            expect(skill?.template).toContain('explicit weakest-dimension rationale reporting');
+            expect(skill?.template).toContain('repo-evidence citation requirement');
+        });
         it('should expose pipeline metadata for deep-interview handoff into omc-plan', () => {
             const skill = getBuiltinSkill('deep-interview');
             expect(skill?.pipeline).toEqual({
@@ -116,6 +219,26 @@ describe('Builtin Skills', () => {
             expect(skill?.template).toContain('Skill("oh-my-claudecode:omc-plan")');
             expect(skill?.template).toContain('`--consensus --direct`');
             expect(skill?.template).toContain('`.omc/specs/deep-interview-{slug}.md`');
+            expect(skill?.template).toContain('Why now: {one_sentence_targeting_rationale}');
+            expect(skill?.template).toContain('cite the repo evidence');
+            expect(skill?.template).toContain('Ontology-style question for scope-fuzzy tasks');
+            expect(skill?.template).toContain('Every round explicitly names the weakest dimension and why it is the next target');
+            expect(skill?.argumentHint).toContain('--autoresearch');
+            expect(skill?.template).toContain('zero-learning-curve setup lane for `omc autoresearch`');
+            expect(skill?.template).toContain('autoresearch --mission "<mission>" --eval "<evaluator>"');
+        });
+        it('rewrites built-in skill command examples to plugin-safe bridge invocations when omc is unavailable', () => {
+            process.env.CLAUDE_PLUGIN_ROOT = '/plugin-root';
+            process.env.PATH = '';
+            clearSkillsCache();
+            const deepInterviewSkill = getBuiltinSkill('deep-interview');
+            const askSkill = getBuiltinSkill('ask');
+            expect(deepInterviewSkill?.template)
+                .toContain('zero-learning-curve setup lane for `node "$CLAUDE_PLUGIN_ROOT"/bridge/cli.cjs autoresearch`');
+            expect(deepInterviewSkill?.template)
+                .toContain('node "$CLAUDE_PLUGIN_ROOT"/bridge/cli.cjs autoresearch --mission "<mission>" --eval "<evaluator>"');
+            expect(askSkill?.template)
+                .toContain('node "$CLAUDE_PLUGIN_ROOT"/bridge/cli.cjs ask {{ARGUMENTS}}');
         });
         it('should expose pipeline metadata for omc-plan handoff into autopilot', () => {
             const skill = getBuiltinSkill('omc-plan');
@@ -174,32 +297,25 @@ describe('Builtin Skills', () => {
     describe('listBuiltinSkillNames()', () => {
         it('should return canonical skill names by default', () => {
             const names = listBuiltinSkillNames();
-            expect(names).toHaveLength(44);
+            expect(names).toHaveLength(31);
             expect(names).toContain('ai-slop-cleaner');
             expect(names).toContain('ask');
             expect(names).toContain('autopilot');
             expect(names).toContain('cancel');
             expect(names).toContain('ccg');
             expect(names).toContain('configure-notifications');
-            expect(names).toContain('analyze');
-            expect(names).toContain('multi-model-research');
-            expect(names).toContain('planning-with-files');
-            expect(names).toContain('bdd-generator');
-            expect(names).toContain('tdd-generator');
-            expect(names).toContain('e2e');
-            expect(names).toContain('electron-driver');
-            expect(names).toContain('test-coverage');
-            expect(names).toContain('start-dev');
-            expect(names).toContain('test-gen');
             expect(names).toContain('ralph');
             expect(names).toContain('ultrawork');
             expect(names).toContain('omc-plan');
+            expect(names).toContain('omc-reference');
             expect(names).toContain('deepinit');
             expect(names).toContain('release');
             expect(names).toContain('omc-doctor');
             expect(names).toContain('hud');
             expect(names).toContain('omc-setup');
             expect(names).toContain('setup');
+            expect(names).toContain('trace');
+            expect(names).toContain('visual-verdict');
             expect(names).not.toContain('swarm'); // removed in #1131
             expect(names).not.toContain('psm');
         });
@@ -212,8 +328,10 @@ describe('Builtin Skills', () => {
         it('should include aliases when explicitly requested', () => {
             const names = listBuiltinSkillNames({ includeAliases: true });
             // swarm alias removed in #1131, psm still exists
-            expect(names).toHaveLength(45);
+            expect(names).toHaveLength(32);
             expect(names).toContain('ai-slop-cleaner');
+            expect(names).toContain('trace');
+            expect(names).toContain('visual-verdict');
             expect(names).not.toContain('swarm');
             expect(names).toContain('psm');
         });