fix(tree): adjust both number and hash when removing persisted blocks from memory

[fgimenez]

On TreeState::remove_until we only update the block number when the finalized num is ahead of the upper bound.

reth/crates/engine/tree/src/tree/mod.rs

Lines 329 to 336 in 3fa1e28

	// If the finalized num is ahead of the upper bound, and exists, we need to instead ensure
	// that the only blocks removed, are canonical blocks less than the upper bound
	// finalized_num.take_if(|finalized| *finalized > upper_bound);
	let finalized_num_hash = finalized_num_hash.map(|mut finalized| {
	finalized.number = finalized.number.min(upper_bound);
	debug!(target: "engine::tree", ?finalized, "Adjusted upper bound");
	finalized
	});

We should update the hash too, it is used later in prune_finalized_sidechains

[Rjected]

ahh this makes sense