Skip to content

Commit e953092

Browse files
Fixed regression in experimental "sandbox command assessment" feature (openai#7308)
Recent model updates caused the experimental "sandbox tool assessment" to time out most of the time leaving the user without any risk assessment or tool summary. This change explicitly sets the reasoning effort to medium and bumps the timeout. This change has no effect if the user hasn't enabled the `experimental_sandbox_command_assessment` feature flag.
1 parent 28ff364 commit e953092

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

codex-rs/core/src/sandboxing/assessment.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ use crate::protocol::SandboxPolicy;
1414
use askama::Template;
1515
use codex_otel::otel_event_manager::OtelEventManager;
1616
use codex_protocol::ConversationId;
17+
use codex_protocol::config_types::ReasoningEffort as ReasoningEffortConfig;
1718
use codex_protocol::models::ContentItem;
1819
use codex_protocol::models::ResponseItem;
1920
use codex_protocol::protocol::SandboxCommandAssessment;
@@ -23,7 +24,8 @@ use serde_json::json;
2324
use tokio::time::timeout;
2425
use tracing::warn;
2526

26-
const SANDBOX_ASSESSMENT_TIMEOUT: Duration = Duration::from_secs(5);
27+
const SANDBOX_ASSESSMENT_TIMEOUT: Duration = Duration::from_secs(15);
28+
const SANDBOX_ASSESSMENT_REASONING_EFFORT: ReasoningEffortConfig = ReasoningEffortConfig::Medium;
2729

2830
#[derive(Template)]
2931
#[template(path = "sandboxing/assessment_prompt.md", escape = "none")]
@@ -130,7 +132,7 @@ pub(crate) async fn assess_command(
130132
Some(auth_manager),
131133
child_otel,
132134
provider,
133-
config.model_reasoning_effort,
135+
Some(SANDBOX_ASSESSMENT_REASONING_EFFORT),
134136
config.model_reasoning_summary,
135137
conversation_id,
136138
session_source,

0 commit comments

Comments
 (0)