@jpkme/diff-review 0.1.6 → 0.1.10

package/lib/prompt.mjs

@@ -21,16 +21,16 @@ export const OUTPUT_SCHEMA_BLOCK = `{
     "branch": "<branch under review>",
     "base": "develop",
     "remoteUrl": "<origin URL; optional when source=url>",
-    "generatedAt": "<ISO-8601 datetime; optional, informational>",
-    "agent": "<tool name e.g. Claude Code / Cursor; optional, informational>",
-    "model": "<model id e.g. claude-opus-…; optional, informational>"
+    "generatedAt": "<ISO-8601 datetime; optional>",
+    "agent": "<required: product name you are, e.g. Claude Code>",
+    "model": "<required: exact model id you are running>"
   },
   "intent": "string",
   "groups": [
     {
       "id": "g1",
       "kind": "feat|fix|refactor|perf|test|chore|docs|infra",
-      "title": "string ≤72 chars, imperative, no kind prefix",
+      "title": "string, aim for ≤72 chars, imperative, no kind prefix",
       "intent": "string"
     }
   ],
@@ -44,8 +44,8 @@ export const OUTPUT_SCHEMA_BLOCK = `{
       "start": 40,
       "end": 58,
       "op": "add|mod|del|rename",
-      "what": "string",
-      "why": "string",
+      "what": "string, aim for ≤120 chars",
+      "why": "string, aim for ≤120 chars",
       "source": "code|commit|pr|issue|inferred"
     }
   ],
@@ -59,8 +59,8 @@ export const OUTPUT_SCHEMA_BLOCK = `{
       "file": "relative/path.ext",
       "line": 87,
       "block": "b1",
-      "what": "string",
-      "fix": "string"
+      "what": "string, aim for ≤120 chars",
+      "fix": "string, aim for ≤120 chars"
     }
   ],
   "skipped": [
@@ -71,18 +71,19 @@ export const OUTPUT_SCHEMA_BLOCK = `{
 
 const RULES = `- Coverage (hard): every path in the diff MUST appear in "blocks" and/or "skipped". Prefer a skipped glob when many files share one reason.
 - Do NOT invent files, hunks, or line ranges that are not in the diff.
-- Line anchors: "lines"/"start"/"end"/"line" MUST match the chosen "side" ("new" = post-image / right side of @@; "old" = pre-image, typically deletions). One block per coherent change span — not one per line. Point findings at that block; leave "block" empty only if cross-cutting.
+- Line anchors: "lines"/"start"/"end"/"line" MUST match the chosen "side" ("new" = post-image / right side of @@; "old" = pre-image, typically deletions). One block per coherent change span, not one per line. Point findings at that block; leave "block" empty only if cross-cutting.
 - Do NOT report: pre-existing code outside the diff; linter/formatter noise; naming/style prefs with no real consequence; intentional branch behavior; silenced lint rules; "missing docs/coverage" without a concrete broken scenario.
 - Quality findings must name a concrete cost, not a vague feeling.
-- Signal over volume: prefer fewer high-signal findings. Order findings by severity (high → nit). Use "nit" sparingly.
+- Signal over volume: prefer fewer high-signal findings. Order findings by severity (high, then nit). Use "nit" sparingly.
 - "blocking": true ONLY if the branch should not merge as-is. Use only with class "risk".
-- Write human-readable strings (intent, title, what, why, fix, notes) in Spanish. Keep enums/ids/paths/JSON keys exactly as in the schema.`;
+- Write human-readable strings (intent, title, what, why, fix, notes) in Spanish. Keep enums/ids/paths/JSON keys exactly as in the schema.
+- Length is a target, not a check: aim for title ≤72 characters and what/why/fix ≤120 characters. Longer text is still valid. Do not rewrite the file only to shorten it.`;
 
 function workflowLocal(base, branch) {
   return `Workflow:
 1. Resolve the merge-base of \`${base}\` and \`${branch}\`.
 2. Run exactly: \`git diff ${base}...${branch}\` (three-dot / merge-base diff). Do not use two-dot unless three-dot is impossible.
-3. Read enough surrounding code (types, callers, tests) to judge behavior — not only the hunk lines.
+3. Read enough surrounding code (types, callers, tests) to judge behavior, not only the hunk lines.
 4. Emit the JSON file. Do not modify the repo.
 5. Validate it with the command in the output section. If it prints INVALID, follow "How to continue" and re-run until it prints OK.`;
 }
@@ -98,7 +99,7 @@ function workflowUrl(base, branch) {
 1. If the repo is not local, clone it (or use whatever access you have). Do not modify it.
 2. Resolve the merge-base of \`${base}\` and \`${branch}\`.
 3. Run exactly: \`git diff ${base}...${branch}\` (three-dot / merge-base diff). Do not use two-dot unless three-dot is impossible.
-4. Read enough surrounding code (types, callers, tests) to judge behavior — not only the hunk lines.
+4. Read enough surrounding code (types, callers, tests) to judge behavior, not only the hunk lines.
 5. Emit the JSON file.
 6. Validate it with the command in the output section. If it prints INVALID, follow "How to continue" and re-run until it prints OK.`;
 }
@@ -122,8 +123,8 @@ export function outputSchemaBlock(meta) {
   if (remoteUrl) metaLines.push(`    "remoteUrl": ${JSON.stringify(remoteUrl)},`);
   metaLines.push(
     `    "generatedAt": "<ISO-8601 datetime>",`,
-    `    "agent": "<tool name, e.g. Claude Code / Cursor>",`,
-    `    "model": "<model id>"`
+    `    "agent": "<required: product name you are, e.g. Claude Code>",`,
+    `    "model": "<required: exact model id you are running>"`
   );
   const metaBlock = `"meta": {\n${metaLines.join("\n")}\n  }`;
   return OUTPUT_SCHEMA_BLOCK.replace(/"meta": \{[\s\S]*?\n  \}/, metaBlock);
@@ -156,13 +157,13 @@ ${RULES}
 Write the result to \`${outputFilename}\` at the repo root.
 - One JSON object only: first character \`{\`, last character \`}\`.
 - No prose before/after, no markdown fences.
-- Shape (copy "meta" source/repo/branch/base/remoteUrl as given; fill generatedAt/agent/model):
+- Shape (copy "meta" source/repo/branch/base/remoteUrl as given; "agent" and "model" are required):
 
 ${outputSchemaBlock(meta)}
 
-Include "meta" with the source/repo/branch/base above (and origin "remoteUrl" when local). Also set informational fields: "generatedAt" (ISO-8601 now), "agent" (tool name), "model" (model id). That lets someone reopen the report elsewhere without re-picking the repo.
+Include "meta" with the source/repo/branch/base above (and origin "remoteUrl" when local). "agent" and "model" are required: set "agent" to the product you are (Claude Code, Cursor, Codex, Copilot, Gemini, …) and "model" to the exact model id you are running. Do not leave placeholders or omit them; validation fails without both. "generatedAt" is optional (ISO-8601 now).
 
-Do NOT include diffs or a "files" array — the tool computes them with git when opening the report. Do not invent hunks or paste patches into the JSON.
+Do NOT include diffs or a "files" array. The tool computes them with git when opening the report. Do not invent hunks or paste patches into the JSON.
 
 After writing the file, validate it (npx so it works even if \`diff-review\` is not on PATH):
 

package/lib/review-meta.mjs

@@ -0,0 +1,33 @@
+/**
+ * Estampa en el JSON quién generó la review. La CLI lo sabe mejor que el modelo.
+ */
+import { readFileSync, writeFileSync } from "node:fs";
+import { decodePayload } from "./json-payload.mjs";
+
+/**
+ * @param {string} filePath
+ * @param {{ agent?: string, model?: string }} provenance
+ * @returns {boolean}
+ */
+export function stampReviewProvenance(filePath, { agent, model } = {}) {
+  const agentName = String(agent || "").trim();
+  const modelId = String(model || "").trim();
+  if (!agentName && !modelId) return false;
+
+  let payload;
+  try {
+    payload = decodePayload(readFileSync(filePath, "utf8"));
+  } catch {
+    return false;
+  }
+  if (!payload || typeof payload !== "object" || Array.isArray(payload)) return false;
+
+  const current = payload.meta;
+  const meta =
+    current && typeof current === "object" && !Array.isArray(current) ? { ...current } : {};
+  if (agentName) meta.agent = agentName;
+  if (modelId) meta.model = modelId;
+  payload.meta = meta;
+  writeFileSync(filePath, `${JSON.stringify(payload, null, 2)}\n`, "utf8");
+  return true;
+}

package/lib/review-meta.test.mjs

@@ -0,0 +1,30 @@
+import { test } from "node:test";
+import assert from "node:assert/strict";
+import { mkdtempSync, readFileSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { stampReviewProvenance } from "./review-meta.mjs";
+
+test("stampReviewProvenance escribe agente y modelo sin perder el resto", () => {
+  const dir = mkdtempSync(join(tmpdir(), "diff-review-meta-"));
+  const file = join(dir, "diff-rev_a_b.json");
+  writeFileSync(file, JSON.stringify({ intent: "algo concreto", meta: { branch: "feat" } }));
+
+  assert.equal(stampReviewProvenance(file, { agent: "Claude Code", model: "opus" }), true);
+
+  const saved = JSON.parse(readFileSync(file, "utf8"));
+  assert.equal(saved.intent, "algo concreto");
+  assert.deepEqual(saved.meta, { branch: "feat", agent: "Claude Code", model: "opus" });
+});
+
+test("stampReviewProvenance no pisa el modelo si no hay uno", () => {
+  const dir = mkdtempSync(join(tmpdir(), "diff-review-meta-"));
+  const file = join(dir, "diff-rev_a_b.json");
+  writeFileSync(file, JSON.stringify({ meta: { model: "sonnet" }, intent: "x" }));
+
+  stampReviewProvenance(file, { agent: "Codex" });
+
+  const saved = JSON.parse(readFileSync(file, "utf8"));
+  assert.equal(saved.meta.agent, "Codex");
+  assert.equal(saved.meta.model, "sonnet");
+});

package/lib/review.test.mjs

@@ -132,6 +132,7 @@ test("buildReview deduplica hallazgos repetidos y limpia referencias colgadas",
 
 test("una respuesta JSON del modelo atraviesa decode, validación y normalización", () => {
   const answer = JSON.stringify({
+    meta: { agent: "Claude Code", model: "claude-opus" },
     intent: "Hace que las búsquedas con varias palabras matcheen los tokens en cualquier orden.",
     groups: [
       { id: "g1", kind: "feat", title: "feat(search): matchear tokens", intent: "el cambio" },

package/lib/validate.mjs

@@ -13,9 +13,18 @@ export const SKIP_REASONS = ["generated", "lockfile", "format", "vendored", "bin
 export const REVIEW_SCHEMA = {
   $id: "diff-review",
   type: "object",
-  required: ["intent", "groups", "blocks"],
+  required: ["meta", "intent", "groups", "blocks"],
   additionalProperties: true,
   properties: {
+    meta: {
+      type: "object",
+      required: ["agent", "model"],
+      additionalProperties: true,
+      properties: {
+        agent: { type: "string", minLength: 1 },
+        model: { type: "string", minLength: 1 },
+      },
+    },
     intent: { type: "string", minLength: 40 },
     groups: {
       type: "array",
@@ -45,8 +54,8 @@ export const REVIEW_SCHEMA = {
           file: { type: "string", minLength: 1 },
           lines: { type: "string", minLength: 1 },
           op: { type: "string", enum: BLOCK_OPS },
-          what: { type: "string", minLength: 1, maxLength: 120 },
-          why: { type: "string", maxLength: 120 },
+          what: { type: "string", minLength: 1 },
+          why: { type: "string" },
           source: { type: "string", enum: WHY_SOURCES },
         },
       },
@@ -66,8 +75,8 @@ export const REVIEW_SCHEMA = {
           file: { type: "string", minLength: 1 },
           line: { type: ["integer", "null"] },
           block: { type: "string" },
-          what: { type: "string", minLength: 1, maxLength: 120 },
-          fix: { type: "string", maxLength: 120 },
+          what: { type: "string", minLength: 1 },
+          fix: { type: "string" },
         },
       },
     },
@@ -112,9 +121,6 @@ export function validateAgainst(schema, data, path = "$") {
   if (actual === "string" && schema.minLength && data.length < schema.minLength) {
     errors.push(`${path}: texto demasiado corto (mínimo ${schema.minLength})`);
   }
-  if (actual === "string" && schema.maxLength && data.length > schema.maxLength) {
-    errors.push(`${path}: texto demasiado largo (máximo ${schema.maxLength})`);
-  }
   if (schema.enum && (actual === "string" || actual === "number") && !schema.enum.includes(data)) {
     errors.push(`${path}: valor no permitido (${data}); usá uno de: ${schema.enum.join(", ")}`);
   }
@@ -145,6 +151,16 @@ export function validateAgainst(schema, data, path = "$") {
  */
 export function reviewIssues(payload, knownPaths = []) {
   const errors = [];
+  const meta = payload?.meta && typeof payload.meta === "object" ? payload.meta : {};
+  const agentName = typeof meta.agent === "string" ? meta.agent.trim() : "";
+  const modelId = typeof meta.model === "string" ? meta.model.trim() : "";
+  if (!agentName || agentName.startsWith("<")) {
+    errors.push("meta.agent es obligatorio: nombre del producto (Claude Code, Cursor, Codex, …), sin placeholder");
+  }
+  if (!modelId || modelId.startsWith("<")) {
+    errors.push("meta.model es obligatorio: id exacto del modelo, sin placeholder");
+  }
+
   const groups = Array.isArray(payload?.groups) ? payload.groups : [];
   const blocks = Array.isArray(payload?.blocks) ? payload.blocks : [];
   const findings = Array.isArray(payload?.findings) ? payload.findings : [];
@@ -202,9 +218,10 @@ TEXT LANGUAGE: Spanish (rioplatense, technical) for intent, groups.intent, block
 
 {
   "intent": "2-4 sentences on what the branch solves",
-  "groups": [{ "id": "g1", "kind": "feat", "title": "scope: summary ≤72 chars (do not repeat kind)", "intent": "why this theme exists" }],
-  "blocks": [{ "id": "b1", "group": "g1", "file": "exact/path.ts", "lines": "L40-58", "op": "mod", "what": "≤120 chars", "why": "≤120 chars", "source": "code" }],
-  "findings": [{ "id": "f1", "class": "risk", "severity": "high", "blocking": true, "kind": "bug", "file": "path.ts", "line": 42, "block": "b1", "what": "≤120 chars", "fix": "≤120 chars" }],
+  "groups": [{ "id": "g1", "kind": "feat", "title": "scope: summary (do not repeat kind)", "intent": "why this theme exists" }],
+  "meta": { "agent": "<required: product name, e.g. Claude Code>", "model": "<required: exact model id>" },
+  "blocks": [{ "id": "b1", "group": "g1", "file": "exact/path.ts", "lines": "L40-58", "op": "mod", "what": "what changed", "why": "why", "source": "code" }],
+  "findings": [{ "id": "f1", "class": "risk", "severity": "high", "blocking": true, "kind": "bug", "file": "path.ts", "line": 42, "block": "b1", "what": "what is wrong", "fix": "what to do" }],
   "skipped": [{ "file": "bun.lock", "reason": "lockfile" }]
 }
 

package/lib/validate.test.mjs

@@ -3,6 +3,7 @@ import assert from "node:assert/strict";
 import { REVIEW_SCHEMA, reviewIssues, jsonContract, validateAgainst } from "./validate.mjs";
 
 const valid = () => ({
+  meta: { agent: "Claude Code", model: "claude-opus" },
   intent: "Agrega el ABM de depósitos locales end-to-end para dejar de administrarlos por SQL.",
   groups: [{ id: "g1", kind: "feat", title: "feat(product): CRUD de depósitos", intent: "base de todo" }],
   blocks: [
@@ -16,6 +17,30 @@ test("validateAgainst acepta una review bien formada", () => {
   assert.deepEqual(validateAgainst(REVIEW_SCHEMA, valid()), []);
 });
 
+test("validateAgainst exige meta.agent y meta.model", () => {
+  const missing = valid();
+  delete missing.meta;
+  assert.ok(validateAgainst(REVIEW_SCHEMA, missing).some((e) => e.includes("meta")));
+  assert.ok(reviewIssues(missing).some((e) => e.includes("meta.agent")));
+  assert.ok(reviewIssues(missing).some((e) => e.includes("meta.model")));
+
+  const blank = valid();
+  blank.meta = { agent: "   ", model: "<required: exact model id>" };
+  assert.ok(reviewIssues(blank).some((e) => e.includes("meta.agent")));
+  assert.ok(reviewIssues(blank).some((e) => e.includes("meta.model")));
+});
+
+test("validateAgainst no limita el largo de what, why ni fix", () => {
+  const doc = valid();
+  const long = "x".repeat(500);
+  doc.blocks[0].what = long;
+  doc.blocks[0].why = long;
+  doc.findings = [
+    { id: "f1", class: "risk", severity: "high", blocking: true, kind: "bug", file: "src/a.ts", line: 4, what: long, fix: long },
+  ];
+  assert.deepEqual(validateAgainst(REVIEW_SCHEMA, doc), []);
+});
+
 test("validateAgainst exige una intención de verdad", () => {
   const errors = validateAgainst(REVIEW_SCHEMA, { ...valid(), intent: "corto" });
   assert.ok(errors.some((e) => e.includes("intent")));

package/lib/wait-output.mjs

@@ -1,15 +1,53 @@
 /**
  * Espera a que aparezca un JSON de review válido en el repo.
  */
-import { existsSync, readFileSync, watch } from "node:fs";
+import { existsSync, readFileSync, statSync, watch } from "node:fs";
 import { basename, join } from "node:path";
 import { decodePayload } from "./json-payload.mjs";
 import { REVIEW_SCHEMA, validateAgainst } from "./validate.mjs";
 
 /**
+ * Lee y valida el JSON. `null` si no está, está vacío, no parsea o es anterior a `since` (ms).
+ * @param {string} target
+ * @param {number} [since]
+ * @returns {{ payload: unknown, errors: string[] } | null}
+ */
+function readReview(target, since) {
+  if (!existsSync(target)) return null;
+  let text;
+  try {
+    if (since != null && statSync(target).mtimeMs < since) return null;
+    text = readFileSync(target, "utf8");
+  } catch {
+    return null;
+  }
+  if (!text.trim()) return null;
+  let payload;
+  try {
+    payload = decodePayload(text);
+  } catch {
+    return null;
+  }
+  return { payload, errors: validateAgainst(REVIEW_SCHEMA, payload) };
+}
+
+/**
+ * JSON de review válido que ya está en el repo (p. ej. de una corrida anterior sobre los mismos commits).
  * @param {string} repoRoot
  * @param {string} filename
- * @param {{ onInvalid?: (errors: string[]) => void, pollMs?: number }} [opts]
+ * @returns {{ path: string, payload: unknown } | null}
+ */
+export function existingReviewOutput(repoRoot, filename) {
+  const target = join(repoRoot, filename);
+  const read = readReview(target);
+  return read && !read.errors.length ? { path: target, payload: read.payload } : null;
+}
+
+/**
+ * @param {string} repoRoot
+ * @param {string} filename
+ * @param {{ onInvalid?: (errors: string[]) => void, pollMs?: number, since?: number }} [opts]
+ *   `since`: ignorar el archivo si se modificó antes (ms epoch), para no tomar uno de una corrida anterior.
  * @returns {Promise<{ path: string, payload: unknown }>}
  */
 export function waitForReviewOutput(repoRoot, filename, opts = {}) {
@@ -19,21 +57,9 @@ export function waitForReviewOutput(repoRoot, filename, opts = {}) {
   let lastInvalidKey = "";
 
   const tryRead = () => {
-    if (!existsSync(target)) return null;
-    let text;
-    try {
-      text = readFileSync(target, "utf8");
-    } catch {
-      return null;
-    }
-    if (!text.trim()) return null;
-    let payload;
-    try {
-      payload = decodePayload(text);
-    } catch {
-      return null;
-    }
-    const errors = validateAgainst(REVIEW_SCHEMA, payload);
+    const read = readReview(target, opts.since);
+    if (!read) return null;
+    const { payload, errors } = read;
     if (errors.length) {
       const key = errors.join("\n");
       if (key !== lastInvalidKey) {

package/lib/wait-output.test.mjs

@@ -0,0 +1,68 @@
+import assert from "node:assert/strict";
+import { mkdtempSync, utimesSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { describe, it } from "node:test";
+import { existingReviewOutput, waitForReviewOutput } from "./wait-output.mjs";
+
+const valid = () => ({
+  meta: { agent: "Codex", model: "gpt-5" },
+  intent: "Agrega el ABM de depósitos locales end-to-end para dejar de administrarlos por SQL.",
+  groups: [{ id: "g1", kind: "feat", title: "feat(product): CRUD de depósitos", intent: "base de todo" }],
+  blocks: [
+    { id: "b1", group: "g1", file: "src/a.ts", lines: "L1-20", op: "add", what: "controller nuevo", why: "", source: "code" },
+  ],
+  findings: [],
+  skipped: [],
+});
+
+const FILE = "diff-rev_a_b.json";
+
+function repoWith(payload) {
+  const repo = mkdtempSync(join(tmpdir(), "diff-review-wait-"));
+  if (payload !== undefined) writeFileSync(join(repo, FILE), JSON.stringify(payload));
+  return repo;
+}
+
+/** Atrasa el mtime del archivo, como si viniera de una corrida anterior. */
+function age(repo, ms) {
+  const past = new Date(Date.now() - ms);
+  utimesSync(join(repo, FILE), past, past);
+}
+
+describe("existingReviewOutput", () => {
+  it("devuelve el JSON si existe y es válido", () => {
+    const repo = repoWith(valid());
+    assert.equal(existingReviewOutput(repo, FILE)?.path, join(repo, FILE));
+  });
+
+  it("null si no existe o es inválido", () => {
+    assert.equal(existingReviewOutput(repoWith(), FILE), null);
+    assert.equal(existingReviewOutput(repoWith({ intent: "corto" }), FILE), null);
+  });
+});
+
+describe("waitForReviewOutput", () => {
+  it("sin since toma un JSON válido que ya estaba", async () => {
+    const repo = repoWith(valid());
+    age(repo, 60_000);
+    const result = await waitForReviewOutput(repo, FILE, { pollMs: 20 });
+    assert.equal(result.path, join(repo, FILE));
+  });
+
+  it("con since ignora el JSON anterior y resuelve cuando se reescribe", async () => {
+    const repo = repoWith(valid());
+    age(repo, 60_000);
+    let resolved = false;
+    const waiting = waitForReviewOutput(repo, FILE, { pollMs: 20, since: Date.now() - 1000 }).then((r) => {
+      resolved = true;
+      return r;
+    });
+    await new Promise((r) => setTimeout(r, 100));
+    assert.equal(resolved, false);
+
+    writeFileSync(join(repo, FILE), JSON.stringify(valid()));
+    const result = await waiting;
+    assert.equal(result.path, join(repo, FILE));
+  });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jpkme/diff-review",
-  "version": "0.1.6",
+  "version": "0.1.10",
   "description": "Local UI to prompt a coding agent for a branch review and browse the result.",
   "type": "module",
   "bin": {

package/schema.json

@@ -4,12 +4,12 @@
   "title": "Diff Review Document",
   "description": "JSON que escribe el agente (meta/intent/groups/blocks/findings/skipped). La UI completa files con git al abrir el reporte; no hace falta que el modelo incluya diffs.",
   "type": "object",
-  "required": ["intent", "groups", "blocks", "findings"],
+  "required": ["meta", "intent", "groups", "blocks", "findings"],
   "properties": {
     "meta": {
       "type": "object",
       "description": "Contexto del repo para reabrir el reporte sin rellenar el formulario. Copiá source/repo/branch/base del prompt.",
-      "required": ["source", "repo", "branch", "base"],
+      "required": ["source", "repo", "branch", "base", "agent", "model"],
       "properties": {
         "source": { "type": "string", "enum": ["local", "url"] },
         "repo": {
@@ -28,11 +28,13 @@
         },
         "agent": {
           "type": "string",
-          "description": "Nombre del tool/agente (informativo), ej. Claude Code / Cursor."
+          "minLength": 1,
+          "description": "Nombre del producto que escribió la review, ej. Claude Code o Cursor. Obligatorio."
         },
         "model": {
           "type": "string",
-          "description": "Id del modelo usado (informativo)."
+          "minLength": 1,
+          "description": "Id exacto del modelo que escribió la review. Obligatorio."
         }
       }
     },
@@ -57,10 +59,9 @@
           },
           "title": {
             "type": "string",
-            "maxLength": 72,
             "description": "Resumen del tema en una línea sin repetir kind (el chip ya lo muestra), ej. 'warehouses: CRUD de depósitos locales' o 'product: alta de depósitos'."
           },
-          "intent": { "type": "string", "maxLength": 240, "description": "Por qué existe este tema." }
+          "intent": { "type": "string", "description": "Por qué existe este tema." }
         }
       }
     },
@@ -86,12 +87,10 @@
           "op": { "type": "string", "enum": ["add", "mod", "del", "rename"] },
           "what": {
             "type": "string",
-            "maxLength": 200,
             "description": "Qué cambia este bloque."
           },
           "why": {
             "type": "string",
-            "maxLength": 200,
             "description": "Por qué. Vacío solo si no es inferible."
           },
           "source": {
@@ -123,8 +122,8 @@
           "file": { "type": "string" },
           "line": { "type": ["integer", "null"], "description": "Línea de la post-imagen (columna derecha / @@ +N)." },
           "block": { "type": ["string", "null"], "description": "Id del bloque, o vacío/null si es transversal." },
-          "what": { "type": "string", "maxLength": 300 },
-          "fix": { "type": "string", "maxLength": 300, "description": "Acción concreta." }
+          "what": { "type": "string" },
+          "fix": { "type": "string", "description": "Acción concreta." }
         }
       }
     },

package/web/build/client/_app/immutable/assets/2.CNFDEnjx.css

@@ -0,0 +1 @@
+.page.svelte-1uha8ag{flex-direction:column;min-height:100vh;display:flex}.diff-backdrop.svelte-1uha8ag{z-index:80;background:color-mix(in srgb, var(--bg) 88%, transparent);-webkit-backdrop-filter:blur(28px);place-items:center;padding:24px;display:grid;position:fixed;inset:0}.diff-modal.svelte-1uha8ag{background:var(--bg-card);border:1px solid var(--border);text-align:center;width:min(380px,100%);padding:28px 24px;box-shadow:0 16px 48px #00000047}.diff-modal.svelte-1uha8ag p:where(.svelte-1uha8ag){color:var(--text);margin:14px 0 0;font-size:15px}.diff-spinner.svelte-1uha8ag{border:2px solid var(--border);border-top-color:var(--accent);border-radius:50%;width:28px;height:28px;margin:0 auto;animation:.7s linear infinite svelte-1uha8ag-diff-spin}@keyframes svelte-1uha8ag-diff-spin{to{transform:rotate(360deg)}}.toast.svelte-1uha8ag{z-index:50;border:1px solid var(--border);background:var(--bg-card);color:var(--text);padding:10px 16px;font-size:13px;position:fixed;top:16px;left:50%;transform:translate(-50%);box-shadow:0 8px 24px #00000029}.topbar.svelte-1uha8ag{border-bottom:1px solid var(--border);background:var(--header-bg);z-index:5;justify-content:space-between;align-items:center;gap:12px;padding:16px 28px;display:flex;position:sticky;top:0}.brand.svelte-1uha8ag{color:var(--text);align-items:center;gap:10px;text-decoration:none;display:flex}.brand-dot.svelte-1uha8ag{background:linear-gradient(135deg,#6ea8fe,#c9a5ff);border-radius:50%;flex-shrink:0;width:9px;height:9px;box-shadow:0 0 12px #6ea8fe88}.title-block.svelte-1uha8ag{text-align:left;flex-direction:column;line-height:1.25;display:flex}.title-block.svelte-1uha8ag span:where(.svelte-1uha8ag):first-child{font-size:15px;font-weight:700}.title-block.svelte-1uha8ag small:where(.svelte-1uha8ag){color:var(--text-faint);font-size:11.5px;font-weight:400}.workspace.svelte-1uha8ag{flex:1;grid-template-columns:minmax(0,1fr) minmax(300px,380px);align-items:start;gap:28px;width:100%;max-width:1280px;margin:0 auto;padding:22px 28px 40px;display:grid;overflow:auto}.setup.svelte-1uha8ag{min-width:0}h2.svelte-1uha8ag{letter-spacing:.06em;text-transform:uppercase;color:var(--text-faint);margin:0 0 4px;font-size:13px;font-weight:600}.result-head.svelte-1uha8ag{margin-top:24px}.section-head.svelte-1uha8ag{justify-content:space-between;align-items:baseline;margin-top:22px;display:flex}.hint-text.svelte-1uha8ag{color:var(--text-dim);margin:0 0 12px;font-size:12.5px;line-height:1.5}.hint-text.svelte-1uha8ag code:where(.svelte-1uha8ag){font-family:var(--mono);color:var(--text);font-size:12px}.detect-banner.svelte-1uha8ag,.detect-wait.svelte-1uha8ag{color:var(--text-dim);margin:0 0 10px;font-size:13px}button.file-link.svelte-1uha8ag{color:var(--accent);font:inherit;cursor:pointer;background:0 0;border:0;padding:0;text-decoration:underline}.error-line.svelte-1uha8ag{color:var(--danger);margin:0 0 10px;font-size:12px}.ok-line.svelte-1uha8ag{color:var(--ok);margin:0 0 10px;font-size:12px}.row.svelte-1uha8ag{flex-wrap:wrap;gap:8px;display:flex}.repo-setup.svelte-1uha8ag{flex-direction:column;gap:10px;display:flex}.field.svelte-1uha8ag{flex-direction:column;flex:1;gap:4px;min-width:180px;display:flex}.repo-field.svelte-1uha8ag{flex:none;width:100%;min-width:0}.branches-row.svelte-1uha8ag .field:where(.svelte-1uha8ag){flex:1;min-width:160px}.field.svelte-1uha8ag span:where(.svelte-1uha8ag){color:var(--text-faint);font-size:11px}.repo-row.svelte-1uha8ag{gap:6px;min-width:0;display:flex}.repo-row.svelte-1uha8ag input[type=text]:where(.svelte-1uha8ag){flex:1;min-width:0}.repo-row.svelte-1uha8ag button:where(.svelte-1uha8ag){flex-shrink:0}.source-toggle.svelte-1uha8ag{margin-bottom:10px}input[type=text].svelte-1uha8ag,select.svelte-1uha8ag{background:var(--bg-card);border:1px solid var(--border);flex:1;min-width:0;padding:9px 12px}button.svelte-1uha8ag{background:var(--bg-card);border:1px solid var(--border);color:var(--text);padding:9px 12px;font-size:12.5px}button.svelte-1uha8ag:hover:not(:disabled){border-color:var(--accent)}button.svelte-1uha8ag:disabled{opacity:.5;cursor:not-allowed}.mode-toggle.svelte-1uha8ag{border:1px solid var(--border);width:fit-content;margin-bottom:10px;display:flex;overflow:hidden}.mode-toggle.svelte-1uha8ag button:where(.svelte-1uha8ag){background:0 0;border:0;padding:6px 12px;font-size:12px}.mode-toggle.svelte-1uha8ag button:where(.svelte-1uha8ag)+button:where(.svelte-1uha8ag){border-left:1px solid var(--border)}.mode-toggle.svelte-1uha8ag button.on:where(.svelte-1uha8ag){background:var(--accent-soft)}textarea.svelte-1uha8ag{resize:vertical;background:var(--bg);border:1px solid var(--border);width:100%;height:220px;font-family:var(--mono);color:var(--text);padding:12px;font-size:12px;line-height:1.5}.command-box.svelte-1uha8ag{background:var(--bg);border:1px solid var(--border);padding:10px 12px;overflow:auto}.command-box.svelte-1uha8ag code:where(.svelte-1uha8ag){font-family:var(--mono);color:var(--text);white-space:nowrap;font-size:12.5px}.actions.svelte-1uha8ag{gap:8px;margin-top:10px;display:flex}button.primary.svelte-1uha8ag{background:var(--btn-primary-bg);border-color:var(--btn-primary-border);color:var(--btn-primary-text);font-weight:600}button.ghost.svelte-1uha8ag{color:var(--text-dim);background:0 0}.skill-block.svelte-1uha8ag{border-top:1px solid var(--border);margin-top:16px;padding-top:14px}.skill-block.svelte-1uha8ag h3:where(.svelte-1uha8ag){letter-spacing:.06em;text-transform:uppercase;color:var(--text-faint);margin:0 0 4px;font-size:11.5px;font-weight:650}.skill-buttons.svelte-1uha8ag{flex-wrap:wrap;gap:8px;display:flex}.coverage-box.svelte-1uha8ag{border:1px solid var(--border);border-left:3px solid var(--sev-med);background:var(--bg-card);flex-direction:column;gap:10px;margin-top:12px;padding:12px 14px;display:flex}.coverage-head.svelte-1uha8ag{flex-wrap:wrap;align-items:baseline;gap:8px;display:flex}.coverage-head.svelte-1uha8ag strong:where(.svelte-1uha8ag){color:var(--sev-med);font-size:13px}.coverage-head.svelte-1uha8ag span:where(.svelte-1uha8ag){color:var(--text-faint);font-size:11.5px}.coverage-box.svelte-1uha8ag summary:where(.svelte-1uha8ag){color:var(--text-dim);cursor:pointer;font-size:12px}.coverage-list.svelte-1uha8ag{max-height:180px;font-family:var(--mono);color:var(--text-faint);margin:8px 0 0;padding-left:18px;font-size:11px;overflow:auto}.coverage-box.svelte-1uha8ag textarea:where(.svelte-1uha8ag){resize:vertical;background:var(--bg);border:1px solid var(--border);width:100%;font-family:var(--mono);color:var(--text);padding:10px;font-size:11.5px;line-height:1.5}.coverage-actions.svelte-1uha8ag{flex-wrap:wrap;gap:8px;display:flex}.dropzone.svelte-1uha8ag{border:1.5px dashed var(--border);text-align:center;background:var(--bg-elev);padding:26px;transition:background .15s,border-color .15s}.dropzone.drag.svelte-1uha8ag{border-color:var(--accent);background:var(--accent-soft)}.dropzone.busy.svelte-1uha8ag{opacity:.7}.dropzone.svelte-1uha8ag p:where(.svelte-1uha8ag){color:var(--text-dim);margin:0 0 6px;font-size:14px}.visually-hidden.svelte-1uha8ag{clip:rect(0 0 0 0);width:1px;height:1px;position:absolute;overflow:hidden}.drop-errors.svelte-1uha8ag{background:var(--danger-bg);color:var(--danger);margin-top:10px;padding:10px 12px;font-size:12.5px}.drop-errors.svelte-1uha8ag strong:where(.svelte-1uha8ag){margin-bottom:4px;display:block}.drop-errors.svelte-1uha8ag ul:where(.svelte-1uha8ag){margin:0;padding-left:18px;line-height:1.6}.reports-col.svelte-1uha8ag{min-width:0;max-height:calc(100vh - 110px);position:sticky;top:88px;overflow:auto}.report-list.svelte-1uha8ag{flex-direction:column;gap:8px;margin:0;padding:0;list-style:none;display:flex}.report-item.svelte-1uha8ag{border:1px solid var(--border);background:var(--bg-card);align-items:stretch;display:flex;position:relative;overflow:visible}.report-item.svelte-1uha8ag:hover{background:var(--bg-card-hover);border-color:#33507a}.report-card.svelte-1uha8ag{min-width:0;color:var(--text);flex:1;padding:12px 8px 12px 14px;text-decoration:none;display:block}.report-actions.svelte-1uha8ag{flex-shrink:0;padding:8px 8px 0 0;position:relative}.kebab.svelte-1uha8ag{width:26px;height:26px;color:var(--text-faint);cursor:pointer;background:0 0;border:1px solid #0000;justify-content:center;align-items:center;padding:0;display:inline-flex}.kebab.svelte-1uha8ag:hover{border-color:var(--border);color:var(--text);background:var(--bg)}.report-menu.svelte-1uha8ag{z-index:40;border:1px solid var(--border);background:var(--bg-card);width:148px;padding:4px;position:fixed;box-shadow:0 8px 24px #0000002e}.report-menu.svelte-1uha8ag .menu-item:where(.svelte-1uha8ag){width:100%;color:var(--text);text-align:left;cursor:pointer;background:0 0;border:0;padding:8px 10px;font-size:12.5px;display:block}.report-menu.svelte-1uha8ag .menu-item:where(.svelte-1uha8ag):hover{background:var(--accent-soft)}.report-menu.svelte-1uha8ag .menu-item.danger:where(.svelte-1uha8ag){color:var(--danger)}.report-top.svelte-1uha8ag{justify-content:space-between;align-items:baseline;gap:12px;display:flex}.report-top.svelte-1uha8ag strong:where(.svelte-1uha8ag){font-size:14px}.report-prov.svelte-1uha8ag{min-width:0;color:var(--text-dim);align-items:center;gap:6px;margin-top:4px;font-size:12px;display:flex}.report-prov.svelte-1uha8ag .prov-agent:where(.svelte-1uha8ag){text-overflow:ellipsis;white-space:nowrap;flex:0 auto;min-width:0;overflow:hidden}.report-prov.svelte-1uha8ag .prov-sep:where(.svelte-1uha8ag){color:var(--text-faint);flex-shrink:0}.report-prov.svelte-1uha8ag .prov-model:where(.svelte-1uha8ag){min-width:0;font-family:var(--mono);text-overflow:ellipsis;white-space:nowrap;flex:auto;font-size:11.5px;overflow:hidden}.when.svelte-1uha8ag{color:var(--text-dim);font-size:12px}.report-meta.svelte-1uha8ag{color:var(--text-dim);justify-content:space-between;align-items:baseline;gap:12px;margin-top:6px;font-size:12px;display:flex}.report-base.svelte-1uha8ag{text-overflow:ellipsis;white-space:nowrap;min-width:0;overflow:hidden}.base-arrow.svelte-1uha8ag{color:var(--text-faint)}.report-counts.svelte-1uha8ag{flex-shrink:0;align-items:baseline;gap:8px;display:flex}.pill.svelte-1uha8ag{font-family:var(--mono);font-size:11px}.pill.bad.svelte-1uha8ag{color:var(--danger)}.report-intent.svelte-1uha8ag{color:var(--text-dim);-webkit-line-clamp:2;line-clamp:2;-webkit-box-orient:vertical;margin:8px 0 0;font-size:12.5px;line-height:1.45;display:-webkit-box;overflow:hidden}@media (width<=960px){.workspace.svelte-1uha8ag{grid-template-columns:1fr}.reports-col.svelte-1uha8ag{max-height:none;position:static}}