specatlas 0.1.8 → 0.1.10

package/dist/{approvals-HCBC7QDM-PSBJC7EV.js → approvals-4QHIT7F6-JAI5IHNX.js}

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   signApproval
-} from "./chunk-IFALGLUJ.js";
+} from "./chunk-X4FIHYOJ.js";
 import "./chunk-DKFIPJ74.js";
 export {
   signApproval

package/dist/bin.js

@@ -8,7 +8,7 @@ import {
   sha256,
   shortHash,
   signApproval
-} from "./chunk-IFALGLUJ.js";
+} from "./chunk-X4FIHYOJ.js";
 import {
   __commonJS,
   __toESM,
@@ -9864,6 +9864,14 @@ function verifyApproval(change, approvals, cfg, specContent) {
   if (current !== approval.hash) return { status: "stale", approvedBy: approval.by, approvedAt: approval.at };
   return { status: "valid", approvedBy: approval.by, approvedAt: approval.at };
 }
+function requiresMockups(meta, cfg) {
+  if (meta?.mockups === "required") return true;
+  if (meta?.mockups === "skip") return false;
+  return cfg.gates.mockup.require_approval;
+}
+function mockupOverride(change) {
+  return (change.meta?.overrides ?? []).some((override) => override.gate === "mockup");
+}
 function deriveState(input) {
   const { change, cfg, approval, blockingFindings } = input;
   const lane = change.meta?.lane ?? cfg.lanes.default;
@@ -9896,6 +9904,14 @@ function deriveState(input) {
     blockedBy.push(`${blockingFindings} hallazgo(s) bloqueante(s)`);
     return { state: "spec_draft", blockedBy, nextAction: next(`satlas validate --change ${change.slug}`, "Corregir los hallazgos de la especificaci\xF3n"), progress };
   }
+  if ((approval.status === "missing" || approval.status === "stale") && requiresMockups(change.meta, cfg) && input.mockupsReady !== true && !mockupOverride(change)) {
+    return {
+      state: "awaiting_mockups",
+      blockedBy: ["mockups requeridos y no listos"],
+      nextAction: next(`/satlas-mockup ${change.slug}`, "Generar los mockups (contrato visual) antes de aprobar", true),
+      progress
+    };
+  }
   if (approval.status === "missing" || approval.status === "stale") {
     blockedBy.push(approval.status === "stale" ? "la firma de la spec qued\xF3 obsoleta (el archivo cambi\xF3)" : "la spec no est\xE1 aprobada");
     const presented = change.presentationPath !== void 0;
@@ -9926,6 +9942,7 @@ function stateLabel(state) {
   const labels = {
     draft: "borrador",
     spec_draft: "spec en borrador",
+    awaiting_mockups: "esperando mockups",
     awaiting_approval: "esperando aprobaci\xF3n",
     approved: "aprobado",
     planned: "planificado",
@@ -10498,6 +10515,7 @@ function changeMetaYaml(meta) {
   if (meta.risk) doc["risk"] = meta.risk;
   if (meta.created) doc["created"] = meta.created;
   if (meta.owner) doc["owner"] = meta.owner;
+  if (meta.mockups) doc["mockups"] = meta.mockups;
   return `# Estado del cambio. La fase se DERIVA de los artefactos; aqu\xED solo hechos.
 ` + stringifyYaml3(doc, { lineWidth: 120 });
 }
@@ -10783,7 +10801,7 @@ async function approveFromGithub(opts) {
   if (!by) {
     return { approved: false, label, issueNumber: number, diagnostics: [diag("ATLAS-GH-011", "error", "No se pudo determinar qui\xE9n aprueba (--by ni `gh api user`)")] };
   }
-  const { signApproval: signApproval2 } = await import("./approvals-HCBC7QDM-PSBJC7EV.js");
+  const { signApproval: signApproval2 } = await import("./approvals-4QHIT7F6-JAI5IHNX.js");
   const signed = await signApproval2({
     root: opts.root,
     artifact: path5.posix.join("changes", opts.slug, "spec.md"),
@@ -11302,6 +11320,28 @@ async function captureMockups(root, slug, manifest, now = /* @__PURE__ */ new Da
   void now;
   return { screenshots, diagnostics };
 }
+async function mockupsReady(root, slug, change) {
+  const check = await checkMockups(root, slug, change);
+  return Boolean(check.manifest && check.manifest.screens.length > 0 && !check.stale);
+}
+async function setMockupRequirement(root, slug, value) {
+  const file = path8.join(root, ".sdd", "changes", slug, "meta.yaml");
+  const raw = await readTextIfExists(file) ?? `schema_version: 1
+slug: ${slug}
+lane: standard
+`;
+  const lines = raw.replace(/\r\n?/g, "\n").split("\n");
+  const index = lines.findIndex((line) => /^mockups:/.test(line));
+  if (index >= 0) {
+    lines[index] = `mockups: ${value}`;
+  } else {
+    let last = lines.length;
+    while (last > 0 && (lines[last - 1] ?? "").trim() === "") last -= 1;
+    lines.splice(last, 0, `mockups: ${value}`);
+  }
+  await writeText(file, lines.join("\n"));
+  return { path: file };
+}
 var checkSchema = z5.object({
   id: z5.string().min(1),
   title: z5.string().min(1),
@@ -11853,6 +11893,8 @@ async function generatePresentation(opts) {
   const mockupInfo = await copyMockups(root, change, mockupsCopyDir);
   const proposalRaw = await readTextIfExists(path12.join(change.dir, "proposal.md"));
   const proposalBody = proposalRaw ? parseFrontmatter(proposalRaw).body : "";
+  const approvals = await loadApprovals(path12.join(root, ".sdd"));
+  const approval = verifyApproval(change, approvals.byArtifact, config, deltaContent);
   const labels = labelsFor(language);
   const html = page({
     language,
@@ -11869,7 +11911,8 @@ async function generatePresentation(opts) {
     evidence: new Map((change.verify?.evidence ?? []).map((e) => [e.scenario, e.result])),
     mockups: mockupInfo.screens,
     screenshots: mockupInfo.screenshots,
-    approveHint: `satlas approve ${change.slug} --by "<nombre>" --channel presentation`
+    approveHint: `satlas approve ${change.slug} --by "<nombre>" --channel presentation`,
+    ...approval.status === "valid" && approval.approvedBy ? { approval: { by: approval.approvedBy, at: approval.approvedAt ?? "" } } : {}
   });
   const outFile = path12.join(presentationDir, "index.html");
   await writeText(outFile, html);
@@ -11920,6 +11963,7 @@ function labelsFor(language) {
       mockups: "Mockups",
       approve: "Approval",
       approveText: "This proposal is approved by signing the spec (hash + author). Any later change invalidates the signature.",
+      approveDone: "Approved by {by} on {at}. Any later change invalidates the signature.",
       command: "Command",
       hash: "Hash",
       generated: "Generated",
@@ -11941,6 +11985,7 @@ function labelsFor(language) {
     mockups: "Mockups",
     approve: "Aprobaci\xF3n",
     approveText: "Esta propuesta se aprueba firmando la spec (hash + autor). Cualquier cambio posterior invalida la firma.",
+    approveDone: "Aprobada por {by} el {at}. Cualquier cambio posterior invalida la firma.",
     command: "Comando",
     hash: "Hash",
     generated: "Generado",
@@ -12056,8 +12101,8 @@ function page(input) {
 
   <section>
     <h2>${esc(l.approve)}</h2>
-    <div class="callout">${esc(l.approveText)}</div>
-    <p>${esc(l.command)}: <code>${esc(input.approveHint)}</code></p>
+    ${input.approval ? `<div class="callout" style="border-left-color:#15803d;background:rgba(21,128,61,.12)">\u2714 ${esc(l.approveDone.replace("{by}", input.approval.by).replace("{at}", input.approval.at))}</div>` : `<div class="callout">${esc(l.approveText)}</div>
+    <p>${esc(l.command)}: <code>${esc(input.approveHint)}</code></p>`}
   </section>
 
   <footer>SpecAtlas \xB7 ${esc(input.project)} \xB7 ${esc(input.generatedAt)}</footer>
@@ -13475,10 +13520,26 @@ async function runApprove(ctx) {
       diagnostics: [{ code: "ATLAS-APPROVE-000", severity: "error", message: "Falta --by <nombre>: toda aprobaci\xF3n es nominal y auditada" }]
     };
   }
-  const { root } = await requireWorkspace(ctx);
+  const { root, workspace, config } = await requireWorkspace(ctx);
   const artifact = target.includes("/") || target.includes("\\") ? target : path26.join("changes", target, "spec.md");
   const channelFlag = flagString(ctx.flags, "channel");
   const channel = channelFlag === "presentation" || channelFlag === "editor" || channelFlag === "pr" || channelFlag === "tracker" ? channelFlag : "cli";
+  const change = workspace.changes.find(
+    (candidate) => candidate.slug === target || artifact.includes(`changes/${candidate.slug}/`) || artifact.includes(`changes\\${candidate.slug}\\`)
+  );
+  if (change && requiresMockups(change.meta, config) && !(change.meta?.overrides ?? []).some((override) => override.gate === "mockup") && !await mockupsReady(root, change.slug, change)) {
+    return {
+      exitCode: 1,
+      diagnostics: [
+        {
+          code: "ATLAS-APPROVE-002",
+          severity: "error",
+          message: `El cambio "${change.slug}" exige mockups (meta.yaml: mockups: required) y no est\xE1n listos`,
+          suggestion: `Genera el contrato visual con \`/satlas-mockup ${change.slug}\` y vuelve a aprobar (o registra un override del gate "mockup" en meta.yaml)`
+        }
+      ]
+    };
+  }
   const result = await signApproval({
     root,
     artifact,
@@ -13871,12 +13932,15 @@ async function runMockup(ctx) {
   const { root, workspace, config } = await requireWorkspace(ctx);
   const slug = ctx.positionals[0];
   if (!slug) {
-    return { exitCode: 2, diagnostics: [{ code: "ATLAS-MKP-000", severity: "error", message: "Falta el slug: satlas mockup <slug> [--plan|--check|--capture]" }] };
+    return { exitCode: 2, diagnostics: [{ code: "ATLAS-MKP-000", severity: "error", message: "Falta el slug: satlas mockup <slug> [--plan|--check|--capture|--require]" }] };
   }
   const change = workspace.changes.find((c) => c.slug === slug);
   if (!change) {
     return { exitCode: 2, diagnostics: [{ code: "ATLAS-MKP-000", severity: "error", message: `No existe el cambio "${slug}"` }] };
   }
+  if (flagBool(ctx.flags, "require")) {
+    await setMockupRequirement(root, slug, "required");
+  }
   if (flagBool(ctx.flags, "check")) {
     const result = await checkMockups(root, slug, change);
     const errors = result.findings.filter((d) => d.severity === "error").length;
@@ -14045,7 +14109,14 @@ async function evaluateChange(workspace, config, change, approvals) {
     requireEvidence: config.gates.verify.mode !== "off" && config.gates.verify.require_evidence
   });
   const blocking = lintFindings.filter((d) => d.severity === "error").length + trace.findings.filter((d) => d.severity === "error").length;
-  const state = deriveState({ change, cfg: config, approval, blockingFindings: change.delta ? blocking : 0 });
+  const mockupsAreReady = requiresMockups(change.meta, config) ? await mockupsReady(workspace.root, change.slug, change) : void 0;
+  const state = deriveState({
+    change,
+    cfg: config,
+    approval,
+    blockingFindings: change.delta ? blocking : 0,
+    ...mockupsAreReady !== void 0 ? { mockupsReady: mockupsAreReady } : {}
+  });
   return { change, approval, lintFindings, trace, blocking, state };
 }
 

package/dist/{chunk-IFALGLUJ.js → chunk-X4FIHYOJ.js}

@@ -5,7 +5,7 @@ import {
   writeText
 } from "./chunk-DKFIPJ74.js";
 
-// ../core/dist/chunk-2BFXQ65Q.js
+// ../core/dist/chunk-LMJURGRU.js
 import path from "path";
 import { stringify as stringifyYaml } from "yaml";
 import { createHash } from "crypto";
@@ -70,6 +70,7 @@ var changeMetaSchema = z.object({
   risk: z.enum(["low", "medium", "high"]).optional(),
   created: z.string().optional(),
   owner: z.string().optional(),
+  mockups: z.enum(["required", "skip"]).optional(),
   tracker: z.object({ provider: z.string(), id: z.string() }).optional(),
   paused: z.object({ reason: z.string(), at: z.string(), by: z.string() }).optional(),
   lane_history: z.array(z.object({ from: z.enum(["fix", "standard", "full"]), to: z.enum(["fix", "standard", "full"]), at: z.string(), by: z.string() })).optional(),
@@ -99,6 +100,7 @@ function parseChangeMeta(raw, filePath) {
   if (v.risk !== void 0) meta.risk = v.risk;
   if (v.created !== void 0) meta.created = v.created;
   if (v.owner !== void 0) meta.owner = v.owner;
+  if (v.mockups !== void 0) meta.mockups = v.mockups;
   if (v.tracker !== void 0) meta.tracker = v.tracker;
   if (v.paused !== void 0) meta.paused = v.paused;
   if (v.lane_history !== void 0) meta.laneHistory = v.lane_history;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "specatlas",
-  "version": "0.1.8",
+  "version": "0.1.10",
   "description": "SpecAtlas: kernel determinista de Spec-Driven Development (CLI)",
   "license": "MIT",
   "type": "module",
@@ -40,7 +40,7 @@
   },
   "devDependencies": {
     "@specatlas/adapters": "0.1.1",
-    "@specatlas/core": "0.1.6"
+    "@specatlas/core": "0.1.8"
   },
   "scripts": {
     "typecheck": "tsc --noEmit",

package/workflow/phases/mockup.md

@@ -19,10 +19,10 @@ El contenido se escribe en **{{LANGUAGE_NAME}}**.
 
 ## Pasos
 
-1. Prepara el plan y el manifiesto:
+1. Prepara el plan y el manifiesto (y marca el cambio como que exige mockups si aún no lo declara):
 
 ```
-satlas mockup {{SLUG}}
+satlas mockup {{SLUG}} --require
 ```
 
 2. Lee el plan (`changes/{{SLUG}}/mockups/plan.yaml`), la spec, el glosario y, si existen, `design/tokens.json` o `DESIGN.md` (sistema de diseño del proyecto).

package/workflow/phases/specify.md

@@ -34,13 +34,14 @@ El contenido se escribe en **{{LANGUAGE_NAME}}**.
    - `MODIFIED` copia el bloque **completo** del requisito tal como está en la spec viva y lo edita.
    - `REMOVED` declara `- Motivo:` y `- Migración:`.
 5. Completa `.sdd/changes/{{SLUG}}/proposal.md` en lenguaje de negocio (sin tecnología y sin dejar los textos entre paréntesis de la plantilla): **Por qué** (problema u oportunidad, con la historia de usuario), **Qué cambia** (alcance funcional), **Fuera de alcance** (lo que no se hará) y **Cómo se mide el éxito** (indicadores observables).
-6. Verifica con el CLI y corrige hasta que no haya errores:
+6. Si el cambio toca interfaz, **pregunta si llevará mockups** y anótalo en `.sdd/changes/{{SLUG}}/meta.yaml`: `mockups: required` (no se podrá aprobar sin ellos; el siguiente paso será `/satlas-mockup`) o `mockups: skip` (se aprueba sin contrato visual).
+7. Verifica con el CLI y corrige hasta que no haya errores:
 
 ```
 satlas validate --change {{SLUG}}
 ```
 
-7. Reporta: número de requisitos y escenarios, supuestos, y las preguntas que quedaron abiertas.
+8. Reporta: número de requisitos y escenarios, supuestos, y las preguntas que quedaron abiertas.
 
 ## Prohibido