Skip to content

Fix browser operator profile staging and ChatGPT wrapper #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
lisihao wants to merge 1 commit into
base: codex/runtime-drift-writepath-fix
Choose a base branch
from
Open

Fix browser operator profile staging and ChatGPT wrapper #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 22 additions & 2 deletions harness/lib/browser_job_runtime.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -436,6 +436,26 @@ def _remove_profile_restore_artifacts(root: Path, profile_directory: str) -> Non
pass


def _browser_profile_copy_ignore(_dir: str, names: list[str]) -> set[str]:
# Chrome extension assets can change while the live profile is being copied.
# They are not required for ChatGPT auth/session reuse and have repeatedly
# caused flaky FileNotFoundError during Browser Agent profile staging.
ignored = {
"Extensions",
"Extension State",
"Extension Rules",
"Extension Scripts",
"Extension Cookies",
"ShaderCache",
"GrShaderCache",
"GPUCache",
"Code Cache",
"Cache",
"DawnCache",
}
return {name for name in names if name in ignored}


def refresh_browser_profile_cache(user_data_dir: str | Path | None, profile_directory: str | None) -> Optional[Path]:
if not user_data_dir or not profile_directory:
return None
Expand All @@ -450,7 +470,7 @@ def refresh_browser_profile_cache(user_data_dir: str | Path | None, profile_dire
cache_root.mkdir(parents=True, exist_ok=True)
if cache_profile.exists():
shutil.rmtree(cache_profile, ignore_errors=True)
shutil.copytree(source_profile, cache_profile, dirs_exist_ok=True)
shutil.copytree(source_profile, cache_profile, dirs_exist_ok=True, ignore=_browser_profile_copy_ignore)
local_state_src = source_root / "Local State"
if local_state_src.exists():
shutil.copy(local_state_src, cache_root / "Local State")
Expand Down Expand Up @@ -497,7 +517,7 @@ def _stage_browser_profile(user_data_dir: str | Path | None, profile_directory:

staged_root = Path(tempfile.mkdtemp(prefix=_STAGED_PROFILE_PREFIX))
staged_profile = staged_root / profile_directory
shutil.copytree(source_profile, staged_profile, dirs_exist_ok=True)
shutil.copytree(source_profile, staged_profile, dirs_exist_ok=True, ignore=_browser_profile_copy_ignore)

local_state_src = source_root / "Local State"
if local_state_src.exists():
Expand Down
Loading