CtrlK
BlogDocsLog inGet started
Tessl Logo

omc-plan

Strategic planning with optional interview workflow

45

Quality

47%

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

Quality

Content

70%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 highly actionable with excellent per-mode workflow sequencing, validation gates, and feedback loops — a model of executable instruction for a complex orchestration skill. Its main weaknesses are severe redundancy (the sequencing and state-lifecycle rules are each repeated 4-5 times, roughly doubling token cost) and a monolithic single-file structure with no progressive disclosure layer for the RALPLAN-DR, /goal, and advanced material.

Suggestions

State the sequential-review rule and the ralplan state lifecycle once each (e.g., in Execution_Policy or a single Tool_Usage bullet) and reference it from the steps instead of repeating full sentences five times — this alone would cut substantial tokens without losing any instruction.

Move the RALPLAN-DR summary structure, /goal workflow comparison policies, and Design Option Presentation / Question Classification material into a references/ file (e.g., references/ralplan-dr.md and references/goal-workflow.md) with clear one-level-deep pointers from the body.

Consolidate the Good/Bad examples down to the two most instructive pairs; the option-presentation and batching examples restate rules already specified verbatim in Execution_Policy.

DimensionReasoningScore

Conciseness

The sequential-review rule ('never parallel', 'Architect output MUST NOT be passed to Critic') is stated at least five times across steps 3, 4, the blockquote, step 5a, and the Tool_Usage CRITICAL bullet, and the ralplan state-lifecycle rules are repeated across Steps, Tool_Usage, Escalation, and the Final_Checklist. This matches 'Noticeably verbose; several unnecessary explanations or padded sections'; the content itself is skill-specific (not concepts Claude already knows), which keeps it above 1.

2 / 5

Actionability

Guidance is fully executable: exact state_write/state_clear calls with arguments, concrete Task(subagent_type=...) and Skill(...) invocations, real file paths (.omc/plans/, .omc/drafts/), quantitative thresholds (80%+ file/line citations, 90%+ testable criteria, max 5 iterations), and copy-ready option labels for the approval UI. Specific examples cover the common cases.

5 / 5

Workflow Clarity

Each of the four modes has an explicit numbered sequence with validation checkpoints: a re-review feedback loop (Critic rejects -> revise -> re-review), approval gates via structured AskUserQuestion, iteration caps with defined fallback behavior, state lifecycle rules for every exit path, and a final checklist. This matches the anchor with explicit validation steps, feedback loops, and checklists.

5 / 5

Progressive Disclosure

The file is a single ~250-line monolith with no bundle files at all: RALPLAN-DR deliberation structure, the /goal workflow comparison policies, the deprecation notice, and the examples all live inline where reference files would serve better. Section organization (XML-tagged sections, tables) is decent, which lifts it above anchor 2 ('content that clearly belongs in separate files is inlined' with minimal structure), but there is no reference layer to signal — matching 'Some structure but could be better organized... content that should be separate is inline'.

3 / 5

Total

15

/

20

Passed

Description

25%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 too thin to function as a trigger: it names the domain in abstract terms but gives no concrete capabilities, no natural trigger phrases, and no 'use when' guidance — all of which exist in the body's Use_When section but are absent from the frontmatter. It reads like a one-line summary rather than a routing signal, creating real conflict risk with other planning-type skills.

Suggestions

Move the body's Use_When triggers into the description, e.g. append 'Use when the user says "plan this", wants requirements gathered for a vague idea, wants consensus on a plan, or wants an existing plan reviewed.'

Replace the abstract 'Strategic planning' with concrete actions: 'Creates actionable work plans through interactive interviews, direct planning, or Planner/Architect/Critic consensus; reviews existing plans.'

Add distinguishing keywords users would actually say — 'plan', 'work plan', 'requirements', 'consensus', 'ralplan', 'review this plan' — to reduce overlap with generic planning skills.

DimensionReasoningScore

Specificity

The description names the domain ('Strategic planning') but lists no concrete actions — there is no 'creates work plans', 'interviews the user', or 'runs consensus review'. This matches the anchor 'Names the domain but actions are minimal or generic'; it is not a 3 because 1-2 concrete actions are never actually enumerated.

2 / 5

Completeness

It has a vague 'what' ('Strategic planning with optional interview workflow') and no 'when' clause at all — the trigger guidance exists only in the body, not the description. Per the guideline, a missing 'Use when...' clause caps completeness at 3, and with the vague what and entirely absent when this sits at the anchor 'Has a vague what and no when'.

2 / 5

Trigger Term Quality

Only 'Strategic planning' and 'interview workflow' appear — technical/jargon phrasing rather than natural user phrases like 'plan this', 'make a plan', or 'review this plan' (which the body's Use_When section actually contains but the description omits). Not a 3 since even the common variations are missing.

2 / 5

Distinctiveness Conflict Risk

'Strategic planning' is broad and would overlap with any roadmap, architecture, or general planning skill; the only distinguishing hint is 'optional interview workflow', which is not a trigger a user would emit. Anchor 2 ('Very broad; high overlap risk with many similar skills').

2 / 5

Total

8

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

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

Warning

Total

15

/

16

Passed

Repository
Yeachan-Heo/oh-my-claudecode
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.