PDK Dev Call Updates - August 18th 2025 #969

arminsabouri · 2025-08-18T18:08:06Z

arminsabouri
Aug 18, 2025
Collaborator

Discovered with Ben an issue where receiver and sender sessions become starved if one session fails transiently or fatally.
Continued progress on the receiver decoupling effort Decoupling v2 receiver typestates from v1 typestates #951
On review duty for several open PRs, prioritizing Refactor sender to validate PjParam by version #901.
Made a series of smaller miscellaneous changes on the Liana front and rust-payjoin. No other constraints.

Fixed and merged a bug in Liana where sessions looped by session ID could cause failures if one session expired or crashed.
Identified unwrap and expect calls that triggered panics on expired sessions; the original POC incorrectly assumed expiration wouldn’t occur.
Upgraded MSRV to 1.85.0, enabling upgrades to rust-tls and other dependencies.
Encountered dependency issues: the ohttp-relay crate still uses older versions, causing compiler complaints. Plan is to upgrade ohttp-relay first, then address downstream dependencies. Nothingmuch offered to pair on this.
This week: fix additional panics in Liana and upgrade dependencies.
Discussed dependency management:
- We need to specify full x.y.z versions in Cargo.toml.
- Current lockfile handling isn’t consistent Cargo-minimal.lock and Cargo-recent.lock have the same contents. Is this right? #454
- Goal is to enforce complete versioning and resolve lockfile issues.
Observed flaky GitHub Actions tests due to failed downloads of test utility dependencies. Suggested ensuring the bitcoind binary is available in PATH or using some
Proposed bumping testcontainers to restore Dart integration tests, but API changes need to be addressed. Spacebear offered to help with the migration.

Experimented with rewriting the git history in the Dart-UniFFI repo.
Preparing a HashMap PR (?), pending history rewrite.
Getting familiar with the rust-payjoin codebase.
Investigating trait method support in UniFFI-Dart (Allow replacing output and inserting new outputs #69).

Focused on finishing the sender common abstraction PR Refactor sender to validate PjParam by version #901
Called for a feature freeze to move toward 1.0 now that the BIP is stable.
Next steps: stabilize existing integrations, which remain somewhat flaky.
Clarified that Payjoin-CLI changes can continue to flow outside the freeze.
Attended the Africa Bitcoin Conference.
Conducted risk assessment on directory infrastructure; emphasized need to distribute directory hosting.
Main priority: ship the next release as 1.0.0-alpha (milestone is open -- maybe we create a tracking issue as well).
Once Liana integration is usable, remaining integrations will follow quickly.
Investigating an issue with BB integration: OHTTP relay connections can hang for over 60 seconds. BB currently selects relays randomly, and connection timeout defaults to 60s. GitHub issues will be filed.

Last week focused on external commitments.
On the Redis removal front Replace redis with file based mailbox storage #915 : some architectural changes made, moving toward persisting all state on disk.

Spent last week on PR reviews, unblocking others.
This week: allocate time to address long-standing PRs open for over two months.
Resume work on Bull Bitcoin Dart integration, migrating from rust-bridge to UniFFI-Dart bindings.

Pushed additional fixes for the Sled-to-SQLite migration Migrate payjoin-cli from sled to rusqlite #873
Continued working on the system time handling issue
Collaborated with Nothingmuch on using Axum for routing directory reqs

Addressing pending PR for configuration saving payjoin-cli should have allow saving configs #896 and Prometheus metrics Introduce metrics Collection via prometheus for payjoin directory #848
Opened or opening PR to remove custom request size constants