refactor(engine): always connect buffered blocks on r/w hook finish

[shekhirin]

This PR removes a separate ConnectBufferedBlocks engine hook action, and instead connects the buffered blocks upon completion of any hook with read-write access to the database.

It is logically correct, because when we start a hook with read-write access to the database, we can't process incoming payloads, and buffer them instead, hence we need to check the buffered blocks (payloads) for any such hook.

[codecov]

Codecov Report

Merging #4657 (df50e3a) into main (8012942) will decrease coverage by 0.01%.
Report is 9 commits behind head on main.
The diff coverage is 81.25%.

Files Changed	Coverage Δ
crates/consensus/beacon/src/engine/hooks/mod.rs	62.50% <0.00%> (-2.72%)	⬇️
...es/consensus/beacon/src/engine/hooks/controller.rs	79.41% <76.92%> (+5.16%)	⬆️
crates/consensus/beacon/src/engine/mod.rs	74.30% <88.23%> (-0.28%)	⬇️
crates/consensus/beacon/src/engine/hooks/prune.rs	87.01% <100.00%> (+0.76%)	⬆️

... and 23 files with indirect coverage changes

Flag	Coverage Δ
integration-tests	16.74% <0.00%> (-0.06%)	⬇️
unit-tests	63.35% <81.25%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
reth binary	32.17% <ø> (-0.01%)	⬇️
blockchain tree	83.75% <ø> (ø)
pipeline	88.54% <ø> (ø)
storage (db)	73.01% <ø> (-0.47%)	⬇️
trie	94.73% <ø> (-0.04%)	⬇️
txpool	49.44% <ø> (ø)
networking	77.10% <ø> (-0.15%)	⬇️
rpc	57.82% <ø> (+0.36%)	⬆️
consensus	63.25% <81.25%> (+0.58%)	⬆️
revm	28.16% <ø> (+0.12%)	⬆️
payload builder	8.45% <ø> (ø)
primitives	86.53% <ø> (+0.02%)	⬆️

[mattsse]

It is logically correct, because when we start a hook with read-write access to the database, we can't process incoming payloads, and buffer them instead, hence we need to check the buffered blocks (payloads) for any such hook.

this is correct, however it looks like this breaks a few tests

[shekhirin]

@mattsse god bless engine tests, fixed