local-diff-reviewer 1.0.7 → 1.0.11

package/README.md

@@ -42,6 +42,7 @@ npx skills add Mone-Lee/diff-review
 local-diff-reviewer
 local-diff-reviewer staged
 local-diff-reviewer HEAD~1 HEAD
+local-diff-reviewer stop
 local-diff-reviewer --repo /path/to/project
 ```
 
@@ -51,6 +52,15 @@ local-diff-reviewer --repo /path/to/project
 - `staged`：审查已经 `git add`、但还没有提交的改动。
 - `revision`：审查两个 revision 之间的差异，例如 `local-diff-reviewer HEAD~1 HEAD` 会比较 `HEAD~1..HEAD`。
 
+停止当前项目已启动的 review 进程：
+
+- `stop`：按当前 Git 仓库范围清理该仓库启动过的 review 运行进程（包含其 API 端口进程）。
+
+```bash
+local-diff-reviewer stop
+local-diff-reviewer --repo /path/to/project stop
+```
+
 如果命令不是在目标项目目录里启动，可以用 `--repo <path>` 显式指定要审查的 Git 仓库：
 
 ```bash
@@ -77,6 +87,7 @@ local-diff-reviewer --repo /path/to/project staged
 /diff-review
 /diff-review staged
 /diff-review HEAD~1 HEAD
+/diff-review stop
 ```
 
 安装 skill：
@@ -139,12 +150,20 @@ npm run release major
 
 该命令会按顺序执行：
 
+- 发布前检查 npmjs 认证与连通性（`npm whoami --registry=https://registry.npmjs.org/` + `npm ping --registry=https://registry.npmjs.org/`）
 - `npm run release:check`
 - `npm version <patch|minor|major>`（默认 `patch`）
 - `npm publish`
 - `git push`
 - `git push --tags`
 
+若 npmjs 认证缺失或过期，发布会在 preflight 阶段提前失败，并提示执行：
+
+```bash
+npm login --registry=https://registry.npmjs.org/
+npm whoami --registry=https://registry.npmjs.org/
+```
+
 只有在 `npm publish` 成功后，才会自动推送提交和标签到 GitHub。
 
 ## 本地开发

package/SKILL.md

@@ -15,6 +15,7 @@ Use this skill when the user asks for `/diff-review`, wants to inspect current w
 - `/diff-review working`: review current working tree diff.
 - `/diff-review staged`: review staged diff.
 - `/diff-review <base> <target>`: review diff between two Git revisions.
+- `/diff-review stop`: stop all review runtimes created for the current workspace repository.
 
 Do not ask the user to run a shell CLI manually. Determine the target workspace/repository from the user's active environment context, then run the package command with that repository as the command working directory:
 

package/dist/cli/start.js

@@ -3,7 +3,7 @@
 // src/cli/start.ts
 import { spawn } from "node:child_process";
 import { existsSync as existsSync2 } from "node:fs";
-import { basename as basename2, dirname as dirname2, join as join4, resolve as resolve2 } from "node:path";
+import { basename as basename3, dirname as dirname3, join as join5, resolve as resolve2 } from "node:path";
 import { fileURLToPath } from "node:url";
 
 // src/server/storage.ts
@@ -510,11 +510,96 @@ function binaryAddedDiff(path) {
   return [`diff --git a/${path} b/${path}`, "new file mode 100644", "index 0000000..0000000", `Binary files /dev/null and b/${path} differ`].join("\n");
 }
 
+// src/cli/runtime-registry.ts
+import { createHash as createHash3 } from "node:crypto";
+import { access, mkdir as mkdir2, readFile as readFile3, writeFile as writeFile2 } from "node:fs/promises";
+import { homedir as homedir2, platform as platform2 } from "node:os";
+import { basename as basename2, dirname as dirname2, join as join3 } from "node:path";
+async function recordRuntime(entry) {
+  const path = runtimePath(entry.repoRoot);
+  const store = await readRuntime(path);
+  const aliveEntries = store.entries.filter((item) => isPidAlive(item.pid));
+  const deduped = aliveEntries.filter((item) => item.pid !== entry.pid);
+  deduped.push(entry);
+  await writeRuntime(path, { entries: deduped });
+}
+async function stopRecordedRuntimes(repoRoot) {
+  const path = runtimePath(repoRoot);
+  const store = await readRuntime(path);
+  const stopped = [];
+  const stale = [];
+  for (const entry of store.entries) {
+    const parentAlive = isPidAlive(entry.pid);
+    const viteAlive = typeof entry.vitePid === "number" && isPidAlive(entry.vitePid);
+    if (!parentAlive && !viteAlive) {
+      stale.push(entry);
+      continue;
+    }
+    try {
+      if (parentAlive) process.kill(entry.pid, "SIGTERM");
+      if (viteAlive) process.kill(entry.vitePid, "SIGTERM");
+      stopped.push(entry);
+    } catch {
+      stale.push(entry);
+    }
+  }
+  await writeRuntime(path, { entries: [] });
+  return { stopped, stale };
+}
+function runtimePath(repoRoot) {
+  const repoName = basename2(repoRoot) || "repo";
+  const repoHash = createHash3("sha256").update(repoRoot).digest("hex").slice(0, 12);
+  return join3(runtimeDir(), `${repoName}-${repoHash}.runtime.json`);
+}
+function runtimeDir() {
+  if (platform2() === "win32") {
+    return join3(process.env.LOCALAPPDATA ?? join3(homedir2(), "AppData", "Local"), "diff-review", "runtime");
+  }
+  return join3(homedir2(), ".local", "diff-review", "runtime");
+}
+async function readRuntime(path) {
+  try {
+    const text = await readFile3(path, "utf8");
+    const parsed = JSON.parse(text);
+    if (!Array.isArray(parsed.entries)) return { entries: [] };
+    return { entries: parsed.entries.filter(isRuntimeEntry) };
+  } catch {
+    return { entries: [] };
+  }
+}
+async function writeRuntime(path, store) {
+  await mkdir2(dirname2(path), { recursive: true });
+  await writeFile2(path, `${JSON.stringify(store, null, 2)}
+`, "utf8");
+}
+function isRuntimeEntry(value) {
+  if (!value || typeof value !== "object") return false;
+  const entry = value;
+  return typeof entry.pid === "number" && (typeof entry.vitePid === "undefined" || typeof entry.vitePid === "number") && typeof entry.repoRoot === "string" && typeof entry.repoName === "string" && typeof entry.startedAt === "string" && typeof entry.apiPort === "number" && typeof entry.usesVite === "boolean";
+}
+function isPidAlive(pid) {
+  if (!Number.isInteger(pid) || pid <= 0) return false;
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch {
+    return false;
+  }
+}
+async function hasRuntimeRecord(repoRoot) {
+  try {
+    await access(runtimePath(repoRoot));
+    return true;
+  } catch {
+    return false;
+  }
+}
+
 // src/server/index.ts
 import express from "express";
 import { existsSync } from "node:fs";
 import { createServer } from "node:http";
-import { join as join3, normalize as normalize2, resolve, sep } from "node:path";
+import { join as join4, normalize as normalize2, resolve, sep } from "node:path";
 
 // src/core/markdown-source-map.ts
 import GithubSlugger from "github-slugger";
@@ -855,10 +940,10 @@ async function startServer(state, port = 4966) {
       next(error);
     }
   });
-  const webDist = state.webDist ?? join3(process.cwd(), "dist", "web");
+  const webDist = state.webDist ?? join4(process.cwd(), "dist", "web");
   if (existsSync(webDist)) {
     app.use(express.static(webDist));
-    app.get(/.*/, (_req, res) => res.sendFile(join3(webDist, "index.html")));
+    app.get(/.*/, (_req, res) => res.sendFile(join4(webDist, "index.html")));
   }
   app.use((error, _req, res, _next) => {
     res.status(500).json({ error: error.message });
@@ -908,30 +993,41 @@ function selectPromptThreads(threads, scope) {
 }
 
 // src/cli/start.ts
-var packageRoot = resolve2(dirname2(fileURLToPath(import.meta.url)), "../..");
-var builtWebDist = join4(packageRoot, "dist", "web");
+var packageRoot = resolve2(dirname3(fileURLToPath(import.meta.url)), "../..");
+var builtWebDist = join5(packageRoot, "dist", "web");
 async function main() {
-  const { dev, repo, reviewArgs, comments } = parseCliOptions(process.argv.slice(2));
+  const { command, dev, repo, reviewArgs, comments } = parseCliOptions(process.argv.slice(2));
+  if (command === "stop") {
+    await stopCommand(repo);
+    return;
+  }
   const mode = parseReviewMode(reviewArgs);
   const repoRoot = await getRepoRoot(repo ?? process.cwd());
   const diff = await getDiff(mode, repoRoot);
   const diffFiles = parseUnifiedDiff(diff);
   const session = {
     id: crypto.randomUUID(),
-    repoName: basename2(repoRoot),
+    repoName: basename3(repoRoot),
     repoRoot,
     mode,
     diffHash: diffHash(diff),
     createdAt: (/* @__PURE__ */ new Date()).toISOString()
   };
   const importResult = await importAgentComments(repoRoot, diffFiles, comments);
-  const hasBuiltWeb = existsSync2(join4(builtWebDist, "index.html"));
+  const hasBuiltWeb = existsSync2(join5(builtWebDist, "index.html"));
   const apiUrl = await startServer({ session, diffFiles, webDist: hasBuiltWeb ? builtWebDist : void 0 });
   const useVite = dev || !hasBuiltWeb;
   const uiUrl = useVite ? "http://127.0.0.1:5173" : apiUrl;
-  if (useVite) {
-    startVite();
-  }
+  const vitePid = useVite ? startVite() : void 0;
+  await recordRuntime({
+    pid: process.pid,
+    vitePid,
+    repoRoot,
+    repoName: session.repoName,
+    startedAt: session.createdAt,
+    apiPort: parsePort(apiUrl),
+    usesVite: useVite
+  });
   openBrowser(uiUrl);
   console.log(`Diff Review is running: ${uiUrl}`);
   console.log(`Repo: ${session.repoName} (${repoRoot})`);
@@ -948,6 +1044,7 @@ async function main() {
   }
 }
 function parseCliOptions(args) {
+  let command = "review";
   const reviewArgs = [];
   const comments = [];
   let repo;
@@ -958,6 +1055,10 @@ function parseCliOptions(args) {
       dev = true;
       continue;
     }
+    if (arg === "stop") {
+      command = "stop";
+      continue;
+    }
     if (arg === "--repo") {
       const value = args[index + 1];
       if (!value) throw new Error("--repo requires a path value");
@@ -986,7 +1087,35 @@ function parseCliOptions(args) {
     }
     reviewArgs.push(arg);
   }
-  return { dev, repo, reviewArgs, comments };
+  return { command, dev, repo, reviewArgs, comments };
+}
+async function stopCommand(repo) {
+  const repoRoot = await getRepoRoot(repo ?? process.cwd());
+  const hasRecord = await hasRuntimeRecord(repoRoot);
+  const { stopped, stale } = await stopRecordedRuntimes(repoRoot);
+  const total = stopped.length + stale.length;
+  if (total === 0) {
+    if (hasRecord) {
+      console.log("No running review process found for this repo.");
+      return;
+    }
+    console.log("No review runtime record found for this repo.");
+    return;
+  }
+  console.log(`Stopped review runtimes: ${stopped.length}`);
+  for (const entry of stopped) {
+    console.log(
+      `- pid=${entry.pid} vitePid=${entry.vitePid ?? "-"} apiPort=${entry.apiPort} vite=${entry.usesVite ? "yes" : "no"} startedAt=${entry.startedAt}`
+    );
+  }
+  if (stale.length > 0) {
+    console.log(`Skipped stale records: ${stale.length}`);
+  }
+}
+function parsePort(url) {
+  const parsed = new URL(url);
+  const port = parsed.port ? Number(parsed.port) : 80;
+  return Number.isNaN(port) ? 0 : port;
 }
 function modeLabel(mode) {
   if (mode.kind === "revision") return `${mode.base}..${mode.target}`;
@@ -1001,6 +1130,7 @@ function startVite() {
   });
   process.on("SIGINT", () => child.kill("SIGINT"));
   process.on("SIGTERM", () => child.kill("SIGTERM"));
+  return child.pid;
 }
 function openBrowser(url) {
   const child = process.platform === "darwin" ? spawn("open", [url], { stdio: "ignore", detached: true }) : process.platform === "win32" ? spawn("cmd", ["/c", "start", "", url], {

package/dist/web/assets/{arc-FnFNT5Eh.js → arc-BOpgS1XG.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-D2UD_g6s.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-BD8BTEle.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};