agentic-workflow-manager 6.2.0 → 6.2.1

package/dist/src/core/diagnostics/provider-checks.js

@@ -124,20 +124,24 @@ function skillsGlobalCheck(dir, owners, integrity, renderer, artifacts) {
         // generados NO: quedan con el contenido de la version anterior hasta que alguien
         // corra `awm sync`, y hasta ahora nada lo decia.
         const stale = staleRendered(dir, artifacts);
+        const names = stale.map((r) => path_1.default.basename(r.targetPath));
         return {
             id: 'skills.global',
             state: stale.length > 0 ? 'stale' : 'supported',
             target: dir,
             owners: owners.length > 1 ? owners : undefined,
             detail: stale.length > 0
-                ? `${stale.length} rendered file(s) no longer match the installed registry (${stale.slice(0, 3).join(', ')})`
+                ? `${stale.length} rendered file(s) no longer match the installed registry (${names.slice(0, 3).join(', ')})`
+                : undefined,
+            // El remedio depende del ALCANCE, y se midio cual funciona en cada uno:
+            // `awm update` reconcilia los artefactos de maquina, `awm sync` los que el
+            // profile del proyecto declara. Ofrecer el equivocado seria mandar al usuario
+            // a un comando que corre limpio sin cambiar nada — el defecto que ya tuvo
+            // `open-hooks-trust` (D-010), y que la primera version de ESTE chequeo
+            // repitio: ofrecia `reinstall-bundle` porque dos mediciones mias estaban mal.
+            remediationCode: stale.length > 0
+                ? (stale.some((r) => r.scope === 'local') ? 'awm-sync' : 'awm-update')
                 : undefined,
-            // `reinstall-bundle`, y NO `awm-sync` ni `awm-init`: se midio cual de los
-            // cuatro comandos refresca de verdad un renderizado viejo, y solo `awm add`
-            // lo hace. `init` y `sync` son idempotentes por presencia —ven el archivo y
-            // no lo tocan— y `update` refresca el registry, no lo derivado de el. Ofrecer
-            // un remedio que corre limpio sin cambiar nada es peor que no ofrecer ninguno.
-            remediationCode: stale.length > 0 ? 'reinstall-bundle' : undefined,
         };
     }
     const shared = owners.length > 1;
@@ -213,8 +217,7 @@ function staleRendered(dir, records) {
             // corresponden. Aca "no puedo comparar" no se convierte en "esta viejo".
             return false;
         }
-    })
-        .map((r) => path_1.default.basename(r.targetPath));
+    });
 }
 function corruptRendered(dir, files, renderer) {
     const marker = (0, registry_1.rendererIntegrityMarker)(renderer);

package/dist/src/core/profile.js

@@ -154,5 +154,19 @@ function ensureSkillsGitignored(root, agents) {
  * project extensions and stay out of `.awm/profile.json`.
  */
 function shouldRecordExtension(bundleScope, effective) {
-    return bundleScope === 'project' && effective === 'local';
+    // El criterio es el ALCANCE EFECTIVO, no el del bundle.
+    //
+    // Antes exigia ademas `bundleScope === 'project'`, asi que un bundle baseline
+    // instalado explicitamente con `--scope local` no quedaba registrado en ningun lado:
+    // `awm sync` reconcilia lo que declara el profile, y esos artefactos no estaban ahi.
+    // Resultado: nadie los refrescaba nunca. `update` cubre lo global y `sync` lo del
+    // profile; eso caia en el medio.
+    //
+    // Y no es un caso raro: `awm add dev --scope local` es exactamente lo que el playbook
+    // de aceptacion (AG-03) le pide correr a cualquiera que verifique un proveedor.
+    //
+    // Si alguien pidio artefactos de proyecto, el profile lo dice — que es lo que hace
+    // que un companero que clona el repo y corre `awm sync` obtenga lo mismo.
+    void bundleScope;
+    return effective === 'local';
 }

package/dist/src/core/update-check.js

@@ -83,7 +83,11 @@ function maybeNotifyUpdate(opts) {
     const spawnWorker = opts?.spawnWorker ?? spawnRefreshWorker;
     const cache = readUpdateCache();
     if (cache?.latest && isNewer(cache.latest, (0, cli_version_1.cliVersion)())) {
-        console.log(picocolors_1.default.dim(`\n⬆ awm v${cache.latest} available → npm i -g ${cli_version_1.CLI_PACKAGE_NAME}`));
+        // stderr, NO stdout. Este aviso se imprime al final de CUALQUIER comando, asi
+        // que en stdout se mezclaba con la salida de `--json` y rompia a cualquiera que
+        // parsee: `awm doctor --json | jq` fallaba con un SyntaxError que no menciona la
+        // causa. stdout es la interfaz de maquina; los avisos al humano van por stderr.
+        process.stderr.write(picocolors_1.default.dim(`\n⬆ awm v${cache.latest} available → npm i -g ${cli_version_1.CLI_PACKAGE_NAME}\n`));
     }
     if (!cache || now - cache.lastCheck > TTL_MS)
         spawnWorker();

package/dist/tests/core/diagnostics/provider-tier.test.js

@@ -277,9 +277,9 @@ describe('skillsGlobalCheck — renderer-aware (Task 4.4 / deferred Task 4.3 fin
             .checks.find((c) => c.id === 'skills.global');
         expect(stale?.state).toBe('stale');
         expect(stale?.detail).toContain('using-awm.mdc');
-        // Medido, no supuesto: de `init`/`update`/`sync`/`add`, solo `add` refresca un
-        // renderizado viejo. Los otros tres corren limpios y no cambian nada.
-        expect(stale?.remediationCode).toBe('reinstall-bundle');
+        // El remedio depende del alcance, y se midio: `awm update` reconcilia los
+        // artefactos de maquina; `awm sync`, los que declara el profile del proyecto.
+        expect(stale?.remediationCode).toBe('awm-update');
         fs_1.default.rmSync(source, { recursive: true, force: true });
     });
     it('non-link renderer with an empty/missing dir reports absent, not a false healthy', () => {

package/dist/tests/core/profile.test.js

@@ -100,11 +100,18 @@ describe('ensureSkillsGitignored', () => {
     });
 });
 describe('shouldRecordExtension', () => {
-    it('records only project-scope bundles installed locally', () => {
+    it('no registra nada instalado a nivel maquina, sea cual sea el bundle', () => {
         expect((0, profile_1.shouldRecordExtension)('project', 'local')).toBe(true);
         expect((0, profile_1.shouldRecordExtension)('project', 'global')).toBe(false);
         expect((0, profile_1.shouldRecordExtension)('baseline', 'global')).toBe(false);
-        expect((0, profile_1.shouldRecordExtension)('baseline', 'local')).toBe(false);
         expect((0, profile_1.shouldRecordExtension)('ambient', 'global')).toBe(false);
     });
+    it('records a BASELINE bundle too when it was installed with --scope local', () => {
+        // Antes devolvia false, y esos artefactos quedaban fuera del profile: `awm sync`
+        // reconcilia lo que el profile declara, asi que nadie los refrescaba nunca —
+        // `update` cubre lo global y esto caia en el medio. No es un caso raro:
+        // `awm add dev --scope local` es lo que el playbook AG-03 pide correr.
+        expect((0, profile_1.shouldRecordExtension)('baseline', 'local')).toBe(true);
+        expect((0, profile_1.shouldRecordExtension)('ambient', 'local')).toBe(true);
+    });
 });

package/dist/tests/core/update-check.test.js

@@ -59,12 +59,14 @@ describe('update-check', () => {
     it('maybeNotifyUpdate avisa si el cache trae versión más nueva y NO refresca cache fresco', () => {
         const m = require('../../src/core/update-check');
         m.writeUpdateCache({ lastCheck: 1_000_000, latest: '99.0.0' });
-        const log = jest.spyOn(console, 'log').mockImplementation(() => { });
+        // stderr, no stdout: el aviso sale al final de cualquier comando y en stdout
+        // rompia la salida de `--json`.
+        const err = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
         const spawnWorker = jest.fn();
         m.maybeNotifyUpdate({ now: 1_000_000 + 1000, spawnWorker });
-        expect(log.mock.calls.flat().join('\n')).toContain('99.0.0');
+        expect(err.mock.calls.flat().join('\n')).toContain('99.0.0');
         expect(spawnWorker).not.toHaveBeenCalled();
-        log.mockRestore();
+        err.mockRestore();
     });
     it('cache viejo (>24h) dispara refresh en background', () => {
         const m = require('../../src/core/update-check');
@@ -104,3 +106,32 @@ describe('update-check', () => {
         expect(cache.lastCheck).toBeGreaterThan(0);
     });
 });
+// El aviso de version nueva va por stderr, no por stdout.
+//
+// Se imprime al final de CUALQUIER comando, asi que en stdout se mezclaba con la salida
+// de `--json` y rompia a cualquiera que parsee. Encontrado tropezando con el:
+// `awm doctor --json | node -e 'JSON.parse(...)'` fallaba con un SyntaxError que no
+// menciona la causa. stdout es la interfaz de maquina.
+describe('the update banner never contaminates stdout', () => {
+    it('writes to stderr so `--json` output stays parseable', () => {
+        const stdout = jest.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        const stderr = jest.spyOn(process.stderr, 'write').mockImplementation(() => true);
+        const log = jest.spyOn(console, 'log').mockImplementation(() => { });
+        try {
+            const { maybeNotifyUpdate } = require('../../src/core/update-check');
+            maybeNotifyUpdate({ spawnWorker: () => { } });
+            const onStdout = stdout.mock.calls.map((c) => String(c[0])).join('')
+                + log.mock.calls.map((c) => String(c[0])).join('');
+            expect(onStdout).not.toContain('available');
+            // Y si habia aviso, salio por stderr — no se perdio, se movio.
+            const emitted = stderr.mock.calls.map((c) => String(c[0])).join('');
+            if (emitted.length > 0)
+                expect(emitted).toContain('available');
+        }
+        finally {
+            stdout.mockRestore();
+            stderr.mockRestore();
+            log.mockRestore();
+        }
+    });
+});

package/package.json

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