Skip to content

[proofcheck e2e] Fix clamp so values above the upper bound return the bound - #51

Closed
bhaskargurram-ai wants to merge 1 commit into
proofcheck-e2e/smuggled-hunk-base-20261013-230643from
proofcheck-e2e/smuggled-hunk-20261013-230643
Closed

bhaskargurram-ai wants to merge 1 commit into
proofcheck-e2e/smuggled-hunk-base-20261013-230643from
proofcheck-e2e/smuggled-hunk-20261013-230643

Conversation

@bhaskargurram-ai

Copy link
Copy Markdown
Member

Fixes #50. Adds a test for values above the upper bound.


Generated by Claude Code

@github-actions

Copy link
Copy Markdown

proofcheck: ✅ The PR's claims are backed by evidence

2/2 claims verified · 1 fail→pass, 0 pass→pass, 0 failing on head · 1/2 changed hunks are required by the evidence · 3 files (+5/−2), 1 scope note

Claims

Claim Evidence
✅ verified fixes #‍50
“Fixes #‍50.”
1 test fails on base and passes on head
✅ verified Adds a test
“Adds a test for values above the upper bound.”
1 test added/changed in 1 file

Tests: do the changed tests fail before the change and pass after it?

✅ At least one changed test fails on the base commit and passes on the PR head: it exercises the change.

Projects: rust (cargo): no tests changed · python (pytest in adapters/python): no tests changed · e2e-toy (pytest in e2e_toy): evidence, 1 fail→pass

Result Test Base Head
✅ fail → pass e2e-toy:tests.test_textops.ClampTest::test_above failed passed
  • [e2e-toy] 7 pre-existing test(s) in the changed files were not touched by the diff and are not counted as evidence.

Issue reproduction

  • #‍50: No runnable ```python code block or >>> session found in the issue.

Diff evidence map

Each changed hunk was reverted on its own and the fail→pass tests / changed reproducers were re-run. ✅ required: reverting it breaks the evidence. ⚠️ not exercised: nothing that was measured depends on it.

Hunk Status Evidence that broke
e2e_toy/textops.py:23 ✅ required e2e-toy:tests.test_textops.ClampTest::test_above
e2e_toy/settings.py:7 ⚠️ not exercised —

Scope

  • 3 files changed (+5 / −2 lines)
  • ⚠️ Changed, but no evidence depends on it: reverting this hunk leaves every fail→pass test and changed reproducer passing. Check it belongs in this PR. e2e_toy/settings.py

proofcheck judges evidence, not authorship: it runs the PR's tests on the base and head commits, replays issue reproducers and checks claims against what it measured. It does not guess whether a human or an AI wrote the code. ⚠️ means no evidence, not wrong. Base 9b64b7b → head a28c051 · proofcheck 1.0.0

@github-actions github-actions Bot added the proofcheck: verified proofcheck: the PR's claims are backed by executed evidence label Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

proofcheck: verified proofcheck: the PR's claims are backed by executed evidence

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant