PR Summary

Medium Risk
Primarily a performance refactor (memoization, debounced search, stable refs) but it touches many callbacks/state transitions around save/delete/navigation, so regressions would show up as stale actions or incorrect preview mode changes.

Overview
Reduces unnecessary rerenders on the workspace Files page by memoizing preview/context-menu components and stabilizing many handlers with useRef-backed values (e.g., save/delete/download/rename/context-menu actions).

Adds a 200ms debounced search input and introduces lightweight row caching so ResourceRow objects aren’t recreated on every render, while preserving inline-rename behavior.

Refactors preview-mode initialization and route-change handling to avoid effect churn, and extracts the file row context menu into a memoized component; FileViewer save now uses a ref to avoid re-creating the autosave onSave callback.

^{Written by Cursor Bugbot for commit 3750f6e. Configure here.}

@adithyaakrishna is attempting to deploy a commit to the Sim Team on Vercel.

A member of the Team first needs to authorize it.

Greptile Summary

This PR applies a broad set of memoization and ref-stabilization optimizations to the files page to reduce unnecessary re-renders. The changes include debounced search with immediate UI feedback, row caching via rowCacheRef, stable callbacks via the ref pattern, memo wrapping for all leaf preview components, extraction of a new FileRowContextMenu component, merged keydown/beforeunload effects, and a lazy previewMode initializer. Two issues found: headerRename callbacks missing from fileDetailBreadcrumbs deps (latent maintenance risk), and a render-time prevFileIdRef.current mutation that can leave previewMode stuck in React 18 concurrent mode.

Confidence Score: 4/5

Safe to merge after addressing the concurrent-mode ref-mutation issue in previewMode derivation; the missing headerRename deps are a low-risk maintenance concern.

The PR delivers a large, well-structured rerender optimization. The two remaining findings are both in files.tsx: one is a latent (not currently broken) dep-array inconsistency, and the other is a genuine correctness risk under React 18 concurrent rendering where previewMode can get stuck after a route change.

apps/sim/app/workspace/[workspaceId]/files/files.tsx — specifically the render-time prevFileIdRef mutation (lines 567–590) and the fileDetailBreadcrumbs missing deps (lines 425–465).

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User types in search box] --> B[setInputValue - immediate]
    B --> C[Debounce timer 200ms]
    C --> D[setDebouncedSearchTerm]
    D --> E[filteredFiles useMemo recalculates]
    E --> F{Row in rowCacheRef?}
    F -- Yes, same file+members --> G[Return cached ResourceRow]
    F -- No --> H[Build new ResourceRow]
    G & H --> I[baseRows array]
    I --> J{listRename.editingId set?}
    J -- No --> K[rows = baseRows]
    J -- Yes --> L[Inject InlineRenameInput into matching row]
    L --> K
    K --> M[Resource table renders]

    N[fileIdFromRoute changes] --> O{prevFileIdRef check during render}
    O -- Different --> P[Mutate prevFileIdRef.current]
    P --> Q[Compute nextMode via filesRef.current]
    Q --> R[setPreviewMode if different]
    O -- Same --> S[No-op]

    style P fill:#f9c,stroke:#c00
    style R fill:#ffc,stroke:#990

_{Reviews (2): Last reviewed commit: "chore: fix review changes" | Re-trigger Greptile}

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 27, 2026 0:59am

@greptile

@cursor review