refactor: removed multiple unused includes; added some must-have headers

It improves compilation time insignificantly (1minute out of 98minutes,  assuming 1 core compilation, 1% improvement)

Author: knst

src/bls/bls_worker.cpp

@@ -4,7 +4,6 @@
 
 #include <bls/bls_worker.h>
 #include <hash.h>
-#include <serialize.h>
 
 #include <util/ranges.h>
 #include <util/system.h>

src/coinjoin/client.cpp

@@ -7,7 +7,6 @@
 #include <chain.h>
 #include <chainparams.h>
 #include <coinjoin/options.h>
-#include <consensus/validation.h>
 #include <core_io.h>
 #include <evo/deterministicmns.h>
 #include <masternode/meta.h>
@@ -25,10 +24,9 @@
 #include <version.h>
 #include <wallet/coincontrol.h>
 #include <wallet/coinjoin.h>
-#include <wallet/fees.h>
+#include <wallet/coinselection.h>
 #include <wallet/receive.h>
 #include <wallet/spend.h>
-#include <walletinitinterface.h>
 
 #include <memory>
 #include <univalue.h>

src/coinjoin/coinjoin.cpp

@@ -7,14 +7,10 @@
 #include <bls/bls.h>
 #include <chain.h>
 #include <chainparams.h>
-#include <consensus/validation.h>
-#include <governance/common.h>
 #include <instantsend/instantsend.h>
 #include <llmq/chainlocks.h>
 #include <masternode/node.h>
 #include <masternode/sync.h>
-#include <messagesigner.h>
-#include <netmessagemaker.h>
 #include <txmempool.h>
 #include <util/moneystr.h>
 #include <util/system.h>

src/coinjoin/interfaces.cpp

@@ -9,11 +9,8 @@
 #include <coinjoin/options.h>
 #include <node/context.h>
 #include <util/check.h>
-#include <wallet/wallet.h>
 #include <walletinitinterface.h>
 
-#include <univalue.h>
-
 #include <memory>
 #include <string>
 

src/coinjoin/server.cpp

@@ -4,7 +4,6 @@
 
 #include <coinjoin/server.h>
 
-#include <consensus/validation.h>
 #include <core_io.h>
 #include <evo/deterministicmns.h>
 #include <masternode/meta.h>
@@ -21,7 +20,6 @@
 #include <util/ranges.h>
 #include <util/system.h>
 #include <validation.h>
-#include <version.h>
 
 #include <univalue.h>
 

src/coinjoin/util.cpp

@@ -3,10 +3,8 @@
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
 #include <coinjoin/util.h>
-#include <net.h>
 #include <policy/fees.h>
 #include <policy/policy.h>
-#include <script/sign.h>
 #include <util/translation.h>
 #include <wallet/fees.h>
 #include <wallet/spend.h>

src/dsnotificationinterface.cpp

@@ -2,7 +2,6 @@
 // Distributed under the MIT software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
-#include <chainparams.h>
 #include <coinjoin/coinjoin.h>
 #ifdef ENABLE_WALLET
 #include <coinjoin/client.h>
@@ -11,7 +10,6 @@
 #include <dsnotificationinterface.h>
 #include <governance/governance.h>
 #include <masternode/sync.h>
-#include <net_processing.h>
 #include <validation.h>
 
 #include <evo/deterministicmns.h>

src/evo/chainhelper.cpp

@@ -4,7 +4,7 @@
 
 #include <evo/chainhelper.h>
 
-#include <consensus/params.h>
+#include <chainparams.h>
 #include <evo/specialtxman.h>
 #include <instantsend/instantsend.h>
 #include <instantsend/lock.h>

src/evo/deterministicmns.cpp

@@ -11,7 +11,6 @@
 #include <evo/specialtx.h>
 #include <index/txindex.h>
 
-#include <base58.h>
 #include <chainparams.h>
 #include <coins.h>
 #include <consensus/validation.h>

src/evo/dmnstate.cpp

@@ -3,14 +3,10 @@
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
 #include <evo/dmnstate.h>
-#include <evo/providertx.h>
 
-#include <chainparams.h>
-#include <consensus/validation.h>
 #include <script/standard.h>
 
 #include <univalue.h>
-#include <messagesigner.h>
 
 std::string CDeterministicMNState::ToString() const
 {