Stop visual-test scripts leaving empty folders and popping Finder

[roylibman]

Summary

Visual-test runs that capture no screenshots no longer leave empty timestamped folders behind or open Finder on an empty directory. This keeps repeated local visual-test runs quiet while preserving screenshot folders when captures actually exist.

What changed

• visual-test-stop.sh opens the screenshot folder only when it contains captured files.
• Empty run directories are removed during stop instead of being left under .playwright-mcp/visual-test/.
• visual-test-start.sh prunes leftover empty timestamped directories before a new run starts.
• The startup prune preserves the screenshot directory for the currently active run recorded in .playwright-mcp/visual-test-state.env when that Radar process is still alive, so relaunching with --skip-build cannot delete the active capture target.

Testing

• bash -n scripts/visual-test-start.sh
• bash -n scripts/visual-test-stop.sh
• git diff --check
• visual-test: skipped (script-only cleanup; no UI delta)

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 46d66f2. Configure here.}