Add filesystem browse API and command palette project picker

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[Noojuno]

Summary

• Add filesystem.browse RPC endpoint with directory listing, cross-platform path resolution (Windows drive/UNC, Unix, ~/ expansion), and relative path support against the active project
• Wire filesystem browse through contracts (FilesystemBrowseInput/Result/Error), shared path helpers (@t3tools/shared/path), the RPC group, server handler, WsRpcClient, and EnvironmentApi
• Add "Add project" action to the command palette with an interactive filesystem browser: type a path to browse directories, navigate with keyboard/click, press Enter or click "Add" to create the project
• Add projectPaths utility library for browse navigation (path segment append/parent, trailing separator detection, project deduplication, relative path resolution, title inference)
• Fix pre-existing Effect v3→v4 type errors in the server filesystem handler (Effect.catchAll → Effect.mapError, explicit type annotations)

Testing

• bun run typecheck — all 8 packages pass (0 errors)
• bun run test — contracts (65 tests), shared (75 tests including new path helpers), web (projectPaths 11 tests, wsRpcClient 1 test) all pass
• Manual: command palette "Add project" action opens browse mode, directory listing populates, keyboard navigation and Enter-to-add work

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Add filesystem browse API and command palette project picker

• Adds a filesystem.browse WebSocket RPC that returns subdirectories under a given path, supporting prefix filtering, ~ expansion, relative paths with a cwd, and cross-platform path validation (WorkspaceEntries.ts, ws.ts).
• Replaces the inline sidebar 'Add project' UI with a command palette flow: clicking the sidebar trigger opens the command palette in browse mode, where users can navigate directories with keyboard or click and submit a path to create a project via project.create.
• Adds a comprehensive path utility library (projectPaths.ts) covering Windows drive/UNC/Unix path normalization, relative path resolution, browse query detection, and parent path traversal.
• Extends ProjectCreateCommand with an optional createWorkspaceRootIfMissing flag; when set, normalizeWorkspaceRoot creates the directory before project creation.
• Adds an 'Add project starts in' field to the General settings panel, letting users configure the default browse directory.
• Risk: The normalizeWorkspaceRoot call in dispatch normalization now creates directories on disk when createWorkspaceRootIfMissing is true, which is a new side effect in the orchestration normalization path.

^{Macroscope summarized 4d4786d.}

---

Note

Medium Risk
Adds a new filesystem.browse RPC path and changes project.create/normalizeWorkspaceRoot to optionally create missing workspace directories, which can affect filesystem behavior and error handling across environments.

Overview
Adds a new filesystem browsing capability via filesystem.browse in contracts/RPC/client/server, returning matching subdirectories for a partial path with ~ expansion, platform-aware Windows path guards, and optional relative-path resolution against a provided cwd.

Updates project creation path handling by extending project.create with createWorkspaceRootIfMissing and teaching WorkspacePaths.normalizeWorkspaceRoot (and orchestration normalization) to create missing workspace root directories when requested, including new error types and coverage in server tests.

Reworks “Add project” UX in the web app to run through the command palette: adds a browse mode with directory navigation, environment selection, prefetching, and an optional native folder picker (desktop bridge now accepts pickFolder({ initialPath })), plus a new addProjectBaseDirectory setting and broad UI/test updates (including removing the sidebar inline add-project form).

^{Reviewed by Cursor Bugbot for commit 4d4786d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a246714d-6e2e-4c60-81bc-4cb6bdd3c542

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cursor]

Tilde paths bypass project deduplication check

Medium Severity

When browseResult hasn't loaded yet (or there's no exactBrowseEntry), resolvedAddProjectPath falls back to the raw query.trim() — a tilde-prefixed string like "~/projects/myapp". This unresolved path is passed to handleAddProject, where findProjectByPath compares it (via normalizeProjectPathForComparison) against existing projects whose cwd values are server-expanded absolute paths like "/home/user/projects/myapp". Since the tilde is never expanded client-side, the comparison always fails, and a duplicate project is created for the same directory.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L497-L501

^{Reviewed by Cursor Bugbot for commit 9448e45. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR introduces a significant new feature (filesystem browse API and command palette project picker) with new RPC endpoints, UI flows, and server-side workspace creation capabilities. Additionally, there are multiple unresolved review comments identifying medium-severity bugs related to path handling and deduplication logic that could cause incorrect behavior.

^{You can customize Macroscope's approvability policy. Learn more.}

[cursor]

Client-side filter hides dot directories in directory listing mode

Medium Severity

filterBrowseEntries sets showHidden based solely on whether browseFilterQuery starts with ".". When the user navigates into a directory (trailing separator), browseFilterQuery is "", so showHidden is false and all dot-prefixed entries are filtered out. The server intentionally includes hidden directories in directory-listing mode (tested in WorkspaceEntries.test.ts — "shows dot directories in directory mode"), but the client discards them. Users can never see directories like .config or .vscode when browsing into a folder.

^{Reviewed by Cursor Bugbot for commit 0ddcb1a. Configure here.}

[cursor]

Case-sensitive exact match mismatches case-insensitive filter

Medium Severity

filterBrowseEntries uses case-insensitive prefix matching for filteredEntries but case-sensitive equality for exactEntry. When a user types a directory name with different casing (e.g., "development" vs "Development"), the entry appears in results but exactEntry is null. This causes resolvedAddProjectPath to fall back to the raw query.trim() (preserving the unexpanded tilde and wrong casing) instead of using the server-resolved fullPath from the browse entry. The project may be created with an incorrect, unexpanded path like ~/development instead of /Users/user/Development.

Additional Locations (1)

• apps/web/src/components/CommandPalette.tsx#L584-L587

^{Reviewed by Cursor Bugbot for commit 5dc6d92. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 4 total unresolved issues (including 3 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}

[cursor]

Unmemoized array defeats downstream memoization chain

Low Severity

addProjectEnvironmentItems is created as a plain array every render (via .map()) without useMemo, then used as the sole dependency of the addProjectEnvironmentGroupsuseMemo. Since the array reference changes every render, the memo always recomputes, cascading instability to openAddProjectFlow (a useCallback depending on addProjectEnvironmentGroups) and the openIntentuseEffect. Wrapping addProjectEnvironmentItems in useMemo with [addProjectEnvironmentOptions, startAddProjectBrowse] would stabilize the entire chain.

^{Reviewed by Cursor Bugbot for commit 4d4786d. Configure here.}