Skip to content

feat(browser): explain browser control before runs#179

Merged
steipete merged 3 commits intosteipete:mainfrom
pdurlej:pdurlej/calmer-browser-ux
May 6, 2026
Merged

feat(browser): explain browser control before runs#179
steipete merged 3 commits intosteipete:mainfrom
pdurlej:pdurlej/calmer-browser-ux

Conversation

@pdurlej
Copy link
Copy Markdown
Contributor

@pdurlej pdurlej commented May 5, 2026

Summary

  • add a small browser control-plan helper so dry-runs and live browser sessions say whether Oracle will launch visible Chrome, hide a new window, attach to an existing browser, or use remote/headless Chrome
  • surface those browser-control lines through non-verbose session logs so MCP/agent callers get a visible progress signal before long GPT-5.5 Pro waits
  • document calmer shared-desktop/Claude Code usage and clarify that long browser consults should be inspected via sessions/status before retrying
  • let --browser-attach-running ignore conflicting launch-only defaults from user config, while still allowing explicitly provided conflicts to fail validation

Why

Agents such as Claude Code can look stuck during long browser-backed GPT-5.5 Pro consults, and visible Chrome automation can surprise the operator. This PR keeps behavior explicit before touching ChatGPT and makes the attach-running path usable even when a user config defaults to manual-login browser launches.

Tests

  • pnpm install --frozen-lockfile
  • pnpm vitest run tests/cli/browserDefaults.test.ts tests/cli/browserConfig.test.ts tests/cli/dryRun.test.ts tests/cli/dryRun.coverage.test.ts tests/browser/controlPlan.test.ts tests/browser/sessionRunner.test.ts
  • pnpm run check
  • pnpm run build
  • pnpm test
  • git diff --check

Smoke / Dogfood

  • node ./dist/bin/oracle-cli.js --dry-run summary --engine browser --model gpt-5.5-pro --browser-attach-running -p "CHECK_CONTROL_PLAN" --file README.md
    • confirmed preview prints Browser control: attach to an already-running local Chrome session without inheriting manual-login config conflicts
  • Oracle GPT-5.5 Pro browser review, slug calmer-browser-ux-review: ACCEPTABLE
  • Oracle GPT-5.5 Pro browser follow-up, slug calmer-browser-ux-review-2: ACCEPTABLE; during the wait it emitted [browser] Waiting for ChatGPT response ..., matching the agent-facing progress case this PR documents

@pdurlej pdurlej force-pushed the pdurlej/calmer-browser-ux branch from 278429c to 25ed018 Compare May 5, 2026 20:51
@pdurlej
Copy link
Copy Markdown
Contributor Author

pdurlej commented May 5, 2026

Rebased onto current upstream main after the live ChatGPT tab harvest landed. I kept tab cleanup conservative with the new current/ref tab mode: blank-tab cleanup only runs for Oracle-owned targets, so user-selected tabs are not closed. Local focused tests, pnpm run check, pnpm run build, and git diff --check passed; GitHub CI is green on macOS, Ubuntu, and Windows.

@pdurlej pdurlej force-pushed the pdurlej/calmer-browser-ux branch from 25ed018 to 635a1c8 Compare May 5, 2026 23:41
@steipete steipete force-pushed the pdurlej/calmer-browser-ux branch from 635a1c8 to 7752295 Compare May 6, 2026 05:48
@steipete steipete merged commit 8465379 into steipete:main May 6, 2026
4 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