Repository navigation
Conversation
Throwaway file with a deliberate, obvious logic bug (is_even returns True for odd numbers) to force the reviewer model to generate a real, non-empty findings list -- the case that broke in every prior strict-mode failure. Will be deleted; this PR is not meant to merge. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
🤖 marginal review
PR #103: DO NOT MERGE -- verify non-empty findings on v0.2.4 · open · 2d1ad43f6114b0119ee426be3d6433ef356fbfe9 → 16da681c9b4719bac027087f1a6e82376c3691a9 · 1 file changed
Changed files (1)
scratch_verify_findings.py
|
|
||
| def is_even(n: int) -> bool: | ||
| """Return whether `n` is even.""" | ||
| return n % 2 == 1 |
There was a problem hiding this comment.
🟠 High · 98% confidence
The function is_even has an inverted/incorrect implementation: n % 2 == 1 returns True for odd numbers, not even ones. This bug would cause the function to always return the wrong result.
| @@ -0,0 +1,11 @@ | |||
| """Throwaway file to force a non-empty marginal review finding. | |||
There was a problem hiding this comment.
🟡 Medium · 85% confidence
This file is explicitly marked as a throwaway/scratch file with 'DO NOT MERGE' in its docstring, indicating it should not be part of the final PR. It should be removed before merging.
|
Verification complete -- v0.2.4 correctly generated and posted 2 real findings (98% and 85% confidence) for this deliberate bug. Closing without merging, deleting the branch. |
1 participant
Throwaway verification PR, not meant to merge. Deliberately introduces an obvious bug (
is_evenreturns True for odd numbers) so the reviewer model has to generate a real, non-emptyfindingspayload -- the exact case every prior strict-mode failure (the $PARAMETER_NAME wrapper, the stringified array, the 400 rejection) only ever happened on. Will close this and delete the branch once verified either way.