CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

player-internals

Deep implementation details of the IterablePlayer state machine, tick loop, and data source iteration patterns.

Quality

73%

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 ./.github/skills/player-internals/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

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

The body is a dense, well-organized reference that respects the reader's intelligence: concrete state transitions, real paths, specific constants, and no padding. Its weaknesses are minor — a duplicated caveat about DeserializingIterableSource, a code sample marked as simplified rather than executable, and an external skill path that is not part of the bundle.

DimensionReasoningScore

Conciseness

The body assumes Claude's competence — no basic concepts are explained, and specifics are dense ("300ms" tick budget, "~600MB" cache, "default { sec: 10 }"). The one redundancy is that DeserializingIterableSource's optional status is stated both in the diagram ("ONLY for serialized sources") and again in the blockquote warning, which could be trimmed.

4 / 5

Actionability

Concrete guidance throughout: exact state transitions, a real file path ("packages/suite-base/src/players/IterablePlayer/DeserializingIterableSource.ts"), named interfaces, and specific constants. The tick-loop snippet is explicitly labeled "Simplified tick loop logic" rather than executable code, a minor gap that keeps it below anchor 5.

4 / 5

Workflow Clarity

This is a reference/knowledge skill rather than a multi-step workflow, and its content is unambiguously organized (state machine, tick loop, iterator architecture, backfill, subscriptions, performance). The Backfill Strategy section is properly sequenced 1-4. It is not a 5 because there is no explicit guidance on how a reader should apply this knowledge when investigating or modifying the player.

4 / 5

Progressive Disclosure

No bundle files exist (no references/, scripts/, or assets/), and at ~110 lines of cohesive reference material, keeping everything in SKILL.md with clear section headers is appropriate. The one organizational gap is the pointer to ".github/skills/remote-caching/SKILL.md", which is not part of this bundle and reads as a dangling external reference rather than a well-signaled one-level-deep reference.

4 / 5

Total

16

/

20

Passed

Description

58%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 is specific and clearly scoped to the IterablePlayer internals, making it distinct from generic skills. Its main weakness is the complete absence of a "when to use" trigger clause and of natural user-facing keywords like "playback", which limits discoverability and caps completeness and trigger-term quality.

Suggestions

Add an explicit trigger clause, e.g. "Use when working on the IterablePlayer, playback/seek behavior, or data source iteration in suite-base."

Include natural synonyms users would actually say — "playback", "player", "seek", "message pipeline" — alongside the existing internal jargon.

State the description in action terms (e.g. "Explains the IterablePlayer state machine...") to sharpen the 'what' beyond a topic listing.

DimensionReasoningScore

Specificity

The description lists several concrete topics — "state machine, tick loop, and data source iteration patterns" — giving specific coverage of the skill's contents, though they are framed as subject areas rather than actions. It is above anchor 3 (which expects only 1-2 concrete actions) but not fully comprehensive of anchor 5.

4 / 5

Completeness

It clearly answers "what" ("Deep implementation details of the IterablePlayer state machine, tick loop, and data source iteration patterns") but contains no "Use when..." clause or equivalent trigger guidance, which caps completeness at 3 per the judging guidelines. It is not a 2 because the "what" is specific and unambiguous.

3 / 5

Trigger Term Quality

Terms like "IterablePlayer", "state machine", and "tick loop" are relevant but are internal jargon; common natural variations a user would say ("playback", "player", "seek", "message pipeline") are missing. This matches anchor 3 (some relevant keywords, missing common variations) and is not anchor 4's broader coverage.

3 / 5

Distinctiveness Conflict Risk

Naming a single specific class ("IterablePlayer") gives a clear niche with minimal conflict risk against unrelated skills. It falls short of anchor 5 only because no explicit trigger phrases distinguish it from other player/playback-related skills in the same codebase.

4 / 5

Total

14

/

20

Passed

Validation

100%

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

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

lichtblick-suite/lichtblick

Path

.github/skills/player-internals/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.