feat(gl-sdk): NodeState snapshot, hex identifiers, list_payments fixes

NodeState
- Unified node_state() snapshot aggregating get_info,
  list_peer_channels, and list_funds into a single record with
  balances, capacity, utxos, and connected-channel-peer set.
- Reserved UTXOs (in-flight PSBTs) excluded from on-chain balance.
  FundOutput now carries `reserved` from listfunds.outputs[].reserved.
- Closing-channel double-count fixed. Onchain-state channels only
  contribute to pending_onchain_balance_msat when we initiated the
  close and our payout is still timelocked (DELAYED_OUTPUT_TO_US);
  otherwise the payout is already a wallet UTXO and would be counted
  twice. PeerChannel exposes `closer` (ChannelSide) and `status` to
  support this gate.
- ChannelState::from_i32 unknown fallback now maps to a new
  ChannelState::Unknown variant rather than Onchain (which silently
  counted unmapped states as closing). Unknown is treated as neither
  open nor closing by balance math. gl-sdk-cli output formatter
  handles the new variant too.
- Immature outputs (confirmed but timelocked, e.g. coinbase
  maturation) surface as immature_onchain_balance_msat instead of
  being silently dropped. OutputStatus match is exhaustive.
- The three underlying RPCs run concurrently via tokio::join!.

NodeState aggregate fields:
- total_onchain_msat (confirmed + unconfirmed + immature)
- total_balance_msat (everything the user owns)
- spendable_balance_msat (send-screen gate)
- max_chan_reserve_msat (protocol reserve locked across channels)
- utxos: Vec<FundOutput> for coin-control UIs

Other NodeState changes:
- channels_balance_msat / max_payable_msat docstrings explicitly
  name their roles: home-screen display vs send-button gate.
- num_active/pending/inactive_channels docstrings clarify which
  balance fields each count contributes to.
- connected_peers renamed to connected_channel_peers (only lists
  peers we have a channel with); dedup via HashSet.
- fees_collected_msat removed from NodeState (still on
  GetInfoResponse); it's a routing-node concern.

Hex identifier surface
- Every identifier (pubkey, payment hash, txid, preimage, funding
  txid, channel id) on public structs is now a lowercase hex String
  instead of Vec<u8>. Affected: NodeState, PeerChannel, FundOutput,
  FundChannel, GetInfoResponse, Peer, Invoice, Pay, Payment,
  SendResponse, OnchainSendResponse (txid only; raw tx stays bytes),
  InvoicePaidEvent, ParsedInvoice.
- Kept as Vec<u8>: Credentials::load/save bytes, raw on-chain tx
  bytes, Peer.features bitfield, DeveloperCert/Signer constructor
  arguments.

list_payments fixes
- Unpaid (open) and expired invoices are dropped from list_payments.
  Only Paid invoices appear as Payment rows on the received side;
  list_invoices() still surfaces the full invoice list for callers
  that want to inspect open invoices directly.
- Payment.destination is documented as always-None for
  PaymentType::Received: Lightning's privacy model does not reveal
  the sender's pubkey to the recipient, and the only pubkey
  derivable from a paid invoice is the payee (our own node), which
  is uninteresting per-row. PaymentStatus::Pending stays on the
  public type — it remains valid for in-flight sent payments.

NAPI mirrors synced (Buffer→String everywhere the underlying field
became hex), gl-sdk-cli output formatters simplified, NodeExtensions
defaults adjusted, and Python + Kotlin tests updated to match.

Author: angelix

libs/gl-sdk-android/lib/src/androidInstrumentedTest/kotlin/com/blockstream/glsdk/NodeOperationsTest.kt

@@ -2,6 +2,8 @@ package com.blockstream.glsdk
 
 import android.system.Os
 import androidx.test.ext.junit.runners.AndroidJUnit4
+import org.junit.Assert.assertEquals
+import org.junit.Assert.assertTrue
 import org.junit.Before
 import org.junit.Test
 import org.junit.runner.RunWith
@@ -41,4 +43,22 @@ class NodeOperationsTest {
             println("Lightning Invoice: ${invoice.toString()}")
         }
     }
+
+    @Test
+    fun test_node_state_returns_valid_snapshot() {
+        val config = Config()
+        val node = registerOrRecover(mnemonic = testMnemonic, inviteCode = null, config = config)
+        node.use { n ->
+            val state = n.nodeState()
+            assertTrue(state.id.isNotEmpty())
+            assertTrue(state.blockHeight > 0u)
+            assertEquals("bitcoin", state.network)
+            assertTrue(state.version.isNotEmpty())
+            assertEquals(0uL, state.channelsBalanceMsat)
+            assertEquals(0uL, state.maxPayableMsat)
+            assertEquals(0uL, state.totalChannelCapacityMsat)
+            assertEquals(0uL, state.onchainBalanceMsat)
+            assertEquals(0uL, state.totalInboundLiquidityMsat)
+        }
+    }
 }

libs/gl-sdk-android/lib/src/commonMain/kotlin/com/blockstream/glsdk/NodeExtensions.kt

@@ -30,7 +30,7 @@ public fun Node.listPayments(
     filters: List<PaymentTypeFilter>? = null,
     fromTimestamp: ULong? = null,
     toTimestamp: ULong? = null,
-    includeFailures: Boolean? = null,
+    includeFailures: Boolean = false,
     offset: UInt? = null,
     limit: UInt? = null,
 ): List<Payment> = listPayments(ListPaymentsRequest(filters, fromTimestamp, toTimestamp, includeFailures, offset, limit))

libs/gl-sdk-cli/src/output.rs

@@ -27,9 +27,9 @@ pub struct GetInfoOutput {
 impl From<glsdk::GetInfoResponse> for GetInfoOutput {
     fn from(r: glsdk::GetInfoResponse) -> Self {
         Self {
-            id: hex::encode(&r.id),
+            id: r.id,
             alias: r.alias,
-            color: hex::encode(&r.color),
+            color: r.color,
             num_peers: r.num_peers,
             num_pending_channels: r.num_pending_channels,
             num_active_channels: r.num_active_channels,
@@ -73,7 +73,7 @@ impl From<glsdk::ListPeersResponse> for ListPeersOutput {
 impl From<glsdk::Peer> for PeerOutput {
     fn from(p: glsdk::Peer) -> Self {
         Self {
-            id: hex::encode(&p.id),
+            id: p.id,
             connected: p.connected,
             num_channels: p.num_channels,
             netaddr: p.netaddr,
@@ -130,18 +130,19 @@ fn channel_state_str(s: &glsdk::ChannelState) -> &'static str {
         glsdk::ChannelState::DualopendAwaitingLockin => "DUALOPEND_AWAITING_LOCKIN",
         glsdk::ChannelState::DualopendOpenCommitted => "DUALOPEND_OPEN_COMMITTED",
         glsdk::ChannelState::DualopendOpenCommitReady => "DUALOPEND_OPEN_COMMIT_READY",
+        glsdk::ChannelState::Unknown => "UNKNOWN",
     }
 }
 
 impl From<glsdk::PeerChannel> for PeerChannelOutput {
     fn from(c: glsdk::PeerChannel) -> Self {
         Self {
-            peer_id: hex::encode(&c.peer_id),
+            peer_id: c.peer_id,
             peer_connected: c.peer_connected,
             state: channel_state_str(&c.state).to_string(),
             short_channel_id: c.short_channel_id,
-            channel_id: c.channel_id.map(|v| hex::encode(&v)),
-            funding_txid: c.funding_txid.map(|v| hex::encode(&v)),
+            channel_id: c.channel_id,
+            funding_txid: c.funding_txid,
             funding_outnum: c.funding_outnum,
             to_us_msat: c.to_us_msat,
             total_msat: c.total_msat,
@@ -205,7 +206,7 @@ impl From<glsdk::ListFundsResponse> for ListFundsOutput {
 impl From<glsdk::FundOutput> for FundOutputOutput {
     fn from(o: glsdk::FundOutput) -> Self {
         Self {
-            txid: hex::encode(&o.txid),
+            txid: o.txid,
             output: o.output,
             amount_msat: o.amount_msat,
             status: output_status_str(&o.status).to_string(),
@@ -218,15 +219,15 @@ impl From<glsdk::FundOutput> for FundOutputOutput {
 impl From<glsdk::FundChannel> for FundChannelOutput {
     fn from(c: glsdk::FundChannel) -> Self {
         Self {
-            peer_id: hex::encode(&c.peer_id),
+            peer_id: c.peer_id,
             our_amount_msat: c.our_amount_msat,
             amount_msat: c.amount_msat,
-            funding_txid: hex::encode(&c.funding_txid),
+            funding_txid: c.funding_txid,
             funding_output: c.funding_output,
             connected: c.connected,
             state: channel_state_str(&c.state).to_string(),
             short_channel_id: c.short_channel_id,
-            channel_id: c.channel_id.map(|v| hex::encode(&v)),
+            channel_id: c.channel_id,
         }
     }
 }
@@ -267,7 +268,7 @@ impl From<glsdk::SendResponse> for SendOutput {
     fn from(r: glsdk::SendResponse) -> Self {
         Self {
             status: pay_status_str(&r.status).to_string(),
-            preimage: hex::encode(&r.preimage),
+            preimage: r.preimage,
             amount_msat: r.amount_msat,
             amount_sent_msat: r.amount_sent_msat,
             parts: r.parts,
@@ -301,7 +302,7 @@ impl From<glsdk::OnchainSendResponse> for OnchainSendOutput {
     fn from(r: glsdk::OnchainSendResponse) -> Self {
         Self {
             tx: hex::encode(&r.tx),
-            txid: hex::encode(&r.txid),
+            txid: r.txid,
             psbt: r.psbt,
         }
     }
@@ -330,9 +331,9 @@ impl From<glsdk::NodeEvent> for NodeEventOutput {
     fn from(e: glsdk::NodeEvent) -> Self {
         match e {
             glsdk::NodeEvent::InvoicePaid { details } => NodeEventOutput::InvoicePaid {
-                payment_hash: hex::encode(&details.payment_hash),
+                payment_hash: details.payment_hash,
                 bolt11: details.bolt11,
-                preimage: hex::encode(&details.preimage),
+                preimage: details.preimage,
                 label: details.label,
                 amount_msat: details.amount_msat,
             },