Skip to content

[GEN-1815] Create FF for coin selection closest strategy - #562

Open
imclvr wants to merge 1 commit into
mainfrom
feat/improve-coin-selection
Open

[GEN-1815] Create FF for coin selection closest strategy#562
imclvr wants to merge 1 commit into
mainfrom
feat/improve-coin-selection

Conversation

@imclvr

@imclvr imclvr commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

FF for enabling new utxo selection from NG.


Stack created with GitHub Stacks CLIGive Feedback 💬

@imclvr
imclvr requested review from Jossec101, RodriFS and daliclovr and a lite review from Copilot August 12, 2026 12:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an environment-driven feature flag to enable/disable a new “closest amount” UTXO coin selection strategy from NodeGuard configuration, with local dev defaults set to disabled.

Changes:

  • Introduces ENABLE_COIN_SELECTION_BY_CLOSEST feature flag in Constants and wires it to ENABLE_COIN_SELECTION_BY_CLOSEST environment variable parsing.
  • Adds the new environment variable (default "false") to local launch configurations for both dotnet and VS Code.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/Properties/launchSettings.json Adds ENABLE_COIN_SELECTION_BY_CLOSEST=false to local dev environment variables.
src/Helpers/Constants.cs Defines and initializes the new feature flag from environment variables.
.vscode/launch.json Adds ENABLE_COIN_SELECTION_BY_CLOSEST=false to VS Code debug environment variables.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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