@fabiofiorita/porcelain 0.60.0 → 0.60.1

package/README.md

@@ -1,4 +1,4 @@
-# @fabiofiorita/porcelain (0.60.0)
+# @fabiofiorita/porcelain (0.60.1)
 
 Headless **Porcelain** backend — the Electron-free daemon + renderer, packaged for
 plain Node on any machine (Linux mini-PC, cloud VM, laptop). Same credential-gated

package/main/daemon/server.js

@@ -3261,6 +3261,8 @@ var evidenceAssetBodySchema = import_zod22.z.object({
 }).strict();
 var reviewCommentSchema = import_zod22.z.object({
   id: import_zod22.z.string(),
+  /** Absent only for comments written before authorship was recorded; clients treat those as user. */
+  author: import_zod22.z.enum(["user", "agent"]).optional(),
   path: import_zod22.z.string().min(1),
   startLine: import_zod22.z.number().int().positive().optional(),
   endLine: import_zod22.z.number().int().positive().optional(),
@@ -7496,11 +7498,11 @@ function createCommitGeneration() {
   });
 }
 function createGitDiffReadingSources(options) {
-  const layersFor = async (repoPath) => options === void 0 ? [] : options.review.layersForRepo(repoPath);
+  const layersFor = async (repoPath, commitHash) => options === void 0 ? [] : options.review.layersForRepo(repoPath, commitHash);
   return Object.freeze({
     loadWorkingFlow: async (repoPath) => loadWorkingFlow(repoPath, await layersFor(repoPath)),
     loadRangeFlow: async (repoPath, base) => loadRangeFlow(repoPath, await layersFor(repoPath), base),
-    loadCommitFlow: async (repoPath, hash) => loadCommitFlow(repoPath, hash, await layersFor(repoPath)),
+    loadCommitFlow: async (repoPath, hash) => loadCommitFlow(repoPath, hash, await layersFor(repoPath, hash)),
     workingHunks: (repoPath, path, context) => gitDiffFile(repoPath, path, context).then((result) => result.hunks),
     rangeHunks: (repoPath, base, path, context) => gitRangeDiffFile(repoPath, base, path, context).then((result) => result.hunks),
     diffFile: gitDiffFile,
@@ -10677,9 +10679,10 @@ function structuredSource(document, assetsDir, extraFiles = []) {
     ...extraFiles.length === 0 ? {} : { extraFiles }
   };
 }
-function reviewBundleSource(data, assetsDir) {
+function reviewBundleSource(data, assetsDir, commitHash) {
   const metadata = {
     name: data.title,
+    ...commitHash === void 0 ? {} : { commitHash },
     layers: data.layers,
     files: data.files,
     sections: []
@@ -10830,7 +10833,7 @@ function createMcpToolHandlers(deps) {
     const resolved = await resolveWorkspace(ref, inventory.value);
     return resolved.ok ? { ok: true, value: resolved.value } : { ok: false, result: fail(resolved.message) };
   }
-  async function canvasSource(args) {
+  async function canvasSource(args, reviewCommitHash) {
     if (args.document !== void 0) {
       if (args.templateData !== void 0 || args.template !== void 0 || args.files !== void 0 || args.kind !== void 0 || args.entry !== void 0 || args.title !== void 0) {
         return {
@@ -10884,7 +10887,11 @@ function createMcpToolHandlers(deps) {
         };
       }
       const assetsDir = stringField(args, "sourceDir");
-      const source = template === "review" ? reviewBundleSource(reviewCanvasTemplateDataSchema.parse(parsed.data), assetsDir) : planBundleSource(planCanvasTemplateDataSchema.parse(parsed.data), assetsDir);
+      const source = template === "review" ? reviewBundleSource(
+        reviewCanvasTemplateDataSchema.parse(parsed.data),
+        assetsDir,
+        reviewCommitHash
+      ) : planBundleSource(planCanvasTemplateDataSchema.parse(parsed.data), assetsDir);
       return {
         ok: true,
         title: parsed.data.title,
@@ -10984,7 +10991,17 @@ function createMcpToolHandlers(deps) {
       }
       if (op !== "create" && op !== "update")
         return fail("op must be list, get, create, update, delete, or promote.");
-      const source = await canvasSource(args);
+      const reviewCommitHash = await (async () => {
+        if (args.template !== "review" || place.worktreePath === null) return void 0;
+        try {
+          const status = await operations.git.statusGit(place.worktreePath);
+          if (!status.ok || status.value.length > 0) return void 0;
+          return (await operations.git.logGit({ repoPath: place.worktreePath, limit: 1 }))[0]?.hash;
+        } catch {
+          return void 0;
+        }
+      })();
+      const source = await canvasSource(args, reviewCommitHash);
       if (!source.ok) return source.result;
       const canvasId = stringField(args, "id");
       if (op === "create" && canvasId !== void 0)
@@ -11022,10 +11039,17 @@ function createMcpToolHandlers(deps) {
           ...replacing ? { replace: true } : {}
         });
         return promoted.ok ? ok(
-          `Canvas ${written.value.id} written to the tracked overlay. Files written; nothing staged or committed.`
+          source.template === "review" ? `Review Canvas ${written.value.id} written to the tracked overlay${reviewCommitHash === void 0 ? " for live Changes; update it after committing to bind History" : ` and bound to commit ${reviewCommitHash} for History`}. Files written; nothing staged or committed.` : `Canvas ${written.value.id} written to the tracked overlay. Files written; nothing staged or committed.`
         ) : fail(`Could not update the tracked Canvas: ${describeError(promoted.error)}`);
       }
-      return workspaceResult(place, written.value);
+      return workspaceResult(
+        place,
+        source.template === "review" ? {
+          ...written.value,
+          historyCommit: reviewCommitHash ?? null,
+          history: reviewCommitHash === void 0 ? "live-only; update this Canvas after committing to bind History" : "bound"
+        } : written.value
+      );
     },
     async porcelain_comment(args, place) {
       if (place?.worktreePath === null || place === null)
@@ -11054,6 +11078,7 @@ function createMcpToolHandlers(deps) {
           return fail("comment.path and comment.body are required to create a comment.");
         const created = await operations.review.addReviewComment({
           projectPath,
+          author: "agent",
           path,
           body,
           ...typeof comment.startLine === "number" ? { startLine: comment.startLine } : {},
@@ -11404,7 +11429,7 @@ var MCP_TOOLS = Object.freeze([
   {
     name: "porcelain_canvas",
     title: "Manage a Canvas",
-    description: "List, read, create, update, delete, or promote an agent-authored Canvas. Prefer document for a custom validated structured Canvas; sourceDir may provide bundle assets. Built-in review and plan templates compile into that same renderer. Review fixes Why/How and owns its file layers; Plan chooses bounded tabs. Each Review create is scoped to the addressed Worktree, and update requires its id. Promotion writes files but never stages or commits.",
+    description: "List, read, create, update, delete, or promote an agent-authored Canvas. Prefer document for a custom validated structured Canvas; sourceDir may provide bundle assets. Built-in review and plan templates compile into that same renderer. Review fixes Why/How and owns its file layers; Plan chooses bounded tabs. Each Review create is scoped to the addressed Worktree, and update requires its id. A clean Review write binds current HEAD for cross-Worktree History; a dirty write reports live-only and must be updated after commit. Promotion writes files but never stages or commits.",
     inputSchema: {
       type: "object",
       properties: {
@@ -12688,7 +12713,7 @@ function parseComment(value, index) {
     throw new CommentsFileParseError("invalid-comment", `comments[${index}] is not an object`);
   }
   for (const key of Object.keys(value)) {
-    if (key !== "id" && key !== "path" && key !== "startLine" && key !== "endLine" && key !== "anchorText" && key !== "body" && key !== "resolved" && key !== "createdAt" && key !== "agentReply") {
+    if (key !== "id" && key !== "author" && key !== "path" && key !== "startLine" && key !== "endLine" && key !== "anchorText" && key !== "body" && key !== "resolved" && key !== "createdAt" && key !== "agentReply") {
       throw new CommentsFileParseError(
         "invalid-comment",
         `comments[${index}] has unknown field ${key}`
@@ -12698,6 +12723,9 @@ function parseComment(value, index) {
   if (typeof value.id !== "string" || value.id.length === 0) {
     throw new CommentsFileParseError("invalid-comment", `comments[${index}].id is invalid`);
   }
+  if (value.author !== void 0 && value.author !== "user" && value.author !== "agent") {
+    throw new CommentsFileParseError("invalid-comment", `comments[${index}].author is invalid`);
+  }
   if (typeof value.path !== "string" || value.path.length === 0) {
     throw new CommentsFileParseError("invalid-comment", `comments[${index}].path is invalid`);
   }
@@ -12732,6 +12760,7 @@ function parseComment(value, index) {
     resolved: value.resolved,
     createdAt: value.createdAt
   };
+  if (value.author !== void 0) comment.author = value.author;
   if (value.startLine !== void 0) comment.startLine = value.startLine;
   if (value.endLine !== void 0) comment.endLine = value.endLine;
   if (value.anchorText !== void 0) comment.anchorText = value.anchorText;
@@ -12800,6 +12829,7 @@ function cloneComment(comment) {
     resolved: comment.resolved,
     createdAt: comment.createdAt
   };
+  if (comment.author !== void 0) next.author = comment.author;
   if (comment.startLine !== void 0) next.startLine = comment.startLine;
   if (comment.endLine !== void 0) next.endLine = comment.endLine;
   if (comment.anchorText !== void 0) next.anchorText = comment.anchorText;
@@ -12817,6 +12847,7 @@ function planAddReviewComment(file, input) {
   }
   const comment = {
     id: input.id,
+    author: input.author ?? "user",
     path: input.path,
     body: input.body,
     resolved: false,
@@ -12909,6 +12940,7 @@ function createAddReviewComment(deps) {
     const result = await deps.store.transact(input.projectPath, (current) => {
       const planned = planAddReviewComment(current, {
         id,
+        author: input.author,
         path: input.path,
         startLine: input.startLine,
         endLine: input.endLine,
@@ -14844,7 +14876,7 @@ function daemonIdentity(host = process.env.PORCELAIN_DAEMON_HOST?.trim() || (0,
 
 // ../daemon/src/net/daemon-version.ts
 function daemonVersion() {
-  return "0.60.0";
+  return "0.60.1";
 }
 
 // ../daemon/src/stores/review-store.ts
@@ -14878,6 +14910,8 @@ var reviewSectionSchema2 = import_zod49.z.object({
 var reviewSetSchema = import_zod49.z.object({
   name: import_zod49.z.string().default("Active review"),
   thesis: import_zod49.z.string().max(4096).optional(),
+  /** The clean checkout HEAD this Review was last authored against. */
+  commitHash: import_zod49.z.string().regex(/^[0-9a-f]{40,64}$/).optional(),
   layers: import_zod49.z.array(profileLayerSchema).default([]),
   files: import_zod49.z.array(reviewSetFileSchema).default([]),
   sections: import_zod49.z.array(reviewSectionSchema2).max(30).default([])
@@ -14950,7 +14984,21 @@ function projectIdentity(repoPath) {
     return null;
   }
 }
-async function readCanvasReviewSet(repoPath) {
+async function readReviewCandidate(projectId, record) {
+  try {
+    const raw = JSON.parse(
+      await (0, import_promises33.readFile)(
+        (0, import_node_path48.join)(canvasBundleDir(porcelainHome(), projectId, record.id), "review.json"),
+        "utf8"
+      )
+    );
+    const parsed = reviewSetSchema.safeParse(raw);
+    return parsed.success ? parsed.data : null;
+  } catch {
+    return null;
+  }
+}
+async function readCanvasReviewSet(repoPath, commitHash) {
   const identity2 = projectIdentity(repoPath);
   if (identity2 === null) return null;
   try {
@@ -14960,29 +15008,31 @@ async function readCanvasReviewSet(repoPath) {
     const candidates = (index.value?.canvases ?? []).filter(
       (entry) => typeof entry === "object" && entry !== null && typeof entry.id === "string"
     );
-    const reviews = candidates.filter((entry) => entry.template === "review");
+    const reviews = candidates.filter((entry) => entry.template === "review").sort((a, b) => (b.updatedAt ?? "").localeCompare(a.updatedAt ?? ""));
+    if (commitHash !== void 0) {
+      for (const record2 of reviews) {
+        const review = await readReviewCandidate(identity2.projectId, record2);
+        if (review?.commitHash !== void 0 && (review.commitHash === commitHash || review.commitHash.startsWith(commitHash))) {
+          return review;
+        }
+      }
+      return null;
+    }
     const exact = reviews.filter((entry) => entry.worktreeId === identity2.worktreeId);
-    const record = (exact.length === 0 ? reviews.filter((entry) => entry.worktreeId == null) : exact).sort((a, b) => (b.updatedAt ?? "").localeCompare(a.updatedAt ?? ""))[0];
+    const record = (exact.length === 0 ? reviews.filter((entry) => entry.worktreeId == null) : exact)[0];
     if (record === void 0) return null;
-    const raw = JSON.parse(
-      await (0, import_promises33.readFile)(
-        (0, import_node_path48.join)(canvasBundleDir(porcelainHome(), identity2.projectId, record.id), "review.json"),
-        "utf8"
-      )
-    );
-    const parsed = reviewSetSchema.safeParse(raw);
-    return parsed.success ? parsed.data : null;
+    return readReviewCandidate(identity2.projectId, record);
   } catch {
     return null;
   }
 }
-async function readReviewSet(repoPath) {
-  const canvas = await readCanvasReviewSet(repoPath);
+async function readReviewSet(repoPath, commitHash) {
+  const canvas = await readCanvasReviewSet(repoPath, commitHash);
   if (canvas !== null) return sanitizeReview(repoPath, lenientReviewSetSchema.parse(canvas));
   return null;
 }
-async function reviewLayersForRepo(repoPath) {
-  return (await readReviewSet(repoPath))?.layers ?? [];
+async function reviewLayersForRepo(repoPath, commitHash) {
+  return (await readReviewSet(repoPath, commitHash))?.layers ?? [];
 }
 
 // ../daemon/src/daemon-composition/daemon-operations.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fabiofiorita/porcelain",
-  "version": "0.60.0",
+  "version": "0.60.1",
   "description": "Headless Porcelain daemon — plain Node backend for remote machines (npx @fabiofiorita/porcelain@latest serve)",
   "license": "MIT",
   "author": "Fabio Fiorita <fabiolfp@gmail.com>",