agent-kitchen 0.7.0 → 0.8.0

package/README.md

@@ -59,7 +59,7 @@ npx agent-kitchen doctor                       # revisa cada pieza con su propio
 
 - **Sin terminal y sin `--yes`** (por ejemplo, desde un agente), solo muestra lo que ofrecería.
 - **Corre en el checkout principal,** no en un worktree, y desde tu terminal. Dentro de una sesión de Claude, el sandbox de `claude-repo-guard` no deja escribir `.claude/settings.json`.
-- **Al terminar,** corre el `install` de tu gestor de paquetes, revisa el diff y haz commit. O usa `--commit`.
+- **Al terminar,** si los pasos cambiaron `package.json`, corre el `install` de tu gestor de paquetes. Te queda revisar el diff y hacer commit. O usa `--commit`.
 
 **`--commit`** deja los cambios commiteados sin que lo haga un agente:
 

package/dist/commands/sync.js

@@ -11,7 +11,8 @@
  * flow's version), one line each, with the
  * tools' output only when one fails; one that fails does not stop the rest. With `--yes`
  * it does what is suggested without asking (and keeps the repo's own skills); without a
- * terminal it only shows the steps. With `--commit` it first checks the repo is clean and on
+ * terminal it only shows the steps. Without `--commit`, it installs at the end if the steps
+ * changed package.json. With `--commit` it first checks the repo is clean and on
  * its base branch, and at the end installs if package.json changed and asks what to do with
  * the changes (see git-finish.ts), also when the only change is the decisions file below;
  * the steps then show as plain lines, since a spinner before a question leaves it deaf.
@@ -33,11 +34,11 @@ import { issueFlowStatus, NAME as FLOW } from "../pieces/issue-flow.js";
 import { keepOwn, NAME as SKILLS, ownInPick, skillsStatus } from "../pieces/skills.js";
 import { agentsMdStatus, NAME as AGENTS_MD } from "../pieces/agents-md.js";
 import { NAME as SKILLS_MANAGER, skillsManagerStatus } from "../pieces/skills-manager.js";
-import { mainCheckout, packageManager } from "../repo.js";
+import { mainCheckout, packageManager, packageText } from "../repo.js";
 import { download, failedOutput, runInstall, setVerbose } from "../tools.js";
 import { commitLine, DECISIONS_LINE } from "../commit-texts.js";
 import { progressFor } from "../progress.js";
-import { stepLabel, ui } from "../texts/index.js";
+import { phrases, stepLabel, ui } from "../texts/index.js";
 /** Every piece, in the order its steps run. */
 const PIECES = [SKILLS_MANAGER, FLOW, CLOUD, GUARD, SKILLS, AGENTS_MD];
 /**
@@ -367,6 +368,7 @@ export async function sync(options, out) {
             out.stdout(noStepsMessage("none", wrote));
         return endWithoutSteps();
     }
+    const pkgBefore = packageText(root);
     const failedSteps = [];
     for (const step of chosen) {
         if (!(await runStep(root, step, progress, out)))
@@ -375,9 +377,28 @@ export async function sync(options, out) {
     const failed = failedSteps.map(stepLabel);
     if (base !== null)
         return commitChanges(root, base, commitLabels(chosen.filter((step) => !failedSteps.includes(step)), wrote), failed, progress, options, out);
-    out.stdout(`\n${ui.summary(failed)} ${ui.next(packageManager(root))}\n`);
+    const installFailed = packageText(root) !== pkgBefore ? !(await install(root, progress, out)) : false;
+    if (installFailed)
+        failed.push(phrases({ kind: "install-deps", manager: packageManager(root) }).menu);
+    out.stdout(`\n${ui.summary(failed)} ${ui.next(installFailed ? packageManager(root) : null)}\n`);
     return failed.length === 0 ? 0 : 1;
 }
+/**
+ * Runs the repo's install with the same progress as a step, and shows its output if it fails.
+ * @param root - The repo.
+ * @param progress - How progress is shown.
+ * @param out - Where to write.
+ * @returns Whether it worked.
+ */
+async function install(root, progress, out) {
+    const text = { kind: "install-deps", manager: packageManager(root) };
+    progress.start(text);
+    const installed = await runInstall(root, text.manager);
+    progress.stop(text, installed.ok);
+    if (!installed.ok)
+        out.stderr(failedOutput(installed.output));
+    return installed.ok;
+}
 /**
  * The end of `--commit`: the install if package.json changed, the changes, the question (or
  * the branch with a PR under --yes) and what git did.
@@ -399,16 +420,9 @@ async function commitChanges(root, base, done, failed, progress, options, out) {
     }
     const manager = packageManager(root);
     const packageChanged = changed.includes("package.json");
-    if (packageChanged) {
-        const text = { kind: "install-deps", manager };
-        progress.start(text);
-        const installed = await runInstall(root, manager);
-        progress.stop(text, installed.ok);
-        if (!installed.ok) {
-            out.stderr(failedOutput(installed.output));
-            out.stdout(`\n${summary} No commiteé: el install falló y los cambios quedan sin commit.\n`);
-            return 1;
-        }
+    if (packageChanged && !(await install(root, progress, out))) {
+        out.stdout(`\n${summary} No commiteé: el install falló y los cambios quedan sin commit.\n`);
+        return 1;
     }
     const paths = changedPaths(root);
     out.stdout(`\n${summary} Cambios:\n${paths.map((path) => `  ${path}`).join("\n")}\n`);

package/dist/progress.d.ts

@@ -6,7 +6,8 @@ export interface Progress {
     stop: (text: StepText, ok: boolean) => void;
 }
 /**
- * Progress as plain lines: the start, a line now and then while it runs, and the end.
+ * Progress as plain lines: the start (marked ◒, like the spinner), a line now and then while it
+ * runs, and the end (marked ✓ or ✗).
  * @param out - Where to write.
  * @param options - Whether to print while it runs, how often, and the clock.
  * @returns The progress.

package/dist/progress.js

@@ -20,7 +20,8 @@ function endLine(text, ok, ms) {
     return `${ok ? said.done : ui.failed(said.menu)} (${ui.duration(ms)})`;
 }
 /**
- * Progress as plain lines: the start, a line now and then while it runs, and the end.
+ * Progress as plain lines: the start (marked ◒, like the spinner), a line now and then while it
+ * runs, and the end (marked ✓ or ✗).
  * @param out - Where to write.
  * @param options - Whether to print while it runs, how often, and the clock.
  * @returns The progress.
@@ -33,7 +34,7 @@ export function plainProgress(out, options) {
     return {
         start: (text) => {
             started = now();
-            out.stdout(`${phrases(text).doing}\n`);
+            out.stdout(`◒ ${phrases(text).doing}\n`);
             if (options.ticks)
                 timer = setInterval(() => out.stdout(`${ui.still(ui.duration(now() - started))}\n`), options.tickMs ?? TICK_MS);
         },

package/dist/repo.d.ts

@@ -46,6 +46,13 @@ export declare function dependencyVersion(pkg: Record<string, unknown> | null, n
  * @example packageManager(root) // "pnpm"
  */
 export declare function packageManager(root: string): string;
+/**
+ * The text of the repo's package.json, to tell later whether a run changed it.
+ * @param root - The repo.
+ * @returns Its text, or null if there is none.
+ * @example packageText(root) // '{ "devDependencies": … }'
+ */
+export declare function packageText(root: string): string | null;
 /**
  * The version of a package installed in the repo's `node_modules`.
  * @param root - The repo.

package/dist/repo.js

@@ -115,6 +115,16 @@ export function packageManager(root) {
         return field;
     return LOCKFILES.find(([file]) => existsSync(join(root, file)))?.[1] ?? "npm";
 }
+/**
+ * The text of the repo's package.json, to tell later whether a run changed it.
+ * @param root - The repo.
+ * @returns Its text, or null if there is none.
+ * @example packageText(root) // '{ "devDependencies": … }'
+ */
+export function packageText(root) {
+    const file = join(root, "package.json");
+    return existsSync(file) ? readFileSync(file, "utf8") : null;
+}
 /**
  * The version of a package installed in the repo's `node_modules`.
  * @param root - The repo.

package/dist/texts/catalog.d.ts

@@ -37,8 +37,8 @@ export interface Catalog {
     suggested: (dryRun: boolean) => string;
     /** The last line when every step worked or some failed. */
     summary: (failed: string[]) => string;
-    /** What is left to do by hand after the steps, without --commit. */
-    next: (manager: string) => string;
+    /** What is left to do by hand after the steps, without --commit; with the manager if its install failed. */
+    next: (failedInstall: string | null) => string;
     /** A new piece left out because no one was asked about it. */
     undecided: (names: string[]) => string;
     /** The repo's own skills left out of a pick. */

package/dist/texts/es.js

@@ -127,7 +127,7 @@ export const es = {
     onlyOptional: (menus) => `Todo al día. Opcional, para elegir en una terminal: ${menus.join("; ")}.`,
     suggested: (dryRun) => `Pasos sugeridos${dryRun ? "" : " (para hacerlos, corre agent-kitchen apply en una terminal, o con --yes)"}:`,
     summary: (failed) => (failed.length === 0 ? "Listo." : `Terminó, pero fallaron: ${failed.join("; ")}.`),
-    next: (manager) => `Falta: corre '${manager} install', revisa el diff y haz commit.`,
+    next: (failedInstall) => `Falta: ${failedInstall === null ? "" : `corre '${failedInstall} install', `}revisa el diff y haz commit.`,
     undecided: (names) => `Pieza nueva sin decisión: ${list(names)}. Corre agent-kitchen apply en una terminal para elegir.`,
     ownSkillsKept: (names) => `Se dejan fuera las skills que el repo ya tiene como propias: ${list(names)}. Para reemplazarlas, corre agent-kitchen apply en una terminal.`,
     saved: (file, commits) => `Guardé qué piezas usa el repo en ${file}${commits ? "" : "; commitéalo"}.`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-kitchen",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "One command to install, update and check a repo's agent tooling: the issue flow and its skills, the general skills, Claude cloud sessions, claude-repo-guard and the installed-versions rule in AGENTS.md.",
   "license": "MIT",
   "type": "module",