CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

ac-qa-reviewer

Quality assurance review for implementations. Use when reviewing code quality, checking implementation standards, performing QA cycles, or validating feature quality.

Quality

71%

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

Quality

Content

61%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 well-structured, verifiable skill body: real executable Quick Start code, concrete quality gates, and appropriate deferral to an existing script. The main gaps are a generic dimension checklist that pads token spend, and a batch auto-fix workflow lacking an explicit re-validation loop before approval.

Suggestions

Trim the Review Dimensions bullet lists (which restate standard review knowledge like DRY and SQL injection) to the five dimension names, or move the checklists into a reference file.

Add an explicit feedback loop to the QA workflow: after the FIX step, re-run the quality gates and only proceed to APPROVE when all required_checks pass.

Expand the API Reference pointer with one or two key entry points from scripts/qa_reviewer.py (e.g. review_feature, gates configuration) so deferral is clearly signaled.

DimensionReasoningScore

Conciseness

The body is mostly efficient with no padded prose, but the five Review Dimensions sections (~25 lines) enumerate knowledge Claude already has — "DRY (Don't Repeat Yourself)", "Input validation", "SQL injection prevention", "Algorithm efficiency", "Type annotations" — generic checklists that could be collapsed to dimension names or moved to a reference. This exceeds the 'minor instances of over-explanation' of anchor 4 but is well short of the heavily padded anchor 2.

3 / 5

Actionability

The Quick Start is real, near-executable code verified against the bundle ("from scripts.qa_reviewer import QAReviewer" / "await reviewer.review_feature(\"auth-001\")" matches the actual async method at scripts/qa_reviewer.py:79), and the gates are concrete ("minimum_score: 70", named required_checks). Not 5 because `project_dir` is undefined in the example and the 7 workflow steps are abstract labels with no runnable commands; not 3 because the guidance is genuine executable code, not pseudocode.

4 / 5

Workflow Clarity

The 7-step QA workflow (SCAN → ... → APPROVE) is a clearly sequenced process backed by an explicit Quality Gates section, but the FIX step batch-modifies changed files with no stated feedback loop — there is no 're-run checks after fixes before approving' step. Per the rubric's cap for batch operations missing validate→fix→retry loops, workflow clarity is capped at 3 despite the clear sequence.

3 / 5

Progressive Disclosure

The body is a well-sectioned overview that defers implementation to a real one-level-deep reference ("See `scripts/qa_reviewer.py` for full implementation" — the file exists and exports QAReviewer), with the Quick Start also pointing at it. Not 5 because the API Reference section is a single terse pointer with no signposting of what the script provides (key methods, config options), leaving navigation into the bundle weakly signaled.

4 / 5

Total

14

/

20

Passed

Description

66%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 correct third-person voice, an explicit 'Use when' clause, and several natural trigger phrases. Its weaknesses are a generic, near-synonymous action list in place of concrete capabilities and moderate conflict risk with general code-review skills.

Suggestions

Replace the near-synonymous trigger verbs with the skill's concrete capabilities, e.g. 'Scores features across code quality, security, performance, testing, and documentation dimensions; enforces quality gates; auto-fixes simple issues.'

Sharpen distinctiveness by scoping the triggers to the QA-gate workflow (e.g. 'Use when running a QA pass on a completed feature or deciding whether to approve it') so it does not fire on ordinary code-review requests.

DimensionReasoningScore

Specificity

The 'what' is a single generic action ("Quality assurance review for implementations") and the trigger clauses ("reviewing code quality, checking implementation standards, ... validating feature quality") restate near-synonymous review verbs rather than distinct capabilities; no mention of scoring, quality gates, or auto-fix. It lists actions but they are overlapping and generic, matching the anchor 'names domain and 1-2 concrete actions, but not comprehensive' rather than the anchor 4 example of several distinct concrete actions.

3 / 5

Completeness

Both 'what' ("Quality assurance review for implementations") and an explicit 'when' ("Use when reviewing code quality, checking implementation standards, performing QA cycles, or validating feature quality") are present in third person. Not 5 because the 'what' is thin — one generic phrase with no concrete capability list — whereas the when clause carries all the detail.

4 / 5

Trigger Term Quality

Natural phrases users would say are present: "code quality", "implementation standards", "QA cycles", "validating feature quality". Not 5 because common synonyms and variations ("code review", "audit", "best practices", "lint") are missing; not 3 because coverage spans several natural terms beyond a single keyword.

4 / 5

Distinctiveness Conflict Risk

The trigger "reviewing code quality" would naturally fire on a generic code-review request, creating real overlap with code-review, lint, and security-review skills; "performing QA cycles" and "validating feature quality" provide only a partial niche. This matches 'somewhat specific but could still overlap with similar skills' rather than the minor-overlap anchor 4.

3 / 5

Total

14

/

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

fernandezbaptiste/Skrillz

Path

skills/ac-qa-reviewer/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.