Mateo/dev 42 revamp calling tools in your custom apps quickstart #591
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Converted this back to draft because Vercel builds are failing |
…stom-apps-quickstart
vfanelle
left a comment
vfanelle
left a comment
There was a problem hiding this comment.
Left a few points of feedback:
- Updated apps to agents in the title
- Asked to put 'from arcadepy import Arcade' in Step 2 code snippet
- I got stuck completing the QuickStart and left a screenshot of the last output I got.
| Create a new script called `example.py`: | ||
|
|
||
| ```python filename="example.py" | ||
| from arcadepy import Arcade |
There was a problem hiding this comment.
Can we add the import of Arcade to the top of the "setup the client" code snippet?
from arcadepy import Arcade
Co-authored-by: vfanelle <[email protected]>
Co-authored-by: vfanelle <[email protected]>
Co-authored-by: vfanelle <[email protected]>
Co-authored-by: vfanelle <[email protected]>
Co-authored-by: vfanelle <[email protected]>
Co-authored-by: vfanelle <[email protected]>
Co-authored-by: vfanelle <[email protected]>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| // You can also set the `ARCADE_API_KEY` environment variable instead of passing it as a parameter. | ||
| const client = new Arcade( | ||
| apiKey: "{arcade_api_key}", | ||
| ); |
There was a problem hiding this comment.
Bug: JavaScript syntax error in Arcade constructor
The JavaScript example code in the "Example Code" section has invalid syntax. The new Arcade() constructor call is missing the opening curly brace for the object literal. It reads new Arcade( apiKey: ...) but it needs to be new Arcade({ apiKey: ... }). Compare this with the correctly formatted version earlier in the document (lines 94-96). Users copying this code will get a syntax error.
| tool_name: string, | ||
| input: any, | ||
| user_id: string, | ||
| }) { |
There was a problem hiding this comment.
Bug: TypeScript syntax in JavaScript .mjs file examples
The authorize_and_run_tool function uses TypeScript type annotations (: { tool_name: string, input: any, user_id: string }) but the file is named example.mjs (JavaScript module). TypeScript syntax won't run in a plain JavaScript file. Users copying this code will get a syntax error when running it with node example.mjs.
Additional Locations (1)
Co-authored-by: vfanelle <[email protected]>
Preview here
This PR reworks the main quickstart to move from 3 disconnected tool calls to a more useful workflow that features
The workflow itself:
Since there is a helper function now, the structure of the code went from a single piece of code, to 3 sections of code to make it easier and faster to digest.
Note
Reworks the quickstart into an end-to-end agent tool-calling workflow with auth, updates nav titles, and bumps @next/third-parties to 16.0.7.
app/en/home/quickstart/page.mdxto a workflow-based guide:GoogleNews.SearchNewsStories,GoogleDocs.CreateDocumentFromText, andGmail.SendEmail.app/en/home/_meta.tsxto"Calling tools in your agent".@next/third-partiesfrom16.0.1to16.0.7(lockfile updated).Written by Cursor Bugbot for commit ccffeca. This will update automatically on new commits. Configure here.