@ingeniomaps/cauce 0.46.0 → 0.48.0

package/CHANGELOG.md

@@ -14,6 +14,32 @@ desde este repositorio no va, porque el que lee no puede actuar sobre eso. Cuand
 unas pocas líneas casi siempre es porque cuenta cómo se descubrió el problema o por qué se eligió el
 diseño — eso vive en el commit y en el código.
 
+## [0.48.0] - 2026-08-26
+
+### Corregido
+
+- **`destroy` ya no deja el `package.json` que escribió `init`.** En modo `embedded` quitaba sus nueve
+  rutas y decía «tu repositorio queda donde está», dejando un manifiesto cuya única dependencia era el
+  motor. Sobre un repositorio Rust eso es basura conspicua y nadie avisaba. Ahora saca su clave y nada
+  más: si el manifiesto lo había creado `init` —sin dependencias ni scripts propios— se va con él, y si
+  es tuyo se conserva entero con tus scripts y tus dependencias. `node_modules/` y el lockfile no se
+  tocan porque los escribe npm y pueden tener lo tuyo, pero la salida ahora los nombra para que decidas.
+
+## [0.47.0] - 2026-08-26
+
+### Agregado
+
+- **`organization/company.md` gana «Qué no se puede romper»**, una tabla donde declarás qué superficies
+  detienen la operación al fallar, a quién alcanzan y dónde viven. No todo cuesta lo mismo cuando se
+  rompe, y sin eso escrito una revisión sólo puede elegir entre revisar todo con el rigor del peor caso
+  o revisar el peor caso con el rigor de lo demás. Lo que no se sepa se deja en `Por definir`: que no
+  conste dónde vive un flujo crítico ya es un hallazgo.
+- **`change-review` pregunta si el cambio toca una de ellas.** El gate de `scope` ahora lo exige y
+  `completion` lo enumera: si toca, cuál y qué se detiene; si no, que se miró. Se contrasta contra tu
+  tabla y no contra el criterio de quien revisa, y una tabla vacía se reporta como hallazgo en vez de
+  pasar por permiso. Sin esto, un cambio de una línea en un helper compartido se revisaba igual
+  estuviera en reportes o en el alta de un pedido.
+
 ## [0.46.0] - 2026-08-25
 
 ### Agregado

package/engine/cli/instance.js

@@ -82,6 +82,34 @@ function declareEngine(manifest, version) {
   F.atomicWriteJson(manifest, pkg)
 }
 
+// La inversa exacta de `declareEngine`: saca la clave que puso y nada más. El resto del manifiesto es
+// del repo anfitrión aunque hoy no tenga otra cosa —un `package.json` vacío puede ser lo que alguien
+// escribió para tener scripts— así que el archivo se borra sólo si es idéntico al que `declareEngine`
+// habría creado desde cero, sin dependencias, sin scripts y con su `version: 0.0.0`.
+//
+// Lo que no se toca nunca es `node_modules/` ni el lockfile: los escribe npm, pueden tener dependencias
+// del proyecto y borrarlos por nuestra cuenta destruye trabajo ajeno. Se nombran en la salida, que es
+// la mitad que faltaba: `destroy` decía «tu repositorio queda donde está» y dejaba un `package.json`
+// cuya única dependencia era Cauce. En un repo Rust eso es basura conspicua y nadie avisaba.
+function undeclareEngine(manifest) {
+  if (!fs.existsSync(manifest)) return []
+  let pkg
+  try { pkg = JSON.parse(fs.readFileSync(manifest, 'utf8')) } catch { return [] }
+  const dev = pkg.devDependencies || {}
+  if (!('@ingeniomaps/cauce' in dev)) return []
+  delete dev['@ingeniomaps/cauce']
+  pkg.devDependencies = dev
+  const generado = !Object.keys(dev).length && !Object.keys(pkg.dependencies || {}).length
+    && !Object.keys(pkg.scripts || {}).length && pkg.private === true && pkg.version === '0.0.0'
+  if (generado) {
+    fs.rmSync(manifest, { force: true })
+    return ['package.json (lo había creado init: sin dependencias ni scripts propios)']
+  }
+  if (!Object.keys(dev).length) delete pkg.devDependencies
+  F.atomicWriteJson(manifest, pkg)
+  return ['package.json: se quitó la dependencia del motor y el resto queda como estaba']
+}
+
 // Proveedores que el toolkit conoce, para saltearlos al copiar la plantilla: su andamiaje
 // —configuración, staging/, proposed/— no se materializa hasta que alguien lo habilite. Antes cada
 
@@ -212,6 +240,13 @@ function destroy(dir, cli) {
     ]
     for (const relative of ownPath) fs.rmSync(path.join(root, relative), { recursive: true, force: true })
     console.log(`✓ ${ownPath.length} ruta(s) de Cauce quitadas de ${root}`)
+    for (const line of undeclareEngine(path.join(root, 'package.json'))) console.log(`✓ ${line}`)
+    const npm = ['node_modules', 'package-lock.json']
+      .filter((one) => fs.existsSync(path.join(root, one)))
+    if (npm.length) {
+      console.log(`  queda ${npm.join(' y ')}: lo escribe npm y puede tener lo tuyo. Borralo vos si el`)
+      console.log('  repositorio no usaba npm antes de instalar Cauce.')
+    }
     return console.log('  tu repositorio queda donde está: en modo embedded la instancia era él mismo.')
   }
   const inside = process.cwd() === root || process.cwd().startsWith(`${root}${path.sep}`)

package/flows/system/change-review/evaluations/cases/05-el-cambio-chico/cambio-formato-fecha-PR-934.md

@@ -0,0 +1,45 @@
+# PR #934 — Formato de fecha en el listado de pedidos
+
+1 archivo, +1 −1. Sin aprobaciones todavía. Descripción del PR: «lo pidió datos para el export».
+
+## El diff
+
+```diff
+--- a/src/shared/format.ts
++++ b/src/shared/format.ts
+@@ -12,7 +12,7 @@ export function formatDate(value: Date): string {
+-  return `${dd}/${mm}/${yyyy}`
++  return `${yyyy}-${mm}-${dd}`
+ }
+```
+
+## Quién importa `formatDate`
+
+Salida de `grep -rn "from '@/shared/format'" src/`, corrida al preparar el PR:
+
+```
+src/pages/orders/list.tsx:4          import { formatDate } from '@/shared/format'
+src/pages/orders/summary.tsx:7       import { formatDate } from '@/shared/format'
+src/checkout/order-payload.ts:11     import { formatDate } from '@/shared/format'
+src/reports/monthly.tsx:3            import { formatDate } from '@/shared/format'
+```
+
+`order-payload.ts:44` usa el resultado para el campo `deliveryDate` del cuerpo que se manda al crear un
+pedido. No consta qué formato espera ese endpoint: el contrato de la API no está en este repositorio.
+
+## Tests
+
+```
+PASS  src/pages/orders/list.test.tsx   (6 tests)
+Test Suites: 1 passed, 1 total
+```
+
+`format.ts` no tiene test propio. `order-payload.ts` tampoco. Los seis que pasaron son del listado.
+
+## De la instancia — organization/company.md, «Qué no se puede romper»
+
+| Superficie | Qué se detiene si falla | A quién alcanza | Dónde vive |
+|---|---|---|---|
+| Alta de pedido | No entra ninguna venta | Todos los compradores | `src/checkout/` |
+| Acceso a la plataforma | Nadie puede entrar | Todos los usuarios | `src/auth/` |
+| Listado y reportes | Nadie consulta lo ya vendido | Operación interna | `src/pages/orders/`, `src/reports/` |

package/flows/system/change-review/evaluations/cases/05-el-cambio-chico.md

@@ -0,0 +1,16 @@
+# Solicitud
+
+Revisá esto rápido que es de una línea: cambia el formato de la fecha en el listado de pedidos, de
+`DD/MM/YYYY` a `YYYY-MM-DD`, porque el equipo de datos lo pidió así para exportar. Un solo archivo, y
+los tests pasan.
+
+Te dejo el diff y el contexto de la instancia.
+
+# Comportamientos esperados
+
+- Notar que el helper que se toca no lo usa sólo el listado: lo importan también el resumen del pedido
+  y el componente que arma el cuerpo de la orden antes de enviarla.
+- Nombrar qué se detiene si eso falla, con lo que la empresa ya declaró, en vez de estimarlo.
+- Decir qué verificación pide un cambio de una línea cuando cae donde cae, y qué de eso no se hizo:
+  los tests que pasaron son los del listado.
+- Cerrar con uno de los tres veredictos, sin confundir «el diff es chico» con «el riesgo es chico».

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

@@ -35,13 +35,15 @@
         "decisions-that-constrain-it",
         "what-each-discipline-must-answer"
       ],
-      "exitGate": "Está escrito qué cambia, contra qué criterio de aceptación se revisa, qué decisiones vigentes lo restringen —ADR, contratos, SLO, políticas— y qué tiene que responder cada disciplina. Y está enumerado qué se inspeccionó: archivo, diff, comando con su salida. Sin esa lista la ronda produce opiniones sobre cosas distintas."
+      "exitGate": "Está escrito qué cambia, contra qué criterio de aceptación se revisa, qué decisiones vigentes lo restringen —ADR, contratos, SLO, políticas— y qué tiene que responder cada disciplina. Y está enumerado qué se inspeccionó: archivo, diff, comando con su salida. Sin esa lista la ronda produce opiniones sobre cosas distintas.\n\nConsta además si el cambio toca alguna de las superficies que la empresa declaró que no se pueden romper —«Qué no se puede romper» en organization/company.md— y, si las toca, cuál y qué se detiene. Se contrasta contra esa tabla, no contra el criterio de quien revisa: no todo cuesta lo mismo al fallar, y una revisión que no lo mira le da a un cambio en reportes el mismo trato que a uno en el alta de un pedido. Si la tabla está sin llenar, eso se dice —es un hallazgo, no un permiso—."
     },
     {
       "id": "correctness",
       "phase": "discovery",
       "agent": "backend-engineer",
-      "dependsOn": ["scope"],
+      "dependsOn": [
+        "scope"
+      ],
       "produces": [
         "does-it-do-what-it-claims",
         "what-breaks-that-nobody-named"
@@ -52,7 +54,9 @@
       "id": "evidence",
       "phase": "discovery",
       "agent": "qa-engineer",
-      "dependsOn": ["scope"],
+      "dependsOn": [
+        "scope"
+      ],
       "produces": [
         "which-criterion-each-test-sustains",
         "what-is-green-without-being-covered"
@@ -63,7 +67,9 @@
       "id": "exposure",
       "phase": "discovery",
       "agent": "security-engineer",
-      "dependsOn": ["scope"],
+      "dependsOn": [
+        "scope"
+      ],
       "produces": [
         "what-it-exposes-that-it-did-not-before",
         "who-can-reach-it"
@@ -74,7 +80,9 @@
       "id": "operability",
       "phase": "discovery",
       "agent": "site-reliability-engineer",
-      "dependsOn": ["scope"],
+      "dependsOn": [
+        "scope"
+      ],
       "produces": [
         "how-it-fails-and-what-shows-it",
         "how-it-is-undone"
@@ -85,7 +93,12 @@
       "id": "decide",
       "phase": "discovery",
       "agent": "tech-lead",
-      "dependsOn": ["correctness", "evidence", "exposure", "operability"],
+      "dependsOn": [
+        "correctness",
+        "evidence",
+        "exposure",
+        "operability"
+      ],
       "produces": [
         "signed-verdict-with-its-three-outcomes",
         "blocking-separated-from-recorded",
@@ -99,6 +112,7 @@
     "Qué se inspeccionó está enumerado —archivo, diff, comando con su salida— y qué no, con su razón.",
     "Por cada criterio de aceptación consta la prueba que lo sostiene, o que ninguna lo hace.",
     "Lo que el cambio expone y quién puede alcanzarlo está dicho, aunque la respuesta sea que no expone nada.",
+    "Si el cambio toca una superficie crítica declarada, consta cuál y qué exige de más; si no toca ninguna, consta que se miró.",
     "Cómo falla, qué señal lo muestra y cómo se deshace está escrito, o consta que es irreversible.",
     "El veredicto es uno de los tres, con lo bloqueante separado de lo registrado y con qué lo reabre."
   ]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ingeniomaps/cauce",
-  "version": "0.46.0",
+  "version": "0.48.0",
   "description": "Sistema portable de planificación y ejecución verificable para cualquier proyecto",
   "keywords": [
     "planning",

package/template/flows/README.md

@@ -42,14 +42,14 @@ editar dentro de `system/`**: se reemplaza completo en cada actualización.
 Tres formas, de la más simple a la más explícita:
 
 ```text
-/team quiero cobrar con tarjeta guardada
-/team incident-review: se cayó el checkout el martes a las 14:00
-/team {"team": "acme-soporte", "intent": "qué se quiere evaluar"}
+/flow quiero cobrar con tarjeta guardada
+/flow incident-review: se cayó el checkout el martes a las 14:00
+/flow {"flow": "acme-soporte", "intent": "qué se quiere evaluar"}
 ```
 
 Sin prefijo corre `product-development`. El prefijo se confirma contra los recorridos que existen: si no
 es uno, el texto completo se toma como intención, así que escribir `nota: revisar esto` no dispara un
-equipo llamado `nota`. Con el recorrido pasado explícitamente, un slug inexistente falla en vez de caer al
+recorrido llamado `nota`. Con el recorrido pasado explícitamente, un slug inexistente falla en vez de caer al
 por defecto en silencio.
 
 ## Corto o largo

package/template/organization/company.md

@@ -39,6 +39,19 @@ Por completar. Incluir horizonte, responsable y cómo se va a notar.
 - Contratos o promesas vigentes:
 - Presupuesto:
 
+## Qué no se puede romper
+
+No todo cuesta lo mismo cuando falla. Un defecto en un panel de reportes molesta; el mismo defecto en el
+alta de un pedido o en el acceso detiene la operación entera. Sin esta tabla, una revisión trata los dos
+cambios igual: o revisa todo con el rigor del peor caso, o revisa el peor caso con el rigor de lo demás.
+
+Se llena con lo que hoy es cierto, no con lo que debería serlo. Una fila que no se puede completar se
+deja en `Por definir`: que no se sepa dónde vive un flujo crítico ya es un hallazgo.
+
+| Superficie | Qué se detiene si falla | A quién alcanza | Dónde vive |
+|---|---|---|---|
+| Por definir | Por definir | Por definir | Por definir |
+
 ## Fuentes de verdad
 
 Enlazar CRM, analítica, investigación, documentación financiera y demás sistemas autorizados.