agentic-workflow-manager 9.7.0 → 9.8.0

package/dist/tests/structural/bundle-skill-reference-contract.test.js

@@ -0,0 +1,56 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const fs_1 = __importDefault(require("fs"));
+const path_1 = __importDefault(require("path"));
+const ROOT = path_1.default.resolve(__dirname, '../../..');
+const BUNDLES_SOURCE = path_1.default.join(ROOT, 'cli/src/core/bundles.ts');
+const GUIDE = path_1.default.join(ROOT, 'docs/guides/authoring-a-registry-with-an-orchestrator.md');
+const read = (file) => fs_1.default.readFileSync(file, 'utf8');
+function sourceFiles(dir) {
+    return fs_1.default.readdirSync(dir, { withFileTypes: true }).flatMap((entry) => {
+        const file = path_1.default.join(dir, entry.name);
+        if (entry.isDirectory())
+            return sourceFiles(file);
+        return entry.isFile() && file.endsWith('.ts') ? [file] : [];
+    });
+}
+function jsonFences(markdown) {
+    return Array.from(markdown.matchAll(/```json\n([\s\S]*?)\n```/g), ([, json]) => json);
+}
+describe('bundle skill-reference contract', () => {
+    it('keeps active bundle consumers string-only, without BundleSkillRef', () => {
+        const source = read(BUNDLES_SOURCE);
+        const bundleSkillRefs = sourceFiles(path_1.default.join(ROOT, 'cli/src'))
+            .filter((file) => /\bBundleSkillRef\b/.test(read(file)));
+        expect(bundleSkillRefs).toEqual([]);
+        expect(source).toMatch(/interface BundleDefinition\s*\{[\s\S]*?\bskills:\s*string\[\];/);
+    });
+    it('uses canonical string skill references in the active authoring guide', () => {
+        const guide = read(GUIDE);
+        const bundleExample = jsonFences(guide).find((json) => JSON.parse(json).name === 'mi-proceso');
+        expect(bundleExample).toBeDefined();
+        const skills = JSON.parse(bundleExample).skills;
+        expect(skills).toEqual(['mi-proceso']);
+        expect(skills.every((skill) => typeof skill === 'string')).toBe(true);
+    });
+    it('places the CLI 9 migration boundary immediately after the bundle example', () => {
+        const guide = read(GUIDE);
+        const bundleFence = '```json\n{\n  "name": "mi-proceso"';
+        const start = guide.indexOf(bundleFence);
+        const end = guide.indexOf('```', start) + 3;
+        const following = guide.slice(end, guide.indexOf('## 5.', end));
+        expect(start).toBeGreaterThanOrEqual(0);
+        expect(following).toMatch(/CLI 9[\s\S]*(?:legacy object|objeto legacy)[\s\S]*\{\s*name\s*,\s*onSignal:\s*true\s*\}/i);
+        expect(following).toMatch(/ignor(?:e|a)[\s\S]*boolean[\s\S]*onSignal/i);
+        expect(following).toMatch(/warning/i);
+        expect(following).toMatch(/CLI 10[\s\S]*(?:rejects?|rechaza)[\s\S]*(?:objects?|objetos?)/i);
+    });
+    it('keeps every JSON fence in the touched guide parseable', () => {
+        for (const json of jsonFences(read(GUIDE))) {
+            expect(() => JSON.parse(json)).not.toThrow();
+        }
+    });
+});

package/dist/tests/structural/jest-environment-is-isolated.test.js

@@ -6,44 +6,49 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const os_1 = __importDefault(require("os"));
 const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
+function isWithin(parent, candidate, pathApi = path_1.default) {
+    const relative = pathApi.relative(parent, candidate);
+    return relative === '' || (relative !== '..' && !relative.startsWith(`..${pathApi.sep}`) && !pathApi.isAbsolute(relative));
+}
 describe('Jest environment isolation', () => {
+    it('treats a path on another volume as outside the operator home', () => {
+        expect(isWithin('C:\\operator-home', 'D:\\suite-root', path_1.default.win32)).toBe(false);
+    });
     it('does not inherit the operator Codex home', () => {
         expect(process.env.CODEX_HOME).toBeUndefined();
     });
     it('uses a suite-owned temporary directory', () => {
         expect(os_1.default.tmpdir()).toContain('awm-jest-');
     });
-    it('falls back to the system temp directory when the home cache is read-only', async () => {
+    it('routes every unscoped AWM write into the suite-owned temporary directory', () => {
+        expect(process.env.AWM_HOME).toBe(path_1.default.join(process.env.AWM_JEST_TMPDIR, 'awm-home'));
+    });
+    it('keeps the suite root and both homes outside the operator home', () => {
+        const suiteRoot = process.env.AWM_JEST_TMPDIR;
+        const operatorHome = fs_1.default.realpathSync(os_1.default.userInfo().homedir);
+        expect(isWithin(operatorHome, suiteRoot)).toBe(false);
+        expect(process.env.HOME).toBe(path_1.default.join(suiteRoot, 'home'));
+        expect(process.env.AWM_HOME).toBe(path_1.default.join(suiteRoot, 'awm-home'));
+    });
+    it('creates its physical root below the system temp directory, not the operator home', async () => {
         const originalEnv = { ...process.env };
-        const fallback = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'awm-jest-fallback-'));
+        const systemTemp = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'awm-jest-system-'));
+        const operatorHome = path_1.default.join(systemTemp, 'operator-home');
         try {
+            fs_1.default.mkdirSync(operatorHome);
             jest.resetModules();
             jest.doMock('os', () => ({
                 ...jest.requireActual('os'),
-                homedir: () => '/readonly-home',
-                tmpdir: () => fallback,
-            }));
-            const actualFs = jest.requireActual('fs');
-            jest.doMock('fs', () => ({
-                ...actualFs,
-                mkdirSync: jest.fn((target, ...args) => {
-                    if (String(target).includes('readonly-home')) {
-                        return undefined;
-                    }
-                    return actualFs.mkdirSync(target, ...args);
-                }),
-                mkdtempSync: jest.fn((prefix, ...args) => {
-                    if (prefix.includes('readonly-home')) {
-                        const error = new Error('read-only');
-                        error.code = 'EROFS';
-                        throw error;
-                    }
-                    return actualFs.mkdtempSync(prefix, ...args);
-                }),
+                homedir: () => operatorHome,
+                tmpdir: () => systemTemp,
             }));
             const setup = require('../../jest.global-setup.js');
             await expect(setup()).resolves.toBeUndefined();
-            expect(process.env.AWM_JEST_TMPDIR?.startsWith(path_1.default.join(fallback, 'awm-cache', 'awm-jest-'))).toBe(true);
+            const suiteRoot = process.env.AWM_JEST_TMPDIR;
+            expect(suiteRoot.startsWith(path_1.default.join(fs_1.default.realpathSync(systemTemp), 'awm-jest-'))).toBe(true);
+            expect(isWithin(operatorHome, suiteRoot)).toBe(false);
+            expect(process.env.HOME).toBe(path_1.default.join(suiteRoot, 'home'));
+            expect(process.env.AWM_HOME).toBe(path_1.default.join(suiteRoot, 'awm-home'));
         }
         finally {
             jest.dontMock('fs');
@@ -52,7 +57,7 @@ describe('Jest environment isolation', () => {
             for (const key of Object.keys(process.env))
                 delete process.env[key];
             Object.assign(process.env, originalEnv);
-            fs_1.default.rmSync(fallback, { recursive: true, force: true });
+            fs_1.default.rmSync(systemTemp, { recursive: true, force: true });
         }
     });
 });

package/dist/tests/structural/native-release-artifacts.test.js

@@ -9,6 +9,22 @@ const path_1 = __importDefault(require("path"));
 const CLI_ROOT = path_1.default.resolve(__dirname, '..', '..');
 const REPO_ROOT = path_1.default.resolve(CLI_ROOT, '..');
 describe('native release artifacts', () => {
+    it('checks Windows admission before full suites and stops a failed suite without weakening publication', () => {
+        for (const name of ['ci.yml', 'release.yml']) {
+            const workflow = fs_1.default.readFileSync(path_1.default.join(REPO_ROOT, '.github', 'workflows', name), 'utf8');
+            expect(workflow).toContain('- name: Windows admission smoke');
+            expect(workflow).toContain("if: runner.os == 'Windows'");
+            expect(workflow).toContain('run: npx jest tests/commands/plan/index.test.ts --runInBand --bail');
+            expect(workflow).toContain('run: npx jest --runInBand --bail');
+            expect(workflow.indexOf('- name: Windows admission smoke')).toBeLessThan(workflow.indexOf('run: npx jest --runInBand --bail'));
+        }
+        const release = fs_1.default.readFileSync(path_1.default.join(REPO_ROOT, '.github', 'workflows', 'release.yml'), 'utf8');
+        expect(release).toMatch(/release:\s*\n\s*needs: test/);
+        for (const target of ['linux-x64', 'linux-arm64', 'darwin-x64', 'darwin-arm64', 'win32-x64', 'win32-arm64']) {
+            expect(release).toContain(`target: ${target}`);
+        }
+        expect(release).toContain('cancel-in-progress: false');
+    });
     it('keeps native prebuilds generated instead of versioned source', () => {
         const gitignore = fs_1.default.readFileSync(path_1.default.join(REPO_ROOT, '.gitignore'), 'utf8');
         const tracked = (0, child_process_1.execFileSync)('git', ['ls-files', '--', 'cli/prebuilds'], {

package/dist/tests/utils/grouping.test.js

@@ -7,10 +7,10 @@ const bundle = (over) => ({
 });
 const processes = [
     bundle({ name: 'core-dev', description: 'Core development skills',
-        skills: [{ name: 'brainstorming', onSignal: false }, { name: 'shared-skill', onSignal: false }],
+        skills: ['brainstorming', 'shared-skill'],
         workflows: [], agents: [] }),
     bundle({ name: 'docs', description: 'Documentation skills',
-        skills: [{ name: 'docs-assistant', onSignal: false }, { name: 'shared-skill', onSignal: false }],
+        skills: ['docs-assistant', 'shared-skill'],
         workflows: [], agents: [] }),
 ];
 const formatLabel = (c) => c.baseName;

package/dist/tests/utils/registry-view.test.js

@@ -10,10 +10,10 @@ const bundle = (over) => ({
 });
 const processes = [
     bundle({ name: 'core-dev', description: 'Dev lifecycle', scope: 'baseline',
-        skills: [{ name: 'brainstorming', onSignal: false }, { name: 'shared', onSignal: false }],
+        skills: ['brainstorming', 'shared'],
         workflows: ['exec'], agents: ['plan'] }),
     bundle({ name: 'docs', description: 'Docs as code',
-        skills: [{ name: 'shared', onSignal: false }] }),
+        skills: ['shared'] }),
 ];
 describe('buildPackageView', () => {
     it('groups artifacts under their package with correct counts and install names', () => {
@@ -108,7 +108,7 @@ describe('visibility', () => {
     it('marks a private bundle on its PackageView', () => {
         const priv = [
             bundle({ name: 'secret', description: 'private', visibility: 'private',
-                skills: [{ name: 'a', onSignal: false }] }),
+                skills: ['a'] }),
         ];
         const view = (0, registry_view_1.buildPackageView)([skill('a')], [], [], priv);
         expect(view.find((p) => p.name === 'secret').visibility).toBe('private');
@@ -121,7 +121,7 @@ describe('visibility', () => {
 describe('packageSummaryLines width-awareness', () => {
     it('truncates the description column when a width is given', () => {
         const view = (0, registry_view_1.buildPackageView)([skill('brainstorming', 'x'.repeat(200))], [], [], [
-            bundle({ name: 'core-dev', description: 'x'.repeat(200), skills: [{ name: 'brainstorming', onSignal: false }] }),
+            bundle({ name: 'core-dev', description: 'x'.repeat(200), skills: ['brainstorming'] }),
         ]);
         const lines = (0, registry_view_1.packageSummaryLines)(view, 60);
         for (const l of lines)
@@ -130,7 +130,7 @@ describe('packageSummaryLines width-awareness', () => {
     });
     it('does not truncate when no width is given (piped output)', () => {
         const view = (0, registry_view_1.buildPackageView)([skill('brainstorming', 'y'.repeat(200))], [], [], [
-            bundle({ name: 'core-dev', description: 'z'.repeat(200), skills: [{ name: 'brainstorming', onSignal: false }] }),
+            bundle({ name: 'core-dev', description: 'z'.repeat(200), skills: ['brainstorming'] }),
         ]);
         const lines = (0, registry_view_1.packageSummaryLines)(view); // no width
         expect(lines.some((l) => l.includes('z'.repeat(200)))).toBe(true);
@@ -139,7 +139,7 @@ describe('packageSummaryLines width-awareness', () => {
 describe('artifactPickerItems', () => {
     it('prepends an "install entire package" sentinel item, then one per artifact', () => {
         const view = (0, registry_view_1.buildPackageView)([skill('a', 'desc a'), skill('b', 'desc b')], [], [], [
-            bundle({ name: 'p', description: 'pkg', skills: [{ name: 'a', onSignal: false }, { name: 'b', onSignal: false }] }),
+            bundle({ name: 'p', description: 'pkg', skills: ['a', 'b'] }),
         ]);
         const items = (0, registry_view_1.artifactPickerItems)(view.find((p) => p.name === 'p'));
         expect(items[0].value).toBe(registry_view_1.ALL_SENTINEL);
@@ -150,7 +150,7 @@ describe('artifactPickerItems', () => {
 describe('packagePickerItems', () => {
     it('builds one item per package with a count+description summary', () => {
         const view = (0, registry_view_1.buildPackageView)([skill('a')], [], [], [
-            bundle({ name: 'p', description: 'pkg desc', skills: [{ name: 'a', onSignal: false }] }),
+            bundle({ name: 'p', description: 'pkg desc', skills: ['a'] }),
         ]);
         const items = (0, registry_view_1.packagePickerItems)(view);
         expect(items[0].value).toBe('p');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-workflow-manager",
-  "version": "9.7.0",
+  "version": "9.8.0",
   "main": "dist/src/index.js",
   "bin": {
     "awm": "./dist/src/index.js"
@@ -47,7 +47,7 @@
   },
   "description": "CLI to manage Agentic Workflows - install skills, workflows, and processes for AI agents",
   "dependencies": {
-    "@clack/prompts": "^1.0.1",
+    "@clack/prompts": "^0.11.0",
     "commander": "^14.0.3",
     "picocolors": "^1.1.1",
     "semver": "^7.7.4",