Open
Conversation
A miner could claim a third-party tx to the user's dest address as their own fulfillment. Store miner_dest_address on the swap struct at initiation and validate tx_info.sender against it during dest-side verification.
Miners can now post separate rates for each swap direction (e.g., 340 TAO/BTC for BTC->TAO, 350 TAO/BTC for TAO->BTC) to capture tx fee asymmetry. Commitment format bumped to v3 with two rate fields. Backend unchanged — each swap still stores one rate, selected by direction at initiation time.
Replace TAO-centric assumptions with canonical chain ordering so any pair (including non-TAO pairs like BTC-ETH) works correctly. TAO is preferred as the rate denomination when present, otherwise alphabetical ordering is used as a deterministic fallback. - Add canonical_pair() to chains.py with TAO-preference ordering - Rename rate_reverse → counter_rate across all files - Change get_rate_for_direction(source_is_tao) to accept chain name - Rename rate.py params: source_is_tao → is_reverse, tao_decimals/asset_decimals → dest_decimals/source_decimals - Filter miners with rate=0 for the desired direction (CLI + validator) - Fix test_scale.py swap encoding (missing miner_source_address, miner_dest_address, rate) - Fix float comparison for rate display → use string comparison - Add 5 unit tests: canonical_pair, direction after normalization, direction-specific rate calc
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary
Miners post a single bilateral rate that applies to both swap directions. But tx fees are asymmetric across chains — sending BTC costs more than receiving it. A single rate forces miners to average out that cost, meaning users get a worse rate on the cheap direction.
This adds direction-specific rates so miners can price each direction honestly (e.g., 340 TAO/BTC for BTC->TAO, 350 TAO/BTC for TAO->BTC). The spread captures the tx fee differential.
Approach
Commitment format bumped to v3 with two rate fields. The correct rate is selected based on swap direction at initiation time. The entire backend (validator verification, miner fulfillment, smart contract, voting) is unchanged — each swap on-chain still stores one rate, it's just the right one for that direction now.
Test plan
Reviewer experience
Start the dev environment and explore these flows:
Post a pair with directional rates (as miner):