fix(web): the pull request badge reads at the meta size again

[juliusmarminge]

#13034 rendered the thread pull request badge through InlineButton in the sidebar and Button size="xs" in the composer. Both brought their own type: font-medium, plus 14px from the one-line sidebar row or from Button below sm. The ghost Button also muted the glyph, so the badge grew and lost its state color.

The badge now takes a render prop for the control it sits in, and owns only its content: the glyph and number at text-xs, regular weight, tabular figures, in the state tone. Base UI's useRender merges the badge's element (the link, or the stack button) and its click behavior into that control, the same way every components/ui export composes render.

• Sidebar row: render={<InlineButton />}
• Composer: render={<ComposerControl size="xs" />}, the same control as the model and branch pickers next to it

This also drops one no-restyle finding (the tone className on Button), so the ceiling goes 628 → 627.

Measured in a real client: 12px/400 in the state color in both places. The composer control is 24px tall on desktop and 28px on narrow, matching its neighbours.

	Before (main)	After
Sidebar row
Composer (narrow)

Composer on desktop, after:

Claude Opus 5.5 via Claude Code.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Refactor
  • Updated thread status controls with more consistent link and button styling.
  • Preserved existing pull request and stack action behavior, accessibility labels, pointer handling, and tooltips.
  • Improved control rendering in branch selectors and sidebars for a more consistent interface.
  • Pull request badges now integrate more naturally with surrounding controls while maintaining their existing functionality.

[macroscopeapp]

Approvability

Verdict: Approved at d6287ee

Macroscope's review found this PR approvable — This is a narrowly scoped UI bug fix that reuses existing button primitives to restore the pull-request badge’s meta-sized presentation without changing data, navigation intent, or broader workflows. The lint-threshold update only tightens CI enforcement, and the open feedback is limited to a non-blocking primitive-consistency concern.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Shared primitives over custom controls: this styled raw <button> rebuilds the existing Button control. Please render it through Button from components/ui/button instead.

Suggested fix: restore the Button primitive and its import; this requires changes outside this hunk, so no inline diff is provided.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review completed with one inline finding.

^{Posted via Macroscope — UI Consistency}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.5 KiB	13.5 KiB	+18 B (+0.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.1 KiB	7.0 KiB	−4 B (−0.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.5 KiB	6.5 KiB	+22 B (+0.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	56.3 KiB	56.3 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.5 KiB	13.5 KiB	−42 B (−0.3%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	−3 B (−0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.4 KiB	−39 B (−0.6%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.1 KiB	57.0 KiB	−44 B (−0.1%)	66.4 KiB	✅
Claude	Live turn messages	10	9	−1 (−10.0%)	21	✅

Baseline: 2eaff08 · PR result: d6287ee · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 114.0 KiB
• Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 94b1e215-b98d-4d16-a127-988e12afe0cc

📥 Commits

Reviewing files that changed from the base of the PR and between 7c6fc07 and d6287ee.

📒 Files selected for processing (1)

• apps/web/src/components/ThreadStatusIndicators.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 4 remain after this review.

---

📝 Walkthrough

Walkthrough

ThreadPullRequestBadgeControl now accepts a caller-provided render element. The branch toolbar and sidebar provide specific controls. The component preserves stack and pull-request behavior. The restyle lint ceiling decreases from 628 to 627.

Changes

Thread status controls

Layer / File(s)	Summary
Direct trigger rendering apps/web/src/components/ThreadStatusIndicators.tsx	ThreadPullRequestBadgeControl accepts a render element instead of a variant. It uses useRender to add stack-button or pull-request-link behavior, accessibility labels, pointer handling, click handling, and badge styling.
Caller renderer integrations apps/web/src/components/BranchToolbarBranchSelector.tsx, apps/web/src/components/Sidebar.tsx	The branch toolbar supplies ComposerControl. The sidebar supplies InlineButton.

Restyle ceiling

Layer / File(s)	Summary
Lint threshold update scripts/lint-restyle-ceiling.ts	RESTYLE_CEILING decreases from 628 to 627.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Bug fix

Suggested reviewers: maria-rcks

Merge Risk: ⚪ Minimal · up to d6287

The trigger-rendering and styling update appears ready to merge.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 4 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the pull request badge styling fix.
Description check	✅ Passed	The description clearly explains the change, the reason, the implementation, and the UI results. It includes before-and-after screenshots. The formal Checklist section is absent, but the required info…

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

Dismissing prior approval to re-evaluate 7c6fc07

Dismissing prior approval to re-evaluate d6287ee