agentic-workflow-manager 9.22.0 → 9.22.1

package/dist/src/core/versioning.js

@@ -1,7 +1,4 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.normalizePin = normalizePin;
 exports.resolveTargetRef = resolveTargetRef;
@@ -12,7 +9,7 @@ exports.compareSemver = compareSemver;
 //
 // Resolver de versiones por tags semver — WS-3. Sin estado propio: git es la
 // única fuente de verdad (la versión vigente se deriva de `git describe`).
-const simple_git_1 = __importDefault(require("simple-git"));
+const registry_git_1 = require("./registry-git");
 const config_1 = require("../utils/config");
 const TAG_RE = /^v(\d+)\.(\d+)\.(\d+)$/;
 function normalizePin(pin) {
@@ -59,7 +56,7 @@ async function fetchVersionTags(git) {
  * pin declarado > último tag semver (canal stable) > HEAD (canal dev o repo sin tags).
  */
 async function resolveTargetRef(repoDir, opts) {
-    const git = (0, simple_git_1.default)(repoDir);
+    const git = (0, registry_git_1.createRegistryGit)(repoDir);
     const tags = await fetchVersionTags(git);
     if (opts.pin) {
         const want = `v${normalizePin(opts.pin)}`;
@@ -82,7 +79,7 @@ async function resolveTargetRef(repoDir, opts) {
 /** Versión checkouteada actual ("X.Y.Z") si HEAD coincide exactamente con un tag semver; null si sigue un branch. */
 async function currentVersion(repoDir) {
     try {
-        const out = (await (0, simple_git_1.default)(repoDir).raw(['describe', '--tags', '--exact-match', 'HEAD'])).trim();
+        const out = (await (0, registry_git_1.createRegistryGit)(repoDir).raw(['describe', '--tags', '--exact-match', 'HEAD'])).trim();
         return TAG_RE.test(out) ? out.slice(1) : null;
     }
     catch {

package/dist/src/index.js

@@ -19,6 +19,7 @@ const provider_preflight_1 = require("./ui/provider-preflight");
 const discovery_1 = require("./core/discovery");
 const bundles_1 = require("./core/bundles");
 const registries_1 = require("./core/registries");
+const registry_sync_report_1 = require("./core/registry-sync-report");
 const install_planner_1 = require("./core/install-planner");
 const install_transaction_1 = require("./core/install-transaction");
 const profile_1 = require("./core/profile");
@@ -129,10 +130,8 @@ exports.program.command('add [name]')
     s.start('Syncing registries...');
     const results = await (0, registries_1.syncRegistries)();
     s.stop('Registries synced.');
-    for (const r of results) {
-        if (r.action === 'error')
-            console.warn(picocolors_1.default.yellow(`  ⚠  registry ${r.name}: ${r.error}`));
-    }
+    // SG-7 soft path: usable cache may continue after stating cached version + SG-6 cause.
+    await (0, registry_sync_report_1.reportRegistrySyncSoftFailures)(results, (m) => console.warn(picocolors_1.default.yellow(m)));
     try {
         (0, registries_1.assertSyncedRegistriesUsable)(results);
     }
@@ -458,10 +457,8 @@ exports.program.command('list [package]')
     s.start('Syncing registries...');
     const listSyncResults = await (0, registries_1.syncRegistries)();
     s.stop('Registries synced.');
-    for (const r of listSyncResults) {
-        if (r.action === 'error')
-            console.warn(picocolors_1.default.yellow(`  ⚠  registry ${r.name}: ${r.error}`));
-    }
+    // SG-7 soft path: usable cache may continue after stating cached version + SG-6 cause.
+    await (0, registry_sync_report_1.reportRegistrySyncSoftFailures)(listSyncResults, (m) => console.warn(picocolors_1.default.yellow(m)));
     try {
         (0, registries_1.assertSyncedRegistriesUsable)(listSyncResults);
     }

package/dist/tests/commands/change/new.test.js

@@ -92,6 +92,37 @@ function sharedRepo() {
     beforeAll(() => { newRepo(); receiptForHead(); });
     afterAll(() => repo.cleanup());
 }
+/**
+ * Prebuilt states: each is built once, lazily, in its own clone of the template (the git work happens there once), and every test
+ * that starts from it gets a private file copy (no git process) that it may change freely. `build` returns what the test needs
+ * of how the state was made (a command result, a text), handed back on every use.
+ */
+const fixtures = new Map();
+afterAll(() => { for (const f of fixtures.values())
+    f.state.cleanup(); });
+function fromFixture(key, build) {
+    let built = fixtures.get(key);
+    if (built === undefined) {
+        const own = repo;
+        repo = (0, helpers_1.cloneRepo)(template);
+        try {
+            built = { state: repo, value: build() };
+            fixtures.set(key, built);
+        }
+        catch (error) {
+            repo.cleanup();
+            throw error;
+        }
+        finally {
+            repo = own;
+        }
+    }
+    repo.cleanup();
+    repo = (0, helpers_1.cloneRepo)(built.state);
+    return built.value;
+}
+/** The repository after `receiptForHead()` and `run('change-record')` (the record is created, not committed): that run's result. */
+const created = () => fromFixture('created', () => { receiptForHead(); return run('change-record'); });
 // The command writes only into the throwaway repository: the real checkout is never touched.
 afterEach(() => expect(listReal()).toEqual(realBefore));
 afterAll(() => expect(listReal()).toEqual(realBefore));
@@ -181,20 +212,17 @@ describe('awm change new: receipt and clean tree (R3.6)', () => {
 describe('awm change new: skeleton (R3.7, R3.9)', () => {
     freshRepo();
     test('prints the created path and writes exactly that file', () => {
-        receiptForHead();
-        const r = run('change-record');
+        const r = created();
         expect(r).toEqual({ code: 0, stdout: 'Created docs/changes/2026-10-17-change-record.md\n', stderr: '' });
         expect(recordFiles()).toEqual(['2026-10-17-change-record.md']);
     });
     test('derives type, scope, semver, breaking and modules from the branch commits', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const fm = frontmatter(read());
         expect(fm).toMatchObject({ schema: 'awm.change-record/v1', id: '2026-10-17-change-record', type: 'feat', scope: 'changes', semver: 'minor', breaking: false, modules: ['cli/src'] });
     });
     test('copies commit, overall, sensors and the digest of the receipt into verification', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const expected = {
             commit: repo.git('rev-parse', 'HEAD'),
             overall: 'pass',
@@ -211,8 +239,7 @@ describe('awm change new: skeleton (R3.7, R3.9)', () => {
         expect(frontmatter(read()).verification.receipt_digest).toBe(digest);
     });
     test('every narrative field and every body section carries the pending marker', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const text = read();
         const fm = frontmatter(text);
         expect(fm.title).toBe(types_1.PENDING_MARKER);
@@ -225,8 +252,7 @@ describe('awm change new: skeleton (R3.7, R3.9)', () => {
         }
     });
     test('the skeleton fails validation only because of its pending markers', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const { diagnostics } = (0, validate_shape_1.validateRecordShape)('2026-10-17-change-record.md', read(), { root: repo.root });
         expect(diagnostics.map((d) => d.code)).toEqual(['pending-content']);
     });
@@ -246,13 +272,11 @@ describe('awm change new: skeleton (R3.7, R3.9)', () => {
         expect(frontmatter(read()).links).toEqual({ plan: 'docs/plans/p.md', issue: 'https://github.com/o/r/issues/217' });
     });
     test('without --plan and --issue there is no links key', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         expect(frontmatter(read())).not.toHaveProperty('links');
     });
     test('the record ends with a newline and uses LF only', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const text = read();
         expect(text.endsWith('\n')).toBe(true);
         expect(text).not.toContain('\r');
@@ -310,8 +334,20 @@ describe('awm change new: the 65536-byte boundary of the record (R3.4, R1.8)', (
         ...Array.from({ length: fillers }, (_, i) => ({ name: `f${String(i).padStart(4, '0')}${'x'.repeat(95)}`, status: 'pass', errors: [] })),
         { name: `v${'x'.repeat(variable - 1)}`, status: 'pass', errors: [] },
     ];
-    const setReceipt = (fillers, variable) => {
-        (0, receipt_1.writeReceipt)(repo.root, (0, receipt_1.buildReceipt)({ commit: repo.git('rev-parse', 'HEAD'), ranAt: RECEIPT_TIME, cliVersion: '9.20.0', output: { overall: 'pass', sensors: sensorList(fillers, variable) } }));
+    /** The receipt for HEAD of (fillers, variable), not written. */
+    const receiptOf = (head, fillers, variable) => (0, receipt_1.buildReceipt)({ commit: head, ranAt: RECEIPT_TIME, cliVersion: '9.20.0', output: { overall: 'pass', sensors: sensorList(fillers, variable) } });
+    const setReceipt = (fillers, variable) => { (0, receipt_1.writeReceipt)(repo.root, receiptOf(repo.git('rev-parse', 'HEAD'), fillers, variable)); };
+    /** The verification block the command copies from a receipt. */
+    const verificationOf = (r) => ({ commit: r.commit, overall: r.overall, sensors: r.sensors, receipt_digest: r.digest });
+    /** The derived fields a record states. */
+    const fieldsOf = (text) => {
+        const fm = frontmatter(text);
+        return { type: fm.type, scope: fm.scope, semver: fm.semver, breaking: fm.breaking, modules: fm.modules };
+    };
+    const bytesOf = (text) => {
+        if (text === null)
+            throw new Error('the record could not be rendered');
+        return Buffer.byteLength(text, 'utf8');
     };
     /** Largest n in [lo, hi] for which `fits(n)` holds, `fits` being true up to a point and false after it. */
     const largest = (lo, hi, fits) => {
@@ -326,12 +362,17 @@ describe('awm change new: the 65536-byte boundary of the record (R3.4, R1.8)', (
         }
         return a;
     };
-    /** Runs `attempt` for a receipt of (fillers, variable); `bytes` is the size of the file it wrote, null when it failed as too large. */
-    function findBoundary(attempt) {
-        const fits = (fillers, variable) => { setReceipt(fillers, variable); return attempt().bytes; };
-        const fillers = largest(0, 700, (k) => fits(k, 1) !== null);
-        const variable = largest(1, 128, (l) => fits(fillers, l) !== null);
-        return { fillers, variable, atLimit: fits(fillers, variable), oneMore: variable < 128 ? fits(fillers, variable + 1) : -1 };
+    /**
+     * The receipt (fillers, variable) whose record is exactly at the limit is searched with `predict` (the size of the record the
+     * command would write for it: the pure renderer the command writes with, no git), then the command itself is run on that receipt
+     * and on the one a byte larger: `attempt` reports the size of the file it wrote, null when it failed as too large.
+     */
+    function findBoundary(predict, attempt) {
+        const fits = (fillers, variable) => predict(fillers, variable) <= types_1.MAX_RECORD_BYTES;
+        const fillers = largest(0, 700, (k) => fits(k, 1));
+        const variable = largest(1, 128, (l) => fits(fillers, l));
+        const real = (f, v) => { setReceipt(f, v); return attempt().bytes; };
+        return { fillers, variable, atLimit: real(fillers, variable), oneMore: variable < 128 ? real(fillers, variable + 1) : -1 };
     }
     test('change new: a record of exactly 65536 bytes is written, one of 65537 is record-too-large', () => {
         const attempt = () => {
@@ -344,7 +385,15 @@ describe('awm change new: the 65536-byte boundary of the record (R3.4, R1.8)', (
             expect(r.stderr).toContain('record-too-large');
             return { bytes: null };
         };
-        const found = findBoundary(attempt);
+        const head = repo.git('rev-parse', 'HEAD');
+        setReceipt(0, 1);
+        expect(run('change-record').code).toBe(0);
+        const fields = fieldsOf(read());
+        const predict = (f, v) => bytesOf(renderModule.renderSkeleton({ id: '2026-10-17-change-record', fields, verification: verificationOf(receiptOf(head, f, v)), links: {}, provenance: {} }));
+        // The prediction is the command's own output: the record it just wrote, byte for byte.
+        expect(read()).toBe(renderModule.renderSkeleton({ id: '2026-10-17-change-record', fields, verification: verificationOf(receiptOf(head, 0, 1)), links: {}, provenance: {} }));
+        fs_1.default.rmSync(file());
+        const found = findBoundary(predict, attempt);
         expect(found.variable).toBeLessThan(128);
         expect(found.atLimit).toBe(65536);
         expect(found.oneMore).toBeNull();
@@ -366,7 +415,16 @@ describe('awm change new: the 65536-byte boundary of the record (R3.4, R1.8)', (
             expect(read()).toBe(original);
             return { bytes: null };
         };
-        const found = findBoundary(attempt);
+        const head = repo.git('rev-parse', 'HEAD');
+        setReceipt(0, 1);
+        expect(run('change-record', { refresh: true }).code).toBe(0);
+        const refreshed = read();
+        const fields = fieldsOf(refreshed);
+        fs_1.default.writeFileSync(file(), original);
+        const predict = (f, v) => bytesOf(renderModule.refreshDerived(original, fields, verificationOf(receiptOf(head, f, v))));
+        // The prediction is the command's own output: the record it just refreshed, byte for byte.
+        expect(renderModule.refreshDerived(original, fields, verificationOf(receiptOf(head, 0, 1)))).toBe(refreshed);
+        const found = findBoundary(predict, attempt);
         expect(found.variable).toBeLessThan(128);
         expect(found.atLimit).toBe(65536);
         expect(found.oneMore).toBeNull();
@@ -389,8 +447,7 @@ describe('awm change new: HEAD moving while the command runs', () => {
         expect(recordFiles()).toEqual([]);
     });
     test('a HEAD that moves before a refresh is written is head-moved and the record is untouched', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         repo.git('add', '-A');
         repo.git('commit', '-q', '-m', 'docs(changes): record');
         receiptForHead();
@@ -462,8 +519,7 @@ describe('awm change new: provenance (R3.8)', () => {
         return r.state;
     }
     test('omits the provenance key entirely when no source exists', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         expect(frontmatter(read())).not.toHaveProperty('provenance');
     });
     test('model and session come only from flags', () => {
@@ -526,23 +582,20 @@ describe('awm change new: provenance (R3.8)', () => {
 describe('awm change new: no overwrite (R3.10)', () => {
     freshRepo();
     test('refuses to overwrite on the same date and leaves the file untouched', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         fs_1.default.writeFileSync(file(), 'edited by hand');
         expectFailure(run('change-record'), 'record-exists', 'use --refresh to update derived fields');
         expect(read()).toBe('edited by hand');
     });
     test('refuses the same slug on another date', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const before = read();
         expectFailure(run('change-record', {}, { now: () => new Date('2026-10-20T12:00:00.000Z') }), 'record-exists', 'use --refresh to update derived fields');
         expect(recordFiles()).toEqual(['2026-10-17-change-record.md']);
         expect(read()).toBe(before);
     });
     test('a different slug is not blocked by an existing record', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         expect(run('change-record-2').code).toBe(0);
         expect(recordFiles()).toEqual(['2026-10-17-change-record-2.md', '2026-10-17-change-record.md']);
     });
@@ -561,8 +614,7 @@ describe('awm change new: no overwrite (R3.10)', () => {
         expect(fs_1.default.readFileSync(outside, 'utf8')).toBe('precious');
     });
     test('a listing that misses an existing file still never overwrites it (exclusive create)', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         fs_1.default.writeFileSync(file(), 'edited by hand');
         interceptFs('readdirSync', (a) => isRecordsDirArg(a[0]), () => []);
         expectFailure(run('change-record'), 'record-exists', 'use --refresh to update derived fields');
@@ -842,7 +894,7 @@ describe('awm change new: base and derivation diagnostics (R3.1, R3.2, R3.5)', (
 });
 describe('awm change new --refresh (R3.11)', () => {
     freshRepo();
-    function recordCommitted(mutate = (t) => t) {
+    function buildRecordCommitted(mutate) {
         receiptForHead();
         run('change-record');
         const edited = mutate(read().split(types_1.PENDING_MARKER).join('written by agent'));
@@ -853,6 +905,10 @@ describe('awm change new --refresh (R3.11)', () => {
         receiptForHead();
         return edited;
     }
+    /** A filled record committed, a later fix(core) commit and a receipt for it. Without `mutate` the state is prebuilt (fromFixture). */
+    function recordCommitted(mutate) {
+        return mutate === undefined ? fromFixture('record committed', () => buildRecordCommitted((t) => t)) : buildRecordCommitted(mutate);
+    }
     const strip = (s) => s.replace(/^(type|scope|semver|breaking|modules|verification):[\s\S]*?(?=^[a-z_]+:|^---$)/gm, '');
     test('recomputes the derived fields and verification', () => {
         recordCommitted();
@@ -908,8 +964,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expectFailure(run('change-record', { refresh: true }), 'record-missing', 'run awm change new change-record to create it');
     });
     test('two records with the same slug are record-ambiguous and neither is modified', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         run('change-record', {}, { now });
         fs_1.default.copyFileSync(file(), path_1.default.join(recordsDir(), '2026-11-01-change-record.md'));
         const before = recordFiles().map((f) => fs_1.default.readFileSync(path_1.default.join(recordsDir(), f), 'utf8'));
@@ -927,8 +982,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(read()).toBe('---\n: : :\n---\n');
     });
     test('a record missing a derived key is record-invalid and untouched', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const broken = read().replace(/^semver: .*\n/m, '');
         fs_1.default.writeFileSync(file(), broken);
         repo.git('add', '-A');
@@ -938,8 +992,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(read()).toBe(broken);
     });
     test('R3.4 a refresh that would push the record over 64 KiB is record-too-large with a remedy, and the record is untouched', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         repo.git('add', '-A');
         repo.git('commit', '-q', '-m', 'docs(changes): record');
         const files = Object.fromEntries(Array.from({ length: 4000 }, (_, i) => [`vendor/package-${String(i).padStart(5, '0')}/index.js`, '1']));
@@ -950,8 +1003,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(read()).toBe(before);
     });
     test('a record already over 64 KiB on disk is record-invalid and untouched (it cannot be read for a rewrite)', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         fs_1.default.writeFileSync(file(), `${read()}\n${'x'.repeat(70 * 1024)}\n`);
         repo.git('add', '-A');
         repo.git('commit', '-q', '-m', 'docs(changes): huge');
@@ -961,8 +1013,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(read()).toBe(before);
     });
     test('a record that is not valid UTF-8 is record-invalid and untouched', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const bytes = Buffer.concat([fs_1.default.readFileSync(file()), Buffer.from([0xff, 0xfe, 0x0a])]);
         fs_1.default.writeFileSync(file(), bytes);
         repo.git('add', '-A');
@@ -972,8 +1023,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(fs_1.default.readFileSync(file()).equals(bytes)).toBe(true);
     });
     test('a record with a UTF-8 BOM is record-invalid and its bytes are untouched', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const bytes = Buffer.concat([Buffer.from([0xef, 0xbb, 0xbf]), fs_1.default.readFileSync(file())]);
         fs_1.default.writeFileSync(file(), bytes);
         repo.git('add', '-A');
@@ -983,8 +1033,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(fs_1.default.readFileSync(file()).equals(bytes)).toBe(true);
     });
     test('a record without a BOM but with non-ASCII text still refreshes and keeps those bytes', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         fs_1.default.writeFileSync(file(), fs_1.default.readFileSync(file(), 'utf8').split(types_1.PENDING_MARKER).join('Qué se probó ñandú \u{1F600}'));
         repo.git('add', '-A');
         repo.git('commit', '-q', '-m', 'docs(changes): text');
@@ -995,8 +1044,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(read().charCodeAt(0)).toBe('-'.charCodeAt(0));
     });
     test('a record larger than the record limit is record-invalid', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         fs_1.default.appendFileSync(file(), 'x'.repeat(70 * 1024));
         repo.git('add', '-A');
         repo.git('commit', '-q', '-m', 'docs(changes): big');
@@ -1004,8 +1052,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expectFailure(run('change-record', { refresh: true }), 'record-invalid');
     });
     test('a record whose derived keys are quoted is refreshed and still validates', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const quoted = read().replace(/^(type|semver) *:/gm, '"$1" :');
         fs_1.default.writeFileSync(file(), quoted.split(types_1.PENDING_MARKER).join('written by agent'));
         repo.git('add', '-A');
@@ -1043,8 +1090,7 @@ describe('awm change new --refresh (R3.11)', () => {
         expect(run('change-record', { refresh: true, base: 'main' }).code).toBe(0);
     });
     test('a derived key spelled with a YAML escape is record-invalid and the bytes are untouched', () => {
-        receiptForHead();
-        run('change-record');
+        created();
         const bytes = Buffer.from(read().replace(/^type: .*$/m, '"t\\x79pe": fix'));
         expect(bytes.toString('utf8')).toContain('"t\\x79pe": fix');
         fs_1.default.writeFileSync(file(), bytes);