Skip to content

fix(module): transpile reka-ui to prevent injection errors#6286

Merged
benjamincanac merged 4 commits intov4from
deps/reka-ui-peer
Apr 3, 2026
Merged

fix(module): transpile reka-ui to prevent injection errors#6286
benjamincanac merged 4 commits intov4from
deps/reka-ui-peer

Conversation

@benjamincanac
Copy link
Copy Markdown
Member

@benjamincanac benjamincanac commented Apr 2, 2026

🔗 Linked issue

Resolves #6250

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@github-actions github-actions Bot added the v4 #4488 label Apr 2, 2026
@benjamincanac benjamincanac changed the title fix(module): add reka-ui as optional peer dependency to prevent duplicate installs fix(module): add reka-ui as optional peer to prevent duplicate installs Apr 2, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 2, 2026

npm i https://pkg.pr.new/@nuxt/ui@6286

commit: 8aa2490

@benjamincanac benjamincanac changed the title fix(module): add reka-ui as optional peer to prevent duplicate installs fix(module): deduplicate reka-ui to prevent injection errors Apr 2, 2026
@benjamincanac benjamincanac changed the title fix(module): deduplicate reka-ui to prevent injection errors fix(module): transpile reka-ui to prevent injection errors Apr 3, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 3, 2026

📝 Walkthrough

Walkthrough

The change modifies src/module.ts to add reka-ui to Nuxt's build transpile configuration during module setup. This ensures that the reka-ui dependency is transpiled as part of the Nuxt build pipeline rather than being processed differently. The modification consists of adding a single configuration statement to the module's setup logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is incomplete and does not provide details about the changes, but it is linked to issue #6250 which contextualizes the fix. Add a description explaining why reka-ui needs to be transpiled and how this prevents the injection errors mentioned in issue #6250.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding transpile configuration for reka-ui to fix injection errors.
Linked Issues check ✅ Passed The PR code change directly addresses the linked issue #6250 by adding reka-ui to build.transpile to prevent symbol injection conflicts.
Out of Scope Changes check ✅ Passed The change is narrowly focused on adding reka-ui transpilation, which is directly related to resolving the symbol injection error in issue #6250.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch deps/reka-ui-peer

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

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

@benjamincanac benjamincanac merged commit b822c43 into v4 Apr 3, 2026
18 checks passed
@benjamincanac benjamincanac deleted the deps/reka-ui-peer branch April 3, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v4 #4488

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Symbol injection error when using NavigationMenu from "reka-ui" and UDropdownMenu

1 participant