Skip to content

EXSC-142 Finalize tron configuration#1665

Merged
mirooon merged 109 commits intomainfrom
exsc-142-finalize-tron-configuration
Apr 3, 2026
Merged

EXSC-142 Finalize tron configuration#1665
mirooon merged 109 commits intomainfrom
exsc-142-finalize-tron-configuration

Conversation

@mirooon
Copy link
Copy Markdown
Contributor

@mirooon mirooon commented Mar 19, 2026 •
edited
Loading

Which Linear task belongs to this PR?

https://linear.app/lifi-linear/issue/EXSC-142/finalize-tron-configuration-refundwallet-pauserwallet-permissions

Why did I implement it this way?

Checklist before requesting a review

Checklist for reviewer (DO NOT DEPLOY and contracts BEFORE CHECKING THIS!!!)

  • I have checked that any arbitrary calls to external contracts are validated and or restricted
  • I have checked that any privileged calls (i.e. storage modifications) are validated and or restricted
  • I have ensured that any new contracts have had AT A MINIMUM 1 preliminary audit conducted on by <company/auditor>

purriza and others added 30 commits January 28, 2026 10:09
@purriza
Added new Tron deployments
81e62f9
@purriza
Add retry for RPC rate limits
c1eaf61
@purriza
Updated targetState.json with Tron deployments
d647466
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
4ee5d29
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
49b6ea6
@purriza
Added execWithRateLimitRetry test
56e1692
@purriza
Unreacheable error removed
15e95be
@purriza
Removed test for retries in Tron healthCheck. Refactor duplicated cod…
b4b1c88 
…e, added checks for whitelist integrity
@purriza
tighten forge script success check and healthCheck lint
defaf13
@purriza
Removed duplicate import
4ea3294
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
2d50c97
@purriza
Refactor imports and fix rate limit retries
34a5c63
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
cb596db
@purriza
Fix address check in Tron
d1a6189
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
b94fc46
@purriza
Refactor tron functions, delay calls, constants
7701488
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
084196e
@purriza
refactor: centralize delay utilities and improve Tron health check mo…
79daabc 
…dularity

- Consolidate delay constants in script/deploy/tron/constants.ts
- Extract Tron health check functions to healthCheckTronUtils.ts
- Replace setTimeout with centralized sleep() utility across 9 files
- Improve RPC URL handling to prioritize environment variables
@purriza
Removed ts-nocheck and refactored scripts. Added cursor rules
d321ab5
@purriza
Added new isRateLimitErrorFunction
0ad3751
@purriza
Moved TargetState to types.ts. Consolidate shared/constants.ts. Refac…
cc99bed 
…tor sleep management on tron contract calls
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
caafe0e
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
dee006d
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
7f3ffae
@purriza
Refactor constants management, delays in tron calls and checkWhitelis…
cbab0aa 
…tIntegrity
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
7b2e1e4
@purriza
Updated comments and refactored expectedPairs functionality
d4846db
@purriza
Merge branch 'main' into SMAR-62-Tron-ensure-all-scripts-work
39cbf3f
@purriza
Created new helper spawnAndCapture, checkAndLogDeployment and refacto…
7c368c0 
…r Tron RPC usage
@purriza
Use spawnAndCapture helper in tron/utils
becd274
@lifi-action-bot lifi-action-bot changed the title EXSC-142 Finalize tron configuration [MayanFacet v1.2.2] EXSC-142 Finalize tron configuration Mar 30, 2026
gvladika
gvladika reviewed Mar 31, 2026
View reviewed changes
mirooon added 14 commits April 1, 2026 13:44
@mirooon
refactor: update EVM version retrieval from foundry.toml to use TOML …
2b2ebc2 
…parsing
@mirooon
fix: improve error handling for EVM version retrieval in deploy-safe.ts
6733e8a
@mirooon
refactor: streamline address normalization in deployment utilities
c2dc986
@mirooon
refactor: restructure Tron wallet configurations in global.json and u…
ddc5b5f 
…pdate related scripts
@mirooon
Merge branch 'exsc-142-finalize-tron-configuration-addressed-comments…
57ea134 
…' into exsc-142-finalize-tron-configuration-refactoring
@mirooon
Merge pull request #1685 from lifinance/exsc-142-finalize-tron-config…
320aa74 
…uration-refactoring

EXSC-142 Refactor of dependencies
@mirooon
Merge branch 'exsc-142-finalize-tron-configuration' into exsc-142-fin…
123dd66 
…alize-tron-configuration-addressed-comments
@mirooon
Merge pull request #1681 from lifinance/exsc-142-finalize-tron-config…
3085d6c 
…uration-addressed-comments

EXSC-142 finalize tron configuration addressed comments
@mirooon
fix: ensure correct RPC URL format for Tron network in getRpcUrl func…
99cbcd3 
…tion
@mirooon
removed duplication for formatDecodedArg and left one function
8bd1931
@mirooon
make chainExecutor chain agnostic in init
350f2d3
@mirooon
fixed gaszipId for tron to 0. GasZip is not deployed to tron
02c8af0
@mirooon
Merge branch 'main' into exsc-142-finalize-tron-configuration
9170df4
@mirooon
refactor: enhance Tron wallet handling and global configuration types…
c490579 
… in scripts
@gvladika gvladika self-requested a review April 3, 2026 06:38
gvladika
gvladika approved these changes Apr 3, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@gvladika gvladika left a comment

Choose a reason for hiding this comment

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

Lgtm

@mirooon mirooon merged commit 9eef226 into main Apr 3, 2026
31 of 33 checks passed
@mirooon mirooon deleted the exsc-142-finalize-tron-configuration branch April 3, 2026 09:48
@coderabbitai coderabbitai bot mentioned this pull request Apr 3, 2026
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@gvladika gvladika gvladika approved these changes

@0xDEnYO 0xDEnYO 0xDEnYO approved these changes

+1 more reviewer

@aikido-pr-checks aikido-pr-checks[bot] aikido-pr-checks[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

AuditNotRequired

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@mirooon @gvladika @github-advanced-security @0xDEnYO @lifi-action-bot @purriza