Skip to content

fix: declare ON_USE rather than ON_INSTALL for the Codex plugin policy - #38

Merged
ziyiunity merged 1 commit into
mainfrom
fix/codex-authentication-policy
Sep 9, 2026
Merged

fix: declare ON_USE rather than ON_INSTALL for the Codex plugin policy#38
ziyiunity merged 1 commit into
mainfrom
fix/codex-authentication-policy

Conversation

@ziyiunity

Copy link
Copy Markdown
Collaborator

What

One line in .agents/plugins/marketplace.json:

-"authentication": "ON_INSTALL"
+"authentication": "ON_USE"

Why

This plugin ships skills only. No MCP server, no API, nothing to sign in to. ON_INSTALL is what a service connector declares, and it risks Codex asking users for credentials we never take.

Both values are in production use, so this is not a spec question but a semantic one, and the closest analogues settle it. OpenAI's own skills-only local plugins all declare ON_USE: documents, pdf, spreadsheets, presentations, template-creator. ON_INSTALL is what github, slack, stripe, notion and figma declare. Across the plugins visible in a local Codex install, 21 declare ON_USE and none of them connect to an external service the way ours does not.

The published spec names only ON_INSTALL and does not cover the no-auth case, which is why this was left alone in #37 pending evidence rather than guessed at.

Scope

One line. Nothing else in the manifest changes.

This plugin ships skills only: no MCP server, no API, nothing to sign in
to. ON_INSTALL is what service connectors declare, and it risks Codex
asking users for credentials we never take.

Both values are in production use, so this is not a spec question but a
semantic one, and the closest analogues settle it: OpenAI's own
skills-only local plugins -- documents, pdf, spreadsheets, presentations,
template-creator -- all declare ON_USE, while ON_INSTALL is what github,
slack, stripe, notion and figma use. 21 shipped plugins declare ON_USE.
@ziyiunity
ziyiunity requested a review from a team as a code owner September 9, 2026 15:24
@ziyiunity
ziyiunity merged commit da46d37 into main Sep 9, 2026
@ziyiunity
ziyiunity deleted the fix/codex-authentication-policy branch September 9, 2026 15:24
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