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 2:18am

PR Summary

Medium Risk
Moderate risk: adds new React Query fetches in Sidebar and new result groups in the cmd-k modal, which could impact performance or permissions/visibility if misconfigured, but stays within client-side navigation/search UI.

Overview
Expands the cmd-k search modal to include individual tables, files, and knowledgeBases as selectable results that navigate directly to the entity page.

Sidebar now fetches these entity lists via existing query hooks and passes them into SearchModal, respecting hideTablesTab, hideFilesTab, and hideKnowledgeBaseTab. The search UI adds a reusable MemoizedIconItem plus an icon-group factory (createIconGroup) to render the new result sections.

^{Written by Cursor Bugbot for commit 2100c85. Configure here.}

Greptile Summary

This PR extends the cmd-k search modal to surface individual tables, files, and knowledge bases as navigable search results, complementing the existing static "Pages" links. It introduces a MemoizedIconItem component and a createIconGroup factory to avoid repetition, fetches entity data via three existing React Query hooks in Sidebar, respects per-entity permission flags, and maps each entity to a valid deep-link href.\n\nKey changes:\n- command-items.tsx: New MemoizedIconItem memoized component with a four-field custom comparator (including onSelect, fixed from prior review).\n- search-groups.tsx: createIconGroup factory produces three exported group components (TablesGroup, FilesGroup, KnowledgeBasesGroup) with minimal duplication.\n- search-modal.tsx: Three new filteredX memos and the three new group components wired in; reuses the existing handleTaskSelect callback since the navigation behaviour is identical to tasks.\n- sidebar.tsx: Calls useTablesList, useWorkspaceFiles, and useKnowledgeBasesQuery; maps results to TaskItem with entity-specific hrefs (individual file routes at [fileId]/page.tsx confirmed to exist).\n- Jobs and files with ambiguous routing (original concern from prior review) have been addressed: jobs are absent and files now carry per-file deep-link hrefs.

Confidence Score: 5/5

Safe to merge — prior routing concerns are resolved, logic is correct, and the single remaining note is a non-blocking style preference.

All prior review concerns have been addressed: the onSelect comparator was added to MemoizedIconItem, jobs were removed (no per-job routes exist), and files now carry valid individual deep-link hrefs that map to an existing [fileId]/page.tsx route. The only open item is a minor style issue (factory defined after its callers due to hoisting), which does not affect correctness or reliability.

No files require special attention.

Important Files Changed

_{Reviews (2): Last reviewed commit: "fix(search): add files back with per-fil..." | Re-trigger Greptile}

@greptile

@cursor review