Skip to content

feat: add 750 fee tier for Alien Base V3#533

Open
Haikane wants to merge 3 commits intomainfrom
feat/alienbase-v3
Open

feat: add 750 fee tier for Alien Base V3#533
Haikane wants to merge 3 commits intomainfrom
feat/alienbase-v3

Conversation

@Haikane
Copy link
Copy Markdown
Contributor

@Haikane Haikane commented Feb 19, 2026

Summary

  • Add MediumLow2 = 750 (0.075%) fee tier to the FeeAmount enum with tick_spacing=15
  • Required for Alien Base V3 on Base, which uses this tier for blue chip pairs (ETH, BTC)
  • Without this change, pools with fee=750 fail deserialization with InvalidSnapshotError::ValueError("Unsupported fee amount")

Changes

  • enums.rs: Add MediumLow2 = 750 variant and TryFrom match arm
  • state.rs: Add MediumLow2 => 15 to get_spacing()

Verification

  • All 21 UniswapV3 tests pass
  • Tick spacing=15 confirmed via cast call on Alien Base V3 factory (feeAmountTickSpacing(750))

Related PRs

Test plan

  • All 21 V3 simulation tests pass
  • cargo check clean
  • Tick spacing verified on-chain via factory contract

🤖 Generated with Claude Code

Alien Base V3 on Base uses a custom 0.075% fee tier (750) with
tick_spacing=15 for blue chip pairs. Without this, pools with
fee=750 fail deserialization with "Unsupported fee amount".

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add doc comment to MediumLow2 variant documenting its origin and
tick spacing. Add unit tests for fee=750 round-trip through TryFrom
and tick_spacing=15 mapping to prevent regressions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Haikane Haikane changed the title Add 750 fee tier for Alien Base V3 feat: add 750 fee tier for Alien Base V3 Feb 19, 2026
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@dianacarvalho1 dianacarvalho1 left a comment

Choose a reason for hiding this comment

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

Thank you! this looks good

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Tycho Feb 23, 2026
Copy link
Copy Markdown
Collaborator

@tamaralipows tamaralipows left a comment

Choose a reason for hiding this comment

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

Thanks, just one comment but otherwise good

Comment on lines +43 to +47
fn test_fee_amount_750_round_trip() {
let fee = FeeAmount::try_from(750).expect("750 should be a valid fee tier");
assert_eq!(fee, FeeAmount::MediumLow2);
assert_eq!(fee as i32, 750);
}
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.

I don't think these tests are very meaningful, we are essentially testing that the rust enum works?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants