fix(automation): issue watcher must not open PRs without tests #138

Merged

shad0w merged 1 commit from fix/watcher-test-gate into master

2026-06-25 22:24:54 +00:00

Author	SHA1	Message	Date
Hardik	a5248e5c23	fix(automation): issue watcher must not open PRs without tests All checks were successful PR checks / checks (pull_request) Successful in 48s Details PR checks / integration (pull_request) Successful in 30s Details PR #137 shipped a code change under App/app with no test — the test-presence gate in pr-checks.yml would reject it, but the watcher opened the PR anyway. Add the same gate to the watcher's fix phase: before pushing/opening a PR, run the pr-checks.yml test-presence check against the branch diff. If code under App/(app\|lib\|components\|hooks) changed with no accompanying test, the watcher does NOT open a PR — it marks the issue claude-failed and comments, so the queue can retry. Never raises a PR the CI would immediately fail. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2026-06-26 03:47:59 +05:30

Author

SHA1

Message

Date

Hardik

a5248e5c23

fix(automation): issue watcher must not open PRs without tests

PR checks / checks (pull_request) Successful in 48s

Details

PR checks / integration (pull_request) Successful in 30s

Details

PR #137 shipped a code change under App/app with no test — the test-presence
gate in pr-checks.yml would reject it, but the watcher opened the PR anyway.

Add the same gate to the watcher's fix phase: before pushing/opening a PR, run
the pr-checks.yml test-presence check against the branch diff. If code under
App/(app|lib|components|hooks) changed with no accompanying test, the watcher
does NOT open a PR — it marks the issue claude-failed and comments, so the
queue can retry. Never raises a PR the CI would immediately fail.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

2026-06-26 03:47:59 +05:30