fix(desktop): wire v2 sidebar project settings to settings route

[Kitenite]

Summary

• The v2 dashboard sidebar's project context-menu "Project Settings" item was showing a "coming soon" toast. Navigate to /settings/project/$projectId instead, matching v1's ProjectHeader.handleOpenSettings logic.

Test plan

• Right-click a project in the v2 dashboard sidebar → click Project Settings → verify it navigates to the project settings page (general tab)

---

Summary by cubic

Clicking Project Settings in the v2 dashboard sidebar now opens the project settings page for the selected project. We navigate to /settings/project/$projectId using @tanstack/react-router, replacing the “coming soon” toast and matching v1 behavior.

^{Written for commit 1a0df5b. Summary will update on new commits.}

Summary by CodeRabbit

• Bug Fixes
  • Fixed project settings access in the dashboard sidebar - users can now navigate to project settings directly.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a081b3d7-005d-45c8-acd0-8ca970003d3f

📥 Commits

Reviewing files that changed from the base of the PR and between b2278b1 and 1a0df5b.

📒 Files selected for processing (1)

• apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/DashboardSidebar/components/DashboardSidebarProjectSection/hooks/useDashboardSidebarProjectSectionActions/useDashboardSidebarProjectSectionActions.ts

---

📝 Walkthrough

Walkthrough

A single hook was modified to replace a "coming soon" toast notification with actual navigation. The handleOpenSettings function now directs users to the project settings page by navigating to /settings/project/$projectId instead of displaying a placeholder message.

Changes

Cohort / File(s)	Summary
Navigation Implementation apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/DashboardSidebar/components/DashboardSidebarProjectSection/hooks/useDashboardSidebarProjectSectionActions/useDashboardSidebarProjectSectionActions.ts	Added useNavigate import and replaced "coming soon" toast with navigation to project settings route, passing project.id as route parameter.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A settings button, once just a dream,
Now hops straight where it should be!
No more "soon" in a toast so bright,
Navigation works—what a delight! 🌟

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description includes a clear summary and test plan, but is missing several required template sections: Related Issues, Type of Change, Testing (formal section), Screenshots, and Additional Notes.	Complete the template by adding Related Issues, selecting appropriate Type of Change checkbox(es), expanding the Testing section, and including any Additional Notes as needed.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix(desktop): wire v2 sidebar project settings to settings route' is specific and directly describes the main change—wiring the project settings menu item to the settings route.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch curse-calculator

---

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.}

[greptile-apps]

Greptile Summary

This PR wires up the v2 dashboard sidebar's "Project Settings" context-menu action, replacing a placeholder toast.info("Project Settings coming soon") with a proper navigate call to /settings/project/$projectId. The implementation matches v1's ProjectHeader.handleOpenSettings exactly. The target route has an index-redirect component that automatically redirects to the /general sub-route.

Key changes:

• handleOpenSettings now calls useNavigate() to navigate to the project settings page instead of showing a "coming soon" toast.
• The navigation target (/settings/project/$projectId) is consistent with the v1 sidebar implementation and is handled by an existing <Navigate> redirect to /general.

Confidence Score: 5/5

Safe to merge — targeted one-function fix with no side effects.

The change is minimal and correct: it replaces a stub toast with navigation logic identical to v1, the target route exists and properly redirects to the general tab, and no imports or state are left dangling.

No files require special attention.

Important Files Changed

Filename	Overview
apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/DashboardSidebar/components/DashboardSidebarProjectSection/hooks/useDashboardSidebarProjectSectionActions/useDashboardSidebarProjectSectionActions.ts	Replaces the 'coming soon' toast in handleOpenSettings with a navigate call to /settings/project/$projectId, matching v1 ProjectHeader behavior exactly. The index route at that path correctly redirects to /general.

Sequence Diagram

sequenceDiagram
    actor User
    participant Sidebar as DashboardSidebar (v2)
    participant Hook as useDashboardSidebarProjectSectionActions
    participant Router as TanStack Router
    participant IndexRoute as /settings/project/$projectId/
    participant GeneralPage as /settings/project/$projectId/general

    User->>Sidebar: Right-click project → "Project Settings"
    Sidebar->>Hook: handleOpenSettings()
    Hook->>Router: navigate({ to: "/settings/project/$projectId", params: { projectId } })
    Router->>IndexRoute: Load index route
    IndexRoute->>GeneralPage: Navigate replace to /general
    GeneralPage-->>User: Renders ProjectSettings (general tab)

Loading

_{Reviews (1): Last reviewed commit: "fix(desktop): wire v2 sidebar project se..." | Re-trigger Greptile}

[cubic-dev-ai]

No issues found across 1 file

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ⚠️ Neon database branch
• ⚠️ Electric Fly.io app

Thank you for your contribution! 🎉