Skip to content

cmd/loop: drop misleading static loop-in fee warning#1125

Merged
hieblmi merged 1 commit intolightninglabs:masterfrom
hieblmi:correct-static-text
Apr 23, 2026
Merged

cmd/loop: drop misleading static loop-in fee warning#1125
hieblmi merged 1 commit intolightninglabs:masterfrom
hieblmi:correct-static-text

Conversation

@hieblmi
Copy link
Copy Markdown
Collaborator

@hieblmi hieblmi commented Apr 20, 2026

No description provided.

@gemini-code-assist
Copy link
Copy Markdown

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request cleans up the user interface by removing a potentially confusing warning message regarding on-chain fees for static address loop-ins. The changes ensure that the command output is more concise and accurate, while also updating the documentation and adding a test to prevent regressions.

Highlights

  • Removed misleading warning: Removed the static loop-in fee warning message from the command line output as it was deemed misleading.
  • Updated documentation: Updated the regtest README to reflect the removal of the fee warning message in the command output examples.
  • Added regression test: Introduced a new test case to verify that the fee warning message is no longer displayed during static loop-in operations.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@hieblmi hieblmi requested a review from starius April 20, 2026 11:13
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes redundant informational messages regarding on-chain fees for static address loop-ins from the CLI output and updates the documentation to match. It also introduces a new test suite to verify the display logic. Feedback was provided to improve the test's resource management by ensuring all pipe writers are closed within the cleanup function to prevent potential leaks or hangs.

Comment thread cmd/loop/main_test.go Outdated
@hieblmi hieblmi requested a review from alexbosworth April 20, 2026 14:28
@hieblmi hieblmi force-pushed the correct-static-text branch from e14ad47 to 7669abe Compare April 20, 2026 14:31
@hieblmi hieblmi requested a review from bhandras April 23, 2026 06:03
Copy link
Copy Markdown
Collaborator

@starius starius left a comment

Choose a reason for hiding this comment

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

LGTM! 💯

Comment thread cmd/loop/main_test.go Outdated
"Previously deposited on-chain: 250000 sat")
require.Contains(t, output,
"Estimated total fee: 386 sat")
}
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.

Do we really need a test for this specific change? We don't have CLI tests overall yet. When #1072 is ready, we can just add it there as a recorded session.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Removed it.

@hieblmi hieblmi force-pushed the correct-static-text branch from 7669abe to 08e94a7 Compare April 23, 2026 17:20
@hieblmi hieblmi force-pushed the correct-static-text branch from 08e94a7 to b5d2677 Compare April 23, 2026 17:22
@hieblmi hieblmi merged commit 787519e into lightninglabs:master Apr 23, 2026
13 checks passed
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.

4 participants