Skip to content

Show configured provider logos in t3sidebar - #1164

Draft
smsunarto wants to merge 1 commit into
get-bb:mainfrom
smsunarto:scott/fix-t3sidebar-provider-icon
Draft

Show configured provider logos in t3sidebar#1164
smsunarto wants to merge 1 commit into
get-bb:mainfrom
smsunarto:scott/fix-t3sidebar-provider-icon

Conversation

@smsunarto

Copy link
Copy Markdown

Summary

  • load BB's provider metadata once through the t3sidebar backend
  • render a provider's configured logo before the built-in Codex/Claude glyphs
  • keep the neutral fallback when provider metadata or logo loading is unavailable
  • cover the Amp ACP provider logo path in the sidebar test

Why

Custom ACP providers already expose their configured logoUrl, but t3sidebar only recognized Codex and Claude. That left providers such as Amp as a neutral dot even though BB had their brand mark available.

Validation

  • pnpm exec turbo run typecheck --filter=bb-plugin-t3sidebar
  • pnpm exec turbo run test --filter=bb-plugin-t3sidebar --force (81 tests)
  • pnpm exec prettier --check examples/plugins/t3sidebar/src/server.ts examples/plugins/t3sidebar/src/ProviderGlyph.tsx examples/plugins/t3sidebar/src/ThreadCard.tsx examples/plugins/t3sidebar/src/ThreadInbox.tsx examples/plugins/t3sidebar/src/ThreadInbox.test.tsx examples/plugins/t3sidebar/README.md
  • pnpm run bb -- plugin build examples/plugins/t3sidebar

@smsunarto
smsunarto marked this pull request as draft August 7, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant