CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

scene-history-integrity

Review Excalidraw editor changes for scene graph, history, and undo/redo integrity. Use as one lens in a code review run.

Quality

84%

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

86%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 tight, well-structured instruction-only review-lens skill that respects the context window and gives specific, repo-aware guidance with clear report/do-not-report criteria. The main growth areas are making the method's follow-through sequence more stepwise and adding one or two concrete examples of the update paths or findings it wants.

DimensionReasoningScore

Conciseness

The ~24-line body is lean with no padding: it never explains what Excalidraw, undo/redo, or history capture are, and every section (Scope, Method, Threshold, Reporting) carries unique operational content. This matches the anchor 'Lean and efficient; assumes Claude's competence; every token earns its place.'

5 / 5

Actionability

The guidance is concrete and repo-specific — 'element ids, fractional ordering, deleted-element handling, and selection state', 'CaptureUpdateAction or the repository's equivalent update path', and explicit report/do-not-report criteria — but it stops short of naming specific files/functions or example findings. Mostly actionable guidance with minor gaps, matching anchor 4 rather than 5.

4 / 5

Workflow Clarity

The Method section lays out a clear follow-through sequence ('Start at the changed editor operation... Follow the update through the element map, app state, history capture, and any tests'), and the Threshold section provides an explicit decision checkpoint. It is not 5 because the sequence is one flowing sentence rather than explicit steps with verification checkpoints, and not 3 because the sequence and decision points are clearly present.

4 / 5

Progressive Disclosure

The skill is under 50 lines, single-purpose, and needs no external references (none exist in the bundle and the body references none, which is consistent). Its four well-organized sections satisfy the simple-skill exception: well-organized sections alone merit 5.

5 / 5

Total

18

/

20

Passed

Description

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

A distinctive, appropriately concise description that clearly states what the skill does and when to invoke it, anchored by the strong Excalidraw niche. Its main limitation is that it compresses the review work into one generic verb, leaving the concrete actions (checking element identity, bound geometry, history capture) implicit.

Suggestions

Replace the single 'Review ... for ... integrity' verb with the concrete checks the skill performs, e.g. 'Check element ids, fractional ordering, and selection state; verify bound text and arrows stay connected through undo/redo; confirm each user operation becomes the right undo unit via CaptureUpdateAction.'

Make the when-clause carry concrete trigger phrases, e.g. 'Use when reviewing Excalidraw changes that mutate editor state, element maps, or history capture, or when the user mentions undo/redo or scene integrity.'

Add natural synonyms such as 'elements', 'canvas', or 'state updates' to broaden keyword coverage beyond 'scene graph, history, and undo/redo'.

DimensionReasoningScore

Specificity

It names the domain concretely ('Excalidraw editor changes', 'scene graph, history, and undo/redo integrity') but offers a single action verb — 'Review ... for ... integrity' — with the sub-areas as objects of that one action. That matches anchor 3 ('names domain and 1-2 concrete actions, but not comprehensive') better than 4, which expects several distinct actions.

3 / 5

Completeness

The 'what' is clear (review Excalidraw changes for scene/history/undo-redo integrity) and an explicit 'Use as one lens in a code review run' clause supplies the 'when', so it exceeds anchor 3's weakly-implied 'when'. It is not 5 because the when-clause lacks concrete trigger phrases a user would say.

4 / 5

Trigger Term Quality

'Excalidraw', 'editor changes', 'undo/redo', 'history', and 'code review' are natural terms a user reviewing this codebase would actually say, giving good keyword coverage. It falls short of anchor 5 because natural variations like 'elements', 'canvas', or 'state management' are absent.

4 / 5

Distinctiveness Conflict Risk

'Excalidraw editor changes' carves out a clear niche with distinct triggers — no other skill category would plausibly claim this description — and conflict risk with generic code-review skills is minimal given the domain specificity.

5 / 5

Total

16

/

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

guypod/excalidraw-factory

Path

review-lenses/scene-history-integrity/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.