bark: Don't auto-exit in the LN receive claim code
- check or remove this line after you've added a CHANGELOG entry or if your PR doesn't need one (they should go into
CHANGELOG/unreleased/)
Sorry guys, missed this one, Nitesh encountered in and spent more sats than the VTXO was worth.
Summary by CodeRabbit
-
Bug Fixes
- Refreshes receive state after persisting preimage reveal so failure handling observes the updated status; when claim retries are exhausted, receives remain pending (preimage revealed, no automatic on‑chain exit) and normal failure handling runs.
-
Tests
- Updated tests to exercise zero‑retry/fail‑fast and exhausted‑retry scenarios, asserting receives stay pending until an explicit on‑chain exit is triggered.
-
Documentation
- Changelog entry clarifies new pending behavior and documents explicit on‑chain exit as a fallback.
Edited by coderabbit