Skip to content

Update provider branding color palettes - #3419

Open
elijahfriedman wants to merge 3 commits into
steipete:mainfrom
elijahfriedman:main
Open

Update provider branding color palettes#3419
elijahfriedman wants to merge 3 commits into
steipete:mainfrom
elijahfriedman:main

Conversation

@elijahfriedman

@elijahfriedman elijahfriedman commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update provider branding colors across 37 provider descriptors to better match current provider identities.
  • Align confetti palettes with the refreshed primary branding colors while preserving supporting neutral/secondary palette colors.
  • Keep provider registry constraints intact by ensuring every confetti palette remains 2-3 colors.
  • Preserve existing widget palettes by adding explicit widgetColor values where descriptors previously derived them from the old accent color.
  • Update the Command Code regression expectation to the refreshed 0x8C4EDD accent.

Details

A table of updated colors:

Provider Old Color New Color
Abacus AI #38BDF8 #814EE8
ai& #E25C2B #C70007
Amp #DC2626 #F34E3F
Augment #6366F1 #1AA049
AWS Bedrock #FF9900 #01A88D
Chutes #3184FF #63D297
ClawRouter #596EF6 #1F5AE0
ClinePass #61A3FA #5487C8
Codebuff #44FF00 #00FF95
Command Code #A04DFD #8C4EDD
Copilot #A855F7 #8534F3
Crof #2EAB94 #8B7CFF
Cursor #00BFA5 #F54E00
DeepSeek #527DF0 #4D6BFE
Deepgram #6467F2 #13EF93
Devin #46B482 #317CFF
Doubao #3370FF #0057FF
Fireworks #F25B1C #6720FF
Groq #F56844 #F55036
JetBrains AI #FF3399 #955AE0
Kilo #F27027 #FAF74F
Kimi Code #FE603C #007CFF
Kiro #FF9900 #9046FF
LiteLLM #4C89F0 #5B3FD1
LongCat #FFD100 #29E154
Mistral #FF500F #FF5229
Moonshot #205DEB #000000
Neuralwatt #38D98C #D55934
Notion AI #337EA9 #2EAADC
OpenCode #3B82F6 #3B7DD8
Perplexity #20B2AA #20808D
Qoder #10B981 #2ADB5C
Sakana AI #2975DB #CC2B2B
sub2api #2DC6D8 #14B8A6
T3 Chat #F56647 #A3004C
Venice #3399FF #3C8FDD
Warp #938BB4 #01A4FF

Testing

  • make check passes
  • swift test --filter MenuCardProviderRegressionTests --filter ProviderArchitectureGatekeeperTests passes
  • swift test --filter ProviderArchitectureGatekeeperTests passes

Screenshots

Screenshot 2026-09-04 at 9 47 37 AM

@clawsweeper

clawsweeper Bot commented Sep 4, 2026

Copy link
Copy Markdown

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 83799f37ae

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@clawsweeper clawsweeper Bot added P3 Low-risk cleanup, docs, polish, ergonomics, or speculative feature. merge-risk: 🚨 compatibility 🚨 Merging this PR could break existing users, config, migrations, defaults, or upgrades. proof: sufficient Contributor real behavior proof is sufficient. proof: 📸 screenshot Contributor real behavior proof includes screenshot evidence. rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. labels Sep 4, 2026
@clawsweeper

clawsweeper Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codex review: needs maintainer review before merge. Reviewed September 4, 2026, 10:04 AM ET / 14:04 UTC.

ClawSweeper review

What this changes

The PR refreshes menu and confetti accent colors across 37 provider descriptors, explicitly retains existing widget colors, and updates the Command Code color regression test.

Merge readiness

Ready for maintainer review

The previously reported widget-palette and Command Code test regressions are resolved at the current PR head. The verified test merge still contains this PR’s 38-file palette update, so the work remains distinct from current main and is ready for normal merge review once checks finish.

Priority: P3
Reviewed head: d47cede6ae586ae8774e2cf51bb967b3c730497b

Review scores

Measure Result What it means
Overall readiness 🐚 platinum hermit (4/6) A focused visual update with direct UI evidence, preserved widget behavior, and both prior blockers corrected.
Proof confidence 🐚 platinum hermit (4/6) ✨ media proof bonus Sufficient (screenshot): The PR body supplies an after-change 828×108 menu screenshot that directly demonstrates the descriptor-derived visual output; the corrected static widget separation is additionally protected by the repository’s descriptor invariant.
Patch quality 🐚 platinum hermit (4/6) No actionable review findings were identified.

Verification

Check Result Evidence
Real behavior Verified Sufficient (screenshot): The PR body supplies an after-change 828×108 menu screenshot that directly demonstrates the descriptor-derived visual output; the corrected static widget separation is additionally protected by the repository’s descriptor invariant.
Evidence reviewed 6 items Separate widget-color contract: ProviderBranding uses the menu accent only when widgetColor is omitted; the corrected descriptors now pass explicit widget values where needed, preserving the widget palette while changing the menu accent.
Representative corrected descriptor: Abacus now has the refreshed purple menu and confetti accent while explicitly retaining its former blue widget color.
Widget invariant remains protected: The architecture gatekeeper fingerprints every registered descriptor’s widget colors against the preserved legacy table, covering the earlier regression concern.
Findings None None.
Security None None.

How this fits together

CodexBar provider descriptors supply branding colors to menu usage cards, celebration effects, and widgets. Each descriptor’s branding is read by those UI surfaces, with the widget color intentionally separate from the menu accent.

flowchart LR
  A[Provider descriptors] --> B[Branding configuration]
  B --> C[Menu usage cards]
  B --> D[Confetti effects]
  B --> E[Widgets]
  F[Regression tests] --> B
Loading

Before merge

None.

Agent review details

Security

None.

Review metrics

Metric Value Why it matters
Palette scope 37 provider descriptors + 1 regression test The broad visual refresh remains confined to declarative branding values and one focused assertion.
Line delta production +126/-96, tests +1/-1 The production growth is the explicit widget-color preservation needed to keep the existing widget contract intact.

Technical review

Best possible solution:

Merge the scoped palette refresh with explicit widget-color preservation so menu branding can evolve without changing existing widget visuals.

Do we have a high-confidence way to reproduce the issue?

Not applicable: the PR updates static visual palette values rather than repairing a reported runtime failure.

Is this the best way to solve the issue?

Yes. Keeping widgetColor explicit while updating menu and confetti colors follows the shared branding model’s intended separation and preserves the existing widget invariant.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against 30f881aee0e8.

Labels

Label changes:

  • add rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🐚 platinum hermit and patch quality is 🐚 platinum hermit.
  • add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (screenshot): The PR body supplies an after-change 828×108 menu screenshot that directly demonstrates the descriptor-derived visual output; the corrected static widget separation is additionally protected by the repository’s descriptor invariant.
  • remove rating: 🦪 silver shellfish: Current PR rating is rating: 🐚 platinum hermit, so this older rating label is no longer current.
  • remove merge-risk: 🚨 compatibility: Current PR review selected no merge-risk labels.
  • remove status: ⏳ waiting on author: Current PR status label is status: 👀 ready for maintainer look.

Label justifications:

  • P3: This is a contained visual branding refresh with limited user impact.
  • rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🐚 platinum hermit and patch quality is 🐚 platinum hermit.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (screenshot): The PR body supplies an after-change 828×108 menu screenshot that directly demonstrates the descriptor-derived visual output; the corrected static widget separation is additionally protected by the repository’s descriptor invariant.
  • proof: sufficient: Contributor real behavior proof is sufficient. The PR body supplies an after-change 828×108 menu screenshot that directly demonstrates the descriptor-derived visual output; the corrected static widget separation is additionally protected by the repository’s descriptor invariant.
  • proof: 📸 screenshot: Contributor real behavior proof includes screenshot evidence. The PR body supplies an after-change 828×108 menu screenshot that directly demonstrates the descriptor-derived visual output; the corrected static widget separation is additionally protected by the repository’s descriptor invariant.

Evidence

What I checked:

Likely related people:

  • Peter Steinberger: Suggested for follow-up; no historical authorship or introduction is verified. (role: unverified routing candidate; confidence: low)
  • Peter Urda: Suggested for follow-up; no historical authorship or introduction is verified. (role: unverified routing candidate; confidence: low)

Rating scale

Score Internal tier Crab rank Meaning
6/6 S 🦀 challenger crab Exceptional readiness
5/6 A 🦞 diamond lobster Very strong readiness
4/6 B 🐚 platinum hermit Good normal PR; ordinary maintainer review
3/6 C 🦐 gold shrimp Useful, but confidence is limited
2/6 D 🦪 silver shellfish Proof or implementation needs work
1/6 F 🧂 unranked krab Not merge-ready
N/A NA 🌊 off-meta tidepool Rating does not apply

Overall follows the weaker of proof and patch quality.
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

Workflow

  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

History

Review history (1 earlier review cycle)
  • reviewed 2026-09-04T13:54:32.199Z sha 83799f3 :: blocked before merge. :: [P1] Preserve the existing widget palette | [P1] Update the Command Code regression expectation

@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. and removed rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. merge-risk: 🚨 compatibility 🚨 Merging this PR could break existing users, config, migrations, defaults, or upgrades. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. labels Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low-risk cleanup, docs, polish, ergonomics, or speculative feature. proof: 📸 screenshot Contributor real behavior proof includes screenshot evidence. proof: sufficient Contributor real behavior proof is sufficient. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant