refactor: remove dead m_dsq client-side relay from MessageProcessingResult

Client-side DSQ relay was removed in dashpay#7070 but the m_dsq field and its
population in CCoinJoinClientQueueManager::ProcessMessage were left
behind as dead code. The network has been running without client-side
DSQ relay with no issues since masternodes handle DSQ propagation
through their own CCoinJoinServer::ProcessDSQUEUE path.

Remove the dead code and fix transitive include dependencies that were
masked by msg_result.h pulling in coinjoin/coinjoin.h. This also
resolves two circular dependency chains that went through that link.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: UdjinM6

src/coinjoin/client.cpp

@@ -142,7 +142,6 @@ MessageProcessingResult CCoinJoinClientQueueManager::ProcessMessage(NodeId from,
             WITH_LOCK(cs_vecqueue, vecCoinJoinQueue.push_back(dsq));
         }
     } // cs_ProcessDSQueue
-    ret.m_dsq.push_back(dsq);
     return ret;
 }
 

src/instantsend/net_instantsend.h

@@ -15,6 +15,8 @@
 #include <thread>
 #include <vector>
 
+class CChainState;
+
 namespace Consensus {
 struct LLMQParams;
 } // namespace Consensus

src/instantsend/signing.h

@@ -7,6 +7,7 @@
 
 #include <instantsend/lock.h>
 #include <llmq/signing.h>
+#include <primitives/transaction.h>
 
 #include <optional>
 

src/msg_result.h

@@ -5,8 +5,6 @@
 #ifndef BITCOIN_MSG_RESULT_H
 #define BITCOIN_MSG_RESULT_H
 
-#include <coinjoin/coinjoin.h>
-
 #include <protocol.h>
 #include <uint256.h>
 
@@ -50,9 +48,6 @@ struct MessageProcessingResult
     //! @m_inventory will relay these inventories to connected peers
     std::vector<CInv> m_inventory;
 
-    //! @m_dsq will relay DSQs to connected peers
-    std::vector<CCoinJoinQueue> m_dsq;
-
     //! @m_transactions will relay transactions to peers which is ready to accept it (some peers does not accept transactions)
     std::vector<uint256> m_transactions;
 
@@ -68,7 +63,7 @@ struct MessageProcessingResult
         m_error(error)
     {}
 
-    bool empty() const { return !m_error.has_value() && m_inventory.empty() && m_dsq.empty() && m_transactions.empty() && !m_to_erase.has_value(); }
+    bool empty() const { return !m_error.has_value() && m_inventory.empty() && m_transactions.empty() && !m_to_erase.has_value(); }
 };
 
 #endif // BITCOIN_MSG_RESULT_H

src/rpc/quorums.cpp

@@ -24,6 +24,7 @@
 #include <util/helpers.h>
 
 #include <chainparams.h>
+#include <core_io.h>
 #include <deploymentstatus.h>
 #include <index/txindex.h>
 #include <net_processing.h>

test/lint/lint-circular-dependencies.py

@@ -23,9 +23,7 @@
     # Dash
     "active/context -> active/dkgsessionhandler -> llmq/dkgsessionhandler -> net_processing -> active/context",
     "banman -> common/bloom -> evo/assetlocktx -> llmq/quorumsman -> net -> banman",
-    "chainlock/chainlock -> spork -> msg_result -> coinjoin/coinjoin -> chainlock/chainlock",
     "chainlock/chainlock -> spork -> net -> evo/deterministicmns -> evo/providertx -> validation -> chainlock/chainlock",
-    "coinjoin/coinjoin -> instantsend/instantsend -> spork -> msg_result -> coinjoin/coinjoin",
     "coinjoin/client -> coinjoin/util -> wallet/wallet -> psbt -> node/transaction -> net_processing -> coinjoin/walletman -> coinjoin/client",
     "common/bloom -> evo/assetlocktx -> llmq/commitment -> evo/deterministicmns -> evo/simplifiedmns -> merkleblock -> common/bloom",
     "common/bloom -> evo/assetlocktx -> llmq/quorumsman -> net -> common/bloom",