fix(mothership): fix build error

[Sg312]

Summary

Fix build

Type of Change

• Bug fix

Testing

Manual

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

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-apps]

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

Filename	Overview
apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/file-viewer.tsx	Adds missing dataUpdatedAt destructuring from useWorkspaceFileBinary, fixing a TypeScript build error where the variable was referenced in the useEffect dependency array but never declared.

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

Loading

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