Include failure context in splice events by jkczyz · Pull Request #4514 · lightningdevkit/rust-lightning

jkczyz · 2026-03-26T19:30:06Z

When a splice negotiation round fails, the user needs to know why it failed and what they can do about it. Previously, SpliceFailed gave no indication of the failure cause and didn't return the contribution, making it difficult to retry.

This PR adds failure context to the splice events so users can make informed retry decisions:

NegotiationFailureReason indicates what went wrong — peer disconnect, counterparty abort, invalid contribution, insufficient feerate, channel closing, etc. Each variant documents how to resolve it.
The FundingContribution from the failed round is returned in the event. Users can feed it back to funding_contributed to retry as-is, or inspect it to understand which inputs, outputs, and feerate were used when constructing a new contribution.
SplicePending and SpliceFailed are renamed to SpliceNegotiated and SpliceNegotiationFailed to reflect that each negotiation round (initial or RBF) independently resolves to one of these two outcomes.
DiscardFunding is now emitted before SpliceNegotiationFailed so wallet inputs are unlocked before the failure handler runs. Otherwise, a retry during SpliceNegotiationFailed handling would leave inputs locked, and the subsequent DiscardFunding would incorrectly unlock inputs committed to the new attempt.

Additionally, SpliceFundingFailed internals are simplified:

Dead funding_txo and channel_type fields are removed.
An unreachable was_negotiated check is removed from the contribution pop.
Inputs and outputs are derived from FundingContribution via a unified splice_funding_failed_for! macro, replacing the old approach of extracting them from FundingNegotiation variants. Unused conversion methods are removed.

Also fixes a bug in into_unique_contributions where outputs were compared by full TxOut equality instead of script_pubkey. This could fail to filter change outputs that reused the same address but had different values across RBF rounds.

ldk-reviews-bot · 2026-03-26T19:30:10Z

🎉 This PR is now ready for review!
Please choose at least one reviewer by assigning them on the right bar.
If no reviewers are assigned within 10 minutes, I'll automatically assign one.
Once the first reviewer has submitted a review, a second will be assigned if required.

jkczyz · 2026-03-26T19:42:35Z

lightning/src/events/mod.rs

+	/// sent an invalid message). Retry by calling [`ChannelManager::splice_channel`].
+	///
+	/// [`ChannelManager::splice_channel`]: crate::ln::channelmanager::ChannelManager::splice_channel
+	NegotiationError,


Note we could have this wrap AbortReason -- we had an internal NegotiationError struct that already wrapped this along with contributions -- but it would require making AbortReason public. Not sure if we want to expose all its variants.

jkczyz · 2026-03-26T19:44:38Z

lightning/src/ln/channel.rs

+	fn with_negotiation_failure_reason(mut self, reason: NegotiationFailureReason) -> Self {
+		match self {
+			QuiescentError::FailSplice(_, ref mut r) => *r = reason,
+			_ => debug_assert!(false, "Expected FailSplice variant"),


Little unfortunate we need to do this, but I couldn't find a better way other than passing NegotiationFailureReason to abandon_quiescent_action and quiescent_action_into_error, which doesn't seem right since they could be for future QuiescentAction variants.

jkczyz · 2026-03-26T19:50:17Z

lightning/src/events/mod.rs

-		/// The outpoint of the channel's splice funding transaction, if one was created.
-		abandoned_funding_txo: Option<OutPoint>,
-		/// The features that this channel will operate with, if available.
-		channel_type: Option<ChannelTypeFeatures>,


Didn't feel like there was much value to these.

abandoned_funding_txo would be set only if there's a failure after the transaction was negotiated but before signed.

In the future, when channel_type can change, maybe it is just part of FundingContribution? Could also keep it if you prefer.

ldk-reviews-bot · 2026-03-30T00:10:32Z

🔔 1st Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

ldk-reviews-bot · 2026-03-30T00:10:33Z

🔔 1st Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

ldk-reviews-bot · 2026-04-01T00:11:17Z

🔔 2nd Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

ldk-reviews-bot · 2026-04-01T00:11:18Z

🔔 2nd Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

ldk-claude-review-bot · 2026-04-02T17:19:19Z

lightning/src/ln/channel.rs

+impl QuiescentError {
+	fn with_negotiation_failure_reason(mut self, reason: NegotiationFailureReason) -> Self {
+		match self {
+			QuiescentError::FailSplice(_, ref mut r) => *r = reason,
+			_ => debug_assert!(false, "Expected FailSplice variant"),
+		}
+		self
+	}
 }


Minor: The debug_assert!(false) path is only reachable in non-production builds if a caller passes a non-FailSplice variant. Currently the only caller passes through quiescent_action_into_error which always produces FailSplice for production code (only DoNothing in test builds). But debug_assert!(false) will silently return self unchanged in release builds — consider using debug_assert! with a more specific message about which variant was unexpected, or restructuring so the method only accepts FailSplice to make this impossible.

See #4514 (comment)

ldk-claude-review-bot · 2026-04-02T17:19:28Z

lightning/src/ln/channel.rs

+		let is_initiator = pending_splice
+			.funding_negotiation
+			.take()
+			.map(|negotiation| negotiation.is_initiator())
+			.unwrap_or(false);


The is_initiator fallback when funding_negotiation is None defaults to false, which means splice_funding_failed_for! will return None for non-initiators (since the macro has None if !$is_initiator => None). This means if the funding negotiation was already taken before reset_pending_splice_state is called and the node was actually the initiator, the splice failure event would be silently dropped.

Is funding_negotiation ever None when reset_pending_splice_state is called? If so, defaulting to false may suppress failure events for the initiator. If not, this should be a debug_assert or .expect() to catch violations of that invariant.

Added debug assertions.

ldk-claude-review-bot · 2026-04-02T17:19:36Z

lightning/src/ln/channel.rs

+		let is_initiator = pending_splice
+			.funding_negotiation
+			.as_ref()
+			.map(|negotiation| negotiation.is_initiator())
+			.unwrap_or(false);


Same concern here: funding_negotiation.as_ref() being None causes is_initiator to default to false, which will cause the splice_funding_failed_for! macro to return None for non-initiators. Since maybe_splice_funding_failed is the read-only version used during serialization, silently returning None means the splice failure events won't be persisted and the user won't learn about the failure after restart.

Consider whether funding_negotiation can actually be None in this path, and if not, using expect to enforce the invariant.

Added debug assertions.

lightning/src/ln/funding.rs

ldk-claude-review-bot · 2026-04-02T17:20:30Z

I've completed a thorough re-review of the entire PR diff. All previously flagged issues have been verified and no significant new bugs were found.

Review Summary

No new bugs or security issues found beyond what was already flagged in the prior review pass.

Previously flagged issues (still applicable):

lightning/src/ln/channel.rs:3184 — with_negotiation_failure_reason silently no-ops in release for non-FailSplice variants.
lightning/src/ln/channel.rs:7267 — is_initiator defaults to false when funding_negotiation is None in reset_pending_splice_state, potentially suppressing initiator failure events if the invariant is violated in production.
lightning/src/ln/channel.rs:7317 — Same is_initiator defaulting issue in maybe_splice_funding_failed (serialization path).
lightning/src/ln/channel.rs:14020 — Duplicate quiescent action returns NegotiationFailureReason::Unknown because .with_negotiation_failure_reason() is not chained.

Previously flagged issues (now resolved):

Inconsistent DiscardFunding emission — all sites now use into_parts() with consistent if let Some(funding_info) guard.
Unnecessary .clone() on contribution — resolved by into_parts() consuming the struct.

Cross-cutting concerns:

Forward compatibility of NegotiationFailureReason: All 9 variant IDs are even (0, 2, 4, ..., 16). In impl_writeable_tlv_based_enum_upgradable!, unknown even variants return Err(UnknownRequiredFeature). Even though the reason field is read as upgradable_option, the error from an unknown even variant propagates through the ? operator in _decode_tlv!, failing the entire event deserialization and preventing ChannelManager loading. Future variants should use odd IDs for forward compatibility.
Changelog completeness: The changelog mentions feerate() and inputs() accessors on FundingContribution, but the PR also adds outputs(), change_output(), and value_added() as new public API methods that are not listed.

Each splice negotiation round can fail for different reasons, but Event::SpliceFailed previously gave no indication of what went wrong. Add a NegotiationFailureReason enum so users can distinguish failures and take appropriate action (e.g., retry with a higher feerate vs. wait for the channel to become usable). The reason is determined at each channelmanager emission site based on context rather than threaded through channel.rs internals, since the channelmanager knows the triggering context (disconnect, tx_abort, shutdown, etc.) while channel.rs functions like abandon_quiescent_action handle both splice and non-splice quiescent actions. The one exception is QuiescentError::FailSplice, which carries a reason alongside the SpliceFundingFailed. This is appropriate because FailSplice is already splice-specific, and the channel.rs code that constructs it (e.g., contribution validation, feerate checks) knows the specific failure cause. A with_negotiation_failure_reason method on QuiescentError allows callers to override the default when needed. Older serializations that lack the reason field default to Unknown via default_value in deserialization. The persistence reload path uses PeerDisconnected since a reload implies the peer connection was lost. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

jkczyz · 2026-04-03T01:25:47Z

Rebased

ldk-claude-review-bot · 2026-04-03T01:36:37Z

lightning/src/ln/channelmanager.rs

@@ -4128,6 +4118,16 @@ impl<
 								},


nit: Unlike handle_quiescent_error (line ~6619), this site unconditionally emits DiscardFunding even when contributed_inputs and contributed_outputs are both empty (which happens for the initiator when all inputs are committed to prior rounds). The same inconsistency exists at lines ~4417, ~4920, ~11853, ~12012, ~12180, ~12329, ~15448, and ~18122.

Consider adding the same guard here:

if !splice_funding_failed.contributed_inputs.is_empty() || !splice_funding_failed.contributed_outputs.is_empty() { // emit DiscardFunding }

Or, equivalently, have the macro return None for the initiator when there are no unique contributions (matching the non-initiator behavior) and emit SpliceNegotiationFailed without DiscardFunding in those cases.

Fixed by making SpliceFundingFailed consumable into Option-al parts rather than exposing fields. The into_parts method now checks for empty contributions.

lightning/src/ln/channelmanager.rs

codecov · 2026-04-03T02:29:54Z

Codecov Report

❌ Patch coverage is 84.69945% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.10%. Comparing base (2adb690) to head (c469423).

Files with missing lines	Patch %	Lines
lightning/src/ln/channelmanager.rs	79.61%	21 Missing ⚠️
lightning/src/ln/funding.rs	40.00%	3 Missing ⚠️
lightning/src/ln/channel.rs	96.55%	1 Missing and 1 partial ⚠️
lightning/src/events/mod.rs	85.71%	1 Missing ⚠️
lightning/src/ln/functional_test_utils.rs	90.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4514   +/-   ##
=======================================
  Coverage   87.09%   87.10%           
=======================================
  Files         163      163           
  Lines      108856   108776   -80     
  Branches   108856   108776   -80     
=======================================
- Hits        94808    94744   -64     
+ Misses      11563    11545   -18     
- Partials     2485     2487    +2

Flag	Coverage Δ
fuzzing	`40.25% <30.63%> (+0.05%)`	⬆️
tests	`86.21% <84.69%> (+<0.01%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…cence Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Unknown future variants (using odd IDs) will deserialize as Unknown instead of blocking ChannelManager loading. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Replace the abandoned_funding_txo and channel_type fields on Event::SpliceFailed with an Option<FundingContribution> from the failed round. Users can feed this back to funding_contributed to retry or use it to inform a fresh attempt via splice_channel. Also makes FundingContribution::feerate() public so users can inspect the feerate when deciding whether to retry or bump. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

jkczyz · 2026-04-03T15:34:00Z

Forward compatibility of NegotiationFailureReason: Uses impl_writeable_tlv_based_enum! with all-even variant IDs. A future variant added by a newer LDK version would cause UnknownRequiredFeature on deserialization, blocking ChannelManager loading on older versions. Consider impl_writeable_tlv_based_enum_upgradable! or documenting the constraint.

Changed to use impl_writeable_tlv_based_enum_upgradable and changed the reason from default_value to upgradable_option, defaulting to NegotiationFailureReason::Unknown.

… events Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Reverse the event ordering at all emission sites so that Event::DiscardFunding is emitted before Event::SpliceFailed. If the user retries the splice when handling SpliceFailed, the contributed inputs would still be locked. A subsequent DiscardFunding would then incorrectly unlock inputs that are now committed to the new attempt. Emitting DiscardFunding first avoids this by ensuring inputs are unlocked before any retry occurs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Rename Event::SplicePending to Event::SpliceNegotiated and Event::SpliceFailed to Event::SpliceNegotiationFailed. These names better reflect the per-round semantics: each negotiation attempt resolves to one of these two outcomes, independent of the overall splice lifecycle. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

The was_negotiated check is unnecessary because reset_pending_splice_state only runs when funding_negotiation is present, meaning on_tx_signatures_exchange hasn't been called yet. Since the feerate is only recorded in last_funding_feerate_sat_per_1000_weight during on_tx_signatures_exchange, the current round's feerate can never match it. So the contribution can always be unconditionally popped. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Filter outputs by script_pubkey rather than full TxOut equality. Outputs reusing the same address as a prior round are still considered committed even if the value differs (e.g., different change amounts across RBF rounds with different feerates). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Replace the maybe_create_splice_funding_failed! macro and splice_funding_failed_for method with a unified splice_funding_failed_for! macro that derives contributed inputs and outputs from the FundingContribution rather than extracting them from the negotiation state. Callers pass ident parameters for which PendingSplice filtering methods to use: contributed_inputs/contributed_outputs when the current round's contribution has been popped or was never pushed, and prior_contributed_inputs/prior_contributed_outputs for the read-only persistence path where the contribution is cloned instead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…gotiationContext Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…hods Now that splice_funding_failed_for! derives inputs and outputs from FundingContribution directly, remove the unused NegotiationError struct and into_negotiation_error methods from the interactive tx types, along with the into/to_contributed_inputs_and_outputs methods on ConstructedTransaction. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

ldk-reviews-bot · 2026-04-04T00:00:39Z

🔔 3rd Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

ldk-reviews-bot · 2026-04-04T00:00:40Z

🔔 3rd Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

ldk-reviews-bot · 2026-04-06T00:01:40Z

🔔 4th Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

ldk-reviews-bot · 2026-04-06T00:01:44Z

🔔 4th Reminder

Hey @TheBlueMatt @wpaulino! This PR has been waiting for your review.
Please take a look when you have a chance. If you're unable to review, please let us know so we can find another reviewer.

TheBlueMatt · 2026-04-03T18:31:39Z

lightning/src/events/mod.rs

+	/// [`ChannelManager::splice_channel`], or wait for the counterparty to initiate.
+	///
+	/// [`ChannelManager::splice_channel`]: crate::ln::channelmanager::ChannelManager::splice_channel
+	CounterpartyAborted,


Is the "Message data" in tx_abort not a string that we can include here?

TheBlueMatt · 2026-04-03T18:32:01Z

lightning/src/events/mod.rs

+	///
+	/// [`ChannelManager::splice_channel`]: crate::ln::channelmanager::ChannelManager::splice_channel
+	PeerDisconnected,
+	/// The counterparty aborted the negotiation by sending `tx_abort`. Retry by calling


Almost each docstring says "Retry by calling splice_channel". That doesn't seem like particularly helpful advice especially in cases where the counterparty rejected implying just trying again is unlikely to fix it?

TheBlueMatt · 2026-04-03T19:55:13Z

lightning/src/events/mod.rs

+	/// Adjust the contribution and retry via [`ChannelManager::splice_channel`].
+	///
+	/// [`ChannelManager::splice_channel`]: crate::ln::channelmanager::ChannelManager::splice_channel
+	ContributionInvalid,


It does seem like some of these could use more details (error message string?). As-is given the recommendation for ~all enum variants is "retry by calling splice_channel again" its not really clear to me why we should bother with an enum - an enum is useful if the downstream code has different handling for different cases, but here that doesn't really seem likely/possible. If we can't expose more details that are useful for automated action, we could probably do with an enum with three variants and a developer-readable string.

TheBlueMatt · 2026-04-06T00:52:31Z

lightning/src/ln/channelmanager.rs

 						if let Some(splice_funding_failed) = splice_funding_failed {
 							let (funding_info, contribution) = splice_funding_failed.into_parts();
 							let mut pending_events = self.pending_events.lock().unwrap();
+							if let Some(funding_info) = funding_info {


We should document which comes first and maybe test it more exhaustively - eg in get_and_clear_pending_events assert the order if both events appear for the same channel?

jkczyz self-assigned this Mar 26, 2026

jkczyz added this to Weekly Goals Mar 26, 2026

jkczyz force-pushed the 2026-03-splice-rbf-fail-event branch 2 times, most recently from 333980d to c2cad50 Compare March 28, 2026 00:08

jkczyz requested review from TheBlueMatt and wpaulino March 28, 2026 00:10

jkczyz commented Mar 28, 2026

View reviewed changes

jkczyz marked this pull request as ready for review April 2, 2026 17:04

ldk-claude-review-bot reviewed Apr 2, 2026

View reviewed changes

lightning/src/ln/funding.rs Show resolved Hide resolved

jkczyz force-pushed the 2026-03-splice-rbf-fail-event branch from c2cad50 to c469423 Compare April 3, 2026 01:25

ldk-claude-review-bot reviewed Apr 3, 2026

View reviewed changes

lightning/src/ln/channelmanager.rs Outdated Show resolved Hide resolved

jkczyz and others added 2 commits April 2, 2026 22:13

f - Fix broken rustdoc links in NegotiationFailureReason

1035615

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

f - Add TODO about NegotiationFailureReason layering in propose_quies…

f320e58

…cence Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

jkczyz force-pushed the 2026-03-splice-rbf-fail-event branch from c469423 to 4f6b1a6 Compare April 3, 2026 13:42

jkczyz and others added 3 commits April 3, 2026 10:14

f - Use upgradable serialization for NegotiationFailureReason

47bd416

Unknown future variants (using odd IDs) will deserialize as Unknown instead of blocking ChannelManager loading. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

f - Fix FundingTemplate rustdoc link path

b51a7d5

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

f - Destructure SpliceFundingFailed to avoid unnecessary clone

36c511b

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

jkczyz force-pushed the 2026-03-splice-rbf-fail-event branch from 4f6b1a6 to 0609565 Compare April 3, 2026 15:33

jkczyz and others added 12 commits April 3, 2026 10:39

f - Add SpliceFundingFailed::into_parts to avoid empty DiscardFunding…

f707945

… events Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

f - Remove dead funding_txo and channel_type from SpliceFundingFailed

2fd414e

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

f - Update changelog to use SpliceNegotiationFailed name

e34b3ac

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

f - Add debug_asserts for funding_negotiation invariant

3ee03ae

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

f - Remove dead contributed_inputs_and_outputs methods from FundingNe…

8b0148a

…gotiationContext Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Add pending changelog for splice negotiation event changes

44ecc06

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

jkczyz force-pushed the 2026-03-splice-rbf-fail-event branch from 0609565 to 44ecc06 Compare April 3, 2026 15:39

TheBlueMatt reviewed Apr 6, 2026

View reviewed changes

Conversation

jkczyz commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ldk-reviews-bot commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ldk-reviews-bot commented Mar 30, 2026

Uh oh!

ldk-reviews-bot commented Mar 30, 2026

Uh oh!

ldk-reviews-bot commented Apr 1, 2026

Uh oh!

ldk-reviews-bot commented Apr 1, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ldk-claude-review-bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review Summary

Previously flagged issues (still applicable):

Previously flagged issues (now resolved):

Cross-cutting concerns:

Uh oh!

jkczyz commented Apr 3, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

codecov bot commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

jkczyz commented Apr 3, 2026

Uh oh!

ldk-reviews-bot commented Apr 4, 2026

Uh oh!

ldk-reviews-bot commented Apr 4, 2026

Uh oh!

ldk-reviews-bot commented Apr 6, 2026

Uh oh!

ldk-reviews-bot commented Apr 6, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

jkczyz commented Mar 26, 2026 •

edited

Loading

ldk-reviews-bot commented Mar 26, 2026 •

edited

Loading

ldk-claude-review-bot commented Apr 2, 2026 •

edited

Loading

codecov bot commented Apr 3, 2026 •

edited

Loading