Content
88%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A strong, highly actionable routing skill: every section defers detail to real, verified reference files while surfacing Bedrock-specific gotchas (quota reservation mechanics, prepare-agent, inference-profile requirements) that generic AWS knowledge would miss. The main weaknesses are mild redundancy — the maxTokens warning and AgentCore routing each repeated across several sections — and a two-level-deep nested reference bundle for the migration guide.
Suggestions
Consolidate the maxTokens quota warning to the Critical Warnings section (or a single quota reference) and have other sections link to it, instead of restating the full explanation in the Converse-vs-InvokeModel, two Examples, ThrottlingException, and Deploy sections.
Merge the three overlapping routing surfaces (the 'Which Bedrock Capability Do You Need?' table, the AgentCore Services table, and the 'Deploy an agent to AgentCore' prose) into one routing table to remove duplicated guidance.
Flatten or inline the migration guide's nested sub-references (references/bedrock-agents-to-agentcore-harness/{cli,deploy,discovery,eligibility,mapping}.md) so all references are one level deep from SKILL.md, or note in the body that these are sub-pages of the migration guide so navigation stays predictable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Efficient overall — dense tables, service-specific gotchas ("unset values default to the model's maximum and silently reserve far more quota than needed"), and no explanations of concepts Claude already knows. Not score 5 because the maxTokens warning is repeated in 4+ sections and routing is spread across three overlapping tables (capability table, AgentCore services table, Deploy-an-agent prose); not score 3 because nearly every section carries non-obvious, load-bearing information. | 4 / 5 |
Actionability | Fully executable commands throughout: `aws bedrock list-foundation-models --region us-east-1`, a complete converse invocation with model-id and inference-config, `Config(retries={"max_attempts": 5, "mode": "adaptive"})`, plus concrete diagnoses like "Run prepare-agent after ANY configuration change." Not score 4 because commands are copy-paste ready and cover the common cases, including exact error-message remediations. | 5 / 5 |
Workflow Clarity | Sequences are explicit with validation checkpoints: create → data source → ingest → "verify with Retrieve", "polling get-harness until status READY", "check for cacheReadInputTokens", and the 5-point mandatory coverage list for Harness answers. Not score 4 because validation/feedback loops are explicit, and destructive operations carry a confirm-before-proceeding constraint. | 5 / 5 |
Progressive Disclosure | Good structure: a TOC, routing tables with well-signaled links to 24 reference files (all verified to exist on disk), and a clear MCP-vs-local file-resolution guardrail. Not score 5 because the migration path nests two levels deep (SKILL.md → migrate-bedrock-agents-to-agentcore-harness.md → bedrock-agents-to-agentcore-harness/{cli,deploy,discovery,eligibility,mapping}.md) rather than the one-level-deep ideal; not score 3 because all references are clearly signaled and appropriately split. | 4 / 5 |
Total | 18 / 20 Passed |