@jpkme/diff-review 0.1.4 → 0.1.5

package/lib/output-name.test.mjs

@@ -1,54 +1,37 @@
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
 import {
-  formatOutputStamp,
   isOutputFilename,
   makeOutputFilename,
   OUTPUT_FILENAME_LEGACY,
 } from "./output-name.mjs";
 
 describe("output-name", () => {
-  const when = new Date(2026, 8, 15, 14, 30, 45);
-  const stamp = "20260915-143045";
-
-  it("formats stamp", () => {
-    assert.equal(formatOutputStamp(when), stamp);
-  });
-
   it("builds name with short tip hashes", () => {
     const name = makeOutputFilename({
       branchSha: "aaaaaaaaaa1111111111",
       baseSha: "bbbbbbbbbb2222222222",
-      when,
     });
-    assert.equal(name, "diff-report_aaaaaaaaaa-bbbbbbbbbb_20260915-143045.json");
+    assert.equal(name, "diff-rev_aaaaaaaaaa_bbbbbbbbbb.json");
     assert.equal(isOutputFilename(name), true);
   });
 
-  it("reuses an explicit stamp", () => {
-    const name = makeOutputFilename({
-      branchSha: "abcdef0123456789",
-      baseSha: "fedcba9876543210",
-      stamp: "20260101-000000",
-    });
-    assert.equal(name, "diff-report_abcdef0123-fedcba9876_20260101-000000.json");
-  });
-
-  it("falls back to legacy stamp without tips", () => {
-    const name = makeOutputFilename({ when });
-    assert.equal(name, "diff-review-output-20260915-143045.json");
-    assert.equal(isOutputFilename(name), true);
+  it("falls back to legacy without tips", () => {
+    assert.equal(makeOutputFilename({}), OUTPUT_FILENAME_LEGACY);
+    assert.equal(isOutputFilename(OUTPUT_FILENAME_LEGACY), true);
   });
 
-  it("accepts legacy names", () => {
-    assert.equal(OUTPUT_FILENAME_LEGACY, "diff-review-output.json");
-    assert.equal(isOutputFilename(OUTPUT_FILENAME_LEGACY), true);
+  it("accepts legacy stamped and report names", () => {
     assert.equal(isOutputFilename("diff-review-output-20260915-143045.json"), true);
+    assert.equal(
+      isOutputFilename("diff-report_aaaaaaaaaa-bbbbbbbbbb_20260915-143045.json"),
+      true
+    );
   });
 
   it("rejects other names", () => {
     assert.equal(isOutputFilename("review.json"), false);
     assert.equal(isOutputFilename("diff-review-output-foo.json"), false);
-    assert.equal(isOutputFilename("diff-report_short-tips_20260915-143045.json"), false);
+    assert.equal(isOutputFilename("diff-rev_short_tips.json"), false);
   });
 });

package/lib/prompt.mjs

@@ -4,14 +4,12 @@
  */
 import {
   makeOutputFilename,
-  formatOutputStamp,
   isOutputFilename,
   OUTPUT_FILENAME_LEGACY,
 } from "./output-name.mjs";
 
 export {
   makeOutputFilename,
-  formatOutputStamp,
   isOutputFilename,
   OUTPUT_FILENAME_LEGACY,
 };
@@ -22,7 +20,10 @@ export const OUTPUT_SCHEMA_BLOCK = `{
     "repo": "<absolute path or git URL>",
     "branch": "<branch under review>",
     "base": "develop",
-    "remoteUrl": "<origin URL; optional when source=url>"
+    "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>"
   },
   "intent": "string",
   "groups": [
@@ -108,9 +109,14 @@ export function outputSchemaBlock(meta) {
     `    "source": ${JSON.stringify(source)},`,
     `    "repo": ${JSON.stringify(repo)},`,
     `    "branch": ${JSON.stringify(branch)},`,
-    `    "base": ${JSON.stringify(base)}${remoteUrl ? "," : ""}`,
+    `    "base": ${JSON.stringify(base)},`,
   ];
-  if (remoteUrl) metaLines.push(`    "remoteUrl": ${JSON.stringify(remoteUrl)}`);
+  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>"`
+  );
   const metaBlock = `"meta": {\n${metaLines.join("\n")}\n  }`;
   return OUTPUT_SCHEMA_BLOCK.replace(/"meta": \{[\s\S]*?\n  \}/, metaBlock);
 }
@@ -142,11 +148,11 @@ ${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" exactly as given):
+- Shape (copy "meta" source/repo/branch/base/remoteUrl as given; fill generatedAt/agent/model):
 
 ${outputSchemaBlock(meta)}
 
-Include "meta" with the source/repo/branch/base above (and origin "remoteUrl" when local). 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). 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.
 
 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.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jpkme/diff-review",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "Local UI to prompt a coding agent for a branch review and browse the result.",
   "type": "module",
   "bin": {
@@ -49,6 +49,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@bindrs/rfd": "^2.2.2"
+    "@bindrs/rfd": "^2.2.2",
+    "@clack/prompts": "^1.8.1"
   }
 }

package/schema.json

@@ -21,6 +21,18 @@
         "remoteUrl": {
           "type": "string",
           "description": "URL del remote (origin). Fallback si la carpeta local no existe en otra máquina."
+        },
+        "generatedAt": {
+          "type": "string",
+          "description": "ISO-8601 de cuándo se escribió la review (informativo)."
+        },
+        "agent": {
+          "type": "string",
+          "description": "Nombre del tool/agente (informativo), ej. Claude Code / Cursor."
+        },
+        "model": {
+          "type": "string",
+          "description": "Id del modelo usado (informativo)."
         }
       }
     },

package/web/build/client/_app/immutable/chunks/{CAnYe5db.js → CO0mUwMJ.js}

@@ -1 +1 @@
-import"./Dur-vdUf.js";import{c as e}from"./Gya4IU1I.js";var t={get data(){return e.data},get error(){return e.error},get form(){return e.form},get params(){return e.params},get route(){return e.route},get state(){return e.state},get status(){return e.status},get url(){return e.url}};export{t};
+import"./Dur-vdUf.js";import{c as e}from"./DzqTpz7_.js";var t={get data(){return e.data},get error(){return e.error},get form(){return e.form},get params(){return e.params},get route(){return e.route},get state(){return e.state},get status(){return e.status},get url(){return e.url}};export{t};

package/web/build/client/_app/immutable/chunks/CO0mUwMJ.js.br

@@ -0,0 +1,2 @@
+%
@�r[�Kbqh#DȢ;��ʬ�8x4��Bzo�*�����+!�I��8���͊l~�)�+�B�)�ٟJ�}s�2� ���o����\@F�
+J&��|R��|k��u��u�/�jbij/��`���_:���