agentic-workflow-manager 9.14.1 → 9.15.0

package/dist/tests/integration/cursor-subset-gate.test.js

@@ -0,0 +1,123 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+/**
+ * Plan B R3.1 — seven non-empty subsets of {claude-code, codex, cursor}.
+ * Isolated HOME/AWM_HOME; no real ~/.claude or ~/.codex required for {cursor}.
+ */
+const fs_1 = __importDefault(require("fs"));
+const os_1 = __importDefault(require("os"));
+const path_1 = __importDefault(require("path"));
+const TRIO = ['claude-code', 'codex', 'cursor'];
+function subsetsOf(agents) {
+    const out = [];
+    for (let mask = 1; mask < (1 << agents.length); mask++) {
+        out.push(agents.filter((_, i) => (mask & (1 << i)) !== 0));
+    }
+    return out;
+}
+describe('Cursor subset gate (Plan B R3.1)', () => {
+    let tmpHome;
+    let tmpRegistry;
+    let tmpWork;
+    let originalHome;
+    let originalAwmHome;
+    beforeEach(() => {
+        tmpHome = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'awm-subset-home-'));
+        tmpRegistry = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'awm-subset-reg-'));
+        tmpWork = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'awm-subset-work-'));
+        originalHome = process.env.HOME;
+        originalAwmHome = process.env.AWM_HOME;
+        process.env.HOME = tmpHome;
+        process.env.AWM_HOME = path_1.default.join(tmpHome, '.awm');
+        jest.resetModules();
+        const skill = path_1.default.join(tmpRegistry, 'skills/using-awm');
+        fs_1.default.mkdirSync(skill, { recursive: true });
+        fs_1.default.writeFileSync(path_1.default.join(skill, 'SKILL.md'), '---\nname: using-awm\ndescription: guide\n---\nMUST invoke skills.\n');
+        const hooks = path_1.default.join(tmpRegistry, 'hooks');
+        fs_1.default.mkdirSync(hooks, { recursive: true });
+        for (const name of ['cursor-session-start', 'cursor-pre-compact', 'cursor-post-tool-use', 'codex-session-start', 'session-start', 'run-hook.cmd']) {
+            fs_1.default.writeFileSync(path_1.default.join(hooks, name), '#!/usr/bin/env node\nconsole.log("{}")\n', { mode: 0o755 });
+        }
+        fs_1.default.mkdirSync(path_1.default.join(tmpHome, '.awm'), { recursive: true });
+        fs_1.default.writeFileSync(path_1.default.join(tmpHome, '.awm/preferences.json'), JSON.stringify({
+            defaultAgent: 'cursor',
+            enabledAgents: TRIO,
+            installMethod: 'symlink',
+            defaultScope: 'local',
+        }));
+        fs_1.default.writeFileSync(path_1.default.join(tmpHome, '.awm/registries.json'), JSON.stringify([
+            { name: 'baseline', remote: 'https://example.invalid/baseline.git' },
+        ]));
+        // Point baseline content root at our temp registry via junction/symlink of registries/baseline
+        fs_1.default.mkdirSync(path_1.default.join(tmpHome, '.awm/registries'), { recursive: true });
+        fs_1.default.symlinkSync(tmpRegistry, path_1.default.join(tmpHome, '.awm/registries/baseline'), process.platform === 'win32' ? 'junction' : 'dir');
+    });
+    afterEach(() => {
+        fs_1.default.rmSync(tmpHome, { recursive: true, force: true });
+        fs_1.default.rmSync(tmpRegistry, { recursive: true, force: true });
+        fs_1.default.rmSync(tmpWork, { recursive: true, force: true });
+        if (originalHome === undefined)
+            delete process.env.HOME;
+        else
+            process.env.HOME = originalHome;
+        if (originalAwmHome === undefined)
+            delete process.env.AWM_HOME;
+        else
+            process.env.AWM_HOME = originalAwmHome;
+    });
+    it.each(subsetsOf(TRIO).map((s) => [s.join('+'), s]))('subset %s installs a single using-awm skill delivery and Cursor hooks when included', (_label, subset) => {
+        const { planInstall } = require('../../src/core/install-planner');
+        const { applyInstallPlan } = require('../../src/core/install-transaction');
+        const { installHook } = require('../../src/commands/hooks/install');
+        const { providerFor } = require('../../src/providers');
+        const skillSource = path_1.default.join(tmpRegistry, 'skills/using-awm');
+        const plan = planInstall({
+            artifacts: [{
+                    name: 'using-awm',
+                    installName: 'using-awm',
+                    type: 'skill',
+                    sourcePath: skillSource,
+                }],
+            selectedAgents: subset,
+            enabledAgents: subset,
+            scope: 'global',
+            projectRoot: tmpWork,
+            method: 'symlink',
+        });
+        applyInstallPlan(plan);
+        // Shared agents skills root (Cursor/Codex/OpenCode) — at most one using-awm.
+        // Claude may also have ~/.claude/skills/using-awm; that is a separate channel.
+        const sharedRoot = path_1.default.join(tmpHome, '.agents/skills');
+        const sharedSkill = path_1.default.join(sharedRoot, 'using-awm');
+        const wantsShared = subset.some((agent) => {
+            const global = providerFor(agent).skill.global;
+            return global === sharedRoot;
+        });
+        if (wantsShared) {
+            expect(fs_1.default.existsSync(sharedSkill)).toBe(true);
+        }
+        expect(fs_1.default.existsSync(path_1.default.join(tmpHome, '.cursor/rules/using-awm.mdc'))).toBe(false);
+        if (subset.includes('cursor')) {
+            // {cursor} alone must not require ~/.claude or ~/.codex
+            if (subset.length === 1) {
+                expect(fs_1.default.existsSync(path_1.default.join(tmpHome, '.claude'))).toBe(false);
+                expect(fs_1.default.existsSync(path_1.default.join(tmpHome, '.codex'))).toBe(false);
+            }
+            installHook({
+                agent: 'cursor',
+                registryRoot: tmpRegistry,
+                installMethod: 'copy',
+                baselineRegistryVersion: '4.9.0',
+            });
+            expect(fs_1.default.existsSync(path_1.default.join(tmpHome, '.awm/hooks/cursor/session-start'))).toBe(true);
+            const hooksJson = JSON.parse(fs_1.default.readFileSync(path_1.default.join(tmpHome, '.cursor/hooks.json'), 'utf8'));
+            expect(hooksJson.version).toBe(1);
+            expect(hooksJson.hooks.sessionStart.some((e) => e.command.endsWith(`${path_1.default.sep}session-start`))).toBe(true);
+            expect(hooksJson.hooks.preCompact.some((e) => e.command.endsWith(`${path_1.default.sep}pre-compact`))).toBe(true);
+            expect(hooksJson.hooks.postToolUse.some((e) => e.command.endsWith(`${path_1.default.sep}post-tool-use`))).toBe(true);
+        }
+    });
+});

package/dist/tests/providers/index.test.js

@@ -170,10 +170,25 @@ describe('Providers Routing', () => {
             expect((0, providers_1.isAgentTarget)('cursor')).toBe(true);
             expect((0, providers_1.isAgentTarget)('copilot')).toBe(true);
         });
-        it('resolves Cursor skill paths for both scopes', () => {
-            expect((0, providers_1.getTargetPath)('skill', 'cursor', 'local')).toBe('.cursor/rules');
+        it('joins the shared agents skills dirs with link renderer (Plan B R1/R2)', () => {
+            expect((0, providers_1.providerFor)('cursor').skill).toEqual({
+                global: path_1.default.join(process.env.HOME, '.agents/skills'),
+                local: '.agents/skills',
+                renderer: 'link',
+            });
+            expect((0, providers_1.getTargetPath)('skill', 'cursor', 'local')).toBe('.agents/skills');
             expect((0, providers_1.getTargetPath)('skill', 'cursor', 'global'))
-                .toBe(path_1.default.join(process.env.HOME, '.cursor/rules'));
+                .toBe(path_1.default.join(process.env.HOME, '.agents/skills'));
+        });
+        it('installs canonical Cursor agents under .cursor/agents with link (Plan B R6/R9)', () => {
+            expect((0, providers_1.providerFor)('cursor').agent).toEqual({
+                global: path_1.default.join(process.env.HOME, '.cursor/agents'),
+                local: '.cursor/agents',
+                renderer: 'link',
+            });
+            expect((0, providers_1.getTargetPath)('agent', 'cursor', 'local')).toBe('.cursor/agents');
+            expect((0, providers_1.getTargetPath)('agent', 'cursor', 'global'))
+                .toBe(path_1.default.join(process.env.HOME, '.cursor/agents'));
         });
         it('resolves the Copilot local skill path', () => {
             expect((0, providers_1.getTargetPath)('skill', 'copilot', 'local')).toBe('.github/instructions');
@@ -181,31 +196,29 @@ describe('Providers Routing', () => {
         it('throws a specific, non-generic reason when Copilot global skills are requested', () => {
             expect(() => (0, providers_1.getTargetPath)('skill', 'copilot', 'global')).toThrow('skill global scope is not supported by Copilot: GitHub Copilot has no user-level skill discovery mechanism — skills must be installed per-project.');
         });
-        it('keeps workflow/agent unsupported (null) for both, via the existing generic message', () => {
+        it('keeps workflow unsupported for cursor and agent unsupported for copilot', () => {
             expect(() => (0, providers_1.getTargetPath)('workflow', 'cursor', 'local')).toThrow('workflows are not supported by Cursor.');
             expect(() => (0, providers_1.getTargetPath)('agent', 'copilot', 'local')).toThrow('agents are not supported by Copilot.');
         });
-        it('declares no hooks config for cursor or copilot', () => {
-            expect((0, providers_1.providerFor)('cursor').hooks).toBeUndefined();
+        it('declares cursor-hooks-json for cursor and none for copilot', () => {
+            expect((0, providers_1.providerFor)('cursor').hooks).toEqual({
+                type: 'cursor-hooks-json',
+                settingsPath: path_1.default.join(tmpHome, '.cursor/hooks.json'),
+                scriptsDir: path_1.default.join(process.env.AWM_HOME, 'hooks/cursor'),
+                matcher: '',
+                eventName: '',
+            });
             expect((0, providers_1.providerFor)('copilot').hooks).toBeUndefined();
         });
-        it('assigns the Cursor .mdc and Copilot instructions renderers to their skill artifact config (Task 4.3)', () => {
-            expect((0, providers_1.providerFor)('cursor').skill.renderer).toBe('cursor-mdc');
+        it('assigns Copilot instructions renderer; Cursor skills use link (cursor-mdc kept for migration)', () => {
+            expect((0, providers_1.providerFor)('cursor').skill.renderer).toBe('link');
             expect((0, providers_1.providerFor)('copilot').skill.renderer).toBe('copilot-instructions');
         });
-        it('assertLinkRenderer still refuses the Cursor/Copilot skill renderers (Task 4.3 code-quality-review fix)', () => {
-            // Regression: an earlier version of this task widened assertLinkRenderer to
-            // allow these two through, on the theory that a raw unrendered copy is "at
-            // least a plausible degraded install" — wrong. assertLinkRenderer's only
-            // callers (core/provider-artifacts.ts's legacy preflight, src/index.ts's
-            // legacy interactive `awm add`) can only symlink/copy verbatim; they never
-            // render. A raw SKILL.md copy at `.cursor/rules/<name>` or
-            // `.github/instructions/<name>` has no `.mdc`/`.instructions.md` extension
-            // and no frontmatter (`alwaysApply`/`applyTo`) — neither Cursor nor Copilot
-            // would ever read it. This must keep throwing, same as codex-agent-toml
-            // always has, directing users to commands/add.ts's real render pipeline
-            // instead (which never calls assertLinkRenderer at all).
-            expect(() => (0, providers_1.assertLinkRenderer)('skill', 'cursor')).toThrow(/not implemented yet/);
+        it('assertLinkRenderer accepts Cursor link skills and still refuses Copilot', () => {
+            expect(() => (0, providers_1.assertLinkRenderer)('skill', 'cursor')).not.toThrow();
+            expect((0, providers_1.assertLinkRenderer)('skill', 'cursor')?.renderer).toBe('link');
+            // Copilot still renders via the add pipeline only — a raw SKILL.md under
+            // `.github/instructions/` lacks `.instructions.md` + applyTo frontmatter.
             expect(() => (0, providers_1.assertLinkRenderer)('skill', 'copilot')).toThrow(/not implemented yet/);
         });
     });

package/dist/tests/structural/provider-paths-honor-config-home.test.js

@@ -86,11 +86,12 @@ describe('a provider that declares a config-home override honors it everywhere',
         });
     }
     it('shared, cross-agent conventions are NOT moved by one agent’s override', () => {
-        // `~/.agents/skills` lo comparten Codex y OpenCode: es del ecosistema, no de un
-        // agente. Moverlo con CODEX_HOME desconectaria a OpenCode de sus propias skills.
+        // `~/.agents/skills` lo comparten Codex, OpenCode y Cursor: es del ecosistema, no de un
+        // agente. Moverlo con CODEX_HOME desconectaria a OpenCode/Cursor de sus propias skills.
         const moved = providersWith({ HOME, CODEX_HOME: ELSEWHERE });
         expect(moved.codex.skill.global).toBe(path_1.default.join(HOME, '.agents/skills'));
         expect(moved.opencode.skill.global).toBe(path_1.default.join(HOME, '.agents/skills'));
+        expect(moved.cursor.skill.global).toBe(path_1.default.join(HOME, '.agents/skills'));
     });
     it('every agent declares a config home, so a new provider cannot skip the question', () => {
         const table = providersWith({ HOME });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-workflow-manager",
-  "version": "9.14.1",
+  "version": "9.15.0",
   "main": "dist/src/index.js",
   "bin": {
     "awm": "./dist/src/index.js"