Add configurable sidebar thread preview count#1856
Add configurable sidebar thread preview count#1856Marve10s wants to merge 14 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. |
ApprovabilityVerdict: Approved Straightforward UI feature adding a user-configurable sidebar thread preview count setting. Replaces a hardcoded constant (6) with a bounded configurable value (2-10, default 6). Changes are self-contained to the sidebar UI and settings schema with no backend or runtime behavior implications beyond the visual preference. 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 |
Resolve merge conflicts: - settings.ts: keep sidebarThreadPreviewCount with Effect.succeed() pattern - useSettings.ts: drop removed migration functions, keep upstream's reset helper - useSettings.test.ts: delete (migration tests no longer needed)
The clientPersistence test was missing the new field after merge.
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 bb60b2c. Configure here.
Dismissing prior approval to re-evaluate 02fce1a

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
Primarily UI and client-side settings schema changes; risk is limited to sidebar rendering/interaction and settings persistence defaults.
Overview
Adds a new client setting,
sidebarThreadPreviewCount(default6, clamped2–10), to the shared settings contracts and updates persistence tests to include it.Updates the web sidebar to replace the hardcoded thread preview limit with the user setting, and adds a "Visible threads" +/- + numeric input control to the sidebar options menu to update the setting. Settings restore-defaults confirmation now also lists "Visible threads" when this value differs from default.
Reviewed by Cursor Bugbot for commit 02fce1a. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add configurable sidebar thread preview count with controls in sidebar options menu
sidebarThreadPreviewCountfield toClientSettingswith a default of 6 and a valid range of [2, 10], defined in settings.ts.THREAD_PREVIEW_LIMITin the sidebar with the user-configured value for both collapsed project previews and per-project thread slicing.ProjectSortMenu) to adjust the count, which persists to client settings.sidebarThreadPreviewCountto 6.Macroscope summarized 02fce1a.