CtrlK
BlogDocsLog inGet started
Tessl Logo

add-new-jit-ee-api

Add a new API to the JIT-VM (aka JIT-EE) interface in the codebase.

58

Quality

66%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

—

The risk profile of this skill

Fix and improve this skill with Tessl

tessl review fix ./.github/skills/add-new-jit-ee-api/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.

This is a strong procedural skill: an ordered 8-step workflow with concrete diff examples for each layer, exact repository paths, the generation script invocation, and a Definition of Done checklist. Weaknesses are minor — some duplicated file listings, no build/compile verification for a ten-file change, and one long example block that could be split into a reference file.

Suggestions

Add a final verification step after the gen.sh/build, e.g. 'Compile the affected projects (or run a jit-diff) and confirm CorInfoImpl_generated.cs contains the new API' to create a feedback loop for the batch edit.

Provide a concrete diff for the agnostic.h step (an Agnostic_* type example) so every step is copy-paste ready.

Collapse the duplicated superpmi-shared file listing (bulleted group followed by per-file repetition) into the per-file sections only.

DimensionReasoningScore

Conciseness

The body is almost entirely project-specific knowledge Claude cannot infer (exact paths, diff examples, macro conventions) with no filler explaining known concepts, so it respects the token budget. It is not 5 because of minor redundancy: the four superpmi-shared files are listed twice (once as a bulleted group, then repeated one-by-one) and 'Go through each of them one by one' plus repeated path headers could be collapsed.

4 / 5

Actionability

Nearly every step gives exact file paths and copy-paste-ready diff examples for the common case, plus a concrete command (gen.sh/gen.bat), matching 'Mostly executable guidance; concrete code or commands with minor gaps'. It is not 5 because the agnostic.h step has no example diff and the lwmlist/enum entries use placeholders ('DLD', '<last value + 1>') rather than a fully concrete instance.

4 / 5

Workflow Clarity

Steps are explicitly numbered and declared 'must be completed in order', with placement notes and a Definition of Done self-check list covering all ten files — a clear sequence with most checkpoints present, matching the 4 anchor. It is not 5 because for a batch edit across ~10 C++/C# files there is no build/compile or gen.sh-output verification step to confirm correctness, i.e. no feedback loop; it stays above 3 because the DoD checklist and the step-4 instruction to inspect the auto-generated file provide explicit verification checkpoints.

4 / 5

Progressive Disclosure

The skill has no bundle files and uses clear section headers (Goal, Required inputs, Implementation steps, DoD) with a single self-contained procedure where every step is needed each run, so inline placement is appropriate and navigation is easy — 'Good structure; most content is appropriately placed... minor organization gaps'. It is not 5 because the ~50-line methodcontext.cpp example diff is long enough that it (or the SuperPMI step) could live in a reference file to keep SKILL.md a leaner overview.

4 / 5

Total

16

/

20

Passed

Description

57%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 short, concrete, and highly distinctive for its niche, correctly covering the primary name variation (JIT-VM aka JIT-EE). Its main weaknesses are the missing 'Use when...' trigger guidance and thin keyword coverage beyond the interface's two names, which leave a user relying on exact jargon to discover it.

Suggestions

Add a 'Use when...' clause, e.g. 'Use when adding or extending a JIT-VM/JIT-EE (corinfo) API, updating ThunkInput.txt, or extending SuperPMI recording/replay.'

Include natural trigger keywords users would say: 'corinfo.h', 'JIT interface', 'ICorStaticInfo', 'SuperPMI', 'JIT-EE API'.

Briefly hint at scope so the 'what' is more comprehensive, e.g. '...including thunk generation, CorInfoImpl, and SuperPMI record/replay plumbing.'

DimensionReasoningScore

Specificity

The description names the domain ("JIT-VM (aka JIT-EE) interface") and one concrete action ("Add a new API"), but does not enumerate the several supporting actions the skill actually covers (thunk generation, corinfo.h, SuperPMI plumbing). It clearly fits the anchor 'Names domain and 1-2 concrete actions, but not comprehensive' — below 4 because only a single action is listed, above 2 because the action and domain are concrete, not generic.

3 / 5

Completeness

There is a clear 'what' ("Add a new API to the JIT-VM (aka JIT-EE) interface in the codebase") but no 'Use when...' clause or equivalent trigger guidance, which per the judging guidelines caps completeness at 3. It is not 2 because the 'what' is specific and unambiguous; it cannot be 4 without any 'when' element at all.

3 / 5

Trigger Term Quality

It includes the relevant keywords "JIT-VM", "JIT-EE", "API", and "interface", covering the primary name variation via "(aka JIT-EE)", but misses natural phrases a user would likely say such as "corinfo", "JIT interface", "SuperPMI", or "ThunkInput". This matches 'Some relevant keywords but missing common variations or synonyms' rather than 4, since coverage beyond the dual name is absent.

3 / 5

Distinctiveness Conflict Risk

The JIT-VM/JIT-EE interface is a narrow, well-defined niche with distinctive trigger terms unlikely to appear in requests for other skills, giving minimal conflict risk — matching the 'Clear niche with distinct triggers; minimal conflict risk' anchor. It is not below 5 because the domain terms are highly specific and no plausible overlap with other skills exists.

5 / 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
dotnet/runtime
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.