Skip to content

feat: enforce extension manifest + shared registry runtime parity#321

Open
eva57gr wants to merge 5 commits intoLight-Heart-Labs:mainfrom
eva57gr:feat/extension-manifest-enforcement
Open

feat: enforce extension manifest + shared registry runtime parity#321
eva57gr wants to merge 5 commits intoLight-Heart-Labs:mainfrom
eva57gr:feat/extension-manifest-enforcement

Conversation

@eva57gr
Copy link
Contributor

@eva57gr eva57gr commented Mar 17, 2026

No description provided.

Copy link
Collaborator

@Lightheartdevs Lightheartdevs left a comment

Choose a reason for hiding this comment

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

additionalProperties: false on the manifest schema will break all existing extension manifests that have undeclared fields. Either bump the schema version so old manifests aren't suddenly invalid, or add a migration script. The importlib-based service-registry sharing also adds fragile path inference at import time.

@Lightheartdevs
Copy link
Collaborator

The schema tightening is good work, but additionalProperties: false with many new required fields is a breaking change for existing manifests. Please:

  1. Audit all existing manifests in the repo to ensure they comply
  2. Add a migration guide or script
  3. Consider using additionalProperties: false only in a future version bump

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