catalyst icon indicating copy to clipboard operation
catalyst copied to clipboard

chore: rename /vibes/soul/ to /ui/

Open jorgemoya opened this issue 9 months ago • 4 comments

What/Why?

Rename /vibes/soul/ to /ui/, replacing all imports and configuration rules to match new pathname.

Testing

No eslint errors.

Migration instructions

Rebase conflicts

  • Custom components in /vibes/soul/* need to be moved to /ui/.
  • Change import name from @/vibes/soul/ to @/ui/.
  • If import order errors occur, run pnpm run lint -- --fix to fix import order or fix manually.

Manual migration

  • Replace /vibes/soul/ for /ui/.
  • Rename all imports from @/vibes/soul/ to @/ui/.
  • Update tsconfig.json to point to the @/ui path.
  • Update next.config.ts to replaces vibes with ui in the included eslint dirs.
  • Update global.css to replace ./vibes with ./ui.
  • Run pnpm run lint -- --fix to auto fix import order.

jorgemoya avatar Apr 18 '25 15:04 jorgemoya

🦋 Changeset detected

Latest commit: a157ba4a158d4568e7a882d592e4d931e08be661

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst-core Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Apr 18 '25 15:04 changeset-bot[bot]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
catalyst-canary ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 18, 2025 5:10pm
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
catalyst ⬜️ Ignored (Inspect) Apr 18, 2025 5:10pm
catalyst-au ⬜️ Ignored (Inspect) Visit Preview Apr 18, 2025 5:10pm
catalyst-soul ⬜️ Ignored (Inspect) Visit Preview Apr 18, 2025 5:10pm
catalyst-uk ⬜️ Ignored (Inspect) Visit Preview Apr 18, 2025 5:10pm

vercel[bot] avatar Apr 18 '25 15:04 vercel[bot]

Thanks for the migration steps!

bookernath avatar Apr 18 '25 16:04 bookernath

⚡️🏠 Lighthouse report

Lighthouse ran against https://catalyst-canary-hzajnbm2p-bigcommerce-platform.vercel.app

🖥️ Desktop

We ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:

Category Score
🟢 Performance 99
🟢 Accessibility 92
🟠 Best practices 78
🟠 SEO 82

📱 Mobile

We ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:

Category Score
🟠 Performance 83
🟢 Accessibility 92
🟠 Best practices 78
🟠 SEO 85

github-actions[bot] avatar Apr 18 '25 17:04 github-actions[bot]

Closing for now

jorgemoya avatar Jul 08 '25 18:07 jorgemoya