Skip to content

Commit 2f13043

Browse files
guyernestclaude
andcommitted
fix(browser-agent): Handle None values for llm_provider and llm_model
When command has explicit null values for llm_provider or llm_model, the executor would crash with AttributeError on NoneType.lower(). Now uses defaults ("openai" and "gpt-4o-mini") when None is passed. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent abe609e commit 2f13043

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

lambda/tools/local-browser-agent/python/openai_playwright_executor.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,8 +128,9 @@ def __init__(
128128
user_data_dir: Optional[Path] = None,
129129
navigation_timeout: int = 60000,
130130
):
131-
self.llm_provider = llm_provider.lower()
132-
self.llm_model = llm_model
131+
# Handle None values with defaults
132+
self.llm_provider = (llm_provider or "openai").lower()
133+
self.llm_model = llm_model or "gpt-4o-mini"
133134
self.llm_api_key = llm_api_key
134135
self.s3_bucket = s3_bucket
135136
self.aws_profile = aws_profile

lambda/tools/local-browser-agent/src-tauri/tauri.conf.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
},
99
"package": {
1010
"productName": "Local Browser Agent",
11-
"version": "0.4.23"
11+
"version": "0.4.24"
1212
},
1313
"tauri": {
1414
"allowlist": {

0 commit comments

Comments
 (0)