chore: save Codex config seed - #3834
Conversation
Reviewer's GuideAdds a tracked Codex configuration seed file capturing safe, non-secret default preferences to be merged into the live ~/.codex/config.toml without tracking generated state or credentials. Flow diagram for using the Codex config seed to populate live configflowchart LR
RepoSeed["wsl/home/.codex/config.seed.toml\n(safe, tracked preferences)"] --> MergeStep["merge/install step\n(future, #3833)"]
LiveConfig["~/.codex/config.toml\n(untracked, includes generated state & secrets)"]
MergeStep --> LiveConfig
subgraph SeedContents["Key safe preferences in config.seed.toml"]
Reasoning["model_reasoning_effort = high\nplan_mode_reasoning_effort = xhigh"]
Sandbox["sandbox_mode = danger-full-access\napproval_policy = never"]
ServiceTier["service_tier = default"]
TUI["[tui].status_line"]
Idle["[features].prevent_idle_sleep = true"]
CloudflareMCP["[mcp_servers.cloudflare-docs]\n[mcp_servers.cloudflare-api]"]
end
Reasoning --> RepoSeed
Sandbox --> RepoSeed
ServiceTier --> RepoSeed
TUI --> RepoSeed
Idle --> RepoSeed
CloudflareMCP --> RepoSeed
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="wsl/home/.codex/config.seed.toml" line_range="7-8" />
<code_context>
+model_reasoning_effort = "high"
+plan_mode_reasoning_effort = "xhigh"
+
+sandbox_mode = "danger-full-access"
+approval_policy = "never"
+
+# Explicitly use standard speed instead of the Fast/priority service tier.
</code_context>
<issue_to_address>
**π¨ issue (security):** Reconsider combining `danger-full-access` sandbox with `approval_policy = "never"` for safety-critical operations.
This setup gives the model unrestricted, fully automated access to the host without any human checks, greatly amplifying the impact of prompt injection, mis-specification, or tool bugs. If you need full-access for power users, consider keeping a stricter `approval_policy` (or using a separate high-trust profile) so that clearly sensitive or dangerous operations still require explicit confirmation.
</issue_to_address>Help me be more useful! Please click π or π on each comment and I'll use the feedback to improve your reviews.
| sandbox_mode = "danger-full-access" | ||
| approval_policy = "never" |
There was a problem hiding this comment.
π¨ issue (security): Reconsider combining danger-full-access sandbox with approval_policy = "never" for safety-critical operations.
This setup gives the model unrestricted, fully automated access to the host without any human checks, greatly amplifying the impact of prompt injection, mis-specification, or tool bugs. If you need full-access for power users, consider keeping a stricter approval_policy (or using a separate high-trust profile) so that clearly sensitive or dangerous operations still require explicit confirmation.
Summary
wsl/home/.codex/config.seed.tomlcontaining the current safe, non-default Codex preferences~/.codex/config.tomluntracked so generated state and literal OTEL credentials cannot enter the repositoryWhy
Codex currently stores durable preferences alongside project trust, hook hashes, UI state, connector state, and OTEL credentials. Tracking or symlinking the live file would both dirty the repository and risk committing secrets.
This seed is the safe canonical input for a future merge/install step tracked in #3833.
Validation
tombi lint wsl/home/.codex/config.seed.tomltombi format --check wsl/home/.codex/config.seed.tomlbetterleaksRelated to #3833.
Summary by Sourcery
Enhancements: