Chore/cleanup

[LGLabGreg]

Summary by CodeRabbit

• Chores

  • Updated package manager to pnpm 10.26.1
  • Updated Next.js and related tooling to version 16.1.0
  • Updated development dependencies including testing and build tools

• Refactor

  • Removed the playground application
  • Removed multiple UI component libraries from documentation site
  • Removed ad configuration file

• Style

  • Adjusted shadow styling in switch component

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[cursor]

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on January 3.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[vercel]

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

Project	Deployment	Review	Updated (UTC)
reactqrcode-com	Ready	Preview, Comment	Dec 20, 2025 0:32am

[coderabbitai]

Walkthrough

The PR removes numerous Radix UI-based UI component wrappers from the docs application, deletes the entire playground app directory, updates tooling dependencies to newer versions, and removes several unused npm packages from dependency lists.

Changes

Cohort / File(s)	Change Summary
UI Component Removals apps/docs/src/components/ui/alert-dialog.tsx, alert.tsx, aspect-ratio.tsx, avatar.tsx, breadcrumb.tsx, carousel.tsx, collapsible.tsx, command.tsx, context-menu.tsx, drawer.tsx, dropdown-menu.tsx, hover-card.tsx, input-otp.tsx, menubar.tsx, navigation-menu.tsx, pagination.tsx, progress.tsx, radio-group.tsx, scroll-area.tsx, sonner.tsx, textarea.tsx, toggle-group.tsx	Complete removal of 22 Radix UI-based component modules, each exposing 1–15+ public component exports (AlertDialog, Avatar, Breadcrumb, Carousel, ContextMenu, Drawer, DropdownMenu, etc.). All internal forwardRef wrappers, styling logic, and re-exports eliminated.
Switch Component Update apps/docs/src/components/ui/switch.tsx	Changed Tailwind shadow utility from shadow-xs to shadow-sm in the root element className.
Resizable Component Removal apps/docs/src/components/ui/resizable.tsx	Removed ResizablePanelGroup, ResizablePanel, ResizableHandle wrapper around react-resizable-panels primitives and all associated imports.
Playground App Deletion apps/playground/ (all files)	Complete removal of playground application: package.json, vite.config.ts, tsconfig.json, tsconfig.app.json, tsconfig.node.json, src/App.tsx, src/main.tsx, src/index.css, src/vite-env.d.ts, src/constants.ts, src/components/ui/select.tsx, index.html.
Dependency & Configuration Updates apps/docs/package.json, packages/react-qr-code/package.json, package.json, pnpm-workspace.yaml, vitest.workspace.ts	Removed Radix UI packages (react-alert-dialog, react-aspect-ratio, react-avatar, react-collapsible, react-context-menu, react-dropdown-menu, react-hover-card, react-menubar, react-navigation-menu, react-progress, react-radio-group, react-scroll-area) and additional deps (cmdk, embla-carousel-react, input-otp, react-resizable-panels, sonner, vaul) from apps/docs; bumped Next.js to 16.1.0, @types/node to ^25.0.3, vite to ^7.3.0, vitest to 4.0.16, typescript-eslint to ^8.50.0; removed playground-related dev scripts and linting entries; updated pnpm version to 10.26.1; added onlyBuiltDependencies: esbuild to pnpm-workspace.yaml; removed vitest.workspace.ts default export.
Public Metadata Cleanup apps/docs/public/ads.txt	Removed the sole Google ad network entry line.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20–25 minutes

Areas requiring attention:

• Verify no active code or examples in docs still reference the removed UI components (alert-dialog, avatar, breadcrumb, carousel, context-menu, drawer, dropdown-menu, menubar, navigation-menu, pagination, etc.).
• Confirm that removal of Radix UI dependencies from apps/docs/package.json does not break any remaining components that may still use them indirectly.
• Check that playground app removal is intentional and no critical examples or test fixtures relied on it.
• Validate that the pnpm-workspace.yaml change and vitest.workspace.ts deletion do not break build/test pipelines or workspace resolution.
• Review switch.tsx shadow utility change for visual correctness in context.

Possibly related PRs

• Chore/deps 09 11 25 #313: Directly related conflict—removes Radix UI wrapper components and dependencies that this PR also removes, suggesting potential duplication or coordination needed.
• Chore deps #87: Modifies apps/playground/package.json, directly related to the playground deletion in this PR.
• Chore docs #98: Deletes the same apps/docs/src/components/ui files and playground app that this PR removes; likely a conflicting or duplicate change.

Poem

🐰 Old Radix wrappers fade away,
Components cleared, a lighter day,
Playground packed and tucked away,
Dependencies tidy, versions stay,
The docs now dance a simpler way! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Chore/cleanup' is vague and generic, using non-descriptive terms that don't convey meaningful information about the extensive changes in the changeset.	Consider a more specific title that captures the main change, such as 'Remove unused UI components and playground app' or 'Remove Radix UI components and playground application'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/cleanup

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7b727f5 and 09efa4f.

⛔ Files ignored due to path filters (2)

• apps/playground/public/vite.svg is excluded by !**/*.svg
• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (42)

• apps/docs/package.json (1 hunks)
• apps/docs/public/ads.txt (0 hunks)
• apps/docs/src/components/ui/alert-dialog.tsx (0 hunks)
• apps/docs/src/components/ui/alert.tsx (0 hunks)
• apps/docs/src/components/ui/aspect-ratio.tsx (0 hunks)
• apps/docs/src/components/ui/avatar.tsx (0 hunks)
• apps/docs/src/components/ui/breadcrumb.tsx (0 hunks)
• apps/docs/src/components/ui/carousel.tsx (0 hunks)
• apps/docs/src/components/ui/collapsible.tsx (0 hunks)
• apps/docs/src/components/ui/command.tsx (0 hunks)
• apps/docs/src/components/ui/context-menu.tsx (0 hunks)
• apps/docs/src/components/ui/drawer.tsx (0 hunks)
• apps/docs/src/components/ui/dropdown-menu.tsx (0 hunks)
• apps/docs/src/components/ui/hover-card.tsx (0 hunks)
• apps/docs/src/components/ui/input-otp.tsx (0 hunks)
• apps/docs/src/components/ui/menubar.tsx (0 hunks)
• apps/docs/src/components/ui/navigation-menu.tsx (0 hunks)
• apps/docs/src/components/ui/pagination.tsx (0 hunks)
• apps/docs/src/components/ui/progress.tsx (0 hunks)
• apps/docs/src/components/ui/radio-group.tsx (0 hunks)
• apps/docs/src/components/ui/resizable.tsx (0 hunks)
• apps/docs/src/components/ui/scroll-area.tsx (0 hunks)
• apps/docs/src/components/ui/sonner.tsx (0 hunks)
• apps/docs/src/components/ui/switch.tsx (1 hunks)
• apps/docs/src/components/ui/textarea.tsx (0 hunks)
• apps/docs/src/components/ui/toggle-group.tsx (0 hunks)
• apps/playground/index.html (0 hunks)
• apps/playground/package.json (0 hunks)
• apps/playground/src/App.tsx (0 hunks)
• apps/playground/src/components/ui/select.tsx (0 hunks)
• apps/playground/src/constants.ts (0 hunks)
• apps/playground/src/index.css (0 hunks)
• apps/playground/src/main.tsx (0 hunks)
• apps/playground/src/vite-env.d.ts (0 hunks)
• apps/playground/tsconfig.app.json (0 hunks)
• apps/playground/tsconfig.json (0 hunks)
• apps/playground/tsconfig.node.json (0 hunks)
• apps/playground/vite.config.ts (0 hunks)
• package.json (2 hunks)
• packages/react-qr-code/package.json (1 hunks)
• pnpm-workspace.yaml (1 hunks)
• vitest.workspace.ts (0 hunks)

💤 Files with no reviewable changes (37)

• apps/playground/src/vite-env.d.ts
• apps/docs/src/components/ui/toggle-group.tsx
• apps/docs/src/components/ui/sonner.tsx
• vitest.workspace.ts
• apps/docs/src/components/ui/aspect-ratio.tsx
• apps/docs/src/components/ui/textarea.tsx
• apps/playground/src/constants.ts
• apps/docs/src/components/ui/command.tsx
• apps/docs/src/components/ui/collapsible.tsx
• apps/docs/src/components/ui/drawer.tsx
• apps/docs/src/components/ui/radio-group.tsx
• apps/playground/src/components/ui/select.tsx
• apps/docs/src/components/ui/resizable.tsx
• apps/docs/src/components/ui/pagination.tsx
• apps/docs/src/components/ui/hover-card.tsx
• apps/docs/src/components/ui/scroll-area.tsx
• apps/docs/src/components/ui/alert-dialog.tsx
• apps/docs/src/components/ui/breadcrumb.tsx
• apps/docs/src/components/ui/carousel.tsx
• apps/docs/src/components/ui/context-menu.tsx
• apps/playground/index.html
• apps/docs/src/components/ui/avatar.tsx
• apps/docs/src/components/ui/alert.tsx
• apps/docs/public/ads.txt
• apps/playground/vite.config.ts
• apps/docs/src/components/ui/menubar.tsx
• apps/docs/src/components/ui/progress.tsx
• apps/playground/src/index.css
• apps/docs/src/components/ui/navigation-menu.tsx
• apps/playground/src/main.tsx
• apps/playground/tsconfig.node.json
• apps/playground/tsconfig.app.json
• apps/playground/tsconfig.json
• apps/docs/src/components/ui/input-otp.tsx
• apps/docs/src/components/ui/dropdown-menu.tsx
• apps/playground/src/App.tsx
• apps/playground/package.json

🔇 Additional comments (11)

pnpm-workspace.yaml (2)

2-3: LGTM: Valid glob pattern format.

The unquoted glob syntax is standard for pnpm workspace configurations and has no functional impact.

---

5-6: onlyBuiltDependencies allows lifecycle scripts; consider using allowBuilds instead.

The onlyBuiltDependencies configuration allows esbuild to execute its "preinstall", "install", and/or "postinstall" scripts during installation. Note that pnpm 10.26 introduced allowBuilds as a more flexible replacement that accepts a map of package matchers to explicitly allow or disallow script execution. For consistency with the preferred pattern, consider migrating to allowBuilds:

allowBuilds:
  esbuild: true

apps/docs/src/components/ui/switch.tsx (1)

14-14: LGTM: Standardized to Tailwind's built-in shadow utility.

The change from shadow-xs to shadow-sm aligns the component with Tailwind's standard utilities. Ensure this visual change has been reviewed in the UI to confirm the shadow appearance matches design expectations.

apps/docs/package.json (3)

49-49: LGTM: Consistent type definitions update.

The @types/node update to ^25.0.3 aligns with the same update in other packages, maintaining consistency across the monorepo.

---

34-34: No action needed for Next.js 16.1.0 compatibility.

Next.js 16 uses the latest React Canary release, which includes React 19.2 features. The minor version bump from 16.0.10 to 16.1.0 introduces faster development workflows and improved stability, with major updates to Turbopack and tooling. This update is compatible with React 19.2.3 with no breaking changes requiring code modification.

Also applies to: 36-36

---

12-44: No action needed. Verification confirms that none of the removed dependencies are imported in the codebase, so no orphaned imports or runtime errors will occur.

package.json (4)

8-8: LGTM: Package manager version updated.

The pnpm version bump to 10.26.1 aligns with the workspace configuration changes. Ensure this version is used consistently in CI/CD pipelines.

---

22-31: LGTM: Lint-staged configuration updated for playground removal.

The removal of playground-related entries from lint-staged is consistent with the broader cleanup of the playground app.

---

11-11: Playground removal is complete.

The watch:playground script has been removed from the dev workflow. No remaining references to the playground app exist in configuration files, scripts, or documentation.

---

38-38: Verify test suite passes with updated dependencies.

These devDependency updates include major versions for testing and tooling packages. While the versions are compatible, confirm that:

• vitest 4.0.16 and @testing-library/react 16.3.1 work with your test setup
• Test coverage reporting runs without errors
• Linting completes successfully with typescript-eslint 8.50.0

packages/react-qr-code/package.json (1)

52-52: The devDependency version updates are compatible with TypeScript ~5.9.3.

typescript-eslint 8.50.0 supports TypeScript >=4.8.4 <6.0.0, which includes 5.9.3. Vite 7.3.0 is a smooth update from Vite 6 with no breaking changes in patch versions. All three devDependency updates are safe to use.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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