Skip to content

fix: expose probe route fee - #1083

Merged
pwltr merged 1 commit into
masterfrom
fix/probe-route-fee
Jul 16, 2026
Merged

fix: expose probe route fee#1083
pwltr merged 1 commit into
masterfrom
fix/probe-route-fee

Conversation

@pwltr

@pwltr pwltr commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Expose routeFeeMsat in debug probe devtools responses.
  • Include the route fee for both successful and failed probe outcomes.
  • Enables e2e probe reports to consume the route fee already captured by the Android probe outcome path.

Testing

  • Ran ./gradlew :app:compileMainnetDebugKotlin successfully.

@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown

Greptile Summary

This PR exposes probe route fees through debug devtools responses. The main changes are:

  • Adds nullable routeFeeMsat fields to successful and failed probe results.
  • Copies the existing route fee from both probe outcome variants.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • Both outcome variants preserve the existing ULong? type and null behavior.

Important Files Changed

Filename Overview
app/src/debug/java/to/bitkit/dev/DevToolsProvider.kt Adds the existing nullable probe route fee to both serialized devtools result variants.

Reviews (1): Last reviewed commit: "fix: expose probe route fee" | Re-trigger Greptile

@pwltr
pwltr requested a review from piotr-iohk July 15, 2026 15:09
@pwltr
pwltr merged commit e85b33b into master Jul 16, 2026
45 of 49 checks passed
@pwltr
pwltr deleted the fix/probe-route-fee branch July 16, 2026 08:24
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.

2 participants