agentic-workflow-manager 3.3.0 → 3.3.1

package/dist/src/commands/init.js

@@ -49,6 +49,7 @@ const bundles_1 = require("../core/bundles");
 const registries_1 = require("../core/registries");
 const orchestrator_1 = require("../core/init/orchestrator");
 const steps_1 = require("../core/init/steps");
+const failure_1 = require("../core/init/failure");
 const mutation_targets_1 = require("../core/init/mutation-targets");
 const provider_facts_1 = require("../core/init/provider-facts");
 const install_transaction_1 = require("../core/install-transaction");
@@ -94,6 +95,35 @@ function renderInitOutcome(o) {
     lines.push(`status: ${status} · ${pendingCount} steps require an agent (skills above)`);
     return lines.join('\n');
 }
+// ---------------------------------------------------------------------------
+// Failure reporting
+// ---------------------------------------------------------------------------
+/**
+ * Single exit point for every failed `awm init`. Honours `--json`'s contract on
+ * the error path — the whole point of the flag for a headless bootstrap is to
+ * learn WHICH step failed — and, in human mode, renders the same evidence
+ * through the normal init dashboard instead of discarding it.
+ *
+ * stdout carries the machine-readable document (JSON mode) or the dashboard
+ * (human mode); stderr always carries the one-line summary plus the
+ * transaction verdict, so `2>&1`-free scripts still see a reason.
+ */
+function reportInitFailure(o) {
+    const payload = (0, failure_1.buildInitFailureOutput)({
+        error: o.error,
+        outcome: o.outcome,
+        transaction: o.transaction,
+    });
+    if (o.json) {
+        process.stdout.write(JSON.stringify(payload, null, 2) + '\n');
+    }
+    else if (o.outcome) {
+        process.stdout.write(renderInitOutcome(o.outcome) + '\n');
+    }
+    process.stderr.write(`awm init: ${payload.error}\n`);
+    process.stderr.write(`awm init: ${payload.transaction.note}\n`);
+    return 2;
+}
 async function runInit(opts = {}) {
     const cwd = opts.cwd ?? process.cwd();
     const agent = opts.agent === undefined ? 'claude-code' : (0, providers_1.requireAgentTarget)(opts.agent);
@@ -125,6 +155,10 @@ async function runInit(opts = {}) {
     // and would incorrectly be flagged as a violation.
     const beforeClaudeFacts = agent === 'claude-code' ? null : (0, provider_facts_1.gatherProviderFacts)('claude-code');
     let outcome;
+    // Populated as soon as each becomes available, so the failure reporter can
+    // emit whatever evidence THIS run got far enough to produce.
+    let pipelineOutcome;
+    let transaction;
     try {
         const mergedActions = {
             ...steps_1.defaultActions,
@@ -155,7 +189,11 @@ async function runInit(opts = {}) {
             (0, config_1.savePreferences)(nextPreferences);
             (0, registries_1.seedBaselineRegistry)();
             if ((0, registries_1.listRegistries)().some((r) => !fs_1.default.existsSync(r.contentRoot))) {
-                await mergedActions.syncCache();
+                // `syncRegistries()` reports per-registry failures as RESULTS,
+                // never as throws (core/registries.ts) — swallowing them here
+                // let an unavailable registry degrade silently into some later
+                // step's failure instead of being reported as its own cause.
+                (0, registries_1.assertSyncedRegistriesUsable)((await mergedActions.syncCache()) ?? []);
             }
             const bundles = (0, bundles_1.discoverAllBundles)();
             const ctx = (0, context_1.gatherContext)({ cwd, bundles, agent });
@@ -177,8 +215,13 @@ async function runInit(opts = {}) {
                 confirmExtensions,
                 actions: mergedActions,
             });
+            pipelineOutcome = outcome;
             if (outcome.failed > 0) {
-                throw new Error('one or more init steps failed');
+                // Typed so the outcome — the only record of WHICH step failed
+                // and why — survives the rollback below and reaches the
+                // reporter. A bare Error here is what made `--json` emit
+                // nothing on the one path that most needed it.
+                throw new failure_1.InitStepsFailedError(outcome);
             }
             if (beforeClaudeFacts) {
                 (0, provider_facts_1.assertClaudeBaselinePreserved)(beforeClaudeFacts, (0, provider_facts_1.gatherProviderFacts)('claude-code'));
@@ -188,21 +231,49 @@ async function runInit(opts = {}) {
             outcome.modifiedFiles = backup.targetPaths;
         }
         catch (error) {
-            backup.rollback();
+            // Rollback is best-effort and must never mask the original failure
+            // (same policy as applyInstallPlan's own rollback loop): the
+            // operator needs the step that failed, not the restore that also did.
+            let rollbackError;
+            try {
+                backup.rollback();
+            }
+            catch (e) {
+                rollbackError = e instanceof Error ? e.message : String(e);
+            }
+            transaction = {
+                committed: false,
+                rolledBack: rollbackError === undefined,
+                transactionId: backup.transactionId,
+                restoredFiles: backup.targetPaths,
+                ...(rollbackError === undefined ? {} : { rollbackError }),
+                note: (0, failure_1.transactionNote)(rollbackError === undefined),
+            };
             throw error;
         }
     }
     catch (err) {
-        process.stderr.write(`awm init: internal error: ${err.message}\n`);
-        return 2;
+        // The typed error carries its own outcome (so it stays self-sufficient
+        // for any caller of runInit); `pipelineOutcome` covers the other way a
+        // run can fail *after* the pipeline produced one — e.g. the R19
+        // Claude-baseline assertion.
+        return reportInitFailure({
+            error: err,
+            outcome: err instanceof failure_1.InitStepsFailedError ? err.outcome : pipelineOutcome,
+            transaction,
+            json: opts.json,
+        });
     }
+    // `result` mirrors the exit code, so a consumer can branch on one field
+    // instead of correlating stdout with $?: ok → 0, degraded → 1, failed → 2.
+    const result = outcome.after.overall === 'healthy' ? 'ok' : 'degraded';
     if (opts.json) {
-        process.stdout.write(JSON.stringify(outcome, null, 2) + '\n');
+        process.stdout.write(JSON.stringify({ result, ...outcome }, null, 2) + '\n');
     }
     else {
         process.stdout.write(renderInitOutcome(outcome) + '\n');
     }
-    return outcome.after.overall === 'healthy' ? 0 : 1;
+    return result === 'ok' ? 0 : 1;
 }
 // ---------------------------------------------------------------------------
 // Extension confirmation factory
@@ -240,7 +311,7 @@ function registerInitCommand(program) {
         .option('-y, --yes', 'Skip confirmation prompts')
         .option('-a, --agent <agent>', 'Target agent (default: claude-code)')
         .option('--machine-only', 'Only run machine-level steps (skip project steps)')
-        .option('--json', 'Emit the InitOutcome as JSON')
+        .option('--json', 'Emit the InitOutcome as JSON — on success and on failure (failed steps + rollback)')
         .action(async (options) => {
         (0, paths_1.warnIfUnsupportedPlatform)((m) => console.warn(picocolors_1.default.yellow(`⚠ ${m}`)));
         const code = await runInit({

package/dist/src/core/init/failure.js

@@ -0,0 +1,74 @@
+"use strict";
+// src/core/init/failure.ts
+//
+// Failure evidence for `awm init`.
+//
+// `awm init` is transactional: any failure rolls every write back. That part
+// was always right — what was missing is the EVIDENCE. `runInitSteps` already
+// records, per step, `{ id, action: 'failed', error }` (orchestrator.ts's
+// `wrapStep`), but commands/init.ts used to collapse the whole outcome into a
+// bare `new Error('one or more init steps failed')`, so `--json` — whose only
+// job is to emit that outcome — printed nothing at all on the exact path an
+// operator needs it: a headless cloud bootstrap that just aborted.
+//
+// `InitStepsFailedError` carries the outcome across the rollback boundary, and
+// `buildInitFailureOutput` shapes it into the JSON envelope the CLI emits.
+// The envelope is deliberately self-describing (`result`, `failedSteps`,
+// `transaction`) so a bootstrap script can branch on it without re-deriving
+// anything from prose on stderr.
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.InitStepsFailedError = void 0;
+exports.failedSteps = failedSteps;
+exports.transactionNote = transactionNote;
+exports.buildInitFailureOutput = buildInitFailureOutput;
+/** Only the steps that actually failed — the subset an operator needs first. */
+function failedSteps(outcome) {
+    return outcome.steps.filter((s) => s.action === 'failed');
+}
+/**
+ * Thrown when the step pipeline ran to completion but ≥1 step failed. Carries
+ * the full `InitOutcome` so the rollback path can still emit it, and builds a
+ * message that names every failed step instead of the old generic sentence.
+ */
+class InitStepsFailedError extends Error {
+    outcome;
+    constructor(outcome) {
+        const detail = failedSteps(outcome)
+            .map((s) => `${s.id}: ${s.error ?? 'no error recorded'}`)
+            .join('; ');
+        super(`one or more init steps failed — ${detail || 'no failed step recorded'}`);
+        this.name = 'InitStepsFailedError';
+        this.outcome = outcome;
+    }
+}
+exports.InitStepsFailedError = InitStepsFailedError;
+const NO_TRANSACTION_NOTE = 'init failed before a backup session was opened — nothing was written, so there was nothing to roll back.';
+/** The note explaining what a given rollback outcome means for the machine. */
+function transactionNote(rolledBack) {
+    return rolledBack
+        ? 'the transaction was NOT committed: every path in restoredFiles was restored to its pre-init state. '
+            + '`after` reflects the state observed at the end of the step pipeline, BEFORE this rollback ran.'
+        : 'the transaction was NOT committed and the rollback did not complete — see rollbackError and '
+            + 'restore manually with `awm backup restore <transactionId>`.';
+}
+function buildInitFailureOutput(o) {
+    const steps = o.outcome?.steps ?? [];
+    return {
+        result: 'failed',
+        error: o.error.message,
+        steps,
+        failedSteps: steps.filter((s) => s.action === 'failed'),
+        applied: o.outcome?.applied ?? 0,
+        pending: o.outcome?.pending ?? 0,
+        failed: o.outcome?.failed ?? 0,
+        before: o.outcome?.before ?? null,
+        after: o.outcome?.after ?? null,
+        transaction: o.transaction ?? {
+            committed: false,
+            rolledBack: false,
+            transactionId: null,
+            restoredFiles: [],
+            note: NO_TRANSACTION_NOTE,
+        },
+    };
+}

package/dist/src/core/init/steps.js

@@ -39,7 +39,7 @@ const codex_agents_1 = require("../context/strategies/codex-agents");
 // ---------------------------------------------------------------------------
 const realInjectionOrchestrator = new orchestrator_1.InjectionOrchestrator();
 exports.defaultActions = {
-    syncCache: async () => { await (0, registries_1.syncRegistries)(); },
+    syncCache: async () => (0, registries_1.syncRegistries)(),
     installHook: (o) => (0, install_1.installHook)({
         agent: o.agent,
         registryRoot: o.registryRoot,
@@ -135,14 +135,30 @@ async function stepCache(d) {
     const needsSync = !registryCache.present || registryCache.gitState === 'behind';
     if (!needsSync)
         return ok('machine.cache', 'machine', 'skipped');
+    let results;
     try {
-        await d.actions.syncCache();
-        return ok('machine.cache', 'machine', 'applied');
+        results = (await d.actions.syncCache()) ?? [];
     }
     catch (e) {
         const msg = e instanceof Error ? e.message : String(e);
         return failed('machine.cache', 'machine', msg);
     }
+    // `syncRegistries()` reports per-registry failures as RESULTS, not throws
+    // (registries.ts), so the `try` above catches none of them. Ignoring them
+    // is what let an unavailable base registry degrade silently into some
+    // later step's failure — with its own cause already gone.
+    const errors = (0, registries_1.registrySyncErrors)(results);
+    if (errors.length === 0)
+        return ok('machine.cache', 'machine', 'applied');
+    // A registry that errored and has no content on disk is unusable, and every
+    // later step that reads it will fail for a reason that no longer names this
+    // cause — so machine.cache owns it here. One that errored but still has
+    // content is stale, not broken: record it and carry on.
+    const unusable = (0, registries_1.unusableSyncedRegistries)(results);
+    if (unusable.length > 0) {
+        return failed('machine.cache', 'machine', `registry unavailable — ${(0, registries_1.describeRegistrySyncErrors)(unusable)}`);
+    }
+    return ok('machine.cache', 'machine', 'applied', `stale registries — ${(0, registries_1.describeRegistrySyncErrors)(errors)}`);
 }
 /** Step 2 – Install the session-start hook for the target agent. */
 function stepHook(d) {

package/dist/src/core/registries.js

@@ -13,6 +13,10 @@ exports.contentRoots = contentRoots;
 exports.capabilityRoot = capabilityRoot;
 exports.validateRegistryLayout = validateRegistryLayout;
 exports.syncRegistries = syncRegistries;
+exports.registrySyncErrors = registrySyncErrors;
+exports.describeRegistrySyncErrors = describeRegistrySyncErrors;
+exports.unusableSyncedRegistries = unusableSyncedRegistries;
+exports.assertSyncedRegistriesUsable = assertSyncedRegistriesUsable;
 exports.readRegistryManifest = readRegistryManifest;
 exports.registryNameForPath = registryNameForPath;
 exports.verifyMinCliVersions = verifyMinCliVersions;
@@ -144,6 +148,46 @@ async function syncRegistries() {
     }
     return results;
 }
+/** The errored entries of a `syncRegistries()` run, in `listRegistries()` order. */
+function registrySyncErrors(results) {
+    return results
+        .filter((r) => r.action === 'error')
+        .map((r) => ({ name: r.name, error: r.error }));
+}
+/** `name: reason; name: reason` — the shape both init and stepCache report errors in. */
+function describeRegistrySyncErrors(errors) {
+    return errors.map((e) => `${e.name}: ${e.error}`).join('; ');
+}
+/**
+ * Registries that both errored during sync AND have no content on disk
+ * afterwards — i.e. genuinely unusable, as opposed to merely stale.
+ *
+ * `syncRegistries()` deliberately reports per-registry failures as results
+ * rather than throwing, so a flaky secondary registry never aborts a whole
+ * run. Callers that go on to READ registry content (init) still need to know
+ * whether what they are about to read exists: otherwise a missing registry
+ * resurfaces much later as an unrelated step's failure, with its real cause
+ * already discarded. Note that "usable" is deliberately about content on disk,
+ * not about being a healthy git clone — a seeded content root with no `.git`
+ * fails to sync every time and is still perfectly readable.
+ */
+function unusableSyncedRegistries(results) {
+    const errors = registrySyncErrors(results);
+    if (errors.length === 0)
+        return [];
+    const roots = new Map(listRegistries().map((r) => [r.name, r.contentRoot]));
+    return errors.filter((e) => {
+        const root = roots.get(e.name);
+        return root === undefined || !fs_1.default.existsSync(root);
+    });
+}
+/** `unusableSyncedRegistries` as a guard: throws naming every unusable registry. */
+function assertSyncedRegistriesUsable(results) {
+    const unusable = unusableSyncedRegistries(results);
+    if (unusable.length === 0)
+        return;
+    throw new Error(`registry sync failed and left no content on disk — ${describeRegistrySyncErrors(unusable)}`);
+}
 exports.REGISTRY_MANIFEST_NAME = 'awm-registry.json';
 function readRegistryManifest(root) {
     const file = path_1.default.join(root, exports.REGISTRY_MANIFEST_NAME);

package/dist/tests/commands/init.test.js

@@ -91,6 +91,7 @@ describe('runInit', () => {
         const parsed = JSON.parse(written);
         expect(Array.isArray(parsed.steps)).toBe(true);
         expect(parsed.after.overall).toBe('degraded');
+        expect(parsed.result).toBe('degraded'); // success envelope is self-describing too
         expect(code).toBe(1);
     });
     const prefsFile = () => path_1.default.join(process.env.AWM_HOME, 'preferences.json');
@@ -291,4 +292,85 @@ describe('runInit', () => {
         // rollback must restore its exact pre-run content, not just leave it be.
         expect(JSON.parse(fs_1.default.readFileSync(settingsPath, 'utf8'))).toEqual({ pristine: true, unrelated: 'keep' });
     });
+    // -----------------------------------------------------------------------
+    // Failure evidence — `--json` must honour its contract on the ERROR path
+    // -----------------------------------------------------------------------
+    describe('failure evidence', () => {
+        let errSpy;
+        beforeEach(() => {
+            errSpy = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
+        });
+        afterEach(() => errSpy.mockRestore());
+        const stdout = () => writeSpy.mock.calls.map((c) => c[0]).join('');
+        const stderr = () => errSpy.mock.calls.map((c) => c[0]).join('');
+        /** InitActions whose `machine.hook` step blows up inside the step pipeline. */
+        function actionsWithFailingHook() {
+            const actions = fakeActions([]);
+            actions.installHook = () => { throw new Error('hook boom'); };
+            return actions;
+        }
+        it('--json emits parseable JSON carrying the failed step id and error', async () => {
+            const { runInit } = require('../../src/commands/init');
+            const code = await runInit({
+                cwd: tmpHome,
+                yes: true,
+                json: true,
+                actions: actionsWithFailingHook(),
+            });
+            expect(code).not.toBe(0);
+            expect(code).toBe(2);
+            const parsed = JSON.parse(stdout());
+            expect(parsed.result).toBe('failed');
+            const failedStep = parsed.steps.find((s) => s.action === 'failed');
+            expect(failedStep.id).toBe('machine.hook');
+            expect(failedStep.error).toBe('hook boom');
+            expect(parsed.failedSteps).toEqual([failedStep]);
+            expect(parsed.failed).toBe(1);
+            // The top-level message names the step too — no generic "internal error".
+            expect(parsed.error).toContain('machine.hook');
+            expect(parsed.error).toContain('hook boom');
+            // `before` is the pre-run snapshot the issue asks for.
+            expect(parsed.before.results.length).toBeGreaterThan(0);
+        });
+        it('--json reports the transaction as not committed and rolled back', async () => {
+            const { runInit } = require('../../src/commands/init');
+            await runInit({ cwd: tmpHome, yes: true, json: true, actions: actionsWithFailingHook() });
+            const { transaction } = JSON.parse(stdout());
+            expect(transaction.committed).toBe(false);
+            expect(transaction.rolledBack).toBe(true);
+            expect(typeof transaction.transactionId).toBe('string');
+            expect(Array.isArray(transaction.restoredFiles)).toBe(true);
+            expect(transaction.note).toBeTruthy();
+            // The backup manifest on disk agrees: nothing was committed.
+            const { listBackups } = require('../../src/core/install-transaction');
+            const backups = listBackups();
+            expect(backups.length).toBeGreaterThan(0);
+            expect(backups.every((b) => b.committed)).toBe(false);
+            // …and the run really did not persist anything.
+            expect(fs_1.default.existsSync(prefsFile())).toBe(false);
+        });
+        it('--json still emits a failure envelope when init fails before the step pipeline', async () => {
+            const actions = fakeActions([]);
+            actions.syncCache = async () => { throw new Error('registry unreachable'); };
+            const { runInit } = require('../../src/commands/init');
+            const code = await runInit({ cwd: tmpHome, yes: true, json: true, actions });
+            expect(code).toBe(2);
+            const parsed = JSON.parse(stdout());
+            expect(parsed.result).toBe('failed');
+            expect(parsed.error).toContain('registry unreachable');
+            expect(parsed.steps).toEqual([]);
+            expect(parsed.before).toBeNull();
+            expect(parsed.transaction.committed).toBe(false);
+        });
+        it('human mode renders the failed outcome instead of swallowing it', async () => {
+            const { runInit } = require('../../src/commands/init');
+            const code = await runInit({ cwd: tmpHome, yes: true, actions: actionsWithFailingHook() });
+            expect(code).toBe(2);
+            expect(stdout()).toContain('AWM · init');
+            expect(stdout()).toContain('machine.hook');
+            expect(stdout()).toContain('hook boom');
+            expect(stderr()).toContain('machine.hook');
+            expect(stderr()).toContain('hook boom');
+        });
+    });
 });

package/dist/tests/core/init/steps-registry-sync.test.js

@@ -0,0 +1,109 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+// stepCache vs. syncRegistries()'s RESULT-shaped failures.
+//
+// `syncRegistries()` never throws on a per-registry failure — it returns
+// `{ action: 'error', error }` for that registry and keeps going. stepCache
+// used to `await` it and report a blanket 'applied', so a registry that never
+// landed on disk degraded silently into some later step's failure with its own
+// cause already gone.
+//
+// Isolated in its own file (not steps.test.ts) because these assertions reach
+// `listRegistries()`, which resolves AWM_HOME at module require time — the env
+// override therefore has to happen before the module is required, which means
+// `jest.resetModules()` + `require`, not a static import.
+const fs_1 = __importDefault(require("fs"));
+const os_1 = __importDefault(require("os"));
+const path_1 = __importDefault(require("path"));
+describe('stepCache — registry sync error results', () => {
+    let tmpHome;
+    let originalHome;
+    let originalAwmHome;
+    beforeEach(() => {
+        tmpHome = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'awm-stepcache-'));
+        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();
+    });
+    afterEach(() => {
+        fs_1.default.rmSync(tmpHome, { 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;
+    });
+    const awmHome = () => process.env.AWM_HOME;
+    /** Declares `names` in registries.json; creates a content root only for `withContent`. */
+    function configureRegistries(names, withContent) {
+        fs_1.default.mkdirSync(awmHome(), { recursive: true });
+        fs_1.default.writeFileSync(path_1.default.join(awmHome(), 'registries.json'), JSON.stringify(names.map((name) => ({ name, remote: `https://example.com/${name}.git` })), null, 2));
+        for (const name of withContent) {
+            const skills = path_1.default.join(awmHome(), 'registries', name, 'skills');
+            fs_1.default.mkdirSync(skills, { recursive: true });
+            fs_1.default.writeFileSync(path_1.default.join(skills, 'placeholder.md'), '# placeholder\n');
+        }
+    }
+    /** InitDeps just complete enough for stepCache: a machine with no registry cache yet. */
+    function deps(results) {
+        const ctx = {
+            machine: {
+                registryCache: { present: false },
+                hook: { present: true, degraded: false },
+                devCore: { present: true, brokenLinks: [] },
+                ambient: { wanted: [], installed: [] },
+                contextInjection: [],
+                globalSkills: { valid: [], repairable: [], dead: [] },
+            },
+            project: null,
+        };
+        const actions = { syncCache: async () => results };
+        return {
+            cwd: tmpHome, ctx, bundles: [], agent: 'claude-code', enabledAgents: ['claude-code'],
+            installMethod: 'symlink', registryRoot: '', contentDir: '', sensorPacksRoot: '',
+            confirmExtensions: async (p) => p, actions,
+        };
+    }
+    async function run(results) {
+        const { stepCache } = require('../../../src/core/init/steps');
+        return stepCache(deps(results));
+    }
+    it('fails, naming the registry, when a sync error left no content on disk', async () => {
+        configureRegistries(['baseline'], []);
+        const r = await run([{ name: 'baseline', action: 'error', error: 'could not clone' }]);
+        expect(r.action).toBe('failed');
+        expect(r.error).toContain('baseline');
+        expect(r.error).toContain('could not clone');
+    });
+    it('stays applied but records the error when content is already on disk', async () => {
+        configureRegistries(['baseline'], ['baseline']);
+        const r = await run([{ name: 'baseline', action: 'error', error: 'pull timed out' }]);
+        expect(r.action).toBe('applied');
+        expect(r.detail).toContain('baseline');
+        expect(r.detail).toContain('pull timed out');
+    });
+    it('fails on a secondary registry that never landed, even when the base one synced', async () => {
+        configureRegistries(['baseline', 'documentation'], ['baseline']);
+        const r = await run([
+            { name: 'baseline', action: 'pulled', version: 'v1.0.0' },
+            { name: 'documentation', action: 'error', error: 'host unreachable' },
+        ]);
+        expect(r.action).toBe('failed');
+        expect(r.error).toContain('documentation');
+        expect(r.error).not.toContain('baseline');
+    });
+    it('reports applied with no detail when every registry synced', async () => {
+        configureRegistries(['baseline'], ['baseline']);
+        const r = await run([{ name: 'baseline', action: 'pulled', version: 'v1.0.0' }]);
+        expect(r.action).toBe('applied');
+        expect(r.detail).toBeUndefined();
+    });
+});

package/package.json

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