Skip to content

fix(manifest): use canonical CLI flag field#35

Merged
intel352 merged 1 commit into
mainfrom
codex/canonical-plugin-manifest
Jun 26, 2026
Merged

fix(manifest): use canonical CLI flag field#35
intel352 merged 1 commit into
mainfrom
codex/canonical-plugin-manifest

Conversation

@intel352

Copy link
Copy Markdown
Contributor

Summary

  • replace legacy flags_passthrough with canonical flagsPassthrough in plugin.json
  • prevents registry sync from reimporting unsupported manifest spelling for both migrations registry entries

Validation

  • GOWORK=off go test ./...
  • jq empty plugin.json

Copilot AI review requested due to automatic review settings June 26, 2026 20:40

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.

Pull request overview

Updates the plugin manifest to use the canonical CLI flag passthrough field name, avoiding registry sync re-importing an unsupported legacy spelling for the migrations registry entries.

Changes:

  • Replaced legacy flags_passthrough with canonical flagsPassthrough in plugin.json.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@intel352
intel352 merged commit f81cb16 into main Jun 26, 2026
6 checks passed
@intel352
intel352 deleted the codex/canonical-plugin-manifest branch June 26, 2026 22:13
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