@youtyan/code-viewer 0.8.5 → 0.8.6

package/package.json

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

package/web/app.js

@@ -9672,6 +9672,10 @@ ${frontmatter.yaml}
   }
 
   // web-src/views/file-shell.ts
+  function isMediaPreviewOnlySource(path) {
+    const kind = sourceDisplayKind(path);
+    return kind === "image" || kind === "video" || kind === "audio" || kind === "pdf";
+  }
   function isBlobOrBlameFileRoute(route) {
     return route.screen === "file" && (route.view === "blob" || route.view === "blame");
   }
@@ -9724,14 +9728,18 @@ ${frontmatter.yaml}
   }
   function appendFileViewTabs(tabs, deps, target, activeTab, options = {}) {
     const includeFileTabs = options.includeFileTabs !== false;
-    const showPreview = options.previewable ?? (isPreviewableSource(target.path) || activeTab === "preview");
+    const mediaOnly = options.previewable === undefined && isMediaPreviewOnlySource(target.path);
+    const showPreview = mediaOnly || (options.previewable ?? (isPreviewableSource(target.path) || activeTab === "preview"));
     let previewButton = null;
     if (showPreview) {
-      previewButton = createBlobSourceTabButton(deps, target, "preview", activeTab === "preview", options);
+      previewButton = createBlobSourceTabButton(deps, target, "preview", activeTab === "preview" || mediaOnly && activeTab === "code", options);
       tabs.appendChild(previewButton);
     }
-    const codeButton = createBlobSourceTabButton(deps, target, "code", activeTab === "code", options);
-    tabs.appendChild(codeButton);
+    let codeButton = null;
+    if (!mediaOnly) {
+      codeButton = createBlobSourceTabButton(deps, target, "code", activeTab === "code", options);
+      tabs.appendChild(codeButton);
+    }
     if (includeFileTabs) {
       tabs.appendChild(createFileViewTabButton(deps, target, "blame", "Blame", activeTab === "blame"));
       tabs.appendChild(createFileViewTabButton(deps, target, "history", "History", activeTab === "history"));
@@ -26044,7 +26052,7 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
                 },
                 {
                   kind: "paragraph",
-                  text: "A file detail page exposes four tabs — Preview, Code, Blame, History. Code is the default and ?preview=1 opts in to the Markdown / media preview. Blame and History have their own canonical URLs (view=blame, view=history) so deep links and the browser back/forward stay in sync, and they keep the Repository sidebar visible."
+                  text: "A file detail page exposes up to four tabs — Preview, Code, Blame, History. For text files Code is the default and ?preview=1 opts in to the Markdown / HTML preview; media files (images, video, audio, PDF) show a Preview tab only, with no Code tab. Blame and History have their own canonical URLs (view=blame, view=history) so deep links and the browser back/forward stay in sync, and they keep the Repository sidebar visible."
                 },
                 {
                   kind: "paragraph",
@@ -26715,7 +26723,7 @@ code-viewer query agent-help`
                 },
                 {
                   kind: "paragraph",
-                  text: "ファイル詳細ページには Preview / Code / Blame / History の 4 つのタブがあります。Code がデフォルトで、?preview=1 を付けると Markdown / メディアプレビューに切り替わります。Blame と History はそれぞれ専用 URL (view=blame, view=history) を持つため、ディープリンクとブラウザの戻る / 進むが同期し、いずれも Repository サイドバーは表示されたままです。"
+                  text: "ファイル詳細ページには最大 4 つのタブ (Preview / Code / Blame / History) があります。テキストファイルは Code がデフォルトで、?preview=1 を付けると Markdown / HTML プレビューに切り替わります。画像・動画・音声・PDF などのメディアファイルは Preview タブのみ表示され、Code タブは表示されません。Blame と History はそれぞれ専用 URL (view=blame, view=history) を持つため、ディープリンクとブラウザの戻る / 進むが同期し、いずれも Repository サイドバーは表示されたままです。"
                 },
                 {
                   kind: "paragraph",
@@ -34083,6 +34091,8 @@ code-viewer query agent-help`
       });
     }
     function applyRenderedSourceTab(tab, codeButton, previewButton, codePane, previewPane, updateRoute = true) {
+      if (!codeButton)
+        return false;
       if (tab === "preview" && (!previewButton || !previewPane))
         return false;
       codeButton.classList.toggle("active", tab === "code");
@@ -34332,6 +34342,7 @@ code-viewer query agent-help`
         body.replaceWith(view);
       else
         card.appendChild(view);
+      setSourceCardState(card, "error");
     }
     function renderSourceCancelled(card, target) {
       const body = card.querySelector(".gdp-file-detail-body, .d2h-files-diff, .d2h-file-diff, .gdp-media, .gdp-source-viewer");
@@ -34356,6 +34367,7 @@ code-viewer query agent-help`
         body.replaceWith(view);
       else
         card.appendChild(view);
+      setSourceCardState(card, "cancelled");
     }
     function renderSourceUnsupported(card, target) {
       const body = card.querySelector(".gdp-file-detail-body, .d2h-files-diff, .d2h-file-diff, .gdp-media, .gdp-source-viewer");
@@ -34374,6 +34386,7 @@ code-viewer query agent-help`
         body.replaceWith(view);
       else
         card.appendChild(view);
+      setSourceCardState(card, "done");
     }
     function renderSourceInternalPath(card, target, kind) {
       const body = card.querySelector(".gdp-file-detail-body, .d2h-files-diff, .d2h-file-diff, .gdp-media, .gdp-source-viewer");
@@ -34398,6 +34411,7 @@ code-viewer query agent-help`
         body.replaceWith(view);
       else
         card.appendChild(view);
+      setSourceCardState(card, "done");
     }
     function renderHtmlPreview(target, html) {
       return renderHtmlPreviewFrame(`${target.path} preview`, html);
@@ -34511,7 +34525,7 @@ code-viewer query agent-help`
           previewButton2?.addEventListener("click", () => {
             applyRenderedSourceTab("preview", codeButton2, previewButton2, virtualCode, preview);
           });
-          codeButton2.addEventListener("click", () => {
+          codeButton2?.addEventListener("click", () => {
             applyRenderedSourceTab("code", codeButton2, previewButton2, virtualCode, preview);
           });
           if (header)
@@ -34606,7 +34620,7 @@ code-viewer query agent-help`
         previewButton?.addEventListener("click", () => {
           applyRenderedSourceTab("preview", codeButton, previewButton, table2, preview);
         });
-        codeButton.addEventListener("click", () => {
+        codeButton?.addEventListener("click", () => {
           applyRenderedSourceTab("code", codeButton, previewButton, table2, preview);
         });
         if (header)
@@ -34926,7 +34940,7 @@ code-viewer query agent-help`
         e2.preventDefault();
         const url = new URL(full.href, window.location.origin);
         setRoute(parseRoute(url.pathname, url.search, currentRange()), true);
-        renderStandaloneSource(target);
+        renderStandaloneSource(target, { refresh: true });
       });
       actions.append(copy, full);
       info.append(badge, summary, actions);
@@ -35042,7 +35056,7 @@ code-viewer query agent-help`
         e2.preventDefault();
         const url = new URL(full.href, window.location.origin);
         setRoute(parseRoute(url.pathname, url.search, currentRange()), true);
-        renderStandaloneSource(target);
+        renderStandaloneSource(target, { refresh: true });
       });
       actions.append(raw, full);
       info.append(badge, summary, actions);
@@ -35308,6 +35322,7 @@ code-viewer query agent-help`
         body.replaceWith(view);
       else
         card.appendChild(view);
+      setSourceCardState(card, "done");
     }
     function _renderSourceBinary(card, target) {
       const body = card.querySelector(".gdp-file-detail-body, .d2h-files-diff, .d2h-file-diff, .gdp-media, .gdp-source-viewer");
@@ -35331,7 +35346,30 @@ code-viewer query agent-help`
       else
         card.appendChild(view);
     }
-    async function renderStandaloneSource(target) {
+    function mountedStandaloneSourceCard(target) {
+      const card = document.querySelector(".gdp-standalone-source");
+      if (!card)
+        return null;
+      if (card.dataset.path !== target.path)
+        return null;
+      if (card.dataset.sourceRef !== (target.ref || "worktree"))
+        return null;
+      return card;
+    }
+    function setSourceCardState(card, state) {
+      card.dataset.sourceState = state;
+    }
+    async function renderStandaloneSource(target, options = {}) {
+      if (!options.refresh) {
+        const mounted = mountedStandaloneSourceCard(target);
+        const state = mounted?.dataset.sourceState;
+        if (mounted && state === "done") {
+          scrollStandaloneSourceLine(mounted, lineTargetStart(STATE.route.screen === "file" ? STATE.route.line : undefined));
+          return;
+        }
+        if (mounted && state === "loading")
+          return;
+      }
       cancelActiveSourceLoad("navigation");
       const req = ++SOURCE_REQ_SEQ;
       const repoTarget = repoFileTargetFromRoute();
@@ -35343,10 +35381,13 @@ code-viewer query agent-help`
       const card = document.createElement("article");
       card.className = "gdp-file-shell loaded gdp-standalone-source gdp-source-mode";
       card.dataset.path = target.path;
+      card.dataset.sourceRef = target.ref || "worktree";
+      setSourceCardState(card, "loading");
       const internalKind = sourceInternalPathKind(target.path);
       const wrapper = document.createElement("div");
       wrapper.className = "gdp-file-detail-wrapper";
-      const activeTab = !internalKind && STATE.route.screen === "file" && STATE.route.preview ? "preview" : "code";
+      const mediaOnly = !internalKind && isMediaPreviewOnlySource(target.path);
+      const activeTab = mediaOnly || !internalKind && STATE.route.screen === "file" && STATE.route.preview ? "preview" : "code";
       const { sticky, header } = createFileShellSticky({
         currentRange,
         setRoute,
@@ -35427,9 +35468,13 @@ code-viewer query agent-help`
             const rendered2 = await renderPagedSourceText(card, target, meta.size, controller.signal);
             if (req !== SOURCE_REQ_SEQ || !sourceTargetsEqual(sourceTargetFromRoute(), target))
               return;
-            if (!rendered2)
+            if (!rendered2) {
+              finishSourceLoad(req);
+              renderSourceError(card, target, `Cannot load ${target.path} at ${target.ref}`);
               return;
+            }
             scrollStandaloneSourceLine(card, lineTargetStart(STATE.route.screen === "file" ? STATE.route.line : undefined));
+            setSourceCardState(card, "done");
             finishSourceLoad(req);
             return;
           }
@@ -35450,6 +35495,7 @@ code-viewer query agent-help`
           if (!rendered)
             return;
           scrollStandaloneSourceLine(card, lineTargetStart(STATE.route.screen === "file" ? STATE.route.line : undefined));
+          setSourceCardState(card, "done");
           finishSourceLoad(req);
         }
       } catch (err) {
@@ -35477,7 +35523,7 @@ code-viewer query agent-help`
       if (row)
         row.scrollIntoView({ block: "center" });
     }
-    function applySourceRouteToShell() {
+    function applySourceRouteToShell(options = {}) {
       const target = sourceTargetFromRoute();
       setPageMode();
       if (!target) {
@@ -35487,7 +35533,7 @@ code-viewer query agent-help`
         });
         return;
       }
-      renderStandaloneSource(target);
+      renderStandaloneSource(target, options);
     }
     function inferLang(path) {
       const name = sourceFileName(path);
@@ -37636,12 +37682,12 @@ code-viewer query agent-help`
         history.replaceState(historyStateForRoute(STATE.route), "", url + window.location.hash);
       }
     }
-    function dispatchFileRoute(route) {
+    function dispatchFileRoute(route, options = {}) {
       if (route.view === "blob") {
         setStatus("live");
         removeFileHistoryShell2();
         BLAME_VIEW.removeBlamePage();
-        applySourceRouteToShell();
+        applySourceRouteToShell(options);
         return true;
       }
       if (route.view === "blame") {
@@ -38649,7 +38695,7 @@ code-viewer query agent-help`
         setStatus("live");
         return Promise.resolve(null);
       }
-      if (STATE.route.screen === "file" && !(isFileHistoryRoute(STATE.route) && activeHistoryPathFilter) && dispatchFileRoute(STATE.route)) {
+      if (STATE.route.screen === "file" && !(isFileHistoryRoute(STATE.route) && activeHistoryPathFilter) && dispatchFileRoute(STATE.route, { refresh: true })) {
         return Promise.resolve({
           structureChanged: false,
           invalidatedCards: 0,
@@ -39342,7 +39388,7 @@ code-viewer query agent-help`
       if (!shouldAutoLoadCurrentRoute(route))
         return;
       if (isBlobOrBlameFileRoute(route)) {
-        dispatchFileRoute(route);
+        dispatchFileRoute(route, { refresh: true });
         return;
       }
       doSseLoad(paths);
@@ -39382,7 +39428,7 @@ code-viewer query agent-help`
         const viewingPath = route.path;
         if (paths && viewingPath && !paths.has(viewingPath))
           return;
-        dispatchFileRoute(route);
+        dispatchFileRoute(route, { refresh: true });
         return;
       }
       if (route.screen === "repo") {