fix(predict): round fee decimals to avoid underflow errors #22361
+3
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.




Description
Fixed a floating-point precision issue in the Polymarket fee calculation that was causing order placement failures. When calculating fees (e.g., for a $7.40 bet), JavaScript floating-point arithmetic would produce values with excessive decimal places (e.g., 0.29600000000000004), which would later fail when passed to
parseUnits. The fix rounds the fee to 4 decimal places to prevent these precision errors while maintaining sufficient accuracy for fee calculations.Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
Order placement would fail with parseUnits error when fee calculation resulted in values like 0.29600000000000004
After
Order placement succeeds as fee is properly rounded to 0.2960 (4 decimal places)
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Round
totalFeeto 4 decimals incalculateFeesto prevent floating-point precision issues.Written by Cursor Bugbot for commit b198cda. This will update automatically on new commits. Configure here.