agentic-workflow-manager 5.0.1 → 5.0.2

package/dist/src/commands/hooks/codex.js

@@ -129,18 +129,24 @@ function computeCodexHookStatus(agent) {
     const settingsEntry = checkCodexSettingsEntry(config.settingsPath, config.scriptsDir, config.matcher, config.eventName);
     const allOk = sessionStartScript.ok && settingsEntry.ok;
     const settingsOnlyMissing = !settingsEntry.ok && sessionStartScript.ok;
-    let overall;
-    if (allOk)
-        overall = 'HEALTHY';
-    else if (settingsOnlyMissing)
-        overall = 'NOT_INSTALLED';
-    else
-        overall = 'DEGRADED';
     // Trust only makes sense once the AWM entry is actually present in
     // hooks.json — otherwise there's nothing installed to trust.
     const trust = settingsEntry.ok
         ? computeCodexTrust(sessionStartScript, scriptPath, heartbeatPath)
         : undefined;
+    // `overall` IGNORABA `trust` por completo, asi que `awm hooks status` imprimia
+    // `Status: HEALTHY` dos lineas debajo de `Trust: … pending-trust`. Una corrida real
+    // del playbook leyo esa salida y concluyo que el hook andaba. `doctor --json` decia
+    // la verdad; la vista humana, no — y es la que mira una persona. Ver D-010.
+    let overall;
+    if (!allOk)
+        overall = settingsOnlyMissing ? 'NOT_INSTALLED' : 'DEGRADED';
+    else if (trust === 'stale')
+        overall = 'DEGRADED';
+    else if (trust === 'pending-trust')
+        overall = 'PENDING_TRUST';
+    else
+        overall = 'HEALTHY';
     return {
         overall,
         trust,

package/dist/src/commands/hooks/index.js

@@ -13,6 +13,16 @@ const status_1 = require("./status");
 const providers_1 = require("../../providers");
 const registries_1 = require("../../core/registries");
 const HOOK_TARGETS = providers_1.AGENT_TARGETS.filter((a) => (0, providers_1.getHookConfig)(a));
+/**
+ * `hooks` nacio con `-t, --target` y el resto del CLI (`add`, `remove`, `sync`,
+ * `update`, `doctor`) usa `-a, --agent`. La asimetria se cobro una corrida real del
+ * playbook: `awm hooks status --agent codex` fallaba con `unknown option`. Se acepta
+ * `--agent` como alias en vez de renombrar, para no romper a quien ya scriptea
+ * `--target`. Ver D-010.
+ */
+function resolveHookTarget(options) {
+    return (options.agent ?? options.target ?? 'claude-code');
+}
 function targetOptionDescription() {
     return `Target harness (${HOOK_TARGETS.join('|')})`;
 }
@@ -21,9 +31,10 @@ function registerHooksCommand(program) {
     hooks.command('install')
         .description('Install the AWM bootstrap hook into the target harness')
         .option('-t, --target <target>', targetOptionDescription(), 'claude-code')
+        .option('-a, --agent <agent>', 'Alias de --target, por simetria con el resto del CLI')
         .option('-y, --yes', 'Skip interactive confirmations', false)
         .action(async (options) => {
-        const agent = (options.target ?? 'claude-code');
+        const agent = resolveHookTarget(options);
         const prefs = (0, config_1.getPreferences)();
         const hooksRoot = (0, registries_1.capabilityRoot)('hooks');
         if (!hooksRoot) {
@@ -69,9 +80,10 @@ function registerHooksCommand(program) {
     hooks.command('uninstall')
         .description('Remove the AWM bootstrap hook')
         .option('-t, --target <target>', targetOptionDescription(), 'claude-code')
+        .option('-a, --agent <agent>', 'Alias de --target, por simetria con el resto del CLI')
         .option('-y, --yes', 'Skip interactive confirmations', false)
         .action(async (options) => {
-        const agent = (options.target ?? 'claude-code');
+        const agent = resolveHookTarget(options);
         if (!options.yes && process.stdin.isTTY) {
             const ok = await (0, prompts_1.confirm)({ message: 'Remove AWM bootstrap hook from settings.json?' });
             if ((0, prompts_1.isCancel)(ok) || ok !== true) {
@@ -98,8 +110,9 @@ function registerHooksCommand(program) {
     hooks.command('status')
         .description('Check the bootstrap hook installation status')
         .option('-t, --target <target>', targetOptionDescription(), 'claude-code')
+        .option('-a, --agent <agent>', 'Alias de --target, por simetria con el resto del CLI')
         .action((options) => {
-        const agent = (options.target ?? 'claude-code');
+        const agent = resolveHookTarget(options);
         try {
             const result = (0, status_1.computeHookStatus)(agent);
             const symbol = (ok) => ok ? picocolors_1.default.green('✓') : picocolors_1.default.red('✗');
@@ -118,10 +131,17 @@ function registerHooksCommand(program) {
             }
             console.log('');
             const overall = result.overall === 'HEALTHY' ? picocolors_1.default.green(result.overall) :
-                result.overall === 'NOT_INSTALLED' ? picocolors_1.default.yellow(result.overall) :
-                    picocolors_1.default.red(result.overall);
+                result.overall === 'NOT_INSTALLED' || result.overall === 'PENDING_TRUST'
+                    ? picocolors_1.default.yellow(result.overall)
+                    : picocolors_1.default.red(result.overall);
             console.log(`  Status: ${overall}`);
-            if (result.overall !== 'HEALTHY') {
+            if (result.overall === 'PENDING_TRUST') {
+                console.log(picocolors_1.default.dim('  El hook esta instalado y bien formado, pero nunca se lo vio correr.'));
+                console.log(picocolors_1.default.dim('  Abri una sesion del agente; si sigue igual, no se esta disparando.'));
+            }
+            // PENDING_TRUST no sale 1: no hay nada roto que arreglar, y salir 1 en
+            // toda instalacion recien hecha convertiria el comando en ruido.
+            if (result.overall !== 'HEALTHY' && result.overall !== 'PENDING_TRUST') {
                 process.exit(1);
             }
         }

package/dist/tests/commands/hooks/codex.test.js

@@ -167,6 +167,27 @@ describe('installHook / computeHookStatus / uninstallHook — Codex adapter', ()
         const { computeHookStatus } = require('../../../src/commands/hooks/status');
         expect(computeHookStatus('codex').trust).toBe(expected); // verifies R18
     });
+    // `overall` NUNCA se assertaba — solo `.trust` — y por eso el bug vivio: se calculaba
+    // ignorando trust por completo, asi que `awm hooks status` imprimia `Status: HEALTHY`
+    // dos lineas debajo de `Trust: … pending-trust`. Una corrida real del playbook leyo esa
+    // salida y concluyo que el hook andaba. Ver D-010.
+    it.each([
+        [false, 'PENDING_TRUST'],
+        [true, 'HEALTHY'],
+    ])('overall refleja el trust, no solo la presencia de archivos (heartbeat=%s)', (heartbeat, expected) => {
+        installCodexFixture({ heartbeat: heartbeat });
+        const { computeHookStatus } = require('../../../src/commands/hooks/status');
+        expect(computeHookStatus('codex').overall).toBe(expected);
+    });
+    it('un heartbeat obsoleto degrada: el script cambio desde la ultima corrida confirmada', () => {
+        installCodexFixture({ heartbeat: true });
+        fs_1.default.writeFileSync(path_1.default.join(codexScriptsDir, 'session-start'), '#!/usr/bin/env bash\necho "changed"', { mode: 0o755 });
+        const { computeHookStatus } = require('../../../src/commands/hooks/status');
+        const status = computeHookStatus('codex');
+        expect(status.trust).toBe('stale');
+        // Esto era `HEALTHY`: la rotura mas clara de las tres, reportada en verde.
+        expect(status.overall).toBe('DEGRADED');
+    });
     it('reports stale trust when the heartbeat hash no longer matches the installed script', () => {
         installCodexFixture({ heartbeat: true });
         fs_1.default.writeFileSync(path_1.default.join(codexScriptsDir, 'session-start'), '#!/usr/bin/env bash\necho "changed"', { mode: 0o755 });

package/package.json

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