Skip to content

Fix refund checks for JSON string amounts - #16

Merged
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
be-student:codex/11-string-refund-amounts
Sep 6, 2026
Merged

Fix refund checks for JSON string amounts#16
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
be-student:codex/11-string-refund-amounts

Conversation

@be-student

Copy link
Copy Markdown
Contributor

Fixes #11.

  • Compare refund caps and amounts through finite decimal normalization, including JSON numeric strings.
  • Reject booleans, empty or malformed strings, and non-finite values as missing amounts.
  • Add regression and invalid-input coverage, plus document the accepted input contract.

Validation: pytest -q (21 passed), and both documented CLI fixture checks passed. Ruff reports the same six findings as the base revision; this patch adds none.

AI disclosure: Implemented with Codex assistance and manually reviewed and tested.

Signed-off-by: eunwoo song <presentsong@naver.com>
@royalpinto007
royalpinto007 merged commit 5753716 into AgentPostmortem:main Sep 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

check_policy_violation ignores string amounts

2 participants