CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

factory-review

Review a pull request for a Factory work item — history and context first, then a verdict published on the PR — and mark the review complete

Quality

70%

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 ./mastracode/factory/factory-skills/factory-review/SKILL.md
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 delivers exceptionally concrete, executable guidance with a rigorously sequenced workflow and validation loops at every fork — top-anchor actionability and workflow clarity. Its weaknesses are structural: extreme paragraph density and ~50KB of inlined operational protocol that a reference bundle (which already exists for category guidance) should absorb.

Suggestions

Move the Phase 6 publishing protocol (verdict edit-vs-post rules, author-identity handling, follow-up PR steps) and the Phase 3 test-check/test-report procedures into reference files under references/, keeping a short procedure summary in SKILL.md.

Break the 400-600-word single-paragraph bullets (e.g. the test-check and test-report bullets in Phase 3) into numbered steps or tables so each instruction is scannable.

Deduplicate verdict-publication mechanics, which are restated across the Phase 6 bullets, into one authoritative sequence.

DimensionReasoningScore

Conciseness

The body is ~50KB of unbroken mega-paragraphs — the Phase 3 test-check bullet alone is a ~600-word single paragraph, and verdict-publication mechanics are restated across multiple Phase 6 bullets. Not 1 because nothing explains concepts Claude already knows and there is no fluff padding; not 3 because the verbosity and density are pervasive throughout, not occasional — the rubric penalizes verbosity even when accurate.

2 / 5

Actionability

Fully executable throughout: exact `gh` commands with flags, a complete GraphQL reviewThreads query with pagination logic, exact tool invocations (`factory_review_source`, `source_control_publish_test_check`, `factory_record_review_verdict`) with their argument shapes, exact approval gates, and concrete fallback commands for every failure path. Copy-paste ready and covers the common cases.

5 / 5

Workflow Clarity

Six phases are clearly sequenced (context → existing signal → quality gate → history/architecture → verdict → handoff/publish) with explicit validation checkpoints and feedback loops: adversarial check before every approve, the in-run cross-check with stop-and-retry on mismatch, the head-moved check before publishing, gate-the-file checks before posting, and retry-only-after-rejection for the terminal call. This matches the top anchor's validate-fix-revalidate pattern.

5 / 5

Progressive Disclosure

The references that exist are real, clearly signaled, and one level deep (`references/categories/README.md` with 15 category pages, `references/archaeology.md` for command recipes), and the body points to them at the right moment. But the main file still inlines the bulk of the operational protocol — the entire Phase 6 publishing/edit/fallback rules and the test-check procedure are exactly the content that belongs in a reference file the body could point to. Not 4 because 'most content appropriately placed' fails for a 50KB monolith; not 2 because the references that do exist are well-used and correctly signaled.

3 / 5

Total

15

/

20

Passed

Description

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

The description states concrete, well-sequenced actions and is clearly scoped to Factory work items, but it has no 'Use when...' trigger clause and lacks natural synonym coverage ('PR', 'code review'). It reads as a capable summary of the skill's flow rather than a discovery-optimized trigger description.

Suggestions

Add an explicit trigger clause, e.g. 'Use when a Factory work item enters the review phase and its pull request needs a verdict.'

Include natural synonyms users would say — 'PR', 'code review', 'review the diff' — alongside 'pull request' to improve trigger term coverage.

Briefly enumerate the review's substantive checks (correctness, tests, scope, pattern-consistency) to raise specificity toward comprehensive coverage.

DimensionReasoningScore

Specificity

Names several concrete actions — 'Review a pull request', 'history and context first', 'a verdict published on the PR', 'mark the review complete' — but coverage of the review activity itself is summary-level, omitting testing, quality gating, and security review that the body treats as core. Not 5 because comprehensiveness would enumerate the substantive review activities; not 3 because it lists more than 1-2 actions.

4 / 5

Completeness

Has a clear 'what' (review PR, build history/context, publish verdict, mark complete) but no 'Use when...' clause or equivalent trigger guidance, which the judging guidelines explicitly cap at 3. The role-based trigger ('factory-phase signal shows role="review"') lives in the body, not the description, so 'when' is not even weakly implied here.

3 / 5

Trigger Term Quality

Includes the natural phrases 'Review a pull request' and 'Factory work item', but misses common variations and synonyms users would actually say — 'PR', 'code review', 'review the diff'. Anchor 4 requires good keyword coverage with only a few natural terms missing; the abbreviation and synonym space is entirely absent.

3 / 5

Distinctiveness Conflict Risk

'Review a pull request for a Factory work item' carves a clear niche tied to Factory-bound sessions, distinguishable from generic code-review skills. Not 5 because without the Factory qualifier the phrase 'review a pull request' would overlap directly with generic PR-review skills; minor overlap risk remains.

4 / 5

Total

14

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

referenced_paths_exist

Referenced path issues: 1 deeper-than-1-level

Warning

Total

15

/

16

Passed

Repository

mastra-ai/mastra

Path

mastracode/factory/factory-skills/factory-review/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.