backend-client: add workspace attribution settings transport#31994
Draft
jayp-oai wants to merge 1 commit into
Draft
backend-client: add workspace attribution settings transport#31994jayp-oai wants to merge 1 commit into
jayp-oai wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
GET settings/usercall tocodex-backend-client/api/codex/settings/useror ChatGPT's/wham/settings/userChatGPT-Account-IDand FedRAMP routing follow the current CLI accountcommit_attribution_enabledfield tofalsefor safe compatibility with pre-BIZG-546 serversCache-Control: no-storeWhy
BIZG-549 requires Codex CLI sessions to consume the effective workspace attribution policy computed by BIZG-546. The CLI should not read
/accounts/{id}/settings, infer eligibility from local plan claims, or add an enterprise policy to TOML. The existing backend client is the narrow shared transport that already owns Codex API versus/whamrouting and authenticated workspace headers.Scope and dependencies
This draft is an independently mergeable foundation directly on
main; it is not stacked on BIZG-548 and contains no prompt-attribution logic.BIZG-548 is not yet present on
main, so the remaining CLI integration is deliberately not duplicated here. After BIZG-546 and BIZG-548 are available, the follow-up in this PR can fetch these settings from the current auth snapshot at thread bootstrap, treat timeout/fetch/decode failures as disabled without blocking startup, and pass only the resolved gate into the shared contributor. End-to-end enabled/disabled, personal/ineligible, offline, workspace-switch, app-server, and MCP-host coverage depends on that consumer seam.Ticket: BIZG-549
Validation
just fmtjust test -p codex-backend-client— 19 passedjust fix -p codex-backend-clientorigin/maindiff — clean, no accepted/actionable findings