local-diff-reviewer 3.0.11 → 4.0.1

package/dist/cli/start.js

@@ -8,7 +8,7 @@ import { basename as basename3, dirname as dirname3, join as join5, resolve as r
 import { fileURLToPath } from "node:url";
 
 // src/server/storage.ts
-import { mkdir, readFile, writeFile } from "node:fs/promises";
+import { mkdir, readFile, rename, rm, writeFile } from "node:fs/promises";
 import { createHash } from "node:crypto";
 import { homedir, platform } from "node:os";
 import { basename, dirname, join } from "node:path";
@@ -68,16 +68,27 @@ async function attachLegacyComments(repoRoot, diffHash2, diffFiles) {
 async function readCommentStore(repoRoot) {
   const path = commentsPath(repoRoot);
   try {
-    return normalizeStore(JSON.parse(await readFile(path, "utf8")));
-  } catch {
+    const text = await readFile(path, "utf8");
+    return normalizeStore(parseCommentStore(text));
+  } catch (error) {
+    if (!isMissingFileError(error)) {
+      const recovered = await recoverCommentStore(repoRoot, path, error);
+      if (recovered) return recovered;
+    }
     return readLegacyComments(repoRoot);
   }
 }
 async function writeComments(repoRoot, store) {
   const path = commentsPath(repoRoot);
+  const tempPath = `${path}.${process.pid}.${crypto.randomUUID()}.tmp`;
   await mkdir(dirname(path), { recursive: true });
-  await writeFile(path, `${JSON.stringify(store, null, 2)}
+  try {
+    await writeFile(tempPath, `${JSON.stringify(store, null, 2)}
 `, "utf8");
+    await rename(tempPath, path);
+  } finally {
+    await rm(tempPath, { force: true }).catch(() => void 0);
+  }
 }
 function commentsPath(repoRoot) {
   const repoName = basename(repoRoot) || "repo";
@@ -92,11 +103,71 @@ function commentLogsDir() {
 }
 async function readLegacyComments(repoRoot) {
   try {
-    return normalizeStore(JSON.parse(await readFile(join(repoRoot, ".diff-review", "comments.json"), "utf8")));
+    return normalizeStore(parseCommentStore(await readFile(join(repoRoot, ".diff-review", "comments.json"), "utf8")));
   } catch {
     return { threads: [] };
   }
 }
+async function recoverCommentStore(repoRoot, path, cause) {
+  try {
+    const recovered = normalizeStore(parseRecoverableCommentStore(await readFile(path, "utf8")));
+    await writeComments(repoRoot, recovered);
+    return recovered;
+  } catch {
+    console.warn(`Failed to read comment store ${path}: ${cause instanceof Error ? cause.message : String(cause)}`);
+    return void 0;
+  }
+}
+function parseCommentStore(text) {
+  const parsed = JSON.parse(text);
+  if (!Array.isArray(parsed.threads)) return { threads: [] };
+  return parsed;
+}
+function parseRecoverableCommentStore(text) {
+  const end = findRootJsonObjectEnd(text);
+  if (end === -1) throw new Error("Comment store does not contain a complete JSON object");
+  return parseCommentStore(text.slice(0, end));
+}
+function findRootJsonObjectEnd(text) {
+  let depth = 0;
+  let inString = false;
+  let escaped = false;
+  let started = false;
+  for (let index = 0; index < text.length; index += 1) {
+    const char = text[index];
+    if (!started) {
+      if (/\s/.test(char)) continue;
+      if (char !== "{") return -1;
+      started = true;
+    }
+    if (inString) {
+      if (escaped) {
+        escaped = false;
+      } else if (char === "\\") {
+        escaped = true;
+      } else if (char === '"') {
+        inString = false;
+      }
+      continue;
+    }
+    if (char === '"') {
+      inString = true;
+      continue;
+    }
+    if (char === "{") {
+      depth += 1;
+      continue;
+    }
+    if (char === "}") {
+      depth -= 1;
+      if (depth === 0) return index + 1;
+    }
+  }
+  return -1;
+}
+function isMissingFileError(error) {
+  return typeof error === "object" && error !== null && "code" in error && error.code === "ENOENT";
+}
 function normalizeStore(store) {
   const groups = /* @__PURE__ */ new Map();
   for (const thread of store.threads) {
@@ -850,9 +921,108 @@ function getThreadLocation(thread) {
   return `${thread.filePath}:${thread.anchor.lineNumber}`;
 }
 
+// src/server/file-watcher-service.ts
+import { watch } from "node:fs";
+var FileWatcherService = class {
+  constructor(repoRoot) {
+    this.repoRoot = repoRoot;
+    this.startWatching();
+  }
+  repoRoot;
+  clients = /* @__PURE__ */ new Set();
+  watchers = [];
+  pendingChanges = false;
+  debounceTimer = null;
+  closed = false;
+  /**
+   * 注册一个 SSE 客户端，并立即同步当前是否存在待刷新的文件变化。
+   */
+  subscribe(res) {
+    this.clients.add(res);
+    this.send(res, {
+      type: "connected",
+      hasPendingChanges: this.pendingChanges
+    });
+    res.on("close", () => {
+      this.clients.delete(res);
+    });
+  }
+  /**
+   * 在 review 快照已经重算后清空待刷新状态，并通知前端恢复为最新快照。
+   */
+  clearPendingChanges() {
+    if (!this.pendingChanges) return;
+    this.pendingChanges = false;
+    this.broadcast({ type: "synced", hasPendingChanges: false });
+  }
+  /**
+   * 释放底层文件监听资源，避免 runtime 退出后留下悬挂 watcher。
+   */
+  dispose() {
+    this.closed = true;
+    if (this.debounceTimer) {
+      clearTimeout(this.debounceTimer);
+      this.debounceTimer = null;
+    }
+    this.watchers.splice(0).forEach((watcher) => watcher.close());
+    this.clients.forEach((client) => client.end());
+    this.clients.clear();
+  }
+  /**
+   * 优先使用递归监听整个仓库；若运行环境不支持，则退化为监听仓库根和 `.git`。
+   */
+  startWatching() {
+    if (this.tryWatch(this.repoRoot, { recursive: true })) {
+      return;
+    }
+    this.tryWatch(this.repoRoot);
+    this.tryWatch(`${this.repoRoot}/.git`);
+  }
+  /**
+   * 把短时间内的多次磁盘事件合并成一次“可刷新”通知，避免前端按钮频繁闪动。
+   */
+  handleChange() {
+    if (this.closed) return;
+    this.pendingChanges = true;
+    if (this.debounceTimer) {
+      clearTimeout(this.debounceTimer);
+    }
+    this.debounceTimer = setTimeout(() => {
+      this.debounceTimer = null;
+      this.broadcast({
+        type: "change",
+        hasPendingChanges: true,
+        changedAt: (/* @__PURE__ */ new Date()).toISOString()
+      });
+    }, 180);
+  }
+  tryWatch(path, options) {
+    try {
+      const watcher = watch(path, options ?? {}, () => {
+        this.handleChange();
+      });
+      this.watchers.push(watcher);
+      return true;
+    } catch {
+      return false;
+    }
+  }
+  broadcast(event) {
+    this.clients.forEach((client) => {
+      this.send(client, event);
+    });
+  }
+  send(res, event) {
+    res.write(`data: ${JSON.stringify(event)}
+
+`);
+  }
+};
+
 // src/server/index.ts
 async function startServer(state, port = 4966) {
   let markdownPreviews = await buildMarkdownPreviewCache(state);
+  const fileWatcher = new FileWatcherService(state.session.repoRoot);
   const app = express();
   app.use(express.json({ limit: "2mb" }));
   app.get("/api/session", (_req, res) => {
@@ -885,13 +1055,34 @@ async function startServer(state, port = 4966) {
         webDist: state.webDist
       };
       markdownPreviews = await buildMarkdownPreviewCache(nextReviewState);
-      state.session = nextReviewState.session;
-      state.diffFiles = nextReviewState.diffFiles;
+      applyReviewState(state, nextReviewState);
+      fileWatcher.clearPendingChanges();
       res.json({ session: state.session, files: state.diffFiles });
     } catch (error) {
       next(error);
     }
   });
+  app.get("/api/watch", (_req, res) => {
+    res.writeHead(200, {
+      "Content-Type": "text/event-stream",
+      "Cache-Control": "no-cache, no-transform",
+      Connection: "keep-alive"
+    });
+    res.write(": connected\n\n");
+    fileWatcher.subscribe(res);
+  });
+  app.post("/api/refresh", async (_req, res, next) => {
+    try {
+      const nextReviewState = await rebuildReviewState(state);
+      markdownPreviews = await buildMarkdownPreviewCache(nextReviewState);
+      applyReviewState(state, nextReviewState);
+      fileWatcher.clearPendingChanges();
+      const comments = await readComments(state.session.repoRoot);
+      res.json({ session: state.session, files: state.diffFiles, threads: comments.threads });
+    } catch (error) {
+      next(error);
+    }
+  });
   app.get("/api/markdown-preview", async (req, res, next) => {
     try {
       const filePath = String(req.query.path ?? "");
@@ -1168,7 +1359,7 @@ async function startServer(state, port = 4966) {
   app.use((error, _req, res, _next) => {
     res.status(500).json({ error: error.message });
   });
-  return listen(app, port);
+  return listen(app, port, fileWatcher);
 }
 async function buildMarkdownPreviewCache(state) {
   const previews = /* @__PURE__ */ new Map();
@@ -1187,12 +1378,36 @@ async function buildMarkdownPreviewCache(state) {
   );
   return previews;
 }
-function listen(app, port) {
+function applyReviewState(currentState, nextState) {
+  currentState.session = nextState.session;
+  currentState.diffFiles = nextState.diffFiles;
+}
+async function rebuildReviewState(state) {
+  const diff = await getDiff(state.session.mode, state.session.repoRoot);
+  const diffFiles = parseUnifiedDiff(diff);
+  const session = {
+    id: crypto.randomUUID(),
+    repoName: state.session.repoName,
+    repoRoot: state.session.repoRoot,
+    mode: state.session.mode,
+    diffHash: diffHash(diff),
+    createdAt: (/* @__PURE__ */ new Date()).toISOString()
+  };
+  return {
+    session,
+    diffFiles,
+    webDist: state.webDist
+  };
+}
+function listen(app, port, fileWatcher) {
   return new Promise((resolve3, reject) => {
     const server = createServer(app);
+    server.once("close", () => {
+      fileWatcher.dispose();
+    });
     server.once("error", (error) => {
       if (error.code === "EADDRINUSE" && port !== 0) {
-        listen(app, 0).then(resolve3, reject);
+        listen(app, 0, fileWatcher).then(resolve3, reject);
         return;
       }
       reject(error);

package/dist/web/assets/{arc-DXHF8Xjh.js → arc-BR56UJ2q.js}

@@ -1 +1 @@
-import{n as e,t}from"./path-BJQEcSo7.js";import{a as n,c as r,d as i,f as a,i as o,l as s,m as c,n as l,o as u,p as d,r as f,u as p}from"./dist-BvZI7zIw.js";function m(e){return e.innerRadius}function h(e){return e.outerRadius}function g(e){return e.startAngle}function _(e){return e.endAngle}function v(e){return e&&e.padAngle}function y(e,t,n,r,i,a,o,s){var c=n-e,l=r-t,u=o-i,d=s-a,f=d*c-u*l;if(!(f*f<1e-12))return f=(u*(t-a)-d*(e-i))/f,[e+f*c,t+f*l]}function b(e,t,n,r,i,a,o){var c=e-n,l=t-r,u=(o?a:-a)/d(c*c+l*l),f=u*l,p=-u*c,m=e+f,h=t+p,g=n+f,_=r+p,v=(m+g)/2,y=(h+_)/2,b=g-m,x=_-h,S=b*b+x*x,C=i-a,w=m*_-g*h,T=(x<0?-1:1)*d(s(0,C*C*S-w*w)),E=(w*x-b*T)/S,D=(-w*b-x*T)/S,O=(w*x+b*T)/S,k=(-w*b+x*T)/S,A=E-v,j=D-y,M=O-v,N=k-y;return A*A+j*j>M*M+N*N&&(E=O,D=k),{cx:E,cy:D,x01:-f,y01:-p,x11:E*(i/C-1),y11:D*(i/C-1)}}function x(){var s=m,x=h,S=e(0),C=null,w=g,T=_,E=v,D=null,O=t(k);function k(){var e,t,m=+s.apply(this,arguments),h=+x.apply(this,arguments),g=w.apply(this,arguments)-r,_=T.apply(this,arguments)-r,v=l(_-g),k=_>g;if(D||=e=O(),h<m&&(t=h,h=m,m=t),!(h>1e-12))D.moveTo(0,0);else if(v>c-1e-12)D.moveTo(h*u(g),h*a(g)),D.arc(0,0,h,g,_,!k),m>1e-12&&(D.moveTo(m*u(_),m*a(_)),D.arc(0,0,m,_,g,k));else{var A=g,j=_,M=g,N=_,P=v,F=v,I=E.apply(this,arguments)/2,L=I>1e-12&&(C?+C.apply(this,arguments):d(m*m+h*h)),R=p(l(h-m)/2,+S.apply(this,arguments)),z=R,B=R,V,H;if(L>1e-12){var U=o(L/m*a(I)),W=o(L/h*a(I));(P-=U*2)>1e-12?(U*=k?1:-1,M+=U,N-=U):(P=0,M=N=(g+_)/2),(F-=W*2)>1e-12?(W*=k?1:-1,A+=W,j-=W):(F=0,A=j=(g+_)/2)}var G=h*u(A),K=h*a(A),q=m*u(N),J=m*a(N);if(R>1e-12){var Y=h*u(j),X=h*a(j),Z=m*u(M),Q=m*a(M),$;if(v<i)if($=y(G,K,Z,Q,Y,X,q,J)){var ee=G-$[0],te=K-$[1],ne=Y-$[0],re=X-$[1],ie=1/a(f((ee*ne+te*re)/(d(ee*ee+te*te)*d(ne*ne+re*re)))/2),ae=d($[0]*$[0]+$[1]*$[1]);z=p(R,(m-ae)/(ie-1)),B=p(R,(h-ae)/(ie+1))}else z=B=0}F>1e-12?B>1e-12?(V=b(Z,Q,G,K,h,B,k),H=b(Y,X,q,J,h,B,k),D.moveTo(V.cx+V.x01,V.cy+V.y01),B<R?D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,h,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),!k),D.arc(H.cx,H.cy,B,n(H.y11,H.x11),n(H.y01,H.x01),!k))):(D.moveTo(G,K),D.arc(0,0,h,A,j,!k)):D.moveTo(G,K),!(m>1e-12)||!(P>1e-12)?D.lineTo(q,J):z>1e-12?(V=b(q,J,Y,X,m,-z,k),H=b(G,K,Z,Q,m,-z,k),D.lineTo(V.cx+V.x01,V.cy+V.y01),z<R?D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,m,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),k),D.arc(H.cx,H.cy,z,n(H.y11,H.x11),n(H.y01,H.x01),!k))):D.arc(0,0,m,N,M,k)}if(D.closePath(),e)return D=null,e+``||null}return k.centroid=function(){var e=(+s.apply(this,arguments)+ +x.apply(this,arguments))/2,t=(+w.apply(this,arguments)+ +T.apply(this,arguments))/2-i/2;return[u(t)*e,a(t)*e]},k.innerRadius=function(t){return arguments.length?(s=typeof t==`function`?t:e(+t),k):s},k.outerRadius=function(t){return arguments.length?(x=typeof t==`function`?t:e(+t),k):x},k.cornerRadius=function(t){return arguments.length?(S=typeof t==`function`?t:e(+t),k):S},k.padRadius=function(t){return arguments.length?(C=t==null?null:typeof t==`function`?t:e(+t),k):C},k.startAngle=function(t){return arguments.length?(w=typeof t==`function`?t:e(+t),k):w},k.endAngle=function(t){return arguments.length?(T=typeof t==`function`?t:e(+t),k):T},k.padAngle=function(t){return arguments.length?(E=typeof t==`function`?t:e(+t),k):E},k.context=function(e){return arguments.length?(D=e??null,k):D},k}export{x as t};
+import{n as e,t}from"./path-BJQEcSo7.js";import{a as n,c as r,d as i,f as a,i as o,l as s,m as c,n as l,o as u,p as d,r as f,u as p}from"./dist-Ck1Zve1a.js";function m(e){return e.innerRadius}function h(e){return e.outerRadius}function g(e){return e.startAngle}function _(e){return e.endAngle}function v(e){return e&&e.padAngle}function y(e,t,n,r,i,a,o,s){var c=n-e,l=r-t,u=o-i,d=s-a,f=d*c-u*l;if(!(f*f<1e-12))return f=(u*(t-a)-d*(e-i))/f,[e+f*c,t+f*l]}function b(e,t,n,r,i,a,o){var c=e-n,l=t-r,u=(o?a:-a)/d(c*c+l*l),f=u*l,p=-u*c,m=e+f,h=t+p,g=n+f,_=r+p,v=(m+g)/2,y=(h+_)/2,b=g-m,x=_-h,S=b*b+x*x,C=i-a,w=m*_-g*h,T=(x<0?-1:1)*d(s(0,C*C*S-w*w)),E=(w*x-b*T)/S,D=(-w*b-x*T)/S,O=(w*x+b*T)/S,k=(-w*b+x*T)/S,A=E-v,j=D-y,M=O-v,N=k-y;return A*A+j*j>M*M+N*N&&(E=O,D=k),{cx:E,cy:D,x01:-f,y01:-p,x11:E*(i/C-1),y11:D*(i/C-1)}}function x(){var s=m,x=h,S=e(0),C=null,w=g,T=_,E=v,D=null,O=t(k);function k(){var e,t,m=+s.apply(this,arguments),h=+x.apply(this,arguments),g=w.apply(this,arguments)-r,_=T.apply(this,arguments)-r,v=l(_-g),k=_>g;if(D||=e=O(),h<m&&(t=h,h=m,m=t),!(h>1e-12))D.moveTo(0,0);else if(v>c-1e-12)D.moveTo(h*u(g),h*a(g)),D.arc(0,0,h,g,_,!k),m>1e-12&&(D.moveTo(m*u(_),m*a(_)),D.arc(0,0,m,_,g,k));else{var A=g,j=_,M=g,N=_,P=v,F=v,I=E.apply(this,arguments)/2,L=I>1e-12&&(C?+C.apply(this,arguments):d(m*m+h*h)),R=p(l(h-m)/2,+S.apply(this,arguments)),z=R,B=R,V,H;if(L>1e-12){var U=o(L/m*a(I)),W=o(L/h*a(I));(P-=U*2)>1e-12?(U*=k?1:-1,M+=U,N-=U):(P=0,M=N=(g+_)/2),(F-=W*2)>1e-12?(W*=k?1:-1,A+=W,j-=W):(F=0,A=j=(g+_)/2)}var G=h*u(A),K=h*a(A),q=m*u(N),J=m*a(N);if(R>1e-12){var Y=h*u(j),X=h*a(j),Z=m*u(M),Q=m*a(M),$;if(v<i)if($=y(G,K,Z,Q,Y,X,q,J)){var ee=G-$[0],te=K-$[1],ne=Y-$[0],re=X-$[1],ie=1/a(f((ee*ne+te*re)/(d(ee*ee+te*te)*d(ne*ne+re*re)))/2),ae=d($[0]*$[0]+$[1]*$[1]);z=p(R,(m-ae)/(ie-1)),B=p(R,(h-ae)/(ie+1))}else z=B=0}F>1e-12?B>1e-12?(V=b(Z,Q,G,K,h,B,k),H=b(Y,X,q,J,h,B,k),D.moveTo(V.cx+V.x01,V.cy+V.y01),B<R?D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,h,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),!k),D.arc(H.cx,H.cy,B,n(H.y11,H.x11),n(H.y01,H.x01),!k))):(D.moveTo(G,K),D.arc(0,0,h,A,j,!k)):D.moveTo(G,K),!(m>1e-12)||!(P>1e-12)?D.lineTo(q,J):z>1e-12?(V=b(q,J,Y,X,m,-z,k),H=b(G,K,Z,Q,m,-z,k),D.lineTo(V.cx+V.x01,V.cy+V.y01),z<R?D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,m,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),k),D.arc(H.cx,H.cy,z,n(H.y11,H.x11),n(H.y01,H.x01),!k))):D.arc(0,0,m,N,M,k)}if(D.closePath(),e)return D=null,e+``||null}return k.centroid=function(){var e=(+s.apply(this,arguments)+ +x.apply(this,arguments))/2,t=(+w.apply(this,arguments)+ +T.apply(this,arguments))/2-i/2;return[u(t)*e,a(t)*e]},k.innerRadius=function(t){return arguments.length?(s=typeof t==`function`?t:e(+t),k):s},k.outerRadius=function(t){return arguments.length?(x=typeof t==`function`?t:e(+t),k):x},k.cornerRadius=function(t){return arguments.length?(S=typeof t==`function`?t:e(+t),k):S},k.padRadius=function(t){return arguments.length?(C=t==null?null:typeof t==`function`?t:e(+t),k):C},k.startAngle=function(t){return arguments.length?(w=typeof t==`function`?t:e(+t),k):w},k.endAngle=function(t){return arguments.length?(T=typeof t==`function`?t:e(+t),k):T},k.padAngle=function(t){return arguments.length?(E=typeof t==`function`?t:e(+t),k):E},k.context=function(e){return arguments.length?(D=e??null,k):D},k}export{x as t};