CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-work

Post-implementation gate review: run manual QA on the real surface yourself, then launch ONE gate reviewer (never a panel) to audit goal, constraints, code quality, security, missed context, and QA evidence. Use before a PR handoff or when the user explicitly asks to review completed work.

Quality

85%

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

SKILL.md
Quality
Evals
Security

Quality

Content

77%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 orchestration core is excellent — a clearly sequenced, validated, fully executable two-lane review workflow with concrete templates. It is dragged down by a verbose, partially duplicated harness-compatibility preamble and a monolithic single-file layout that inlines material better kept in reference files.

Suggestions

Deduplicate the subagent-reliability guidance: the 'WORKING:/BLOCKED:' protocol, timeout-means-no-mailbox-update rule, and fallback conditions each appear in both the compatibility preamble and the 'Codex Subagent Reliability' section — state them once.

Move the Codex/OpenCode tool-translation tables and the multi_agent_v1/v2 surface-rewriting rules into a references/ file (e.g. references/harness-compat.md), keeping only a one-line pointer in SKILL.md.

Extract the ~70-line gate-reviewer prompt template into its own reference file (e.g. references/gate-reviewer-prompt.md) so the phase instructions stay scannable.

DimensionReasoningScore

Conciseness

The core Phases 0-4 are tight and every token earns its place, but ~45 lines of Codex harness-compatibility preamble and a 'Codex Subagent Reliability' section repeat the same WAIT/BLOCKED/fallback guidance nearly verbatim ('A timeout only means no new mailbox update arrived. Treat a running child as alive. Fallback only when...' appears twice), which is unnecessary duplication. Mostly efficient with clear tightening opportunities, rather than noticeably padded throughout.

3 / 5

Actionability

Fully executable throughout: copy-paste bash for context collection and mining, concrete per-channel QA artifact instructions, a complete gate-reviewer prompt template with placeholders, a QA matrix schema, explicit verdict logic, and a final report format. These cover the common cases a practitioner would hit.

5 / 5

Workflow Clarity

Phases 0-4 are explicitly sequenced with validation checkpoints and error-recovery loops: a QA FAIL ends the review before wasting the reviewer, INCONCLUSIVE lanes get a bounded respawn with a retry budget, re-reviews are a fresh delta-scoped pass rather than a stale followup, and worktree teardown ordering is specified relative to reviewer completion.

5 / 5

Progressive Disclosure

The single ~280-line file has clear section headers, but no bundle files exist and content that clearly belongs in separate reference files is inlined: the Codex/OpenCode tool-compatibility tables and the 70-line gate-reviewer prompt template. Structure is present but the split is missing, matching the 'could be better organized; content that should be separate is inline' anchor.

3 / 5

Total

16

/

20

Passed

Description

85%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 strong description: concrete, third-person, and explicit about both capability and triggers, with an unusually clear enumeration of the audit dimensions. Its main weakness is trigger overlap with the broad family of code-review skills and modest keyword variation.

DimensionReasoningScore

Specificity

The description enumerates multiple concrete actions — 'run manual QA on the real surface yourself, then launch ONE gate reviewer' — and comprehensively lists the audit dimensions ('goal, constraints, code quality, security, missed context, and QA evidence'), covering essentially everything the skill does. It fits the 'comprehensive coverage' anchor rather than the 'minor gaps' anchor below it.

5 / 5

Completeness

Both halves are explicit: what it does (manual QA plus one gate reviewer auditing a listed set of dimensions) and when to use it ('Use before a PR handoff or when the user explicitly asks to review completed work'). This matches the anchor for clearly answering both what and when with concrete trigger phrases.

5 / 5

Trigger Term Quality

'Use before a PR handoff or when the user explicitly asks to review completed work' plus 'gate review' and 'manual QA' give good natural keyword coverage a user would actually say. Not 5: common variations like 'before opening a PR', 'check my work', 'QA', or 'sign off' are missing.

4 / 5

Distinctiveness Conflict Risk

'Post-implementation gate review' and 'review completed work' carve a real niche, but 'review' is an extremely common trigger that overlaps with generic code-review and PR-review skills. It is somewhat specific yet could still overlap with similar skills, rather than being mostly distinct with only minor risk.

3 / 5

Total

17

/

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

code-yeongyu/oh-my-openagent

Path

packages/shared-skills/skills/review-work/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.