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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 29, 2026 6:32am

PR Summary

Medium Risk
Moderate risk: broad UI refactors add memoization, refs, and debounced state across files/knowledge/sidebar flows, which could introduce stale-state or event-handling regressions despite being mostly behavior-preserving.

Overview
Improves perceived performance across resource screens by aggressively memoizing components (Resource, table rows/cells, headers/options bars, file previews/modals/menus) and stabilizing callbacks/props; the files and knowledge list pages also switch to debounced search and ref-based state access to reduce dependency churn and rerenders.

Enhances UX for knowledge connectors by showing a per-connector syncing spinner while a sync is pending, and updates the collapsed/expanded sidebar interactions with consistent --surface-hover vs --surface-active styling plus new collapsed-workflow actions (open-in-new-tab/rename via submenus) and more reliable “more” button visibility.

Polishes destructive confirmation copy across multiple modals to emphasize permanence (error-colored warning text), introduces --surface-hover theme tokens, and removes global body antialiasing.

^{Written by Cursor Bugbot for commit 34df333. Configure here.}

Greptile Summary

This release bundles five commits that collectively improve rendering performance across the resource/table layer, refine sidebar UX in both expanded and collapsed states, and surface a sync-pending spinner on knowledge-base connectors. The changes are well-structured and follow the codebase's architecture conventions.\n\nKey changes:\n- Performance (resource.tsx, files.tsx, knowledge.tsx) — Resource, DataRow, CellContent, CreateRow, ResourceColGroup, DataTableSkeleton, Pagination, and SearchSection are all wrapped in memo(); stable JSX constants replace inline element creation; manual debounce timer refs replace the useDebounce hook to avoid unnecessary re-renders in search inputs.\n- Sidebar UX — Hover and active item states are now semantically separated via a new --surface-hover / --surface-active CSS token split; collapsed-sidebar workflow actions migrated from an external context-menu to an inline DropdownMenuSub; active-state highlight persists on items while their context menu is open.\n- Collapsed sidebar quick-create / upgrade — Free-plan users now see an "Upgrade" button in the workspace header (gated behind isBillingEnabled); the subscription fetch is similarly guarded.\n- Knowledge connectors UX — syncingConnectorId state drives an immediate spinner on the connector chip as soon as sync is triggered, before the server reflects the syncing status.\n- Font antialiasing removed — @apply antialiased removed from body in globals.css.\n- EMCN input focus styles — focus-visible:border-[var(--text-muted)] removed from Input, Textarea, Combobox, DatePicker, and TimePicker base variants; worth verifying whether an alternative focus indicator is provided.

Confidence Score: 5/5

Safe to merge; all remaining findings are P2 style/accessibility suggestions with no runtime or data-integrity risk.

The logic changes (connector sync spinner, isCollapsedRef pattern, SearchSection render-time sync, collapsed workflow DropdownMenuSub) are all correct. The two flagged items are P2: the globals.css edit (justified for CSS-variable addition, debatable for the antialiasing line) and the removed focus-visible border on EMCN inputs (accessibility polish, not a breakage). No data-loss, crash, or security risk identified.

EMCN input components (input.tsx, textarea.tsx, combobox.tsx, date-picker.tsx, time-picker.tsx) — verify that an alternative keyboard focus indicator exists before these land in production.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User types in SearchSection] --> B[setLocalValue - immediate]
    B --> C[search.onChange called]
    C --> D[Parent debounces / filters data]

    E[Parent clears search externally] --> F{search.value !== lastReportedRef?}
    F -- yes --> G[render-time setLocalValue sync]
    G --> H[Re-render with synced value]

    I[User triggers connector Sync] --> J[setSyncingConnectorId]
    J --> K[Spinner shown immediately on chip]
    K --> L{triggerSync}
    L -- onSuccess --> M[setSyncingConnectorId null]
    L -- onError --> M

    N[User opens context menu on workflow item] --> O[setMenuOpenTaskId / actionsOpen]
    O --> P[Item stays bg-surface-active while menu is open]
    P --> Q{Menu closes}
    Q --> R[menuOpenTaskId cleared / actionsOpen false]

_{Reviews (1): Last reviewed commit: "improvement(sidebar): collapsed sidebar ..." | Re-trigger Greptile}

@cursor review