Fix partial transactions

[n8maninger]

No description provided.

[Copilot]

Pull Request Overview

This PR corrects the selection of partial-block transactions by switching from using FullHash to MerkleLeafHash when matching requested transactions.

• Updated syncer/peer.go to filter by MerkleLeafHash()
• Updated chain/manager.go to filter by MerkleLeafHash()
• Added a changeset file documenting the fix

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
syncer/peer.go	Use MerkleLeafHash instead of FullHash when matching txns
chain/manager.go	Use MerkleLeafHash instead of FullHash in transaction lookup
.changeset/...partial_blocks.md	Documented the patch in the changeset file

Comments suppressed due to low confidence (3)

syncer/peer.go:347

• [nitpick] The variable name want is ambiguous; consider renaming it to something like desiredLeafHashes to make its purpose clearer.

if want[txn.MerkleLeafHash()] {

chain/manager.go:884

• [nitpick] The want map name is vague in this context; renaming it to targetLeafHashes or similar would improve readability.

if h := txn.MerkleLeafHash(); want[h] {

chain/manager.go:881

• There are no tests covering the switch to using MerkleLeafHash() for partial-block retrieval; consider adding unit tests to verify correct behavior when matching by leaf hash.

// TODO: might want to cache these

[peterjan]

wow 👀

[lukechampine]

dang. Forgot about that interaction between the Manager and block outlines. :/