Skip to content

Add strict known marketplace requirements#19857

Closed
abhinav-oai wants to merge 1 commit into
mainfrom
abhinav/strict-known-marketplaces
Closed

Add strict known marketplace requirements#19857
abhinav-oai wants to merge 1 commit into
mainfrom
abhinav/strict-known-marketplaces

Conversation

@abhinav-oai
Copy link
Copy Markdown
Collaborator

@abhinav-oai abhinav-oai commented Apr 27, 2026

Why

Enterprises need a managed way to restrict user-added plugin marketplaces to approved sources while keeping Codex's built-in catalogs available. This gives admins a source-based allowlist for marketplace installs without turning curated or backend catalogs into managed-config surface area.

What

  • add a strict_known_marketplaces managed requirement with exact matching over source_type, normalized source, ref, and sparse_paths
  • enforce the allowlist for configured user marketplaces when adding, discovering, installing from, and upgrading them
  • keep openai-curated plus backend-provided catalogs intentionally outside this requirement, and document that boundary inline
  • expose the requirement through configRequirements/read and regenerate the app-server schema/TypeScript fixtures

Verification

  • cargo test -p codex-config deserialize_strict_known_marketplaces
  • cargo test -p codex-core-plugins add_marketplace_sync_rejects_git_source_blocked_by_requirements_before_clone
  • cargo test -p codex-core install_plugin_rejects_disallowed_configured_marketplace
  • cargo test -p codex-app-server-protocol config_requirements
  • cargo test -p codex-core-plugins -p codex-hooks

@abhinav-oai abhinav-oai changed the base branch from main to dev/abhinav/plugin-hooks-discovery April 27, 2026 18:36
@abhinav-oai abhinav-oai changed the base branch from dev/abhinav/plugin-hooks-discovery to main April 27, 2026 18:36
@abhinav-oai abhinav-oai force-pushed the abhinav/strict-known-marketplaces branch from 97c4b67 to e441128 Compare April 27, 2026 18:40
@abhinav-oai abhinav-oai deleted the abhinav/strict-known-marketplaces branch April 27, 2026 18:44
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