The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 23, 2026 6:15pm

Greptile Summary

This PR fixes a TypeScript build error in PptxPreview where dataUpdatedAt was already referenced in the useEffect dependency array (line 631) but was never destructured from the useWorkspaceFileBinary hook. The one-line fix properly destructures the field, resolving the compiler error.

Key changes:

• Destructures dataUpdatedAt from useWorkspaceFileBinary in PptxPreview so the useEffect dependency array reference is valid.

Behavioral note: Beyond fixing the build, this change also ensures correctness at runtime — dataUpdatedAt (a React Query timestamp that updates on every refetch) guarantees the PPTX render effect re-triggers whenever the file binary is refreshed, even if fileData happens to share the same reference or byteLength as the prior fetch.

Confidence Score: 5/5

• This PR is safe to merge — it is a minimal, targeted fix with no functional regressions.
• Single-line change that correctly destructures a standard React Query field (dataUpdatedAt) already referenced in the dependency array. The hook (useWorkspaceFileBinary wrapping useQuery) always returns this field, so the destructuring is always valid. No logic paths are altered beyond ensuring the effect dependency is properly declared.
• No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant RQ as React Query
    participant Hook as useWorkspaceFileBinary
    participant Comp as PptxPreview
    participant Effect as useEffect (render slides)

    RQ->>Hook: refetch (on window focus / cache invalidation)
    Hook-->>Comp: { fileData, dataUpdatedAt, ... }
    Note over Comp: dataUpdatedAt changes on every refetch
    Comp->>Effect: dependency [fileData, dataUpdatedAt, ...] changed
    Effect->>Effect: re-render PPTX slides

_{Reviews (1): Last reviewed commit: "Fix build" | Re-trigger Greptile}