Skip to content

Update zerion extension #18165

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

Closed
wants to merge 4 commits into from
Closed

Update zerion extension #18165

wants to merge 4 commits into from

Conversation

zerts
Copy link
Contributor

@zerts zerts commented Mar 26, 2025

Description

AI functionality for Zerion Extension. Based on the London Hackathon project

Screencast

Zerts.Screenshot.2025-03-07.at.18.32-converted-converted.mp4

Checklist

- Merge branch \'contributions/merge-1743011894653\' into ai-extension-support
- Pull contributions
- ✨ Run Prettier
- 📝 Update Changelog
- 🤖 AI Extension Support
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: zerion Issues related to the zerion extension OP is contributor The OP of the PR is a contributor of the extension labels Mar 26, 2025
@raycastbot
Copy link
Collaborator

raycastbot commented Mar 26, 2025

Thank you for your contribution! 🎉

🔔 @qd-qd @zerts you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

You can expect an initial review within five business days.

@zerts zerts marked this pull request as ready for review March 26, 2025 22:07
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR adds comprehensive AI functionality to the Zerion extension, introducing new tools for blockchain interactions and wallet management along with AI evaluation tests.

  • Several tools lack proper error handling for API failures and input validation, particularly in check-portfolio.ts, look-wallets.ts, and search-blockchain.ts
  • Typos found in package.json tool descriptions: "breif" should be "brief" and "ususlly" should be "usually"
  • The CHANGELOG.md entry for "Zerion AI Extension" is missing description of the actual changes/features
  • The auto-detection of best chain functionality mentioned in comments for create-a-swap-link.ts is not implemented
  • The 10-address limit in look-wallets.ts should be documented in the function's description

💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!

9 file(s) reviewed, 19 comment(s)
Edit PR Review Bot Settings | Greptile

@zerts
Copy link
Contributor Author

zerts commented Apr 9, 2025

Hey @pernielsentikaer !
Do I need to do anything else about this pull request?
Just checking :))

@pernielsentikaer
Copy link
Collaborator

Our AI team will pick it up 😊

@andreaselia andreaselia self-assigned this May 2, 2025
@pernielsentikaer
Copy link
Collaborator

Hi @zerts 👋

Can you look into the CI errors so we can continue the review 👋

@zerts
Copy link
Contributor Author

zerts commented May 13, 2025

Hi @zerts 👋

Can you look into the CI errors so we can continue the review 👋

Looking into it!

@zerts zerts closed this May 13, 2025
@zerts zerts deleted the ext/zerion branch May 13, 2025 08:47
@zerts
Copy link
Contributor Author

zerts commented May 13, 2025

Hey @pernielsentikaer !
Not sure why, but after I run npm run publish, this branch was removed and a new pr was openned. Is this expected?
Looks like I filed something during the merge process 🫠
#19132

@pernielsentikaer
Copy link
Collaborator

It should normally just update this PR. Thanks for flagging 😊

@zerts
Copy link
Contributor Author

zerts commented May 13, 2025

@pernielsentikaer , thank you for the response.
What should I do with this?
Should I prepare new PR the same way this was? Or we can somehow restore this one? :)

@zerts
Copy link
Contributor Author

zerts commented May 13, 2025

I see! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Extension extension fix / improvement Label for PRs with extension's fix improvements extension: zerion Issues related to the zerion extension OP is contributor The OP of the PR is a contributor of the extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants