Skip to content

feat: prepare managed controls for skills and marketplaces#21462

Open
viyatb-oai wants to merge 2 commits intomainfrom
codex/viyatb/managed-artifact-exposure
Open

feat: prepare managed controls for skills and marketplaces#21462
viyatb-oai wants to merge 2 commits intomainfrom
codex/viyatb/managed-artifact-exposure

Conversation

@viyatb-oai
Copy link
Copy Markdown
Collaborator

@viyatb-oai viyatb-oai commented May 7, 2026

Why

The enforcement PRs need a shared in-memory representation for managed skill-source and plugin-marketplace policy before the public requirements.toml surface is activated. Keeping that plumbing separate lets the later activation PR land only after the behavior exists.

This PR is intentionally internal-only: it does not yet make any new managed config keys parseable from requirements.toml.

What changed

  • Added the normalized ConfigRequirements fields for skill-source and plugin-marketplace policy.
  • Added shared requirement value types for skill sources and plugin marketplace allowlists.
  • Exported those shared types for the enforcement layers that follow in the stack.

Example config.toml

Once the later activation PR is present, an approved marketplace can still appear in user config.toml like this:

[features]
plugins = true

[marketplaces.approved-marketplace]
source_type = "git"
source = "https://github.com/example/approved-marketplace.git"

There is intentionally no user-writable allowed_sources key in config.toml; skill-source restrictions are enforced only from managed requirements.

Verification

  • This PR only adds the internal model consumed by the enforcement PRs above it.

Stack

--> #21462: internal managed artifact requirement plumbing

Co-authored-by: Codex noreply@openai.com
@viyatb-oai viyatb-oai changed the base branch from codex/viyatb/managed-artifact-requirements to main May 7, 2026 02:20
@viyatb-oai viyatb-oai requested a review from a team as a code owner May 7, 2026 02:20
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/managed-artifact-exposure branch from 58cde56 to 45c47d2 Compare May 7, 2026 02:20
@viyatb-oai viyatb-oai changed the title feat: expose managed artifact requirements feat: add managed artifact requirement plumbing May 7, 2026
@viyatb-oai viyatb-oai changed the title feat: add managed artifact requirement plumbing feat: prepare managed controls for skills and marketplaces May 7, 2026
Co-authored-by: Codex noreply@openai.com
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