@dforce2055/dai 0.4.0 → 0.5.0

package/CHANGELOG.md

@@ -3,6 +3,25 @@
 Formato basado en [Keep a Changelog](https://keepachangelog.com/). Versionado semver
 (ver `VERSION`).
 
+## [0.5.0] — 2026-07-10
+
+**`archive` en el flujo** ([ADR-0011](docs/adr/0011-archive-gate-de-aprobacion.md)): cerrar el CÓMO
+del lado de las specs canónicas, atado a la aprobación de la PR.
+
+### Agregado
+- **`dai archive [<change>]`**: funde los delta specs del change en las specs canónicas
+  (`openspec/specs/`) y lo archiva. Lo corre el **aprobador** de la PR, en la branch, al aprobar
+  (el fold viaja en la PR → elude la base protegida). Detecta el change activo por su `implements.yaml`
+  o le pasás el nombre; envuelve `openspec archive --yes` (mecánico → comando, no skill). Flag `--skip-specs`.
+
+### Cambiado
+- **`dai check` y `dai ls` saltean `openspec/changes/archive/`**: un change shippeado ya no genera
+  ⚠️ de drift falso ni aparece en el listado. `discoverImplements` acepta `includeArchived` (default
+  `true`); `check`/`ls` lo pasan `false`. `stamp`/`done` mantienen el default (lo necesitan post-merge).
+
+### Interno
+- **110 tests** (+1 desde 0.4.0: filtro `includeArchived`).
+
 ## [0.4.0] — 2026-07-10
 
 **Versionado y upgrade** ([ADR-0010](docs/adr/0010-versionado-y-upgrade.md)): mantené tu repo al
@@ -147,6 +166,7 @@ ClickUp y Jira Cloud.
 - Tests de las rutas de red (jira/clickup/forge) con `fetch` mockeado. Sin links rotos;
   `files` de npm sin tests ni secretos.
 
+[0.5.0]: https://github.com/dforce2055/dai/releases/tag/v0.5.0
 [0.4.0]: https://github.com/dforce2055/dai/releases/tag/v0.4.0
 [0.3.1]: https://github.com/dforce2055/dai/releases/tag/v0.3.1
 [0.3.0]: https://github.com/dforce2055/dai/releases/tag/v0.3.0

package/README.md

@@ -167,6 +167,7 @@ flowchart TD
 | `dai pr [--assignee u] [--base b] [--draft] [--yes]` | crea TU PR/MR precargada: pregunta la branch base (default `main`), muestra el texto y confirma antes de publicar |
 | `dai stamp` | estampa la cobertura inversa en el tracker (branch + commit-ancla) |
 | `dai done [--base main] [--force]` | cierra la US: vuelve a la base, `fetch --prune` + `pull`, y borra la branch local **si está mergeada** (chequeo estricto; `--force` la borra igual). Redes: no estar en la base, sin cambios sueltos, sin commits sin pushear |
+| `dai archive [<change>] [--skip-specs]` | **funde los delta specs del change en las specs canónicas** (`openspec/specs/`) y lo archiva. Lo corre el **aprobador** de la PR (gate de aprobación, [ADR-0011](docs/adr/0011-archive-gate-de-aprobacion.md)); detecta el change activo o le pasás el nombre. Envuelve `openspec archive` |
 | `dai forge comment <ref> --body-file <f>` · `dai forge pr <ref>` | comentar / leer una PR/MR (GitHub/GitLab) |
 | `dai ac-hash <us.md>` | calcula el hash de los criterios de aceptación de una US |
 | `dai doctor` · `dai docs <dest>` · `dai version` | diagnóstico del entorno (incluye **version-drift** del scaffold) · copiar la doc · versión (`dai version` avisa si tu repo quedó atrás) |

package/VERSION

@@ -1 +1 @@
-0.4.0
+0.5.0

package/cli/dai.mjs

@@ -71,7 +71,7 @@ function cmdAcHash(arg) {
 // ── ls ──────────────────────────────────────────────────────────────────────
 function cmdLs(opts) {
   const root = opts.root || process.cwd();
-  const found = discoverImplements(root);
+  const found = discoverImplements(root, { includeArchived: false });
   const rows = [];
   for (const f of found) {
     for (const im of f.implements || []) {
@@ -170,7 +170,7 @@ function gitCommit() { try { return git(["rev-parse", "HEAD"]); } catch { return
 async function cmdCheck() {
   loadEnv();
   const adapter = getAdapter(process.env);
-  const found = discoverImplements(process.cwd());
+  const found = discoverImplements(process.cwd(), { includeArchived: false });
   let worst = 0, n = 0;
   const atrasadas = [];
   for (const f of found) for (const im of f.implements || []) {
@@ -682,6 +682,39 @@ function cmdDocs(dest) {
   ok(`documentación copiada a ${dest}`);
 }
 
+// ── archive: funde los delta specs del change en los specs canónicos y lo archiva ─
+// Lo corre el APROBADOR de la PR, en la branch, al aprobar: es el gate de aprobación
+// (el fold va atado a la aprobación, no a la autoría). Envuelve `openspec archive
+// <change> --yes` — mecánico, va al CLI y no a una skill (ADR-0002). El fold queda
+// sin commitear para que el aprobador lo revise y lo incluya en la PR antes de mergear.
+function cmdArchive(changeArg, opts) {
+  const repo = process.cwd();
+  if (!existsSync(join(repo, "openspec"))) fail("no hay OpenSpec en este repo (falta openspec/). Nada que archivar.", 2);
+
+  let change = changeArg;
+  if (!change) {
+    // Detectar el change ACTIVO (no archivado) por su implements.yaml.
+    const active = discoverImplements(repo, { includeArchived: false })
+      .filter((f) => /[/\\]openspec[/\\]changes[/\\]/.test(f.path))
+      .map((f) => basename(dirname(f.path)));
+    const uniq = [...new Set(active)];
+    if (uniq.length === 0) fail("no encontré un change activo para archivar (¿ya está archivado, o falta `dai link-us`?).", 2);
+    if (uniq.length > 1) fail(`hay varios changes activos: ${uniq.join(", ")}.\n  Pasá cuál: dai archive <change>`, 2);
+    change = uniq[0];
+  }
+
+  info(`archivando '${change}' — funde los delta specs en openspec/specs/ y mueve el change a archive/…`);
+  const args = ["archive", change, "--yes"];
+  if (opts.skipSpecs) args.push("--skip-specs");
+  try {
+    execFileSync(npmBin("openspec"), args, { stdio: "inherit", cwd: repo });
+  } catch (e) {
+    fail(`openspec archive falló (¿tasks incompletas? ¿change inexistente?): ${String(e.message).split("\n")[0]}`, 1);
+  }
+  process.stdout.write("\n");
+  ok(`'${change}' archivado. Revisá los cambios (specs fundidas + change en archive/) y commitealos en la PR antes de mergear.`);
+}
+
 // ── sync: refresca las copias scaffoldeadas a la versión del CLI (ADR-0010) ───
 // Las copias (skills, constitución, templates, PR template) son un CACHÉ derivable
 // del CLI: `dai sync` las re-genera a la versión instalada, aditivo (no pisa la
@@ -864,6 +897,7 @@ switch (cmd) {
   case "publish": cmdPublish(pos[0]).catch((e) => fail(String(e.message))); break;
   case "pr":      cmdPr(opts).catch((e) => fail(String(e.message))); break;
   case "done":    cmdDone(opts); break;
+  case "archive": cmdArchive(pos[0], opts); break;
   case "install": cmdInstall(opts).catch((e) => fail(String(e.message))); break;
   case "init":    cmdInit(pos[0], opts).catch((e) => fail(String(e.message))); break;
   case "sync":    cmdSync(pos[0], opts); break;
@@ -882,6 +916,7 @@ switch (cmd) {
       "  check                        compara vs la US viva → atrasado (ADR-0003)\n" +
       "  stamp                        estampa la cobertura en el tracker (ADR-0005)\n" +
       "  done [--base main] [--force] cierra la US: vuelve a la base, actualiza y borra la branch local (si está mergeada)\n" +
+      "  archive [<change>] [--skip-specs]   funde los delta specs del change en las specs canónicas y lo archiva (lo corre el aprobador en la PR)\n" +
       "  pr [--assignee u] [--base b] [--draft] [--yes]   crea TU PR/MR precargada (muestra + confirma)\n" +
       "  forge comment <ref> --body-file <f> · forge pr <ref>   comentar/leer una PR ajena (github/gitlab)\n\n" +
       "Instalación:\n" +

package/cli/lib/implements.mjs

@@ -71,7 +71,10 @@ export function parseImplements(text) {
 }
 
 // Camina el árbol desde root y devuelve { path, ...parsed } por cada implements.yaml.
-export function discoverImplements(root) {
+// `includeArchived: false` saltea `openspec/changes/archive/` — los changes shippeados
+// no deben aparecer en `check`/`ls` (ADR-0010). Por defecto los incluye (stamp/done los
+// necesitan post-merge).
+export function discoverImplements(root, { includeArchived = true } = {}) {
   const found = [];
   const walk = (dir) => {
     let entries;
@@ -81,7 +84,9 @@ export function discoverImplements(root) {
       let st;
       try { st = statSync(full); } catch { continue; }
       if (st.isDirectory()) {
-        if (!SKIP_DIRS.has(name)) walk(full);
+        if (SKIP_DIRS.has(name)) continue;
+        if (!includeArchived && name === "archive") continue;
+        walk(full);
       } else if (name === "implements.yaml") {
         try {
           found.push({ path: full, ...parseImplements(readFileSync(full, "utf8")) });

package/docs/adr/0011-archive-gate-de-aprobacion.md

@@ -0,0 +1,68 @@
+# ADR-0011 — `archive` es un gate de aprobación (`dai archive`)
+
+- **Estado:** aceptado
+- **Fecha:** 2026-07-10
+- **Decide:** lead / arquitecto de la metodología
+
+## Contexto
+
+OpenSpec tiene `archive`: cuando un change está completo, **funde sus delta specs** en las
+specs canónicas (`openspec/specs/<capability>/spec.md`) y **mueve el change** a
+`openspec/changes/archive/`. Pero en el flujo dai (`link-us → propose → apply/tdd → check
+→ pr → dai-review → merge → stamp → done`) el `archive` **no tenía lugar**: ni cuándo, ni
+quién. Sin eso, las specs vivas del repo quedan desincronizadas de lo shippeado.
+
+Dos preguntas de fondo: **¿quién lo corre?** y **¿en qué momento?**
+
+## Decisión
+
+### `archive` = el acto de aceptar el change en las specs canónicas → lo hace **quien aprueba la PR**
+
+Fundir un change en las specs oficiales es **bendecir** ese cambio. Por eso lo ata a la
+**aprobación**, no a la autoría: lo corre **quien aprueba la PR**, no el autor. Es un
+**gate de aprobación**, no un paso de cierre del autor (un autor no debería fundir sus
+propios cambios en la spec oficial de un equipo).
+
+Ownership por **dial de ceremonia** (N1/N2/N3):
+
+- **N1/N2 — `dai archive` (comando):** el aprobador lo corre **en la branch de la PR** al
+  aprobar → funde los deltas + mueve el change + lo deja **sin commitear** para revisarlo →
+  lo incluye en la PR → mergea normal. **Resuelve solo el problema de base protegida**: el
+  fold viaja dentro de la PR, no hay push directo a `main`.
+- **N3 — workflow de CI opt-in** (roadmap): el merge aprobado dispara el `archive` solo.
+
+### `dai archive` es un **comando**, no una skill
+
+Archivar es **mecánico** (detectar el change + `openspec archive <change> --yes`): sin
+juicio → va al CLI, como `dai stamp`/`check`/`done` (ADR-0002). `dai-review` es skill
+porque revisar código sí requiere juicio; archivar no. `dai archive` **envuelve**
+`openspec archive`, detectando el change activo por su `implements.yaml` (reusa
+`discoverImplements`); si hay varios, pide el nombre.
+
+### `dai check` y `dai ls` saltean `openspec/changes/archive/`
+
+Un change archivado está **shippeado**: no debe aparecer en `check` (daría un ⚠️ de drift
+falso si el PO edita esa US después) ni en `ls`. `discoverImplements` acepta
+`includeArchived` (default `true`); `check`/`ls` lo pasan en `false`. **`dai stamp`/`done`/
+`archive` mantienen el default** — sí necesitan encontrar el `implements.yaml` (p. ej.
+`stamp` post-merge, cuando el change ya se archivó en la branch).
+
+## Consecuencias
+
+- ✅ El `archive` queda **atado a la aprobación** — las specs canónicas reflejan solo lo bendecido.
+- ✅ La variante `dai archive` (N1/N2) **elude la base protegida** sin tokens ni CI: el fold entra por la PR.
+- ✅ `check`/`ls` dejan de reportar ruido de changes shippeados.
+- ✅ Coherente con el ADN: es una **herramienta opcional** (no obliga), mecánica → CLI.
+- ⚠️ Edge: en PRs desde **fork**, el aprobador puede no tener push a la branch del autor →
+  ahí cae al workflow de CI, o lo corre el autor.
+- ⚠️ El **nudge** (que `/dai-review` recuerde archivar al aprobar) y el **workflow de CI**
+  quedan como follow-up (roadmap), igual que documentar el paso en `SCRUM-CON-IA`.
+
+## Alternativas consideradas
+
+- **Que lo corra el autor, en el PR** — descartado: fundir en la spec oficial es un acto de
+  aprobación; el autor no debería bendecir su propio cambio. Va atado a quien aprueba.
+- **Post-merge sobre la base (commit directo)** — descartado como default: choca con branch
+  protection (push directo a `main`). Queda para el workflow de CI (con follow-up PR o App token).
+- **Una skill `/dai-archive`** — descartado: archivar es mecánico, no necesita un LLM (ADR-0002).
+  El comando puede invocarse desde el asistente igual, sin ser skill.

package/docs/adr/README.md

@@ -16,6 +16,7 @@ decisión cambia, se escribe un ADR nuevo que supersede al viejo. Molde en
 | [0008](0008-estrategia-de-i18n.md) | Estrategia de i18n: fuente única (español) + traducciones derivadas, `DAI_LANG` en el CLI, por fases | propuesto |
 | [0009](0009-adaptador-cursor.md) | Adaptador nativo para Cursor (skills + rules) con `dai init`/`install`/`doctor` | propuesto |
 | [0010](0010-versionado-y-upgrade.md) | Versionado y upgrade: compatibilidad por semver, `doctor` version-drift, `dai sync` aditivo | propuesto |
+| [0011](0011-archive-gate-de-aprobacion.md) | `archive` es un gate de aprobación: `dai archive` (comando) lo corre el aprobador; `check`/`ls` saltean `archive/` | aceptado |
 
 > Estas son las decisiones que cierran las "Decisiones abiertas" de
 > [`METODOLOGIA.md §7`](../METODOLOGIA.md) y las enmiendas al

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dforce2055/dai",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "Metodología de desarrollo asistido por IA — CLI de acciones deterministas (trazabilidad QUÉ↔CÓMO).",
   "repository": { "type": "git", "url": "git+https://github.com/dforce2055/dai.git" },
   "homepage": "https://dforce2055.github.io/dai/",