Skip to content

Add XenForo addon.json schema#5499

Open
jerpr wants to merge 4 commits intoSchemaStore:masterfrom
jerpr:xenforo-addon-schema
Open

Add XenForo addon.json schema#5499
jerpr wants to merge 4 commits intoSchemaStore:masterfrom
jerpr:xenforo-addon-schema

Conversation

@jerpr
Copy link
Copy Markdown

@jerpr jerpr commented Mar 26, 2026

This adds a schema for XenForo addon.json files. We may land support for naming them addon.xf.json in the future to allow resolution from add-on root folders.

@jerpr jerpr requested a review from madskristensen as a code owner March 26, 2026 07:02
Copilot AI review requested due to automatic review settings March 26, 2026 07:02
@jerpr jerpr requested a review from hyperupcall as a code owner March 26, 2026 07:02
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds SchemaStore catalog support for detecting XenForo add-on manifests (addon.json) so editors can auto-associate the XenForo schema for validation and completion.

Changes:

  • Register “XenForo add-on manifest” in the JSON schema catalog.
  • Add a fileMatch glob targeting XenForo add-on manifest locations.
  • Point the catalog entry to XenForo’s published schema URL.

jerpr and others added 2 commits March 26, 2026 00:06
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This is too narrow as there isn't necessarily a vendor.

This reverts commit 13c902f.
@madskristensen
Copy link
Copy Markdown
Contributor

is this ready to merge?

@jerpr
Copy link
Copy Markdown
Author

jerpr commented Mar 26, 2026

Yes 🙂

@hyperupcall
Copy link
Copy Markdown
Member

I myself wont be merging this as-is, since I think match "**/src/addons/**/addon.json" is too generic, but I will merge a pattern that matches addon.xf.json instead.

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.

4 participants