agentic-workflow-manager 8.1.3 → 8.1.4

package/dist/src/release/index.js

@@ -40,6 +40,7 @@ function parseArgs(argv) {
 }
 function realIO(repoRoot, cliDir) {
     const pkgPath = path_1.default.join(cliDir, 'package.json');
+    const lockPath = path_1.default.join(cliDir, 'package-lock.json');
     const changelogPath = path_1.default.join(repoRoot, 'CHANGELOG.md');
     const npmrcPath = path_1.default.join(cliDir, '.npmrc');
     return {
@@ -52,6 +53,18 @@ function realIO(repoRoot, cliDir) {
             pkg.version = v;
             fs_1.default.writeFileSync(pkgPath, JSON.stringify(pkg, null, 2) + '\n');
         },
+        // El round-trip JSON.parse/stringify preserva el orden de claves y el formato que
+        // npm escribe (2 espacios + newline final), asi que el diff de un bump son las dos
+        // lineas de version y nada mas — verificado sobre el lockfile real de este repo.
+        writeLockVersion: (v) => {
+            if (!fs_1.default.existsSync(lockPath))
+                return;
+            const lock = JSON.parse(fs_1.default.readFileSync(lockPath, 'utf8'));
+            lock.version = v;
+            if (lock.packages?.[''])
+                lock.packages[''].version = v;
+            fs_1.default.writeFileSync(lockPath, JSON.stringify(lock, null, 2) + '\n');
+        },
         readChangelog: () => (fs_1.default.existsSync(changelogPath) ? fs_1.default.readFileSync(changelogPath, 'utf8') : ''),
         writeChangelog: (c) => fs_1.default.writeFileSync(changelogPath, c),
         writeNpmrc: (token) => fs_1.default.writeFileSync(npmrcPath, `//registry.npmjs.org/:_authToken=${token}\n`),

package/dist/src/release/orchestrator.js

@@ -65,9 +65,10 @@ function release(opts, io) {
     }
     // aplicar
     io.writePackageVersion(version);
+    io.writeLockVersion(version);
     const section = (0, core_1.renderChangelog)(version, io.today(), commits);
     io.writeChangelog(section + '\n' + io.readChangelog());
-    io.run('git', ['add', 'cli/package.json', 'CHANGELOG.md']);
+    io.run('git', ['add', 'cli/package.json', 'cli/package-lock.json', 'CHANGELOG.md']);
     io.run('git', ['commit', '-m', `chore(release): v${version} [skip ci]`]);
     io.run('git', ['tag', '-a', `v${version}`, '-m', `v${version}`]);
     // OIDC mode: GitHub inyecta ACTIONS_ID_TOKEN_REQUEST_URL con id-token:write

package/dist/tests/release/orchestrator.test.js

@@ -6,6 +6,10 @@ const core_1 = require("../../src/release/core");
 function makeIO(over = {}) {
     const calls = [];
     let pkgVersion = '2.1.1';
+    // El lockfile arranca en la MISMA version que el package.json, como en el repo real.
+    // Si el release no lo sincroniza, este valor se queda atras — que es exactamente el
+    // estado que dejaba `main` en rojo (#92).
+    let lockVersion = '2.1.1';
     const io = {
         run(cmd, args) {
             const full = `${cmd} ${args.join(' ')}`;
@@ -26,6 +30,7 @@ function makeIO(over = {}) {
         },
         readPackageVersion: () => pkgVersion,
         writePackageVersion: (v) => { pkgVersion = v; calls.push(`WRITE_PKG ${v}`); },
+        writeLockVersion: (v) => { lockVersion = v; calls.push(`WRITE_LOCK ${v}`); },
         readChangelog: () => '',
         writeChangelog: (c) => calls.push(`WRITE_CHANGELOG ${c.split('\n')[0]}`),
         writeNpmrc: () => calls.push('WRITE_NPMRC'),
@@ -35,7 +40,7 @@ function makeIO(over = {}) {
         env: { NPM_TOKEN: 'tok' },
         ...over,
     };
-    return { io, calls };
+    return { io, calls, versions: () => ({ pkg: pkgVersion, lock: lockVersion }) };
 }
 const opts = (o = {}) => ({ dryRun: false, force: null, push: true, branch: 'main', cliDir: '/cli', ...o });
 describe('release — happy path', () => {
@@ -54,6 +59,25 @@ describe('release — happy path', () => {
         expect(calls).toContain('WRITE_NPMRC');
         expect(calls).toContain('REMOVE_NPMRC');
     });
+    // #92: el bump escribia solo package.json, asi que el lockfile quedaba una version
+    // atras en CADA release. `r3-cli-major-version.test.ts` exige que coincidan, y el
+    // `[skip ci]` del commit de bump hacia que ese rojo no apareciera hasta el PR
+    // siguiente — que llegaba roto por algo que no habia hecho.
+    it('sincroniza package-lock.json con package.json y lo incluye en el commit de bump', () => {
+        const { io, calls, versions } = makeIO({ commits: `feat: x${core_1.US}${core_1.RS}` });
+        (0, orchestrator_1.release)(opts(), io);
+        expect(versions()).toEqual({ pkg: '2.2.0', lock: '2.2.0' });
+        expect(calls).toContain('WRITE_LOCK 2.2.0');
+        // Escribir el archivo no alcanza: si no se stagea, el commit de release lo deja
+        // fuera y el lockfile queda sucio en el working tree del runner.
+        expect(calls).toContain('git add cli/package.json cli/package-lock.json CHANGELOG.md');
+    });
+    it('no toca el lockfile cuando no hay nada que publicar', () => {
+        const { io, calls, versions } = makeIO({ commits: `docs: solo docs${core_1.US}${core_1.RS}` });
+        (0, orchestrator_1.release)(opts(), io);
+        expect(versions()).toEqual({ pkg: '2.1.1', lock: '2.1.1' });
+        expect(calls.some((c) => c.startsWith('WRITE_LOCK'))).toBe(false);
+    });
     it('sin commits releasables → no publica (exit 0 lógico)', () => {
         const { io, calls } = makeIO({ commits: `docs: solo docs${core_1.US}${core_1.RS}` });
         const res = (0, orchestrator_1.release)(opts(), io);

package/package.json

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