Merge pull request #106 from TheBlueMatt/main

0.0.109.0

Update CI branches to 0.0.109

[TS] Update auto-generated TS bindings

[Java] Update auto-generated Java bindings

[Java] Update tests to match 0.0.109 API

Merge pull request #104 from TheBlueMatt/main

Fix TS u64 call semantics (and test it)

[TS] Explicitly include `index.d.mts` (and `.mjs`) in package

As apparently npm doesn't include it just because its listed in
`types`.

[TS] Auto-update version in package.json in genbindings.sh

[TS] Update auto-generated typescript bindings

[TS] Add a test that uses `PeerManager` to exchange channel open

[TS] Fix typo in inconsistent-versions error message

[TS] Free arrays passed as slices from TS -> C -> Rust

[TS] Split C -> JS function calls based on u32/u64 parameters/return

We were calling all C -> JS functions with u32 parameters and
return types. This is fine for all of our pointers, as well as any
smaller types - u32 always gets mapped to a JavaScript `number`, so
its all consistent.

However, for u64 parameters/returns, we map the values to
JavaScript `bigint`s, which are not compatible with `numbers`, and
the correct type is checked at the FFI when returning or when users
ultimately try to use a passed `bigint` as if it were a `number`.

Thus, we have to split the `js_invoke_function` family by the
parameters and return values, using `u` for u32s and `b` for u64s
to do so.

Merge pull request #103 from TheBlueMatt/main

0.0.108.0

Update CI install to match latest debian bookworm state

Update CI MacOS clang version to 14.0.5

Update CI cbindgen install script to work with MSRV

Update CI bindings branches and tags to 0.0.108

[TS] Update auto-generated typescript bindings

[TS] Update tests to new upstream API changes in 0.0.108

[TS] Link wasm32-wasi libc as rustc no longer does by default

[Java] Update auto-generated Java bindings for 0.0.108

[Java] Handle different ldk lib paths in the ar in genbindings.sh

[Java] Use public channels and test Access interface in tests

[Java] Update batteries and tests to 0.0.108

This is mostly pretty obvious changes to keep up with the latest
API, though it also includes a change to make `PeerTest` work
correctly in a world where `PeerManager` supports multithreaded
parallel access.

[Java] Do not put a org.ldk.structs prefix for usize types

[Java] Map `LDKReadOnly*` the same as `LDKLocked*` ie via `AutoClosable`

Pass unnamed enum field comments through to field processing

Add handling for the new WitnessVersion type, similar to u5 but 4-bits

Use fully-qualified types more often in conversions

This ensures to-human conversions still work correctly inside
enums where there is an enum variant named identically to the type
which is being converted to (eg in the new `GraphSyncError` enum).

Merge pull request #100 from TheBlueMatt/main

Add note about garbage collection of the chan man constructor

Add note about garbage collection of the chan man constructor

Merge pull request #98 from tnull/2022-05-fix-genbindings-aarch64

Some fixes for aarch64 macos

Some fixes for aarch64 macos

Merge pull request #97 from TheBlueMatt/main

[Java] Fix PATH order to use upstream clang in OSX CI

[Java] Fix PATH order to use upstream clang in OSX CI

Thanks to @tnull for spotting this one.

Merge pull request #96 from TheBlueMatt/main

[Java] Correct trivial race condition in HumanObjectPeerTest

Update ts bindings version to 0.0.106.0beta1

[Java] Correct trivial race condition in HumanObjectPeerTest

On my (very slow) old armv7 device, HumanObjectPeerTest can fail as
`maybe_exchange_peer_messages` will set `ran = false` *after* a
message is added to the runqueue, but before it runs, then when we
call `process_events` there will be no events to process, then
before we get into the `runqueue` `synchronized` block the message
will be delivered, causing us to think we're done, even though
there are now events pending. Because going around the loop twice
is free, we should just not set `ran` to false at the top of
`maybe_exchange_peer_messages`.

Merge pull request #95 from TheBlueMatt/main

Convert primitives before passing to host language

Update OSX CI clang to 14.0.1

[TS] Update auto-generated bindings

[Java] Update auto-generated bindings

Convert primitives before passing to host language

This resolves an issue where we map unsigned primitives to their
signed variants, and occasionally pass the unsigned variant (which
is outside the bounds of the signed variant) to the host language.

In Java this can cause a panic as the VM sees an int outside the
range of a short (for example with network ports).

Merge pull request #94 from TheBlueMatt/main

Fix spurious inclusion of the OSX leaktracking bins in the release jar

Upload OSX CI builds for analysis to debug nondeterminism

Update clang to 14.0.0 (sadly 14.0.1 darwin binaries are not yet up)

Update CI branch to 0.0.106

Pin cbindgen in CI as 0.21 broke the MSRV

[TS] Remove checked-in WASM binary - it should be in ldk-java-bins

[TS] Update tests to latest upstream API

[TS] Update auto-generated bindings

[TS] Implement trait_struct_inc_refcnt properly (identical to java)

[Java] Update auto-generated bindings

[Java] Update LDK batteries to latest upstream API

[Java] Update tests to latest upstream API

Add a new manual type - UnqualifiedError/LDKError

Fully-Qualify Paths in some contexts instead of assuming enums

Instead of assuming enum variants that have a name the same as a
field's type, actually build a (simple) qualified-name method and
start using it.

Merge pull request #91 from TheBlueMatt/main

[Java] Correct the checked-in bindings to match generated ones

[Java] Correct the checked-in bindings to match generated ones

Merge pull request #90 from TheBlueMatt/main

Expose ProbabilisticScorer

[Java] Link libmath for `log10` (REVERT FOR 0.0.106)

[TS/CI] Update dependent package list

[TS] Update auto-generated bindings

[Java] Update auto-generated bindings to include ProbabilisticScorer

[Java] Use ProbabilisticScorer in HumanObjectPeerTest

Don't assume from-var conversion in opaque ret conv

When we do a ret-conv of an opaque struct, don't assume that the
variable as-parsed exists, instead use the variable that's being
created in the ret-conv string if we need to clone.

[Java] Make HumanObjectPeerTest status print a callback

...to make it easier to improve the Android LDKTester app

Merge pull request #88 from TheBlueMatt/main

Fix 32-bit Java builds

[TS] Update auto-generated C bindings with latest changes

[Java] Update auto-generated bindings with latest changes

Make pointer integer type language-dependent, instead of intptr_t

This is more consistent as we already did this in some places and
resolves issues on 32-bit Java platforms (ie Android ARMv7)
introduced in b0ba503b308bcdcbf1b7609876705c12408e44e6 and
071b5fa40465132896ffca3c8ad29d0374e983d2.

[Android] Drop i686 as a build target

I don't believe i686 phones ever existed, and emulators should
prefer x86_64 instead.

Merge pull request #86 from TheBlueMatt/main

Update to 0.0.105.0

Update CI to the ldk-c-bindings 0.0.105 branch

Bump clang to 13.0.1 on OSX in CI

[TS] Update auto-generated bindings for 0.0.105.0

[TS] Handle UInt5s and arrays of UInt5s

[Java] Update HumanObjectPeerTest to test Scorer serialization

This doesn't appear to change much, but it seems nicer to.

As of this commit, the total leaks during a full Java test run are
    25 allocations remained for 1115532 bytes.

[Java] Update auto-generated bindings to 0.0.105.0

[Java] Add a reachabilityFence on underlying trait impl calls

Because the implementation instance is implicitly held as a
reference in the `held` field, it seems unlikely this could ever
matter, hwoever, in the simplified implementation in `PeerTest`
(where there is no Human instance to hold a reference to the
underlying bindings instance), we do see the implementation being
free'd while its still in use. Thus, to ensure we can't hit any
such issue in the future, we add one in normal human objects as
well.

[Java] Update tests + batteries to latest upstream API

[Java] Handle UInt5 array conversion

Correct handling of arrays of 32-byte-array equivalents (Payment*)

Properly match warning comments, not structs with "Warning"

Merge pull request #82 from TheBlueMatt/main

TypeScript Package cleanups and Java socket closure

[TS] Fix CI when we log while the browser is shutting down

[Java] Close all sockets cleanly on `NioPeerHandler::interrupt`

[Java] More carefully ensure sockets are closed in `NioPeerHandler`

There is no indication that the sockets are leaking in the handling
loop, but its good to be careful and ensure we always both close
the SocketChannel and cancel the key registration.

During connection, however, it appears we may leak a SocketChannel
if the connection times out, leaking a file descriptor at least
until the GC kicks in and cleans up after us.  Here we are more
careful.

[Java] Stop processing net before we stop the background processor

See https://github.com/lightningdevkit/rust-lightning/issues/1237A

Fixes #77.

[TS] Copy README.md instead of symlinking, so npm publish accepts it

[TS] Add a types tag in package.json to point to TS types

Merge pull request #81 from TheBlueMatt/main

[TS] Unify Web + Node Codebases (with different init functions)

[TS] Bump package version to 0.0.104.1alpha4

Update auto-generated TypeScript files (now universal Node+Web!)

[TS] Unify Web and Node.JS output so it can be universal

Merge pull request #80 from TheBlueMatt/main

Fix TypeScript CI + Package Missing FIles

[TS] Bump package version to 0.0.104.1alpha3

[TS] Include missing version.mjs file