fixup! Add ReserveType to ChannelDetails

make channel reserve type optional until negotiation determines reserve behavior

Author: enigbe

src/types.rs

@@ -587,17 +587,19 @@ pub struct ChannelDetails {
 	pub channel_shutdown_state: Option<ChannelShutdownState>,
 	/// The type of on-chain reserve maintained for this channel.
 	///
+	/// Will be `None` until channel negotiation has completed and determined whether
+	/// this channel uses anchor or legacy reserve behavior.
+	///
 	/// See [`ReserveType`] for details on how reserves differ between anchor and legacy channels.
-	pub reserve_type: ReserveType,
+	pub reserve_type: Option<ReserveType>,
 }
 
 impl ChannelDetails {
 	pub(crate) fn from_ldk(
 		value: LdkChannelDetails, anchor_channels_config: Option<&AnchorChannelsConfig>,
 	) -> Self {
-		let reserve_type =
-			if value.channel_type.as_ref().is_some_and(|ct| ct.supports_anchors_zero_fee_htlc_tx())
-			{
+		let reserve_type = value.channel_type.as_ref().map(|channel_type| {
+			if channel_type.supports_anchors_zero_fee_htlc_tx() {
 				if let Some(config) = anchor_channels_config {
 					if config.trusted_peers_no_reserve.contains(&value.counterparty.node_id) {
 						ReserveType::TrustedPeersNoReserve
@@ -616,7 +618,8 @@ impl ChannelDetails {
 				}
 			} else {
 				ReserveType::Legacy
-			};
+			}
+		});
 
 		ChannelDetails {
 			channel_id: value.channel_id,