-
Notifications
You must be signed in to change notification settings - Fork 7
Mateo/dev 42 revamp calling tools in your custom apps quickstart #591
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
base: main
Are you sure you want to change the base?
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
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 <vfanelle@gmail.com>
Co-authored-by: vfanelle <vfanelle@gmail.com>
Co-authored-by: vfanelle <vfanelle@gmail.com>
Co-authored-by: vfanelle <vfanelle@gmail.com>
Co-authored-by: vfanelle <vfanelle@gmail.com>
Co-authored-by: vfanelle <vfanelle@gmail.com>
Co-authored-by: vfanelle <vfanelle@gmail.com>
| // 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 <vfanelle@gmail.com>
| ``` | ||
| ```bash | ||
| uv run example.py | ||
| > The square root of 625 is 25 |
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.
| > The square root of 625 is 25 |
| ```bash | ||
| uv run example.py | ||
| > The square root of 625 is 25 | ||
| > Successfully starred the repository ArcadeAI/arcade-mcp |
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.
| > Successfully starred the repository ArcadeAI/arcade-mcp |
| - Build a simple workflow that uses tools to: | ||
| - search for news with Google News | ||
| - Create a Google Doc with the news | ||
| - Send a link to the Google Doc to a Slack User |
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.
| - Send a link to the Google Doc to a Slack User | |
| - Email a link to the Google Doc to the user |
|
|
||
| print(f"The square root of 625 is {response.output.value}") | ||
| # Get the news results from the response | ||
| news = response_search.output.value["news_results"] |
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.
I get None outputs in the News Results. Can we remove those:
latest news about MCP URL mode elicitation:
Visual Studio Magazine - VS Code 1.107 (November 2025 Update) Expands Multi-Agent Orchestration, Model Management
None
SD Times - Several new updates make their way into the MCP specification
None
AI News - How the MCP spec update boosts security as infrastructure scales
None
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.