CtrlK
BlogDocsLog inGet started
Tessl Logo

oma-qa

Quality assurance specialist for security, performance, accessibility, comprehensive testing, and quality standard alignment. Use for test, review, security audit, OWASP, coverage, lint work, and ISO/IEC 25010 or ISO/IEC 29119-aligned QA recommendations.

59

Quality

68%

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 ./benchmarks/runs/oma/.agents/skills/oma-qa/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

66%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 defines a clear, well-validated QA workflow (ordered scenes, explicit verification and recovery rules, severity rubric, concrete tool commands) that scores at the top of the workflow-clarity anchor. It is dragged down by agent-spec scaffolding that adds tokens without executable guidance, redundant triple listing of resources, and abstract core steps that delegate substance to referenced files not present in the bundle.

Suggestions

Cut the 'SSL primitive' Actions table and the 'Resource scope' table — they describe the agent's own machinery rather than instructing how to perform QA — and consolidate the three resource listings into the single References bullet list.

Add one concrete example finding (severity, file:line, evidence, remediation snippet) and a short report template so the abstract 'REASON/ANALYZE' steps become executable guidance.

Expand the 'ACQUIRE' step with stack-specific command selection (e.g., 'npm audit' for Node, 'pip-audit/bandit' for Python, 'brakeman' for Rails) instead of a bare three-command block, and ensure the referenced resources/ files actually ship with the skill.

DimensionReasoningScore

Conciseness

The body is dense bullets and tables with no basic-concept explanations, but it carries unnecessary scaffolding: the 'SSL primitive' Actions table (READ/SELECT/CALL_TOOL/COMPARE/VALIDATE/WRITE/NOTIFY) adds no executable value, and the resource files are listed three separate times (Dependencies, References prose, References bullets). Mostly efficient with clear cut candidates, matching anchor 3; not 4 because multiple sections are removable redundancy rather than minor trims.

3 / 5

Actionability

Concrete guidance exists — canonical commands ('npm audit', 'bandit -r .', 'lighthouse <url>'), a defined severity scale (CRITICAL/HIGH/MEDIUM/LOW), priority ordering (Security > Performance > Accessibility > Code Quality), and a required finding format ('file:line, description, and fix') — but a large fraction of the body is agent meta-description ('Intent signature', 'Scenes', 'Resource scope', Preconditions) and core steps remain abstract ('Analyze security, performance, accessibility, correctness, and test coverage') with no example finding or report format. Mixed concrete/abstract matches anchor 3; not 4 because the concrete material does not dominate the body.

3 / 5

Workflow Clarity

The sequence is explicit (Entry steps, then PREPARE → ACQUIRE → REASON → VERIFY → FINALIZE) with written validation checkpoints ('VERIFY: Reproduce findings and reject false positives', guardrail 'No false positives - every finding must be reproducible'), feedback loops for error recovery ('If an automated tool is unavailable, document that limit and do manual checks'; 'If a finding cannot be reproduced, do not report it'), and a self-check gate before submitting, plus success/partial-success exit criteria. This matches anchor 5 (clear sequence, explicit validation, error-recovery loops); not 4 because validation is explicitly written rather than merely present. No destructive/batch cap applies.

5 / 5

Progressive Disclosure

References are one level deep and well-signaled with per-file purpose labels and usage conditions ('Use `resources/iso-quality.md` when the user needs enterprise QA, audit readiness…'). However, none of the referenced files (`resources/*.md`, `../_shared/core/*.md`) exist in the provided bundle, the same resources are listed three times, and inline scaffolding (Actions table, Resource scope) arguably belongs in a separate file — matching anchor 4 (good structure, minor organization gaps); not 5 because of the duplicate listings and unverifiable reference targets.

4 / 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 solid description with an explicit and multi-trigger 'Use for' clause covering natural terms (test, review, security audit, OWASP, coverage, lint). Its main weakness is the 'what' half: it names quality domains through a role label rather than concrete actions, costing specificity and keeping completeness below the top anchor.

Suggestions

Lead with concrete third-person verbs instead of a role label, e.g., 'Audits code for OWASP Top 10 vulnerabilities, runs coverage and lint checks, verifies WCAG 2.1 AA accessibility, and maps findings to ISO/IEC 25010 quality characteristics.'

Add natural trigger synonyms users actually say — 'code review', 'QA pass', 'pre-deployment review', 'CI checks' — to broaden trigger coverage.

State the artifact the skill produces (severity-ranked findings with file:line references and remediation) so the 'what' answers what the user gets, not just what the agent is.

DimensionReasoningScore

Specificity

Names specific quality domains ('security, performance, accessibility, comprehensive testing, and quality standard alignment') but contains no concrete action verbs — 'Quality assurance specialist for…' describes a role, not what the skill actually does (e.g., audits code, runs coverage checks, reports severity-ranked findings). Fits anchor 3 (domain and some specifics, not comprehensive); not 4 because every action is generic rather than listed concretely.

3 / 5

Completeness

Both parts are explicit: the 'what' ('Quality assurance specialist for security, performance, accessibility, comprehensive testing, and quality standard alignment') and an explicit 'Use for…' trigger clause. Not 5 because the 'what' is a role label with the abstract phrase 'quality standard alignment' rather than concrete capabilities; not 3 because the 'when' is explicit and multi-trigger, not weakly implied.

4 / 5

Trigger Term Quality

Natural user phrasing is present: 'Use for test, review, security audit, OWASP, coverage, lint work'. Not 5 because common variations users would actually say — 'code review', 'QA', 'CI', 'bug findings', 'pen test' — are missing, so coverage is good but not comprehensive.

4 / 5

Distinctiveness Conflict Risk

Distinct niche triggers ('OWASP', 'ISO/IEC 25010 or ISO/IEC 29119-aligned', 'coverage', 'lint') make this mostly distinguishable, but the generic terms 'test' and 'review' create minor overlap risk with general code-review and testing skills. Matches anchor 4 (mostly distinct, minor overlap with closely related skills).

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
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.