**2 issues found** across 12 files Confidence score: **4/5** - This PR looks safe to merge overall, with risk concentrated in test stability rather than production behavior. - The most significant issue is in `tests/test_client.py`: not clearing `ANTHROPIC_API_KEY` can cause `_resolve_model_api_key` to pick the wrong key based on runner environment, leading to non-deterministic test outcomes. - A similar environment-leak risk exists in `tests/test_sea_binary.py`, where missing `monkeypatch.delenv("STAGEHAND_SEA_BINARY", raising=False)` can bypass the intended monkeypatched path resolution and make the test depend on external env state. - Pay close attention to `tests/test_client.py`, `tests/test_sea_binary.py` - ensure env vars are explicitly cleared so tests remain deterministic across different runners.

Prompt for AI agents (unresolved issues)

```text Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately. P2: This test doesn't omit `ANTHROPIC_API_KEY`, which has higher priority than `GEMINI_API_KEY` in `_MODEL_API_KEY_ENV_VARS`. If `ANTHROPIC_API_KEY` is set in the test environment, `_resolve_model_api_key` will return it instead of the intended `"gemini-key"`, causing the assertion to fail. P2: Missing `monkeypatch.delenv("STAGEHAND_SEA_BINARY", raising=False)`. If this env var is set in the runner's environment, `resolve_binary_path()` returns early before reaching the monkeypatched `_resource_binary_path`/`_copy_to_cache`, causing the test to fail. ```

Architecture diagram

```mermaid sequenceDiagram participant User as Developer / App participant Client as Stagehand Client participant Env as OS Environment participant Manager as SeaServerManager participant Binary as SEA Server Binary participant GitHub as GitHub Releases API Note over User, GitHub: Initialization & Key Resolution Flow User->>Client: NEW: Stagehand(model_api_key=None, ...) Client->>Env: NEW: _resolve_model_api_key() Note right of Env: Iterates provider list:
MODEL_API_KEY, OPENAI_API_KEY,
ANTHROPIC_API_KEY, GEMINI_API_KEY, etc. Env-->>Client: Return first matching value alt No API key found in args or environment Client-->>User: NEW: Raise StagehandError (lists all supported env vars) end Note over Client, Binary: Local Server Startup (Server="local") Client->>Manager: NEW: Initialize SeaServerConfig(model_api_key=...) Note right of Client: local_openai_api_key now
maps to model_api_key Manager->>Binary: resolve_binary_path() opt Version check required Binary->>GitHub: NEW: resolve_latest_server_tag() Note right of GitHub: NEW: Filters for stable tags only
(ignores -dev or +build) GitHub-->>Binary: Latest stable stagehand-server-v3/vX.Y.Z end Binary-->>Manager: Local path to binary Manager->>Manager: NEW: _build_process_env() Note right of Manager: Sets MODEL_API_KEY in child env
regardless of provider source Manager->>Binary: CHANGED: subprocess.Popen(env=proc_env) Binary->>Binary: Start Stagehand Server Note over Binary: Server uses MODEL_API_KEY
for all LLM provider requests Binary-->>Manager: Server ready Manager-->>Client: Connection established Client-->>User: Client ready ```

_{Reply with feedback, questions, or to request a fix. Tag `@cubic-dev-ai` to re-run a review.}