Match hover-card model metadata to the composer - #67
Merged
Conversation
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.
Summary
Sonnet 5, notClaude-Sonnet-5), including decimal versions such asOpus 4.8 (1M)The plugin renders into an injected DOM surface, so it cannot directly reuse the host's React
ModelReasoningPicker. This change intentionally mirrors the composer's presentation semantics and shared color tokens without adding a new SDK or host dependency.Before
The reasoning label is smaller and fainter than the composer, and the provider icon is 13px.
After
The hover card now matches the composer: 12px/15px metadata, a 16px provider icon, grouped model/reasoning, and a quieter safe permission state.
Narrow desktop verification:
Permission hierarchy
Accept Edits and Auto are intentionally muted; Full Access retains the stronger warning treatment.
Accept Edits
Auto
Full Access
Reviewed final state
The single review found and fixed one P1: preserve the decimal in current Claude model IDs. The exact final head now renders
claude-opus-4-8[1m]asOpus 4.8 (1M).Verification
/ottonomous:review branchat151241a48; one validated P1 fixed; no re-review and no P2 expansionget-bb/bb@d81fee6f4; review-affected state recaptured on exact final head2424a0ab5claude-opus-4-8[1m], medium reasoning, and automatic approvalOpus 4.8 (1M)while preservingSonnet 5npm run check --workspace=bb-plugin-thread-hover-cardsnpm run artifacts -- plugins/thread-hover-cardsgit diff --checkBB-Thread-ID: thr_bwikffmsvp