fix decimals handling

[ca333]

add robust normalization utilities

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix_decimals

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Visit the preview URL for this PR (updated for commit 5252473):

https://walletrc--pull-3297-merge-7hhk7a1v.web.app

_{(expires Wed, 05 Nov 2025 14:15:05 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: f66a4ff03faa546f12f0ae5a841bd9eff2714dcc}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

NBSP separators not stripped during normalization

The new normalizeDecimalString tries to remove non‑breaking spaces (U+00A0) and narrow no‑break spaces (U+202F) before parsing, but the literals passed to replaceAll are '\u00A0' and '\u202F'. Those are two-character sequences (backslash + u…) rather than the actual Unicode whitespace characters, so real NBSPs remain in the string and the subsequent regex fails. Inputs copied from locales that use NBSP as the thousands separator (e.g. French 1 234,56) will still throw FormatException despite the tests claiming support. Use ' ' and ' ' to strip the intended characters.

Useful? React with 👍 / 👎.