Skip to content

Commit 250efec

Browse files
joostjagerclaude
andcommitted
Clarify that each pending monitor update ID must be marked complete
The previous wording implied that persisting a full ChannelMonitor would automatically resolve all pending updates. Reword to make clear that each update ID still needs to be individually marked complete via channel_monitor_updated, even after a full monitor persistence. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 98393b3 commit 250efec

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

lightning/src/chain/chainmonitor.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,10 @@ use core::sync::atomic::{AtomicUsize, Ordering};
8383
/// the background with [`ChainMonitor::list_pending_monitor_updates`] and
8484
/// [`ChainMonitor::get_monitor`].
8585
///
86-
/// Once a full [`ChannelMonitor`] has been persisted, all pending updates for that channel can
87-
/// be marked as complete via [`ChainMonitor::channel_monitor_updated`].
86+
/// Each pending update must be individually marked as complete by calling
87+
/// [`ChainMonitor::channel_monitor_updated`] with the corresponding update ID. Note that
88+
/// persisting a full [`ChannelMonitor`] covers all prior updates, but each update ID still
89+
/// needs to be marked complete separately.
8890
///
8991
/// If at some point no further progress can be made towards persisting the pending updates, the
9092
/// node should simply shut down.

0 commit comments

Comments
 (0)