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 lean, highly actionable instruction skill: nearly every line is a concrete project-specific command or policy, with a clearly sequenced workflow and explicit validation checkpoints. Weaknesses are minor — some redundancy around PR-body handling, no explicit recovery loop for failed checks, and a dense preview-linking section that could be factored out.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and teaches only project-specific facts Claude cannot know (Oxfmt usage, Vercel preview-link policy, PR-body file handling) with no concept re-explanation. Minor redundancy — "never pass the replacement body inline" and "do not pass the PR body inline through a shell command" both appear, and the Vercel paragraph is convoluted — keeps it below a 5. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout: "bunx oxfmt <changed-file-or-package-directory>... --write", "bun run build", "bun run stylecheck", "git push -u origin HEAD", and "gh pr create --title \"<title>\" --body-file <path-to-temp-md-file>" with a worked example. Specific commands cover the common cases, including edge handling (skip Oxfmt for unsupported files, vercel inspect fallback). | 5 / 5 |
Workflow Clarity | The sequence is clear and ordered (branch check → existing-PR check → test policy → formatting → build/stylecheck → commit → push → create → preview links) with explicit validation checkpoints ("Inspect any formatter changes before committing", "to ensure we compile and CI linting/formatting passes", report when the preview URL cannot be resolved). It falls short of a 5 because no fix-and-retry loop is specified for failed builds or style checks — recovery is only implied. | 4 / 5 |
Progressive Disclosure | Good structure for a single-file skill: the main flow is linear and the preview-linking workflow is separated under "## Link directly changed website pages", with both file references (the Element contribution guide and the pr-name skill) clearly signaled and only one level deep. Not a 5 because the dense Vercel preview procedure (~10 lines of conditional policy) would sit more appropriately in a separate reference file, and the body has no section headers for the main flow. | 4 / 5 |
Total | 17 / 20 Passed |