-
Notifications
You must be signed in to change notification settings - Fork 781
add quickstart examples #577
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
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Caution Review failedThe pull request is closed. WalkthroughThis PR adds comprehensive example applications and templates to the MCP-Agent framework. Changes include: refactoring CLI example handling to use unified template mapping; four new cloud-deployable examples (hello-world, mcp, temporal, chatgpt-app); asyncio-based MCP server examples with client code; two use-case examples (financial-analyzer, researcher); and seven workflow pattern examples (deep-orchestrator, evaluator-optimizer, intent-classifier, orchestrator-worker, parallel, router, swarm). Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~55 minutes Rationale: Large scope (100+ files across 11 distinct example categories) with heterogeneous types (Python logic, TypeScript/React, YAML configs, Markdown docs, CSS styling). While repetitive patterns reduce per-file complexity, the breadth demands multi-domain reasoning: init.py refactoring logic (medium); new web client implementation for chatgpt_app (medium); dense agent/workflow orchestration code across multiple examples (medium-to-high); and consistency checks across similar templates. The changes span CLI core, cloud deployment examples, framework patterns, and use-case demonstrations, each requiring contextual review. Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (102)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
20288a3 to
b52015f
Compare
d37a1f2 to
44fc459
Compare
Merge activity
|
33e08d0 to
a1c14eb
Compare
| @@ -0,0 +1,6 @@ | |||
| # Core framework dependency | |||
| mcp-agent @ file://../../../ # Link to the local mcp-agent project root | |||
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.
@jtcorbett just checking, did you test all these quickstart examples -- mainly wondering about the paths for the schema requirements files being correct
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 tested them, but I did test them in the repo, so I should probably also test from the package as well to make sure they're loading right there
### TL;DR Added new cloud-based example templates to the MCP Agent CLI initialization command. ### How to test? ``` mcp init --template hello-world my-hello-world mcp init --template mcp my-mcp-example mcp init --template temporal my-temporal-workflow mcp init --template chatgpt-app my-chatgpt-app ```

TL;DR
Added new cloud-based example templates to the MCP Agent CLI initialization command.
How to test?
Summary by CodeRabbit
hello-world,mcp,temporal, andchatgpt-appfor faster project initialization.