-
Notifications
You must be signed in to change notification settings - Fork 54
fix(platform-wallet): auto_select_inputs honors Σ inputs == Σ outputs #3554
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Claudius-Maginificent
wants to merge
62
commits into
v3.1-dev
Choose a base branch
from
fix/rs-platform-wallet-auto-select-inputs
base: v3.1-dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 28 commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
aaf8be7
fix(rs-platform-wallet): auto_select_inputs honors Σ inputs == Σ outputs
lklimek 9ea9e70
fix(rs-platform-wallet): reserve fee headroom at DeductFromInput(0) t…
lklimek 687b1f8
test(rs-platform-wallet): protocol-level reproduction of CodeRabbit f…
lklimek 60f7850
refactor(rs-platform-wallet): sort auto-select candidates by balance …
lklimek 9ff937f
fix(rs-platform-wallet): enforce min_input_amount, restrict fee_strat…
lklimek 79c2b28
ci(rs-packages-filter): trigger Rust workspace tests on rs-platform-w…
lklimek d610502
Merge remote-tracking branch 'origin/v3.1-dev' into fix/rs-platform-w…
lklimek 3c4f919
fix(rs-platform-wallet): clippy-clean for Rust 1.92 (workspace tests …
lklimek d0c772f
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek 45779e3
docs(rs-platform-wallet): trim verbose comments in auto_select_inputs…
lklimek 74b81d1
feat(rs-platform-wallet): support ReduceOutput(0) fee strategy in aut…
lklimek e16a819
Merge remote-tracking branch 'origin/v3.1-dev' into fix/rs-platform-w…
lklimek 357ddb8
docs(rs-platform-wallet): note platform #3040 fee-estimation bug in R…
lklimek 545fa99
Merge remote-tracking branch 'origin/v3.1-dev' into fix/rs-platform-w…
lklimek ea7810f
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek 5e50a0b
fix(rs-platform-wallet): exclude output addresses from auto_select_in…
lklimek 923992a
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek b6e7d05
chore(rs-platform-wallet): fix macOS clippy lints in manager/accessor…
lklimek ef4345a
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek 376ef56
chore(rs-platform-wallet): remove stale QA-001-followup TODO
lklimek dac5a7a
feat(rs-platform-wallet): add address_derivation_info and fee_paid ac…
lklimek 468e774
revert(rs-platform-wallet): drop test-only production additions; abso…
lklimek bdd46f3
fix(rs-platform-wallet): defensive checked arithmetic on Credits in t…
lklimek 47997a2
refactor(rs-platform-wallet): collapse estimate_fee_for_inputs_pub wr…
lklimek f81cdf2
test(rs-platform-wallet): tighten non_fee_target_below_min_input_redi…
lklimek 196f6c5
feat(rs-platform-wallet): typed OnlyOutputAddressesFunded error [CMT-…
lklimek b326a18
chore(rs-platform-wallet): drop useless vec! in detect_only_output_ad…
lklimek 92bc962
chore(rs-platform-wallet): drop stray ManagedIdentitySigner re-export
lklimek 6b530e5
ci: clean stale mac runner artifacts before rust tests
vivekgsharma bf99f0b
fix(rs-platform-wallet): saturating arithmetic on Credits in selector…
lklimek 0cec379
fix(rs-platform-wallet): explicit runtime invariant checks in selecto…
lklimek 61c6ab8
fix(rs-platform-wallet): rename OnlyOutputAddressesFunded to NoSelect…
lklimek d67fd1d
feat(rs-platform-wallet): balance-descending donor in ReduceOutput Ph…
lklimek 07d046b
feat(rs-platform-wallet): borderline warn-log for ReduceOutput(0) cha…
lklimek 7b45af2
refactor(rs-platform-wallet): extract select_inputs shared helpers + …
lklimek 4853cf9
fix(rs-platform-wallet): split NoSelectableInputs into discrete varia…
lklimek fa3e676
fix(rs-platform-wallet): use saturating arithmetic for outputs aggreg…
lklimek 00ac7f2
chore(rs-platform-wallet): document accepted-risk on selector heurist…
lklimek 2dec6a6
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek 6574b84
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek 5ef1b44
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek e46d12a
refactor(platform-wallet/auto_select_inputs): inline helpers, collaps…
lklimek e986342
fix(rs-platform-wallet): pin unreachable selector branches with debug…
lklimek 9e9d289
refactor(rs-platform-wallet): tighten transfer.rs narration, TODO lin…
lklimek eb68dba
fix(rs-platform-wallet): NoSelectableInputs Display omits zero-valued…
lklimek 35f0e98
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek 776cbd7
feat(rs-platform-wallet): output_change_address override on platform-…
lklimek 22149b0
Merge remote-tracking branch 'origin/v3.1-dev' into fix/rs-platform-w…
lklimek 0a5aa2e
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek 25bbb3f
Merge branch 'v3.1-dev' into fix/rs-platform-wallet-auto-select-inputs
lklimek b66e7dd
chore: fmt
lklimek ea8dc63
test(rs-platform-wallet): cover reduce_output Phase 1/3 + wrapper Aut…
lklimek 9a4fc30
docs(rs-platform-wallet): document DeductFromInput contract + trim ve…
lklimek 047ffca
docs(rs-platform-wallet): clarify donor selection + sub-minimum guard
lklimek 79c83c6
refactor(rs-platform-wallet): extract saturating_sum + change-addr co…
lklimek 9ee4e3c
refactor(rs-platform-wallet): remove format_address (duplicates Display)
lklimek aba7ca3
refactor(rs-platform-wallet): split NoSelectableInputs into typed var…
lklimek 4526836
ci: revert tests-rs-workspace.yml disk-cleanup step
lklimek 72fee50
fix(rs-platform-wallet-ffi): drop stale ptr arg from integration_tests
lklimek edd3ea5
chore: cargo fmt
lklimek c5511a7
chore(rs-platform-wallet-ffi): use Result::is_err in group_info tests
lklimek 30e612b
Merge remote-tracking branch 'origin/v3.1-dev' into fix/rs-platform-w…
lklimek File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.