Content
82%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, action-dense router skill: concrete CLI commands, exact error-to-cause mappings, explicit decision defaults (Managed KB first, Converse over InvokeModel, Harness for new agents), and every referenced bundle file verified to exist. The main deductions are deliberate repetition of the maxTokens and 'read the reference first' mandates, and second-level reference nesting inside the migration and payments bundles.
Suggestions
Consolidate the maxTokens/ThrottlingException warning into the Critical Warnings section and reference it once from the Converse, Troubleshooting, and Example sections instead of restating the full explanation four times.
Vary or compress the repeated 'You MUST read [X] before responding' mandate into a single up-front rule listing the reference-per-topic table, cutting several near-duplicate sentences.
Flatten the second-level references in the migration bundle (inline the small eligibility/cli notes into migrate-bedrock-agents-to-agentcore-harness.md or link them directly from SKILL.md's routing table) so all references remain one level deep.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is an efficient router — tables, decision guides, and pointers rather than concept explanations Claude already knows. However, the maxTokens/ThrottlingException warning is repeated in Critical Warnings, the Converse section, Throttleshooting, and Examples 1 and 7, and 'You MUST read [X] before responding' is restated nearly verbatim across many sections. This is deliberate emphasis but is trimming-able, matching 'Efficient; minor instances of over-explanation that could be trimmed' rather than the lean anchor 5. | 4 / 5 |
Actionability | Concrete, copy-paste-ready commands cover the common cases: `aws bedrock list-foundation-models --region us-east-1`, the full Example 4 converse command with model ID and `--inference-config '{"maxTokens":1024}'`, quota-check commands, retry config `Config(retries={"max_attempts": 5, "mode": "adaptive"})`, and exact error names mapped to specific fixes (e.g., the double-underscore action-group name pattern, `prepare-agent` after config changes). | 5 / 5 |
Workflow Clarity | Workflows are sequenced with most checkpoints present: the Invoke-a-model checklist, KB create → connector → ingest → 'verify with Retrieve', 'polling get-harness until status READY', and 'Run prepare-agent after ANY configuration change'. It falls short of anchor 5 because several multi-step procedures (agents, guardrails, AgentCore deploy) delegate their detailed sequence and validation checkpoints to reference files rather than showing feedback loops inline — the body states the gates exist but the fix-and-retry loops themselves live one level down. | 4 / 5 |
Progressive Disclosure | The body is a well-signaled overview with one-level-deep references (all 25 referenced `references/*.md` paths and `scripts/fetch_bedrock_agent.py` verified to exist), a TOC, and routing tables per capability. Scored against the actual bundle structure, it does not quite reach anchor 5 because two areas nest a second level: `migrate-bedrock-agents-to-agentcore-harness.md` links into `references/bedrock-agents-to-agentcore-harness/{cli,deploy,discovery,eligibility,mapping}.md`, and `agentcore-payments.md` links to `agentcore-payments-setup-script.md` and `agentcore-payments-wiring.md`. Good structure overall with minor organization gaps. | 4 / 5 |
Total | 17 / 20 Passed |