Use real ButtonV2 prop values in the Skills tab - #259
Closed
kateebonner wants to merge 1 commit into
Closed
Conversation
The Skills tab shipped with four size values and three variants that are not in ButtonV2's unions — size accepts "small" | "normal" | "large" and variant has no "secondary" — so the workspace has failed `bun typecheck` with 7 errors since it landed. The pre-push hook runs that same typecheck, which means this has been blocking pushes on every branch in the fork, not just its own. Mapped to the values the rest of settings-v2 already uses: "xs"/"sm" → "small" (13 existing uses), "secondary" → "neutral" (10). Rendering is unchanged in intent — these are the nearest real names for what the markup was asking for. Typecheck is now 30/30 across the workspace.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Unblocks
bun typecheckonlocal/amicode.The Skills tab (a6c8255, amicode #573) uses four
sizevalues and threevariantvalues that are not inButtonV2's unions:"xs","sm"and"secondary"are none of those, so the workspace has failedtypecheck with 7 errors since it merged.
Why it matters beyond the errors
.husky/pre-pushruns a full-workspacebun typecheck. So this has beenblocking pushes on every branch in the fork, not just its own — anyone who
hit it had to either stop or reach for
--no-verify, which is how genuinebreakage slips through later. (#258 needed exactly that bypass for its final
push, disclosed there.)
The change
Seven call sites in one file, mapped to the values the rest of
settings-v2already uses rather than to my preference:
size="xs"/size="sm"size="small"variant="secondary"variant="neutral"Every invalid value in the whole
settings-v2directory was in this one file,so this brings it in line with its siblings. Rendering is unchanged in intent —
these are the nearest real names for what the markup was already asking for.
Verified
bun typecheck— 30/30 tasks, workspace clean. First green since #573.bun run check:design— passes.bun run lint— 5366 warnings / 1 error, byte-identical to base; adds nothing.--no-verify.Note for the author
@ the #573 author — flagging rather than assuming: I picked
neutralfor thethree
secondarybuttons (Add directory, Autodiscover, and the discovered-rowaction) because it is the settings-v2 default. If you meant those to read as
lower-emphasis than the primary action,
ghost-mutedis the other establishedchoice and is what the one other in-row action uses. Happy to switch.