code/cl8y-dex-terraclassic
4
0
Fork 0
Code Issues 35 Pull requests 6 Projects Releases Packages Wiki Activity

fix: scale gas limit for multi-hop router swaps #725

Merged
Brouie merged 3 commits from fix/issue-22-multihop-gas into main 2026-03-16 07:27:54 +00:00
Conversation 6 Commits 3 Files changed 3 +23 -5
Brouie commented 2026-03-16 05:43:04 +00:00 (Migrated from gitlab.com)
Copy link

Single-hop swaps use 600k gas. Multi-hop swaps via the router need more gas per hop. This adds a countSwapHops function that reads the operations array from execute_swap_operations and scales gas at 400k per hop. For example: 2-hop = 800k, 3-hop = 1200k. Fixes #22.

Single-hop swaps use 600k gas. Multi-hop swaps via the router need more gas per hop. This adds a countSwapHops function that reads the operations array from execute_swap_operations and scales gas at 400k per hop. For example: 2-hop = 800k, 3-hop = 1200k. Fixes #22.
PlasticDigits (Migrated from gitlab.com) approved these changes 2026-03-16 05:43:04 +00:00
Brouie commented 2026-03-16 05:50:08 +00:00 (Migrated from gitlab.com)
Copy link

added 1 commit

Compare with previous version

added 1 commit <ul><li>7790a6d6 - 1 commit from branch <code>master</code></li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/7/diffs?diff_id=1706020947&start_sha=217d253ad324054340af57f09325f719eefc885e)
PlasticDigits (Migrated from gitlab.com) changed target branch from master to main 2026-03-16 07:24:08 +00:00
PlasticDigits commented 2026-03-16 07:26:51 +00:00 (Migrated from gitlab.com)
Copy link

added 1 commit

  • 64889fb1 - refactor: move SWAP_GAS_PER_HOP to constants.ts

Compare with previous version

added 1 commit <ul><li>64889fb1 - refactor: move SWAP_GAS_PER_HOP to constants.ts</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/7/diffs?diff_id=1706133298&start_sha=7790a6d69be19f58f3410e64e51d7614376692cc)
PlasticDigits commented 2026-03-16 07:27:51 +00:00 (Migrated from gitlab.com)
Copy link

Avoid hardcoding constants in files, instead should be constants.ts. merged @Brouie

Avoid hardcoding constants in files, instead should be constants.ts. merged @Brouie
PlasticDigits commented 2026-03-16 07:27:52 +00:00 (Migrated from gitlab.com)
Copy link

approved this merge request

approved this merge request
PlasticDigits (Migrated from gitlab.com) merged commit 46a71f04dd into main 2026-03-16 07:27:54 +00:00
PlasticDigits commented 2026-03-16 07:27:55 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in commit 46a71f04dd

mentioned in commit 46a71f04dd9046c8e0ac4e636d4c75bb74b125ac
Sign in to join this conversation.
Reviewers
No reviewers
PlasticDigits
Labels
Clear labels   
agent:fix_bugfix

   
agent:fix_conflicts

   
agent:fix_security

   
agent:gap_analysis

   
agent:implement

   
agent:implement
Queue implement gate

  
agent:implement
Queue implement gate

  
agent:open_issues

   
agent:ready

   
agent:research

   
agent:security_audit

   
agent:verify

   
architecture
Architecture

  
backend
API and services

  
blocker:hybrid
Blocks hybrid launch

  
blocker:launch
Blocks launch

  
blocker:limit-orders
Blocks limits launch

  
blocker:v2
Blocks v2

  
block:log_only

   
block:security

   
bug

   
ci

   
contracts
Smart contracts

  
correctness

   
deploy

   
dev

   
devops

   
docs

   
documentation

   
duplicate

   
e2e

   
enhancement

   
epic
Epic

  
feature

   
frontend

   
functional-completion
Functional

  
gas

   
good first issue

   
governance

   
help wanted

   
high-risk

   
hooks

   
hybrid
Hybrid swaps

  
indexer
Indexer

  
infra
CI/CD

  
infrastructure

   
integrators

   
invalid

   
launch-blocker

   
limit-orders
Limit orders

  
localnet

   
localterra

   
low priority

   
missing-implementation
Implementation

  
needs-design
Design decision required

  
ops

   
performance
  
priority
high

  
priority
medium

  
product

   
qa

   
QA

   
question

   
ready
Definition of Ready

  
ready
Definition of Ready

  
research

   
scripts

   
security
Security

  
security-hardening
Security hardening

  
smartcontracts

   
tech-debt

   
testing
Testing

  
ux

   
UX

   
v2
Pool-only swaps

  
verification

   
wontfix
No labels
agent:fix_bugfix
agent:fix_conflicts
agent:fix_security
agent:gap_analysis
agent:implement
agent:implement
agent:implement
agent:open_issues
agent:ready
agent:research
agent:security_audit
agent:verify
architecture
backend
blocker:hybrid
blocker:launch
blocker:limit-orders
blocker:v2
block:log_only
block:security
bug
ci
contracts
correctness
deploy
dev
devops
docs
documentation
duplicate
e2e
enhancement
epic
feature
frontend
functional-completion
gas
good first issue
governance
help wanted
high-risk
hooks
hybrid
indexer
infra
infrastructure
integrators
invalid
launch-blocker
limit-orders
localnet
localterra
low priority
missing-implementation
needs-design
ops
performance
priority
high
priority
medium
product
qa
QA
question
ready
ready
research
scripts
security
security-hardening
smartcontracts
tech-debt
testing
ux
UX
v2
verification
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
code/cl8y-dex-terraclassic!725
No description provided.