fix: button-spec subvalues + lobe-ify openai/deepseek action art#95
Merged
Conversation
Two fixes prompted by a review:
1. Subvalue captions now follow Button Spec §7. Was using long
sentence-style captions ("Org cost (admin API)", "Avg daily DeepSeek
spend (last 7d)"); now uses short, source-tagged captions matching
the Claude/Codex "Cost (local)" precedent:
- anthropic admin: "Cost (admin)" for the windows, "$/day (7d avg)"
for burn rate, "MTD pace" for projected month
- openai admin: same as anthropic
- deepseek bridge path: "Cost (API)" for the windows (the "API"
tag also disambiguates the platform/api source from
chat.deepseek.com — DeepSeek's consumer chat product, which has
no public usage signal), "$/day (7d avg)", "MTD pace",
"Tokens (API)"
- deepseek API fallback stubs: "Needs Helper"
- deepseek platform error path: "Bridge error" / "Sign in to
platform.deepseek.com"; the full error string still lives in
Snapshot.Error for the PI / logs
2. Action library + key-preview thumbnails for OpenAI and DeepSeek
now use the lobe-icons paths (matching the runtime button-face
glyphs from internal/icons/lobe_generated.go). Earlier sync only
covered claude + anthropic, leaving openai still on the old
rosette and deepseek on the CodexBar-derived whale path. Mirrors
the same swap to docs/provider-icons/ for the marketing pills.
Display Names also tweaked to clarify DeepSeek = API platform: each
metric Name now reads "DeepSeek API …" (e.g. "DeepSeek API balance")
so the Property Inspector and tooltips don't conflate the API side
with chat.deepseek.com.
Co-Authored-By: Claude <noreply@anthropic.com>
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
1 participant
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
Two review-prompted fixes:
Test plan
🤖 Generated with Claude Code