fix(browser): restore stale profile permissions

pyproject.toml

@@ -34,6 +34,7 @@ dependencies = [
     "boto3>=1.40.61",
     "harness-browser>=0.7.4",
     "playwright>=1.40",
+    "socksio>=1.0.0",
     "psycopg[binary]>=3.2",
     "langgraph-checkpoint-postgres>=2.0",
 ]
@@ -176,6 +177,7 @@ disable_error_code = ["attr-defined"]
 [tool.pytest.ini_options]
 testpaths = ["tests"]
 asyncio_mode = "auto"
+tmp_path_retention_count = 1
 markers = [
     "live: tests that hit real LLM endpoints (skipped by default)",
     "postgresql: tests that need a live PostgreSQL (set OCTOP_TEST_DATABASE_URL)",

tests/unit/browser/test_browser_setup.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import contextlib
 import json
 import os
 from pathlib import Path
@@ -191,8 +192,12 @@ def test_ensure_profile_writable_recreates_when_not_writable(
             assert result == profile or "harness-browser-profiles" in str(result)
             assert _probe_dir_writable(result)
         finally:
-            if profile.exists():
-                os.chmod(profile, 0o700)
+            # Restore permissions on the profile and any stale dirs renamed
+            # by ensure_profile_writable (they keep 0o000 → pytest can't rm).
+            for d in [profile, *tmp_path.glob("default.stale-*")]:
+                if d.exists():
+                    with contextlib.suppress(OSError):
+                        os.chmod(d, 0o700)
     else:
         assert ensure_profile_writable(profile) == profile