Repository navigation
Drive verify-graph with the Playwright CLI - #66
Merged
Merged
Conversation
Swap the Playwright MCP calls for playwright-cli (one session per branch, so parallel checks don't share a browser), and refresh the checks to match develop and the zenoh simulator: BMS/Segment_Total_Volt/0 as the topic, the Quick Select historical range, and develop's known NG0100 console errors as the baseline. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PVbhpw5Qyzvh4MP7UgZJx9
… run, per-branch browser output serve.sh claims its port under a short mkdir lock, so parallel runs stop picking the same port while ng serve starts. verify-graph gets scripts: seed.sh makes one 50-hour run spanning two midnights (once, under a lock) and prints its id; select-run.sh picks a run by id in the autoplaying carousel; open.sh opens a per-branch session with its output under pictures/<branch>/verify-graph/. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PVbhpw5Qyzvh4MP7UgZJx9
check.sh seeds the shared run, opens the branch's session and runs check.js: page load, a tree topic drawing live, the seeded run, historical Quick Select, back to real time and new console errors. It prints a checklist and screenshots to judge. The skill now points the agent at hand checks for what the diff touches, with the CLI patterns to drive them. helpers.js is gone: select-run.sh replaced it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PVbhpw5Qyzvh4MP7UgZJx9
…throwaway code to .playwright-cli/<branch>/ Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PVbhpw5Qyzvh4MP7UgZJx9
…nd throwaway code), so it's excluded too Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PVbhpw5Qyzvh4MP7UgZJx9
bracyw
force-pushed
the
verify-graph-playwright-cli
branch
from
October 18, 2026 16:07
f2a589d to
34f5d97
Compare
bracyw
marked this pull request as ready for review
October 18, 2026 16:08
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
The Argos
verify-graphskill now drives the graph page withplaywright-cli(from@playwright/cli) instead of the Playwright MCP tools. Each branch gets its own browser session, so several branches can be verified side by side against one backend.The base checks are scripted. check.sh opens the branch's session and runs check.js against a seeded run, and prints a checklist plus screenshots for the agent to judge. It checks that:
The skill then points the agent at hand checks for whatever the diff touches, and gives the CLI patterns to drive them.
Supporting scripts, from running seven branches in parallel:
run-local's serve.sh now claims its port under a short lock, so parallel runs stop picking the same port while ng serve starts.
Diagram
flowchart LR subgraph Before direction TB A1["agent"] -->|"Playwright MCP"| B1["clicks through all 7 groups by hand"] B1 --> C1["Select Run picks whichever run the carousel shows"] P1["parallel serve.sh"] -->|"both see the same port free"| X1["port collision"] end subgraph After direction TB A2["agent"] --> S["check.sh"] S --> SD["seed.sh: one 50-hour run, made once under a lock"] S --> O["open.sh: session per branch; output in pictures/branch, config in .playwright-cli/branch"] S --> CJ["check.js: 6 base checks"] CJ --> R["checklist + screenshots to judge"] R --> H["hand checks where the diff points: playwright-cli, select-run.sh"] P2["parallel serve.sh"] -->|"port claimed under a short lock"| U["one port each"] end Before ~~~ AfterTesting
Checklist
🤖 Generated with Claude Code
https://claude.ai/code/session_01PVbhpw5Qyzvh4MP7UgZJx9