Skip to content

Exclude uncategorized channels from fee management and rebalancing - #605

Merged
markettes merged 1 commit into
mainfrom
feat/uncategorized-not-automated
Sep 14, 2026
Merged

markettes merged 1 commit into
mainfrom
feat/uncategorized-not-automated

Conversation

@markettes

Copy link
Copy Markdown
Contributor

The engine now leaves channels alone until it has a committed flow verdict on them.
The fee optimizer filters them in OptimizeNode and writes no ChannelFeeState, so the
first managed cycle still seeds from the category baseline. The rebalancer drops them
in AutoRebalanceJob before building signals, so they are neither drained nor counted
toward a destination peer. Idle is a real verdict and stays fully managed.

Worth flagging explicitly in the PR body, since it's a deliberate trade rather than an oversight: filtering the channels out also hides their balance from the destination aggregate, so a peer with a mix of categorized and uncategorized channels can be over-refilled. It's pinned by Execute_MixedPeer_OverRefillsBecauseUncategorizedSiblingIsInvisible and written up in docs/rebalance-algorithm.md §8.


Stack created with GitHub Stacks CLIGive Feedback 💬

@markettes
markettes added this pull request to stack #606 September 11, 2026 15:32
Jossec101
Jossec101 previously approved these changes Sep 14, 2026
@markettes
markettes force-pushed the feat/uncategorized-not-automated branch from b7b2541 to 2037dfa Compare September 14, 2026 13:13
Base automatically changed from feat/idle-state to main September 14, 2026 13:15
@markettes
markettes force-pushed the feat/uncategorized-not-automated branch from 2037dfa to dc829a6 Compare September 14, 2026 13:15
@markettes
markettes merged commit f8ce0e1 into main Sep 14, 2026
5 checks passed
@markettes
markettes deleted the feat/uncategorized-not-automated branch September 14, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants