CtrlK
BlogDocsLog inGet started
Tessl Logo

maple-otel-spec-review

Review a diff, PR, or specific file in this repo for OpenTelemetry *specification* compliance, grounded in the source-linked spec corpus at docs/otel-spec/ (snapshot v1.58.0). Triggers on requests like 'is this spec compliant', 'review this PR against the OTel spec', 'spec-review this diff', 'check my partial-success handling', 'are these retryable status codes right', 'does apps/ingest honor the OTLP spec', and on reviews of changes touching the OTLP server surface in apps/ingest (partial success, retryable set {429, 502, 503, 504}, protobuf Status bodies, gzip, OTLP/JSON encoding), self-instrumentation (apps/api tracer setup, apps/ingest/src/otel.rs, packages/effect-sdk), or consumers of span status / SeverityNumber / db.query.text (WarehouseQueryService, query-engine). Spec MUSTs and SHOULDs only. For Maple house conventions use maple-telemetry-conventions; for whole-project instrumentation audits use maple-audit; for general diff correctness use /code-review.

76

Quality

96%

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

SKILL.md
Quality
Evals
Security

Quality

Content

92%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.

A tightly engineered review workflow: every section is operational (severity rules, routing table, cached MUSTs, hard rules), the six-step process has real validation checkpoints and feedback loops, and corpus loading is budgeted and section-scoped. The only blemish is the duplicated time-sensitive version/date stamp, which keeps conciseness just short of the top anchor.

DimensionReasoningScore

Conciseness

The body is dense and operational — tables carry rules without padding, and nothing explains concepts Claude already knows. However, the time-sensitive snapshot stamp ('spec v1.58.0, researched 2026-07-05') appears twice (intro and report template) and is not isolated in a deprecated/old-patterns section, which the guidelines explicitly penalize. Between anchors: clearly above the 3-anchor's 'some unnecessary explanation' but not the 5-anchor's 'every token earns its place'.

4 / 5

Actionability

Fully executable guidance: concrete commands ('Grep -n '^##' docs/otel-spec/<file>.md' then 'Read' with offset/limit, 'git diff', 'git show <sha>', 'bun typecheck', 'cargo check'), a copy-paste findings-report template with a worked example row, and a classification example showing exact output format. Matches the 5-anchor's copy-paste-ready commands covering common cases.

5 / 5

Workflow Clarity

Six clearly sequenced steps with explicit checkpoints and feedback loops: classification is shown to the user for correction before reviewing (Step 2), the findings report precedes any edit (Step 5), and fixes are followed by build re-verification with a fix-or-revert loop ('A fix that breaks the build is a regression: fix or revert it'). The 5-anchor (clear sequence, explicit validation, error-recovery feedback loops) fits; the 4-anchor's 'minor validation gaps' is not present.

5 / 5

Progressive Disclosure

The skill is a routing hub over one-level-deep corpus files (docs/otel-spec/*.md) with a routing table, a grep-then-read loading rule, and an explicit read budget — references are clearly signaled and easy to navigate. The inline 16-row quick table is justified as a cache so trivial reviews load zero spec files, which is progressive disclosure working correctly, not misplaced content. Matches the 5-anchor; no nested or buried references exist.

5 / 5

Total

19

/

20

Passed

Description

100%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.

An exemplary description: it states exactly what the skill does and when to use it in the user's own words, scopes the corpus and version it grounds itself in, and preempts mis-triggering by explicitly routing to the three neighboring skills. Third-person voice is used throughout, with no fluff or over-claims.

DimensionReasoningScore

Specificity

Names the concrete deliverable ('Review a diff, PR, or specific file … for OpenTelemetry specification compliance', 'Spec findings: MUST/SHOULD violations with normative citations') and enumerates the exact surfaces and file paths it covers (OTLP server in apps/ingest, self-instrumentation, span status / SeverityNumber / db.query.text consumers). This matches the 5-anchor of multiple specific concrete actions with comprehensive coverage; the 4-anchor's 'minor gaps in coverage' doesn't apply since the scope is fully enumerated.

5 / 5

Completeness

Explicitly answers both 'what' (spec-compliance review of diffs/PRs/files, MUSTs and SHOULDs only, grounded in the docs/otel-spec/ corpus) and 'when' ('Triggers on requests like … and on reviews of changes touching …' with concrete trigger phrases). This is the 5-anchor verbatim; the 4-anchor's 'when could be more explicit' does not apply.

5 / 5

Trigger Term Quality

Includes six quoted natural trigger phrases users would actually say ('is this spec compliant', 'spec-review this diff', 'are these retryable status codes right', 'does apps/ingest honor the OTLP spec') plus content-signal triggers, covering synonyms and phrasing variations. Comprehensive natural-term coverage matches the 5-anchor; the 4-anchor's 'a few natural terms missing' is not the case.

5 / 5

Distinctiveness Conflict Risk

Explicitly disambiguates the three neighbor skills by name ('For Maple house conventions use maple-telemetry-conventions; for whole-project instrumentation audits use maple-audit; for general diff correctness use /code-review') and carves a tight niche (OTel spec compliance, repo-specific paths). Clear niche with distinct triggers and minimal conflict risk — the 5-anchor; the 4-anchor's 'minor overlap risk' is not present.

5 / 5

Total

20

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

metadata_version

'metadata.version' is missing

Warning

metadata_field

'metadata' should map string keys to string values

Warning

Total

14

/

16

Passed

Repository
MapleTechLabs/maple
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.