[only21mil]

Problem

MessageCopyButton accepts optional size and className props and forwards them to Button, but no caller passes either one. Every copy button already renders at the default size="xs" with no extra class, so the props are unused API that suggests the button can be resized or restyled from outside.

Change

The two props, their types, and their forwarding are removed from apps/web/src/components/chat/MessageCopyButton.tsx. Button now gets size="xs" directly. label, icon, extraFlavors and variant are unchanged. One file, 1 line added, 6 removed.

Scope and approval

No issue. This is maintenance work with no behavior change: the button looks and works the same at every call site, so it needs no product-direction approval under CONTRIBUTING.md. If maintainers would rather keep the props for future callers, this PR can simply be closed.

Verification

• Every call site on main was checked: TimelineAssistantRows.tsx:224, TimelineUserMessageRow.tsx:334, 336, 787, 800 and MessageCopyButton.browser.test.tsx:11. None passes size or className. The one spread (TimelineUserMessageRow.tsx:789) only adds extraFlavors. The mobile UserMessageCopyButton is a separate component.
• Button receives the same props as before for every caller: size was always the "xs" default, and className was always undefined, which cn(buttonVariants(...)) treats the same as a missing prop.
• vp run check:changed 1d626d02d9: format, lint, knip, typecheck (2 packages), the MessageCopyButton browser test and the React compiler check all pass.

No screenshots: nothing renders differently, because every caller already got these exact Button props.

Not checked:

• The full CI test suites and the build.

Prior PRs

No open or closed PR makes this change. #13208 by juliusmarminge last reworked how this button forwards className and variant to Button. #12865 by saphid changed the action names, and #23679 by only21mil changed clipboard errors; neither changed these props.

Generated with Claude Opus 5.5 and GPT-6.1 Sol in T3 Code Nightly

[macroscopeapp]

Approvability

Verdict: Approved at add0139

Macroscope's review found this PR approvable — Removes unused copy-button customization props while preserving the existing size and behavior at all call sites. No product defaults or diagnostic suppressions change.

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

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 3fd2b85
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 6099ea29-f5e0-48d5-b86d-40cf40e4b553

Commits

Reviewing files that changed from the base of the PR and between 3fd2b85 and add0139.

Files selected for processing: 1

• apps/web/src/components/chat/MessageCopyButton.tsx

Limit details: You’ve used all 10 included reviews currently available.

Walkthrough

MessageCopyButton no longer accepts size or className props. Its button size is fixed to xs. Variant behavior remains unchanged.

Changes: Message copy button

Layer / File(s)	Summary
Button props and fixed size apps/web/src/components/chat/MessageCopyButton.tsx	The component no longer accepts size or className. The button always uses size xs; variant mapping is unchanged.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Refactor

Merge Risk: Minimal · up to add01

The change is limited to removing unused props and fixing the button size, with no identified merge-blocking risk.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly and concisely identifies the main change: removing the unused size and className props from MessageCopyButton.
Description check		The description includes all required sections, explains the problem and change, documents scope and approval rationale, reports focused verification results, identifies unchecked full CI and build ch…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description includes all required sections, explains the problem and change, documents scope and approval rationale, reports focused verification results, identifies unchecked full CI and build checks, and explains why screenshots are not applicable.

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Usage-based review receipt

• Mode: Continue automatically
• Reviewed files: 1
• Charged: $0.25
• View usage details

Note

This review was completed with usage-based billing: files reviewed beyond your plan's included limits are billed at $0.25/file. View usage-based billing.