@weniatall/dsh-file-viewer 0.3.2 → 0.3.3

package/README.md

@@ -86,7 +86,9 @@ dsh plugin --profile web add <release-tgz-url>
 - **提交记录**：按 30 条一页加载，显示标题、短 SHA、作者与日期，底部「加载更多」按需追加，并显示「已显示 N / 总数」。
 - **改动文件列表**：点击任一提交，右侧显示该提交的完整信息（标题、完整 SHA、作者、时间、正文）与相对**第一父提交**的改动文件列表（合并提交同理；根提交与空树比对）。`A / M / D / R / C` 等状态用颜色区分。
 - **该提交的 diff**：提交详情有「变更文件 / 改动内容」两个页签，后者是该提交针对同一基准（第一父提交）的完整 patch，按文件分段并各自标注 `+N -M`。两个页签取的是同一对 revision，不会出现列表与 diff 基准不一致的情况。
-- **跳转到文件**：点击改动文件会在右侧已有的预览器里打开**该提交中的版本**，而不是工作区当前内容——两者恰恰在历史最有价值的时候不同。被该提交删除的文件从父提交读取，所以删除行也能打开；重命名显示旧路径的内容。预览顶部会标明来源提交，避免被误认为当前文件。`‹` 返回侧栏继续浏览提交。
+- **跳转到文件**：点击改动文件会在右侧已有的预览器里打开**该提交中的版本**，而不是工作区当前内容——两者恰恰在历史最有价值的时候不同。被该提交删除的文件从父提交读取，所以删除行也能打开；重命名显示旧路径的内容。预览顶部会标明来源提交，避免被误认为当前文件。`‹` 返回 「变更文件」列表继续浏览。
+
+  打开后有两个页签：**改动**（这个提交对该文件做了什么，默认显示）和**完整内容**（该版本文件的全文，可翻页、切换语法高亮）。重命名的 diff 会保留为「重命名」而不是退化成「删除 + 新增」——宿主端把新旧两侧路径一起交给 git，否则 `-M` 无法配对。
 
 只读操作（列出仓库、分支、历史、改动文件）不会修改仓库；面板中唯一的写操作是用户显式点击「检出」。
 

package/lib/client.js

@@ -3347,6 +3347,9 @@ window.__ModuleLoader__.load({
         // from the working tree: the two differ exactly when the history is
         // interesting, and a deleted file has no working-tree copy at all.
         const fromCommit = typeof commit === "string" && commit !== "";
+        // Whether the reader has picked a view themselves. The automatic landing
+        // on the diff happens once per mount and then stays out of the way.
+        const viewChosen = react.useRef(false);
 
         react.useEffect(() => {
           let live = true;
@@ -3366,10 +3369,40 @@ window.__ModuleLoader__.load({
           }).catch((error) => {
             if (live) setState({ status: "error", data: null, error });
           });
-          // The working-tree diff is meaningless for a historical blob: it
-          // compares the file with the index, not with the parent commit.
+          // A historical file is diffed against its own commit, not against the
+          // index: the working-tree comparison would answer a different
+          // question and show changes made after this commit.
+          const diffSha = typeof meta.diffSha === "string" && meta.diffSha !== "" ? meta.diffSha : "";
           if (fromCommit) {
-            setDiffData(null);
+            if (diffSha === "") {
+              setDiffData(null);
+              return () => { live = false; };
+            }
+            request("gitCommitDiff", {
+              root,
+              repo,
+              sha: diffSha,
+              path: meta.rowPath || meta.path,
+              ...(meta.oldPath ? { oldPath: meta.oldPath } : {}),
+            }).then((val) => {
+              if (!live) return;
+              // `gitCommitDiff` returns per-file sections; this view shows the
+              // one file it asked for.
+              const first = val !== null && Array.isArray(val.files) ? val.files[0] : null;
+              const loaded = first === null || first === undefined
+                ? null
+                : { hasDiff: first.rows.length > 0, rows: first.rows, diffs: [], summary: { added: first.added, removed: first.removed } };
+              setDiffData(loaded);
+              // The reader opened this file *from* the change list, so the
+              // change is what they came to see; the full text is one press
+              // away and their own choice wins from then on.
+              if (!viewChosen.current && loaded !== null && loaded.hasDiff) {
+                viewChosen.current = true;
+                setDiffMode(true);
+              }
+            }).catch(() => {
+              if (live) setDiffData(null);
+            });
             return () => { live = false; };
           }
           request("diff", { root, path: meta.path }).then((val) => {
@@ -3378,7 +3411,7 @@ window.__ModuleLoader__.load({
             if (live) setDiffData(null);
           });
           return () => { live = false; };
-        }, [root, meta.path, offset, lang, fromCommit, commit, repo]);
+        }, [root, meta.path, offset, lang, fromCommit, commit, repo, meta.diffSha, meta.rowPath, meta.oldPath]);
 
         if (state.status === "error") {
           return react.createElement("div", { className: "fv-note fv-error" }, messageOf(state.error));
@@ -3389,7 +3422,10 @@ window.__ModuleLoader__.load({
         // A rendered preview needs the whole document. Past the host's preview
         // budget only windowed lines arrive, so the source view is all there is.
         const canPreview = (data.kind === "markdown" || data.kind === "json") && typeof data.text === "string";
-        const hasDiff = Boolean(diffData && diffData.hasDiff && diffData.diffs && diffData.diffs.length > 0);
+        const hasDiff = Boolean(diffData && diffData.hasDiff && (
+          (Array.isArray(diffData.rows) && diffData.rows.length > 0)
+          || (Array.isArray(diffData.diffs) && diffData.diffs.length > 0)
+        ));
         const diffSummary = diffData && diffData.summary;
         const showTabs = canPreview || hasDiff;
 
@@ -3465,36 +3501,42 @@ window.__ModuleLoader__.load({
         // Only the source view scrolls sideways; a rendered preview already
         // reflows, so the attribute is pointless there and would just confuse.
         const wrapping = wrap && (!canPreview || raw);
+        // Both answers belong to the same question when a file is opened from a
+        // commit: "what changed here" and "what does it contain". The diff leads
+        // in that case, because it is what the reader came for; opening the
+        // working-tree copy keeps the content tab first.
+        const diffTab = hasDiff
+          ? react.createElement("button", {
+            key: "diff",
+            type: "button", className: "fv-tab fv-tab-diff", "aria-selected": diffMode ? "true" : "false",
+            onClick: () => { viewChosen.current = true; setDiffMode(true); },
+          },
+            "改动",
+            diffSummary && (diffSummary.added > 0 || diffSummary.removed > 0)
+              ? react.createElement("span", { className: "fv-diff-stat" },
+                  diffSummary.added > 0 ? react.createElement("span", { className: "fv-diff-add" }, `+${diffSummary.added}`) : null,
+                  (diffSummary.added > 0 && diffSummary.removed > 0) ? " " : "",
+                  diffSummary.removed > 0 ? react.createElement("span", { className: "fv-diff-del" }, `-${diffSummary.removed}`) : null)
+              : null)
+          : null;
+        const contentTab = react.createElement("button", {
+          key: "content",
+          type: "button", className: "fv-tab", "aria-selected": (!raw && !diffMode) ? "true" : "false",
+          onClick: () => { viewChosen.current = true; setRaw(false); setDiffMode(false); },
+        }, canPreview ? (data.kind === "markdown" ? "预览" : "结构") : (fromCommit ? "完整内容" : "内容"));
+        const sourceTab = canPreview
+          ? react.createElement("button", {
+            key: "source",
+            type: "button", className: "fv-tab", "aria-selected": (raw && !diffMode) ? "true" : "false",
+            onClick: () => { viewChosen.current = true; setRaw(true); setDiffMode(false); },
+          }, "源码")
+          : null;
+
         const tabs = showTabs
           ? react.createElement("div", { className: "fv-sheet-tabs", role: "tablist" },
-            canPreview
-              ? react.createElement("button", {
-                type: "button", className: "fv-tab", "aria-selected": (!raw && !diffMode) ? "true" : "false",
-                onClick: () => { setRaw(false); setDiffMode(false); },
-              }, data.kind === "markdown" ? "预览" : "结构")
-              : react.createElement("button", {
-                type: "button", className: "fv-tab", "aria-selected": (!raw && !diffMode) ? "true" : "false",
-                onClick: () => { setRaw(false); setDiffMode(false); },
-              }, "内容"),
-            canPreview
-              ? react.createElement("button", {
-                type: "button", className: "fv-tab", "aria-selected": (raw && !diffMode) ? "true" : "false",
-                onClick: () => { setRaw(true); setDiffMode(false); },
-              }, "源码")
-              : null,
-            hasDiff
-              ? react.createElement("button", {
-                type: "button", className: "fv-tab fv-tab-diff", "aria-selected": diffMode ? "true" : "false",
-                onClick: () => setDiffMode(true),
-              },
-                "改动",
-                diffSummary && (diffSummary.added > 0 || diffSummary.removed > 0)
-                  ? react.createElement("span", { className: "fv-diff-stat" },
-                      diffSummary.added > 0 ? react.createElement("span", { className: "fv-diff-add" }, `+${diffSummary.added}`) : null,
-                      (diffSummary.added > 0 && diffSummary.removed > 0) ? " " : "",
-                      diffSummary.removed > 0 ? react.createElement("span", { className: "fv-diff-del" }, `-${diffSummary.removed}`) : null)
-                  : null)
-              : null)
+            ...(fromCommit && diffTab !== null
+              ? [diffTab, contentTab, sourceTab]
+              : [contentTab, sourceTab, diffTab]).filter((tab) => tab !== null))
           : null;
         return react.createElement(react.Fragment, null,
           tabs,
@@ -4456,6 +4498,13 @@ window.__ModuleLoader__.load({
               displayPath: baseNameOf(blobPath) + " @ " + readSha.slice(0, 7),
               kind: component,
               commit: readSha,
+              // Opening a changed file must offer both halves of the question:
+              // what this commit changed in it, and what the file looks like.
+              // `diffSha` is the commit being viewed (the blob may come from
+              // its parent for a deletion), and `oldPath` is what lets the host
+              // keep rename detection intact when filtering the patch.
+              diffSha: sha,
+              oldPath: typeof file.oldPath === "string" && file.oldPath !== "" ? file.oldPath : null,
               // The commit records whether the file was added, modified or
               // deleted; the preview says so instead of looking like a normal
               // working-tree file.

package/lib/index.js

@@ -1931,11 +1931,24 @@ export async function readGitCommitDiff(ctx, options, payload, signal) {
   if (wanted === null && typeof payload?.path === 'string' && payload.path !== '') {
     throw new ViewerError('outside-root', '该文件不在当前根目录内')
   }
+  // A rename has two paths, and filtering by only one of them defeats rename
+  // detection: git then reports the pair as an unrelated delete plus an add
+  // (measured — `-- new-name` prints "new file mode", `-- old-name` prints
+  // "deleted file mode"). Passing both keeps `-M` able to pair them.
+  const wantedOld = typeof payload?.oldPath === 'string' && payload.oldPath !== ''
+    ? repoRelativeFromRoot(placement, payload.oldPath)
+    : null
+  if (wantedOld === null && typeof payload?.oldPath === 'string' && payload.oldPath !== '') {
+    throw new ViewerError('outside-root', '该文件不在当前根目录内')
+  }
 
   const base = hasParent
     ? ['diff', '--no-color', '-M', parentSha, sha]
     : ['show', '--no-color', '--format=', '--first-parent', sha]
-  const args = wanted === null ? base : [...base, '--', wanted]
+  const paths = wanted === null
+    ? []
+    : [...new Set([wanted, ...(wantedOld === null ? [] : [wantedOld])])]
+  const args = paths.length === 0 ? base : [...base, '--', ...paths]
   const result = await run(repo.toplevel, args, signal)
   if (result?.ok !== true) throw new ViewerError('git-failed', '无法读取该提交的改动内容')
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@weniatall/dsh-file-viewer",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "description": "Workspace file viewer and Git panel for DeepSeek Harness: file tree, syntax-highlighted code, images, PDF, Excel and Word, plus local/remote branches, paged commit history, commit diffs, working-tree changes and changed-file navigation",
   "type": "module",
   "main": "lib/index.js",