Drop SerialId type as bindings don't support primitive aliasing

Author: TheBlueMatt

lightning/src/ln/interactivetxs.rs

@@ -24,8 +24,9 @@ use crate::events::bump_transaction::{BASE_INPUT_WEIGHT, EMPTY_SCRIPT_SIG_WEIGHT
 use crate::events::MessageSendEvent;
 use crate::ln::channel::TOTAL_BITCOIN_SUPPLY_SATOSHIS;
 use crate::ln::msgs;
-use crate::ln::msgs::{SerialId, TxSignatures};
+use crate::ln::msgs::TxSignatures;
 use crate::ln::types::ChannelId;
+type SerialId = u64;
 use crate::sign::{EntropySource, P2TR_KEY_PATH_WITNESS_WEIGHT, P2WPKH_WITNESS_WEIGHT};
 use crate::util::ser::TransactionU16LenLimited;
 

lightning/src/ln/msgs.rs

@@ -443,10 +443,6 @@ pub struct ChannelReady {
 	pub short_channel_id_alias: Option<u64>,
 }
 
-/// A randomly chosen number that is used to identify inputs within an interactive transaction
-/// construction.
-pub type SerialId = u64;
-
 /// An `stfu` (quiescence) message to be sent by or received from the stfu initiator.
 ///
 // TODO(splicing): Add spec link for `stfu`; still in draft, using from https://github.com/lightning/bolts/pull/1160
@@ -514,7 +510,7 @@ pub struct TxAddInput {
 	pub channel_id: ChannelId,
 	/// A randomly chosen unique identifier for this input, which is even for initiators and odd for
 	/// non-initiators.
-	pub serial_id: SerialId,
+	pub serial_id: u64,
 	/// Serialized transaction that contains the output this input spends to verify that it is non
 	/// malleable.
 	pub prevtx: TransactionU16LenLimited,
@@ -535,7 +531,7 @@ pub struct TxAddOutput {
 	pub channel_id: ChannelId,
 	/// A randomly chosen unique identifier for this output, which is even for initiators and odd for
 	/// non-initiators.
-	pub serial_id: SerialId,
+	pub serial_id: u64,
 	/// The satoshi value of the output
 	pub sats: u64,
 	/// The scriptPubKey for the output
@@ -550,7 +546,7 @@ pub struct TxRemoveInput {
 	/// The channel ID
 	pub channel_id: ChannelId,
 	/// The serial ID of the input to be removed
-	pub serial_id: SerialId,
+	pub serial_id: u64,
 }
 
 /// A [`tx_remove_output`] message for removing an output during interactive transaction construction.
@@ -561,7 +557,7 @@ pub struct TxRemoveOutput {
 	/// The channel ID
 	pub channel_id: ChannelId,
 	/// The serial ID of the output to be removed
-	pub serial_id: SerialId,
+	pub serial_id: u64,
 }
 
 /// [`A tx_complete`] message signalling the conclusion of a peer's transaction contributions during