Skip to content

chore(deps): drop @radix-ui/react-icons and react-icons (use lucide only)#34

Open
abdout wants to merge 1 commit into
mainfrom
chore/drop-redundant-icons
Open

chore(deps): drop @radix-ui/react-icons and react-icons (use lucide only)#34
abdout wants to merge 1 commit into
mainfrom
chore/drop-redundant-icons

Conversation

@abdout
Copy link
Copy Markdown
Contributor

@abdout abdout commented Apr 25, 2026

Summary

  • Replace 6 icon usages from `@radix-ui/react-icons` and `react-icons` with `lucide-react` equivalents.
  • Drop both deps from `package.json` — `components.json` declares lucide canonical.

Mapping

Old New
`FaUser` (`react-icons/fa`) `User` (aliased `UserIcon`)
`ExitIcon` `LogOut` (also swapped `mr-2`→`me-2` per RTL plan)
`ExclamationTriangleIcon` `AlertTriangle`
`CheckCircledIcon` `CheckCircle2`

Files

  • `src/components/auth/user-button.tsx`
  • `src/components/auth/form-error.tsx`
  • `src/components/auth/form-success.tsx`
  • `src/components/auth/error-card.tsx`
  • `src/components/auth/error/error-card.tsx`
  • `src/components/auth/error/form-error.tsx`
  • `package.json`, `pnpm-lock.yaml`

Test plan

  • `pnpm install` succeeds; lockfile clean
  • FormError + FormSuccess render correct icons
  • Error card displays AlertTriangle
  • No TS regressions

Closes #33

🤖 Generated with Claude Code

components.json declares lucide-react canonical, but the repo carried
two extra icon libraries used by 6 files combined. Replace with lucide
equivalents (FaUser→User, ExitIcon→LogOut, ExclamationTriangleIcon→
AlertTriangle, CheckCircledIcon→CheckCircle2) and drop the deps.

Closes #33

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 25, 2026

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

Project Deployment Actions Updated (UTC)
marketing Ready Ready Preview, Comment Apr 25, 2026 2:00pm

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.

chore(deps): drop @radix-ui/react-icons and react-icons (use lucide only)

1 participant