@ingeniomaps/cauce 0.53.1 → 0.54.0

package/engine/cli/catalog.js

@@ -132,6 +132,16 @@ function learn(agent, cli) {
     // aplicada editando frontmatter a mano es justo el paso que un agente hace mal en silencio.
     // Un recorrido aprende de sus corridas y un cargo de su profesión: mismo ciclo, distinto insumo.
     const kind = cli.has('--flow') ? 'flow' : 'agent'
+    // El tercer destino: se miró y no cambia nada. Sin esto sólo había aplicarla o dejarla esperando,
+    // y lo que se hacía era mergear el PR sin firmar — que deja el documento diciendo «sin aplicar»
+    // para siempre, indistinguible de una que espera trabajo.
+    if (cli.has('--archived')) {
+      const result = L.archive(opsRoot(), agent, cli.value('--period'), kind)
+      const relative = path.relative(opsRoot(), result.file)
+      return console.log(result.already
+        ? `= ${relative} ya estaba archivada`
+        : `✓ ${relative} queda archivada: se miró y no cambia nada`)
+    }
     if (cli.has('--applied')) {
       const result = L.seal(opsRoot(), agent, cli.value('--period'), kind)
       const relative = path.relative(opsRoot(), result.file)
@@ -139,14 +149,28 @@ function learn(agent, cli) {
         ? `= ${relative} ya estaba aplicada`
         : `✓ ${relative} queda aplicada: no se vuelve a aplicar`)
     }
+    // Un recorrido no tiene informe semanal: su propuesta se compone de los veredictos de sus propias
+    // corridas y nunca lee `learning/reports/`. La forma desnuda —la que para un cargo abre el informe
+    // de la semana— no tiene entonces qué abrir acá, y `prepareReport` lo decía resolviendo con el
+    // `kind` por defecto: «no existe agents/<tipo>/<slug>/SKILL.md», que manda a crear un cargo que no
+    // falta. Negarse nombrando el comando que sí corresponde es lo que cierra R13.
+    if (kind === 'flow' && !cli.has('--proposal')) {
+      fail(`${agent} es un recorrido: aprende de sus corridas, no de informes semanales.\n`
+        + `  Abrí la propuesta con "ops learn ${agent} --flow --proposal".`, 2)
+    }
     // `--period` es para consolidar a mano un mes que no es el de hoy. El ciclo automático no lo
     // pasa: la propuesta se llama por el mes en que se abre y arrastra lo que todavía no entró.
     const result = cli.has('--proposal')
       ? L.prepareProposal(opsRoot(), agent, new Date(), cli.value('--period'), kind)
       : L.prepareReport(opsRoot(), agent)
-    // Sin informes pendientes no hay archivo que nombrar, y decirlo es el resultado: el ciclo corrió
-    // y no había nada que consolidar.
-    if (!result.file) return console.log('= sin informes pendientes: no se abre propuesta')
+    // Sin archivo que nombrar hay dos resultados distintos y decir cuál es el punto: no había material,
+    // o lo había y ninguno propuso un cambio. El segundo no es un error ni un mes perdido —el informe
+    // es histórico igual— y confundirlo con el primero manda a buscar por qué no se investigó.
+    if (!result.file) {
+      return console.log(result.quiet
+        ? `= ${result.quiet} informe(s) sin recomendación: no hay qué proponer, quedan para el mes que viene`
+        : '= sin informes pendientes: no se abre propuesta')
+    }
     console.log(`${result.created ? '+' : '='} ${path.relative(opsRoot(), result.file)}`)
     // Un cargo consolida informes de su profesión; un recorrido, las corridas que lo midieron.
     if (typeof result.reports === 'number') {
@@ -170,7 +194,12 @@ function evaluate(agent, caso, cli) {
     }
     // El caso es el posicional que sigue al cargo: `evaluate <cargo> --bench <caso>`. Sin él se arma
     // un banco suelto, para mirarlo a mano; una corrida real pide uno por caso.
-    return console.log(evaluationBench(root, agent, caso, cli.has('--force'), kind))
+    //
+    // Relativa a la raíz, como la de `--record`: esta salida la lee el agente que prepara los bancos,
+    // y de ahí la ruta viaja a los informes y a las propuestas que después lee otro cargo. Absoluta
+    // nombraba el directorio personal de una máquina, y así quedaron mil cuatrocientas ochenta y cinco
+    // citas que dejaron de resolver el día que este repositorio cambió de nombre.
+    return console.log(path.relative(root, evaluationBench(root, agent, caso, cli.has('--force'), kind)))
   }
   try {
     // Los casos, para que un recorrido los ejecute. Sin `--json` no tiene sentido: es entrada de

package/engine/cli/instance.js

@@ -7,7 +7,6 @@
 
 const fs = require('node:fs')
 const path = require('node:path')
-const { spawnSync } = require('node:child_process')
 const F = require('../core/files')
 const O = require('../core/ownership')
 const CL = require('../core/changelog')
@@ -15,7 +14,6 @@ const M = require('../core/manifest')
 const OB = require('../core/onboarding')
 const P = require('../planning/parser')
 const ST = require('../planning/state')
-const I = require('../integrations/registry')
 const A = require('../automation')
 const { fail } = require('./io')
 

package/engine/cli/ops.js

@@ -4,25 +4,9 @@
 const fs = require('node:fs')
 const path = require('node:path')
 const { spawnSync } = require('node:child_process')
-const P = require('../planning/parser')
-const B = require('../planning/business-rules')
-const PC = require('../planning/contracts')
-const ST = require('../planning/state')
-const I = require('../integrations/registry')
-const L = require('../agents/learning')
 const A = require('../automation')
-const F = require('../core/files')
-const O = require('../core/ownership')
-const CL = require('../core/changelog')
-const M = require('../core/manifest')
-const SC = require('../core/scan')
-const OB = require('../core/onboarding')
-const C = require('../config/validate')
-const T = require('../flows/registry')
-const AG = require('../agents/catalog')
-const EV = require('../agents/evaluations')
 const { FLAGS, parse } = require('./args')
-const { fail, opsRoot } = require('./io')
+const { fail } = require('./io')
 const IN = require('./instance')
 const PL = require('./planning')
 const CAT = require('./catalog')
@@ -170,7 +154,7 @@ function usage() {
   ops automation doctor <ops-root> claude|codex|gemini|antigravity
   ops automation install <ops-root> claude|codex|gemini|antigravity
   ops automation uninstall <ops-root> claude|codex|gemini|antigravity
-  ops learn <agent|flow> [--flow] [--proposal [--period <AAAA-MM>]] [--applied [--period <AAAA-MM>]]
+  ops learn <agent|flow> [--flow] [--proposal [--period <AAAA-MM>]] [--applied|--archived [--period <AAAA-MM>]]
   ops evaluate <agent|flow> [--flow] [--cases [--json]] [--bench [caso]] [--record [AAAA-MM-DD]]
   ops agents list [ops-root] [--own|--system] [--json]
   ops agents fork <cargo> [ops-root]

package/engine/config/validate.js

@@ -45,10 +45,15 @@ function validateWorkspaces(workspaces, errors) {
       continue
     }
     for (const key of Object.keys(workspace)) {
-      if (!['name', 'path'].includes(key)) {
+      if (!['name', 'path', 'verify'].includes(key)) {
         errors.push(`ops.config.json: workspaceRoots[${index}].${key} no está permitido`)
       }
     }
+    // Opcional: la raíz que no lo declara deja que quien verifica descubra la puerta, como siempre.
+    // Declararlo vacío es peor que no declararlo —promete una puerta y no la da—, así que se rechaza.
+    if ('verify' in workspace && (typeof workspace.verify !== 'string' || !workspace.verify.trim())) {
+      errors.push(`ops.config.json: workspaceRoots[${index}].verify debe ser el comando, o no estar`)
+    }
     if (typeof workspace.name !== 'string' || !workspace.name.trim()) {
       errors.push(`ops.config.json: workspaceRoots[${index}].name es obligatorio`)
     }
@@ -78,4 +83,4 @@ function validateRunner(runner, errors) {
   }
 }
 
-module.exports = { MODES, validateOpsConfig }
+module.exports = { validateOpsConfig }

package/engine/core/onboarding.js

@@ -120,8 +120,5 @@ module.exports = {
   missingSections,
   orphanCredentials,
   guide,
-  OPENING,
-  DIMENSIONS,
-  FOLLOW_UPS,
   PLACEHOLDERS,
 }

package/engine/core/scan.js

@@ -218,4 +218,4 @@ module.exports = {
   workspaceRoots,
   candidates,
   inventory,
-  commandsLine, scan, services, expectedEnv, IGNORED, DEPTH, ENV_MAX }
+  commandsLine, scan, services }

package/engine/hooks/input.js

@@ -124,6 +124,6 @@ function findOpsRoot(start) {
 }
 
 module.exports = {
-  readInput, commandOf, fileOf, patchOf, filesOf, contentOf, cwdOf, block, configOf,
+  readInput, commandOf, patchOf, filesOf, contentOf, cwdOf, block, configOf,
   gitDirectory, isCommit, stagedFiles, pushAllowed, findOpsRoot,
 }

package/engine/integrations/proposals.js

@@ -79,4 +79,4 @@ function validate(root, provider, workspaces = []) {
   return errors
 }
 
-module.exports = { STATES, TYPES, read, validate }
+module.exports = { read, validate }

package/engine/integrations/providers/jira.js

@@ -161,4 +161,4 @@ async function fetchItems(config, options = {}) {
   return issues.map((issue) => normalizeIssue(issue, config))
 }
 
-module.exports = { adfToMarkdown, deriveAcceptance, fetchItems, normalizeFixture, normalizeIssue, validateConfig }
+module.exports = { fetchItems, normalizeFixture, validateConfig }

package/engine/integrations/registry.js

@@ -397,7 +397,6 @@ function providerState(root, name) {
 
 module.exports = {
   providerState,
-  STATES,
   adapter,
   frontmatter,
   promote,

package/engine/planning/business-rules.js

@@ -53,4 +53,4 @@ function validate(root) {
   return errors
 }
 
-module.exports = { markdownFiles, validate }
+module.exports = { validate }

package/engine/planning/contracts.js

@@ -297,6 +297,14 @@ function validateState({ epics, milestones, done, wip, roles = new Set(), humanA
   for (const milestone of milestones) {
     if (!/^[a-z0-9]+(?:-[a-z0-9]+)*$/.test(milestone.slug)) errors.push(`hito con slug inválido: ${milestone.slug}`)
     if (milestoneSlugs.has(milestone.slug)) errors.push(`hito duplicado: ${milestone.slug}`)
+    // La misma puerta que la épica y la tarea, en el nivel que no la tenía. Un hito agrupa trabajo bajo
+    // un nombre, y abrirlo con el nombre puesto para después es el camino natural: nadie promueve una
+    // tarea sin aceptación, y en cambio el título del hito se decide al final, cuando ya se construyó
+    // debajo. El marcador es lo único que se puede juzgar sin leer — un título vago sigue pasando, y
+    // eso es deliberado: acá también un heurístico sobre prosa se equivocaría en los dos sentidos.
+    if (PLACEHOLDERS.test(milestone.title)) {
+      errors.push(`hito ${milestone.slug}: el título no está decidido — "${milestone.title}"`)
+    }
     milestoneSlugs.add(milestone.slug)
     for (const task of milestone.tasks) {
       if (!task.service) errors.push(`BACKLOG ${task.slug}: falta (service: <ruta>)`)
@@ -406,7 +414,6 @@ function oversizedUnits({ epics = [], milestones = [] }) {
 }
 
 module.exports = {
-  testedCriteria,
   validateState,
   oversizedUnits,
   validateAdr,

package/engine/planning/parser.js

@@ -153,6 +153,12 @@ function readCast(rest) {
   }
 }
 
+// El texto de la aceptación, delimitado por las cursivas que lo envuelven. El cierre no es «el
+// próximo guión bajo»: un identificador como `MAX_ATTEMPTS` trae los suyos, y cortar en el primero
+// devolvía `MAX` con `check` en verde, que es la forma cara del error —la tarea se lee completa y no
+// lo está—. Cierra el `_` que markdown cerraría: el que no está entre caracteres de palabra.
+const ACCEPTANCE = /_Aceptaci[oó]n:\s*(.*?\S)_(?![A-Za-z0-9])/i
+
 function readBacklog(dir) {
   const text = withoutComments(read(path.join(dir, 'BACKLOG.md')))
   const milestones = []
@@ -175,7 +181,7 @@ function readBacklog(dir) {
       slug: task[1].trim(), tier: task[2] || '', cast: readCast(rest),
       epic: ((rest.match(/\(epic:\s*(\d{3})\)/) || [])[1] || ''),
       service: ((rest.match(/\(service:\s*([^)]+)\)/) || [])[1] || '').trim(),
-      acceptance: ((rest.match(/_Aceptaci[oó]n:\s*([^_]+)_/i) || [])[1] || '').trim(),
+      acceptance: ((rest.match(ACCEPTANCE) || [])[1] || '').trim(),
       criteria: criteriaRefs(rest),
       noSplit: noSplitReason(rest),
     })

package/engine/schemas/ops-config.schema.json

@@ -2,33 +2,80 @@
   "$schema": "https://json-schema.org/draft/2020-12/schema",
   "title": "Cauce engine configuration",
   "type": "object",
-  "required": ["project", "mode", "workspaceRoots", "runner"],
+  "required": [
+    "project",
+    "mode",
+    "workspaceRoots",
+    "runner"
+  ],
   "properties": {
-    "$schema": { "type": "string", "minLength": 1 },
-    "cauceVersion": { "type": "string", "minLength": 1 },
-    "project": { "type": "string", "minLength": 1 },
-    "mode": { "enum": ["embedded", "sidecar", "toolkit"] },
+    "$schema": {
+      "type": "string",
+      "minLength": 1
+    },
+    "cauceVersion": {
+      "type": "string",
+      "minLength": 1
+    },
+    "project": {
+      "type": "string",
+      "minLength": 1
+    },
+    "mode": {
+      "enum": [
+        "embedded",
+        "sidecar",
+        "toolkit"
+      ]
+    },
     "workspaceRoots": {
       "type": "array",
       "minItems": 1,
       "items": {
         "type": "object",
-        "required": ["name", "path"],
+        "required": [
+          "name",
+          "path"
+        ],
         "properties": {
-          "name": { "type": "string", "minLength": 1 },
-          "path": { "type": "string", "minLength": 1 }
+          "name": {
+            "type": "string",
+            "minLength": 1
+          },
+          "path": {
+            "type": "string",
+            "minLength": 1
+          },
+          "verify": {
+            "type": "string",
+            "description": "Comando que corre la puerta de esta raíz —pruebas, lint, typecheck, build—, tal como se invoca desde ella. Sin él, quien verifica tiene que descubrirlo leyendo el repositorio en cada tarea, y eso es trabajo de modelo repetido para siempre sobre una respuesta que no cambia."
+          }
         },
         "additionalProperties": false
       }
     },
     "runner": {
       "type": "object",
-      "required": ["maxTaskHours", "humanCheckpointBetweenMilestones", "commitPerTask", "allowPush"],
+      "required": [
+        "maxTaskHours",
+        "humanCheckpointBetweenMilestones",
+        "commitPerTask",
+        "allowPush"
+      ],
       "properties": {
-        "maxTaskHours": { "type": "number", "exclusiveMinimum": 0 },
-        "humanCheckpointBetweenMilestones": { "type": "boolean" },
-        "commitPerTask": { "type": "boolean" },
-        "allowPush": { "type": "boolean" }
+        "maxTaskHours": {
+          "type": "number",
+          "exclusiveMinimum": 0
+        },
+        "humanCheckpointBetweenMilestones": {
+          "type": "boolean"
+        },
+        "commitPerTask": {
+          "type": "boolean"
+        },
+        "allowPush": {
+          "type": "boolean"
+        }
       },
       "additionalProperties": false
     }

package/flows/system/incident-review/flow.json

@@ -60,7 +60,7 @@
     "No se atribuye responsabilidad a personas; se analizan condiciones, no culpables.",
     "Una causa sin evidencia se declara como hipótesis, aunque cerrar el informe sea más cómodo.",
     "Ninguna comunicación a clientes ni notificación regulatoria sale de este recorrido: se propone y la decide una persona.",
-    "Los seguimientos quedan en el INBOX sin promover; convertirlos en trabajo es una decisión humana."
+    "Los seguimientos quedan en el INBOX sin promover y cada uno dice si previene la repetición o sólo acelera la detección; convertirlos en trabajo es una decisión humana, y esa distinción es lo único que permite ordenarlos."
   ],
   "completion": [
     "La línea de tiempo está sostenida por registros, no por reconstrucción de memoria.",

package/flows/system/incident-review/learning/HISTORY.md

@@ -4,3 +4,4 @@ Registrar únicamente cambios aprobados por una persona, con fecha, la corrida q
 
 | Fecha | Propuesta | Decisión | Aprobó | Cambio aplicado |
 |---|---|---|---|---|
+| 2026-08-30 | `2026-08.md` | aplicada | Manuel Pinzon | Un guardrail, y sólo su texto: cada seguimiento dice si previene la repetición o sólo acelera la detección. |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ingeniomaps/cauce",
-  "version": "0.53.1",
+  "version": "0.54.0",
   "description": "Sistema portable de planificación y ejecución verificable para cualquier proyecto",
   "keywords": [
     "planning",
@@ -33,8 +33,9 @@
     "test": "bash test/tools/hooks-smoke.sh && node --test \"test/**/*.test.js\"",
     "coverage": "bash test/tools/coverage.sh",
     "coverage:update": "bash test/tools/coverage.sh --update",
-    "dead-imports": "node test/tools/dead-imports.js",
-    "ci": "npm run check && npm run automation:check && npm run integration:check && npm run coverage",
+    "dead-code": "node test/tools/dead-code.js",
+    "dead-code:engine": "node test/tools/dead-code.js --engine",
+    "ci": "npm run check && npm run automation:check && npm run integration:check && npm run dead-code:engine && npm run coverage",
     "prepublishOnly": "npm run ci"
   },
   "engines": {

package/template/planning/rules/system/conduct.md

@@ -28,6 +28,25 @@ deja el pedido sin resolver: es la mitad barata del trabajo.
 - Nombrar lo que falta no reemplaza a entregar lo que se puede: lo que sí se pudo establecer se escribe
   igual, aunque quede parcial y marcado como tal.
 
+El primer punto tiene un borde que se cruza sin notarlo. Faltar contexto es no saber **cómo**: qué
+formato, qué borde, qué valor. Eso se supone y se entrega marcado, porque suponerlo mal cuesta una
+corrección. No saber **qué se quiere ni para qué** es otra cosa, y ahí el borrador deja de ser barato:
+todo lo que se construya encima hereda la suposición, y quien lo reciba no tiene cómo ver que el
+objetivo era supuesto. El para qué tampoco es adorno — es lo que permite reconocer una respuesta
+correcta, y sin él se cumple la aceptación al pie y no sirve.
+
+Lo que separa los dos casos es si la respuesta existe en algún lado. Si está escrita —en la aceptación,
+en el criterio que la unidad cita, en el contexto de la épica, en el código— entonces la unidad se
+entiende y lo que falta es leerla: eso lo hace quien ejecuta y no interrumpe a nadie. Si contestarla
+exigiera inventarla, no hay nada que leer y es una decisión que el cargo no tiene: se escala con la
+pregunta exacta y con qué se desbloquea al contestarla. Es el mismo criterio con que R6 reparte el
+hallazgo que aparece a mitad de camino —quién puede resolverlo, no cuán grave parece—, aplicado antes
+de empezar en vez de durante.
+
+Aclarar la redacción no es ampliar el alcance, y partir tampoco: una unidad que no se entiende porque
+son dos resultados con vidas distintas se entiende partida, y eso no pide ninguna autoridad que el
+cargo no tenga (R17). La pide elegir el objetivo, que es lo único que no se puede suponer.
+
 Esto no afloja ningún límite: no promover, no prometer fechas, no inventar evidencia y no exceder la
 autoridad del cargo siguen siendo absolutos. Lo que se cierra es la salida de cumplirlos sin entregar.
 

package/template/tools/ops.js

@@ -4,7 +4,7 @@
 // Punto de entrada estable del proyecto: nadie —ni una persona ni un agente— necesita saber de
 // dónde sale el motor. Viene de la dependencia npm, que el lockfile versiona.
 //
-// Nada de `require`, `__dirname` ni `import`: el shim se instala dentro del proyecto y hereda el
+// Nada de `require`, `__dirname` ni un `import` estático: el shim se instala en el proyecto y hereda el
 // `type` de su `package.json`, así que el mismo archivo se carga como CommonJS en un repo y como
 // ESM en el de al lado. `import()` dinámico y `process.argv[1]` son las dos únicas formas que
 // existen bajo los dos cargadores; cualquier otra revienta en la mitad de los proyectos —`require
@@ -16,7 +16,7 @@ const root = self.slice(0, self.lastIndexOf('/', self.lastIndexOf('/') - 1)) ||
 
 // El shim sabe dónde vive; quien lo invoca, no. En modo sidecar se lo llama desde la carpeta de la
 // compañía —`node <empresa>-ops/tools/ops.js …`— y sin esto cada comando resolvería su raíz contra
-// el cwd: `agents list` y `team list` devolvían vacío en vez de fallar.
+// el cwd: `agents list` y `flow list` devolvían vacío en vez de fallar.
 process.env.OPS_ROOT = process.env.OPS_ROOT || root
 
 // El especificador se resuelve contra este archivo, así que sube a `<raíz>/node_modules` igual que