-
Notifications
You must be signed in to change notification settings - Fork 11
docs(dungeon-adventure): make the tutorial agentic using the newer mcp and agent generators #331
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
…p and agent generators Remove FastAPI from the dungeon adventure tutorial and replace it with a Strands Agent on Bedrock AgentCore Runtime, as well as a TypeScript MCP server for managing a player's inventory of items. Fixes #324
📚 Documentation translations have been updated and committed (63ac4e3) to this PR.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #331 +/- ##
=======================================
Coverage 96.28% 96.28%
=======================================
Files 91 91
Lines 9060 9060
Branches 1092 1092
=======================================
Hits 8723 8723
Misses 327 327
Partials 10 10 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 some minor feedback. Looks good otherwise.
docs/src/content/docs/en/get_started/tutorials/dungeon-game/2.mdx
Outdated
Show resolved
Hide resolved
docs/src/content/docs/en/get_started/tutorials/dungeon-game/2.mdx
Outdated
Show resolved
Hide resolved
docs/src/content/docs/en/get_started/tutorials/dungeon-game/3.mdx
Outdated
Show resolved
Hide resolved
docs/src/content/docs/en/get_started/tutorials/dungeon-game/2.mdx
Outdated
Show resolved
Hide resolved
docs/src/content/docs/en/get_started/tutorials/dungeon-game/2.mdx
Outdated
Show resolved
Hide resolved
📚 Documentation translations have been updated and committed (3f7efe8) to this PR.
|
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.
LGTM
Reason for this change
Demonstrate newer generators and make the tutorial more fun/relevant :)
Description of changes
Remove FastAPI from the dungeon adventure tutorial and replace it with a Strands Agent on Bedrock AgentCore Runtime, as well as a TypeScript MCP server for managing a player's inventory of items.
Description of how you validated changes
Integration test updated.
Ran through the tutorial manually end to end, deploying and testing at the end of each module.
NB: #330 is needed for the updated tutorial to work
Issue # (if applicable)
Fixes #324
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license