Skip to content

refactor: manifest imports#144

Open
MarioCadenas wants to merge 1 commit intomainfrom
refactor-manifest-imports
Open

refactor: manifest imports#144
MarioCadenas wants to merge 1 commit intomainfrom
refactor-manifest-imports

Conversation

@MarioCadenas
Copy link
Collaborator

@MarioCadenas MarioCadenas commented Feb 27, 2026

  • plugin manifests are now imported directly from manifest.json in plugins (analytics, genie, lakebase, server), and old runtime manifest.ts wrappers were removed.
  • Security hardening for CLI: plugin list, plugin sync, and plugin validate are now JSON-first by default (no JS manifest execution unless explicitly enabled with --allow-js-manifest), with warning messaging when JS opt-in is used.
  • New shared manifest resolver: added manifest-resolve utilities to centralize manifest discovery/loading and support optional JS manifest loading behind explicit opt-in.
  • Plugin CLI improvements: recursive plugin discovery expanded, better manifest resolution across commands, clearer error messages, and updated behavior for add-resource (still edits manifest.json only).
  • Scaffolding updates: plugin scaffold no longer generates manifest.ts; generated plugin code imports manifest.json directly.
  • Build/package updates: upgraded tsdown and adjusted build configs (including output extension/copy behavior), plus package metadata improvements (license, include LICENSE in files).
  • Tests/docs: expanded test coverage around manifest resolution and listing behavior, and updated plugin management docs to reflect secure defaults and JS-manifest opt-in.

@MarioCadenas MarioCadenas force-pushed the refactor-manifest-imports branch 9 times, most recently from 891ae6c to d8248f0 Compare March 4, 2026 17:36
@MarioCadenas MarioCadenas force-pushed the refactor-manifest-imports branch from d8248f0 to 111d428 Compare March 4, 2026 20:21
@MarioCadenas MarioCadenas marked this pull request as ready for review March 4, 2026 20:34
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.

1 participant