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.
An exceptionally actionable, dense skill: executable commands for every workflow, explicit statistical validation, and a genuinely useful CI bisecting guide. The main improvement opportunity is moving the CI-artifacts and mock-server detail into reference files to slim the main document.
Suggestions
Move the CI artifacts/retention table and the 'Finding and reading historic runs' gh command details into a references/ci.md file, keeping only the bisecting workflow inline.
Move the mock-server endpoint list and IS_SCENARIO_AUTOMATION setup details into a reference file, retaining just the one-line role of the mock server in the architecture section.
Trim the key-flags table to the most-used flags (--runs, --scenario, --build, --baseline-build, --no-baseline) and defer the rest to `pnpm run perf:chat --help`.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly all content is operational knowledge Claude cannot infer — flags, exit codes, statistical methodology, retention windows — with no concept tutoring. Not 5 because the 20-row flags table, mock-server endpoint list, and CI artifact-retention details are fully inlined where `--help` or a reference file could carry some of the load. | 4 / 5 |
Actionability | Copy-paste-ready commands throughout — `pnpm run perf:chat --scenario text-only --runs 3`, `--resume <path>` examples, `gh run list`/`gh api` with jq filters — covering the common cases, plus concrete interpretation thresholds (">2.0 MB/msg — likely leak"). Fully executable with no pseudocode. | 5 / 5 |
Workflow Clarity | Clear sequences with explicit validation checkpoints: statistical significance gates before flagging regressions, exit code semantics, a resume feedback loop for inconclusive results, and the numbered 4-step regression-pinpointing workflow that includes "Confirm it's real vs. measurement noise" before deep-diving. | 5 / 5 |
Progressive Disclosure | Well-organized sections with clearly signaled one-level-deep references to real repo files (`common/perf-scenarios.js`, `.github/workflows/chat-perf.yml`) and a pointer to `--help` for the full scenario list. Not 5 because no bundle reference files exist — CI artifact/retention detail and mock-server internals are inlined in SKILL.md where they could be split out for a ~310-line document. | 4 / 5 |
Total | 18 / 20 Passed |