refactor: reorganize pnpm catalogs for better scoping#481
Conversation
- Rename 'lint' → 'linting' - Rename 'test' → 'testing' - Rename 'workers' → 'api' - Rename 'dev' → 'build' for build tooling - Consolidate all production deps into 'prod' catalog - Remove 'dev' catalog in favor of 'build' Closes #480
|
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughReorganizes pnpm workspace catalogs: renames Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
🌏 Preview Deployments
Built from commit: 🤖 This comment will be updated automatically when you push new commits to this PR. |
There was a problem hiding this comment.
Pull request overview
This PR reorganizes pnpm workspace catalogs to improve dependency scoping and maintainability by renaming catalogs to be more descriptive and consolidating build tooling dependencies.
Changes:
- Renamed catalogs:
lint→linting,test→testing,workers→api,dev→build - Updated catalog references in pnpm-workspace.yaml and all package.json files across the monorepo
- Updated pnpm-lock.yaml to reflect the catalog reorganization
Reviewed changes
Copilot reviewed 23 out of 24 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-workspace.yaml | Renamed catalogs dev to build and workers to api, updated overrides to use new catalog names |
| pnpm-lock.yaml | Updated catalog definitions and package references to reflect renamed catalogs |
| package.json (root) | Updated typescript reference from catalog:dev to catalog:build |
| apps/api/package.json | Updated all catalog references from old names to new names (workers→api, dev→build) |
| apps/docs/package.json | Updated typescript and wrangler references to use new catalog names |
| apps/web/package.json | Updated typescript and wrangler references to use new catalog names |
| packages/*/package.json | Updated build tool references from catalog:dev to catalog:build across all packages |
| tooling/*/package.json | Updated typescript and build tool references to catalog:build |
| vscode/package.json | Updated tsdown and typescript references to catalog:build |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)
pnpm-lock.yaml:286
- The overrides section in pnpm-lock.yaml needs to be regenerated to match the updated pnpm-workspace.yaml. The workspace file specifies
rolldown: catalog:buildandwrangler: catalog:api, but the lockfile still has the old direct versionsrolldown: 1.0.0-rc.2andwrangler: 4.61.1. This inconsistency will cause issues during dependency installation. Runpnpm installto regenerate the lockfile with the correct override values.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
This PR reorganizes the pnpm workspace catalogs to have dependencies more closely scoped to the packages/apps that use them.
Changes
Catalog Renames
lint→lintingtest→testingworkers→apidev→build(for build tooling)Consolidated Catalogs
prod- All production dependencies including CLI tools and AI/schema generation depsbuild- Build tooling (tsdown, typescript, publint, tsx, rolldown, openapi-typescript, nanotar)Scoped Catalogs
monorepo- Turbo and release scriptsapi- Cloudflare Workers stack (wrangler, hono, @scalar/hono-api-reference, @sentry/cloudflare)web- Web app dependencies (React, TanStack, Tailwind, Vite, etc.)docs- Documentation site (fumadocs)vscode- VS Code extension (reactive-vscode, vscode-ext-gen, @vscode/vsce)types- Type definitions (@types/*)Migration
All package.json files have been updated to reference the new catalog names:
catalog:lint→catalog:lintingcatalog:test→catalog:testingcatalog:workers→catalog:apicatalog:dev→catalog:buildFiles Changed
pnpm-workspace.yaml- Updated catalog definitionspackage.jsonfiles - Updated catalog referencesCloses #480
Summary by CodeRabbit