jorgex-stack 1.0.30 → 1.0.31

package/dist/cli.js

@@ -5,7 +5,7 @@ import * as p6 from "@clack/prompts";
 import { pathToFileURL as pathToFileURL2 } from "url";
 
 // src/install.ts
-import fs14 from "fs";
+import fs13 from "fs";
 import path19 from "path";
 import * as p from "@clack/prompts";
 
@@ -546,8 +546,6 @@ var opencodeAdapter = {
 ${deny}`);
       }
       if (!agent.spawn) lines.push("  task: deny");
-      lines.push("tools:");
-      lines.push(`  write: ${agent.readonly ? "false" : "true"}`);
     }
     return [
       {
@@ -803,16 +801,7 @@ description: ${yamlString2(agent.description)}
 keep-coding-instructions: true
 ---
 ${agent.body}`;
-      const skillDescription = `${agent.description} Invoke to switch into ${agent.name} mode and apply its flow to the current task.`;
-      const skill = `---
-name: ${agent.name}
-description: ${yamlString2(skillDescription)}
----
-${agent.body}`;
-      return [
-        { file: `${agent.name}.md`, content: style, kind: "output-style" },
-        { file: `${agent.name}/SKILL.md`, content: skill, kind: "skill" }
-      ];
+      return [{ file: `${agent.name}.md`, content: style, kind: "output-style" }];
     }
     const lines = [`name: ${agent.name}`, `description: ${yamlString2(agent.description)}`];
     const tools = toolsFor(agent);
@@ -995,16 +984,7 @@ var codexAdapter = {
         `# Uso: codex --profile ${agent.name}`,
         `developer_instructions = ${tomlMultiline(agent.body)}`
       ];
-      const skillDescription = `${agent.description} Invoke to switch into ${agent.name} mode and apply its flow to the current task.`;
-      const skill = `---
-name: ${agent.name}
-description: ${JSON.stringify(skillDescription)}
----
-${agent.body}`;
-      return [
-        { file: `${agent.name}.config.toml`, content: profileLines.join("\n") + "\n", kind: "profile" },
-        { file: `${agent.name}/SKILL.md`, content: skill, kind: "skill" }
-      ];
+      return [{ file: `${agent.name}.config.toml`, content: profileLines.join("\n") + "\n", kind: "profile" }];
     }
     const lines = [
       `name = ${tomlString(agent.name)}`,
@@ -1379,13 +1359,6 @@ import path13 from "path";
 var PROGRAMMATIC_ROOT = ["modes", "programmatic"];
 var PROGRAMMATIC_MARKER = "<!-- jorgex:programmatic-mode -->";
 var LEGACY_RESULT_CONTRACT_SECTION = /\n?##\s+Result contract[\s\S]*$/;
-var LEGACY_SKILL_DELEGATION_SECTION = /\n?##\s+Formato obligatorio[\s\S]*$/;
-var LEGACY_DELEGATION_LINE = /- For each `→ \[agent\]: \.\.\.` line, launch the corresponding specialist\./;
-var LEGACY_PROGRAMMATIC_PHRASES = [
-  [/\bResult contract\b/g, "strict JSON handoff"],
-  [/Status \/ Delegations \/ Risks/g, "status, delegations, and risks"],
-  [LEGACY_DELEGATION_LINE, "- Process the JSON `delegations[]` array and launch the corresponding specialist."]
-];
 var normalize = (value) => value.replace(/\r\n/g, "\n");
 function loadProgrammaticAddendum(stackDir, fileName) {
   return normalize(fs9.readFileSync(path13.join(stackDir, ...PROGRAMMATIC_ROOT, fileName), "utf8")).trim();
@@ -1399,10 +1372,7 @@ ${addendum}
 `;
 }
 function stripLegacyResultContract(body) {
-  return LEGACY_PROGRAMMATIC_PHRASES.reduce(
-    (text2, [pattern, replacement]) => text2.replace(pattern, replacement),
-    normalize(body).replace(LEGACY_RESULT_CONTRACT_SECTION, "")
-  ).trimEnd();
+  return normalize(body).replace(LEGACY_RESULT_CONTRACT_SECTION, "").replace(/\bResult contract\b/g, "strict JSON handoff").trimEnd();
 }
 function concurrencyRule(concurrency) {
   if (concurrency === "parallel") {
@@ -1422,22 +1392,13 @@ function composeProgrammaticSystemPrompt(stackDir, content, mode) {
 }
 function composeProgrammaticAgentBody(stackDir, agent, mode, concurrency) {
   if (mode !== "programmatic") return normalize(agent.body);
-  const fileName = agent.mode === "primary" ? "orchestrator.addendum.md" : "subagent.addendum.md";
-  let addendum = loadProgrammaticAddendum(stackDir, fileName);
   if (agent.mode === "primary") {
-    addendum = addendum.replace("{{CONCURRENCY_RULE}}", concurrencyRule(concurrency ?? "serial"));
+    const addendum2 = loadProgrammaticAddendum(stackDir, "orchestrator.addendum.md").replace("{{CONCURRENCY_RULE}}", concurrencyRule(concurrency ?? "serial"));
+    return appendAddendum(agent.body, addendum2);
   }
+  const addendum = loadProgrammaticAddendum(stackDir, "subagent.addendum.md");
   return appendAddendum(stripLegacyResultContract(agent.body), addendum);
 }
-function composeProgrammaticSkillBody(stackDir, skillPath, content, mode) {
-  if (mode !== "programmatic") return normalize(content);
-  if (skillPath !== path13.join("agent-delegation", "SKILL.md")) return normalize(content);
-  const base = normalize(content).replace(
-    "Las delegaciones van **en tu output final**, en el formato de abajo. El orquestador las lee y decide a qui\xE9n invocar.",
-    "Las delegaciones van como strings en el JSON final `delegations[]`. El orquestador las lee y decide a qui\xE9n invocar."
-  ).replace(LEGACY_SKILL_DELEGATION_SECTION, "").trimEnd();
-  return appendAddendum(base, loadProgrammaticAddendum(stackDir, "agent-delegation.addendum.md"));
-}
 
 // src/components/system-prompt.ts
 var normalize2 = (s) => s.replace(/\r\n/g, "\n");
@@ -1461,12 +1422,11 @@ function planSystemPrompt(adapter, ctx) {
 // src/components/agents.ts
 import path15 from "path";
 function planAgents(adapter, ctx) {
-  const { agentsDir, commandsDir, outputStylesDir, skillsDir, profilesDir, scriptsDir } = adapter.paths(ctx.configDir);
+  const { agentsDir, commandsDir, outputStylesDir, profilesDir, scriptsDir } = adapter.paths(ctx.configDir);
   const dirFor = {
     agent: agentsDir,
     command: commandsDir,
     "output-style": outputStylesDir,
-    skill: skillsDir,
     profile: profilesDir
   };
   const scriptsBase = scriptsDir.replace(/\\/g, "/");
@@ -1489,43 +1449,35 @@ function planAgents(adapter, ctx) {
 
 // src/components/skills.ts
 import path16 from "path";
-import fs11 from "fs";
 function planSkills(adapter, ctx) {
   const { skillsDir } = adapter.paths(ctx.configDir);
   const source = path16.join(ctx.stackDir, "skills");
   return listFilesRecursive(source).map((file) => {
     const relative = path16.relative(source, file);
-    if (relative === path16.join("agent-delegation", "SKILL.md") && ctx.mode === "programmatic") {
-      return {
-        kind: "write",
-        target: path16.join(skillsDir, relative),
-        content: composeProgrammaticSkillBody(ctx.stackDir, relative, fs11.readFileSync(file, "utf8"), ctx.mode)
-      };
-    }
     return { kind: "copy", source: file, target: path16.join(skillsDir, relative) };
   });
 }
 
 // src/components/commands.ts
 import path17 from "path";
-import fs12 from "fs";
+import fs11 from "fs";
 function planCommands(adapter, ctx) {
   const { commandsDir } = adapter.paths(ctx.configDir);
   const source = path17.join(ctx.stackDir, "commands");
-  if (!fs12.existsSync(source)) return [];
+  if (!fs11.existsSync(source)) return [];
   const commandFiles = [
     ...listMarkdownFiles(source),
     ...listMarkdownFiles(path17.join(source, adapter.id))
   ];
   return commandFiles.map(({ file, fullPath }) => {
-    const raw = fs12.readFileSync(fullPath, "utf8").replace(/\r\n/g, "\n");
+    const raw = fs11.readFileSync(fullPath, "utf8").replace(/\r\n/g, "\n");
     const rendered = adapter.renderCommand(file, raw);
     return { kind: "write", target: path17.join(commandsDir, rendered.file), content: rendered.content };
   });
 }
 function listMarkdownFiles(dir) {
-  if (!fs12.existsSync(dir)) return [];
-  return fs12.readdirSync(dir, { withFileTypes: true }).filter((entry) => entry.isFile() && entry.name.endsWith(".md")).map((entry) => ({ file: entry.name, fullPath: path17.join(dir, entry.name) }));
+  if (!fs11.existsSync(dir)) return [];
+  return fs11.readdirSync(dir, { withFileTypes: true }).filter((entry) => entry.isFile() && entry.name.endsWith(".md")).map((entry) => ({ file: entry.name, fullPath: path17.join(dir, entry.name) }));
 }
 
 // src/components/hooks.ts
@@ -1540,22 +1492,22 @@ function planMcp(adapter, ctx) {
 
 // src/components/plugins.ts
 import path18 from "path";
-import fs13 from "fs";
+import fs12 from "fs";
 function planPlugins(adapter, ctx) {
   const { pluginsDir } = adapter.paths(ctx.configDir);
   if (pluginsDir === null) return [];
   const source = path18.join(ctx.stackDir, "plugins", adapter.id);
-  if (!fs13.existsSync(source)) return [];
+  if (!fs12.existsSync(source)) return [];
   return listFilesRecursive(source).filter((f) => f.endsWith(".ts")).map((sourceFile) => {
     const target = path18.join(pluginsDir, path18.relative(source, sourceFile));
-    const raw = fs13.readFileSync(sourceFile, "utf8");
+    const raw = fs12.readFileSync(sourceFile, "utf8");
     let content = raw;
     if (content.includes('"{{ENGRAM_BIN}}"')) {
       content = content.replace(/"\{\{ENGRAM_BIN\}\}"/g, JSON.stringify(ctx.engramBin ?? "engram"));
     }
     if (content.includes('"{{ENGRAM_PROTOCOL}}"')) {
       const protocol = stripLeadingHtmlComments(
-        fs13.readFileSync(path18.join(ctx.stackDir, "system-prompt", "engram-protocol.md"), "utf8").replace(/\r\n/g, "\n")
+        fs12.readFileSync(path18.join(ctx.stackDir, "system-prompt", "engram-protocol.md"), "utf8").replace(/\r\n/g, "\n")
       );
       content = content.replace(/"\{\{ENGRAM_PROTOCOL\}\}"/g, JSON.stringify(protocol));
     }
@@ -1602,7 +1554,7 @@ function diffPlan(plan) {
       if (current === null) return { action, status: "create" };
       return { action, status: current === action.content ? "unchanged" : "update" };
     }
-    if (!fs14.existsSync(action.target)) return { action, status: "create" };
+    if (!fs13.existsSync(action.target)) return { action, status: "create" };
     return { action, status: sameFileContent(action.source, action.target) ? "unchanged" : "update" };
   });
 }
@@ -1734,7 +1686,7 @@ async function runInstall(opts) {
     applyChanges(changes);
     const pruneRoot = useManifest ? HOME : path19.dirname(configDir);
     for (const orphan of orphans) {
-      fs14.rmSync(orphan, { force: true });
+      fs13.rmSync(orphan, { force: true });
       pruneEmptyDirs(orphan, pruneRoot);
     }
     const verifyCtx = { ...ctx, warnings: [] };
@@ -1757,7 +1709,7 @@ async function runInstall(opts) {
 }
 
 // src/uninstall.ts
-import fs15 from "fs";
+import fs14 from "fs";
 import path20 from "path";
 import * as p2 from "@clack/prompts";
 async function runUninstall(opts) {
@@ -1812,7 +1764,7 @@ async function runUninstall(opts) {
     const pruneRoot = usingRealConfig ? HOME : path20.dirname(configDir);
     const planTargets = [
       .../* @__PURE__ */ new Set([...buildPlan(adapter, ctx).map((a) => path20.resolve(a.target)), ...prevOwned.map((t) => path20.resolve(t))])
-    ].filter((t) => !mergedTargets.has(t) && fs15.existsSync(t));
+    ].filter((t) => !mergedTargets.has(t) && fs14.existsSync(t));
     const deleteTargets = planTargets.filter(
       (t) => !retained.has(t) && !(ctx.preserveEngram && path20.basename(t) === "engram.ts") && isContainedIn(t, pruneRoot)
     );
@@ -1822,18 +1774,18 @@ async function runUninstall(opts) {
     if (sharedKept > 0) p2.log.info(`${sharedKept} archivos se conservan: otros runtimes instalados los siguen usando.`);
     if (opts.dryRun) continue;
     const backup = createBackup(
-      [...deleteTargets, ...unmerge.map((a) => a.target).filter((t) => fs15.existsSync(t))],
+      [...deleteTargets, ...unmerge.map((a) => a.target).filter((t) => fs14.existsSync(t))],
       `uninstall-${id}`
     );
     if (backup) p2.log.info(`Backup: ${backup.id} (${backup.files.length} archivos)`);
     for (const target of deleteTargets) {
-      fs15.rmSync(target, { force: true });
+      fs14.rmSync(target, { force: true });
       pruneEmptyDirs(target, pruneRoot);
     }
     for (const action of unmerge) {
       if (action.kind !== "write") continue;
       if (action.content.trim() === "") {
-        fs15.rmSync(action.target, { force: true });
+        fs14.rmSync(action.target, { force: true });
       } else {
         writeText(action.target, action.content);
       }
@@ -1847,7 +1799,7 @@ async function runUninstall(opts) {
 
 // src/doctor.ts
 import path21 from "path";
-import fs16 from "fs";
+import fs15 from "fs";
 import * as p3 from "@clack/prompts";
 function engramVersion(bin) {
   const out = runDetectedBin(bin, ["--version"], 5e3);
@@ -1880,11 +1832,11 @@ async function runDoctor() {
   }
   const engramDataDir = process.env.ENGRAM_DATA_DIR ?? path21.join(HOME, ".engram");
   const engramDb = path21.join(engramDataDir, "engram.db");
-  if (fs16.existsSync(engramDb)) {
-    const sizeMb = (fs16.statSync(engramDb).size / 1024 / 1024).toFixed(1);
+  if (fs15.existsSync(engramDb)) {
+    const sizeMb = (fs15.statSync(engramDb).size / 1024 / 1024).toFixed(1);
     p3.log.info(`Engram DB: ${engramDb} (${sizeMb} MB de memorias \u2014 el stack no la toca JAM\xC1S).`);
   }
-  if (!fs16.existsSync(modelMapFile())) p3.log.info("model-map: a\xFAn no creado (se crea en el primer install o con 'models').");
+  if (!fs15.existsSync(modelMapFile())) p3.log.info("model-map: a\xFAn no creado (se crea en el primer install o con 'models').");
   const manifest = readManifest();
   const modePreference = loadInstallModePreference();
   const current = collectAllCurrentTargets(modePreference);
@@ -1923,10 +1875,10 @@ async function runDoctor() {
       p3.log.warn(`${adapter.name}: ${orphans.length} archivos hu\xE9rfanos de versiones previas \u2192 ejecuta 'sync'.`);
       problems++;
     }
-    if (adapter.id === "codex" && fs16.existsSync(path21.join(detection.configDir, "hooks.json"))) {
+    if (adapter.id === "codex" && fs15.existsSync(path21.join(detection.configDir, "hooks.json"))) {
       p3.log.info("Codex: recuerda que los hooks requieren aprobaci\xF3n manual \u2014 verifica con /hooks dentro de codex.");
     }
-    if (adapter.id === "codex" && fs16.existsSync(path21.join(detection.configDir, "AGENTS.override.md"))) {
+    if (adapter.id === "codex" && fs15.existsSync(path21.join(detection.configDir, "AGENTS.override.md"))) {
       p3.log.warn(
         "Codex: existe ~/.codex/AGENTS.override.md \u2014 tiene prioridad ABSOLUTA y tapa el AGENTS.md gestionado por el stack."
       );
@@ -1940,14 +1892,14 @@ async function runDoctor() {
 }
 
 // src/update.ts
-import fs19 from "fs";
+import fs18 from "fs";
 import path24 from "path";
 import os3 from "os";
 import { execFileSync as execFileSync4 } from "child_process";
 import * as p4 from "@clack/prompts";
 
 // src/lib/github.ts
-import fs17 from "fs";
+import fs16 from "fs";
 import path22 from "path";
 import { execFileSync as execFileSync2 } from "child_process";
 import os2 from "os";
@@ -2022,7 +1974,7 @@ function validateExtractedTree(destDir) {
   const walk = (dir) => {
     let entries;
     try {
-      entries = fs17.readdirSync(dir, { withFileTypes: true });
+      entries = fs16.readdirSync(dir, { withFileTypes: true });
     } catch {
       return false;
     }
@@ -2030,7 +1982,7 @@ function validateExtractedTree(destDir) {
       const full = path22.join(dir, entry.name);
       let stat;
       try {
-        stat = fs17.lstatSync(full);
+        stat = fs16.lstatSync(full);
       } catch {
         return false;
       }
@@ -2047,14 +1999,14 @@ function validateExtractedTree(destDir) {
 function resolveTarBin() {
   if (process.platform !== "win32") return "tar";
   const winTar = path22.join(process.env["SystemRoot"] ?? "C:\\Windows", "System32", "tar.exe");
-  return fs17.existsSync(winTar) ? winTar : "tar";
+  return fs16.existsSync(winTar) ? winTar : "tar";
 }
 async function downloadRepoTarball(repo, sha, destDir, validateSubdir) {
   const url = `https://codeload.github.com/${repo}/tar.gz/${sha}`;
   const tmp = path22.join(os2.tmpdir(), `jorgex-tarball-${Date.now()}.tar.gz`);
   const fail = (reason) => {
     try {
-      fs17.rmSync(destDir, { recursive: true, force: true });
+      fs16.rmSync(destDir, { recursive: true, force: true });
     } catch {
     }
     return { ok: false, reason };
@@ -2073,10 +2025,10 @@ async function downloadRepoTarball(repo, sha, destDir, validateSubdir) {
     if (!res.body) return fail("respuesta HTTP sin cuerpo");
     await pipeline(
       Readable.fromWeb(res.body),
-      fs17.createWriteStream(tmp)
+      fs16.createWriteStream(tmp)
     );
-    fs17.rmSync(destDir, { recursive: true, force: true });
-    fs17.mkdirSync(destDir, { recursive: true });
+    fs16.rmSync(destDir, { recursive: true, force: true });
+    fs16.mkdirSync(destDir, { recursive: true });
     try {
       execFileSync2(resolveTarBin(), ["-xzf", tmp, "--strip-components=1", "-C", destDir], { stdio: "pipe" });
     } catch (err) {
@@ -2089,30 +2041,30 @@ async function downloadRepoTarball(repo, sha, destDir, validateSubdir) {
     if (validateRoot !== resolvedDest && !isContainedIn(validateRoot, resolvedDest)) {
       return fail(`la ruta de validaci\xF3n "${validateSubdir}" escapa del destino`);
     }
-    if (fs17.existsSync(validateRoot) && !validateExtractedTree(validateRoot)) {
+    if (fs16.existsSync(validateRoot) && !validateExtractedTree(validateRoot)) {
       return fail("el \xE1rbol extra\xEDdo contiene symlinks o rutas fuera del destino");
     }
-    const validated = fs17.existsSync(validateRoot);
+    const validated = fs16.existsSync(validateRoot);
     return { ok: true, validated };
   } catch (err) {
     const timedOut = err instanceof Error && (err.name === "TimeoutError" || err.name === "AbortError");
     return fail(timedOut ? "timeout de descarga (120s)" : err instanceof Error ? `fallo de red: ${err.message}` : "error desconocido");
   } finally {
     try {
-      fs17.rmSync(tmp, { force: true });
+      fs16.rmSync(tmp, { force: true });
     } catch {
     }
   }
 }
 
 // src/lib/skill-update.ts
-import fs18 from "fs";
+import fs17 from "fs";
 import path23 from "path";
 import { execFileSync as execFileSync3 } from "child_process";
 var PROTECTED_SKILLS = /* @__PURE__ */ new Set(["agent-delegation", "work-lifecycle", "xreview"]);
 function sameTextContentNormalized(a, b) {
-  const ba = fs18.readFileSync(a);
-  const bb = fs18.readFileSync(b);
+  const ba = fs17.readFileSync(a);
+  const bb = fs17.readFileSync(b);
   if (ba.equals(bb)) return true;
   const sa = ba.toString("utf8").replace(/\r\n/g, "\n");
   const sb = bb.toString("utf8").replace(/\r\n/g, "\n");
@@ -2194,7 +2146,7 @@ function replaceSkill(name, upstreamSkillDir, newCommit, opts) {
     throw new Error(`La skill "${name}" es propia del stack y no se actualiza desde upstream.`);
   }
   const upstreamsFile = upstreamsFilePath ?? path23.join(path23.dirname(stackRoot()), "upstreams.json");
-  const raw = fs18.readFileSync(upstreamsFile, "utf8");
+  const raw = fs17.readFileSync(upstreamsFile, "utf8");
   const data = JSON.parse(raw);
   const skillEntry = data?.skills?.[name];
   if (!skillEntry) {
@@ -2213,7 +2165,7 @@ function replaceSkill(name, upstreamSkillDir, newCommit, opts) {
   try {
     const upstreamFiles = listFilesRecursive(upstreamSkillDir);
     for (const src of upstreamFiles) {
-      const st = fs18.lstatSync(src);
+      const st = fs17.lstatSync(src);
       if (st.isSymbolicLink()) {
         throw new Error(`Symlink rechazado en upstream de skill "${name}": ${src}`);
       }
@@ -2223,14 +2175,14 @@ function replaceSkill(name, upstreamSkillDir, newCommit, opts) {
       copyFile(src, dest);
     }
     const oldDir = `${localSkillDir}.old-${process.pid}`;
-    if (fs18.existsSync(localSkillDir)) {
-      fs18.renameSync(localSkillDir, oldDir);
+    if (fs17.existsSync(localSkillDir)) {
+      fs17.renameSync(localSkillDir, oldDir);
     }
-    fs18.renameSync(stagingDir, localSkillDir);
-    fs18.rmSync(oldDir, { recursive: true, force: true });
+    fs17.renameSync(stagingDir, localSkillDir);
+    fs17.rmSync(oldDir, { recursive: true, force: true });
   } catch (err) {
     try {
-      fs18.rmSync(stagingDir, { recursive: true, force: true });
+      fs17.rmSync(stagingDir, { recursive: true, force: true });
     } catch {
     }
     throw err;
@@ -2245,7 +2197,7 @@ function rateLimitHint(prefix) {
 }
 function loadUpstreams() {
   const file = path24.join(path24.dirname(stackRoot()), "upstreams.json");
-  return JSON.parse(fs19.readFileSync(file, "utf8"));
+  return JSON.parse(fs18.readFileSync(file, "utf8"));
 }
 function skillsToScan(maintainer, upstreams) {
   return maintainer ? Object.keys(upstreams.skills) : [];
@@ -2358,7 +2310,7 @@ function isEngramRunning() {
   }
 }
 function isGitClone(projectRoot = path24.dirname(stackRoot())) {
-  return fs19.existsSync(path24.join(projectRoot, ".git"));
+  return fs18.existsSync(path24.join(projectRoot, ".git"));
 }
 var STACK_METHOD_CLONE = "git pull + pnpm install + pnpm build";
 function resolvePnpm() {
@@ -2368,7 +2320,7 @@ function resolvePnpm() {
 }
 function cleanupTmp(dir) {
   try {
-    fs19.rmSync(dir, { recursive: true, force: true });
+    fs18.rmSync(dir, { recursive: true, force: true });
   } catch {
   }
 }
@@ -2390,7 +2342,7 @@ function updateStackGlobal() {
   execFileSync4(pnpm, ["add", "-g", "jorgex-stack@latest"], { stdio: "inherit" });
 }
 async function downloadSkillToTemp(repo, sha, skillPath) {
-  const root = fs19.mkdtempSync(path24.join(os3.tmpdir(), "jorgex-skill-"));
+  const root = fs18.mkdtempSync(path24.join(os3.tmpdir(), "jorgex-skill-"));
   try {
     const result = await downloadRepoTarball(repo, sha, root, skillPath);
     if (!result.ok) {
@@ -2403,10 +2355,10 @@ async function downloadSkillToTemp(repo, sha, skillPath) {
         cleanupTmp(root);
         return { error: `la ruta "${skillPath}" escapa del directorio temporal` };
       }
-      if (fs19.existsSync(sub)) return { dir: sub, root };
+      if (fs18.existsSync(sub)) return { dir: sub, root };
       const lastSeg = skillPath.split("/").pop();
       const sub2 = path24.resolve(path24.join(root, lastSeg));
-      if (isContainedIn(sub2, root) && fs19.existsSync(sub2)) {
+      if (isContainedIn(sub2, root) && fs18.existsSync(sub2)) {
         if (!validateExtractedTree(sub2)) {
           cleanupTmp(root);
           return { error: `el sub\xE1rbol "${lastSeg}" contiene symlinks o rutas fuera del destino` };
@@ -2425,11 +2377,11 @@ async function downloadSkillToTemp(repo, sha, skillPath) {
 function pruneEngramDbBackups() {
   try {
     const dir = dataDir();
-    if (!fs19.existsSync(dir)) return;
-    const backups = fs19.readdirSync(dir).filter((f) => f.startsWith("engram-db-backup-") && f.endsWith(".db")).map((f) => ({ name: f, mtime: fs19.statSync(path24.join(dir, f)).mtimeMs })).sort((a, b) => b.mtime - a.mtime);
+    if (!fs18.existsSync(dir)) return;
+    const backups = fs18.readdirSync(dir).filter((f) => f.startsWith("engram-db-backup-") && f.endsWith(".db")).map((f) => ({ name: f, mtime: fs18.statSync(path24.join(dir, f)).mtimeMs })).sort((a, b) => b.mtime - a.mtime);
     for (const old of backups.slice(3)) {
       try {
-        fs19.rmSync(path24.join(dir, old.name));
+        fs18.rmSync(path24.join(dir, old.name));
       } catch {
       }
     }
@@ -2437,7 +2389,7 @@ function pruneEngramDbBackups() {
   }
 }
 function rotateLockedBinary(binPath, sweepRoot = HOME) {
-  if (!fs19.existsSync(binPath)) return null;
+  if (!fs18.existsSync(binPath)) return null;
   const dir = path24.dirname(binPath);
   const base = path24.basename(binPath);
   const escapedBase = base.replace(/[.*+?^$()|[\]{}\\]/g, "\\$&");
@@ -2445,10 +2397,10 @@ function rotateLockedBinary(binPath, sweepRoot = HOME) {
   const resolvedDir = path24.resolve(dir);
   if (resolvedDir === path24.resolve(sweepRoot) || isContainedIn(resolvedDir, sweepRoot)) {
     try {
-      for (const entry of fs19.readdirSync(dir)) {
+      for (const entry of fs18.readdirSync(dir)) {
         if (oldPattern.test(entry)) {
           try {
-            fs19.rmSync(path24.join(dir, entry), { force: true });
+            fs18.rmSync(path24.join(dir, entry), { force: true });
           } catch {
           }
         }
@@ -2457,7 +2409,7 @@ function rotateLockedBinary(binPath, sweepRoot = HOME) {
     }
   }
   const rotated = path24.join(dir, `${base}.old-${Date.now()}`);
-  fs19.renameSync(binPath, rotated);
+  fs18.renameSync(binPath, rotated);
   return rotated;
 }
 async function updateEngram(engramRepo, latestVersion) {
@@ -2468,7 +2420,7 @@ async function updateEngram(engramRepo, latestVersion) {
   }
   const engramDataDir = process.env.ENGRAM_DATA_DIR ?? path24.join(HOME, ".engram");
   const engramDb = path24.join(engramDataDir, "engram.db");
-  if (fs19.existsSync(engramDb)) {
+  if (fs18.existsSync(engramDb)) {
     const doBackup = await p4.confirm({
       message: `\xBFHacer backup de la DB de Engram antes de actualizar? (${engramDb})`,
       initialValue: true
@@ -2477,8 +2429,8 @@ async function updateEngram(engramRepo, latestVersion) {
       const ts = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
       const dest = path24.join(dataDir(), `engram-db-backup-${ts}.db`);
       try {
-        if (!fs19.existsSync(dataDir())) fs19.mkdirSync(dataDir(), { recursive: true });
-        fs19.copyFileSync(engramDb, dest);
+        if (!fs18.existsSync(dataDir())) fs18.mkdirSync(dataDir(), { recursive: true });
+        fs18.copyFileSync(engramDb, dest);
         p4.log.success(`DB respaldada en ${dest} (la DB original NO se modifica jam\xE1s).`);
         pruneEngramDbBackups();
       } catch (err) {
@@ -2528,10 +2480,10 @@ async function updateEngram(engramRepo, latestVersion) {
         ["install", `github.com/Gentleman-Programming/engram/cmd/engram@v${latestVersion}`],
         { stdio: "inherit" }
       );
-      const rollbackOk = resolveEngramRollback({ installOk: true, rotated, bin, binExists: fs19.existsSync(bin ?? "") });
+      const rollbackOk = resolveEngramRollback({ installOk: true, rotated, bin, binExists: fs18.existsSync(bin ?? "") });
       if (rollbackOk.action === "restore") {
         try {
-          fs19.renameSync(rotated, bin);
+          fs18.renameSync(rotated, bin);
           p4.log.warn(rollbackOk.messages.onRestore);
         } catch {
           p4.log.warn(rollbackOk.messages.onRenameFail);
@@ -2539,10 +2491,10 @@ async function updateEngram(engramRepo, latestVersion) {
       }
       return true;
     } catch (err) {
-      const rollbackFail = resolveEngramRollback({ installOk: false, rotated, bin, binExists: fs19.existsSync(bin ?? "") });
+      const rollbackFail = resolveEngramRollback({ installOk: false, rotated, bin, binExists: fs18.existsSync(bin ?? "") });
       if (rollbackFail.action === "restore") {
         try {
-          fs19.renameSync(rotated, bin);
+          fs18.renameSync(rotated, bin);
           p4.log.info(rollbackFail.messages.onRestore);
         } catch {
           p4.log.error(rollbackFail.messages.onRenameFail);
@@ -3063,7 +3015,7 @@ function cancelled() {
 }
 
 // src/lib/release.ts
-import fs20 from "fs";
+import fs19 from "fs";
 import path26 from "path";
 import { execFileSync as execFileSync5 } from "child_process";
 import { fileURLToPath as fileURLToPath2 } from "url";
@@ -3071,7 +3023,7 @@ function findPackageJson() {
   let dir = path26.dirname(fileURLToPath2(import.meta.url));
   for (let i = 0; i < 6; i++) {
     const candidate = path26.join(dir, "package.json");
-    if (fs20.existsSync(candidate)) return candidate;
+    if (fs19.existsSync(candidate)) return candidate;
     dir = path26.dirname(dir);
   }
   throw new Error("No se encontr\xF3 package.json cerca del CLI.");
@@ -3081,7 +3033,7 @@ function readPackageVersion() {
 }
 function readPackageMetadata() {
   const packageJson = findPackageJson();
-  const raw = fs20.readFileSync(packageJson, "utf8");
+  const raw = fs19.readFileSync(packageJson, "utf8");
   const parsed = JSON.parse(raw);
   const name = typeof parsed.name === "string" ? parsed.name.trim() : "";
   const version = typeof parsed.version === "string" ? parsed.version.trim() : "";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jorgex-stack",
-  "version": "1.0.30",
+  "version": "1.0.31",
   "description": "Harness multi-agente portable: instala la config JorgeX (agentes, skills, hooks, Engram, MCPs) en Claude Code, Codex CLI y OpenCode",
   "type": "module",
   "license": "MIT",

package/stack/agents/README.md

@@ -1,6 +1,6 @@
 # Agentes canónicos
 
-Una sola fuente por agente. El instalador los traduce al formato de cada runtime (PRD §6): Markdown+frontmatter para Claude Code y OpenCode, TOML para Codex.
+Una sola fuente por agente. El instalador los traduce al formato de cada runtime (PRD §6): Markdown+frontmatter para Claude Code y OpenCode, TOML para Codex. El workflow completo del orchestrator vive únicamente en `skills/orchestrator/SKILL.md`; el agente primary es un wrapper corto que obliga a cargar esa skill.
 
 ## Frontmatter canónico
 
@@ -19,7 +19,7 @@ Una sola fuente por agente. El instalador los traduce al formato de cada runtime
 | Canónico | Claude Code | Codex | OpenCode |
 |---|---|---|---|
 | `tier` | alias `fable`/`opus`/`sonnet`/`haiku` según model-map | `model` + `model_reasoning_effort` | `provider/model` del model-map |
-| `readonly: true` | `tools: Read, Grep, Glob` (+Bash si aplica) | `sandbox_mode = "read-only"` | `tools: { write: false }`, `permission: { edit: deny }` |
+| `readonly: true` | `tools: Read, Grep, Glob` (+Bash si aplica) | `sandbox_mode = "read-only"` | `permission: { edit: deny }` |
 | `bash: git-read` | `Bash` completo en tools — la restricción a git read es solo de prompt (el frontmatter de Claude Code no tiene esa granularidad) | con `readonly: true` el sandbox read-only impide escrituras; con `readonly: false` la restricción es solo de prompt | `permission.bash: { "git diff*": allow, "git log*": allow }` (única aplicación real) |
 | `bash: none` | sin `Bash` en tools | sandbox read-only | `permission: { bash: deny }` |
 | `spawn: false` | sin tool `Agent`/`Task` | n/a | `permission: { task: deny }` |

package/stack/agents/orchestrator.md

@@ -9,224 +9,4 @@ bash: full
 
 # Orchestrator
 
-You coordinate the work. You think, design, split and delegate according to the **Delegation map**. Don't stay implementing yourself except for minimal documentation or coordination tasks.
-
-## Phases
-
-```text
-INIT → EXPLORE → SPEC → PLAN → EXECUTE → VERIFY → SHIP → CLOSE
-```
-
-### Autonomy
-
-The human drives the flow UP TO the plan: the idea, the PRD review and the plan review are interactive. Once the plan is approved, EXECUTE → VERIFY → SHIP run **autonomously** — no confirmation pauses: plan approval authorizes commits, pushes to the work branch, draft PR creation, final review, and the draft-to-ready transition after verification. Task-critical uncertainty from a subagent is an operational blocker, not a pause in autonomy: answer from existing context first; only if the decision genuinely cannot be made from available context may you ask the user, then relaunch with explicit guidance. Control returns to the user at CLOSE. Merging the PR is NEVER yours: it always requires an explicit user order. For multi-PR work, each merge is a checkpoint; keep `work/{name}/PRD.md` and `plan.md` alive until the roadmap is finished. Dependent PRs are sequential: after a checkpoint merge, update the production branch and create the next worktree/branch from that updated base.
-
-## 1. INIT
-
-- Load previous context from Engram memory: for non-trivial reads, delegate to the `engram` subagent (`mem_context` / `mem_search` filtered to the task).
-- Identify the project's constraints.
-- Detect whether there is documentation, issues or artifacts already created.
-
-## 2. EXPLORE
-
-Launch analysts according to scope:
-
-- `backend-analyst` if it affects backend, DB, APIs or server functions
-- `frontend-analyst` if it affects UI, hooks, state or rendering
-- `security-auditor` if the area is sensitive
-
-## Base rule
-
-- Your priority is to delegate.
-- If a task has a clear subagent scope, delegate.
-- If previous context is needed, gather context or analyze before deciding implementation.
-
-### Delegation triggers
-
-Once a task crosses any of these thresholds, delegating stops being optional:
-
-| Trigger | Expected behavior |
-| --- | --- |
-| Reading 4+ files just to understand a flow | Delegate exploration to the matching analyst. |
-| Wrong cwd, git/worktree accident, confusing test or env failure | Stop; re-explore with fresh context before continuing. |
-| Long session with accumulating complexity | Pause and re-plan or delegate — or state explicitly why not. |
-
-The goal is not ceremony: it is one responsible coordinator, one writer per scope, deterministic feedback while the diff is evolving, and fresh eyes at the PR boundary.
-
-## 3. SPEC
-
-- Synthesize findings.
-- Propose a simple approach.
-- Clarify only the real ambiguities.
-- Apply the `lean-code` skill as a scope gate for any code-bearing task: ask whether the code is needed at all, whether stdlib/native/project helpers already solve it, and whether the smallest obvious change is enough.
-- Backlog items phrased as "consider/evaluate X" are questions, not requirements: answer them HERE — who consumes it, what real case needs it — before they enter the PRD as committed scope. A contract nobody consumes is born dead; drop it or defer it explicitly instead of inheriting it as a fact.
-- Create the PRD before moving to PLAN (see PRD rules).
-
-### PRD rules
-
-The PRD is **mandatory by default** when you work as orchestrator. If you were invoked, the work is non-trivial (several layers, several files or coordination) and deserves a spec before executing. The PRD captures decisions before implementing and leaves traceability towards the tasks.
-
-Use the `to-prd` skill to turn the current context into the PRD (`work/{name}/PRD.md`) before planning execution.
-
-**Escape valve (measurable)**: skip the PRD only if one of these applies:
-
-- the user explicitly asks to skip it, or
-- ALL of these hold: the change touches ≤ 3 files, AND stays in a single layer (only backend, only frontend, only docs…), AND changes no public contract (API, schema, exported types consumed elsewhere). In that case, consider returning the work to the normal flow instead of orchestrating.
-
-If you skip it, say so explicitly and state which condition applied.
-
-When presenting the PRD for review, offer a disposable HTML view (rules in the `work-lifecycle` skill).
-
-If the work is large enough to benefit from explicit vertical slices, use the `to-issues` skill after the PRD to split it into independently executable slices before detailed planning.
-
-## 4. PLAN
-
-- Use the PRD as the base input for planning (it normally exists; only absent if the escape valve was used).
-- If a slice breakdown exists from `to-issues`, use it as the structure for planning and task sequencing.
-- Divide the work into clear tasks.
-- One task = one agent = one scope.
-- For tasks that add or grow code, record the lean-code outcome in the task spec/acceptance criteria so implementer and simplifier apply the same ladder.
-- The PRD does not replace the plan or task breakdown: the PRD captures decisions; the plan and tasks turn those decisions into executable work.
-- Materialize the plan per the Work state rules: `work/{name}/plan.md` with the task table, plus one `mem_save` per task with its full self-contained spec (templates in the `work-lifecycle` skill).
-- When presenting the plan for review, offer a disposable HTML view (rules in the `work-lifecycle` skill). Requested changes go to plan.md; delete the HTML once the plan is approved, before EXECUTE.
-
-## Work state
-
-The `work-lifecycle` skill is the single source of this flow. Summary — every piece has exactly ONE home:
-
-- `work/{name}/` (gitignored, exists only while the work is in progress) holds the human-reviewed artifacts: `PRD.md` and `plan.md`. They stay resident across intermediate PR merges; `plan.md` is the ONLY task status board — flip statuses with surgical edits; don't re-read the whole plan after every task (re-read it on resume).
-- The full spec of each atomic task → Engram, one `mem_save` per task under `work/{name}/task/{NN}`. When you delegate a task, pass the subagent its topic_key + title — never the task content inline; it retrieves the spec itself.
-- Phase outcomes, decisions and PR checkpoints → Engram under `work/{name}/{phase}` and `work/{name}/pr/{NN}`; tell each subagent which topic_key to use for its saves.
-- Pending work → the project's single `work/backlog` topic_key, or issues (`to-issues`) if the project uses a tracker. Never a TODOs folder. For Engram, you are the **single writer**: before every change, retrieve the exact observation with `mem_get_observation`, preserve unrelated entries, send the complete content with `mem_update`, then read it again to verify. Never write it concurrently or use a blind topic-key upsert. Do not split it into per-item memories until Engram supports complete paginated topic-prefix listing.
-- On final close: `mem_save` the outcome under `work/{name}/done`, move the PRD to the project's docs only if it has lasting documentation value, then delete `work/{name}/`. `work/{name}/done` is only for the last PR / final outcome. History is memory + git.
-
-## Delegation map
-
-Load the `agent-delegation` skill: it defines the available subagents, the scope of each and when to delegate. It is the single source of the agent map — don't duplicate the list here.
-
-Every subagent ends with a **Result contract** (Status / Delegations / Risks). Process it:
-
-- For each `→ [agent]: ...` line, launch the corresponding specialist.
-- A delegation is unfinished work in another scope, not a request to append a generic quality pipeline. Normal handoffs between `implementer` and `tester` do not by themselves justify reviewers or analyzers.
-- If a subagent reports `partial`, keep the safe work and relaunch only what still needs guidance.
-- If a subagent reports `blocked` with one concrete uncertainty question, answer it from existing context when possible; if it still cannot be resolved, ask the user only if genuinely necessary, then relaunch the original or a suitable specialist with explicit guidance.
-- Don't declare a phase done while a delegation line remains unprocessed.
-- If Status is `partial` or `blocked`, resolve the cause before moving on.
-
-## 5. EXECUTE
-
-### Worktree
-
-Before the first task, create a git worktree for this work and run the ENTIRE execution inside it — implementation, tests, commits and pushes happen there, never on the user's main checkout.
-
-Canonical location is mandatory: resolve the project root with `git rev-parse --show-toplevel`, ensure `worktrees/` is ignored in the repo-local `.git/info/exclude`, create `worktrees/` inside that root if needed, and create the worktree at `<project-root>/worktrees/<canonical-name>` for single-PR work or `<project-root>/worktrees/<canonical-name>-prNN` for multi-PR checkpoints (branch = worktree name). Do not create worktrees next to the repo, in the repo root, under `work/`, or in any external temp/shared folder.
-
-Every delegation prompt must state the worktree path as the ONLY allowed write root. After each writer subagent finishes, verify the user's main checkout is still clean (`git status` there); if the subagent wrote outside the worktree, STOP, move those changes into the worktree (patch/apply) and restore the main checkout before continuing. Subagent obedience is not a safety boundary — this check is.
-
-### Commit cadence
-
-Commit after each task or bounded group of tasks, with a message that reflects that task — the branch history must map to the plan. Never accumulate the whole work into one giant commit at the end.
-
-### Draft PR cadence
-
-- After the first coherent commit, push the branch and create the PR against its real base with `gh pr create --draft`. Do not wait until SHIP to open it.
-- Keep every code change, commit and push inside the draft phase. The PR remains draft until the code, applicable version bump, local tests, project quality command (`pnpm qa:quality` when defined), Vercel preview when applicable, final diff, and full review are complete.
-- Never push to a ready PR. If a ready PR needs changes, first run `gh pr ready --undo <number>`, then modify and push while draft and repeat VERIFY and the final review before readying it again.
-
-### Handoff rule
-
-The analyst's **Recommendation** is the implementer's input. Sequence: analyst (map + design) → you turn it into tasks → `implementer`/`tester` execute. Don't launch `implementer` on an area no analyst has mapped unless the design is already clear from existing context.
-
-### Testing decision
-
-Every implementation task needs a testing decision, not automatically a new test. Establish:
-
-- the meaningful regression risk introduced by the change
-- the existing test that already protects it, if any
-- the new or changed behavior that needs protection
-- the strongest seam closest to that risk
-- the action: TDD/new test, update, reuse existing coverage, or no new test with a concrete trivial/mechanical/already-covered reason
-
-Prefer one authoritative test per behavior. Another layer is justified only when it protects a distinct contract. The task spec carries this decision so `tester` and `implementer` do not invent different strategies.
-
-### TDD mode
-
-Use for business rules, bugs/regressions, public contracts, invariants, security/data boundaries, or other behavior whose risk warrants new protection.
-
-```text
-tester (RED) → implementer (GREEN/REFACTOR)
-```
-
-### Direct mode
-
-Use for styling, wiring, generated code, mechanical refactors, trivial code, or changes already covered by an authoritative test. Direct mode still runs the cheapest sufficient verification and records why no new test was needed.
-
-```text
-implementer (direct change)
-```
-
-### Special delegations
-
-- `translator` for translations or multilingual visible text
-- `docs-maintainer` for documentation
-- `security-auditor` for sensitive review
-
-### Verification cadence
-
-Deterministic checks are the routine feedback loop while implementation is in progress: run the relevant tests, lint and typecheck/build checks at the cheapest seam that can catch the section's regressions. Verify by bounded, coherent sections (e.g. when a wave completes), not after every small change — and don't defer everything to a single big-bang check at the end either.
-
-Each writer verifies its own bounded area (e.g. its test file). The orchestrator runs shared checks such as the global typecheck once when the wave closes, never concurrently or repeatedly through several writers. Reserve the full suite for VERIFY unless a wave changed broad cross-cutting behavior and an earlier run has a concrete benefit.
-
-### Early-review budget
-
-An early review during EXECUTE is an **exception**, not a default phase. Use it only when there is a concrete risk that deterministic checks cannot cover and the feedback can materially change the remaining implementation. Typical candidates are a sensitive authorization boundary, a destructive migration, subtle concurrency/state consistency, or a broad public contract change.
-
-- State the exact risk and the bounded diff section to inspect before launching anyone.
-- Use the single most relevant specialist. Do not load the `xreview` skill or run a generic multi-agent panel during EXECUTE.
-- Run at most one early review per bounded critical section, after that section is coherent rather than after each task inside it.
-- Do not launch `code-reviewer`, `code-simplifier`, `test-analyzer` or `silent-failure-hunter` merely because a writer finished, a test task completed, several files changed or a commit is due.
-- File count, writer completion, commit, push, or draft PR creation are not early-review triggers. The review boundary is the final candidate SHA while the PR is still draft, immediately before `gh pr ready` in SHIP.
-
-## 6. VERIFY
-
-- Validate against the plan's **Success criteria** in plan.md and tick the ones that pass. Tests passing is NOT enough: a criterion left unmet means the work is not done, even with a green suite.
-- Run the minimum verification that is sufficient.
-- Reserve heavy suites for cases where they provide real value or the project requires them.
-- Before SHIP, ensure all applicable preflight work is complete: code, version bump, local tests, the project's quality command (`pnpm qa:quality` when defined), and Vercel preview review when the project uses Vercel. React Doctor is manual/local, never assumed to be a GitHub Actions gate.
-- If something fails, go back to EXECUTE with fix tasks.
-- **Anti-thrashing**: max 3 attempts per failing task or criterion. If the third attempt still fails, STOP retrying — document what was tried and why it fails (save it under the work's topic_key), then re-plan the task with a different approach or stop and report the blocker. A hard blocker is the one legitimate reason to interrupt the autonomous run; retrying blindly is never one.
-
-## 7. SHIP (automatic)
-
-When the plan is fully applied and VERIFY passes:
-
-1. Confirm the draft PR exists, the worktree is clean, and the draft head matches the local HEAD. Inspect the final diff against the PR's real base.
-2. Load and run the portable `xreview` skill against that final diff while the PR is still draft. This is the one multi-agent review per PR and the definitive review boundary; draft PR creation is not. Process the report by its three levels:
-   - **Critical Issues (must fix)**: apply ALL of them — the PR must not reach merge with these open.
-   - **Important Improvements (should fix)**: apply the ones worth doing now, at your judgment.
-   - **Suggestions (nice to have)**: apply only if trivial and safe.
-3. Every finding you decide NOT to apply now goes to the project's `work/backlog` single topic_key — one line each: what + why deferred. Apply the safe serialized backlog protocol above; subagents only return candidate lines.
-4. For what you DO apply: add the new tasks to plan.md and one `mem_save` per task spec, execute them as in EXECUTE, re-verify, and push the fixes while the PR remains draft. Re-run the `xreview` skill only if the fixes materially changed the reviewed diff or introduced a materially different risk; ordinary finding fixes need deterministic re-verification, not another panel.
-5. Once code, verification, preview, final diff, and review are complete, record the candidate SHA and mark the PR ready exactly once with `gh pr ready <number>`.
-6. Determine whether the project has PR checks configured by inspecting project configuration such as workflows, rulesets or integrations. If the project has PR checks configured, wait for the complete Quality Gates, run `gh pr checks <number>`, and verify they pass for the recorded candidate SHA. If no PR checks are configured, confirm and record their absence; it does not block the merge. An empty `gh pr checks` result immediately after ready is not evidence that no checks are configured. In either case, do not push while the PR is ready. Immediately before reporting or merging, compare `gh pr view --json headRefOid` with the recorded candidate SHA.
-7. If any fix is needed, run `gh pr ready --undo <number>` before editing, return to EXECUTE, and repeat the full verification, review, ready, and — when configured — gate cycle. Never treat checks from an older SHA as merge evidence.
-
-## 8. CLOSE
-
-- STOP here and hand control back to the user only after configured Quality Gates pass for the latest commit, or after confirming that the project has no PR checks configured: report the candidate SHA, check result or confirmed absence, review findings applied vs deferred to `work/backlog`, and whether manual testing is advisable (recommend it for big or user-facing changes; small well-tested changes may not need it).
-- NEVER merge the PR yourself — merge only on an explicit user order. After each intermediate merge: persist the checkpoint to `work/{name}/pr/{NN}`, update `plan.md`, and keep `work/{name}/` alive. After the final merge: persist the final outcome to memory, clean up `work/{name}/` and remove the worktree (see Work state).
-- If the repo has its own skill for the closing steps (release, deploy, git, cleanup), that skill takes precedence over the default behavior.
-
-## Task rule
-
-A task must correspond to a single agent and a single scope. Don't mix production, tests, docs and translations in the same task.
-
-## Operational rules
-
-- The coordinator must not mix scopes in a single task.
-- Read-only agents can run in parallel.
-- Write agents only run in parallel if they don't touch the same files.
-
-## Closing rule
-
-Don't declare the task finished if you have only analyzed or planned. There must be real execution by the subagents or a concrete blocker.
+**Mandatory first action**: Load and follow the `orchestrator` skill. It is the single source of truth for the complete workflow. Do not reconstruct or duplicate that flow from this wrapper.

package/stack/agents/test-analyzer.md

@@ -14,7 +14,8 @@ You determine whether the diff has sufficient evidence for its meaningful regres
 **First actions, in order**:
 
 1. **Get the diff.** When given BASE and HEAD, review only `git diff <BASE>...HEAD` using exactly those branches—never assume `main`. Otherwise review the working diff (`git diff`).
-2. Load the `agent-delegation` skill.
+2. Load the `tdd` skill. Use TDD as the canonical testing policy and an analysis rubric only—never run its writer workflow or RED/GREEN loop.
+3. Load the `agent-delegation` skill.
 
 **Final output, last of all**: save memory before the final report. The report ending with the Result contract must be the last thing you emit.
 
@@ -26,22 +27,18 @@ You are read-only. Analyze testing decisions and recommend what to test, reuse,
 
 Focus on behavioral coverage rather than line coverage.
 
-1. Map each changed behavior to a meaningful regression risk, prioritizing external contracts, critical branches, and data/security boundaries.
-2. Identify the existing test that already protects it, if any.
-3. Decide whether proposed coverage adds a distinct contract or repeats the same behavior at another layer.
-4. Evaluate refactor resistance, determinism, accidental `test.only`/exclusive-focus slips, stable UI semantics, negative test cases, and async/concurrency behavior only where relevant to the diff.
-5. Report only actionable gaps, naming the regression, existing test considered, proposed seam, and criticality.
+Apply the risk, existing-protection, behavior, seam, and non-duplication rules from `tdd`, then:
 
-Prefer one authoritative test at the strongest seam closest to the risk. Persistence, SQL, RLS, migrations, and data-transaction atomicity need real database evidence when those are the risks; other concurrency or atomicity must run at its actual boundary. A regex over SQL text or an “integration” suite that mocks every important collaborator is not sufficient boundary evidence.
-
-Styling, decorative DOM, wiring, aliases, wrappers, generated code, function existence, internal call choreography, and mechanical refactors do not need new tests without a meaningful behavior change. Authentication, authorization, tenant separation, billing, privacy, destructive operations, idempotency, public endpoints, privileged functions, complex calculations/dates, accessibility, and real regressions deserve strong evidence at their actual boundary.
+1. Compare each changed behavior with the actual evidence in existing or changed tests.
+2. Evaluate refactor resistance, determinism, accidental `test.only`/exclusive-focus slips, stable UI semantics, negative cases, and async/concurrency behavior only where relevant to the diff.
+3. Report an actionable gap only when the existing evidence cannot catch a meaningful regression. Name that failure, the test considered, the proposed seam, and its criticality.
+4. Separately flag brittle, redundant, nondeterministic, or implementation-coupled tests worth fixing or removing.
 
 ## Rating guidelines
 
-- **9-10**: Data loss, security issue, or system failure
-- **7-8**: Important business logic or substantial user-facing failure
-- **5-6**: Concrete user-facing or operational regression with moderate impact
-- **1-4**: Do not report as a missing-test finding; mention only a brittle or redundant existing test worth removing
+- **8–10 — Critical**: Data loss, security issue, system failure, or substantial business/user failure without sufficient evidence
+- **5–7 — Important**: Concrete user-facing, business, or operational regression with moderate impact
+- **1–4**: Not a missing-test finding; mention only a brittle or redundant existing test worth removing
 
 ## Output format
 

package/stack/skills/agent-delegation/SKILL.md

@@ -15,7 +15,7 @@ Importante sobre el mecanismo:
 
 - Tú (subagente) **no lanzas a otros subagentes**. Solo el agente principal (orquestador) puede invocarlos.
 - No te salgas de tu scope para "ayudar". Si algo no te corresponde, lo dejas sin hacer y lo delegas.
-- Las delegaciones van **en tu output final**, en el formato de abajo. El orquestador las lee y decide a quién invocar.
+- Las delegaciones van en el contrato de resultado activo del agente. El orquestador las lee y decide a quién invocar.
 - `delegations` are only for work that belongs to another specialist; uncertainty questions are not delegations.
 
 ## Agentes disponibles
@@ -39,9 +39,9 @@ Importante sobre el mecanismo:
 
 Nota: `test-analyzer` analiza cobertura pero no escribe tests; escribir los tests recomendados es de `tester`.
 
-## Formato obligatorio
+## Fallback humano
 
-Una línea por delegación, al final de tu output:
+Si el contrato activo no define un campo o formato de delegaciones, usa una línea por delegación al final de tu output:
 
 ```markdown
 → [agente]: [trabajo pendiente] — [archivos/rutas] — [inputs mínimos]

package/stack/skills/orchestrator/SKILL.md

@@ -0,0 +1,228 @@
+---
+name: orchestrator
+description: Main coordinator for non-trivial tasks. Designs, plans and delegates to specialized subagents. Use it when the work spans several layers, several files or requires coordination. Invoke to switch into orchestrator mode and apply its flow to the current task.
+---
+
+# Orchestrator
+
+You coordinate the work. You think, design, split and delegate according to the **Delegation map**. Don't stay implementing yourself except for minimal documentation or coordination tasks.
+
+## Phases
+
+```text
+INIT → EXPLORE → SPEC → PLAN → EXECUTE → VERIFY → SHIP → CLOSE
+```
+
+### Autonomy
+
+The human drives the flow UP TO the plan: the idea, the PRD review and the plan review are interactive. Once the plan is approved, EXECUTE → VERIFY → SHIP run **autonomously** — no confirmation pauses: plan approval authorizes commits, pushes to the work branch, draft PR creation, final review, and the draft-to-ready transition after verification. Task-critical uncertainty from a subagent is an operational blocker, not a pause in autonomy: answer from existing context first; only if the decision genuinely cannot be made from available context may you ask the user, then relaunch with explicit guidance. Control returns to the user at CLOSE. Merging the PR is NEVER yours: it always requires an explicit user order. For multi-PR work, each merge is a checkpoint; keep `work/{name}/PRD.md` and `plan.md` alive until the roadmap is finished. Dependent PRs are sequential: after a checkpoint merge, update the production branch and create the next worktree/branch from that updated base.
+
+## 1. INIT
+
+- Load previous context from Engram memory: for non-trivial reads, delegate to the `engram` subagent (`mem_context` / `mem_search` filtered to the task).
+- Identify the project's constraints.
+- Detect whether there is documentation, issues or artifacts already created.
+
+## 2. EXPLORE
+
+Launch analysts according to scope:
+
+- `backend-analyst` if it affects backend, DB, APIs or server functions
+- `frontend-analyst` if it affects UI, hooks, state or rendering
+- `security-auditor` if the area is sensitive
+
+## Base rule
+
+- Your priority is to delegate.
+- If a task has a clear subagent scope, delegate.
+- If previous context is needed, gather context or analyze before deciding implementation.
+
+### Delegation triggers
+
+Once a task crosses any of these thresholds, delegating stops being optional:
+
+| Trigger | Expected behavior |
+| --- | --- |
+| Reading 4+ files just to understand a flow | Delegate exploration to the matching analyst. |
+| Wrong cwd, git/worktree accident, confusing test or env failure | Stop; re-explore with fresh context before continuing. |
+| Long session with accumulating complexity | Pause and re-plan or delegate — or state explicitly why not. |
+
+The goal is not ceremony: it is one responsible coordinator, one writer per scope, deterministic feedback while the diff is evolving, and fresh eyes at the PR boundary.
+
+## 3. SPEC
+
+- Synthesize findings.
+- Propose a simple approach.
+- Clarify only the real ambiguities.
+- Apply the `lean-code` skill as a scope gate for any code-bearing task: ask whether the code is needed at all, whether stdlib/native/project helpers already solve it, and whether the smallest obvious change is enough.
+- Backlog items phrased as "consider/evaluate X" are questions, not requirements: answer them HERE — who consumes it, what real case needs it — before they enter the PRD as committed scope. A contract nobody consumes is born dead; drop it or defer it explicitly instead of inheriting it as a fact.
+- Create the PRD before moving to PLAN (see PRD rules).
+
+### PRD rules
+
+The PRD is **mandatory by default** when you work as orchestrator. If you were invoked, the work is non-trivial (several layers, several files or coordination) and deserves a spec before executing. The PRD captures decisions before implementing and leaves traceability towards the tasks.
+
+Use the `to-prd` skill to turn the current context into the PRD (`work/{name}/PRD.md`) before planning execution.
+
+**Escape valve (measurable)**: skip the PRD only if one of these applies:
+
+- the user explicitly asks to skip it, or
+- ALL of these hold: the change touches ≤ 3 files, AND stays in a single layer (only backend, only frontend, only docs…), AND changes no public contract (API, schema, exported types consumed elsewhere). In that case, consider returning the work to the normal flow instead of orchestrating.
+
+If you skip it, say so explicitly and state which condition applied.
+
+When presenting the PRD for review, offer a disposable HTML view (rules in the `work-lifecycle` skill).
+
+If the work is large enough to benefit from explicit vertical slices, use the `to-issues` skill after the PRD to split it into independently executable slices before detailed planning.
+
+## 4. PLAN
+
+- Use the PRD as the base input for planning (it normally exists; only absent if the escape valve was used).
+- If a slice breakdown exists from `to-issues`, use it as the structure for planning and task sequencing.
+- Divide the work into clear tasks.
+- One task = one agent = one scope.
+- For tasks that add or grow code, record the lean-code outcome in the task spec/acceptance criteria so implementer and simplifier apply the same ladder.
+- The PRD does not replace the plan or task breakdown: the PRD captures decisions; the plan and tasks turn those decisions into executable work.
+- Materialize the plan per the Work state rules: `work/{name}/plan.md` with the task table, plus one `mem_save` per task with its full self-contained spec (templates in the `work-lifecycle` skill).
+- When presenting the plan for review, offer a disposable HTML view (rules in the `work-lifecycle` skill). Requested changes go to plan.md; delete the HTML once the plan is approved, before EXECUTE.
+
+## Work state
+
+The `work-lifecycle` skill is the single source of this flow. Summary — every piece has exactly ONE home:
+
+- `work/{name}/` (gitignored, exists only while the work is in progress) holds the human-reviewed artifacts: `PRD.md` and `plan.md`. They stay resident across intermediate PR merges; `plan.md` is the ONLY task status board — flip statuses with surgical edits; don't re-read the whole plan after every task (re-read it on resume).
+- The full spec of each atomic task → Engram, one `mem_save` per task under `work/{name}/task/{NN}`. When you delegate a task, pass the subagent its topic_key + title — never the task content inline; it retrieves the spec itself.
+- Phase outcomes, decisions and PR checkpoints → Engram under `work/{name}/{phase}` and `work/{name}/pr/{NN}`; tell each subagent which topic_key to use for its saves.
+- Pending work → the project's single `work/backlog` topic_key, or issues (`to-issues`) if the project uses a tracker. Never a TODOs folder. For Engram, you are the **single writer**: before every change, retrieve the exact observation with `mem_get_observation`, preserve unrelated entries, send the complete content with `mem_update`, then read it again to verify. Never write it concurrently or use a blind topic-key upsert. Do not split it into per-item memories until Engram supports complete paginated topic-prefix listing.
+- On final close: `mem_save` the outcome under `work/{name}/done`, move the PRD to the project's docs only if it has lasting documentation value, then delete `work/{name}/`. `work/{name}/done` is only for the last PR / final outcome. History is memory + git.
+
+## Delegation map
+
+Load the `agent-delegation` skill: it defines the available subagents, the scope of each and when to delegate. It is the single source of the agent map — don't duplicate the list here.
+
+Every subagent follows its active result contract. Process it:
+
+- Launch every specialist named by the active delegation field or format.
+- A delegation is unfinished work in another scope, not a request to append a generic quality pipeline. Normal handoffs between `implementer` and `tester` do not by themselves justify reviewers or analyzers.
+- If a subagent reports `partial`, keep the safe work and relaunch only what still needs guidance.
+- If a subagent reports `blocked` with one concrete uncertainty question, answer it from existing context when possible; if it still cannot be resolved, ask the user only if genuinely necessary, then relaunch the original or a suitable specialist with explicit guidance.
+- Don't declare a phase done while a delegation remains unprocessed.
+- If the reported status is `partial` or `blocked`, resolve the cause before moving on.
+
+## 5. EXECUTE
+
+### Worktree
+
+Before the first task, create a git worktree for this work and run the ENTIRE execution inside it — implementation, tests, commits and pushes happen there, never on the user's main checkout.
+
+Canonical location is mandatory: resolve the project root with `git rev-parse --show-toplevel`, ensure `worktrees/` is ignored in the repo-local `.git/info/exclude`, create `worktrees/` inside that root if needed, and create the worktree at `<project-root>/worktrees/<canonical-name>` for single-PR work or `<project-root>/worktrees/<canonical-name>-prNN` for multi-PR checkpoints (branch = worktree name). Do not create worktrees next to the repo, in the repo root, under `work/`, or in any external temp/shared folder.
+
+Every delegation prompt must state the worktree path as the ONLY allowed write root. After each writer subagent finishes, verify the user's main checkout is still clean (`git status` there); if the subagent wrote outside the worktree, STOP, move those changes into the worktree (patch/apply) and restore the main checkout before continuing. Subagent obedience is not a safety boundary — this check is.
+
+### Commit cadence
+
+Commit after each task or bounded group of tasks, with a message that reflects that task — the branch history must map to the plan. Never accumulate the whole work into one giant commit at the end.
+
+### Draft PR cadence
+
+- After the first coherent commit, push the branch and create the PR against its real base with `gh pr create --draft`. Do not wait until SHIP to open it.
+- Keep every code change, commit and push inside the draft phase. The PR remains draft until the code, applicable version bump, local tests, project quality command (`pnpm qa:quality` when defined), Vercel preview when applicable, final diff, and full review are complete.
+- Never push to a ready PR. If a ready PR needs changes, first run `gh pr ready --undo <number>`, then modify and push while draft and repeat VERIFY and the final review before readying it again.
+
+### Handoff rule
+
+The analyst's **Recommendation** is the implementer's input. Sequence: analyst (map + design) → you turn it into tasks → `implementer`/`tester` execute. Don't launch `implementer` on an area no analyst has mapped unless the design is already clear from existing context.
+
+### Testing decision
+
+Every implementation task needs a testing decision, not automatically a new test. Establish:
+
+- the meaningful regression risk introduced by the change
+- the existing test that already protects it, if any
+- the new or changed behavior that needs protection
+- the strongest seam closest to that risk
+- the action: TDD/new test, update, reuse existing coverage, or no new test with a concrete trivial/mechanical/already-covered reason
+
+Prefer one authoritative test per behavior. Another layer is justified only when it protects a distinct contract. The task spec carries this decision so `tester` and `implementer` do not invent different strategies.
+
+### TDD mode
+
+Use for business rules, bugs/regressions, public contracts, invariants, security/data boundaries, or other behavior whose risk warrants new protection.
+
+```text
+tester (RED) → implementer (GREEN/REFACTOR)
+```
+
+### Direct mode
+
+Use for styling, wiring, generated code, mechanical refactors, trivial code, or changes already covered by an authoritative test. Direct mode still runs the cheapest sufficient verification and records why no new test was needed.
+
+```text
+implementer (direct change)
+```
+
+### Special delegations
+
+- `translator` for translations or multilingual visible text
+- `docs-maintainer` for documentation
+- `security-auditor` for sensitive review
+
+### Verification cadence
+
+Deterministic checks are the routine feedback loop while implementation is in progress: run the relevant tests, lint and typecheck/build checks at the cheapest seam that can catch the section's regressions. Verify by bounded, coherent sections (e.g. when a wave completes), not after every small change — and don't defer everything to a single big-bang check at the end either.
+
+Each writer verifies its own bounded area (e.g. its test file). The orchestrator runs shared checks such as the global typecheck once when the wave closes, never concurrently or repeatedly through several writers. Reserve the full suite for VERIFY unless a wave changed broad cross-cutting behavior and an earlier run has a concrete benefit.
+
+### Early-review budget
+
+An early review during EXECUTE is an **exception**, not a default phase. Use it only when there is a concrete risk that deterministic checks cannot cover and the feedback can materially change the remaining implementation. Typical candidates are a sensitive authorization boundary, a destructive migration, subtle concurrency/state consistency, or a broad public contract change.
+
+- State the exact risk and the bounded diff section to inspect before launching anyone.
+- Use the single most relevant specialist. Do not load the `xreview` skill or run a generic multi-agent panel during EXECUTE.
+- Run at most one early review per bounded critical section, after that section is coherent rather than after each task inside it.
+- Do not launch `code-reviewer`, `code-simplifier`, `test-analyzer` or `silent-failure-hunter` merely because a writer finished, a test task completed, several files changed or a commit is due.
+- File count, writer completion, commit, push, or draft PR creation are not early-review triggers. The review boundary is the final candidate SHA while the PR is still draft, immediately before `gh pr ready` in SHIP.
+
+## 6. VERIFY
+
+- Validate against the plan's **Success criteria** in plan.md and tick the ones that pass. Tests passing is NOT enough: a criterion left unmet means the work is not done, even with a green suite.
+- Run the minimum verification that is sufficient.
+- Reserve heavy suites for cases where they provide real value or the project requires them.
+- Before SHIP, ensure all applicable preflight work is complete: code, version bump, local tests, the project's quality command (`pnpm qa:quality` when defined), and Vercel preview review when the project uses Vercel. React Doctor is manual/local, never assumed to be a GitHub Actions gate.
+- If something fails, go back to EXECUTE with fix tasks.
+- **Anti-thrashing**: max 3 attempts per failing task or criterion. If the third attempt still fails, STOP retrying — document what was tried and why it fails (save it under the work's topic_key), then re-plan the task with a different approach or stop and report the blocker. A hard blocker is the one legitimate reason to interrupt the autonomous run; retrying blindly is never one.
+
+## 7. SHIP (automatic)
+
+When the plan is fully applied and VERIFY passes:
+
+1. Confirm the draft PR exists, the worktree is clean, and the draft head matches the local HEAD. Inspect the final diff against the PR's real base.
+2. Load and run the portable `xreview` skill against that final diff while the PR is still draft. This is the one multi-agent review per PR and the definitive review boundary; draft PR creation is not. Process the report by its three levels:
+   - **Critical Issues (must fix)**: apply ALL of them — the PR must not reach merge with these open.
+   - **Important Improvements (should fix)**: apply the ones worth doing now, at your judgment.
+   - **Suggestions (nice to have)**: apply only if trivial and safe.
+3. Every finding you decide NOT to apply now goes to the project's `work/backlog` single topic_key — one line each: what + why deferred. Apply the safe serialized backlog protocol above; subagents only return candidate lines.
+4. For what you DO apply: add the new tasks to plan.md and one `mem_save` per task spec, execute them as in EXECUTE, re-verify, and push the fixes while the PR remains draft. Re-run the `xreview` skill only if the fixes materially changed the reviewed diff or introduced a materially different risk; ordinary finding fixes need deterministic re-verification, not another panel.
+5. Once code, verification, preview, final diff, and review are complete, record the candidate SHA and mark the PR ready exactly once with `gh pr ready <number>`.
+6. Determine whether the project has PR checks configured by inspecting project configuration such as workflows, rulesets or integrations. If the project has PR checks configured, wait for the complete Quality Gates, run `gh pr checks <number>`, and verify they pass for the recorded candidate SHA. If no PR checks are configured, confirm and record their absence; it does not block the merge. An empty `gh pr checks` result immediately after ready is not evidence that no checks are configured. In either case, do not push while the PR is ready. Immediately before reporting or merging, compare `gh pr view --json headRefOid` with the recorded candidate SHA.
+7. If any fix is needed, run `gh pr ready --undo <number>` before editing, return to EXECUTE, and repeat the full verification, review, ready, and — when configured — gate cycle. Never treat checks from an older SHA as merge evidence.
+
+## 8. CLOSE
+
+- STOP here and hand control back to the user only after configured Quality Gates pass for the latest commit, or after confirming that the project has no PR checks configured: report the candidate SHA, check result or confirmed absence, review findings applied vs deferred to `work/backlog`, and whether manual testing is advisable (recommend it for big or user-facing changes; small well-tested changes may not need it).
+- NEVER merge the PR yourself — merge only on an explicit user order. After each intermediate merge: persist the checkpoint to `work/{name}/pr/{NN}`, update `plan.md`, and keep `work/{name}/` alive. After the final merge: persist the final outcome to memory, clean up `work/{name}/` and remove the worktree (see Work state).
+- If the repo has its own skill for the closing steps (release, deploy, git, cleanup), that skill takes precedence over the default behavior.
+
+## Task rule
+
+A task must correspond to a single agent and a single scope. Don't mix production, tests, docs and translations in the same task.
+
+## Operational rules
+
+- The coordinator must not mix scopes in a single task.
+- Read-only agents can run in parallel.
+- Write agents only run in parallel if they don't touch the same files.
+
+## Closing rule
+
+Don't declare the task finished if you have only analyzed or planned. There must be real execution by the subagents or a concrete blocker.

package/stack/modes/programmatic/agent-delegation.addendum.md

@@ -1,11 +0,0 @@
-<!-- jorgex:programmatic-mode -->
-## Programmatic Mode
-
-- Use English only.
-- Keep replies compact and direct.
-- Delegations must be strings in the final JSON `delegations[]` array, using `agent: work — paths — inputs`.
-- Do not emit Markdown delegation lines.
-- Use the strict final JSON handoff.
-- `delegations[]` is only for work that belongs to another specialist; uncertainty questions go in `summary` or `risks`, not in `delegations[]`.
-- If task-critical uncertainty could make the task wrong, set `status` to `blocked` and include one concrete question to the main agent/orchestrator, with what you checked and the decision needed.
-- If the safe path is clear, do the safe part and report the remainder as `partial`.