Extract second stage of ChannelManager::read into from_channel_manager_data

Move the validation and reconstruction logic (stage 2) from the
ReadableArgs::read implementation into a new pub(super) constructor
`from_channel_manager_data`. This separates the pure deserialization
from the complex reconstruction logic, making the code more modular
and easier to test.

The read function now:
1. Deserializes into ChannelManagerData (stage 1)
2. Calls from_channel_manager_data for validation/reconstruction (stage 2)

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

Author: joostjager

lightning/src/ln/channelmanager.rs

@@ -17840,7 +17840,7 @@ where
 	L::Target: Logger,
 {
 	fn read<Reader: io::Read>(
-		reader: &mut Reader, mut args: ChannelManagerReadArgs<'a, M, T, ES, NS, SP, F, R, MR, L>,
+		reader: &mut Reader, args: ChannelManagerReadArgs<'a, M, T, ES, NS, SP, F, R, MR, L>,
 	) -> Result<Self, DecodeError> {
 		// Stage 1: Pure deserialization into DTO
 		let data: ChannelManagerData<SP> = ChannelManagerData::read(
@@ -17854,6 +17854,44 @@ where
 		)?;
 
 		// Stage 2: Validation and reconstruction
+		ChannelManager::from_channel_manager_data(data, args)
+	}
+}
+
+impl<
+		M: Deref,
+		T: Deref,
+		ES: Deref,
+		NS: Deref,
+		SP: Deref,
+		F: Deref,
+		R: Deref,
+		MR: Deref,
+		L: Deref + Clone,
+	> ChannelManager<M, T, ES, NS, SP, F, R, MR, L>
+where
+	M::Target: chain::Watch<<SP::Target as SignerProvider>::EcdsaSigner>,
+	T::Target: BroadcasterInterface,
+	ES::Target: EntropySource,
+	NS::Target: NodeSigner,
+	SP::Target: SignerProvider,
+	F::Target: FeeEstimator,
+	R::Target: Router,
+	MR::Target: MessageRouter,
+	L::Target: Logger,
+{
+	/// Constructs a `ChannelManager` from deserialized data and runtime dependencies.
+	///
+	/// This is the second stage of deserialization, taking the raw [`ChannelManagerData`] and combining it with the
+	/// provided [`ChannelManagerReadArgs`] to produce a fully functional `ChannelManager`.
+	///
+	/// This method performs validation, reconciliation with [`ChannelMonitor`]s, and reconstruction of internal state.
+	/// It may close channels if monitors are ahead of the serialized state, and will replay any pending
+	/// [`ChannelMonitorUpdate`]s.
+	pub(super) fn from_channel_manager_data(
+		data: ChannelManagerData<SP>,
+		mut args: ChannelManagerReadArgs<'_, M, T, ES, NS, SP, F, R, MR, L>,
+	) -> Result<(BlockHash, Self), DecodeError> {
 		// Extract mutable/complex fields into local variables; simple fields accessed via data.*
 		let mut forward_htlcs_legacy = data.forward_htlcs;
 		let mut claimable_htlcs_list = data.claimable_htlcs;