Skip to content

Conversation

@kiwicopple
Copy link
Member

This PR completes the migration to shadcn/ui components and modernizes the form handling system.

This PR completes the migration to shadcn/ui components and modernizes
the form handling system.

## Component Migrations

- **Toast System**: Replace react-hot-toast with shadcn toast
  - Add toast.success() and toast.error() helper methods
  - Update all 8 files using toasts

- **Search**: Refactor to use shadcn Popover + cmdk Command
  - Add keyboard navigation and accessibility
  - Better semantic structure

- **Cards**: Convert PackageCard and AccessTokenCard to shadcn Card

- **Buttons**: Convert ThemeSwitcher and Edit page to shadcn Button

- **Forms**: Add Alert, Textarea, Dialog components

## Form System Migration

- Replace react-final-form with react-hook-form
- Use @hookform/resolvers for Zod validation
- Add noValidate for consistent JS-based validation
- Remove unused zod-form-validator-utils.ts

## Cleanup

- Remove react-hot-toast, react-final-form, final-form dependencies
- Remove redundant loader.tsx component
- Standardize on Skeleton + Spinner for loading states

## Testing

- Update tests for react-hook-form API
- Add ResizeObserver/scrollIntoView mocks for cmdk
- All 24 tests pass
@vercel
Copy link

vercel bot commented Jan 15, 2026

@claude is attempting to deploy a commit to the Supabase Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 15, 2026

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

Project Deployment Review Updated (UTC)
dbdev Ready Ready Preview, Comment Jan 15, 2026 8:59am

Copy link
Member

@alaister alaister left a comment

Choose a reason for hiding this comment

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

LGTM!

  • Scanned code and tested on preview

@alaister alaister merged commit db357d5 into supabase:master Jan 15, 2026
7 checks passed
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.

3 participants