Add configurable sidebar thread preview count#1856
Add configurable sidebar thread preview count#1856Marve10s wants to merge 5 commits intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Why ? When working on more than 3+ projects with default thread number, I need to scroll. Instead, I'd prefer to have less threads because I don't need 6 by default while working on multiple stuff. |
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 afe3877. Configure here.
ApprovabilityVerdict: Needs human review The sidebar thread preview count feature itself is straightforward, but there is an unresolved high-severity bug comment identifying that the vite.config.ts changes may cause startup crashes when VITE_WS_URL is not configured. This unrelated change bundled into the PR requires human attention. You can customize Macroscope's approvability policy. Learn more. |
Drop the blank-line removal that was left over from the dev bootstrap fix commits — not in scope for this PR.
fyi Added by accident, PR is no longer contains Vite config changes |

What changed
2-10.-/ numeric input /+control.Notes
apps/web/vite.config.tsthat was needed to verify the app locally.Validation
bun fmtbun lintbun typecheckNote
Low Risk
Low risk UI/settings change that replaces a hardcoded sidebar thread preview limit with a validated client setting; main risk is minor UX regressions in the new input control and settings migration edge cases.
Overview
Replaces the sidebar’s hardcoded thread preview limit with a new client setting,
sidebarThreadPreviewCount(default6, clamped to2–10), and uses it wherever the sidebar computes preview/overflowed thread lists.Updates the sidebar options menu (
ProjectSortMenu) to include a compact-/ numeric input /+control for changing this value, persisting viaupdateSettings, and adds schema + legacy migration support (with tests) plus inclusion in the settings “restore defaults” dirty-check list.Reviewed by Cursor Bugbot for commit f120de9. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add configurable sidebar thread preview count per project
sidebarThreadPreviewCountsetting (range 2–10, default 6) stored in client settings.ProjectSortMenu) with input validation and clamping via a newclampSidebarThreadPreviewCounthelper.updateSettingsand migrates valid legacysidebarThreadPreviewCountvalues during settings migration; invalid legacy values are ignored.sidebarThreadPreviewCountin dirty-state detection and the reset confirmation list.Macroscope summarized d81caf8.