mempool: add interface compliance checks for OrphanManager and FeeEstimator

In this commit, we add compile-time interface compliance checks for
OrphanManager and FeeEstimator, ensuring they correctly implement the
OrphanTxManager and TxFeeEstimator interfaces respectively.

These var _ InterfaceType = (*ConcreteType)(nil) declarations are a Go
idiom for compile-time interface verification. If OrphanManager doesn't
implement all methods of OrphanTxManager, or FeeEstimator doesn't
implement TxFeeEstimator, the code will fail to compile with a clear
error message.

This is particularly valuable during refactoring, as it ensures that if
we change an interface method signature, all implementations are updated
accordingly. The checks have zero runtime cost and serve as living
documentation of the implementation relationships.

Author: Roasbeef

mempool/estimatefee.go

@@ -756,3 +756,6 @@ func RestoreFeeEstimator(data FeeEstimatorState) (*FeeEstimator, error) {
 
 	return ef, nil
 }
+
+// Ensure FeeEstimator implements the TxFeeEstimator interface.
+var _ TxFeeEstimator = (*FeeEstimator)(nil)

mempool/orphan.go

@@ -427,3 +427,6 @@ func (om *OrphanManager) ProcessOrphans(
 
 	return promoted, nil
 }
+
+// Ensure OrphanManager implements the OrphanTxManager interface.
+var _ OrphanTxManager = (*OrphanManager)(nil)