Complete E13-T4 browser verification and epic closure

Author: dmoliveira

CHANGELOG.md

@@ -83,6 +83,7 @@ All notable changes to this project are documented in this file.
 - Marked Epic 13 as in progress in the roadmap and completed Task 13.1 definition notes.
 - Expanded install and selftest coverage for browser provider profile switching and missing dependency guidance.
 - Expanded README wizard/browser guidance with provider trade-offs, stable-first defaults, and `/browser` usage examples.
+- Expanded browser verification coverage to assert provider reset readiness and added install smoke checks that run browser status/doctor after switching across providers.
 
 ## v0.2.0 - 2026-02-12
 

IMPLEMENTATION_ROADMAP.md

@@ -49,7 +49,7 @@ Use this map to avoid overlapping implementations.
 | E10 | Auto Slash Command Detector | paused | Medium | E1, E8 | TBD | Resume only if intent precision stays high in prototypes |
 | E11 | Context-Window Resilience Toolkit | done | High | E4 | bd-2tj, bd-n9y, bd-2t0, bd-18e | Improve long-session stability and recovery |
 | E12 | Provider/Model Fallback Visibility | done | Medium | E5 | bd-1jq, bd-298, bd-194, bd-2gq | Explain why model routing decisions happen |
-| E13 | Browser Automation Profile Switching | in_progress | Medium | E1 | bd-3rs, bd-2qy, bd-f6g | Toggle Playwright/agent-browser with checks |
+| E13 | Browser Automation Profile Switching | done | Medium | E1 | bd-3rs, bd-2qy, bd-f6g, bd-393 | Toggle Playwright/agent-browser with checks |
 | E14 | Plan-to-Execution Bridge Command | planned | Medium | E2, E3 | TBD | Execute validated plans with progress tracking |
 | E15 | Todo Enforcer and Plan Compliance | planned | High | E14 | TBD | Keep execution aligned with approved checklists |
 | E16 | Comment and Output Quality Checker Loop | merged | Medium | E23 | TBD | Merged into E23 (PR Review Copilot) |
@@ -531,7 +531,7 @@ Every command-oriented epic must ship all of the following:
 
 ## Epic 13 - Browser Automation Profile Switching
 
-**Status:** `in_progress`
+**Status:** `done`
 **Priority:** Medium
 **Goal:** Add first-class profile switching between browser automation engines with install/runtime checks.
 **Depends on:** Epic 1
@@ -551,12 +551,13 @@ Every command-oriented epic must ship all of the following:
   - [x] Subtask 13.3.2: Document provider trade-offs and examples
   - [x] Subtask 13.3.3: Add recommended defaults for stable-first users
   - [x] Notes: Extended `scripts/install_wizard.py` with browser profile selection (`--browser-profile`) and updated README install/browser guidance with provider trade-offs plus stable-first recommendations.
-- [ ] Task 13.4: Verification
-  - [ ] Subtask 13.4.1: Add tests for profile switching and persistence
-  - [ ] Subtask 13.4.2: Add smoke tests for status/doctor across providers
-  - [ ] Subtask 13.4.3: Add install-test checks
-- [ ] Exit criteria: provider switching is one-command and reversible
-- [ ] Exit criteria: missing dependency states are diagnosed with exact fixes
+- [x] Task 13.4: Verification
+  - [x] Subtask 13.4.1: Add tests for profile switching and persistence
+  - [x] Subtask 13.4.2: Add smoke tests for status/doctor across providers
+  - [x] Subtask 13.4.3: Add install-test checks
+  - [x] Notes: Expanded `scripts/selftest.py` to validate provider reset readiness and updated install smoke checks to run `status` and `doctor` across both providers.
+- [x] Exit criteria: provider switching is one-command and reversible
+- [x] Exit criteria: missing dependency states are diagnosed with exact fixes
 
 ---
 

Makefile

@@ -47,7 +47,7 @@ install-test: ## Run installer smoke test in temp HOME
 	HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/config_command.py" list; \
 	HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/stack_profile_command.py" apply focus; \
 	HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/stack_profile_command.py" status; \
-	if [ -f "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" ]; then HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" status --json; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" profile agent-browser; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" doctor --json; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" profile playwright; fi; \
+	if [ -f "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" ]; then HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" status --json; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" profile agent-browser; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" doctor --json; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" profile playwright; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" status --json; HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/browser_command.py" doctor --json; fi; \
 	HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/nvim_integration_command.py" install minimal --link-init; \
 	HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/nvim_integration_command.py" status; \
 	HOME="$$TMP_HOME" python3 "$$TMP_HOME/.config/opencode/my_opencode/scripts/devtools_command.py" status; \

scripts/selftest.py

@@ -1616,6 +1616,39 @@ def run_bg(*args: str) -> subprocess.CompletedProcess[str]:
             "browser doctor should report missing selected-provider dependencies",
         )
 
+        browser_profile_reset = subprocess.run(
+            [sys.executable, str(BROWSER_SCRIPT), "profile", "playwright"],
+            capture_output=True,
+            text=True,
+            env=refactor_env,
+            check=False,
+            cwd=REPO_ROOT,
+        )
+        expect(
+            browser_profile_reset.returncode == 0,
+            "browser profile should switch back to playwright",
+        )
+        browser_doctor_playwright = subprocess.run(
+            [sys.executable, str(BROWSER_SCRIPT), "doctor", "--json"],
+            capture_output=True,
+            text=True,
+            env=refactor_env,
+            check=False,
+            cwd=REPO_ROOT,
+        )
+        expect(
+            browser_doctor_playwright.returncode == 0,
+            "browser doctor should succeed after switching back to playwright",
+        )
+        browser_doctor_playwright_report = parse_json_output(
+            browser_doctor_playwright.stdout
+        )
+        expect(
+            browser_doctor_playwright_report.get("provider") == "playwright"
+            and browser_doctor_playwright_report.get("selected_ready") is True,
+            "browser doctor should report ready selected provider after reset",
+        )
+
         keyword_report = resolve_prompt_modes(
             "Please safe-apply and deep-analyze this migration; ulw can wait.",
             enabled=True,