feat: Add Hyperliquid Integration #33
Open
Conversation
This file contains 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
OoXooOx
reviewed
Jan 30, 2025
projects/hyperliquid/tools.ts
Outdated
| export const tools: AiTool[] = [ | ||
| { | ||
| name: 'bridgeToHyperliquid', | ||
| description: 'Bridges native USDC tokens from Arbitrum to Hyperliquid.', |
There was a problem hiding this comment.
don't use word "native" corresponding to ERC-20 tokens. It's can be problem for AI
OoXooOx
reviewed
Jan 30, 2025
| }); | ||
| console.log('Transaction result:', result); | ||
|
|
||
| const transferMessage = result.data[result.data.length - 1]; |
There was a problem hiding this comment.
Consider reading event info and displaying it to the user for better awareness
There was a problem hiding this comment.
which data you suggest from event to show?
OoXooOx
reviewed
Jan 30, 2025
| * @returns Promise resolving to function execution result | ||
| */ | ||
| export async function bridgeToHyperliquid({ chainName, account, amount }: Props, { sendTransactions, notify }: FunctionOptions): Promise<FunctionReturn> { | ||
| try { |
There was a problem hiding this comment.
Is it will be possible in future transfer USDT or other tokens? If yes, then consider add argument - tokenAddress
There was a problem hiding this comment.
would it make more sense to add tokenSymbol instead of tokenAddress? how command call would look like with tokenAddress?
OoXooOx
reviewed
Jan 30, 2025
projects/hyperliquid/index.ts
Outdated
| export default { | ||
| tools, | ||
| functions, | ||
| description: 'Integration with Hyperliquid Protocol', |
There was a problem hiding this comment.
here main protocol description for AI. Add slightly more info.
vaniiiii
force-pushed
the
feature/hyperliquid
branch
from
cbf03a9 to
59dce70
Compare
vaniiiii
force-pushed
the
feature/hyperliquid
branch
from
c58df2d to
3b5d7a2
Compare
vaniiiii
force-pushed
the
feature/hyperliquid
branch
from
ab41357 to
c2fd053
Compare
vaniiiii
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Hyperliquid Integration
🚧 Phase 1 of Hyperliquid Integration
Implements trading functionality between Arbitrum and Hyperliquid including bridging, position management, and fund transfers.
This PR delivers a complete trading functionality with Hyperliquid, enabling the full trading cycle from bridging to position management.
What's Included
Action Items 🎯
Coming Soon ⏳
Test Results
✅ All tests passing
✅ 100% coverage across all files
Commands: