Skip to content

[codex] Skip curated repo sync for remote plugins#29005

Merged
xl-openai merged 1 commit into
mainfrom
xl/skip-curated-sync-for-remote-plugins
Jun 19, 2026
Merged

[codex] Skip curated repo sync for remote plugins#29005
xl-openai merged 1 commit into
mainfrom
xl/skip-curated-sync-for-remote-plugins

Conversation

@xl-openai

@xl-openai xl-openai commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • skip the legacy openai-curated startup repository sync when remote plugins are enabled and the current auth uses the Codex backend
  • keep the curated sync for API-key, Bedrock, and unauthenticated sessions that fall back to the local marketplace
  • preserve configured marketplace upgrades and all remote plugin startup warmups

Why

The remote catalog owns plugin discovery and materialization only when it is usable for the current auth mode. Starting the legacy curated repository sync in that case performs an unnecessary Git/HTTP/archive download and cache refresh. API-key and Bedrock sessions still require the local curated marketplace, so they must continue syncing it.

User impact

Codex startup no longer downloads or refreshes the local openai-curated snapshot when the remote catalog is active. Behavior is unchanged for auth modes that use the local curated marketplace.

Validation

  • just fmt
  • git diff --check

Rust tests were not run per the repository's local verification policy for this narrow conditional change.

@xl-openai
xl-openai marked this pull request as ready for review June 18, 2026 23:56

@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: f4fa8c2e24

ℹ️ 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
@xl-openai
xl-openai force-pushed the xl/skip-curated-sync-for-remote-plugins branch from f4fa8c2 to 76f8171 Compare June 19, 2026 00:37
@xl-openai

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: 76f81713b2

ℹ️ 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
@xl-openai
xl-openai merged commit 38c9686 into main Jun 19, 2026
45 of 47 checks passed
@xl-openai
xl-openai deleted the xl/skip-curated-sync-for-remote-plugins branch June 19, 2026 01:02
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 19, 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