Skip to content

Preserve plugin app manifest order#25491

Merged
charlesgong-openai merged 1 commit into
mainfrom
dec/charlesgong/plugin-app-order
Jun 1, 2026
Merged

Preserve plugin app manifest order#25491
charlesgong-openai merged 1 commit into
mainfrom
dec/charlesgong/plugin-app-order

Conversation

@charlesgong-openai
Copy link
Copy Markdown
Contributor

Summary

  • Preserve app declaration order when loading plugin .app.json files.
  • Keep plugin connector summaries in plugin app order after connector metadata is merged and filtered.
  • Add regression coverage for .app.json order and connector summary order.

Validation

  • just fmt
  • just test -p codex-chatgpt connectors_for_plugin_apps_returns_only_requested_plugin_apps
  • just test -p codex-core-plugins effective_apps_preserves_app_config_order
  • just fix -p codex-core-plugins (passes with existing clippy large_enum_variant warning in core-plugins/src/manifest.rs)
  • just fix -p codex-chatgpt
  • just bazel-lock-update
  • just bazel-lock-check

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@charlesgong-openai charlesgong-openai marked this pull request as ready for review June 1, 2026 06:18
@charlesgong-openai
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Jun 1, 2026
@charlesgong-openai charlesgong-openai merged commit 9756316 into main Jun 1, 2026
31 of 32 checks passed
@charlesgong-openai charlesgong-openai deleted the dec/charlesgong/plugin-app-order branch June 1, 2026 18:04
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants