-
Notifications
You must be signed in to change notification settings - Fork 76
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 AI landing pages #674
Add AI landing pages #674
Conversation
Branch preview✅ Deployed successfully in branch deployment: |
Overall readability score: 32.53 (🟢 +0.91)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work Valentin! 💪 Along with some minor edits, I have three remarks:
- I will need more time to test that the code of these pages work; so I excluded it from the scope of this review for now.
- While it is clever to publish these pages to showcase that we care about AI agents, I have doubts on the utility of these in the context of AI, as the quickstart guides are not specific to it. Whether you are an agent or a human does not make any difference here.
- Most AI agents, models, and tooling are developed using Python (with the notable exception of LangChain which also includes some typescript tooling). I am not sure whether JS/TS guides will truly be helpful in the context of building these tools; on the contrary setting up a LangChain project (or similar) would be super cool to see how one could build an AI-powered app, with an agent having Safe capabilities.
These pages are useful in themselves (e.g. the one on spending limits or the ones on swaps), but they do not explain how to set up an AI agent with access to a Safe; only the javascript code that this agent will have to execute in order to interact with it.
Co-authored-by: louis-md <[email protected]>
Co-authored-by: louis-md <[email protected]>
Co-authored-by: louis-md <[email protected]>
Thanks @louis-md. All of your comments are correct. For a Safe Smart Account it doesn't make a difference, who signs. However, we wanted to support AI builders not familiar with Safe to build. |
Ready to be merged.