@youtyan/code-viewer 0.13.0 → 0.13.1

package/package.json

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

package/web/app.js

@@ -875,6 +875,17 @@ Details: ${JSON.stringify(output)}` : "";
     };
   }
 
+  // web-src/core/file-refresh.ts
+  function rawFileInfoSignature(info) {
+    if (info.missing) return "missing";
+    if (info.size == null && !info.updated_at && !info.commit_updated_at)
+      return null;
+    return `${info.size ?? ""}|${info.updated_at ?? ""}|${info.commit_updated_at ?? ""}`;
+  }
+  function fileSignatureUnchanged(stored, key, sig) {
+    return stored !== null && stored.key === key && stored.sig === sig;
+  }
+
   // web-src/core/focus-scope.ts
   function isEditableKeyTarget(target) {
     if (!target) return false;
@@ -25714,6 +25725,18 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         (f2) => `${fileKey2(f2)}\0${f2.path}\0${f2.old_path || ""}\0${f2.status || "M"}`
       ).join("\n");
     }
+    function stripLoadUrlGeneration(url) {
+      const queryStart = url.indexOf("?");
+      if (queryStart === -1) return url;
+      const params = new URLSearchParams(url.slice(queryStart + 1));
+      if (!params.has("generation")) return url;
+      params.delete("generation");
+      const s2 = params.toString();
+      return s2 ? `${url.slice(0, queryStart)}?${s2}` : url.slice(0, queryStart);
+    }
+    function diffResponseSignature(data) {
+      return JSON.stringify({ ...data, generation: void 0 });
+    }
     function computeCardSignature(f2) {
       return [
         fileKey2(f2),
@@ -25725,12 +25748,18 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         f2.size_class || "small",
         f2.force_layout || "",
         f2.highlight ? 1 : 0,
-        f2.load_url,
-        f2.preview_url || "",
+        stripLoadUrlGeneration(f2.load_url),
+        f2.preview_url ? stripLoadUrlGeneration(f2.preview_url) : "",
         f2.estimated_height_px || 0,
         f2.untracked ? 1 : 0
       ].join("\0");
     }
+    function cardNeedsLoad(card) {
+      if (card.classList.contains("loading")) return false;
+      if (card.dataset.staleLoading === "1") return false;
+      if (card.classList.contains("loaded")) return card.dataset.stale === "1";
+      return true;
+    }
     function ensureLazyObserver() {
       if (!lazyObserver) {
         lazyObserver = new IntersectionObserver(
@@ -25738,8 +25767,7 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
             for (const entry of entries) {
               if (!entry.isIntersecting) continue;
               const card = entry.target;
-              if (card.classList.contains("loaded") || card.classList.contains("loading"))
-                continue;
+              if (!cardNeedsLoad(card)) continue;
               const f2 = card._file || STATE.files.find((x) => x.path === card.dataset.path);
               if (f2) enqueueLoad(f2, card, 0);
             }
@@ -25761,15 +25789,27 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         return false;
       if (!changedPathsCoverPath(changedPaths, file.path)) return false;
       card.dataset.reqId = String(++CLIENT_REQ_SEQ);
+      delete card.dataset.stale;
+      delete card.dataset.staleLoading;
       card.style.minHeight = `${measuredHeight ?? card.offsetHeight}px`;
       card.classList.remove("loaded", "loading", "error");
       card.classList.add("pending");
       card._diffData = null;
+      card._loadedSig = null;
+      card._loadedSigUrl = null;
       const indicator = card.querySelector(".loading-indicator");
       if (indicator) indicator.hidden = false;
       activatePendingCard(card, file);
       return true;
     }
+    function revalidateCardSilently(card, file) {
+      if (!card.classList.contains("loaded")) return false;
+      card.dataset.reqId = String(++CLIENT_REQ_SEQ);
+      card.dataset.stale = "1";
+      card._file = file;
+      activatePendingCard(card, file);
+      return true;
+    }
     function updateSidebarStats(files) {
       for (const f2 of files) {
         const li = document.querySelector(
@@ -25800,8 +25840,12 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
       delete card.dataset.manualRendered;
       delete card.dataset.manualLoad;
       delete card.dataset.manualMode;
+      delete card.dataset.stale;
+      delete card.dataset.staleLoading;
       card.style.minHeight = `${file.estimated_height_px || 80}px`;
       card._diffData = null;
+      card._loadedSig = null;
+      card._loadedSigUrl = null;
       card._file = file;
     }
     function renderShell(meta, changedPaths) {
@@ -25877,23 +25921,37 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
             activatePendingCard(card, f2);
             invalidatedCards++;
             sidebarNeedsStatsUpdate = true;
-          } else {
+          } else if (sigChanged) {
             const stats = card.querySelector(".gdp-shell-header .stats");
             if (stats) {
               stats.innerHTML = '<span class="a">+' + (f2.additions || 0) + '</span><span class="d">−' + (f2.deletions || 0) + "</span>";
             }
             card._file = f2;
-            const didInvalidate = sigChanged ? invalidateLoadedCard(card, f2, null, measuredHeights.get(key)) : invalidateLoadedCard(
+            const didInvalidate = invalidateLoadedCard(
               card,
               f2,
-              changedPaths,
+              null,
               measuredHeights.get(key)
             );
             if (didInvalidate) invalidatedCards++;
-            else if ((sigChanged || pathHint) && card.classList.contains("pending")) {
+            else if (card.classList.contains("pending")) {
               activatePendingCard(card, f2);
             }
-            if (sigChanged) sidebarNeedsStatsUpdate = true;
+            sidebarNeedsStatsUpdate = true;
+          } else {
+            card._file = f2;
+            if (!revalidateCardSilently(card, f2)) {
+              const didInvalidate = invalidateLoadedCard(
+                card,
+                f2,
+                changedPaths,
+                measuredHeights.get(key)
+              );
+              if (didInvalidate) invalidatedCards++;
+              else if (card.classList.contains("pending")) {
+                activatePendingCard(card, f2);
+              }
+            }
           }
         }
         if (sidebarNeedsStatsUpdate && canUpdateSidebar)
@@ -25988,8 +26046,7 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
           entries.forEach((entry) => {
             if (!entry.isIntersecting) return;
             const card = entry.target;
-            if (card.classList.contains("loaded") || card.classList.contains("loading"))
-              return;
+            if (!cardNeedsLoad(card)) return;
             const f2 = card._file || STATE.files.find((x) => x.path === card.dataset.path);
             if (!f2) return;
             enqueueLoad(f2, card, 0);
@@ -26030,8 +26087,7 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
       while (ACTIVE_LOADS < MAX_PARALLEL && LOAD_QUEUE.length) {
         const item = LOAD_QUEUE.shift();
         if (item.epoch !== LOAD_EPOCH) continue;
-        if (item.card.classList.contains("loaded") || item.card.classList.contains("loading"))
-          continue;
+        if (!cardNeedsLoad(item.card)) continue;
         ACTIVE_LOADS++;
         loadFile(item.file, item.card).finally(() => {
           if (item.epoch === LOAD_EPOCH) {
@@ -26126,20 +26182,29 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
       });
     }
     function loadFile(file, card, urlOverride, options) {
-      card.classList.remove("pending");
-      card.classList.add("loading");
-      if (lazyObserver) lazyObserver.unobserve(card);
+      const silent = card.classList.contains("loaded") && card.dataset.stale === "1";
       const indicator = card.querySelector(".loading-indicator");
-      if (indicator) indicator.hidden = false;
+      if (silent) {
+        card.dataset.staleLoading = "1";
+      } else {
+        card.classList.remove("pending");
+        card.classList.add("loading");
+        if (indicator) indicator.hidden = false;
+      }
+      if (lazyObserver) lazyObserver.unobserve(card);
       const url = urlOverride || (card.dataset.manualMode === "full" ? file.load_url : file.preview_url || file.load_url);
       const myGen = getServerGeneration();
       const myReq = ++CLIENT_REQ_SEQ;
       card.dataset.reqId = String(myReq);
       const retryStale = () => {
         if (String(myReq) !== card.dataset.reqId) return;
-        card.classList.remove("loading");
-        card.classList.add("pending");
-        if (indicator) indicator.hidden = true;
+        if (silent) {
+          delete card.dataset.staleLoading;
+        } else {
+          card.classList.remove("loading");
+          card.classList.add("pending");
+          if (indicator) indicator.hidden = true;
+        }
         const fresh = card._file || STATE.files.find((x) => x.path === card.dataset.path);
         if (fresh && card.isConnected) enqueueLoad(fresh, card, 0);
       };
@@ -26158,11 +26223,32 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
           retryStale();
           return;
         }
+        const strippedUrl = stripLoadUrlGeneration(url);
+        if (silent) {
+          const sig = diffResponseSignature(data);
+          if (card._loadedSigUrl === strippedUrl && card._loadedSig === sig) {
+            delete card.dataset.staleLoading;
+            delete card.dataset.stale;
+            return;
+          }
+        }
         if (!options?.immediate) await nextIdle();
         if (String(myReq) !== card.dataset.reqId) return;
+        if (silent) {
+          delete card.dataset.staleLoading;
+          delete card.dataset.stale;
+        }
+        card._loadedSig = diffResponseSignature(data);
+        card._loadedSigUrl = strippedUrl;
         renderFile(file, data, card);
       }).catch((error2) => {
         if (String(myReq) !== card.dataset.reqId) return;
+        if (silent) {
+          delete card.dataset.staleLoading;
+          delete card.dataset.stale;
+          console.error("[code-viewer] silent diff revalidation failed", error2);
+          return;
+        }
         console.error("[code-viewer] failed to load diff", error2);
         card.classList.remove("loading");
         card.classList.add("error");
@@ -36086,6 +36172,7 @@ code-viewer annotate add-db --db app.db --tab query \\
     async function loadRawFileInfo(target) {
       try {
         const res = await fetch(buildRawFileUrl(target), { method: "HEAD" });
+        if (res.status === 404) return { missing: true };
         if (!res.ok) return {};
         const rawSize = res.headers.get("content-length");
         const size = rawSize == null ? NaN : Number(rawSize);
@@ -47326,12 +47413,53 @@ ${error2.stack}` : ""}`
         );
       }
     }
+    let fileRouteSignature = null;
+    let fileRouteSignatureSeed = null;
+    let fileRouteSignatureCheck = null;
+    function fileRouteSignatureKey(route) {
+      return `${route.view || "blob"}\0${route.path}\0${route.ref || "worktree"}`;
+    }
+    async function readFileRouteSignature(route) {
+      const info = await REPO_VIEW.loadRawFileInfo({
+        path: route.path,
+        ref: route.ref || "worktree"
+      });
+      return rawFileInfoSignature(info);
+    }
+    function seedFileRouteSignature(route) {
+      const key = fileRouteSignatureKey(route);
+      const seed = trackLoad(readFileRouteSignature(route)).then((sig) => {
+        if (fileRouteSignatureSeed === seed) fileRouteSignatureSeed = null;
+        if (sig === null) return;
+        fileRouteSignature = { key, sig };
+      });
+      fileRouteSignatureSeed = seed;
+    }
+    function refreshFileRouteIfChanged(route) {
+      if (fileRouteSignatureCheck) return;
+      const key = fileRouteSignatureKey(route);
+      const pendingSeed = fileRouteSignatureSeed ?? Promise.resolve();
+      fileRouteSignatureCheck = pendingSeed.then(() => trackLoad(readFileRouteSignature(route))).then((sig) => {
+        fileRouteSignatureCheck = null;
+        if (sig === null) return;
+        const routeNow = STATE.route;
+        if (routeNow.screen !== "file" || !isBlobOrBlameFileRoute(routeNow) || fileRouteSignatureKey(routeNow) !== key)
+          return;
+        if (fileSignatureUnchanged(fileRouteSignature, key, sig)) return;
+        fileRouteSignature = { key, sig };
+        dispatchFileRoute(routeNow, { refresh: true });
+      });
+      fileRouteSignatureCheck.catch(() => {
+        fileRouteSignatureCheck = null;
+      });
+    }
     function dispatchFileRoute(route, options = {}) {
       if (route.view === "blob") {
         setStatus("live");
         removeFileHistoryShell2();
         BLAME_VIEW.removeBlamePage();
         applySourceRouteToShell(options);
+        seedFileRouteSignature(route);
         return true;
       }
       if (route.view === "blame") {
@@ -47339,6 +47467,7 @@ ${error2.stack}` : ""}`
         cancelActiveSourceLoad("navigation");
         removeFileHistoryShell2();
         void BLAME_VIEW.renderBlamePage({ path: route.path, ref: route.ref });
+        seedFileRouteSignature(route);
         return true;
       }
       if (route.view === "history") {
@@ -49510,7 +49639,7 @@ ${error2.stack}` : ""}`
       if (isBlobOrBlameFileRoute(route)) {
         const viewingPath = route.path;
         if (viewingPath && !changedPathsCoverPath(paths, viewingPath)) return;
-        dispatchFileRoute(route, { refresh: true });
+        refreshFileRouteIfChanged(route);
         return;
       }
       if (route.screen === "repo") {