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.

Sign up for GitHub

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

Jump to bottom

fix: Drift protocol team review implementation #210

Merged

thearyanag merged 8 commits into sendaifun:main from michaelessiet:feat/drift-tool

Jan 17, 2025

Collaborator

michaelessiet commented Jan 15, 2025 •

edited

Loading

Pull Request Description

This PR fixes an issue with validating if a user has a drift account or not. I noticed this while preparing a demo video to showcase how it works. It also fixes the gas fee required to borrow funds from drift.

The PR also contains changes requested by the Drift Protocol team. The changes include:

Addition of actions to stake and unstake from their insurance fund
Addition of actions to swap spot tokens
Addition of actions to get information such as the lend and borrow apy, quote trade price impact and prices, and funding rates.
Addition of an action that gets available spot and perp markets.

Tests and prompts used


          fix: drift user account info fetching

484a64d

arihantbansal requested changes

View reviewed changes

src/tools/drift/drift.ts Outdated Show resolved Hide resolved

test/agent_sdks/vercel_ai.ts Outdated Show resolved Hide resolved

michaelessiet added 6 commits

January 16, 2025 15:50


          merge: main

00f033c


          feat: add more drift actions

97e9673


          fix: bugs noticed during testing

79fe5b0


          feat: add more drift actions

0c840d9


          fix: revert all changes in vercel ai test file

67fa821


          chore: add minimum compute price to constants

e4fa501

michaelessiet commented

View reviewed changes

.github/workflows/build.yml Outdated Show resolved Hide resolved

michaelessiet commented

View reviewed changes

src/vercel-ai/index.ts Show resolved Hide resolved

michaelessiet changed the title ~~fix: Drift account existence validation~~ fix: Drift team review

michaelessiet changed the title ~~fix: Drift team review~~ fix: Drift protocol team review implementation


          Merge branch 'main' into feat/drift-tool

d67747b

thearyanag merged commit 68cb11b into sendaifun:main

1 check failed

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

Labels

None yet