Skip to content

[codex] Load API curated marketplace by auth#28383

Merged
felixxia-oai merged 11 commits into
mainfrom
codex/api-curated-marketplace-selection
Jun 16, 2026
Merged

[codex] Load API curated marketplace by auth#28383
felixxia-oai merged 11 commits into
mainfrom
codex/api-curated-marketplace-selection

Conversation

@felixxia-oai

@felixxia-oai felixxia-oai commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • choose the local OpenAI curated marketplace manifest based on auth: Codex backend auth gets the existing marketplace, direct provider auth gets api_marketplace.json
  • include Bedrock API key auth in the direct-provider API marketplace path
  • safely skip the API marketplace when api_marketplace.json is absent

Validation

  • just fmt
  • git diff --check origin/main...HEAD
  • CI should run the full validation

Manual Testing

- New api marketplace not available for API key sign

  1. Safely not display anything from api marketplace
Screenshot 2026-06-15 at 21 37 43

- New api marketplace for API key sign in

  1. Setup api_marketplace.json
{
  "name": "openai-curated",
  "interface": {
    "displayName": "Codex official"
  },
  "plugins": [
    {
      "name": "linear",
      "source": {
        "source": "local",
        "path": "./plugins/linear"
      },
      "policy": {
        "installation": "AVAILABLE",
        "authentication": "ON_INSTALL"
      },
      "category": "Productivity"
    }
  ]
}
  1. Log in with API key, observe that only the defined plugin from api_marketplace.json is available from "Codex Official" (outside of local testing marketplaces)
Screenshot 2026-06-15 at 21 16 53

also checked functionality on codex app

- SiWC users

Still uses 'default' marketplace.json and renders all plugins
Screenshot 2026-06-15 at 21 40 25

also checked functionality on codex app

Notes

  • just test -p codex-core-plugins was started locally before splitting branches, but I stopped relying on local tests per follow-up and left final validation to PR CI.

@felixxia-oai
felixxia-oai force-pushed the codex/api-curated-marketplace-selection branch 5 times, most recently from 61792b7 to 72f1d45 Compare June 15, 2026 20:06
@felixxia-oai

Copy link
Copy Markdown
Contributor Author

@codex review

@felixxia-oai
felixxia-oai force-pushed the codex/api-curated-marketplace-selection branch from 72f1d45 to 637bfec Compare June 15, 2026 20:11

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 72f1d45f20

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread codex-rs/core-plugins/src/manager.rs
Comment thread codex-rs/cli/src/plugin_cmd.rs Outdated
Comment thread codex-rs/core-plugins/src/manager.rs Outdated
@felixxia-oai
felixxia-oai force-pushed the codex/api-curated-marketplace-selection branch 2 times, most recently from 404a105 to d652860 Compare June 15, 2026 20:38
@felixxia-oai
felixxia-oai requested a review from xl-openai June 15, 2026 20:40
@felixxia-oai
felixxia-oai force-pushed the codex/api-curated-marketplace-selection branch 2 times, most recently from 5b2fcdc to f6ed83a Compare June 15, 2026 20:53
@felixxia-oai

Copy link
Copy Markdown
Contributor Author

@codex review again

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f6ed83a970

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread codex-rs/core-plugins/src/discoverable.rs Outdated
Comment thread codex-rs/core-plugins/src/manager.rs Outdated
Comment thread codex-rs/core-plugins/src/loader.rs Outdated
@felixxia-oai

Copy link
Copy Markdown
Contributor Author

@codex review again

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8555ad01dd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread codex-rs/cli/src/plugin_cmd.rs Outdated
Comment thread codex-rs/core-plugins/src/manager.rs
@felixxia-oai
felixxia-oai force-pushed the codex/api-curated-marketplace-selection branch from 20f8629 to 2faee43 Compare June 15, 2026 21:43
@felixxia-oai

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: 2faee43f85

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@felixxia-oai
felixxia-oai marked this pull request as ready for review June 15, 2026 22:00
@felixxia-oai
felixxia-oai requested a review from a team as a code owner June 15, 2026 22:00

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2faee43f85

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread codex-rs/core-plugins/src/manager.rs Outdated
@felixxia-oai
felixxia-oai force-pushed the codex/api-curated-marketplace-selection branch from 86aaba3 to e46e599 Compare June 15, 2026 22:31
Comment thread codex-rs/core-plugins/src/lib.rs
@felixxia-oai
felixxia-oai force-pushed the codex/api-curated-marketplace-selection branch from 5db9788 to e444987 Compare June 16, 2026 00:45
@felixxia-oai
felixxia-oai enabled auto-merge (squash) June 16, 2026 01:12
@felixxia-oai
felixxia-oai merged commit 02dce8e into main Jun 16, 2026
31 checks passed
@felixxia-oai
felixxia-oai deleted the codex/api-curated-marketplace-selection branch June 16, 2026 01:16
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 16, 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