CtrlK
BlogDocsLog inGet started
Tessl Logo

github-code-review

Comprehensive GitHub code review with AI-powered swarm coordination

59

3.69x
Quality

43%

Does it follow best practices?

Impact

85%

3.69x

Average score across 3 eval scenarios

SecuritybySnyk

Low

Low-risk findings worth noting

Fix and improve this skill with Tessl

tessl review fix ./.claude/skills/github-code-review/SKILL.md

The canonical home for this skill is github-code-review in ruvnet/agentic-flow

SKILL.md
Quality
Evals
Security

Quality

Content

42%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 is rich in concrete, executable commands but is formatted as a bloated product brochure: ~1100 lines of padded, repetitive feature documentation that Claude must wade through with no progressive disclosure. Its strongest points are copy-paste-ready gh/npx commands and complete CI workflow examples; its weakest are token efficiency and organization.

Suggestions

Cut the body to a lean quick-start workflow plus a short section per agent, moving comment templates, webhook handler code, configuration references, and agent check catalogs into references/ files linked one level deep.

Delete marketing padding: emoji benefits lists, 'Comprehensive'/'AI-Powered' taglines, license, support, and last-updated sections — they add tokens without adding information.

Present one canonical ordered workflow (fetch PR → init review → collect results → validate against gates → post comments → merge decision) with explicit validation checkpoints, instead of scattering the flow across feature sections.

DimensionReasoningScore

Conciseness

The body runs ~1100 lines with heavy padding: emoji section headers, '✅' benefits lists ('Parallel review by specialized agents'), repeated near-identical npx ruv-swarm invocations across sections, a license section, support/community footers, and a last-updated stamp. Almost none of this adds knowledge Claude lacks, matching the 'severely verbose; heavily padded' anchor.

1 / 5

Actionability

Commands are concrete and mostly executable: complete gh CLI invocations, CI workflow YAML, an inline-comment posting loop with jq, and a working CustomReviewAgent class. It is not level 5 because the webhook handler references an undefined 'body' variable, the nested code fence in the security comment template is broken, and no output format of ruv-swarm commands is documented, leaving some gaps.

4 / 5

Workflow Clarity

The document is a feature catalog rather than a sequenced workflow — the actual flow (fetch PR data → init review → run agents → post comments → gate merge) is scattered across sections with no explicit ordering. It involves batch/destructive operations (auto-merge, pr-fix --commit-fixes --push-changes) and while some conditional checks exist ('grep -q critical', 'REVIEWS -ge 2' before auto-merge), there is no validate→fix→retry feedback loop, capping workflow_clarity at 3.

3 / 5

Progressive Disclosure

There is real structure — a collapsible table of contents and consistent section headers — but the ~1100-line body inlines large blocks that clearly belong in separate reference files (comment templates, webhook handler code, the full review configuration reference, per-agent check catalogs), and the bundle contains no references/ files at all. This matches 'some structure but could be better organized; content that should be separate is inline'.

3 / 5

Total

11

/

20

Passed

Description

45%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 names its domain clearly but relies on marketing buzzwords ('Comprehensive', 'AI-powered', 'swarm coordination') instead of concrete capabilities, and provides no trigger guidance for when to use it. It would benefit from listing specific actions and adding an explicit 'Use when...' clause.

Suggestions

Replace buzzword qualifiers with concrete actions, e.g. 'Runs multi-agent security, performance, and style reviews on pull requests, posts inline review comments, and enforces quality gates'.

Add an explicit trigger clause: 'Use when the user asks to review a PR, pull request, or code changes on GitHub, or mentions code review comments or merge approval.'

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

DimensionReasoningScore

Specificity

The phrase 'GitHub code review' names the domain, but 'Comprehensive' and 'AI-powered swarm coordination' are generic buzzword qualifiers rather than concrete actions like 'posts review comments' or 'analyzes security issues'. It matches the anchor 'names the domain but actions are minimal or generic' and falls short of level 3 because no specific actions are enumerated.

2 / 5

Completeness

The description conveys a vague 'what' (comprehensive code review) but includes no 'when' or 'Use when...' clause at all, which caps completeness at 3 per the judging guidelines. It is not level 4 because the 'when' is entirely absent rather than merely implicit.

3 / 5

Trigger Term Quality

'GitHub code review' is a natural phrase a user might say, but the description omits common variations users would actually use such as 'PR', 'pull request', 'review comments', or 'request changes'. It has some relevant keywords but misses common synonyms, matching level 3 rather than 4.

3 / 5

Distinctiveness Conflict Risk

'GitHub code review' gives it a recognizable niche, but it would overlap with any other code-review or PR-related skill, and buzzwords like 'AI-powered swarm coordination' do not differentiate it. Somewhat specific but overlap risk remains, matching level 3.

3 / 5

Total

11

/

20

Passed

Validation

81%

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

Validation — 13 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

skill_md_line_count

SKILL.md is long (1141 lines); consider splitting into references/ and linking

Warning

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

relative_links

Relative link issues: 2 missing

Warning

Total

13

/

16

Passed

Repository
ruvnet/RuVector
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.