feat(ui): show folder path in search modal#4138
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Workflow search metadata is extended with Reviewed by Cursor Bugbot for commit a5f4a17. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR adds folder path display to the search modal so workflows with the same name can be distinguished by their location. The implementation correctly uses the existing Confidence Score: 5/5Safe to merge — all findings are P2 style suggestions with no functional impact on correctness The logic is correct: apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/search-modal/components/command-items.tsx — layout consideration for long folder paths Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[sidebar.tsx\nregularWorkflows] -->|useMemo| B{workflow.folderId?}
B -->|yes| C[getFolderPath\nfolderMap, folderId]
C --> D[map folder.name\njoin ' / ']
D --> E[folderPath string]
B -->|no| F[folderPath = undefined]
E --> G[WorkflowItem\nfolderPath = string]
F --> G
G --> H[search-groups.tsx\nWorkflowsGroup]
H -->|value includes folderPath| I[cmdk Command.Item\nenables folder search]
H -->|folderPath prop| J[command-items.tsx\nMemoizedWorkflowItem]
J -->|folderPath truthy| K[render right-aligned\nsubtitle span]
J -->|folderPath undefined| L[no subtitle rendered]
|
...pace/[workspaceId]/w/components/sidebar/components/search-modal/components/search-groups.tsx
Show resolved
Hide resolved
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.
Reviewed by Cursor Bugbot for commit 7f905cb. Configure here.

Summary
Currently the search modal doesn't show folders, meaning that workflows with the same name have no distinguishing features. Added folder path to the right side of the modal.
Type of Change
Testing
Checklist
Screenshots/Videos