CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

lading-optimize-review

Reviews optimization patches using a 5-persona peer review system. Requires unanimous approval backed by benchmarks.

59

Quality

74%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.claude/skills/lading-optimize-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

85%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

This is a strong workflow skill: phases are crisply sequenced, every stage has explicit REJECT-based validation with a genuine feedback loop for Kani failures, commands are copy-paste ready, and report templates are properly externalized to assets with clear navigation. Remaining polish is minor — deduplicate the outcomes/decision table and add a concrete profiling command.

DimensionReasoningScore

Conciseness

The body is dense and operational — exact commands, thresholds, and checklists with almost no explanation of concepts Claude already knows — matching anchor 4 (efficient, minor instances that could be trimmed). Not anchor 5 because the identical APPROVED/REJECTED outcomes table appears twice ("## Outcomes" and "## Phase 4: Decision") and a few lines (e.g. the duplicated decision rules) could be cut.

4 / 5

Actionability

Guidance is mostly executable: copy-paste-ready commands with flags (`cargo criterion --bench $ARGUMENTS[0] 2>&1 | tee ...`, `hyperfine --warmup 3 --runs 30 --export-json ...`), a concrete report-ID derivation algorithm, and a deterministic template-fill procedure against real files in `assets/`. Not anchor 5 because a few steps remain high-level — "Hot path verified via profiling (not just guessed)" gives no profiling command despite `samply` being in allowed-tools, and "Run `ci/validate`" lacks expected-output detail.

4 / 5

Workflow Clarity

Phases 1–5 are clearly sequenced with explicit validation checkpoints at every stage (REJECT rules for missing arguments, missing baselines, and each "NO EXCEPTIONS" case), per-persona checklists, and a documented error-recovery feedback loop ("If Kani fails to run: 1. Document why... 2. Verify comprehensive property tests exist instead"). This matches anchor 5 exactly — explicit validation steps, feedback loops, and checklists for a complex process.

5 / 5

Progressive Disclosure

The SKILL.md body stays operational while bulky material is correctly externalized: both report templates live in `assets/approved.template.yaml` and `assets/rejected.template.yaml` (verified to exist), referenced one level deep via a verdict-to-template table plus explicit read-and-fill steps. Nothing in the body is bulk reference material that belongs in a separate file, matching anchor 5 (well-signaled one-level-deep references, appropriately split content, easy navigation).

5 / 5

Total

18

/

20

Passed

Description

53%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description states a clear, specific capability but completely lacks trigger guidance — no "Use when..." phrasing tells Claude (or a user) when to invoke it. Keyword coverage is adequate for the niche yet misses common synonyms. Adding an explicit trigger clause and naming the lading project would materially improve it.

Suggestions

Add an explicit trigger clause, e.g. "Use when an optimization patch has been written and needs benchmark-backed approval before merging" — its absence currently caps completeness at 3.

Include natural trigger variations users would actually say, such as "performance improvement", "speedup", or "benchmark regression", to improve trigger-term coverage.

Name the project (lading) and the concrete actions performed — running micro/macro benchmarks, checking for duplicate optimizations, and returning a structured verdict report — to sharpen distinctiveness and specificity.

DimensionReasoningScore

Specificity

"Reviews optimization patches using a 5-persona peer review system" names the domain and one concrete action with its mechanism, and "Requires unanimous approval backed by benchmarks" adds a criterion, but coverage is not comprehensive — benchmark execution, duplicate detection, and report generation are absent. This matches anchor 3 (domain plus 1-2 concrete actions, not comprehensive) better than anchor 4, which expects several listed specific actions with only minor gaps.

3 / 5

Completeness

The "what" is clear (review optimization patches via 5-persona peer review requiring unanimous benchmark-backed approval), but there is no "Use when..." clause or equivalent trigger guidance anywhere, which per the judging guidelines caps completeness at 3. It is not anchor 4 because the "when" is entirely missing rather than merely imprecise.

3 / 5

Trigger Term Quality

Terms like "optimization patches", "peer review", "unanimous approval", and "benchmarks" are relevant and natural for this niche, but common variations a user would say — "performance", "speedup", "regression", "benchmark results" — are missing, matching anchor 3 (some relevant keywords, missing common variations). Not anchor 4 because keyword coverage is thin rather than good-with-few-gaps.

3 / 5

Distinctiveness Conflict Risk

"optimization patches" + "5-persona peer review" + "unanimous approval backed by benchmarks" carves a fairly distinct niche with low confusion risk against unrelated skills, matching anchor 4 (mostly distinct, minor overlap risk). Not anchor 5 because the description never names the project (lading) or a unique artifact type, leaving minor overlap risk with generic code-review or benchmarking skills.

4 / 5

Total

13

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

Total

15

/

16

Passed

Repository

DataDog/lading

Path

.claude/skills/lading-optimize-review/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.