Skip to content

fix(payment): bound newest-first mempool scans - #38

Merged
Alike001 merged 1 commit into
mainfrom
fix/bound-mempool-scans
Aug 30, 2026
Merged

Alike001 merged 1 commit into
mainfrom
fix/bound-mempool-scans

Conversation

@Alike001

Copy link
Copy Markdown
Owner

Summary

  • filter invoice-relevant mempool entries before decoding
  • sort candidates newest-first so newly broadcast customer payments are checked first
  • cap each poll at 12 getrawtransaction calls instead of scanning the whole Testnet mempool
  • keep truncated post-expiry scans fail-closed
  • retain mined-block confirmation as the final payment authority

Why

A production poll attempted more than 100 getrawtransaction requests. Roughly 20 succeeded and the rest were rejected, filling the checkout evidence panel with red calls and unnecessarily stressing the free QuickNode endpoint.

Verification

  • formatting, lint, and type-check passed
  • payment verifier: 23 passed
  • remaining suite: 129 passed, 1 skipped; two PGlite startup timeouts passed on clean isolated reruns (14 + 3 tests)
  • secret scan passed
  • production build passed

Closes #37

Copilot AI lite review requested due to automatic review settings August 30, 2026 15:07
@vercel

vercel Bot commented Aug 30, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
zecceipt Ready Ready Preview Aug 30, 2026 3:07pm

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Alike001
Alike001 merged commit 405e35f into main Aug 30, 2026
4 checks passed
@Alike001
Alike001 deleted the fix/bound-mempool-scans branch August 30, 2026 15:09

This branch was successfully deployed

1 active deployment
Preview — a11d968c Deployed Aug 30, 2026 by vercel[bot]
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.

fix(payment): bound newest-first mempool transaction scans

2 participants