Restructure use cases#1653
Conversation
Reorganizes all 27 use case samples into category subfolders aligned with the AgentCore GTM workload definitions: - 01-conversational-agents/ (19 samples): user-facing agents with streaming, user OAuth, and session/long-term memory - 02-automation-agents/ (4 samples): event-driven agents with service identity and stateless execution - 03-coding-assistants/ (3 samples): developer tools with sandboxed execution and project-scoped memory Other changes: - Moves okta-auth-three-tier-end-to-end-demo to 01-features/05-authenticate-and-authorize (identity pattern, not a use case) - Adds 02-use-cases/README.md with category index and sample table - Adds category README for each subfolder, aligned with GTM definitions, no em-dashes - Adds use-case-assessment.md with per-sample scoring, recommendations, and a starter toolkit migration list (18 of 27 samples need migrating) - Fixes all cross-folder relative links broken by the restructure (LICENSE, LOCAL_DEVELOPMENT.md, AGENTCORE_DEPLOYMENT.md)
… demos Reviewed all 19 conversational agent samples against actual AgentCore feature usage, blog post references, and code quality. Removed 9 samples that did not meet the bar and moved 2 identity-pattern demos to the correct location. Dropped from 01-conversational-agents/: - gateway-schema-support-agent: misclassified coding agent, 82-line README, only 1 real feature, no blog post - slide-deck-generator-memory-agent: narrow use case, no blog post, deprecated starter toolkit, 5 features - local-prototype-to-agentcore: claimed 9 features, only 3 in code, tutorial format belongs in 01-tutorials/ - role-based-hr-data-agent: no blog post, deprecated starter toolkit, Cedar pattern needs full rebuild to be useful - A2A-realestate-agentcore-multiagents: claimed 10 features, only 3 in code (Runtime, Cognito, A2A), no blog post - cost-optimization-agent: only Runtime in code, notebook-driven, no blog post - DB-performance-analyzer: only Gateway + Cognito in code, no blog post - farm-management-advisor: notebook-only, mixed deprecated/native SDK, 191-line README, no blog post Moved to 01-features/05-authenticate-and-authorize/: - auth0-multi-agent-obo: RFC 8693 OBO token exchange is an identity pattern, not a use case; sits alongside okta and entra OBO samples - okta-auth-three-tier-end-to-end-demo: (previous commit) Other changes: - Rename 02-automation-agents/ to 02-workflow-automation-agents/ - Update use-case-assessment.md with DROPPED/MOVED entries for all removed samples - Update all README counts and sample tables - Add auth0-multi-agent-obo to identity README folder table and auth pattern quick reference
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
Latest scan for commit: Security Scan ResultsScan Metadata
SummaryScanner ResultsThe table below shows findings by scanner, with status based on severity thresholds and dependencies: Column Explanations: Severity Levels (S/C/H/M/L/I):
Other Columns:
Scanner Results:
Severity Thresholds (Thresh Column):
Threshold Source: Values in parentheses indicate where the threshold is configured:
Statistics calculation:
Detailed FindingsShow 717 actionable findingsFinding 1: CFN_NAG_W89
Description: Finding 2: CFN_NAG_W89
Description: Finding 3: CFN_NAG_W92
Description: Finding 4: CFN_NAG_W92
Description: Finding 5: CFN_NAG_W77
Description: Finding 6: CFN_NAG_W77
Description: Finding 7: CFN_NAG_W77
Description: Finding 8: CFN_NAG_W32
Description: Finding 9: CFN_NAG_F38
Description: Finding 10: CFN_NAG_W11
Description: Finding 11: CFN_NAG_W11
Description: Finding 12: CFN_NAG_W11
Description: Finding 13: CFN_NAG_W11
Description: Finding 14: CFN_NAG_W11
Description: Finding 15: CFN_NAG_W11
Description: Finding 16: CFN_NAG_W11
Description: Finding 17: CFN_NAG_W89
Description: Finding 18: CFN_NAG_W89
Description: Finding 19: CFN_NAG_W89
Description: Finding 20: CFN_NAG_W89
Description: Finding 21: CFN_NAG_W92
Description: Finding 22: CFN_NAG_W92
Description: Finding 23: CFN_NAG_W92
Description: Finding 24: CFN_NAG_W92
Description: Finding 25: CFN_NAG_W84
Description: Finding 26: CFN_NAG_W84
Description: Finding 27: CFN_NAG_W86
Description: Finding 28: CFN_NAG_W86
Description: Finding 29: CFN_NAG_F3
Description: Finding 30: CFN_NAG_W11
Description: Finding 31: CFN_NAG_W89
Description: Finding 32: CFN_NAG_W28
Description: Finding 33: CFN_NAG_F3
Description: Finding 34: CFN_NAG_W11
Description: Finding 35: CFN_NAG_W89
Description: Finding 36: CFN_NAG_W28
Description: Finding 37: CFN_NAG_W32
Description: Finding 38: CFN_NAG_W89
Description: Finding 39: CFN_NAG_W92
Description: Finding 40: CFN_NAG_W92
Description: Finding 41: CFN_NAG_W28
Description: Finding 42: CFN_NAG_W35
Description: Finding 43: CFN_NAG_W40
Description: Finding 44: CFN_NAG_W5
Description: Finding 45: CFN_NAG_W9
Description: Finding 46: CFN_NAG_W89
Description: Finding 47: CFN_NAG_W89
Description: Finding 48: CFN_NAG_W92
Description: Finding 49: CFN_NAG_W92
Description: Finding 50: CFN_NAG_W89
Description: Finding 51: CFN_NAG_W92
Description: Finding 52: CFN_NAG_W28
Description: Finding 53: CFN_NAG_W28
Description: Finding 54: CFN_NAG_W11
Description: Finding 55: CFN_NAG_F78
Description: Finding 56: CFN_NAG_W89
Description: Finding 57: CFN_NAG_W11
Description: Finding 58: CFN_NAG_W89
Description: Finding 59: CFN_NAG_W89
Description: Finding 60: CFN_NAG_W92
Description: Finding 61: CFN_NAG_W68
Description: Finding 62: CFN_NAG_W69
Description: Finding 63: CFN_NAG_W64
Description: Finding 64: CFN_NAG_F78
Description: Finding 65: CFN_NAG_F38
Description: Finding 66: CFN_NAG_F3
Description: Finding 67: CFN_NAG_W11
Description: Finding 68: CFN_NAG_W11
Description: Finding 69: CFN_NAG_W89
Description: Finding 70: CFN_NAG_W89
Description: Finding 71: CFN_NAG_W92
Description: Finding 72: CFN_NAG_W92
Description: Finding 73: CFN_NAG_W84
Description: Finding 74: CFN_NAG_W28
Description: Finding 75: CFN_NAG_W28
Description: Finding 76: CFN_NAG_W28
Description: Finding 77: CKV_AWS_116
Description: Code Snippet: Finding 78: CKV_AWS_117
Description: Code Snippet: Finding 79: CKV_AWS_115
Description: Code Snippet: |
Samples moved one level deeper (into category subfolders) so ../../CONTRIBUTING.md no longer resolves to the repo root. Updated to ../../../CONTRIBUTING.md in three files: - 01-conversational-agents/customer-support-assistant/README.md - 02-workflow-automation-agents/event-driven-claims-agent/README.md - 03-coding-assistants/claude-code-gateway-mcp-server/README.md
AWS-operations-agent mcp-tool-template.yaml and mcp-tool-template-zip.yaml: - Scope IAM wildcard resources to account/region-specific ARNs (lambda:InvokeFunction, iam:PassRole, s3:*, logs:*) - Split S3 permissions into bucket-level and object-level statements - Scope bedrock:InvokeModel to foundation-model ARN pattern - Add KMS keys for Lambda env var encryption (CKV_AWS_173) and CloudWatch log group encryption (CKV_AWS_158) - Add SQS dead letter queue and DeadLetterQueue config (CKV_AWS_116) - Add ReservedConcurrentExecutions (CKV_AWS_115) - Add checkov:skip for VPC (CKV_AWS_117) - demo function, VPC not required customer-support-assistant cognito.yaml: - Scope logs resource to account/region ARN prefix - Scope cognito-idp:AdminAddUserToGroup to UserPool ARN - Add SQS DLQ and DeadLetterConfig to PostSignupFunction (CKV_AWS_116) - Add ReservedConcurrentExecutions (CKV_AWS_115) - Add checkov:skip for VPC (CKV_AWS_117) - Cognito trigger, VPC not needed customer-support-assistant infrastructure.yaml: - Add KMS CMK and SSESpecification to WarrantyTable and CustomerProfileTable (CKV_AWS_119) - Scope logs resource to account/region ARN prefix - Add SQS DLQ and DeadLetterConfig to PopulateDataFunction (CKV_AWS_116) - Add ReservedConcurrentExecutions (CKV_AWS_115) - Add checkov:skip for VPC (CKV_AWS_117) - CFn custom resource, VPC not needed
There was a problem hiding this comment.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
Amazon Bedrock AgentCore Samples Pull Request
Important
review readylabel to it. Only PRs withreview readywill be reviewed.Issue number:
Concise description of the PR
Restructures 02-use-cases/ into three category subfolders aligned with
the AgentCore GTM workload definitions, removes slimmer samples that
are not comprehensive enough, and moves identity pattern demos to the correct
location.
Folder restructure
Samples removed (9) — no blog post reference, actual AgentCore feature usage far below required count, or implementation did not meet the bar for a standalone use case:
gateway-schema-support-agent, slide-deck-generator-memory-agent, local-prototype-to-agentcore, role-based-hr-data-agent, A2A-realestate-agentcore-multiagents, cost-optimization-agent,
DB-performance-analyzer, farm-management-advisor
Samples moved to 01-features/05-authenticate-and-authorize/ (2) — these are identity pattern demos, not use cases:
auth0-multi-agent-obo, okta-auth-three-tier-end-to-end-demo
READMEs added/updated
User experience
Developers browsing the repo can now navigate directly to the agent type that matches their workload (conversational, automation, or coding) rather than scanning a flat list of 27 unorganised
samples. Each category README explains the workload definition, the recommended AgentCore service configuration, and which sample to start with. Samples that were retained are higher quality on
average — fewer misleading feature claims, no deprecated starter toolkit samples without a migration path.
Checklist
If your change doesn't seem to apply, please leave them unchecked.
Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.