Skip to content

fix: tune lightning probe routing#623

Merged
pwltr merged 2 commits into
masterfrom
feat/scorer-params
Jul 14, 2026
Merged

fix: tune lightning probe routing#623
pwltr merged 2 commits into
masterfrom
feat/scorer-params

Conversation

@pwltr

@pwltr pwltr commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Description

Tunes Lightning scorer fee parameters to improve route selection reliability.

  • Applies custom scorer fee parameters during LDK node setup.
  • Keeps unspecified scorer fields from the default LDK config.
  • Stores probe failure shortChannelId as a string for diagnostic display.

Android counterpart: synonymdev/bitkit-android#1052

@greptile-apps

greptile-apps Bot commented Jul 14, 2026

Copy link
Copy Markdown

Greptile Summary

This PR tunes Lightning probe routing behavior. The main changes are:

  • Custom scorer fee parameters during Lightning node setup.
  • Default scorer fields preserved for unspecified values.
  • Probe failure short channel IDs stored as strings for display.
  • A changelog entry for route selection reliability.

Confidence Score: 4/5

Lightning setup can crash when the default scorer parameters are absent.

  • The new setup call always builds scorer parameters.
  • That builder path force-unwraps config.scoringFeeParams.
  • The probe short channel ID string change is localized to diagnostic display.

Bitkit/Services/LightningService.swift

Important Files Changed

Filename Overview
Bitkit/Services/LightningService.swift Adds scorer fee tuning during node setup, with a crash risk from force-unwrapping optional default scorer parameters.
Bitkit/ViewModels/WalletViewModel.swift Changes probe failure short channel ID handling from numeric storage to string storage for diagnostic output.
Bitkit/Views/Settings/ProbingTool/ProbingToolScreen.swift Displays the string short channel ID directly in the probe failure message.
changelog.d/next/623.fixed.md Adds a fixed changelog note for improved Lightning route selection reliability.

Reviews (1): Last reviewed commit: "fix: tune lightning probe routing" | Re-trigger Greptile

Comment thread Bitkit/Services/LightningService.swift Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7788d3e637

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread Bitkit/Services/LightningService.swift Outdated
@pwltr pwltr requested a review from piotr-iohk July 14, 2026 11:11

@piotr-iohk piotr-iohk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

utACK

@pwltr pwltr enabled auto-merge (squash) July 14, 2026 11:36
@jvsena42 jvsena42 added this to the 2.4.0 milestone Jul 14, 2026
@pwltr pwltr merged commit c9b3161 into master Jul 14, 2026
11 checks passed
@pwltr pwltr deleted the feat/scorer-params branch July 14, 2026 13:58
@greptile-apps greptile-apps Bot mentioned this pull request Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants