sneak/AutistMask
1
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: preserve multiple token transfers per tx hash in transaction history #75

Closed
clawbot wants to merge 1 commits from fix/issue-72-address-token-tx-history into main
pull from: fix/issue-72-address-token-tx-history
merge into: sneak:main
Conversation 1 Commits 1 Files Changed 1 +13 -4
clawbot commented 2026-02-28 21:08:47 +01:00
Collaborator
Copy Link

When a swap produces multiple ERC-20 token transfers with the same tx hash (e.g. send WETH + receive USDC), only the last transfer was kept because txsByHash was keyed by bare tx hash. This caused the address-token view to show no transactions for tokens obtained via swap.

Use a composite key (hash:contractAddress) so all token transfers are preserved. The bare-hash normal-tx entry is removed when token transfers replace it to avoid duplication.

docker build . passes.

Closes #72

When a swap produces multiple ERC-20 token transfers with the same tx hash (e.g. send WETH + receive USDC), only the last transfer was kept because `txsByHash` was keyed by bare tx hash. This caused the address-token view to show no transactions for tokens obtained via swap. Use a composite key (`hash:contractAddress`) so all token transfers are preserved. The bare-hash normal-tx entry is removed when token transfers replace it to avoid duplication. `docker build .` passes. Closes #72
clawbot added 1 commit 2026-02-28 21:08:47 +01:00
fix: preserve multiple token transfers per tx hash in transaction history
All checks were successful
check / check (push) Successful in 22s
Details
256acbeeec 
When a swap produces multiple ERC-20 token transfers with the same tx
hash (e.g. send WETH + receive USDC), only the last transfer was kept
because txsByHash was keyed by bare tx hash. This caused the
address-token view to show no transactions for tokens obtained via swap.

Use a composite key (hash:contractAddress) so all token transfers are
preserved. The bare-hash normal-tx entry is removed when token transfers
replace it to avoid duplication.

Closes #72
clawbot added the
bot
needs-review
 labels 2026-02-28 21:08:58 +01:00
clawbot self-assigned this 2026-02-28 21:08:59 +01:00
clawbot removed their assignment 2026-02-28 21:09:05 +01:00
clawbot commented 2026-02-28 21:09:08 +01:00
Author
Collaborator
Copy Link

Closing as duplicate of #76.

Closing as duplicate of #76.
clawbot closed this pull request 2026-02-28 21:09:09 +01:00
All checks were successful
check / check (push) Successful in 22s
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bot

Agent's turn to work

  
merge-ready

   
needs-checks

   
needs-rebase

   
needs-review

   
needs-rework
No Label
bot
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sneak/AutistMask#75
No description provided.
Delete Branch "fix/issue-72-address-token-tx-history"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?