agentic-workflow-manager 9.14.1 → 9.15.1

package/dist/tests/core/install-planner.test.js

@@ -83,6 +83,10 @@ describe('install-planner', () => {
             const group = (0, install_planner_1.agentsSharingSkillTarget)('opencode', ['opencode', 'codex', 'copilot'], 'global', tmpWork);
             expect(group.sort()).toEqual(['codex', 'opencode']);
         });
+        it('includes cursor in the shared ~/.agents/skills group with codex and opencode (Plan B R7)', () => {
+            const group = (0, install_planner_1.agentsSharingSkillTarget)('cursor', ['cursor', 'codex', 'opencode', 'claude-code'], 'global', tmpWork);
+            expect(group.sort()).toEqual(['codex', 'cursor', 'opencode']);
+        });
     });
     describe('planInstall', () => {
         it('deduplicates the OpenCode/Codex physical skill write and reports both owners', () => {
@@ -220,7 +224,7 @@ describe('install-planner', () => {
             })).toThrow(/physical target already claimed by a different source/);
         });
         describe('renderer-driven filename computation (Task 4.3)', () => {
-            it('renders the Cursor skill target with a .mdc extension, stripping any pre-existing extension', () => {
+            it('installs Cursor skills via link under .agents/skills (shared group, Plan B R1/R2)', () => {
                 const plan = (0, install_planner_1.planInstall)({
                     artifacts: [skillArtifact('development-process')],
                     selectedAgents: ['cursor'],
@@ -229,8 +233,8 @@ describe('install-planner', () => {
                     projectRoot: tmpWork,
                     method: 'symlink',
                 });
-                expect(plan.operations[0].targetPath).toBe(path_1.default.join(tmpWork, '.cursor', 'rules', 'development-process.mdc'));
-                expect(plan.operations[0].renderer).toBe('cursor-mdc');
+                expect(plan.operations[0].targetPath).toBe(path_1.default.join(tmpWork, '.agents', 'skills', 'development-process'));
+                expect(plan.operations[0].renderer).toBe('link');
             });
             it('renders the Copilot skill target with a .instructions.md extension (not .md.instructions.md)', () => {
                 const plan = (0, install_planner_1.planInstall)({
@@ -261,13 +265,13 @@ describe('install-planner', () => {
                 });
                 expect(path_1.default.basename(plan.operations[0].targetPath)).toBe('development-process.instructions.md');
             });
-            it('does not truncate an installName with an embedded, non-extension dot (e.g. "v1.2-migration")', () => {
+            it('does not truncate a Cursor link installName with an embedded, non-extension dot', () => {
                 // Regression: physicalTarget used to derive the base name via
                 // path.parse(...).name, which strips everything after the LAST
                 // dot — not just a genuine trailing .md extension. A skill
                 // literally named `v1.2-migration` would silently truncate to
                 // `v1`, dropping `2-migration` and risking a collision with any
-                // other skill named `v1`.
+                // other skill named `v1`. Cursor now uses link under .agents/skills.
                 const dottedName = {
                     name: 'v1.2-migration',
                     installName: 'v1.2-migration',
@@ -282,10 +286,10 @@ describe('install-planner', () => {
                     projectRoot: tmpWork,
                     method: 'symlink',
                 });
-                expect(path_1.default.basename(plan.operations[0].targetPath)).toBe('v1.2-migration.mdc');
-                expect(path_1.default.basename(plan.operations[0].targetPath)).not.toBe('v1.mdc');
+                expect(path_1.default.basename(plan.operations[0].targetPath)).toBe('v1.2-migration');
+                expect(path_1.default.basename(plan.operations[0].targetPath)).not.toBe('v1');
             });
-            it('still strips a real trailing .md extension for Cursor (not v1.2-migration.md.mdc)', () => {
+            it('still strips a real trailing .md extension for Copilot (not using-awm.md.instructions.md)', () => {
                 const withMdExtension = {
                     name: 'using-awm',
                     installName: 'using-awm.md',
@@ -294,14 +298,14 @@ describe('install-planner', () => {
                 };
                 const plan = (0, install_planner_1.planInstall)({
                     artifacts: [withMdExtension],
-                    selectedAgents: ['cursor'],
-                    enabledAgents: ['cursor'],
+                    selectedAgents: ['copilot'],
+                    enabledAgents: ['copilot'],
                     scope: 'local',
                     projectRoot: tmpWork,
                     method: 'symlink',
                 });
-                expect(path_1.default.basename(plan.operations[0].targetPath)).toBe('using-awm.mdc');
-                expect(path_1.default.basename(plan.operations[0].targetPath)).not.toBe('using-awm.md.mdc');
+                expect(path_1.default.basename(plan.operations[0].targetPath)).toBe('using-awm.instructions.md');
+                expect(path_1.default.basename(plan.operations[0].targetPath)).not.toBe('using-awm.md.instructions.md');
             });
         });
     });

package/dist/tests/core/journal/durable-custody.test.js

@@ -0,0 +1,44 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const commander_1 = require("commander");
+const durable_custody_1 = require("../../../src/core/journal/durable-custody");
+const watch_1 = require("../../../src/commands/watch");
+describe('durable custody suspension', () => {
+    const original = process.env.AWM_ALLOW_DURABLE_CUSTODY;
+    afterEach(() => {
+        if (original === undefined)
+            delete process.env.AWM_ALLOW_DURABLE_CUSTODY;
+        else
+            process.env.AWM_ALLOW_DURABLE_CUSTODY = original;
+    });
+    it('is suspended for operators (env unset) and only re-enabled by the Jest harness', () => {
+        delete process.env.AWM_ALLOW_DURABLE_CUSTODY;
+        expect((0, durable_custody_1.isDurableCustodySuspended)()).toBe(true);
+        process.env.AWM_ALLOW_DURABLE_CUSTODY = '0';
+        expect((0, durable_custody_1.isDurableCustodySuspended)()).toBe(true);
+        process.env.AWM_ALLOW_DURABLE_CUSTODY = '1';
+        expect((0, durable_custody_1.isDurableCustodySuspended)()).toBe(false);
+    });
+    it('refuses every awm watch entry point while suspended', async () => {
+        delete process.env.AWM_ALLOW_DURABLE_CUSTODY;
+        const program = new commander_1.Command();
+        (0, watch_1.registerWatchCommand)(program);
+        const err = [];
+        const exit = jest.spyOn(process, 'exit').mockImplementation(((code) => {
+            throw Object.assign(new Error(`exit:${code ?? 0}`), { code });
+        }));
+        program.configureOutput({ writeErr: (s) => { err.push(String(s)); } });
+        // stderr write in the refusal helper bypasses configureOutput; capture it too.
+        const stderr = jest.spyOn(process.stderr, 'write').mockImplementation(((chunk) => {
+            err.push(String(chunk));
+            return true;
+        }));
+        await expect(program.parseAsync(['node', 'awm', 'watch', 'journal-status'])).rejects.toThrow(/exit:1/);
+        expect(err.join('')).toContain(durable_custody_1.DURABLE_CUSTODY_SUSPENDED_MESSAGE);
+        err.length = 0;
+        await expect(program.parseAsync(['node', 'awm', 'watch', '--init', '--plan', 'docs/plan.md'])).rejects.toThrow(/exit:1/);
+        expect(err.join('')).toContain(durable_custody_1.DURABLE_CUSTODY_SUSPENDED_MESSAGE);
+        exit.mockRestore();
+        stderr.mockRestore();
+    });
+});

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.1",
   "main": "dist/src/index.js",
   "bin": {
     "awm": "./dist/src/index.js"