CHIA-1862 Fix FullNodeAPI's request_block_headers returned filter

[AmineKhaldi]

Purpose:

This fixes a bug where request_block_headers is not accounting for removals and additions when computing the transactions filter.

Current Behavior:

request_block_headers doesn't account for removals and additions in transactions filter computation.

New Behavior:

request_block_headers takes them properly into account.

[coveralls-official]

Pull Request Test Coverage Report for Build 12394466119

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 75 of 75 (100.0%) changed or added relevant lines in 4 files are covered.
• 43 unchanged lines in 7 files lost coverage.
• Overall coverage increased (+0.05%) to 91.589%

---

Files with Coverage Reduction	New Missed Lines	%
chia/_tests/simulation/test_simulation.py	1	96.45%
chia/plotters/plotters.py	1	90.94%
chia/full_node/full_node.py	2	86.91%
chia/full_node/weight_proof.py	4	90.64%
chia/plotters/madmax.py	6	44.58%
chia/daemon/server.py	7	83.24%
chia/_tests/core/util/test_lockfile.py	22	77.31%

Totals
Change from base Build 12381535704:	0.05%
Covered Lines:	105092
Relevant Lines:	114566

---

💛 - Coveralls

[arvidn]

your benchmark doesn't show that it's safe to remove the serialization optimization.

[arvidn]

looks good! It would be good to have proper test coverage of the new parsing function too

[AmineKhaldi]

looks good! It would be good to have proper test coverage of the new parsing function too

Done. Please note that while the test in question is skipped in main, enabling it and running it (with this extra coverage added to it) passes.

[arvidn]

I don't think your test is being run. I think you need to delete this line:
https://github.com/Chia-Network/chia-blockchain/pull/18923/files#diff-3681c4e6fa79a6c076c109cb43bbd19b2bb909d4be47b6808289926148366fbdR253