Skip to content

feat: collect permission manifests#6

Merged
lusu007 merged 5 commits into
mainfrom
feat/permissions-query-commands
Jul 9, 2026
Merged

feat: collect permission manifests#6
lusu007 merged 5 commits into
mainfrom
feat/permissions-query-commands

Conversation

@lusu007

@lusu007 lusu007 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

Collects permission manifests from every active plugin/module and upserts them into the central permission catalog. Velocity discovers active plugin classloaders; Minestom uses the released runtime API v0.3.0 and activeModuleProviders. Missing or malformed manifests are skipped, while catalog registration uses bounded retries and does not block startup.

Manifests use META-INF/grounds/permissions.json. Command permissions are sourced from the same standardized manifest.

Dependency Order

  1. grounds-minestom-runtime#11 exposes active module provider origins and is released as v0.3.0.
  2. service-permissions#20 persists registered runtime catalog entries for Portal.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation
  • Chore

Testing

  • Unit tests pass: ./gradlew test
  • New tests cover manifest validation, Velocity/Minestom discovery, and registration retries
  • Formatting: ./gradlew spotlessApply
  • Build: ./gradlew build

@lusu007 lusu007 marked this pull request as ready for review July 9, 2026 19:25
Copilot AI review requested due to automatic review settings July 9, 2026 19:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@lusu007 lusu007 force-pushed the feat/permissions-query-commands branch from 2b7f90d to cd7d3d8 Compare July 9, 2026 19:26
@lusu007 lusu007 merged commit c78162e into main Jul 9, 2026
2 checks passed
@lusu007 lusu007 deleted the feat/permissions-query-commands branch July 9, 2026 19:48
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.

2 participants