agentic-workflow-manager 4.0.0 → 4.1.0

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

@@ -477,18 +477,43 @@ program.command('list [package]')
     console.log(picocolors_1.default.dim(`  awm list <pkg>  ·  awm list --all`));
     (0, prompts_1.outro)(`Run ${picocolors_1.default.green('awm add')} to install artifacts.`);
 });
-program.command('remove')
-    .description('Remove an installed skill or workflow')
+// Simetrico con `add`: si se puede instalar scripteado, se tiene que poder desinstalar
+// scripteado. `remove` era interactivo puro — sin nombre posicional, sin `--scope`, sin
+// `--yes` — asi que cualquier limpieza automatizada quedaba bloqueada, y el playbook de
+// aceptacion (CORE-17) scripteaba `awm remove dev --yes`, una invocacion que nunca
+// existio. Tercera vez en esta sesion que un playbook se escribio contra lo que la doc
+// prometia y no contra el comportamiento. Ver docs/decisions.md, D-006.
+//
+// El nombre es de BUNDLE, igual que en `add` (D-001).
+program.command('remove [name]')
+    .description('Remove an installed bundle interactively, or non-interactively with flags')
     .option('-a, --agent <agent>', `Target agent(s), comma-separated: ${providers_1.AGENT_TARGETS.join(', ')} (defaults to every enabled agent)`)
-    .action(async (options) => {
+    .option('-s, --scope <scope>', 'Scope: local or global (skips the prompt)')
+    .option('-y, --yes', 'Skip the confirmation prompt (requires a name)')
+    .action(async (name, options) => {
     (0, prompts_1.intro)(picocolors_1.default.bgCyan(picocolors_1.default.black(' AWM - Remove Artifact ')));
     const prefs = (0, config_1.getPreferences)();
+    // `--yes` sin nombre borraria lo que el usuario nunca eligio: un `rm -rf` silencioso
+    // sobre todo lo instalado. Se rechaza antes de tocar nada.
+    if (options.yes && !name) {
+        console.error(picocolors_1.default.red('--yes requires a bundle name: `awm remove <bundle> --yes`.'));
+        console.error(picocolors_1.default.dim('Without a name, removal stays interactive so you see what you are deleting.'));
+        process.exit(1);
+    }
     // Multi-agent selection (matching the add command flow). --agent skips
     // the interactive prompt and resolves the same way as add/sync/update/doctor (R12/R13).
     let targetAgents;
     if (options.agent) {
         targetAgents = resolveTargetsOrExit(prefs, options.agent);
     }
+    else if (options.yes) {
+        // `--yes` significa CERO prompts, no "cero confirmaciones". Sin esto, un
+        // `awm remove dev --yes` sin `--agent` seguia abriendo el multiselect de
+        // agentes y colgaba cualquier script — el flag prometia no-interactivo y no lo
+        // era. Sin `--agent`, el default son los agentes habilitados, igual que en
+        // `add`, `sync`, `update` y `doctor`.
+        targetAgents = [...prefs.enabledAgents];
+    }
     else {
         const agentChoice = await (0, prompts_1.multiselect)({
             message: 'From which agent(s)?',
@@ -502,16 +527,26 @@ program.command('remove')
         handleCancel(agentChoice);
         targetAgents = agentChoice;
     }
-    const scopeChoice = await (0, prompts_1.select)({
-        message: 'Scope?',
-        options: [
-            { value: 'local', label: 'Project (Local)' },
-            { value: 'global', label: 'Global' }
-        ],
-        initialValue: prefs.defaultScope
-    });
-    handleCancel(scopeChoice);
-    const scopeVal = scopeChoice;
+    let scopeVal;
+    if (options.scope) {
+        if (options.scope !== 'local' && options.scope !== 'global') {
+            console.error(picocolors_1.default.red(`Invalid scope "${options.scope}". Use: local or global.`));
+            process.exit(1);
+        }
+        scopeVal = options.scope;
+    }
+    else {
+        const scopeChoice = await (0, prompts_1.select)({
+            message: 'Scope?',
+            options: [
+                { value: 'local', label: 'Project (Local)' },
+                { value: 'global', label: 'Global' }
+            ],
+            initialValue: prefs.defaultScope
+        });
+        handleCancel(scopeChoice);
+        scopeVal = scopeChoice;
+    }
     // projectRoot explicito: `config.local` es relativo, y resolverlo contra
     // `process.cwd()` hacia que `awm remove` no encontrara nada desde un
     // subdirectorio — y que le pasara rutas RELATIVAS a fs.rmSync cuando si.
@@ -532,16 +567,43 @@ program.command('remove')
         }
         return `${icons} ${c.baseName} ${picocolors_1.default.dim(`(in: ${Array.from(locations).join(', ')})`)}`;
     });
-    const toRemove = await (0, prompts_1.multiselect)({
-        message: 'Select artifact(s) to remove',
-        options: groupedOpts,
-        required: true
-    });
-    handleCancel(toRemove);
-    const resolved = resolveSelectedArtifacts(toRemove);
+    let resolved;
+    if (name) {
+        // Los artefactos del bundle pedido que estan REALMENTE instalados en este
+        // scope. Se cruza contra `installed`: lo que no esta instalado no se puede
+        // remover, y decirlo es mejor que borrar un subconjunto en silencio.
+        const bundle = (0, bundles_1.discoverAllBundles)().find((b) => b.name === name);
+        if (!bundle) {
+            console.error(picocolors_1.default.red(`Bundle "${name}" not found in registry.`));
+            console.error(picocolors_1.default.dim('Run `awm list` to see available packages.'));
+            process.exit(1);
+        }
+        const owned = new Set([
+            ...bundle.skills.map((sk) => sk.name),
+            ...bundle.workflows,
+            ...bundle.agents,
+        ]);
+        resolved = installed.filter((a) => owned.has(a.name));
+        if (resolved.length === 0) {
+            (0, prompts_1.outro)(picocolors_1.default.yellow(`Nothing from "${name}" is installed at ${scopeVal} scope for the selected agent(s).`));
+            process.exit(0);
+        }
+    }
+    else {
+        const toRemove = await (0, prompts_1.multiselect)({
+            message: 'Select artifact(s) to remove',
+            options: groupedOpts,
+            required: true
+        });
+        handleCancel(toRemove);
+        resolved = resolveSelectedArtifacts(toRemove);
+    }
     const names = resolved.map(a => picocolors_1.default.red(a.name)).join(', ');
-    const confirmRemove = await (0, prompts_1.confirm)({ message: `Remove ${names}?` });
-    handleCancel(confirmRemove);
+    // `--yes` salta la confirmacion, nunca la seleccion: lo que se borra siempre sale
+    // de un nombre que el usuario escribio.
+    const confirmRemove = options.yes ? true : await (0, prompts_1.confirm)({ message: `Remove ${names}?` });
+    if (!options.yes)
+        handleCancel(confirmRemove);
     if (confirmRemove) {
         try {
             for (const artifact of resolved) {

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/package.json

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