CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

oma-qa

Review changes for correctness, security, accessibility, and performance. Use for scoped quality reviews or verification plans; bug fixes use oma-debug.

60

Quality

75%

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 ./.agents/skills/oma-qa/SKILL.md

The canonical home for this skill is oma-qa in first-fluke/oh-my-agent

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's core strength is an exceptionally clear, well-guarded review workflow: staged scenes, explicit verification with false-positive rejection, and a concrete severity/priority scheme. Its weaknesses are spec-style scaffolding that inflates token cost without adding instruction, and a reference layer that points at bundle files that do not exist alongside the skill.

Suggestions

Ship the referenced bundle files (resources/execution-protocol.md, iso-quality.md, checklist.md, error-playbook.md, verify-ship-protocol.md) with the skill, or remove the references — the body instructs 'follow for the selected task' for a file that is not present.

Cut the non-instructive scaffolding — the 'SSL primitive' column in the Actions table and the 'Resource scope' table — to reclaim tokens; neither changes what Claude does.

Add a short example of an ordered finding (severity, file:line, evidence, fix) so the required report format is concrete rather than implied by guardrails.

DimensionReasoningScore

Conciseness

The body mostly avoids explaining concepts Claude already knows, but a fair share of tokens go to non-actionable spec scaffolding — the 'SSL primitive' action table (READ/SELECT/CALL_TOOL), the 'Resource scope' table, and the Goal/Intent-signature meta sections — that could be tightened or cut. That matches anchor 3 (mostly efficient, some unnecessary content that could be tightened) rather than anchor 4, whose over-explanation is only minor.

3 / 5

Actionability

There is genuinely executable guidance: a canonical command path ('npm audit', 'bandit -r .', 'lighthouse <url>'), a concrete severity rubric (CRITICAL/HIGH/MEDIUM/LOW with definitions), an explicit priority order, and named standards (OWASP, WCAG 2.2 AA, ISO/IEC 25010/29119). It stays at anchor 4 rather than 5 because no example finding/report format is shown and per-stack commands are left as 'add project lint/test/coverage commands', leaving minor gaps.

4 / 5

Workflow Clarity

The PREPARE → ACQUIRE → REASON → VERIFY → FINALIZE sequence is clearly ordered, with an explicit validation stage ('VERIFY: Reproduce findings and reject false positives'), feedback loops ('If a finding cannot be reproduced, do not report it as a finding'), failure-and-recovery handling, and defined exit criteria — matching anchor 5's explicit validation steps, error-recovery loops, and checklists. It is not anchor 4 because checkpoints are explicit throughout rather than having minor gaps.

5 / 5

Progressive Disclosure

The References section is clearly signaled and one level deep, but every referenced bundle path — resources/execution-protocol.md, resources/iso-quality.md, resources/checklist.md, resources/error-playbook.md, resources/verify-ship-protocol.md, and ../_shared/core/* — is absent from the skill bundle (no resources/, references/, scripts/, or assets/ files exist), so navigation dead-ends for files the body says to 'follow'. This matches anchor 3 (structure present but organization impaired) rather than anchor 4's 'minor organization gaps', since core execution content is delegated to missing files.

3 / 5

Total

15

/

20

Passed

Description

70%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 compact, third-person description that answers both what and when, with an unusually good explicit de-confliction clause ('bug fixes use oma-debug'). Its main limitations are a single-action 'what' and a 'when' clause built from abstract categories rather than the concrete trigger phrases users would actually type.

Suggestions

Turn the when-clause into concrete user triggers, e.g. 'Use when the user asks for a code review, security audit, PR check, or pre-deployment QA pass' instead of 'scoped quality reviews or verification plans'.

Add one or two natural synonyms to broaden triggering — 'audit', 'code review', 'QA', or 'pre-launch check' — since users rarely say 'verification plan'.

Name the primary deliverable in the what-clause (e.g. 'Produces ordered findings with severity, file:line evidence, and fixes') to sharpen the single 'Review changes' action.

DimensionReasoningScore

Specificity

The description names the domain and a single action — 'Review changes for correctness, security, accessibility, and performance' — where the four quality dimensions are the object of one review action rather than several distinct concrete actions. It matches anchor 3 (domain plus 1-2 concrete actions, not comprehensive) rather than anchor 4, which expects several listed actions; no output artifacts (findings, severity, remediation) are named.

3 / 5

Completeness

Both parts are answered: what ('Review changes for correctness, security, accessibility, and performance') and when ('Use for scoped quality reviews or verification plans'), plus a boundary ('bug fixes use oma-debug'). The when-clause uses abstract category terms rather than concrete user-phrase triggers like 'when the user mentions X', which keeps it at anchor 4 instead of anchor 5.

4 / 5

Trigger Term Quality

Natural keywords are present — 'security', 'accessibility', 'performance', 'quality reviews', 'verification plans' — phrases users would plausibly say when requesting a review. Common variations like 'code review', 'audit', 'QA', 'test coverage', or 'OWASP' are absent, so it falls at anchor 4 (good coverage, a few natural terms missing) rather than anchor 5's comprehensive synonym coverage.

4 / 5

Distinctiveness Conflict Risk

'Review changes' carves out a QA-review niche and the explicit routing note 'bug fixes use oma-debug' actively disambiguates a neighboring skill. Minor overlap risk remains with generic code-review / PR-review skills, matching anchor 4 (mostly distinct, minor overlap with closely related skills) rather than anchor 5's minimal conflict risk.

4 / 5

Total

15

/

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

first-fluke/oh-my-agent

Path

.agents/skills/oma-qa/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.