@youtyan/code-viewer 0.8.6 → 0.8.8

package/README.md

@@ -157,6 +157,12 @@ view's row component, so line numbers, drag-selection of `line=` ranges,
 syntax highlighting and the Viewer Settings code font size all match the
 Code tab.
 
+When the repository remote is hosted on GitHub, repository and file headers
+include an **Open on GitHub** action. Selecting source lines opens a compact
+action bar for copying the AI reference, opening the exact line range on
+GitHub, or copying that GitHub URL. Markdown and HTML diff cards also expose a
+direct **Preview** shortcut beside the regular file view action.
+
 Very large text files use a virtualized source viewer. Only visible rows are
 rendered, and the page includes controls to copy the full file or reopen it in
 the full non-virtual view.
@@ -213,9 +219,11 @@ root:
 - `db-snapshots.sqlite` — datastore snapshots and diff blobs.
 - `datastore-connections.json` — saved non-secret datastore connection settings.
 
-The `.code-viewer` directory is tool-internal: it never shows up in the file
-tree, searches, or diffs. Add it to `.gitignore` if you do not want to share
-its contents through git.
+The `.code-viewer` directory is tool-managed. It appears in the repository
+tree, and its text files can be inspected in the Code view, but it remains
+excluded from repository searches and diffs. Treat the files as diagnostic
+state rather than hand-edited configuration. Add `.code-viewer/` to
+`.gitignore` if you do not want to share its contents through git.
 
 ## Datastore Viewer
 
@@ -803,6 +811,8 @@ state is saved per project).
 
 A copy button on each annotation produces a paste-ready prompt block that
 references the annotation by URL for sharing back into the originating agent.
+The panel-open state, selected session, and selected annotation are also kept
+in the URL, so a reload or shared link restores the same walkthrough context.
 
 The `annotate` subcommand talks to the running server for the repository
 (discovered via `~/.cache/code-viewer/servers/`); start one with

package/dist/code-viewer.js

@@ -196,7 +196,7 @@ function buildRoute(route) {
     }
     case "journal": {
       const params = new URLSearchParams;
-      if (route.tab && route.tab !== "journal")
+      if (route.tab && route.tab !== "tasks")
         params.set("tab", route.tab);
       if (route.date)
         params.set("date", route.date);
@@ -1449,18 +1449,13 @@ function resolveGitArgs(args) {
   return [commandForExternal("git"), ...args.slice(1)];
 }
 function gitFailureMessage(res, fallback) {
-  if (isCommandNotFoundResult("git", res))
-    return commandNotFoundDetail("git");
-  return res.stderr?.trim() || fallback;
+  const message = isCommandNotFoundResult("git", res) ? commandNotFoundDetail("git") : res.stderr?.trim() || fallback;
+  console.error(`[code-viewer] ${fallback} (git exit ${res.code}): ${message}`);
+  return message;
 }
 function gitFailureResult(res, fallback) {
-  if (!isCommandNotFoundResult("git", res)) {
-    return { error: fallback };
-  }
-  return {
-    error: commandNotFoundDetail("git"),
-    status: 503
-  };
+  const error = gitFailureMessage(res, fallback);
+  return isCommandNotFoundResult("git", res) ? { error, status: 503 } : { error };
 }
 function runGitRefLookup(args, cwd) {
   const res = run(args, cwd);
@@ -2534,10 +2529,19 @@ async function untrackedMetaAsync(cwd) {
   return results.filter((meta) => meta !== null);
 }
 async function fileMetaResultAsync(args, cwd, includeUntracked = false) {
-  const ns = await nameStatusResultAsync(args, cwd);
+  let ns;
+  let nm;
+  if (args.includes("--")) {
+    [ns, nm] = await Promise.all([
+      nameStatusResultAsync(args, cwd),
+      numstatZResultAsync(args, cwd)
+    ]);
+  } else {
+    ns = await nameStatusResultAsync(args, cwd);
+    nm = await numstatZResultAsync(args, cwd);
+  }
   if (ns.error)
     return { files: [], error: ns.error };
-  const nm = await numstatZResultAsync(args, cwd);
   if (nm.error)
     return { files: [], error: nm.error };
   const byPath = new Map(nm.files.map((file) => [file.path, file]));
@@ -15054,6 +15058,7 @@ function startWorktreeUpdateWatch(options) {
   const initialScanAsync = options.initialScanMode === "async" || (!options.watch || options.watch === nodeWatch) && !options.readdirSync;
   const initialScanQueue = [];
   let initialScanTimer = null;
+  let processingInitialScan = false;
   const pendingPathInspections = new Map;
   let pathInspectionTimer = null;
   let timer = null;
@@ -15141,8 +15146,14 @@ function startWorktreeUpdateWatch(options) {
       return;
     }
     const next = initialScanQueue.shift();
-    if (next)
-      watchDirectory(next, true);
+    if (next) {
+      processingInitialScan = true;
+      try {
+        watchDirectory(next, true);
+      } finally {
+        processingInitialScan = false;
+      }
+    }
     if (watchers.size >= maxWatchedDirectories) {
       reportWatchLimit();
       initialScanQueue.length = 0;
@@ -15160,7 +15171,7 @@ function startWorktreeUpdateWatch(options) {
     if (children.length > remaining)
       reportWatchLimit();
     initialScanQueue.push(...children.slice(0, remaining));
-    if (!initialScanTimer)
+    if (!initialScanTimer && !processingInitialScan)
       initialScanTimer = setTimer(processInitialScanQueue, 5000);
   };
   const processChangedPath = (changed, fullChangedPath) => {
@@ -24819,7 +24830,7 @@ function buildQuery(params) {
   const s = q.toString();
   return s ? `?${s}` : "";
 }
-function fileToMeta(file, range, extraQs) {
+function fileToMeta(file, range, extraQs, responseGeneration) {
   const sizeClass = classify(file);
   const q = {
     path: file.path,
@@ -24827,6 +24838,7 @@ function fileToMeta(file, range, extraQs) {
     status: file.status,
     from: range.from,
     to: range.to,
+    generation: responseGeneration,
     ...extraQs
   };
   if (file.untracked)
@@ -24865,7 +24877,7 @@ async function computePayload(extras, range, pathFilter = "", responseGeneration
     };
   }
   const { args, refs } = buildRangeArgs(range);
-  const fullArgs = [...extras, ...args];
+  const fullArgs = pathFilter ? [...extras, ...args, "--", pathFilter] : [...extras, ...args];
   const metaResult = await fileMetaResultAsync(fullArgs, cwd, false);
   const files = metaResult.files;
   if (!metaResult.error && includeUntracked(range, refs)) {
@@ -24883,7 +24895,7 @@ async function computePayload(extras, range, pathFilter = "", responseGeneration
     if (e === "--ignore-blank-lines")
       extraQs.ignore_blank = "1";
   }
-  const meta = filteredFiles.map((file) => fileToMeta(file, range, extraQs));
+  const meta = filteredFiles.map((file) => fileToMeta(file, range, extraQs, responseGeneration));
   const totals = meta.reduce((acc, file) => {
     acc.additions += file.additions || 0;
     acc.deletions += file.deletions || 0;
@@ -24902,7 +24914,7 @@ async function computePayload(extras, range, pathFilter = "", responseGeneration
   };
 }
 async function handleDiffJson(url) {
-  const responseGeneration = generation;
+  let responseGeneration = generation;
   const extras = [];
   if (url.searchParams.get("ignore_ws") === "1")
     extras.push("-w");
@@ -24928,9 +24940,15 @@ async function handleDiffJson(url) {
   const requestSequence = ++diffMetaRequestSequence;
   latestDiffMetaRequest.set(key, requestSequence);
   try {
-    const payload = await computePayload(extras, range, path, responseGeneration);
-    if (latestDiffMetaRequest.get(key) !== requestSequence || responseGeneration !== generation)
+    let payload = await computePayload(extras, range, path, responseGeneration);
+    if (latestDiffMetaRequest.get(key) !== requestSequence)
       return json2(payload);
+    if (responseGeneration !== generation) {
+      responseGeneration = generation;
+      payload = await computePayload(extras, range, path, responseGeneration);
+      if (latestDiffMetaRequest.get(key) !== requestSequence || responseGeneration !== generation)
+        return json2(payload);
+    }
     const sig = JSON.stringify({ ...payload, generation: undefined });
     if (noCache && (!cached || cached.sig !== sig)) {
       generation++;
@@ -25399,7 +25417,7 @@ async function handleFileBlame(url) {
   return json2({ ...result, base, ref, generation: responseGeneration });
 }
 async function handleFileDiff(url) {
-  const responseGeneration = generation;
+  const serverGenerationAtRequest = generation;
   const path = url.searchParams.get("path") || "";
   if (!safePath(path))
     return text("invalid path", 400);
@@ -25413,6 +25431,9 @@ async function handleFileDiff(url) {
     from: url.searchParams.get("from") || "",
     to: url.searchParams.get("to") || ""
   };
+  const requestedGeneration = Number(url.searchParams.get("generation"));
+  const usesWorktree = !range.from || range.from === "worktree" || !range.to || range.to === "worktree";
+  const responseGeneration = !usesWorktree && Number.isSafeInteger(requestedGeneration) && requestedGeneration > 0 ? requestedGeneration : serverGenerationAtRequest;
   if (isSameWorktreeRange(range)) {
     return json2({
       path,
@@ -25466,7 +25487,7 @@ async function handleFileDiff(url) {
         errStatus = res.status ?? 500;
       }
     }
-    if (!errText && responseGeneration === generation)
+    if (!errText && serverGenerationAtRequest === generation)
       setTimedCacheEntry(fileCache, cacheKey, { diffText });
   }
   if (errStatus)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@youtyan/code-viewer",
-  "version": "0.8.6",
+  "version": "0.8.8",
   "description": "Local browser-based code and git diff viewer",
   "type": "module",
   "bin": {