Skip to content

Commit 87fe29f

Browse files
isPANNclaude
andcommitted
Revert review-quality bullet to main version
The PR's edit added a Critical-severity rule for `[Rule]` PRs that deferred to review-structural Step 4b-3 — now a dead link since that section was reverted. Main's original bullet already captures the intent: verify the extracted solution is optimal via brute-force on both source and target. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 24d7b80 commit 87fe29f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.claude/skills/review-quality/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ Flag tests that:
7272
- **Mirror the implementation**: Tests recomputing the same formula as the code prove nothing
7373
- **Lack adversarial cases**: Only happy path. Tests must include infeasible configs and boundary cases
7474
- **Use trivial instances only**: Single-edge or 2-node tests may pass with bugs. Need 5+ vertex instances
75-
- **Closed-loop without round-trip verification (CRITICAL for `[Rule]` PRs)**: for new reduction rules, defer to the four-criterion check in [`review-structural`](../review-structural/SKILL.md) Step 4b-3. If the test fails any criterion there, raise it as **Critical** here too — not Minor.
75+
- **Closed-loop without verification**: Must verify extracted solution is **optimal** (compare brute-force on both source and target)
7676
- **Assert count too low**: 1-2 asserts for non-trivial code is insufficient
7777

7878
## Output Format

0 commit comments

Comments
 (0)