feat(send): node connecting UI

[pwltr]

Description

• show a “Syncing Node” overlay when opening Send if the node is not running yet
• defer validation checks until after sync

Linked Issues/Tasks

Closes #339

Screenshot / Video

Unified invoice:

Simulator.Screen.Recording.-.iPhone.17.-.2026-01-16.at.01.00.09.mov

QuickPay (deeplink):

Simulator.Screen.Recording.-.iPhone.17.-.2026-01-16.at.01.00.09.mov

QA Notes

Use a deeplink like

xcrun simctl openurl "iPhone 17" "bitcoin:bcrt1qmd722klk04yph86ky8jz9gvj6g8n9kjep9zj7d?amount=0.0000002&lightning=LNBCRT200N1P5H8H4HDQQNP4QDRM0Y4AT84E48QVDN8CSWVE204SF6FFRR3W3AK904GE3JSEGLP4GPP5VCSLQ6RPA2RHQJ40LG6ELHE4JA0548TNQZTX4UFSTKAJ8FAHKLASSP5E2EQTTN20PF2HV7EZQCDKY43DSCVLXXR5LC6E920GDR3GXJ0QYSQ9QYYSGQCQPCXQRRSSRZJQ29GJY9SQJRRP48TZ7HJ2E5VM4L2DUKC4CSF2MN6QM32U3HTED5LEAPYQQQQQQQF95QQQQLGQQQQQQGQ2Q9T08YP2026U697073LS2FRT0EY9MKEYQASUTM5DV9KV5FZS5X2UPPHLFYU4C3Q34LJ8GRTLQ2YG0SWPZJMSK74MV6WHWUEZWG870AHCQGD3XDW"

[piotr-iohk]

There seems to be a regression (from e2e send_2):

• sending unified invoice with onchain = 20 000 and lightning = 200 000 sats: bitcoin:bcrt1qrj2dcxqv2vm2mm94h6lpp6hgnp6hglfxadherq?amount=0.0002&lightning=lnbcrt2m1p5h03d0pp5sjdqh2xylewme5n3gwv9wqa3ua8v8x8esu9td4t4aee5azemgjmqdqqcqzzsxqyz5vqsp5ax0t68lz5fcrnz73rt439hg7sawfys8xlwfw9m6csmchy8dru77q9qxpqysgqddmkhflxt25kugzucn7jjxkqp6jkg9gjsjpyudysadv62ad3cfk3h4t7l4mzr92fnlywdn7vkxhckv7fffagrendvv00fqr9rc6x30gql76ghh
• should trigger tx from savings since balance on lightning exceeded.

Screen.Recording.2026-01-26.at.20.59.54.mov

[pwltr]

Updated and tested against Docs/SCAN_INVOICE_TEST_MATRIX.md

[pwltr]

@piotr-iohk Could you run e2e for this locally, please? Also note the test matrix in Docs/SCAN_INVOICE_TEST_MATRIX.md, would be nice if any missing test cases could be added from there too.

[piotr-iohk]

@piotr-iohk Could you run e2e for this locally, please? Also note the test matrix in Docs/SCAN_INVOICE_TEST_MATRIX.md, would be nice if any missing test cases could be added from there too.

CI E2E unblocked, but passed locally for me in the meantime. I will update e2e with missing cases (currently ~60% covered).

[jvsena42]

tAck

[piotr-iohk]

tACK

[pwltr]

Found a small bug, patching...

[claude]

Code review

After reviewing this PR for bugs and CLAUDE.md compliance, I found one issue:

---

Potential infinite wait state in SendSheet.swift

Location: Bitkit/Views/Wallets/Send/SendSheet.swift lines 197-200

Issue: If Lightning channels exist but never become usable (e.g., peer permanently offline, stuck channel state), users will be trapped in the sync overlay indefinitely with no escape.

The early return at lines 197-200 exits without setting hasValidatedAfterSync = true. The onChange(of: wallet.hasUsableChannels) handler only fires when the value changes. If channels remain permanently unusable, the validation never completes and the user cannot dismiss the sync overlay or proceed.

Suggested fix: Add a timeout mechanism or fallback logic:

// Option 1: Add timeout state
@State private var syncStartTime: Date?

// In .onAppear or when showing overlay:
syncStartTime = Date()

// In validatePaymentAfterSync:
let syncDuration = Date().timeIntervalSince(syncStartTime ?? Date())
if hasAnyChannels, !wallet.hasUsableChannels {
    if syncDuration > 30 { // 30 second timeout
        // Fall back to onchain or show "channels unavailable" error
        hasValidatedAfterSync = true
        // Proceed with validation or dismiss
    } else {
        return
    }
}

Or add a manual dismiss option in the sync overlay UI so users aren't trapped if something goes wrong.

[pwltr]

Can I get another review please @piotr-iohk @jvsena42, I appended a bug fix

[piotr-iohk]

@pwltr is this looks like something that needs to be addressed? #401 (comment)

[pwltr]

@pwltr is this looks like something that needs to be addressed? #401 (comment)

The comment is about channels never becoming usable, in that case the user will not be able to send any payments anyway. Can think about showing an error state but I'd consider that polishing for later.

[piotr-iohk]

LGTM, sanity tested with deeplinks.