Magnus/browser use rust main integration

crates/browser-use-agent/src/config_overrides.rs

@@ -733,41 +733,57 @@ pub fn apply_child_request_runtime_config(
     config: &mut ProviderRunConfig,
     request: &ChildAgentRunRequest,
 ) -> Result<()> {
-    let overrides = &request.config_overrides;
+    apply_runtime_config_overrides(&mut config.options, &request.config_overrides)
+}
+
+/// Apply config keys that mutate in-memory runtime options.
+///
+/// The raw override list is still retained for downstream consumers that read
+/// less common config keys directly, but options that are consulted before those
+/// consumers run must be materialized here.
+pub fn apply_runtime_config_overrides(
+    options: &mut AgentRunOptions,
+    overrides: &ConfigOverrides,
+) -> Result<()> {
+    if let Some(value) = config_override_u64(overrides, "max_turns") {
+        options.max_turns = usize::try_from(value)
+            .context("max_turns does not fit in usize")?
+            .max(1);
+    }
     if let Some(value) = config_override_str(overrides, "browser_mode") {
-        config.options.browser_mode = Some(value);
+        options.browser_mode = Some(value);
     }
     if let Some(value) = config_override_str(overrides, "base_instructions") {
-        config.options.base_instructions = Some(value);
+        options.base_instructions = Some(value);
     }
     if let Some(value) = config_override_str(overrides, "developer_instructions") {
-        config.options.developer_instructions = Some(value);
+        options.developer_instructions = Some(value);
     }
     if let Some(value) = config_override_str(overrides, "compact_prompt") {
-        config.options.compact_prompt = Some(value);
+        options.compact_prompt = Some(value);
     }
     if let Some(value) = config_override_u64(overrides, "python_tool_timeout_seconds") {
-        config.options.python_tool_timeout_seconds = value;
+        options.python_tool_timeout_seconds = value;
     }
     if let Some(value) = config_override_bool(overrides, "model_compaction_enabled") {
-        config.options.model_compaction_enabled = value;
+        options.model_compaction_enabled = value;
     }
     if let Some(value) = config_override_i64(overrides, "model_auto_compact_token_limit") {
-        config.options.model_auto_compact_token_limit = Some(value);
+        options.model_auto_compact_token_limit = Some(value);
     }
     if let Some(value) = config_override_str(overrides, "model_auto_compact_token_limit_scope") {
-        config.options.model_auto_compact_token_limit_scope =
+        options.model_auto_compact_token_limit_scope =
             parse_auto_compact_token_limit_scope(&value)?;
     }
     if let Some(value) = config_override_str(overrides, "approval_policy")
         .or_else(|| config_override_str(overrides, "ask_for_approval"))
     {
-        config.options.approval_policy = parse_approval_policy(&value)?;
+        options.approval_policy = parse_approval_policy(&value)?;
     }
     if let Some(value) = config_override_bool(overrides, "use_guardian")
         .or_else(|| config_override_bool(overrides, "guardian"))
     {
-        config.options.use_guardian = value;
+        options.use_guardian = value;
     }
     Ok(())
 }
@@ -1816,6 +1832,25 @@ command = "profile-server"
         assert!(options.agent_roles.is_empty());
     }
 
+    #[test]
+    fn runtime_config_overrides_materialize_max_turns_and_browser_mode() {
+        let overrides = parse_config_overrides(&ov(&[
+            "max_turns=100",
+            "browser_mode=\"remote-cdp\"",
+            "python_tool_timeout_seconds=45",
+            "model_compaction_enabled=false",
+        ]))
+        .unwrap();
+        let mut options = AgentRunOptions::default();
+
+        apply_runtime_config_overrides(&mut options, &overrides).unwrap();
+
+        assert_eq!(options.max_turns, 100);
+        assert_eq!(options.browser_mode.as_deref(), Some("remote-cdp"));
+        assert_eq!(options.python_tool_timeout_seconds, 45);
+        assert!(!options.model_compaction_enabled);
+    }
+
     #[test]
     fn provider_run_config_new_uses_explicit_source_and_default_options() {
         let config = ProviderRunConfig::new(ProviderBackend::Anthropic, "claude-x");

crates/browser-use-agent/src/context/tests_accounting.rs

@@ -79,6 +79,7 @@ fn from_llm_usage_uses_server_total_when_present() {
     let u = Usage {
         input_tokens: 100,
         cached_input_tokens: 40,
+        cache_creation_input_tokens: 0,
         output_tokens: 30,
         reasoning_output_tokens: 10,
         total_tokens: 123,
@@ -96,6 +97,7 @@ fn from_llm_usage_total_fallback_excludes_cached() {
     let u = Usage {
         input_tokens: 100,
         cached_input_tokens: 40,
+        cache_creation_input_tokens: 0,
         output_tokens: 30,
         reasoning_output_tokens: 10,
         total_tokens: 0,