Skip to content
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

Add Li.fi plugin #450

Merged
merged 8 commits into from
Mar 26, 2025
Merged

Add Li.fi plugin #450

merged 8 commits into from
Mar 26, 2025

Conversation

sunosuporno
Copy link
Contributor

Background

This PR introduces a new LiFi bridge plugin that enables cross-chain token transfers within our application. LiFi is a bridge aggregator that finds the best routes for transferring tokens across different blockchain networks, making it easier and more efficient for users to bridge their assets.

What does this PR do?

  • Implements a new LifiService plugin with two main tools:
    1. lifi_get_quote: Fetches quotes for cross-chain transfers using the LiFi API
    2. lifi_bridge: Executes the bridge transaction, including automatic token approvals
  • Features:
    • Supports multiple EVM chains through chain mapping
    • Handles token approvals automatically before bridging
    • Includes error handling for cases when no bridge routes are available
    • Optional API key support for enhanced rate limits
    • Comprehensive logging for better debugging and user feedback

Testing

Method Prompt Screenshot Transaction Link
Getting Quotes can you give quotes for swapping 1 USDC from Mode to Base? Screenshot 2025-03-25 at 11 40 22 PM NA
Bridging funds bridge 1 USDC from Mode to Base Screenshot 2025-03-25 at 11 44 55 PM Transaction Link

Docs

My changes do not require a change to the project documentation.

Checklist

  • I have tested this change and added the relevant screenshots to the PR description
  • I updated the README if necessary to include the new plugin, wallet, chain, etc.

Copy link

changeset-bot bot commented Mar 25, 2025

⚠️ No Changeset found

Latest commit: ede7ae2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@0xaguspunk
Copy link
Contributor

@sunosuporno could you rebase from main and remove the changes to the main README?

Also, could you add a README for the new plugin? 🙏

@sunosuporno
Copy link
Contributor Author

@0xaguspunk made the readme related changes! What do you mean by rebasing? I already merged all changes in main to this branch? Am I missing something?

@0xaguspunk
Copy link
Contributor

you pushed a bunch of changes to the main README in the root. I dont think these are related to the PR

@sunosuporno
Copy link
Contributor Author

Ohh I removed it now in the latest commit ad6228c. I added as the last item in the checklist of the PR wanted it 😅

@0xaguspunk
Copy link
Contributor

its failing to build, you need to delete the pnpm lock file and run pnpm install again

@sunosuporno
Copy link
Contributor Author

Done! @0xaguspunk

@0xaguspunk
Copy link
Contributor

@sunosuporno you pushed the changes of root project README, could you revert those? not needed for this PR :)

@sunosuporno
Copy link
Contributor Author

sunosuporno commented Mar 25, 2025

@0xaguspunk
Copy link
Contributor

Its still there

Screenshot 2025-03-25 at 21 39 58

@sunosuporno
Copy link
Contributor Author

What about now?

@0xaguspunk 0xaguspunk merged commit a161073 into goat-sdk:main Mar 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants