Skip to content

Fix build break from react-icons 5.7.0 (missing OpenAI icon) - #5079

Merged
Duncanma merged 4 commits into
mainfrom
fix/react-icons-5.7-openai-icon
Aug 12, 2026
Merged

Fix build break from react-icons 5.7.0 (missing OpenAI icon)#5079
Duncanma merged 4 commits into
mainfrom
fix/react-icons-5.7-openai-icon

Conversation

@Duncanma

@Duncanma Duncanma commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bumping react-icons to 5.7.0 (dependabot chore(deps): bump react-icons from 5.6.0 to 5.7.0 #5071) breaks the build: that release bundles a newer simple-icons which removed the SiOpenai export entirely, not renamed it.
  • Root cause: OpenAI's Nov 2025 rebrand triggered simple-icons#12739, leading to simple-icons#13944 ("Remove OpenAI"), done ahead of their v16.0.0 release pending brand permissions from OpenAI that were never granted. It's tracked as blocked from returning via simple-icons#14748 (tagged [Defensible]). So pinning to react-icons 5.6.x is the only way to keep this icon, and that blocks all future updates.
  • Fix: inline the OpenAI glyph as a local OpenAIIcon component in LLMActions.tsx, sourced from the last react-icons release that shipped it (5.6.0), matching react-icons' own rendering conventions (viewBox, fill="currentColor") so it's visually and behaviorally identical. SiClaude is unaffected and still imported from react-icons/si.
  • Bumps react-icons to ^5.7.0, matching chore(deps): bump react-icons from 5.6.0 to 5.7.0 #5071's package.json/yarn.lock diff exactly. This PR supersedes chore(deps): bump react-icons from 5.6.0 to 5.7.0 #5071.

Test plan

  • yarn build succeeds (previously failed the same way CI does on chore(deps): bump react-icons from 5.6.0 to 5.7.0 #5071)
  • Served the production build and visually confirmed the OpenAI icon renders correctly next to the Claude icon, with working hover/color and accessibility attributes preserved

┆Attachments: EDU-6927 Fix build break from react-icons 5.7.0 (missing OpenAI icon)

- Bump react-icons from 5.6.0 to 5.7.0.
- Inline OpenAI icon in LLMActions component due to removal from react-icons package.
@Duncanma
Duncanma requested a review from a team as a code owner August 10, 2026 17:17
@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
temporal-documentation Ready Ready Preview Aug 12, 2026 5:18pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

📖 Docs PR preview links

This PR does not change any pages in /docs. If you make updates, links to the modified pages will appear here.

@Duncanma
Duncanma merged commit 84c41ce into main Aug 12, 2026
12 of 13 checks passed
@Duncanma
Duncanma deleted the fix/react-icons-5.7-openai-icon branch August 12, 2026 17:19
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.

2 participants