Skip to content

Conversation

@suchapalaver
Copy link
Collaborator

Query paymentsEscrowTransactions from the network subgraph to detect redeemed Horizon RAVs, replacing the LatestRavs query approach. This enables more reliable detection of allocation redemptions by checking for actual redeem transactions rather than comparing RAV values.

Also adds allocation redemption checking for the AllocationId check to prevent accepting receipts for already-redeemed Horizon allocations.


Signed off by Joseph Livesey [email protected]

…king

Query paymentsEscrowTransactions from the network subgraph to detect
redeemed Horizon RAVs, replacing the LatestRavs query approach. This
enables more reliable detection of allocation redemptions by checking
for actual redeem transactions rather than comparing RAV values.

Also adds allocation redemption checking for the AllocationId check
to prevent accepting receipts for already-redeemed Horizon allocations.
@suchapalaver suchapalaver marked this pull request as ready for review February 3, 2026 03:21
@coveralls
Copy link

Pull Request Test Coverage Report for Build 21615507385

Details

  • 178 of 219 (81.28%) changed or added relevant lines in 3 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.3%) to 69.149%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/tap-agent/src/agent/sender_allocation.rs 19 21 90.48%
crates/tap-agent/src/tap/context/checks/allocation_id.rs 132 141 93.62%
crates/tap-agent/src/agent/sender_account.rs 27 57 47.37%
Files with Coverage Reduction New Missed Lines %
crates/tap-agent/src/agent/sender_account.rs 3 85.72%
Totals Coverage Status
Change from base Build 21612257228: 0.3%
Covered Lines: 10743
Relevant Lines: 15536

💛 - Coveralls

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.

3 participants