v0.6.14: performance improvements, connectors UX, collapsed sidebar actions#3809
v0.6.14: performance improvements, connectors UX, collapsed sidebar actions#3809waleedlatif1 merged 8 commits intomainfrom
Conversation
* feat: update sidebar and knowledge * chore: fix rernders on knowledge * chore: fix review changes * chore: fix review changes
* chore: fix antialiasing * chore: remove antialiasing
* chore: fix rerenders on files * chore: fix review changes
* fix(knowledge): show spinner on connector chip while syncing * fix(knowledge): scope sync spinner to mutation lifetime, not cooldown
…stency, and UI polish (#3807) * improvement(sidebar): collapsed sidebar UX, quick-create, hover consistency, and UI polish Made-with: Cursor * fix(sidebar): use stable handlers for root workflow items instead of inline lambdas Made-with: Cursor * fix(sidebar): reset actionsOpen state before triggering rename in collapsed dropdown Made-with: Cursor
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview 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 Polishes destructive confirmation copy across multiple modals to emphasize permanence (error-colored warning text), introduces Written by Cursor Bugbot for commit 34df333. Configure here. |
...orkspace/[workspaceId]/w/components/sidebar/components/workspace-header/workspace-header.tsx
Outdated
Show resolved
Hide resolved
Greptile SummaryThis 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 ( Confidence Score: 5/5Safe to merge; all remaining findings are P2 style/accessibility suggestions with no runtime or data-integrity risk. The logic changes (connector sync spinner, EMCN input components ( 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 |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
.../workspace/[workspaceId]/knowledge/[id]/components/connectors-section/connectors-section.tsx
Outdated
Show resolved
Hide resolved
…3812) * fix(knowledge): scope sync/update state per-connector to prevent race conditions * feat(knowledge): add connectors column to knowledge base list * refactor(knowledge): extract set helpers, handleTogglePause, and filter-before-map * refactor(knowledge): use onSettled for syncingIds cleanup, consistent with updatingIds

Uh oh!
There was an error while loading. Please reload this page.