diff --git a/CHANGELOG.md b/CHANGELOG.md index c84e899..df01878 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,10 @@ # Changelog +## 0.8.1 + +### Added +- **`/ship-pr` test evidence gate** — Screenshot Check now also gates on new test files (`*.test.*`, `*.spec.*`). The PR body must include verbose test runner output proving the new tests pass. For Playwright/E2E tests, screenshots of tested behavior are also required. + ## 0.8.0 ### Added diff --git a/VERSION b/VERSION index a3df0a6..6f4eebd 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -0.8.0 +0.8.1 diff --git a/skills/ship-pr/SKILL.md b/skills/ship-pr/SKILL.md index 7165681..af5b30e 100644 --- a/skills/ship-pr/SKILL.md +++ b/skills/ship-pr/SKILL.md @@ -53,6 +53,15 @@ Update VERSION, package.json, or pyproject.toml (whichever exists). Update CHANG - If the diff includes files in UI/frontend packages AND `evidence/screenshots/` is empty or missing: STOP and warn the user. Recommend running /validate or capturing screenshots now. +- If the diff adds new test files (`*.test.*`, `*.spec.*`, `test_*.py`, `*_test.py`, + `*_test.go`, `*_spec.rb`, `test_*.rb`): the PR body MUST include proof that the + new tests pass. At minimum, embed the verbose test runner output. + To detect Playwright/E2E tests: check for `playwright` imports in the test file, + a `playwright.config.*` at the repo root, or test location in an `e2e/` directory. + If E2E tests are detected, also capture and embed screenshots of the tested behavior. + STOP if no evidence exists — run the tests now and capture output. + STOP if the tests fail — do not embed failure output as "evidence." The tests must + be green before the PR can ship. Report the failures and halt. - If screenshots exist: continue and embed them in the PR body using absolute GitHub blob URLs. - Organize screenshots by feature or mode (e.g., "Docker mode" vs "Sprites mode").