CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

ship-task

Use when the user asks to implement a ready task file (its Solution filled), then review, commit and push it. No planning; not for work that stops before the commit. Self-contained — execute the inlined steps directly, do NOT invoke other skills.

60

Quality

76%

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 ./plugins/flowai/skills/ship-task/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

56%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 dense, highly operational composite workflow with excellent sequencing, validation gates, and concrete commands, undermined by verbosity, duplicated rules, garbled generated step numbering, and dangling references to undefined rules (11-13). The monolithic inline structure conflicts with progressive disclosure expectations for a file this size.

Suggestions

Remove the duplicated content: the manifest-detection chain (stated fully in both Implement step 2 and Review step 2) and the near-verbatim rename-grep rule (doc-sync step 2, stated twice) should each appear once and be cross-referenced.

Fix the garbled generated step numbering in the Implement step 6 and Push Phase steps 6-7 blocks, where nested "6. **6. Hand off..." duplications embed step 7's content inside step 6's parenthetical, and resolve the dangling references to Rules 11, 12, and 13 (the rules section only defines 1-10).

Trim the runtime-irrelevant meta-content (the maintainer note, the dated incident log "2026-09-02 and 2026-09-06: commit-doc-sync-gate...", and the multi-clause rationale essays) to tighten conciseness, or move durable rules the phases repeat into a single shared rules section referenced once.

DimensionReasoningScore

Conciseness

The body is noticeably verbose with several padded or duplicated sections: the manifest-detection priority chain is enumerated twice (Implement step 2 and Review step 2), the rename-grep rule appears near-verbatim twice in the doc-sync step, and dated incident logs ("2026-09-02 and 2026-09-06: commit-doc-sync-gate...") plus a maintainer note add meta-content irrelevant to runtime. Long single-bullet rationale essays (e.g. the ~400-word delegation paragraph) further inflate tokens; not a 1 because the content is project-specific operational rules with high signal density, not explanations of concepts Claude already knows.

2 / 5

Actionability

The guidance is largely executable with exact commands (git rev-parse --abbrev-ref HEAD, git rev-list --left-right --count HEAD...@{u}, export SHA=$(git rev-parse HEAD)), exact gate conditions, exit-code semantics, and a verbatim report template. However, steps reference "Rule 11", "Rule 12", and "Rule 13" which the Rules & Constraints section (numbered 1-10) never defines, and `<SCRATCH>` is undefined — concrete gaps that keep it below fully copy-paste-ready.

4 / 5

Workflow Clarity

The five phases are strictly sequenced with four explicit gates, explicit STOP semantics, feedback loops (TDD RED→GREEN→REFACTOR→CHECK, CI poll loop with retry/anomaly handling), and a final verification checklist. Not a 5 because the generated markup is garbled in places: Implement step 6 and Push steps 6/7 render as broken nested duplications ("6. **6. **Hand off to the next phase**", with step 7's content embedded inside step 6's parenthetical), making parts of the Push Phase ambiguous.

4 / 5

Progressive Disclosure

Section headings per phase and per gate give real structure, but the ~470-line body fully inlines four complete atoms (implement, review, commit, push) with no external reference files at all — content that clearly belongs in separate files is inline. This is an explicit self-containment design choice, but judged against the rubric it sits at the 'some structure, could be better organized' anchor rather than the minimal-structure one below it.

3 / 5

Total

13

/

20

Passed

Description

87%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 strong description: explicit what-and-when triggers, concrete pipeline actions, and well-drawn negative boundaries that sharply distinguish it from planning or partial-cycle skills. The only weaknesses are a missing mention of the reflect phase and slightly framework-bound trigger vocabulary.

DimensionReasoningScore

Specificity

The description lists several concrete actions — "implement a ready task file", "review, commit and push it" — but omits the Reflect/audit phase the body actually contains, leaving a minor coverage gap that fits the 'several specific actions; minor gaps' anchor rather than comprehensive coverage.

4 / 5

Completeness

It explicitly answers both questions with concrete trigger phrasing: what ("implement a ready task file... then review, commit and push it") and when ("Use when the user asks to implement a ready task file"), plus explicit negative scope ("No planning; not for work that stops before the commit").

5 / 5

Trigger Term Quality

Natural terms a user would say are present ("implement", "task file", "review", "commit", "push"), but a few common synonyms are missing (e.g. "ship", "land the change") and the trigger leans on framework-internal jargon ("ready task file (its Solution filled)") that users outside the framework would not say.

4 / 5

Distinctiveness Conflict Risk

It carves a clear niche — the post-planning continuation of a ready, Solution-filled task file — and explicitly excludes adjacent skills ("not for work that stops before the commit", "do NOT invoke other skills"), keeping conflict risk with planning or generic implement/review skills minimal.

5 / 5

Total

18

/

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

frontmatter_unknown_keys

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

Warning

referenced_paths_exist

Referenced path issues: 3 missing

Warning

Total

14

/

16

Passed

Repository

korchasa/flowai-plugins

Path

plugins/flowai/skills/ship-task/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.