fix(web): persist sidebar shelf collapse state

[PixPMusic]

Summary

• persist the Sidebar V2 Settled and Snoozed expansion states across desktop restarts and web reloads
• keep the first-run defaults: Settled expanded and Snoozed collapsed
• use the existing schema-validated useLocalStorage hook with an independent key for each shelf

Why

Collapsing a shelf reads as a lasting preference. Today both shelves reset when the desktop app restarts, so users have to restore their preferred sidebar layout every launch.

This includes the Settled persistence change from #4573 by @ipanasenko with the original commit authorship preserved, then adds the missing Snoozed persistence.

Before / After

Both captures use the same isolated synthetic showcase dataset: three settled threads plus two showcase threads explicitly seeded with future snooze times. Before each restart, Snoozed was expanded and Settled was collapsed.

Before: current main after restart	After: this branch after restart

Verification

• pnpm exec vp test run apps/web/src/hooks/useLocalStorage.test.ts (4 tests)
• pnpm exec vp lint --report-unused-disable-directives apps/web/src/components/SidebarV2.tsx
• pnpm --filter @t3tools/web typecheck
• reproduced the reset on current main with a full T3 Code Desktop quit and relaunch against the synthetic showcase fixture
• verified the branch preserves expanded Snoozed and collapsed Settled states through the same full desktop restart

Checklist

• rebased on current main
• tested in the desktop client with isolated synthetic data
• matched before/after restart evidence embedded without committing images
• original fix(web): persist the Settled shelf's collapsed state #4573 authorship preserved

Model: GPT-5.6 Sol | Harness: Codex in T3 Code

---

---

Note

Low Risk
UI-only sidebar preference persistence with safe fallbacks; no auth, data, or server behavior changes.

Overview
Settled and Snoozed shelf expand/collapse in Sidebar V2 now survives reloads and desktop restarts instead of resetting every launch.

A new usePersistentShelfExpanded hook reads and writes each shelf’s state to localStorage under t3code:sidebar-v2:settled-expanded and t3code:sidebar-v2:snoozed-expanded, with Schema.Boolean validation via getLocalStorageItem / setLocalStorageItem. First-run defaults stay the same: Settled expanded, Snoozed collapsed. Failed reads or writes log to the console and fall back to in-memory state so toggles still work when storage is blocked or corrupt.

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

Note

Persist sidebar shelf collapse state to localStorage in SidebarV2

Replaces useState with useLocalStorage for the Settled and Snoozed shelf expanded flags in SidebarV2.tsx, so collapse state survives page reloads. Boolean values are validated using effect/Schema.

^{Macroscope summarized f575c74.}

[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 Plus

Run ID: 9c3b1f3a-d144-4dd7-b161-8edbc9bf4315

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

---

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

[macroscopeapp]

Approvability

Verdict: Approved caf78ae

Simple client-side localStorage persistence for sidebar shelf collapse state. The changes are self-contained UI preference storage with no backend, schema, or security implications. Review comments point to pre-existing issues or style preferences rather than bugs introduced here.

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

[ipanasenko]

Dismissing prior approval to re-evaluate c92420d

[cursor]

Cursor Bugbot has reviewed your changes using high effort 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 caf78ae. Configure here.}

[PixPMusic]

Non-blocking follow-up: I briefly explored a Sidebar-local persistence helper because the current useLocalStorage failure path can leave a control unresponsive within the session. If reading, decoding, or writing local storage fails, the hook logs the error but cannot publish the attempted value. A malformed stored value is especially awkward: the render falls back to the default, then a functional update reads and decodes the same malformed value again and aborts before it can replace it.

That helper was removed because shelf state should follow the repository rule and use the shared abstraction. This PR now uses useLocalStorage directly and does not change the hook. Separately, the hook may be worth hardening so consumers retain usable in-memory state when persistence is unavailable and can recover from malformed values, while preserving its existing cross-tab behavior.

[PixPMusic]

@ipanasenko thanks for the initial PR! My PR workflow is to scan for existing PRs that match the functionality before I bother working on one--yours was most of the way there. In theory, I could target your branch instead and you could merge my changes into yours.

I see you've closed yours, so I'll leave this one open since it covers both shelves.

Since I'm not a core maintainer, I can't push to your PR branch directly. :)