You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: .agents/skills/next-pr/SKILL.md
+6Lines changed: 6 additions & 0 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -65,6 +65,11 @@ Run the same tests and benchmarks used for the baseline so the comparison is mea
65
65
- benchmark regressions or improvements
66
66
- new warnings, flaky behavior, or environment issues
67
67
68
+
If the PR fixes a Java bug and does not include a regression test:
69
+
- add a regression test as part of the review branch before approval
70
+
- verify that the regression test fails against the baseline branch without the fix
71
+
- verify that the same regression test passes on the rebased PR branch with the fix
72
+
68
73
7. Summarize for approval before merging.
69
74
Report:
70
75
- PR number and title
@@ -92,6 +97,7 @@ When a PR was merged by rebasing or otherwise rewriting commits, verify whether
92
97
- If the repo has both fast checks and full checks, run the full set unless the user explicitly narrows scope.
93
98
- If benchmarks are noisy, report broad direction and uncertainty instead of overclaiming precision.
94
99
- If a failure is clearly pre-existing, say that explicitly and separate it from PR-specific findings.
100
+
- For Java bugfix PRs, require a regression test. If one is missing, add it on the review branch and prove it fails before the fix and passes after the fix.
0 commit comments