Skip to content

Handle exact Decimal amounts across authorization - #10

Merged
rudimentall1 merged 5 commits into
fix/authority-ceiling-decimal-stringsfrom
fix/action-intent-sqlite-decimal
Oct 19, 2026
Merged

rudimentall1 merged 5 commits into
fix/authority-ceiling-decimal-stringsfrom
fix/action-intent-sqlite-decimal

Conversation

@rudimentall1

@rudimentall1 rudimentall1 commented Oct 14, 2026 •

Copy link
Copy Markdown
Owner

What changed

  • Normalize exact ActionIntent amounts at the SQLite boundary so Python Decimal values are never passed directly to SQLite.
  • Serialize amount_exact as a decimal string in audit JSON.
  • Compare the signed exact amount and compatibility amount safely against effective authority limits using Decimal, not mixed string/float comparisons.
  • Fail closed if the two amount representations diverge or either amount/limit is invalid.
  • Add regression tests for SQLite persistence and effective-authority checks with decimal strings.

This fixes the PortalDot local integration path. The demo remains isolated to loopback with disposable storage and keys; no RPC call or transaction broadcast is involved.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@rudimentall1 rudimentall1 changed the title Persist exact ActionIntent amounts through SQLite Handle exact Decimal amounts across authorization Oct 14, 2026
@rudimentall1
rudimentall1 merged commit bb3e9d3 into fix/authority-ceiling-decimal-strings Oct 19, 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.

1 participant