Skip to content

Add configurable default plugin activation policy - #52220

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/83b70c5b47fc95474fec292f248683f1f887221e
Oct 8, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/83b70c5b47fc95474fec292f248683f1f887221e

Conversation

@copyberry

@copyberry copyberry Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Add configurable default plugin activation policy

What changed

Support plugins._default.enabled with explicit per-plugin enabled overrides. Omission preserves existing source and account enablement; configuration cannot activate a plugin disabled at its source. Plugin settings such as remembered tool approvals inherit the default unless enabled is explicitly set.

Apply the policy to plugin skills, MCP servers, Apps, and hooks, including selected executor roots and runtime configuration refreshes. Preserve the latest plugin policy when adding MCP dependencies and rebuild hooks when concurrent configuration updates replace their inputs.

Report effective enablement in plugin/list, plugin/installed, plugin/read, and plugin/share/list. Remote installation still installs disabled packages, but skips MCP and Apps authentication setup without writing an enablement override.

Testing

Add regression coverage for default inheritance and explicit overrides, source-disabled plugins, policy refreshes during active turns, disabled or malformed selected plugin roots, and remote installation authentication behavior.

## What changed

Support `plugins._default.enabled` with explicit per-plugin `enabled` overrides. Omission preserves existing source and account enablement; configuration cannot activate a plugin disabled at its source. Plugin settings such as remembered tool approvals inherit the default unless `enabled` is explicitly set.

Apply the policy to plugin skills, MCP servers, Apps, and hooks, including selected executor roots and runtime configuration refreshes. Preserve the latest plugin policy when adding MCP dependencies and rebuild hooks when concurrent configuration updates replace their inputs.

Report effective enablement in `plugin/list`, `plugin/installed`, `plugin/read`, and `plugin/share/list`. Remote installation still installs disabled packages, but skips MCP and Apps authentication setup without writing an enablement override.

## Testing

Add regression coverage for default inheritance and explicit overrides, source-disabled plugins, policy refreshes during active turns, disabled or malformed selected plugin roots, and remote installation authentication behavior.

GitOrigin-RevId: 83b70c5b47fc95474fec292f248683f1f887221e
@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

Note

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/83b70c5b47fc95474fec292f248683f1f887221e branch from bfdfead to 89d68bc Compare October 8, 2026 18:43
@copyberry
copyberry Bot merged commit 89d68bc into main Oct 8, 2026
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/83b70c5b47fc95474fec292f248683f1f887221e branch October 8, 2026 18:43
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 8, 2026
@Kamber88
Kamber88 deployed to issue-triage October 8, 2026 18:51 — with Image GitHub Actions Active
@Kamber88
Kamber88 deployed to issue-triage October 8, 2026 18:51 — with Image GitHub Actions Active
@Kamber88
Kamber88 deployed to issue-triage October 8, 2026 18:51 — with Image GitHub Actions Active

This branch was successfully deployed

1 active deployment
issue-triage — 89d68bc8 Deployed Oct 8, 2026 by Kamber88 via Generate label suggestions #53275
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.

3 participants