dsh-file-activity 0.5.0 → 0.5.2

package/lib/parts/interceptor.part.js

@@ -1,56 +1,56 @@
-    // ── fetch interception: sidebar file operations ───────────────────────
-    function methodOf(init) {
-      return (init?.method ?? 'GET').toUpperCase()
-    }
+// ── fetch interception: sidebar file operations ───────────────────────
+function methodOf(init) {
+  return (init?.method ?? 'GET').toUpperCase()
+}
 
-    /** POST body as a plain object (non-string bodies are ignored). */
-    function parseBody(init) {
-      return typeof init?.body === 'string' ? JSON.parse(init.body) : {}
-    }
+/** POST body as a plain object (non-string bodies are ignored). */
+function parseBody(init) {
+  return typeof init?.body === 'string' ? JSON.parse(init.body) : {}
+}
 
-    /** Record fs.read / fs.write POSTs observed on the sidebar API. */
-    function recordSidebarFs(url, init) {
-      if (url.pathname !== '/sidebar/api/fs.read' && url.pathname !== '/sidebar/api/fs.write') return
-      if (methodOf(init) !== 'POST') return
-      const body = parseBody(init)
-      if (typeof body.sessionId !== 'string' || typeof body.path !== 'string') return
-      postRecord(body.sessionId, body.path, url.pathname === '/sidebar/api/fs.write' ? 'write' : 'read')
-    }
+/** Record fs.read / fs.write POSTs observed on the sidebar API. */
+function recordSidebarFs(url, init) {
+  if (url.pathname !== '/sidebar/api/fs.read' && url.pathname !== '/sidebar/api/fs.write') return
+  if (methodOf(init) !== 'POST') return
+  const body = parseBody(init)
+  if (typeof body.sessionId !== 'string' || typeof body.path !== 'string') return
+  postRecord(body.sessionId, body.path, url.pathname === '/sidebar/api/fs.write' ? 'write' : 'read')
+}
 
-    /** Record sidebar media opens (/sidebar/file?sessionId=...&path=...). */
-    function recordMediaOpen(url, init) {
-      if (url.pathname !== '/sidebar/file' || methodOf(init) !== 'GET') return
-      const sessionId = url.searchParams.get('sessionId')
-      const path = url.searchParams.get('path')
-      if (sessionId !== null && path !== null) postRecord(sessionId, path, 'read')
-    }
+/** Record sidebar media opens (/sidebar/file?sessionId=...&path=...). */
+function recordMediaOpen(url, init) {
+  if (url.pathname !== '/sidebar/file' || methodOf(init) !== 'GET') return
+  const sessionId = url.searchParams.get('sessionId')
+  const path = url.searchParams.get('path')
+  if (sessionId !== null && path !== null) postRecord(sessionId, path, 'read')
+}
 
-    /** Observe a resolved fetch URL and record sidebar file operations. */
-    function observeSidebarFetch(url, init) {
-      try {
-        recordSidebarFs(url, init)
-        recordMediaOpen(url, init)
-      } catch {
-        // observation must never break the underlying call
-      }
-    }
+/** Observe a resolved fetch URL and record sidebar file operations. */
+function observeSidebarFetch(url, init) {
+  try {
+    recordSidebarFs(url, init)
+    recordMediaOpen(url, init)
+  } catch {
+    // observation must never break the underlying call
+  }
+}
 
-    function installFetchInterceptor() {
-      const original = window.fetch.bind(window)
-      window.fetch = (input, init) => {
-        const result = original(input, init)
-        let url
-        try {
-          if (typeof input === 'string') url = new URL(input, window.location.href)
-          else if (input instanceof URL) url = input
-          else return result // Request instances: skip observation
-        } catch {
-          return result
-        }
-        observeSidebarFetch(url, init)
-        return result
-      }
-      return () => {
-        window.fetch = original
-      }
+function installFetchInterceptor() {
+  const original = window.fetch.bind(window)
+  window.fetch = (input, init) => {
+    const result = original(input, init)
+    let url
+    try {
+      if (typeof input === 'string') url = new URL(input, window.location.href)
+      else if (input instanceof URL) url = input
+      else return result // Request instances: skip observation
+    } catch {
+      return result
     }
+    observeSidebarFetch(url, init)
+    return result
+  }
+  return () => {
+    window.fetch = original
+  }
+}

package/lib/parts/preview.part.js

@@ -1,178 +1,223 @@
-    // ── floating preview window (reuses the sidebar's native viewer) ──────
-    /** Resolve a possibly-relative path against the session cwd. */
-    function resolvePath(path, cwd) {
-      if (typeof path !== 'string' || path === '') return path
-      if (path.startsWith('/')) return path
-      if (typeof cwd === 'string' && cwd !== '') return `${cwd.replace(/\/+$/, '')}/${path}`
-      return path
-    }
+// ── floating preview window (reuses the sidebar's native viewer) ──────
+/** Resolve a possibly-relative path against the session cwd. */
+function resolvePath(path, cwd) {
+  if (typeof path !== 'string' || path === '') return path
+  if (path.startsWith('/')) return path
+  if (typeof cwd === 'string' && cwd !== '') return `${cwd.replace(/\/+$/, '')}/${path}`
+  return path
+}
 
-    /** Whether the fs.read API response carries a text content payload. */
-    function isFsReadOk(json) {
-      return json !== null && typeof json === 'object' && json.ok === true && typeof json.value?.content === 'string'
-    }
+/** Whether the fs.read API response carries a text content payload. */
+function isFsReadOk(json) {
+  return json !== null && typeof json === 'object' && json.ok === true && typeof json.value?.content === 'string'
+}
 
-    /** Error load state from an fs.read API response (or a generic message). */
-    function fsReadError(json, viewer) {
-      return { status: 'error', viewer, message: json?.error?.message ?? strings.previewFailed() }
-    }
+/** Error load state from an fs.read API response (or a generic message). */
+function fsReadError(json, viewer) {
+  return { status: 'error', viewer, message: json?.error?.message ?? strings.previewFailed() }
+}
 
-    /**
-     * Load fsRead content through the sidebar API and resolve the viewer's
-     * load state (ready with text, or error with the API message).
-     */
-    async function loadFsReadContent(viewer, path, scope, sessionId) {
-      const target = resolvePath(path, scope?.cwd ?? '')
-      const response = await fetch('/sidebar/api/fs.read', {
-        method: 'POST',
-        headers: { 'content-type': 'application/json' },
-        body: JSON.stringify({ sessionId, path: target }),
-      })
-      const json = await response.json()
-      if (isFsReadOk(json)) return { status: 'ready', viewer, content: json.value.content }
-      return fsReadError(json, viewer)
-    }
+/**
+ * Load fsRead content through the sidebar API and resolve the viewer's
+ * load state (ready with text, or error with the API message).
+ */
+async function loadFsReadContent(viewer, path, scope, sessionId) {
+  const target = resolvePath(path, scope?.cwd ?? '')
+  const response = await fetch('/sidebar/api/fs.read', {
+    method: 'POST',
+    headers: { 'content-type': 'application/json' },
+    body: JSON.stringify({ sessionId, path: target }),
+  })
+  const json = await response.json()
+  if (isFsReadOk(json)) return { status: 'ready', viewer, content: json.value.content }
+  return fsReadError(json, viewer)
+}
 
-    /**
-     * Fetch the bytes the viewer's fetchStrategy needs (fsRead text /
-     * mediaUrl / customData) and resolve its load state.
-     */
-    async function fetchPreviewLoad(viewer, path, scope, sessionId) {
-      const strategy = viewer.fetchStrategy
-      if (strategy === 'fsRead') return loadFsReadContent(viewer, path, scope, sessionId)
-      if (strategy === 'mediaUrl') {
-        return { status: 'ready', viewer, mediaUrl: mediaUrlOf(sessionId, path) }
-      }
-      if (strategy === 'custom') {
-        const data = await (viewer.load?.(path, scope) ?? Promise.resolve(undefined))
-        return { status: 'ready', viewer, customData: data }
+/**
+ * Fetch the bytes the viewer's fetchStrategy needs (fsRead text /
+ * mediaUrl / customData) and resolve its load state.
+ */
+async function fetchPreviewLoad(viewer, path, scope, sessionId) {
+  const strategy = viewer.fetchStrategy
+  if (strategy === 'fsRead') return loadFsReadContent(viewer, path, scope, sessionId)
+  if (strategy === 'mediaUrl') {
+    return { status: 'ready', viewer, mediaUrl: mediaUrlOf(sessionId, path) }
+  }
+  if (strategy === 'custom') {
+    const data = await (viewer.load?.(path, scope) ?? Promise.resolve(undefined))
+    return { status: 'ready', viewer, customData: data }
+  }
+  // 'binary-download' and anything else: mount the viewer's own
+  // component (it handles the download / media itself).
+  return { status: 'ready', viewer }
+}
+
+/**
+ * Resolve the file's viewer through the sidebar registry and load the
+ * bytes it needs; failures become an error state shown in the window.
+ */
+function usePreviewLoader(service, path, sessionId, scope) {
+  const [load, setLoad] = useState({ status: 'loading', viewer: null })
+  useEffect(() => {
+    let cancelled = false
+    const viewer = service?.matchFileViewer?.(path)
+    if (!viewer) {
+      setLoad({ status: 'error', viewer: null, message: strings.previewUnsupported() })
+      return () => {
+        cancelled = true
       }
-      // 'binary-download' and anything else: mount the viewer's own
-      // component (it handles the download / media itself).
-      return { status: 'ready', viewer }
     }
-
-    /**
-     * Resolve the file's viewer through the sidebar registry and load the
-     * bytes it needs; failures become an error state shown in the window.
-     */
-    function usePreviewLoader(service, path, sessionId, scope) {
-      const [load, setLoad] = useState({ status: 'loading', viewer: null })
-      useEffect(() => {
-        let cancelled = false
-        const viewer = service?.matchFileViewer?.(path)
-        if (!viewer) {
-          setLoad({ status: 'error', viewer: null, message: strings.previewUnsupported() })
-          return () => { cancelled = true }
-        }
-        setLoad({ status: 'loading', viewer })
-        fetchPreviewLoad(viewer, path, scope, sessionId)
-          .then((next) => {
-            if (!cancelled) setLoad(next)
-          })
-          .catch((error) => {
-            if (!cancelled) setLoad({ status: 'error', viewer, message: error instanceof Error ? error.message : String(error) })
+    setLoad({ status: 'loading', viewer })
+    fetchPreviewLoad(viewer, path, scope, sessionId)
+      .then((next) => {
+        if (!cancelled) setLoad(next)
+      })
+      .catch((error) => {
+        if (!cancelled)
+          setLoad({
+            status: 'error',
+            viewer,
+            message: error instanceof Error ? error.message : String(error),
           })
-        return () => { cancelled = true }
-      }, [path, sessionId, scope])
-      return load
-    }
-
-    /** Preview window body: loading note / error panel / viewer mount. */
-    function renderPreviewBody(load, ctx, store, scope, path, title, sessionId) {
-      if (load.status === 'loading') {
-        return createElement('div', { className: 'dfa-fp-note' }, strings.loading())
-      }
-      if (load.status === 'error') {
-        return createElement('div', { className: 'dfa-fp-err' },
-          strings.previewFailed(),
-          load.message
-            ? createElement('div', { style: { marginTop: '6px', fontSize: '11px', opacity: 0.85 } }, load.message)
-            : null,
-        )
-      }
-      if (load.viewer.id === 'pdf') {
-        const url = mediaUrlOf(sessionId, path)
-        return createElement(PdfPreview, { src: url, download: `${url}&download=1`, title })
-      }
-      return createElement(load.viewer.component, {
-        ctx, store, scope, path, title,
-        viewerId: load.viewer.id,
-        content: load.content,
-        mediaUrl: load.mediaUrl,
-        customData: load.customData,
       })
+    return () => {
+      cancelled = true
     }
+  }, [path, sessionId, scope])
+  return load
+}
+
+/** Preview window body: loading note / error panel / viewer mount. */
+function renderPreviewBody(load, ctx, store, scope, path, title, sessionId) {
+  if (load.status === 'loading') {
+    return createElement('div', { className: 'dfa-fp-note' }, strings.loading())
+  }
+  if (load.status === 'error') {
+    return createElement(
+      'div',
+      { className: 'dfa-fp-err' },
+      strings.previewFailed(),
+      load.message
+        ? createElement('div', { style: { marginTop: '6px', fontSize: '11px', opacity: 0.85 } }, load.message)
+        : null,
+    )
+  }
+  if (load.viewer.id === 'pdf') {
+    const url = mediaUrlOf(sessionId, path)
+    return createElement(PdfPreview, { src: url, download: `${url}&download=1`, title })
+  }
+  return createElement(load.viewer.component, {
+    ctx,
+    store,
+    scope,
+    path,
+    title,
+    viewerId: load.viewer.id,
+    content: load.content,
+    mediaUrl: load.mediaUrl,
+    customData: load.customData,
+  })
+}
 
-    /**
-     * A floating preview window. Instead of re-implementing rendering, it
-     * asks the sidebar registry for the file's viewer (`matchFileViewer`),
-     * fetches the bytes the viewer's fetchStrategy needs (fsRead text /
-     * mediaUrl / customData), then mounts that viewer's own component — so
-     * code gets syntax highlighting and markdown gets rendered by the SAME
-     * built-in renderers the sidebar's editor tab uses.
-     *
-     * Media caveat: the sidebar's own media route (/sidebar/file) only serves
-     * files inside the session working directory, while file activity records
-     * files the agent touched anywhere (/tmp scratch files, sibling repos…).
-     * Media bytes therefore come from OUR route (/file-activity/file), which
-     * authorizes exactly the paths this session recorded; PDF is the one
-     * built-in viewer that fetches its own URL internally (it ignores the
-     * `mediaUrl` prop), so it gets a small iframe preview instead.
-     */
-    function FloatingPreview({ ctx, store, scope, preview, onClose }) {
-      const sessionId = scope?.sessionId ?? ''
-      const path = preview.abs
-      const title = preview.name
-      const service = ctx.betterSidebar
-      const load = usePreviewLoader(service, path, sessionId, scope)
+/**
+ * A floating preview window. Instead of re-implementing rendering, it
+ * asks the sidebar registry for the file's viewer (`matchFileViewer`),
+ * fetches the bytes the viewer's fetchStrategy needs (fsRead text /
+ * mediaUrl / customData), then mounts that viewer's own component — so
+ * code gets syntax highlighting and markdown gets rendered by the SAME
+ * built-in renderers the sidebar's editor tab uses.
+ *
+ * Media caveat: the sidebar's own media route (/sidebar/file) only serves
+ * files inside the session working directory, while file activity records
+ * files the agent touched anywhere (/tmp scratch files, sibling repos…).
+ * Media bytes therefore come from OUR route (/file-activity/file), which
+ * authorizes exactly the paths this session recorded; PDF is the one
+ * built-in viewer that fetches its own URL internally (it ignores the
+ * `mediaUrl` prop), so it gets a small iframe preview instead.
+ */
+function FloatingPreview({ ctx, store, scope, preview, onClose }) {
+  const sessionId = scope?.sessionId ?? ''
+  const path = preview.abs
+  const title = preview.name
+  const service = ctx.betterSidebar
+  const load = usePreviewLoader(service, path, sessionId, scope)
 
-      // Clicking outside is the primary dismiss (the overlay's onClick);
-      // Escape is a keyboard affordance. Both call onClose.
-      useEffect(() => {
-        if (typeof document === 'undefined' || typeof document.addEventListener !== 'function') return () => {}
-        const handler = (event) => { if (event && event.key === 'Escape') onClose() }
-        document.addEventListener('keydown', handler)
-        return () => document.removeEventListener('keydown', handler)
-      }, [onClose])
+  // Clicking outside is the primary dismiss (the overlay's onClick);
+  // Escape is a keyboard affordance. Both call onClose.
+  useEffect(() => {
+    if (typeof document === 'undefined' || typeof document.addEventListener !== 'function') return () => {}
+    const handler = (event) => {
+      if (event && event.key === 'Escape') onClose()
+    }
+    document.addEventListener('keydown', handler)
+    return () => document.removeEventListener('keydown', handler)
+  }, [onClose])
 
-      return createElement(
+  return createElement(
+    'div',
+    { className: 'dfa-fp-overlay', onClick: onClose },
+    createElement(
+      'div',
+      {
+        className: 'dfa-fp',
+        onClick: (event) => {
+          if (event && event.stopPropagation) event.stopPropagation()
+        },
+      },
+      createElement(
         'div',
-        { className: 'dfa-fp-overlay', onClick: onClose },
+        { className: 'dfa-fp-head' },
+        createElement('span', { className: 'dfa-fp-title' }, title),
         createElement(
-          'div',
-          { className: 'dfa-fp', onClick: (event) => { if (event && event.stopPropagation) event.stopPropagation() } },
+          'span',
+          { className: 'dfa-fp-actions' },
           createElement(
-            'div',
-            { className: 'dfa-fp-head' },
-            createElement('span', { className: 'dfa-fp-title' }, title),
-            createElement(
-              'span',
-              { className: 'dfa-fp-actions' },
-              createElement('button', { className: 'dfa-iconbtn', title: strings.closePreview(), 'aria-label': strings.closePreview(), onClick: () => onClose() },
-                icon.close(15),
-              ),
-            ),
+            'button',
+            {
+              className: 'dfa-iconbtn',
+              title: strings.closePreview(),
+              'aria-label': strings.closePreview(),
+              onClick: () => onClose(),
+            },
+            icon.close(15),
           ),
-          createElement('div', { className: 'dfa-fp-body' }, renderPreviewBody(load, ctx, store, scope, path, title, sessionId)),
         ),
-      )
-    }
+      ),
+      createElement(
+        'div',
+        { className: 'dfa-fp-body' },
+        renderPreviewBody(load, ctx, store, scope, path, title, sessionId),
+      ),
+    ),
+  )
+}
 
-    /**
-     * Lightweight PDF preview. better-sidebar's built-in PdfView fetches
-     * `/sidebar/file` internally (it ignores any injected `mediaUrl` prop),
-     * and that route refuses files outside the session working directory — so
-     * a recorded /tmp PDF would never load. This tiny view embeds the bytes
-     * from OUR media route in a native browser PDF frame, with a download
-     * fallback in its toolbar.
-     */
-    function PdfPreview({ src, download, title }) {
-      return createElement('div', { className: 'dfa-pdf' },
-        createElement('div', { className: 'dfa-pdf-toolbar' },
-          createElement('a', { className: 'dfa-pdf-download', href: download, download: true, title: strings.downloadToView() },
-            strings.downloadToView()),
-        ),
-        createElement('iframe', { className: 'dfa-pdf-frame', src, title }),
-      )
-    }
+/**
+ * Lightweight PDF preview. better-sidebar's built-in PdfView fetches
+ * `/sidebar/file` internally (it ignores any injected `mediaUrl` prop),
+ * and that route refuses files outside the session working directory — so
+ * a recorded /tmp PDF would never load. This tiny view embeds the bytes
+ * from OUR media route in a native browser PDF frame, with a download
+ * fallback in its toolbar.
+ */
+function PdfPreview({ src, download, title }) {
+  return createElement(
+    'div',
+    { className: 'dfa-pdf' },
+    createElement(
+      'div',
+      { className: 'dfa-pdf-toolbar' },
+      createElement(
+        'a',
+        {
+          className: 'dfa-pdf-download',
+          href: download,
+          download: true,
+          title: strings.downloadToView(),
+        },
+        strings.downloadToView(),
+      ),
+    ),
+    createElement('iframe', { className: 'dfa-pdf-frame', src, title }),
+  )
+}