From 1383844a6663e1ae5505ec10ebf2d2f0996a9091 Mon Sep 17 00:00:00 2001 From: orveth Date: Tue, 25 Aug 2026 12:24:45 -0700 Subject: [PATCH] fix: decide the Codex session value for the doctor engine-floor test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `a2b9403f` added `codex_chatgpt` to `SandboxConfig` and updated one of the two longhand initializers in the workspace. This is the other one, in a different crate, so `cargo test -p maxplayer` does not compile at main. The value follows the answer already recorded three lines above it for `file_credentials`: this check asserts the engine-version floor, and a host ChatGPT session is a containment concern that would only add a second reason for the check to move. Its only assertion is that a check named "sandbox engine floor" runs, so no field value here is observable by it. The reasoning goes in beside the field because the tripwire cannot tell a decision from a compile-fix — both produce identical bytes, and the note is the only artifact that survives to distinguish them. Co-Authored-By: Claude Opus 5 (1M context) --- crates/maxplayer/src/doctor.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/crates/maxplayer/src/doctor.rs b/crates/maxplayer/src/doctor.rs index 6212e3be4..a74652e3f 100644 --- a/crates/maxplayer/src/doctor.rs +++ b/crates/maxplayer/src/doctor.rs @@ -2808,6 +2808,9 @@ mod tests { // and a file-sourced credential is a containment concern that would only add a second // reason for the check to move. file_credentials: Vec::new(), + // Same decision and the same reason: a host ChatGPT session is a containment concern, + // and reading one here would give the check a second reason to move. + codex_chatgpt: None, }); home.config.relay_url = "not-a-relay-url".into(); home.config.accepted_mints = Vec::new();