feat: enable patrol develop on web and harden MCP session#3040
feat: enable patrol develop on web and harden MCP session#3040piotruela wants to merge 17 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces web support for the Patrol MCP server and enhances the develop command for web platforms. Key changes include implementing platform-specific session behaviors, adding web screenshot capabilities via the Chrome DevTools Protocol, and improving log surfacing and error handling for web tests. Additionally, the PR includes new web integration tests and updates to the patrol_cli to manage web-specific processes and debugger ports. Feedback includes a critical fix for invalid Dart syntax in patrol_session.dart, a recommendation to remove local absolute paths from configuration files, and suggestions to improve the robustness of WebSocket connections and hot restart suppression logic.
6102110 to
d3c8398
Compare
9e4b472 to
cb7bdce
Compare
Summary
Web develop mode
patrol developfor web targets — previously blocked. Cold-starts the app on each restart since the web runner can't hot-restart like mobile (web_test_backend.dart, develop.ts, develop_service.dart).PATROL_LOGentries from the web runner to MCP so test status is observable in develop mode.MCP improvements
PlatformSessionBehaviorabstraction (platform_session_behavior.dart).E2E app / tooling
run-patrolhelper.