Skip to content

Bump LDK dependency for splice feerate API refactor #859

Merged
tnull merged 1 commit intolightningdevkit:mainfrom
jkczyz:2026-03-splicing-rbf-refactor
Apr 8, 2026
Merged

Bump LDK dependency for splice feerate API refactor #859
tnull merged 1 commit intolightningdevkit:mainfrom
jkczyz:2026-03-splicing-rbf-refactor

Conversation

@jkczyz
Copy link
Copy Markdown
Contributor

@jkczyz jkczyz commented Mar 30, 2026

The upstream LDK API moved the min/max feerate parameters from ChannelManager::splice_channel to FundingTemplate::splice_in and FundingTemplate::splice_out. This associates the feerate range with the funding contribution rather than the splice initiation, allowing different feerates per contribution (e.g., for RBF bumps). Update the splice_in_inner and splice_out call sites accordingly.

@ldk-reviews-bot
Copy link
Copy Markdown

ldk-reviews-bot commented Mar 30, 2026

👋 Thanks for assigning @tnull as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

@jkczyz jkczyz self-assigned this Apr 2, 2026
@tnull
Copy link
Copy Markdown
Collaborator

tnull commented Apr 8, 2026

Alright, #853 landed, so this should be good for a rebase now @jkczyz.

The upstream LDK API moved the min/max feerate parameters from
`ChannelManager::splice_channel` to `FundingTemplate::splice_in` and
`FundingTemplate::splice_out`. This associates the feerate range with
the funding contribution rather than the splice initiation, allowing
different feerates per contribution (e.g., for RBF bumps). Update
the splice_in_inner and splice_out call sites accordingly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jkczyz jkczyz force-pushed the 2026-03-splicing-rbf-refactor branch from 5499175 to c0f6fb9 Compare April 8, 2026 14:25
@jkczyz jkczyz marked this pull request as ready for review April 8, 2026 14:26
@jkczyz jkczyz requested a review from tnull April 8, 2026 14:26
Copy link
Copy Markdown
Collaborator

@tnull tnull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@tnull tnull merged commit fe692f3 into lightningdevkit:main Apr 8, 2026
19 checks passed
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.

3 participants