Skip to content

Add optional headed browser mode - #419

Merged
davidmckayv merged 4 commits into
CopilotKit:mainfrom
maximus792:codex/agent-computer-headed
Sep 8, 2026
Merged

Add optional headed browser mode#419
davidmckayv merged 4 commits into
CopilotKit:mainfrom
maximus792:codex/agent-computer-headed

Conversation

@maximus792

Copy link
Copy Markdown
Contributor

Summary

  • add an opt-in COMPUTER_BROWSER_MODE=headed that launches full Chromium while preserving the existing headless default
  • start a private Xvfb display with collision-free -displayfd allocation and stop the computer if that display dies
  • pass the mode through Compose and supervisor-managed per-Bot computers, with configuration and recreation guidance

This reuses the existing screencast and take-the-wheel flow; it does not expose X11/VNC or change the browser-control API.

Verification

  • agent-computer: 215 passed, 14 integration-gated skipped; typecheck passed
  • supervisor: 16 non-Docker tests passed; typecheck passed
  • repository formatting and lint passed
  • headed container on a Linux Docker host navigated with full Chromium and no --headless flag
  • killing its owned Xvfb process made the computer close cleanly with exit code 1, ready for the container restart policy

The full supervisor Docker suite has an unrelated baseline failure in the older-image readiness fixture on current main; the changed supervisor environment tests pass.

@davidmckayv davidmckayv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deep-reviewed against live code (correctness, governance, no vendor/secret/scale issues). Composed build+tests green. CHANGELOG/format rebase on CI-validated substance.

@davidmckayv
davidmckayv merged commit 33c5ac3 into CopilotKit:main Sep 8, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants