fix(web): surface draft sessions in the legacy sidebar by Lasdw6 · Pull Request #7120 · pingdotgg/t3code

Lasdw6 · 2026-08-15T18:58:42Z

What Changed

The legacy sidebar now renders draft sessions (unsent new-thread composers with typed content), reusing the v2 sidebar's existing SidebarDraftBlock component.

SidebarDraftBlock is exported from Sidebar.tsx (no behavior change there).
LegacySidebar.tsx gains a small self-contained wrapper that supplies project title/cwd/favicon lookups, the active draft route, and /draft/$draftId navigation, mounted above the "Projects" header. It subscribes to the draft store itself, so per-keystroke composer updates re-render only this block and the projects content's memoization is untouched.
With zero drafts the block renders nothing — the empty legacy sidebar is pixel-identical to before.

+72/−2 across the two sidebar files.

Why

Fixes #6084.

Drafts created while using the legacy sidebar were kept by the store but had no UI there: navigate away and there was no row to click back into, while the drafts silently piled up in the v2 sidebar. Reusing the v2 block (rather than reimplementing rows) keeps the two sidebars showing identical draft state with one source of truth.

UI Changes

Sidebar v2 (Unchanged)

Legacy Sidebar Before

Legacy Sidebar After

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

Surface draft sessions in the legacy sidebar above the projects list

Adds a LegacySidebarDraftList component to LegacySidebar.tsx that renders a drafts section at the top of the legacy sidebar. Clicking a draft clears the current thread selection, navigates to /draft/$draftId, and closes the sidebar on mobile. SidebarDraftBlock in Sidebar.tsx is exported so the new component can reuse it.

^{Macroscope summarized 799ba95.}

Note

Low Risk
UI-only parity fix that reuses the existing v2 draft block and navigation; no auth, data, or API changes.

Overview
Unsent composer drafts now appear in the legacy sidebar above the Projects list, matching the v2 sidebar so drafts started there stay one click away instead of only showing up in v2.

The change exports SidebarDraftBlock from Sidebar.tsx and adds LegacySidebarDraftList in LegacySidebar.tsx, which builds project title/cwd/favicon maps, reads the active draft from the route, and navigates to /draft/$draftId (clearing thread selection and closing the mobile sidebar). Draft store subscriptions live in that wrapper so typing in a draft does not bust memoization on the main projects list. With no drafts, the block still renders nothing and the layout stays unchanged.

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

Drafts created under the legacy sidebar were invisible there: the store kept them, but only the v2 sidebar rendered rows, so navigating away lost the way back in and drafts silently piled up in the v2 list. Reuse the v2 SidebarDraftBlock (now exported) above the legacy projects list via a self-contained wrapper that supplies project lookups, the route draft id, and navigation, keeping SidebarProjectsContent's memo intact. The block renders nothing at count 0, so the empty sidebar is unchanged.

coderabbitai · 2026-08-15T18:58:51Z

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: 0de0b4b2-b200-4e7c-90b6-32bb32a3d8ad

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

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

macroscopeapp · 2026-08-15T19:12:57Z

Approvability

Verdict: Needs human review

This PR adds new user-facing functionality (draft sessions appearing in the legacy sidebar where they previously weren't visible). While the implementation reuses existing components, it introduces new UI behavior that warrants human review.

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

Drafts remain in the composer store after navigation, but the legacy sidebar did not render them. That left users with no way to return to an unsent draft while the legacy sidebar was enabled. This ports [pingdotgg#7120](pingdotgg#7120) at pinned head `799ba957f51cd0b48ad8e5020090814318d0388f`. It exports the existing `SidebarDraftBlock` and reuses it above the legacy project list through a small wrapper that supplies project lookup, route state, and navigation. Upstream authorship and the `cherry-pick -x` provenance trailer are preserved. ## Validation - LastCode Quick CI passed on Node 24.13.1 at the current head. - Focused web typecheck and lint passed. - 3 nearby test files passed: 118 tests. - Manual isolated-app verification passed for draft appearance, restoration, discard, zero-draft layout, and default-sidebar parity. - Responsive verification passed at 465 × 800: selecting the persisted draft restored its exact text and closed the sidebar. - Matched desktop and responsive before/after evidence is attached in the PR conversation. Model and harness: GPT-5.6-Sol via the Codex harness in LastCode/T3 Code. Co-authored-by: VIvidh Mahajan <vividhmahajan@gmail.com>

The first complete upstream-import tracer exposed workflow requirements that do not belong in `upstream-fix`, which handles the opposite direction. Without a dedicated lane, future imports risk following moving PR heads, confusing path overlap with conflicts, or treating upstream CI as downstream acceptance. This adds an `import-upstream-pr` skill for evaluating open-unmerged and closed-unmerged T3 Code pull requests, preserving pinned provenance, validating on the current LastCode base, publishing evidence, and handing delivery to the existing guarded LastCode workflow. A focused reference captures intake receipts, duplicate detection, UI evidence, and squash patch-equivalence checks. ## Validation - Skill Creator `quick_validate.py` passed. - `git diff --check` passed. - Fresh-context read-only forward test against pingdotgg#7120 correctly detected the already-merged identical patch and stopped before creating another port. - LastCode Quick CI passed on Node 24.13.1. Model and harness: GPT-5.6-Sol via the Codex harness in LastCode/T3 Code.