Skip to content

(MacOS) Load config requirements from MDM#8743

Merged
gt-oai merged 1 commit intomainfrom
gt/mdm-requirements
Jan 5, 2026
Merged

(MacOS) Load config requirements from MDM#8743
gt-oai merged 1 commit intomainfrom
gt/mdm-requirements

Conversation

@gt-oai
Copy link
Contributor

@gt-oai gt-oai commented Jan 5, 2026 •
edited
Loading

Load managed requirements from MDM key requirements_toml_base64.

Tested on my Mac (using defaults to set the preference, though this would be set by MDM in production):

➜  codex git:(gt/mdm-requirements) defaults read com.openai.codex requirements_toml_base64 | base64 -d
allowed_approval_policies = ["on-request"]

➜  codex git:(gt/mdm-requirements) just c --yolo
cargo run --bin codex -- "$@"
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.26s
     Running `target/debug/codex --yolo`
Error loading configuration: value `Never` is not in the allowed set [OnRequest]
error: Recipe `codex` failed on line 11 with exit code 1

➜  codex git:(gt/mdm-requirements) defaults delete com.openai.codex requirements_toml_base64

➜  codex git:(gt/mdm-requirements) just c --yolo
cargo run --bin codex -- "$@"
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.24s
     Running `target/debug/codex --yolo`
╭──────────────────────────────────────────────────────────╮
│ >_ OpenAI Codex (v0.0.0)                                 │
│                                                          │
│ model:     codex-auto-balanced medium   /model to change │
│ directory: ~/code/codex/codex-rs                         │
╰──────────────────────────────────────────────────────────╯

  Tip: Start a fresh idea with /new; the previous session stays in history.

@gt-oai gt-oai requested a review from bolinfest January 5, 2026 15:22
@gt-oai gt-oai force-pushed the gt/mdm-requirements branch 3 times, most recently from 6226000 to 3843ce5 Compare January 5, 2026 16:22
bolinfest
bolinfest approved these changes Jan 5, 2026
View reviewed changes
Copy link
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

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

LGTM! Just some small things: I'll leave it up to you to decide whether you think they're worth addressing!

@gt-oai gt-oai force-pushed the gt/mdm-requirements branch from 3843ce5 to 454f77e Compare January 5, 2026 17:26
@gt-oai gt-oai enabled auto-merge (squash) January 5, 2026 17:32
@gt-oai gt-oai merged commit 1d8e2b4 into main Jan 5, 2026
26 checks passed
@gt-oai gt-oai deleted the gt/mdm-requirements branch January 5, 2026 17:41
@github-actions github-actions bot locked and limited conversation to collaborators Jan 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@bolinfest bolinfest bolinfest approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gt-oai @bolinfest