From 96cdd0c2cb7ca13ee837813d94a3275db24e2dc8 Mon Sep 17 00:00:00 2001 From: winlin Date: Thu, 28 May 2026 21:31:42 -0400 Subject: [PATCH] Claude: skill: Clarify that all proxy E2E scripts must run. The "Run the proxy E2E tests:" wording read as if running the first script was enough. Make explicit that every E2E script must run, one at a time (fixed ports prevent parallelism), and not to stop early since a later test can fail even when earlier ones pass. Co-authored-by: Claude Opus 4.6 --- skills/srs-develop/SKILL.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/skills/srs-develop/SKILL.md b/skills/srs-develop/SKILL.md index a252e9345..968f490c5 100644 --- a/skills/srs-develop/SKILL.md +++ b/skills/srs-develop/SKILL.md @@ -143,7 +143,7 @@ Only after the user confirms the routing do you proceed to Step 2. ``` bash scripts/proxy-utest.sh --coverage ``` -4. Run the proxy E2E tests: +4. Run **all** of the proxy E2E tests below — every one, not just the first. Run them one at a time (they bind fixed ports, so they cannot run in parallel), and do not stop early: a later test can fail even when the earlier ones pass. - Single-origin RTMP proxy test (starts proxy + one SRS origin, publishes RTMP, verifies playback): ``` bash scripts/proxy-e2e-test.sh