navori 0.2.5 → 0.2.7

package/dist/index.js

@@ -1653,6 +1653,51 @@ function extractManagedContent(existing, id, commentStyle = "html") {
   const match = findMarker(existing, id, syntax);
   return match ? match.content : null;
 }
+function locateManagedBlocks(content, syntax) {
+  const openRegex = new RegExp(
+    `${escapeRegex(syntax.openPrefix)}\\s+id="([^"]+)"${syntax.attrsAndTerminatorPattern}`,
+    "g"
+  );
+  const blocks = [];
+  for (const m of content.matchAll(openRegex)) {
+    if (m.index === void 0) continue;
+    const id = m[1];
+    const close = closeMarker(id, syntax);
+    const closeStart = content.indexOf(close, m.index + m[0].length);
+    if (closeStart < 0) continue;
+    blocks.push({ id, openStart: m.index, closeEnd: closeStart + close.length });
+  }
+  return blocks;
+}
+function reorderManagedBlocks(content, canonicalOrder, commentStyle = "html") {
+  const syntax = syntaxFor(commentStyle);
+  const blocks = locateManagedBlocks(content, syntax);
+  if (blocks.length < 2) {
+    return { output: content, reordered: false, blockedByInterleaving: false };
+  }
+  const rank = /* @__PURE__ */ new Map();
+  canonicalOrder.forEach((id, i) => {
+    if (!rank.has(id)) rank.set(id, i);
+  });
+  const unknownBase = canonicalOrder.length;
+  const desired = blocks.map((b, i) => ({ b, i, key: rank.has(b.id) ? rank.get(b.id) : unknownBase + i })).sort((a, z8) => a.key - z8.key || a.i - z8.i).map((x) => x.b);
+  if (desired.every((b, i) => b === blocks[i])) {
+    return { output: content, reordered: false, blockedByInterleaving: false };
+  }
+  for (let i = 0; i < blocks.length - 1; i++) {
+    const gap = content.slice(blocks[i].closeEnd, blocks[i + 1].openStart);
+    if (gap.trim() !== "") {
+      return { output: content, reordered: false, blockedByInterleaving: true };
+    }
+  }
+  const first = blocks[0];
+  const last = blocks[blocks.length - 1];
+  const preamble = content.slice(0, first.openStart);
+  const suffix = content.slice(last.closeEnd);
+  const body = desired.map((b) => content.slice(b.openStart, b.closeEnd)).join("\n\n");
+  const pre = preamble === "" ? "" : preamble.replace(/\n*$/, "\n\n");
+  return { output: pre + body + suffix, reordered: true, blockedByInterleaving: false };
+}
 function resolveCondition(config, path) {
   const segments = path.split(".");
   let cursor = config;
@@ -1885,6 +1930,16 @@ function computeRenderPlan(existing, inputConfig, repoRoot, options = {}) {
     updatesAvailable
   };
 }
+var CLAUDE_COMPUTED_BLOCK_IDS = [
+  "skills-index",
+  "agentes-disponibles",
+  "contexto-proyecto"
+];
+function canonicalManagedOrder(config, repoRoot) {
+  const plan = computeRenderPlan("", config, repoRoot);
+  const ids = plan.entries.filter((entry) => entry.newContent !== null).map((entry) => entry.asset.id);
+  return [...ids, ...CLAUDE_COMPUTED_BLOCK_IDS];
+}
 
 // src/engines/claude/settings-detection.ts
 var NAVORI_OWNERSHIP_MARKER = "$navori";
@@ -2491,6 +2546,13 @@ function renderClaudeEngine(cwd, inputConfig, options = {}) {
   } else {
     claudeMdContent = removeManagedSection(claudeMdContent, CONTEXTO_PROYECTO_ID);
   }
+  const reorder = reorderManagedBlocks(claudeMdContent, canonicalManagedOrder(config, repoRoot));
+  claudeMdContent = reorder.output;
+  if (reorder.blockedByInterleaving) {
+    warnings.push(
+      "CLAUDE.md: los bloques managed est\xE1n fuera del orden can\xF3nico, pero hay texto tuyo intercalado entre bloques, as\xED que no los reorden\xE9. Mueve ese texto arriba del primer bloque managed o abajo del \xFAltimo para que navori pueda ordenarlos."
+    );
+  }
   if (claudeMdContent !== claudeMdExisting) {
     pending.push({
       path: claudeMdPath,
@@ -3446,6 +3508,7 @@ var ES = {
   detectionFailedYes: "No detect\xE9 el nombre del proyecto. Corre sin --yes/--recommended para darlo.",
   wroteConfig: (path) => `Escrib\xED ${path}`,
   recPluginsEnabled: (list) => `Plugins recomendados activados: ${list}`,
+  pluginsAlwaysOn: (list) => `Incluidos siempre con navori: ${list} (no hace falta elegirlos)`,
   presetGapNotice: (stack) => `Detect\xE9 un proyecto '${stack}', pero todav\xEDa no hay un preset oficial para ese stack. Se instala el harness completo (agentes, gates, protocolo, SDD) y funciona desde ya; lo \xFAnico que falta son skills espec\xEDficas de '${stack}'. Quedas con el baseline (preset: custom). Para cubrir el hueco: crea tu preset local con 'navori preset init ${stack}', o agrega skills sueltas en project.localSkills.`,
   placeholderNameNotice: (name) => `El name '${name}' parece un placeholder de scaffold (heredado del package.json sin renombrar). Ren\xF3mbralo en package.json o edita "name" en navori.config.json si no es el nombre real del repo.`,
   workspaceDefaultsTitle: (name) => `Defaults del workspace \xB7 ${name}`,
@@ -3553,6 +3616,7 @@ var EN = {
   detectionFailedYes: "Could not detect project name. Run without --yes/--recommended to provide one.",
   wroteConfig: (path) => `Wrote ${path}`,
   recPluginsEnabled: (list) => `Recommended plugins enabled: ${list}`,
+  pluginsAlwaysOn: (list) => `Always included with navori: ${list} (no need to pick them)`,
   presetGapNotice: (stack) => `Detected a '${stack}' project, but there's no official preset for that stack yet. The full harness installs (agents, gates, protocol, SDD) and works right away; the only thing missing are '${stack}'-specific skills. You stay on the baseline (preset: custom). To cover the gap: scaffold your local preset with 'navori preset init ${stack}', or add individual skills via project.localSkills.`,
   placeholderNameNotice: (name) => `The name '${name}' looks like a scaffold placeholder (carried over from an un-renamed package.json). Rename it in package.json or edit "name" in navori.config.json if it isn't the repo's real name.`,
   workspaceDefaultsTitle: (name) => `Workspace defaults \xB7 ${name}`,
@@ -3919,7 +3983,8 @@ var initCommand = defineCommand2({
       }
       const wsPlugins2 = wsDefaults?.plugins ?? {};
       const recommendedPlugins = args.recommended ? buildRecommendedPlugins(cwd) : {};
-      const mergedPlugins2 = { ...wsPlugins2, ...recommendedPlugins };
+      const mergedPlugins2 = { ...ALWAYS_ON_PLUGINS, ...wsPlugins2, ...recommendedPlugins };
+      p2.log.info(tr.pluginsAlwaysOn(Object.keys(ALWAYS_ON_PLUGINS).join(", ")));
       if (args.recommended && Object.keys(recommendedPlugins).length > 0) {
         p2.log.info(tr.recPluginsEnabled(Object.keys(recommendedPlugins).join(", ")));
       }
@@ -4236,7 +4301,7 @@ var initCommand = defineCommand2({
       }
     }
     const wsPlugins = wsDefaults?.plugins ?? {};
-    const mergedPlugins = { ...wsPlugins, ...pluginsConfig };
+    const mergedPlugins = { ...ALWAYS_ON_PLUGINS, ...wsPlugins, ...pluginsConfig };
     const monorepoBlock = await buildMonorepoBlock(cwd, detected, {
       scanMonorepo: Boolean(args["scan-monorepo"]),
       autoYes: false,
@@ -4298,6 +4363,7 @@ async function chooseAdoptionMode(cwd, infra, projectName, args) {
   const tr = t(args.lang);
   if (args.yes) {
     p2.log.warn(tr.existingInfraYesMode);
+    p2.note(formatInfraSummary(infra, args.lang), tr.filesFoundTitle);
     return "coexist";
   }
   p2.log.warn(tr.existingInfraDetected);
@@ -4326,7 +4392,9 @@ async function chooseAdoptionMode(cwd, infra, projectName, args) {
   return "coexist";
 }
 async function pickPlugins(lang) {
-  const ids = listKnownPluginIds();
+  const alwaysOn = Object.keys(ALWAYS_ON_PLUGINS);
+  p2.log.info(t(lang).pluginsAlwaysOn(alwaysOn.join(", ")));
+  const ids = listKnownPluginIds().filter((id) => !(id in ALWAYS_ON_PLUGINS));
   if (ids.length === 0) return [];
   const options = ids.map((id) => {
     const plugin = (() => {
@@ -4405,10 +4473,11 @@ ${summary}`);
   }
   return overrides;
 }
+var ALWAYS_ON_PLUGINS = {
+  engram: { enabled: true }
+};
 function buildRecommendedPlugins(cwd) {
-  const result = {
-    engram: { enabled: true }
-  };
+  const result = {};
   if (isGitHubRepo(cwd)) {
     result.gh = { enabled: true };
   }
@@ -4831,21 +4900,45 @@ function scanManagedDrift(cwd, config) {
   }
   return out;
 }
+function scanManagedOrder(cwd, config) {
+  const claudeMdPath = join13(cwd, "CLAUDE.md");
+  if (!existsSync17(claudeMdPath)) return null;
+  const content = readFileSync14(claudeMdPath, "utf-8");
+  const current = listMarkers(claudeMdPath).map((m) => m.id);
+  if (current.length < 2) return null;
+  const canonical = canonicalManagedOrder(config, cwd);
+  const result = reorderManagedBlocks(content, canonical);
+  if (!result.reordered && !result.blockedByInterleaving) return null;
+  const rank = /* @__PURE__ */ new Map();
+  canonical.forEach((id, i) => {
+    if (!rank.has(id)) rank.set(id, i);
+  });
+  const expected = current.map((id, i) => ({ id, i, key: rank.has(id) ? rank.get(id) : canonical.length + i })).sort((a, z8) => a.key - z8.key || a.i - z8.i).map((x) => x.id);
+  return { current, expected, interleaved: result.blockedByInterleaving };
+}
 function suggestNextSteps(state) {
   const steps = [];
   if (!state.claudeMdExists) {
-    steps.push("Corr\xE9 'navori render --apply' para generar CLAUDE.md + .claude/.");
+    steps.push("Corre 'navori render --apply' para generar CLAUDE.md + .claude/.");
   }
   if (state.missingPlugins.length > 0) {
     steps.push(
-      `Resolv\xE9 ${state.missingPlugins.length} plugin(s) faltante(s): instalalos o quitalos del config.`
+      `Resuelve ${state.missingPlugins.length} plugin(s) faltante(s): inst\xE1lalos o qu\xEDtalos del config.`
     );
   }
   if (state.drifts.some((d) => d.kind === "content")) {
-    steps.push("Corr\xE9 'navori sync --interactive' para resolver bloques editados a mano.");
+    steps.push("Corre 'navori sync --interactive' para resolver bloques editados a mano.");
   }
   if (state.drifts.some((d) => d.kind === "version")) {
-    steps.push("Corr\xE9 'navori render --apply' para traer los bloques a la \xFAltima versi\xF3n.");
+    steps.push("Corre 'navori render --apply' para traer los bloques a la \xFAltima versi\xF3n.");
+  }
+  if (state.orderReport && !state.orderReport.interleaved) {
+    steps.push("Corre 'navori render --apply' para reordenar los bloques de CLAUDE.md al orden can\xF3nico.");
+  }
+  if (state.orderReport?.interleaved) {
+    steps.push(
+      "Mueve el texto que tienes entre bloques managed de CLAUDE.md arriba del primer bloque o abajo del \xFAltimo; luego corre 'navori render --apply' para reordenarlos."
+    );
   }
   if (steps.length === 0) {
     steps.push("Todo al d\xEDa \u2014 sin acciones pendientes.");
@@ -4910,6 +5003,7 @@ var doctorCommand = defineCommand3({
     const markers = listMarkers(claudeMdPath);
     const missingPlugins = collectMissingPlugins(config);
     const drifts = scanManagedDrift(cwd, config);
+    const orderReport = scanManagedOrder(cwd, config);
     const corruptedSettings = scanCorruptedSettings(cwd);
     const missingInvariants = scanMissingInvariants(cwd, config);
     const resolvedPreset = config.preset !== "custom" ? resolvePreset(config.preset, cwd) : null;
@@ -4934,6 +5028,7 @@ var doctorCommand = defineCommand3({
       managedBlocks: markers,
       missingPlugins,
       drifts,
+      orderReport,
       corruptedSettings,
       missingInvariants,
       missingPreset,
@@ -5057,10 +5152,26 @@ ${lines.join("\n")}`
 ${lines.join("\n")}`
       );
     }
+    if (orderReport) {
+      if (orderReport.interleaved) {
+        p3.log.warn(
+          `Bloques managed de CLAUDE.md fuera del orden can\xF3nico \u2014 NO se pueden reordenar autom\xE1ticamente porque hay texto tuyo entre bloques. Mueve ese texto arriba del primer bloque managed o abajo del \xFAltimo; luego corre 'navori render --apply'.
+  orden actual:   ${orderReport.current.join(", ")}
+  orden can\xF3nico: ${orderReport.expected.join(", ")}`
+        );
+      } else {
+        p3.log.warn(
+          `Bloques managed de CLAUDE.md fuera del orden can\xF3nico \u2014 corre 'navori render --apply' o 'navori sync' para reordenarlos (el primer bloque marca el centro de gravedad del harness).
+  orden actual:   ${orderReport.current.join(", ")}
+  orden can\xF3nico: ${orderReport.expected.join(", ")}`
+        );
+      }
+    }
     const nextSteps = suggestNextSteps({
       claudeMdExists: report.checks.claudeMdExists,
       missingPlugins,
-      drifts
+      drifts,
+      orderReport
     });
     p3.note(
       nextSteps.map((s) => `  ${color.cyan(sym.bullet)} ${s}`).join("\n"),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "navori",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "Multi-agent harness + SDD scaffolder for Claude Code and other AI engines",
   "type": "module",
   "bin": {