local-diff-reviewer 1.0.0 → 1.0.4

package/README.md

@@ -1,5 +1,7 @@
 # diff-review
 
+![Diff 审查台界面截图](./docs/images/diff-review-ui.jpg)
+
 AI chat 里的本地代码审查工具。可以直接用 CLI 打开，也可以安装成 agent skill。
 
 ## Quick Start
@@ -7,7 +9,7 @@ AI chat 里的本地代码审查工具。可以直接用 CLI 打开，也可以
 Try it first:
 
 ```bash
-npx local-diff-reviewer
+npx --yes local-diff-reviewer
 ```
 
 Install and use:
@@ -20,7 +22,7 @@ local-diff-reviewer
 Enable use from AI agents:
 
 ```bash
-npx skills add limengyi/diff-review
+npx skills add Mone-Lee/diff-review
 ```
 
 ## 功能边界
@@ -34,53 +36,6 @@ npx skills add limengyi/diff-review
 - 支持通过内部 `--comment` 参数预置 agent findings / replies。
 - 支持复制极简 AI prompt。
 
-## 评论定位原理（新手友好版）
-
-很多人会直觉认为：评论只和“第几行”绑定。  
-比如删掉第 10 行后，原来第 11 行顶上来变成第 10 行，评论也应该跟着走。
-
-这个工具不是这样做的。它用的是“锚点”定位，锚点至少包含：
-
-- 文件路径（`filePath`）
-- 行号（`lineNumber`）
-- 行所在版本（`side`：旧版本 `old` / 新版本 `new`，代码 Diff 专用）
-
-这意味着评论绑定的是“哪一个版本里的哪一行”，而不是“屏幕上现在第几行”。
-
-### 为什么删除内容后评论可能不显示？
-
-- 对代码 Diff 行评论：
-  - 如果评论当时加在被删除的旧行上，它的锚点是 `old + 行号`。
-  - 删除后右侧展示的是新版本内容（`new`），`old` 锚点找不到对应渲染位置时，就不会显示在行内。
-- 对 Markdown 行评论：
-  - 评论绑定的是 source line（源文件行号）。
-  - 被评论的那段 Markdown 真被删掉后，预览里没有对应块可挂载，行内自然不显示。
-
-### 一个非常短的例子
-
-1. 你在 `old` 版本第 20 行加了评论。  
-2. 后续改动把第 20 行删除，并让后面内容上移。  
-3. 新内容虽然“占了第 20 行的位置”，但它属于 `new` 版本，不是原来的 `old` 行。  
-4. 所以系统不会把旧评论自动贴到新内容上。
-
-这样做的好处是：避免评论“串行”到语义完全不同的内容上，减少误导。
-
-复制出的 prompt 包含 thread id、定位信息和评论内容。thread id 用于让 agent 后续通过 `--comment '{"type":"reply",...}'` 精确回复原评论：
-
-```text
-[thread:<thread-id>]
-文件路径:行号
-评论内容
-```
-
-文件级评论不包含行号：
-
-```text
-[thread:<thread-id>]
-文件路径
-评论内容
-```
-
 ## CLI 使用方式
 
 ```bash
@@ -108,10 +63,10 @@ local-diff-reviewer HEAD~1 HEAD
 安装 skill：
 
 ```bash
-npx skills add limengyi/diff-review
+npx skills add Mone-Lee/diff-review
 ```
 
-skill 会从目标 workspace 运行 `npx local-diff-reviewer [args...]`。
+skill 会从目标 workspace 运行 `npx --yes local-diff-reviewer [args...]`。
 
 ### 预置 agent 评论
 
@@ -120,28 +75,28 @@ CLI 支持重复传入 `--comment <json>`，用于在打开 UI 前把 agent 审
 代码 diff 行评论：
 
 ```bash
-npx local-diff-reviewer \
+npx --yes local-diff-reviewer \
   --comment '{"type":"thread","filePath":"src/foo.ts","position":{"side":"new","line":36},"body":"这里没有处理空数组，可能导致运行时报错。"}'
 ```
 
 Markdown source line 评论：
 
 ```bash
-npx local-diff-reviewer \
+npx --yes local-diff-reviewer \
   --comment '{"type":"thread","filePath":"README.md","position":{"type":"markdown","line":22},"body":"这里可以补充 old/new side 的例子。"}'
 ```
 
 文件级评论：
 
 ```bash
-npx local-diff-reviewer \
+npx --yes local-diff-reviewer \
   --comment '{"type":"thread","filePath":"src/foo.ts","body":"这个文件的错误处理策略需要统一。"}'
 ```
 
 回复已有 thread：
 
 ```bash
-npx local-diff-reviewer \
+npx --yes local-diff-reviewer \
   --comment '{"type":"reply","threadId":"<thread-id>","body":"同意，这里应该按 repoRoot 隔离评论存储。"}'
 ```
 
@@ -155,6 +110,24 @@ npx local-diff-reviewer \
 - `replied`：已有 agent 内容，或从 resolved 重新打开。
 - `resolved`：用户确认完成后的状态。
 
+## 发布流程
+
+```bash
+npm run release
+npm run release minor
+npm run release major
+```
+
+该命令会按顺序执行：
+
+- `npm run release:check`
+- `npm version <patch|minor|major>`（默认 `patch`）
+- `npm publish`
+- `git push`
+- `git push --tags`
+
+只有在 `npm publish` 成功后，才会自动推送提交和标签到 GitHub。
+
 ## 本地开发
 
 ```bash

package/SKILL.md

@@ -19,13 +19,13 @@ Use this skill when the user asks for `/diff-review`, wants to inspect current w
 Do not ask the user to run a shell CLI manually. Run the package command from the workspace root:
 
 ```bash
-npx local-diff-reviewer [args...]
+npx --yes local-diff-reviewer [args...]
 ```
 
 When you have concrete review findings or answers to existing review comments, preload them with one `--comment` JSON argument per comment before launching the viewer:
 
 ```bash
-npx local-diff-reviewer [args...] \
+npx --yes local-diff-reviewer [args...] \
   --comment '{"type":"thread","filePath":"src/foo.ts","position":{"side":"new","line":36},"body":"Explain the finding in the user language."}' \
   --comment '{"type":"reply","threadId":"existing-thread-id","body":"Answer the existing thread as the agent."}'
 ```

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 { dirname as dirname2, join as join4, resolve } from "node:path";
+import { dirname as dirname2, join as join4, resolve as resolve2 } from "node:path";
 import { fileURLToPath } from "node:url";
 
 // src/server/storage.ts
@@ -513,7 +513,7 @@ function binaryAddedDiff(path) {
 // src/server/index.ts
 import express from "express";
 import { existsSync } from "node:fs";
-import { join as join3 } from "node:path";
+import { join as join3, normalize as normalize2, resolve, sep } from "node:path";
 
 // src/core/markdown-source-map.ts
 import GithubSlugger from "github-slugger";
@@ -649,6 +649,22 @@ async function startServer(state, port = 4966) {
       next(error);
     }
   });
+  app.get("/api/markdown-asset", (req, res) => {
+    const relativePath = String(req.query.path ?? "").trim();
+    if (!relativePath) {
+      res.status(400).json({ error: "Asset path is required" });
+      return;
+    }
+    const repoRoot = resolve(state.session.repoRoot);
+    const normalizedRelativePath = normalize2(relativePath);
+    const absolutePath = resolve(repoRoot, normalizedRelativePath);
+    const inRepo = absolutePath === repoRoot || absolutePath.startsWith(`${repoRoot}${sep}`);
+    if (!inRepo || !existsSync(absolutePath)) {
+      res.status(404).json({ error: "Asset not found" });
+      return;
+    }
+    res.sendFile(absolutePath);
+  });
   app.get("/api/threads", async (_req, res, next) => {
     try {
       res.json(await readComments(state.session.repoRoot));
@@ -844,11 +860,11 @@ async function startServer(state, port = 4966) {
   app.use((error, _req, res, _next) => {
     res.status(500).json({ error: error.message });
   });
-  return new Promise((resolve2) => {
+  return new Promise((resolve3) => {
     const server = app.listen(port, "127.0.0.1", () => {
       const address = server.address();
       const actualPort = typeof address === "object" && address ? address.port : port;
-      resolve2(`http://127.0.0.1:${actualPort}`);
+      resolve3(`http://127.0.0.1:${actualPort}`);
     });
   });
 }
@@ -861,7 +877,7 @@ function selectPromptThreads(threads, scope) {
 }
 
 // src/cli/start.ts
-var packageRoot = resolve(dirname2(fileURLToPath(import.meta.url)), "../..");
+var packageRoot = resolve2(dirname2(fileURLToPath(import.meta.url)), "../..");
 var builtWebDist = join4(packageRoot, "dist", "web");
 async function main() {
   const { dev, reviewArgs, comments } = parseCliOptions(process.argv.slice(2));

package/dist/web/assets/{arc-CR7lDgfr.js → arc-DfZYUaz4.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-BBL8i-qs.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-BbvzfnQH.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};