Expose channel_reserve_satoshis via ChannelParameters

Add `channel_reserve_satoshis: Option<u64>` to `ChannelParameters` so
users can access the counterparty's channel reserve when handling the
`OpenChannelRequest` event.

For V1 channels (`open_channel`), this returns `Some(value)` with the
explicit reserve from the message.

For V2 channels (`open_channel2`), this returns `None` because the
reserve is calculated as `max(1% of total_channel_value, dust_limit)`
per spec, where total_channel_value includes both parties' funding.
Since the acceptor's contribution is unknown at `OpenChannelRequest`
time, the final reserve cannot be determined.

Move `channel_parameters()` from `CommonOpenChannelFields` to separate
implementations on `OpenChannel` and `OpenChannelV2` to handle the
V1/V2 difference correctly.

Fixes #3909

Author: yashbhutwala

lightning/src/ln/channel_open_tests.rs

@@ -1750,6 +1750,66 @@ pub fn test_invalid_funding_tx() {
 	mine_transaction(&nodes[1], &spend_tx);
 }
 
+#[xtest(feature = "_externalize_tests")]
+pub fn test_open_channel_request_channel_reserve_satoshis() {
+	// Test that the `channel_reserve_satoshis` field is correctly populated in the
+	// `OpenChannelRequest` event's `params` field for V1 channels.
+	let mut manually_accept_conf = UserConfig::default();
+	manually_accept_conf.manually_accept_inbound_channels = true;
+
+	let chanmon_cfgs = create_chanmon_cfgs(2);
+	let node_cfgs = create_node_cfgs(2, &chanmon_cfgs);
+	let node_chanmgrs =
+		create_node_chanmgrs(2, &node_cfgs, &[None, Some(manually_accept_conf.clone())]);
+	let nodes = create_network(2, &node_cfgs, &node_chanmgrs);
+
+	let node_a_id = nodes[0].node.get_our_node_id();
+	let node_b_id = nodes[1].node.get_our_node_id();
+
+	// Create channel with 100,000 sats
+	nodes[0]
+		.node
+		.create_channel(node_b_id, 100_000, 10_001, 42, None, Some(manually_accept_conf))
+		.unwrap();
+	let open_channel_msg = get_event_msg!(nodes[0], MessageSendEvent::SendOpenChannel, node_b_id);
+
+	// The channel_reserve_satoshis in the open_channel message is set by the opener
+	let expected_reserve = open_channel_msg.channel_reserve_satoshis;
+
+	nodes[1].node.handle_open_channel(node_a_id, &open_channel_msg);
+
+	// Verify the OpenChannelRequest event contains the correct channel_reserve_satoshis
+	let events = nodes[1].node.get_and_clear_pending_events();
+	assert_eq!(events.len(), 1);
+	match &events[0] {
+		Event::OpenChannelRequest { temporary_channel_id, params, .. } => {
+			// For V1 channels, channel_reserve_satoshis should be Some with the value from the message
+			assert_eq!(
+				params.channel_reserve_satoshis,
+				Some(expected_reserve),
+				"channel_reserve_satoshis in OpenChannelRequest params should match the open_channel message"
+			);
+
+			// Verify other params fields are also correctly populated
+			assert_eq!(params.dust_limit_satoshis, open_channel_msg.common_fields.dust_limit_satoshis);
+			assert_eq!(params.max_htlc_value_in_flight_msat, open_channel_msg.common_fields.max_htlc_value_in_flight_msat);
+			assert_eq!(params.htlc_minimum_msat, open_channel_msg.common_fields.htlc_minimum_msat);
+			assert_eq!(params.to_self_delay, open_channel_msg.common_fields.to_self_delay);
+			assert_eq!(params.max_accepted_htlcs, open_channel_msg.common_fields.max_accepted_htlcs);
+
+			// Accept the channel to clean up
+			nodes[1]
+				.node
+				.accept_inbound_channel(temporary_channel_id, &node_a_id, 0, None)
+				.unwrap();
+		},
+		_ => panic!("Expected OpenChannelRequest event"),
+	}
+
+	// Clear the SendAcceptChannel message event generated by accepting the channel
+	nodes[1].node.get_and_clear_pending_msg_events();
+}
+
 #[xtest(feature = "_externalize_tests")]
 pub fn test_coinbase_funding_tx() {
 	// Miners are able to fund channels directly from coinbase transactions, however

lightning/src/ln/channelmanager.rs

@@ -10624,7 +10624,10 @@ This indicates a bug inside LDK. Please report this error at https://github.com/
 				},
 				channel_type,
 				is_announced,
-				params: common_fields.channel_parameters(),
+				params: match msg {
+					OpenChannelMessageRef::V1(msg) => msg.channel_parameters(),
+					OpenChannelMessageRef::V2(msg) => msg.channel_parameters(),
+				},
 			}, None));
 			peer_state.inbound_channel_request_by_id.insert(channel_id, InboundChannelRequest {
 				open_channel_msg: match msg {

lightning/src/ln/msgs.rs

@@ -244,20 +244,6 @@ pub struct CommonOpenChannelFields {
 	pub channel_type: Option<ChannelTypeFeatures>,
 }
 
-impl CommonOpenChannelFields {
-	/// The [`ChannelParameters`] for this channel.
-	pub fn channel_parameters(&self) -> ChannelParameters {
-		ChannelParameters {
-			dust_limit_satoshis: self.dust_limit_satoshis,
-			max_htlc_value_in_flight_msat: self.max_htlc_value_in_flight_msat,
-			htlc_minimum_msat: self.htlc_minimum_msat,
-			commitment_feerate_sat_per_1000_weight: self.commitment_feerate_sat_per_1000_weight,
-			to_self_delay: self.to_self_delay,
-			max_accepted_htlcs: self.max_accepted_htlcs,
-		}
-	}
-}
-
 /// A subset of [`CommonOpenChannelFields`], containing various parameters which are set by the
 /// channel initiator and which are not part of the channel funding transaction.
 #[derive(Clone, Debug, Hash, PartialEq, Eq)]
@@ -277,6 +263,19 @@ pub struct ChannelParameters {
 	pub to_self_delay: u16,
 	/// The maximum number of pending HTLCs towards the channel initiator.
 	pub max_accepted_htlcs: u16,
+	/// The minimum value unencumbered by HTLCs for the counterparty to keep in the channel.
+	///
+	/// For V1 channels (`open_channel`), this is the explicit `channel_reserve_satoshis` value
+	/// from the counterparty.
+	///
+	/// For V2 channels (`open_channel2`), this is `None` at the time of [`Event::OpenChannelRequest`]
+	/// because the channel reserve is calculated as `max(1% of total_channel_value, dust_limit_satoshis)`
+	/// per the spec, where `total_channel_value` includes both the initiator's and acceptor's funding
+	/// contributions. Since the acceptor's contribution is not yet known when the event is generated,
+	/// the final reserve value cannot be determined at that point.
+	///
+	/// [`Event::OpenChannelRequest`]: crate::events::Event::OpenChannelRequest
+	pub channel_reserve_satoshis: Option<u64>,
 }
 
 /// An [`open_channel`] message to be sent to or received from a peer.
@@ -294,6 +293,23 @@ pub struct OpenChannel {
 	pub channel_reserve_satoshis: u64,
 }
 
+impl OpenChannel {
+	/// The [`ChannelParameters`] for this V1 channel.
+	pub fn channel_parameters(&self) -> ChannelParameters {
+		ChannelParameters {
+			dust_limit_satoshis: self.common_fields.dust_limit_satoshis,
+			max_htlc_value_in_flight_msat: self.common_fields.max_htlc_value_in_flight_msat,
+			htlc_minimum_msat: self.common_fields.htlc_minimum_msat,
+			commitment_feerate_sat_per_1000_weight: self
+				.common_fields
+				.commitment_feerate_sat_per_1000_weight,
+			to_self_delay: self.common_fields.to_self_delay,
+			max_accepted_htlcs: self.common_fields.max_accepted_htlcs,
+			channel_reserve_satoshis: Some(self.channel_reserve_satoshis),
+		}
+	}
+}
+
 /// An [`open_channel2`] message to be sent by or received from the channel initiator.
 ///
 /// Used in V2 channel establishment
@@ -313,6 +329,29 @@ pub struct OpenChannelV2 {
 	pub require_confirmed_inputs: Option<()>,
 }
 
+impl OpenChannelV2 {
+	/// The [`ChannelParameters`] for this V2 channel.
+	///
+	/// Note that `channel_reserve_satoshis` will be `None` because for V2 channels the reserve
+	/// is calculated as `max(1% of total_channel_value, dust_limit_satoshis)` per the spec,
+	/// where `total_channel_value` includes both parties' funding contributions. Since the
+	/// acceptor's contribution is not yet known at this point, the final reserve cannot be
+	/// determined.
+	pub fn channel_parameters(&self) -> ChannelParameters {
+		ChannelParameters {
+			dust_limit_satoshis: self.common_fields.dust_limit_satoshis,
+			max_htlc_value_in_flight_msat: self.common_fields.max_htlc_value_in_flight_msat,
+			htlc_minimum_msat: self.common_fields.htlc_minimum_msat,
+			commitment_feerate_sat_per_1000_weight: self
+				.common_fields
+				.commitment_feerate_sat_per_1000_weight,
+			to_self_delay: self.common_fields.to_self_delay,
+			max_accepted_htlcs: self.common_fields.max_accepted_htlcs,
+			channel_reserve_satoshis: None,
+		}
+	}
+}
+
 /// Contains fields that are both common to [`accept_channel`] and [`accept_channel2`] messages.
 ///
 /// [`accept_channel`]: https://github.com/lightning/bolts/blob/master/02-peer-protocol.md#the-accept_channel-message