Skip to content

Reject non-finite fixture amounts - #13

Merged
rogerchappel merged 3 commits into
mainfrom
agent/oss-f63cbd45bfca-fixture-amount-validation
Aug 11, 2026
Merged

Reject non-finite fixture amounts#13
rogerchappel merged 3 commits into
mainfrom
agent/oss-f63cbd45bfca-fixture-amount-validation

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • reject non-finite invoice and payment amounts during fixture normalization
  • report the source CSV path, row, and amount field through LedgerpetError
  • cover library and CLI failures, including the guarantee that invalid fixtures create no report directory
  • document that finite decimals and zero are supported

Commits

  • fix: reject non-finite fixture amounts
  • test: cover malformed fixture amounts
  • docs: define fixture amount requirements

Verification

  • npm run release:check
  • git diff --check origin/main...HEAD
  • git log --format="%h %an %ae %cn %ce" origin/main..HEAD (all commits: Roger Chappel miscanalysis@gmail.com)

@rogerchappel
rogerchappel merged commit b8a21b2 into main Aug 11, 2026
1 check 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.

1 participant