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

Filipposl/coingecko plugin #10

Merged
merged 7 commits into from
Dec 6, 2024
Merged

Filipposl/coingecko plugin #10

merged 7 commits into from
Dec 6, 2024

Conversation

filipposl
Copy link
Collaborator

@filipposl filipposl commented Dec 6, 2024

Background

What does this PR do?

Introduces a new Coingecko plugin that enables querying cryptocurrency market data directly through the GOAT SDK.

The PR includes:

  • New Coingecko plugin implementation
  • Vercel AI example demonstrating plugin usage
  • Integration with existing SDK architecture
  • Support for querying trending cryptocurrencies and price data for specified listing

What kind of change is this?

New feature

Documentation changes needed?

If a docs change is needed: I have updated the documentation accordingly.

Testing

Detailed testing steps

  1. Action: Install plugin and configure with Coingecko API key

Expected result: The plugin initializes without errors

  1. Action: Query price for a specific cryptocurrency (e.g., "BONK")

Expected result: Returns current price data in specified currency

  1. Action: Request trending cryptocurrencies list

Expected result: Returns array of currently trending cryptocurrencies

  1. Action: Run the example script with a natural language query

Expected result: AI correctly interprets the query and returns relevant cryptocurrency data

For plugins

  • [*] I have tested this change locally with key pair wallets
  • I have tested this change locally with hosted wallets (e.g. Crossmint Smart Wallets, etc.)

Discord username

flympe

@filipposl filipposl requested a review from 0xaguspunk December 6, 2024 06:38
Copy link
Collaborator

@aigustin aigustin left a comment

Choose a reason for hiding this comment

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

Very cool!! Made some comments! But otherwise lgtm!

typescript/examples/vercel-ai/coingecko/index.ts Outdated Show resolved Hide resolved
typescript/packages/plugins/coingecko/README.md Outdated Show resolved Hide resolved
typescript/packages/plugins/coingecko/src/tools.ts Outdated Show resolved Hide resolved
typescript/packages/plugins/coingecko/src/tools.ts Outdated Show resolved Hide resolved
typescript/packages/plugins/coingecko/src/parameters.ts Outdated Show resolved Hide resolved
typescript/packages/plugins/coingecko/src/tools.ts Outdated Show resolved Hide resolved
Copy link

changeset-bot bot commented Dec 6, 2024

⚠️ No Changeset found

Latest commit: 398cb3a

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 0xaguspunk marked this pull request as ready for review December 6, 2024 18:12
@0xaguspunk 0xaguspunk merged commit 8d49602 into main Dec 6, 2024
1 check failed
@0xaguspunk 0xaguspunk deleted the filipposl/coingecko-plugin branch December 6, 2024 18:12
@sachiew
Copy link

sachiew commented Dec 28, 2024

hey @filipposl , thanks for contributing this. I work on coinGecko product, would you be keen to add more support here for CoinGecko plugin. There's a lot more can be done with CG public and pro api. Happy to bounce ideas, and grant you paid api keys for testing & personal use. Feel free to ping TG @soonegg91

ICYMI: CG API now supports onchain data, with 100+ networks
https://docs.coingecko.com/reference/endpoint-overview#-onchain-dex-endpoints-geckoterminal

@filipposl
Copy link
Collaborator Author

for sure - will get in touch via TG!

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.

4 participants