Skip to content

Bump 0.17.3 rc.0#2215

Merged
thesimplekid merged 4 commits into
cashubtc:v0.17.xfrom
thesimplekid:bump_0.17.3-rc.0
Jul 8, 2026
Merged

Bump 0.17.3 rc.0#2215
thesimplekid merged 4 commits into
cashubtc:v0.17.xfrom
thesimplekid:bump_0.17.3-rc.0

Conversation

@thesimplekid

Copy link
Copy Markdown
Collaborator

Description


Notes to the reviewers


Suggested CHANGELOG Updates

CHANGED

ADDED

REMOVED

FIXED


Checklist

  • I followed the code style guidelines
  • I ran just quick-check before committing
  • If the Wallet API was modified (added/removed/changed), I have reflected those changes in the FFI bindings (crates/cdk-ffi)

Map the internal 0 expiry sentinel back to None when converting Bolt12
mint quotes to API responses, matching the onchain conversion behavior.

Add regression coverage for direct and generic Bolt12 quote response paths.
NUT-17 notification payloads are ambiguous without the subscription
kind. Quote response shapes overlap and tolerate unknown fields, so
shape-based decoding can select the wrong payload variant.

Add a kind-aware payload decoder and use it in the wallet subscription
client while preserving the existing generic payload deserialization
surface for backports.

This prevents onchain and custom quote notifications from being decoded
as the wrong payload variant.
Detect swap sagas whose inputs are already spent and whose outputs are signed
before running setup compensation. Recovery deletes only the orphaned saga record
for those finalized swaps, avoiding repeated startup failures while preserving
spent proofs and signed outputs.

Add a regression test that reinserts a completed swap saga and verifies restart
cleanup leaves the finalized swap state intact.
@github-project-automation github-project-automation Bot moved this to Backlog in CDK Jul 8, 2026
@thesimplekid thesimplekid added this to the 0.17.3 milestone Jul 8, 2026
@thesimplekid thesimplekid merged commit 5800fa4 into cashubtc:v0.17.x Jul 8, 2026
47 of 50 checks passed
@thesimplekid thesimplekid deleted the bump_0.17.3-rc.0 branch July 8, 2026 09:48
@github-project-automation github-project-automation Bot moved this from Backlog to Done in CDK Jul 8, 2026
@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.58317% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.12%. Comparing base (5f19a66) to head (5800fa4).
⚠️ Report is 4 commits behind head on v0.17.x.

Files with missing lines Patch % Lines
crates/cashu/src/nuts/nut17/mod.rs 89.30% 34 Missing ⚠️
crates/cdk/src/mint/start_up_check.rs 93.45% 7 Missing ⚠️
crates/cdk-common/src/mint.rs 98.03% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           v0.17.x    #2215      +/-   ##
===========================================
+ Coverage    72.00%   72.12%   +0.11%     
===========================================
  Files          359      359              
  Lines        76239    76684     +445     
===========================================
+ Hits         54896    55308     +412     
- Misses       21343    21376      +33     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant