Skip to content

[codex] add remote plugin HTTP state adapter#26105

Closed
cooper-oai wants to merge 1 commit into
cooper/codex-native-integrity/13from
cooper/codex-native-integrity/14
Closed

[codex] add remote plugin HTTP state adapter#26105
cooper-oai wants to merge 1 commit into
cooper/codex-native-integrity/13from
cooper/codex-native-integrity/14

Conversation

@cooper-oai

@cooper-oai cooper-oai commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add opt-in HTTP-state context to remote plugin transport config
  • attach and rotate state for first-party catalog, sharing, and legacy plugin requests
  • keep issued blob and bundle URLs plus ownerless startup work unattributed

Stack

PR 14 of 15. Depends on #26100. Followed by the final app-server plugin RPC attribution layer.

Validation

  • CARGO_INCREMENTAL=0 just test -p codex-core-plugins
  • CARGO_INCREMENTAL=0 just fix -p codex-core-plugins
  • CARGO_INCREMENTAL=0 just fix -p codex-app-server
  • just bazel-lock-check
  • git diff --check

Local limitation

  • just argument-comment-lint -p codex-core-plugins -p codex-app-server cannot start because the pinned nightly-2025-09-18 toolchain is older than the repository's sqlx 0.9.0 MSRV.

@cooper-oai cooper-oai marked this pull request as ready for review June 3, 2026 08:26
@cooper-oai

Copy link
Copy Markdown
Contributor Author

@codex review

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

ℹ️ 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/remote.rs
Comment thread codex-rs/core-plugins/src/remote/http_state.rs Outdated
Comment thread codex-rs/core-plugins/Cargo.toml
Comment thread codex-rs/core-plugins/src/remote/http_state.rs Outdated
@cooper-oai cooper-oai force-pushed the cooper/codex-native-integrity/13 branch from d5f5fd9 to de0feed Compare June 3, 2026 08:46
@cooper-oai cooper-oai requested a review from a team as a code owner June 3, 2026 08:46
@cooper-oai cooper-oai force-pushed the cooper/codex-native-integrity/14 branch from d1603d8 to fbb6c5d Compare June 3, 2026 08:47
@cooper-oai cooper-oai force-pushed the cooper/codex-native-integrity/13 branch from de0feed to b80025f Compare June 3, 2026 09:17
@cooper-oai cooper-oai force-pushed the cooper/codex-native-integrity/14 branch from fbb6c5d to 6a0405b Compare June 3, 2026 09:33
@cooper-oai

Copy link
Copy Markdown
Contributor Author

@codex review

@cooper-oai cooper-oai force-pushed the cooper/codex-native-integrity/13 branch from b80025f to fb9d914 Compare June 3, 2026 09:54
@cooper-oai cooper-oai force-pushed the cooper/codex-native-integrity/14 branch from 6a0405b to b766a57 Compare June 3, 2026 09:54
@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

Codex Review: Didn't find any major issues. Another round soon, please!

ℹ️ 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".

@cooper-oai

Copy link
Copy Markdown
Contributor Author

Superseded by the collapsed seven-PR stack: #26137 through #26143. Closing this prototype layer to keep review focused.

@cooper-oai cooper-oai closed this Jun 3, 2026
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