Content
77%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 highly actionable, well-sequenced operational skill with excellent validation checkpoints and a clean one-level-deep reference bundle. Its weaknesses are length — repeated guardrail phrasing, marketing copy, and a hardcoded changelog TD list inflate the token budget — and significant detail (Dockerfile customization, credential setup) inlined in SKILL.md that belongs in reference files.
Suggestions
Move the Dockerfile/entrypoint customization walkthrough (Step 6, custom image path) and the private-repo credential setup into references (e.g., remote-execution.md or a new credentials.md), leaving only the decision logic in SKILL.md.
Trim redundant 'Do NOT...' restatements to single directives and remove the duplicated 'If the user declines, suggest local mode' paragraph in Step 6.
Cut or compress the marketing greeting and the changelog's hardcoded TD list (which also conflicts with the 'never hardcode TD names' rule) to reduce token load.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most content is non-obvious operational detail (ATX CLI flags, IAM attachment scripts, S3 sync patterns), but there is real padding: the ~20-line marketing greeting, triple-stacked 'Do NOT skip this step. Do NOT ask... Do NOT condition it...' repetition, a duplicated 'If the user declines, suggest local mode' paragraph in Step 6, and a changelog that hardcodes 13 TD names. This sits between the 2 anchor (noticeably verbose) and 3 (mostly efficient with some unnecessary explanation), closer to 3 since the core guidance earns its tokens. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout: complete IAM policy-attachment bash script, `atx custom def list --json`, `aws s3 sync s3://user-bucket/repos/ ... --exclude "*" --include "*.zip"`, concrete Dockerfile RUN examples per language, and a signal-to-file detection table. Specific examples cover the common cases, matching the 5 anchor. | 5 / 5 |
Workflow Clarity | The 8-step workflow is clearly sequenced with explicit validation checkpoints and feedback loops: 'Do NOT proceed until `aws sts get-caller-identity` succeeds' with a re-run after setup, waiting for user confirmation at Steps 4 and 7, and re-running `atx custom def list --json` after TD creation. Batch operations are gated on explicit consent with per-mode concurrency limits, matching the 5 anchor (explicit validation, error-recovery loops). | 5 / 5 |
Progressive Disclosure | The seven reference files exist, are one level deep, and are well signaled (in-body pointers plus a 'When to Use' table). However, substantial content that belongs in references is inlined in SKILL.md: ~100 lines of Dockerfile/entrypoint customization that duplicates remote-execution.md's scope, ~80 lines of private-repo credential setup, and the full changelog TD list. This fits the 3 anchor ('content that should be separate is inline') better than 4, where such gaps would be minor. | 3 / 5 |
Total | 16 / 20 Passed |