agentic-workflow-manager 3.13.7 → 4.0.1

package/dist/src/commands/doctor.js

@@ -11,6 +11,8 @@ const picocolors_1 = __importDefault(require("picocolors"));
 const context_1 = require("../core/diagnostics/context");
 const checks_1 = require("../core/diagnostics/checks");
 const paths_1 = require("../core/paths");
+// `readPreferences`, no `getPreferences`: este comando esta documentado como
+// read-only y el segundo auto-vivifica el archivo. Ver la nota en config.ts.
 const config_1 = require("../utils/config");
 const agent_targets_1 = require("../core/agent-targets");
 function glyph(status) {
@@ -128,7 +130,7 @@ function runDoctor(opts = {}) {
     // internal-error catch further down.
     let targets;
     try {
-        const prefs = (0, config_1.getPreferences)();
+        const prefs = (0, config_1.readPreferences)();
         targets = resolveTargets({ prefs, explicit: opts.agent });
     }
     catch (err) {

package/dist/src/index.js

@@ -88,9 +88,19 @@ function resolveSelectedArtifacts(selections) {
     }
     return Array.from(result.values());
 }
+// AWM instala BUNDLES, no artefactos sueltos. Es una decision de producto, no una
+// limitacion pendiente: las skills de AWM se apoyan unas en otras (el spine de
+// development-process invoca brainstorming, writing-plans, los gates de QA...), asi que
+// una skill instalada sola casi nunca hace lo que el usuario espera. Ver
+// docs/decisions.md, D-001.
+//
+// El flag `-t, --type` existia y `add.ts` NUNCA lo leia: se aceptaba y se descartaba en
+// silencio. Peor, la doc y los playbooks de aceptacion documentaban `awm add <skill>
+// --type skill` como el camino scripteado — una invocacion que siempre fallo con
+// «Bundle "<skill>" not found». Los playbooks se escribieron contra la documentacion, no
+// contra el comportamiento. Un flag que no se lee es peor que uno ausente: promete.
 program.command('add [name]')
-    .description('Add a skill, workflow, or process interactively (or non-interactively with flags)')
-    .option('-t, --type <type>', 'Artifact type: skill, workflow, or process')
+    .description('Add a bundle (package of skills) interactively, or non-interactively with flags')
     .option('-a, --agent <agent>', `Target agent: ${providers_1.AGENT_TARGETS.join(', ')}`)
     .option('-s, --scope <scope>', 'Scope: local or global')
     .option('-m, --method <method>', 'Install method: symlink or copy')

package/dist/src/utils/config.js

@@ -4,6 +4,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.preferencesExist = preferencesExist;
+exports.readPreferences = readPreferences;
 exports.getPreferences = getPreferences;
 exports.loadPreferences = loadPreferences;
 exports.savePreferences = savePreferences;
@@ -84,6 +85,28 @@ function normalizePreferences(value, allowEnabledAgentsMigration = true) {
             JSON.stringify(source) !== JSON.stringify(enabledAgents),
     };
 }
+/**
+ * Las preferencias TAL COMO ESTAN, sin escribir nada. Devuelve los defaults en memoria
+ * cuando el archivo no existe todavia.
+ *
+ * `getPreferences` auto-vivifica: si no hay archivo, lo crea con los defaults. Eso esta
+ * bien para un comando que va a configurar la maquina, y esta mal para uno que solo
+ * mira. `awm doctor` —documentado como «Read-only dashboard. Changes nothing.»— lo
+ * llamaba, asi que en una maquina limpia el comando de inspeccion dejaba un
+ * `preferences.json` con `claude-code` como agente por defecto, que despues se lee como
+ * «el usuario configuro esto» cuando no lo hizo. Lo encontro el playbook de aceptacion,
+ * no la suite: ningun test preguntaba si un comando de lectura escribia.
+ *
+ * Se midieron los otros siete comandos de inspeccion (`list`, `preflight`, `sensors
+ * status`, `context-budget`, `backup list`, `ledger list`, `registry status`): todos
+ * limpios. `doctor` era el unico.
+ */
+function readPreferences() {
+    const loaded = loadPreferences();
+    return loaded.exists
+        ? loaded.prefs
+        : { ...DEFAULT_PREFS, enabledAgents: [...DEFAULT_PREFS.enabledAgents] };
+}
 function getPreferences() {
     const loaded = loadPreferences();
     if (!loaded.exists) {

package/dist/tests/commands/doctor-is-read-only.test.js

@@ -0,0 +1,97 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+// `awm doctor` está documentado como «Read-only dashboard of machine + project harness
+// state. **Changes nothing.**» — y escribía.
+//
+// `getPreferences()` auto-vivifica `preferences.json` con los defaults cuando el archivo
+// no existe (`config.ts`: `if (!loaded.exists) savePreferences(DEFAULT_PREFS)`). Un
+// getter con efecto de escritura escondido adentro. `doctor` lo llamaba para resolver a
+// qué agentes mirar, así que en una máquina limpia el comando de inspección dejaba un
+// `preferences.json` con `claude-code` como agente por defecto — que después se lee como
+// «el usuario configuró esto» cuando no lo hizo.
+//
+// Lo encontró el playbook de aceptación en su segundo check, no la suite: ningún test
+// preguntaba si un comando de lectura escribía. Se midieron los otros siete comandos de
+// inspección (`list`, `preflight`, `sensors status`, `context-budget`, `backup list`,
+// `ledger list`, `registry status`) y todos estaban limpios — así que acá el sitio ES la
+// clase, con medición en vez de suposición.
+const fs_1 = __importDefault(require("fs"));
+const os_1 = __importDefault(require("os"));
+const path_1 = __importDefault(require("path"));
+describe('awm doctor no escribe nada', () => {
+    let home;
+    let projectRoot;
+    let saved;
+    let writeSpy;
+    beforeEach(() => {
+        home = fs_1.default.realpathSync(fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'awm-doctor-ro-')));
+        saved = { HOME: process.env.HOME, AWM_HOME: process.env.AWM_HOME };
+        process.env.HOME = home;
+        process.env.AWM_HOME = path_1.default.join(home, '.awm');
+        projectRoot = path_1.default.join(home, 'proj');
+        fs_1.default.mkdirSync(path_1.default.join(projectRoot, '.git'), { recursive: true });
+        jest.resetModules();
+        writeSpy = jest.spyOn(process.stdout, 'write').mockImplementation(() => true);
+    });
+    afterEach(() => {
+        writeSpy.mockRestore();
+        if (saved.HOME === undefined)
+            delete process.env.HOME;
+        else
+            process.env.HOME = saved.HOME;
+        if (saved.AWM_HOME === undefined)
+            delete process.env.AWM_HOME;
+        else
+            process.env.AWM_HOME = saved.AWM_HOME;
+        fs_1.default.rmSync(home, { recursive: true, force: true });
+    });
+    /** Todo lo que exista bajo `home`, para comparar el antes y el después. */
+    const treeOf = (dir) => {
+        const out = [];
+        const walk = (sub) => {
+            for (const e of fs_1.default.readdirSync(path_1.default.join(dir, sub), { withFileTypes: true })) {
+                const rel = path_1.default.join(sub, e.name);
+                out.push(rel);
+                if (e.isDirectory())
+                    walk(rel);
+            }
+        };
+        walk('.');
+        return out.sort();
+    };
+    it('no crea AWM_HOME ni preferences.json en una máquina limpia', () => {
+        const { runDoctor } = require('../../src/commands/doctor');
+        const before = treeOf(home);
+        runDoctor({ cwd: projectRoot, json: true });
+        expect(fs_1.default.existsSync(path_1.default.join(home, '.awm', 'preferences.json'))).toBe(false);
+        expect(treeOf(home)).toEqual(before);
+    });
+    it('tampoco escribe cuando ya hay preferencias', () => {
+        // El otro camino de `getPreferences`: el archivo existe. `migrationRequired`
+        // podía reescribirlo — sigue siendo una escritura desde un comando de lectura.
+        fs_1.default.mkdirSync(path_1.default.join(home, '.awm'), { recursive: true });
+        const prefsPath = path_1.default.join(home, '.awm', 'preferences.json');
+        fs_1.default.writeFileSync(prefsPath, JSON.stringify({
+            enabledAgents: ['claude-code'], defaultAgent: 'claude-code',
+            installMethod: 'symlink', defaultScope: 'global',
+        }));
+        const before = fs_1.default.readFileSync(prefsPath, 'utf-8');
+        const beforeTree = treeOf(home);
+        const { runDoctor } = require('../../src/commands/doctor');
+        runDoctor({ cwd: projectRoot, json: true });
+        expect(fs_1.default.readFileSync(prefsPath, 'utf-8')).toBe(before);
+        expect(treeOf(home)).toEqual(beforeTree);
+    });
+    it('sigue reportando sobre el agente por defecto sin archivo de preferencias', () => {
+        // No alcanza con no escribir: tiene que seguir funcionando. Sin preferencias,
+        // los defaults en memoria son la respuesta correcta.
+        const { runDoctor } = require('../../src/commands/doctor');
+        const code = runDoctor({ cwd: projectRoot, json: true });
+        expect([0, 1]).toContain(code); // 0 sano / 1 degradado, nunca 2 (error)
+        const emitted = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        expect(JSON.parse(emitted).providers.map((p) => p.id)).toEqual(['claude-code']);
+    });
+});

package/dist/tests/commands/watch/runner.test.js

@@ -11,7 +11,8 @@ const exec_wrapper_1 = require("../../../src/commands/job/exec-wrapper");
 const store_1 = require("../../../src/core/journal/store");
 const paths_1 = require("../../../src/core/journal/paths");
 const process_1 = require("../../../src/core/journal/process");
-// Presupuesto explicito, como sus tres hermanos de `tests/commands/watch/`
+// Presupuesto explicito POR ENCIMA del global (30s en jest.config.js), como sus tres
+// hermanos de `tests/commands/watch/`
 // (integration 30s, supervisor-loop 60s, e2e-crash 180s). Este archivo era el unico de
 // la familia sin uno, y hace el mismo trabajo que ellos: cada `collectAndReconcile`
 // consulta si el proceso sigue vivo, y en Windows eso spawnea `tasklist` — cientos de ms

package/package.json

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