Skip to content

fix(build): replace SiCss3 with SiCss for react-icons 5.6.0 compatibility#1663

Open
tpaustin wants to merge 1 commit intogeneralaction:mainfrom
tpaustin:fix/react-icons-css3-icon
Open

fix(build): replace SiCss3 with SiCss for react-icons 5.6.0 compatibility#1663
tpaustin wants to merge 1 commit intogeneralaction:mainfrom
tpaustin:fix/react-icons-css3-icon

Conversation

@tpaustin
Copy link
Copy Markdown

@tpaustin tpaustin commented Apr 4, 2026

Summary

  • Replaces the removed SiCss3 export with the new SiCss export in FileIcons.tsx
  • Bumps the minimum react-icons version to ^5.6.0 so the dependency and the code are in sync

Background

SiCss3 was removed from react-icons in 5.6.0. The current package.json specifies ^5.5.0, which allows 5.6.0. The pnpm-lock.yaml still pins to 5.5.0, so contributors with the lockfile intact are unaffected today — but anyone whose lockfile resolves to 5.6.0 hits an immediate build failure:

src/renderer/components/FileExplorer/FileIcons.tsx: "SiCss3" is not exported by "node_modules/react-icons/si/index.mjs"

Bumping to ^5.6.0 makes the lock consistent and prevents the breakage for any contributor who runs pnpm install going forward.

Test plan

  • pnpm run build passes
  • pnpm run type-check passes
  • CSS files display the correct icon in the file explorer

Closes #1662

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores

    • Updated react-icons dependency to the latest version.
  • Style

    • Updated the icon used for CSS files in the file explorer.

…lity

SiCss3 was removed from react-icons in 5.6.0. Replace it with the new
SiCss export and bump the minimum version to ^5.6.0 so the dependency
and the code are in sync.

Closes generalaction#1662

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 4, 2026

@tpaustin is attempting to deploy a commit to the General Action Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 81e126d0-1c2c-425c-9cb1-c5cd5298a4b1

📥 Commits

Reviewing files that changed from the base of the PR and between 6a89f67 and 2550bbe.

📒 Files selected for processing (2)
  • package.json
  • src/renderer/components/FileExplorer/FileIcons.tsx

📝 Walkthrough

Walkthrough

Updated react-icons dependency from version 5.5.0 to 5.6.0 and replaced the SiCss3 icon component with SiCss in FileIcons.tsx to resolve a breaking change in the library where SiCss3 was removed.

Changes

Cohort / File(s) Summary
Dependency Update
package.json
Updated react-icons version constraint from ^5.5.0 to ^5.6.0.
Component Icon Reference
src/renderer/components/FileExplorer/FileIcons.tsx
Replaced SiCss3 with SiCss in CSS file extension icon mapping.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop, a skip, a small refactor too!
CSS icons dancing in their debut,
From SiCss3 to SiCss so bright,
React-icons 5.6 shines in the light!
The build is mended, the code takes flight! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely summarizes the main change: replacing SiCss3 with SiCss for react-icons 5.6.0 compatibility.
Linked Issues check ✅ Passed The pull request fully implements both required changes: replacing SiCss3 with SiCss in FileIcons.tsx and bumping react-icons to ^5.6.0 in package.json.
Out of Scope Changes check ✅ Passed All changes are directly aligned with the objectives stated in issue #1662; no out-of-scope modifications were introduced.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

build: SiCss3 removed in react-icons 5.6.0 breaks renderer build

1 participant