Add UniFFI Kotlin bindings for payjoin-ffi by ram0verflow · Pull Request #1869 · payjoin/rust-payjoin

ram0verflow · 2026-09-08T18:05:08Z

Pull Request Checklist

Please confirm the following before requesting review:

I have disclosed my use of
AI
in the body of this PR.
I have read CONTRIBUTING.md and rebased my branch to produce hygienic commits.

What

UniFFI Kotlin/JVM bindings for payjoin-ffi, plus a BIP77 v2 round-trip
integration test.

Layout matches python/csharp:

payjoin-ffi/kotlin/scripts/generate_bindings.sh
payjoin-ffi/kotlin/contrib/test.sh
PAYJOIN_FFI_FEATURES (default _test-utils; empty string is passed
through) and PAYJOIN_FFI_PROFILE (default dev → target/debug)

Generated sources are not committed (gitignored under
kotlin/src/main/kotlin/org/). Generate is a build step.

CI runs inside nix develop .#kotlin (JDK 21, MSRV, nixpkgs bitcoind),
matching csharp/python/dart.

Unit tests (15) cover URIs, persistence, cancel, and validation — the
same count as the Python unit suite. The second commit adds the v2
integration test (16 Gradle tests on HEAD).

Why `[bindings.kotlin.rename]`

Generated UniFFI objects implement Disposable and AutoCloseable, so
close() drops the Rust handle. Payjoin also exports protocol close() on
ReceiverPendingFallback, SenderPendingFallback, and the four JSON
session persister traits. Kotlin cannot overload on return type.

The table is Kotlin-only. It maps those protocol methods to
closeSession(). No Rust API change. Other language bindings keep close.

Integration test

IntegrationTests.kt drives a full BIP77 v2 round trip, sender and
receiver, against the in-process directory, OHTTP relay, and a real
bitcoind. Under the kotlin nix shell, BITCOIND_EXE comes from nixpkgs
and BITCOIND_SKIP_DOWNLOAD=1. Outside the shell, corepc-node may
download Bitcoin Core into a temp regtest dir.

On Linux, v2ToV2Payjoin completes in about 4s.

TestHttp trusts only the directory's rcgen cert and proxies through the
OHTTP relay.

Wrapper pin

Gradle 9.1.0 wrapper is pinned with distributionSha256Sum.
validateDistributionUrl only checks the URL.

Release / publish

Not published anywhere. No pack, Maven, signing, or attestation jobs.

The workflow does not trigger on payjoin-kotlin-* tags. python.yml's
tag trigger runs wheel build, PyPI publish, and GitHub release assets.
Until Kotlin has something to publish, a tag trigger would only re-run
tests.

Follow-ups (out of this PR)

No ktfmt / treefmt registration for .kt
No Windows CI (same as python/dart/javascript)

Disclosure

This PR was written with Cursor Cloud Agent (Grok 4.6).

Code: Kotlin Gradle project, generate/test scripts, nix kotlin
shell, CI workflow, unit tests, and the v2 integration test were
AI-generated and then hand-reviewed against the Python suite and a
known-good round trip (~4s).
PR body: AI-drafted (this text).

Disclosure: co-authored by Cursor Grok 4.6

chavic · 2026-09-10T07:40:58Z

@ram0verflow Thanks, I'll take a look today

coveralls · 2026-09-10T14:38:12Z

Coverage Report for CI Build 34760182247

Coverage remained the same at 86.646%

Details

Coverage remained the same as the base build.
Patch coverage: No coverable lines changed in this PR.
No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.

Coverage Stats


Relevant Lines:	16549
Covered Lines:	14339
Line Coverage:	86.65%
Coverage Strength:	341.7 hits per line

💛 - Coveralls

chavic · 2026-09-10T15:04:31Z

@ram0verflow it looks like you're hitting an unrelated probelm, that's fixed on master, you want to rebase

chavic

A few comments below.

caarloshenriq

Concept ACK.

Kotlin/JVM bindings alongside the other four is a good addition, and the layout mirrors python/csharp closely enough that it should be easy to keep in sync

ram0verflow · 2026-09-10T16:39:09Z

will fix and update

chavic

A few more comments on the tests.

chavic

A few more comments.

Generate with in-tree uniffi-bindgen --language kotlin, siloed like Python (no extra cargo feature). Rename protocol close to closeSession in Kotlin only so AutoCloseable still works. Pin the Gradle 9.1.0 wrapper with distributionSha256Sum; validateDistributionUrl only checks the download URL. Add a kotlin nix dev shell (JDK 21, MSRV, bitcoind) and run CI inside it, matching the other language bindings. Unit tests cover URIs, persistence, cancel, and validation.

Drive a full BIP77 round trip against the in-process directory, OHTTP relay, and bitcoind, mirroring the Python suite's RPC sequence and assertions. TestHttp trusts only the self-signed rcgen directory cert so HTTPS through the relay succeeds in the JVM. Parse bitcoind RPC responses with kotlinx-serialization-json.

Honor CARGO_TARGET_DIR, load the OS-native library, and assert percent-encoded URIs, overflow variants, and persister close via Rust. Co-authored-by: Cursor <cursoragent@cursor.com>

Honor CARGO_TARGET_DIR and [build] target-dir instead of assuming ../target. Co-authored-by: Cursor <cursoragent@cursor.com>

Use the compiler-artifact filename so generate follows target-dir, CARGO_BUILD_TARGET, and custom profiles.

ram0verflow · 2026-09-13T14:00:49Z

updated

chavic

tACK

ram0verflow requested review from DanGould, benalleng and spacebear21 as code owners September 8, 2026 18:05

ram0verflow mentioned this pull request Sep 8, 2026

Java language bindings for payjoin-ffi #1860

Open

chavic requested review from chavic and removed request for DanGould, benalleng and spacebear21 September 10, 2026 14:30

chavic reviewed Sep 10, 2026

View reviewed changes

Comment thread payjoin-ffi/kotlin/build.gradle.kts

Comment thread payjoin-ffi/kotlin/src/test/kotlin/org/payjoindevkit/UriTests.kt Outdated

Comment thread payjoin-ffi/kotlin/CONTRIBUTING.md Outdated

caarloshenriq suggested changes Sep 10, 2026

View reviewed changes

Comment thread payjoin-ffi/kotlin/CONTRIBUTING.md Outdated

Comment thread payjoin-ffi/kotlin/README.md Outdated

Comment thread payjoin-ffi/kotlin/build.gradle.kts Outdated

ram0verflow force-pushed the kotlin-bindings-pr branch from a991880 to df8922e Compare September 10, 2026 17:21

chavic reviewed Sep 11, 2026

View reviewed changes

Comment thread payjoin-ffi/kotlin/scripts/generate_bindings.sh Outdated

Comment thread payjoin-ffi/kotlin/src/test/kotlin/org/payjoindevkit/IntegrationTests.kt Outdated

Comment thread payjoin-ffi/kotlin/build.gradle.kts Outdated

Comment thread .github/workflows/kotlin.yml

ram0verflow and others added 4 commits September 12, 2026 09:46

Address Kotlin bindings review feedback

7d3ee61

Compile production Kotlin bindings in CI and tighten FFI tests.

64a8082

Honor CARGO_TARGET_DIR, load the OS-native library, and assert percent-encoded URIs, overflow variants, and persister close via Rust. Co-authored-by: Cursor <cursoragent@cursor.com>

ram0verflow force-pushed the kotlin-bindings-pr branch from df8922e to 64a8082 Compare September 12, 2026 05:40

chavic reviewed Sep 12, 2026

View reviewed changes

Comment thread payjoin-ffi/kotlin/scripts/generate_bindings.sh Outdated

Resolve the Kotlin native lib path from cargo metadata.

3e025b2

Honor CARGO_TARGET_DIR and [build] target-dir instead of assuming ../target. Co-authored-by: Cursor <cursoragent@cursor.com>

chavic reviewed Sep 13, 2026

View reviewed changes

Comment thread payjoin-ffi/kotlin/scripts/generate_bindings.sh Outdated

chavic reviewed Sep 13, 2026

View reviewed changes

Comment thread payjoin-ffi/kotlin/scripts/generate_bindings.sh Outdated

Take the Kotlin native library path from cargo build JSON.

7c8ba18

Use the compiler-artifact filename so generate follows target-dir, CARGO_BUILD_TARGET, and custom profiles.

chavic approved these changes Sep 13, 2026

View reviewed changes

ram0verflow requested a review from caarloshenriq September 13, 2026 16:43

chavic merged commit 83a33c8 into payjoin:master Sep 13, 2026
57 checks passed

Uh oh!

Conversation

ram0verflow commented Sep 8, 2026

What

Why [bindings.kotlin.rename]

Integration test

Wrapper pin

Release / publish

Follow-ups (out of this PR)

Disclosure

Uh oh!

chavic commented Sep 10, 2026

Uh oh!

coveralls commented Sep 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report for CI Build 34760182247

Coverage remained the same at 86.646%

Details

Uncovered Changes

Coverage Regressions

Coverage Stats

💛 - Coveralls

Uh oh!

chavic commented Sep 10, 2026

Uh oh!

chavic left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

caarloshenriq left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ram0verflow commented Sep 10, 2026

Uh oh!

chavic left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

chavic left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ram0verflow commented Sep 13, 2026

Uh oh!

chavic left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Why `[bindings.kotlin.rename]`

coveralls commented Sep 10, 2026 •

edited

Loading