Skip to content

Comments

feat: add vue-nuxt-permission module#1402

Open
keroloszakaria wants to merge 12 commits intonuxt:mainfrom
keroloszakaria:main
Open

feat: add vue-nuxt-permission module#1402
keroloszakaria wants to merge 12 commits intonuxt:mainfrom
keroloszakaria:main

Conversation

@keroloszakaria
Copy link

Adds vue-nuxt-permission to the Nuxt modules listing.

Module Info

What it does

A unified permission management system for Vue 3 and Nuxt 3/4 that provides:

  • v-permission directive — show/hide DOM elements based on user permissions
  • usePermission() composable — reactive permission checking with async support (auto-imported in Nuxt)
  • Route guards — permission-based and auth-based route protection
  • 6 matching modes — and, or, not, startWith, endWith, regex
  • Built-in caching & persistence via localStorage/sessionStorage

Closes #1400

@coderabbitai
Copy link

coderabbitai bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Adds a new module descriptor file modules/vue-nuxt-permission.yml containing metadata for the vue-nuxt-permission project (name, description, repo, npm, website, learn_more, category, type), a maintainers list, and compatibility requirements specifying Nuxt >= 3.0.0 with bridge support disabled. No source code or public API/exported entity changes were made.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat: add vue-nuxt-permission module' accurately summarizes the main change: adding a new module descriptor for vue-nuxt-permission.
Description check ✅ Passed The description clearly relates to the changeset by explaining what vue-nuxt-permission is, its features, and the relevant links/metadata being added to the modules listing.
Linked Issues check ✅ Passed The PR adds the vue-nuxt-permission module metadata to the repository as required by issue #1400, providing documentation links, category, compatibility info, and module description.
Out of Scope Changes check ✅ Passed The only change is adding a new module descriptor YAML file, which is directly in scope for the module listing requirement from issue #1400 with no unrelated modifications.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@modules/vue-nuxt-permission.yml`:
- Line 3: The maintainer GitHub handle is cased inconsistently: update the
maintainers[0].github value from "KerolosZakaria" to the canonical lowercase
"keroloszakaria" to match the repo field and ensure profile links resolve
correctly; locate the maintainers array in modules/vue-nuxt-permission.yml and
change the github value for the first maintainer accordingly.
- Around line 3-6: Remove the unsupported duplicate github field and fix the
icon value: delete the github entry that contains
"https://github.com/keroloszakaria/vue-nuxt-permission" since the repo:
keroloszakaria/vue-nuxt-permission already encodes that, and update the icon
from the bare identifier "lock" to a filename with an extension (e.g.,
"lock.svg" or "lock.png") so the registry schema is satisfied.

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.

[Module Listing Request]: vue-nuxt-permission

1 participant