Skip to content

exec-server: expose environment registry payloads#28651

Merged
apanasenko-oai merged 2 commits into
mainfrom
codex/viyatb/export-environment-registry-types
Jun 17, 2026
Merged

exec-server: expose environment registry payloads#28651
apanasenko-oai merged 2 commits into
mainfrom
codex/viyatb/export-environment-registry-types

Conversation

@viyatb-oai

@viyatb-oai viyatb-oai commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Why

Services that proxy the exec-server environment registry endpoints need to deserialize and forward the same Noise registration and harness-key validation payloads. Those wire models currently live as private, serialize-only structs in exec-server, which forces consumers to duplicate the contract.

What changed

  • Add owned serde models for registration and harness-key validation requests and responses.
  • Use those models in the existing exec-server registry client.
  • Re-export the models from codex-exec-server and codex-core-api.
  • Keep the harness authorization request free of a derived Debug implementation so it is not accidentally logged.

Testing

  • Focused exec-server registration and harness-key validation tests: 2 passed.
  • cargo check -p codex-core-api

The full codex-exec-server suite compiled and ran 254 tests: 222 passed, while 32 existing filesystem sandbox tests could not run under the nested macOS sandbox (sandbox_apply: Operation not permitted).

Co-authored-by: Codex <noreply@openai.com>
@apanasenko-oai
apanasenko-oai merged commit a0586ad into main Jun 17, 2026
31 checks passed
@apanasenko-oai
apanasenko-oai deleted the codex/viyatb/export-environment-registry-types branch June 17, 2026 20:27
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 17, 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