@hienlh/ppm 0.13.57 → 0.13.59

package/src/web/components/editor/pdf-preview.tsx

@@ -1,44 +1,36 @@
-import { useCallback, useEffect, useRef, useState } from "react";
+import { useCallback, useEffect, useMemo, useRef, useState } from "react";
 import { Loader2, FileWarning, ExternalLink } from "lucide-react";
-import { useBlobUrl } from "./use-blob-url";
+import { projectUrl, getAuthToken } from "@/lib/api-client";
 
 export function PdfPreview({ filePath, projectName }: { filePath: string; projectName: string }) {
-  const [refreshKey, setRefreshKey] = useState(0);
   const iframeRef = useRef<HTMLIFrameElement>(null);
-  const scrollYRef = useRef(0);
+  const [loaded, setLoaded] = useState(false);
+  const [error, setError] = useState(false);
 
-  const { blobUrl, error } = useBlobUrl(filePath, projectName, "application/pdf", refreshKey);
+  // Build stable direct URL (no blob) so reload() preserves scroll
+  const iframeSrc = useMemo(() => {
+    const isExternal = /^(\/|[A-Za-z]:[/\\])/.test(filePath);
+    const base = isExternal
+      ? `/api/fs/raw?path=${encodeURIComponent(filePath)}`
+      : `${projectUrl(projectName)}/files/raw?path=${encodeURIComponent(filePath)}`;
+    const token = getAuthToken();
+    return token ? `${base}&token=${encodeURIComponent(token)}` : base;
+  }, [filePath, projectName]);
 
-  // Auto-reload: listen for file:changed WS events
+  // Auto-reload on file change — reload() preserves browser PDF viewer scroll
   useEffect(() => {
     const handler = (e: Event) => {
       const detail = (e as CustomEvent).detail;
       if (detail.projectName !== projectName || detail.path !== filePath) return;
-      // Save scroll position before re-fetch (best-effort for browser PDF viewer)
       try {
-        const win = iframeRef.current?.contentWindow;
-        if (win) scrollYRef.current = win.scrollY || 0;
-      } catch { /* cross-origin or plugin restriction */ }
-      setRefreshKey((k) => k + 1);
+        iframeRef.current?.contentWindow?.location.reload();
+      } catch { /* cross-origin fallback — shouldn't happen for same-origin */ }
     };
     window.addEventListener("file:changed", handler);
     return () => window.removeEventListener("file:changed", handler);
   }, [filePath, projectName]);
 
-  // Restore scroll after new blob loads
-  useEffect(() => {
-    if (!blobUrl || !iframeRef.current || scrollYRef.current === 0) return;
-    const iframe = iframeRef.current;
-    const saved = scrollYRef.current;
-    const onLoad = () => {
-      try { iframe.contentWindow?.scrollTo(0, saved); } catch { /* ignore */ }
-      scrollYRef.current = 0;
-    };
-    iframe.addEventListener("load", onLoad);
-    return () => iframe.removeEventListener("load", onLoad);
-  }, [blobUrl]);
-
-  const openInNewTab = useCallback(() => { if (blobUrl) window.open(blobUrl, "_blank"); }, [blobUrl]);
+  const openInNewTab = useCallback(() => { window.open(iframeSrc, "_blank"); }, [iframeSrc]);
 
   if (error) {
     return (
@@ -48,18 +40,27 @@ export function PdfPreview({ filePath, projectName }: { filePath: string; projec
       </div>
     );
   }
-  if (!blobUrl) {
-    return <div className="flex items-center justify-center h-full"><Loader2 className="size-5 animate-spin text-text-subtle" /></div>;
-  }
   return (
     <div className="flex flex-col h-full">
-      <div className="flex items-center justify-between px-3 py-1.5 border-b border-border bg-background shrink-0">
-        <span className="text-xs text-text-secondary truncate">{filePath}</span>
-        <button onClick={openInNewTab} className="flex items-center gap-1 text-xs text-text-secondary hover:text-text-primary transition-colors">
-          <ExternalLink className="size-3" /> Open in new tab
-        </button>
+      {!loaded && (
+        <div className="flex items-center justify-center h-full"><Loader2 className="size-5 animate-spin text-text-subtle" /></div>
+      )}
+      <div className={`flex flex-col h-full ${loaded ? "" : "hidden"}`}>
+        <div className="flex items-center justify-between px-3 py-1.5 border-b border-border bg-background shrink-0">
+          <span className="text-xs text-text-secondary truncate">{filePath}</span>
+          <button onClick={openInNewTab} className="flex items-center gap-1 text-xs text-text-secondary hover:text-text-primary transition-colors">
+            <ExternalLink className="size-3" /> Open in new tab
+          </button>
+        </div>
+        <iframe
+          ref={iframeRef}
+          src={iframeSrc}
+          title={filePath}
+          className="flex-1 w-full border-none"
+          onLoad={() => setLoaded(true)}
+          onError={() => setError(true)}
+        />
       </div>
-      <iframe ref={iframeRef} src={blobUrl} title={filePath} className="flex-1 w-full border-none" />
     </div>
   );
 }

package/dist/web/assets/keybindings-store-DWD9jyRg.js

@@ -1 +0,0 @@
-import"./vendor-markdown-0Mxgxy0L.js";import"./api-client-DiZgVOok.js";import{D as e}from"./index-heahuZzl.js";export{e as useKeybindingsStore};

package/dist/web/assets/notification-store-DcY2JHZt.js

@@ -1 +0,0 @@
-import"./vendor-markdown-0Mxgxy0L.js";import"./api-client-DiZgVOok.js";import{P as e}from"./index-heahuZzl.js";export{e as useNotificationStore};

package/dist/web/assets/pdf-preview-BukxhJ6o.js

@@ -1 +0,0 @@
-import{o as e}from"./rolldown-runtime-FhOqtrmT.js";import{b as t,x as n}from"./vendor-markdown-0Mxgxy0L.js";import{t as r}from"./file-exclamation-point-B__2Hrd6.js";import"./api-client-DiZgVOok.js";import{$ as i,ct as a}from"./index-heahuZzl.js";import{t as o}from"./use-blob-url-DCUIEzjB.js";var s=e(n(),1),c=t();function l({filePath:e,projectName:t}){let[n,l]=(0,s.useState)(0),u=(0,s.useRef)(null),d=(0,s.useRef)(0),{blobUrl:f,error:p}=o(e,t,`application/pdf`,n);(0,s.useEffect)(()=>{let n=n=>{let r=n.detail;if(!(r.projectName!==t||r.path!==e)){try{let e=u.current?.contentWindow;e&&(d.current=e.scrollY||0)}catch{}l(e=>e+1)}};return window.addEventListener(`file:changed`,n),()=>window.removeEventListener(`file:changed`,n)},[e,t]),(0,s.useEffect)(()=>{if(!f||!u.current||d.current===0)return;let e=u.current,t=d.current,n=()=>{try{e.contentWindow?.scrollTo(0,t)}catch{}d.current=0};return e.addEventListener(`load`,n),()=>e.removeEventListener(`load`,n)},[f]);let m=(0,s.useCallback)(()=>{f&&window.open(f,`_blank`)},[f]);return p?(0,c.jsxs)(`div`,{className:`flex flex-col items-center justify-center h-full gap-3 text-text-secondary`,children:[(0,c.jsx)(r,{className:`size-10 text-text-subtle`}),(0,c.jsx)(`p`,{className:`text-sm`,children:`Failed to load PDF.`})]}):f?(0,c.jsxs)(`div`,{className:`flex flex-col h-full`,children:[(0,c.jsxs)(`div`,{className:`flex items-center justify-between px-3 py-1.5 border-b border-border bg-background shrink-0`,children:[(0,c.jsx)(`span`,{className:`text-xs text-text-secondary truncate`,children:e}),(0,c.jsxs)(`button`,{onClick:m,className:`flex items-center gap-1 text-xs text-text-secondary hover:text-text-primary transition-colors`,children:[(0,c.jsx)(a,{className:`size-3`}),` Open in new tab`]})]}),(0,c.jsx)(`iframe`,{ref:u,src:f,title:e,className:`flex-1 w-full border-none`})]}):(0,c.jsx)(`div`,{className:`flex items-center justify-center h-full`,children:(0,c.jsx)(i,{className:`size-5 animate-spin text-text-subtle`})})}export{l as PdfPreview};