Merge pull request #124 from arik-so/arik/2023/10/118-prep-01

Prepare 0.0.118
lightningdevkit · Oct 27, 2023 · 412de9a · 412de9a
2 parents 2abea22 + d6c92fc
commit 412de9a
Show file tree

Hide file tree

Showing 264 changed files with 30,209 additions and 11,894 deletions.
diff --git a/.github/actions/install-dependencies/action.yml b/.github/actions/install-dependencies/action.yml
@@ -22,8 +22,8 @@ runs:
             run: |
                 echo `pwd`
                 mkdir -p ${{ inputs.destination }}/bin
-                git clone --branch 2023-07-0.0.116-java-bindings https://github.com/TheBlueMatt/rust-lightning ${{ inputs.destination }}/rust-lightning
-                git clone --branch v0.0.116.0 https://github.com/lightningdevkit/ldk-c-bindings ${{ inputs.destination }}/ldk-c-bindings
+                git clone --branch 0.0.118-bindings https://github.com/lightningdevkit/rust-lightning ${{ inputs.destination }}/rust-lightning
+                git clone --branch main https://github.com/lightningdevkit/ldk-c-bindings ${{ inputs.destination }}/ldk-c-bindings
         -   name: Install Rust, required targets
             if: ${{ inputs.configureRustNightly == 'true' }}
             shell: bash
@@ -42,6 +42,10 @@ runs:
                     ~/.cargo/git/db/
                     target/
                 key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
+        -   name: Pin cbindgen v24
+            shell: bash
+            run: |
+                cargo install --version 0.24.5 cbindgen
         -   name: Generate C Bindings
             shell: bash
             run: |

diff --git a/.github/workflows/swift.yml b/.github/workflows/swift.yml
@@ -19,6 +19,11 @@ jobs:
         steps:
             -   name: Checkout source code
                 uses: actions/checkout@v3
+            -   name: Print environment versions
+                run: |
+                    node --version
+                    npm --version
+                    yarn --version
             -   name: Install native Rust toolchain and build utilities
                 run: |
                     # sudo apt-get purge grub\*
@@ -35,6 +40,10 @@ jobs:
                     configureRustNightly: false
                 env:
                     LDK_SWIFT_GENERATOR_OUTPUT_DIRECTORY_PATH: ci/LDKSwift/Sources/LDKSwift/bindings
+            -   name: Print header file checksum
+                run: sha256sum $LDK_SWIFT_GENERATOR_INPUT_HEADER_PATH
+                env:
+                    LDK_SWIFT_GENERATOR_INPUT_HEADER_PATH: ci/ldk-c-bindings/lightning-c-bindings/include/lightning.h
             -   name: Copy new headers into bindings
                 run: |
                     python3 ./scripts/copy_c_files.py

diff --git a/README.md b/README.md
@@ -7,7 +7,7 @@ Automatic Swift bindings generation for [`rust-lightning`](https://github.com/li
 ### Swift Package Manager
 To install using Swift Package Manager, add the following line to the depedencies array of your `Package.swift`:
 ```
-.package(url: "https://github.com/lightningdevkit/ldk-swift/", exact: "0.0.116")
+.package(url: "https://github.com/lightningdevkit/ldk-swift/", exact: "0.0.118")
 ```
 Be sure to also include "LightningDevKit" into your packages target like so:
 
@@ -58,13 +58,13 @@ In order to generate these bindings from scratch, you will need to clone two dep
 **[rust-lightning](https://github.com/lightningdevkit/rust-lightning)**, (a specific branch built for bindings compatibility):
 
 ```shell
-git clone --branch 2023-07-0.0.116-java-bindings https://github.com/TheBlueMatt/rust-lightning /path/to/rust-lightning
+git clone --branch 0.0.118-bindings https://github.com/lightningdevkit/rust-lightning /path/to/rust-lightning
 ```
 
 **[ldk-c-bindings](https://github.com/lightningdevkit/ldk-c-bindings)**:
 
 ```shell
-git clone --branch v0.0.116.0 https://github.com/lightningdevkit/ldk-c-bindings /path/to/ldk-c-bindings
+git clone --branch main https://github.com/lightningdevkit/ldk-c-bindings /path/to/ldk-c-bindings
 ```
 
 Take note of where you clone these directories, it's best you save the absolute path somewhere handy for the rest of the remaining steps.

diff --git a/ci/LDKSwift/Sources/LDKSwift/batteries/ChannelManagerConstructor.swift b/ci/LDKSwift/Sources/LDKSwift/batteries/ChannelManagerConstructor.swift
@@ -43,10 +43,10 @@ public struct ChannelManagerConstructionParameters {
     public var enableP2PGossip: Bool = false
     public var scorer: MultiThreadedLockableScore?
     public var scoreParams: ProbabilisticScoringFeeParameters?
-    public var payerRetries: Retry = Retry.initWithAttempts(a: UInt(3))
+    public var payerRetries: Retry = Retry.initWithAttempts(a: UInt32(3))
     public var logger: Logger
 
-    public init (config: UserConfig, entropySource: EntropySource, nodeSigner: NodeSigner, signerProvider: SignerProvider, feeEstimator: FeeEstimator, chainMonitor: ChainMonitor, txBroadcaster: BroadcasterInterface, logger: Logger, enableP2PGossip: Bool = false, scorer: MultiThreadedLockableScore? = nil, scoreParams: ProbabilisticScoringFeeParameters? = nil, payerRetries: Retry = Retry.initWithAttempts(a: UInt(3))) {
+    public init (config: UserConfig, entropySource: EntropySource, nodeSigner: NodeSigner, signerProvider: SignerProvider, feeEstimator: FeeEstimator, chainMonitor: ChainMonitor, txBroadcaster: BroadcasterInterface, logger: Logger, enableP2PGossip: Bool = false, scorer: MultiThreadedLockableScore? = nil, scoreParams: ProbabilisticScoringFeeParameters? = nil, payerRetries: Retry = Retry.initWithAttempts(a: UInt32(3))) {
         self.config = config
         self.entropySource = entropySource
         self.nodeSigner = nodeSigner
@@ -134,7 +134,7 @@ public class ChannelManagerConstructor: NativeTypeWrapper {
         var monitorFundingSet = Set<String>()
 
         for currentSerializedChannelMonitor in channelMonitorsSerialized {
-            let channelMonitorResult: Result_C2Tuple_BlockHashChannelMonitorZDecodeErrorZ = Bindings.readBlockHashChannelMonitor(ser: currentSerializedChannelMonitor, argA: params.entropySource, argB: params.signerProvider)
+            let channelMonitorResult = Bindings.readThirtyTwoBytesChannelMonitor(ser: currentSerializedChannelMonitor, argA: params.entropySource, argB: params.signerProvider)
 
             guard let (blockHash, channelMonitor) = channelMonitorResult.getValue() else {
                 throw InvalidSerializedDataError.invalidSerializedChannelMonitor
@@ -170,8 +170,8 @@ public class ChannelManagerConstructor: NativeTypeWrapper {
         // TODO: figure out better way to obtain a router
         let router = params.router(networkGraph: self.netGraph)
         let channelManagerReadArgs = ChannelManagerReadArgs(entropySource: params.entropySource, nodeSigner: params.nodeSigner, signerProvider: params.signerProvider, feeEstimator: params.feeEstimator, chainMonitor: params.chainMonitor.asWatch(), txBroadcaster: params.txBroadcaster, router: router, logger: params.logger, defaultConfig: params.config, channelMonitors: monitors)
-
-        guard let (latestBlockHash, channelManager) = Bindings.readBlockHashChannelManager(ser: channelManagerSerialized, arg: channelManagerReadArgs).getValue() else {
+        
+        guard let (latestBlockHash, channelManager) = Bindings.readThirtyTwoBytesChannelManager(ser: channelManagerSerialized, arg: channelManagerReadArgs).getValue() else {
             throw InvalidSerializedDataError.invalidSerializedChannelManager
         }
 
@@ -281,7 +281,7 @@ public class ChannelManagerConstructor: NativeTypeWrapper {
             let (outPoint, _) = currentChannelMonitor.getFundingTxo()
             print("watching channel")
             let monitorWatchResult = chainMonitorWatch.watchChannel(fundingTxo: outPoint, monitor: currentChannelMonitor)
-            if monitorWatchResult != .Completed {
+            if !monitorWatchResult.isOk() || monitorWatchResult.getValue() != .Completed {
                 Bindings.print("Some issue occurred with a chainMonitorWatch.watch_channel call: \(monitorWatchResult)", severity: .WARNING)
             }
             // monitorClone.cType?.is_owned = true
@@ -360,16 +360,16 @@ fileprivate class CustomChannelManagerPersister: Persister {
         self.handler = handler
         super.init()
     }
-
-    override func persistManager(channelManager: Bindings.ChannelManager) -> Bindings.Result_NoneErrorZ {
+    
+    override func persistManager(channelManager: Bindings.ChannelManager) -> Bindings.Result_NoneIOErrorZ {
         return self.handler.persistManager(channelManager: channelManager)
     }
-
-    override func persistGraph(networkGraph: Bindings.NetworkGraph) -> Bindings.Result_NoneErrorZ {
+    
+    override func persistGraph(networkGraph: Bindings.NetworkGraph) -> Bindings.Result_NoneIOErrorZ {
         return self.handler.persistGraph(networkGraph: networkGraph)
     }
-
-    override func persistScorer(scorer: Bindings.WriteableScore) -> Bindings.Result_NoneErrorZ {
+    
+    override func persistScorer(scorer: Bindings.WriteableScore) -> Bindings.Result_NoneIOErrorZ {
         return self.handler.persistScorer(scorer: scorer)
     }
 }