CtrlK
BlogDocsLog inGet started
Tessl Logo

reviewer

Skill do Reviewer Final para validação completa antes do deploy. Use quando precisar validar que todos os passos do pipeline foram executados, checar qualidade de código, confirmar segurança, confirmar QA, confirmar documentação, ou gerar relatório de aprovação/rejeição. Trigger em: "review final", "reviewer", "aprovação", "rejeição", "validação final", "gate de deploy", "checklist final", "aprovar deploy", "pode deployar", "pronto pra produção", "validar entrega", "review completo", "última verificação".

63

Quality

73%

Does it follow best practices?

Impact

No eval scenarios have been run

SecuritybySnyk

Passed

No known issues

Fix and improve this skill with Tessl

tessl review fix ./skills/11-reviewer/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

47%

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

This skill provides a comprehensive review gate with well-structured workflows and clear rejection/escalation processes, but suffers from significant verbosity issues including a duplicated section, overly detailed inline checklists that should be externalized, and explanations of concepts Claude already understands. The actionability is moderate — checklists are specific but lack executable examples or concrete command/tool usage patterns.

Suggestions

Remove the duplicated 'Constituicao como rubric primario' section (appears twice verbatim)

Move the detailed checklists (Pipeline, Código, Testes, Segurança, Documentação, Performance) to a separate referenced file like 'checklists/review-checklist.md' and keep only a summary in the SKILL.md

Add a concrete example of using the MCP tool `devkit_suggest_trailers` and show a sample review report output rather than just referencing templates

Remove boilerplate sections like 'Quando Usar/Nao Usar' and 'Entradas/Saidas Esperadas' — these describe obvious context that Claude can infer from the skill's purpose

DimensionReasoningScore

Conciseness

The content is excessively verbose with significant redundancy. The 'Constituicao como rubric primario' section is duplicated verbatim. It explains obvious concepts (what a reviewer does, what DRY/SOLID means to Claude), lists extensive checklists that could be in a referenced file, and includes boilerplate sections (Quando Usar/Nao Usar, Entradas/Saidas) that add little actionable value. The anti-rationalization table, while useful, adds bulk. Much of this content assumes Claude doesn't understand code review fundamentals.

1 / 3

Actionability

The checklists are concrete and specific (e.g., 'no console.log', 'npm audit sem HIGH/CRITICAL', 'coverage >= 80%'), and the rejection workflow has clear steps. However, there is no executable code or commands — everything is descriptive checklists and process descriptions. The skill references templates and MCP tools but doesn't show their actual usage or output format inline.

2 / 3

Workflow Clarity

The workflow is clearly sequenced: receive delivery → execute full checklist → mark OK/FAIL → approve or reject → generate report. The rejection workflow includes explicit feedback loops (reviewer rejects → orchestrator → responsible skill → fix → back to reviewer), a maximum cycle count (3), escalation paths, and conditional re-validation requirements (security changes require skill 06, logic changes require skill 05). Validation checkpoints are explicit throughout.

3 / 3

Progressive Disclosure

The skill references multiple external files (templates/review.md, templates/rejection.md, personas/code-reviewer.md, policies/*, GLOBAL.md) which is good progressive disclosure structure. However, the massive inline checklists (Pipeline, Código, Testes, Segurança, Documentação, Performance — ~60 lines of checkboxes) should arguably be in a referenced file rather than inline, making the SKILL.md itself a near-monolithic document. References are one-level deep and clearly signaled, which is positive.

2 / 3

Total

8

/

12

Passed

Description

100%

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

This is a strong skill description that clearly defines its purpose as a final reviewer before deployment, lists concrete validation actions, and provides an extensive set of natural trigger terms in Portuguese. It explicitly addresses both what the skill does and when to use it, making it easy for Claude to select appropriately from a large skill set.

DimensionReasoningScore

Specificity

Lists multiple specific concrete actions: validating pipeline steps, checking code quality, confirming security, confirming QA, confirming documentation, and generating approval/rejection reports.

3 / 3

Completeness

Clearly answers both 'what' (validate pipeline steps, check code quality, confirm security/QA/documentation, generate approval/rejection report) and 'when' (explicit 'Use quando' clause plus a dedicated 'Trigger em' list with specific phrases).

3 / 3

Trigger Term Quality

Excellent coverage of natural trigger terms in Portuguese including common variations like 'review final', 'aprovação', 'rejeição', 'pode deployar', 'pronto pra produção', 'validar entrega', 'última verificação' — these are terms users would naturally say.

3 / 3

Distinctiveness Conflict Risk

Clearly occupies a distinct niche as a final pre-deploy review/gate skill. The specific triggers like 'gate de deploy', 'aprovação', 'checklist final' are unlikely to conflict with general code review or testing skills.

3 / 3

Total

12

/

12

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.

Validation11 / 11 Passed

Validation for skill structure

No warnings or errors.

Repository
felvieira/claude-skills-fv
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.