CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

implement-posix-command

Implement a new POSIX command as a builtin in the safe shell interpreter

56

Quality

71%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

Fix and improve this skill with Tessl

tessl review fix ./.claude/skills/implement-posix-command/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

77%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 content is an exceptionally actionable, well-gated implementation workflow with concrete code, commands, and validation loops throughout. Its weaknesses are systematic repetition of the gating protocol and a monolithic single-file structure that inlines material (test helpers, pentest and fuzz checklists, GNU-compat table) better kept in one-level-deep reference files.

Suggestions

Move the stable runScript/cmdRun test helper, the GNU-compat coverage table, and the pentest/fuzz seed checklists into a references/ file (e.g. references/testing.md) and link to them from the steps, keeping SKILL.md as an overview.

Consolidate the repeated per-step GATE CHECK boilerplate into one statement of the gating rule in the STOP section, with each step referring back to it.

Trim redundant admonitions (e.g. repeated 'do NOT skip' warnings and re-explained parallelism rules) to reduce token cost without losing the protocol.

DimensionReasoningScore

Conciseness

Most content is genuinely non-inferable project knowledge (file layout, import allowlist rules, echo builtin quirks, gate protocol), but there is noticeable padding: the gating instructions repeat the same TaskList/checkpoint boilerplate across every step, and the STOP section reiterates points made later. This fits 'Mostly efficient but includes some unnecessary explanation or could be tightened' — not 4, since the repetition is systematic rather than minor.

3 / 5

Actionability

The body provides copy-paste-ready Go test helpers (runScript, cmdRun, cmdRunCtxFuzz), exact function signatures, a concrete YAML scenario template, exact bash commands, and specific test matrices (integer edge cases, path edge cases, GNU-compat table). Fully executable guidance covering the common cases — matches the top anchor.

5 / 5

Workflow Clarity

A 10-step workflow is explicitly sequenced with a stated execution order, per-step gate checks (TaskList verification before advancing), parallelization rules, and validation feedback loops (run tests, fix, re-run, plus a second independent review pass). This matches 'Clear sequence with explicit validation steps; feedback loops for error recovery; checklists for complex processes'.

5 / 5

Progressive Disclosure

There are no bundle files (references/, scripts/, assets/ are absent) and the entire ~600-line body is one monolithic file: stable test-helper code, the GNU-compat coverage table, the pentest checklist, and the fuzz seed-corporus guidance are all inlined where they belong in separate reference files. Section headers keep it navigable, which lifts it above the 'minimal structure' level 2, but content that should be separate is inline with no references — anchor 3.

3 / 5

Total

16

/

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 is specific to its codebase and states a clear, concrete purpose, but it is missing any 'use when' trigger guidance and reads as a task title rather than a capability description. Adding explicit trigger conditions and a slightly fuller picture of the workflow would raise completeness and trigger-term quality.

Suggestions

Add a 'Use when...' clause (e.g. 'Use when asked to add or port a POSIX command like tail, head, or sort to the safe shell interpreter') to satisfy the trigger/when requirement.

Include natural trigger synonyms and concrete command examples (e.g. 'POSIX commands such as tail, head, cut, uniq') so users' phrasing matches naturally.

Briefly enumerate the workflow scope (research POSIX/GTFOBins behavior, tests, implementation, pentest, fuzzing) so the 'what' is more comprehensive.

DimensionReasoningScore

Specificity

The description names a concrete action ('Implement a new POSIX command as a builtin') and a specific target ('the safe shell interpreter'), but covers only that single action without enumerating what the implementation entails (tests, review, hardening). This matches the anchor 'Names domain and 1-2 concrete actions, but not comprehensive' — not 4, which requires several specific actions, and not 2, since the action is concrete rather than generic.

3 / 5

Completeness

It clearly answers 'what' (implement a POSIX command as a builtin) but has no 'when to use' clause whatsoever, which caps completeness at 3 per the judging guidelines. Not 4 because the 'when' is entirely absent rather than just imprecise; not 2 because the 'what' is clear, not vague.

3 / 5

Trigger Term Quality

Terms like 'POSIX command', 'builtin', and 'safe shell interpreter' are relevant and natural for this codebase, but there are only a few of them with no synonyms or common variations. This fits 'Some relevant keywords but missing common variations or synonyms' — better than the generic one-or-two-keyword level 2, short of the good-coverage level 4.

3 / 5

Distinctiveness Conflict Risk

'The safe shell interpreter' pins the skill to a specific project, making it mostly distinct with only minor overlap risk against other command-implementation skills. Not 5 because without trigger phrasing the fairly broad 'implement a new POSIX command' could still collide with sibling implementation skills.

4 / 5

Total

13

/

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

skill_md_line_count

SKILL.md is long (611 lines); consider splitting into references/ and linking

Warning

frontmatter_unknown_keys

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

Warning

Total

14

/

16

Passed

Repository

DataDog/rshell

Path

.claude/skills/implement-posix-command/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.