CtrlK
BlogDocsLog inGet started
Tessl Logo

veomni-review

Pre-PR code review gate. Run before opening a pull request, and again before pushing a substantive update to an open one — not per commit. Required when the PR's branch diff touches Python under veomni/, tasks/ or tests/, or CI workflows, pyproject.toml, uv.lock, docker/ or configs/. Also trigger proactively for runtime or configuration changes that span multiple files, touch shared infrastructure (BaseTrainer, distributed, model loading, data pipeline, ops dispatch), or have uncertain safety. Docs, comments and .agents/-only changes use the self-check below. The review launches a subagent that checks implementation quality, multi-file consistency, and known constraint violations, then rates the change as safe/needs-attention/risky.

73

Quality

90%

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

SKILL.md
Quality
Evals
Security

Quality

Content

88%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 highly actionable and well-sequenced, with executable commands, a complete subagent prompt, and an explicit verdict-gated feedback loop. Conciseness and progressive-disclosure are the only slightly weaker dimensions, due to minor git-rationale over-explanation and a large inline prompt block.

DimensionReasoningScore

Conciseness

Mostly efficient imperative prose with concrete commands and tables, but the git three-dot rationale ("it diffs from the merge base, so commits that landed on <base> after you branched do not show up") and the once-per-PR justification are mild over-explanation Claude could largely infer, fitting score-4 rather than the lean score-5.

4 / 5

Actionability

Fully executable guidance throughout — copy-paste commands ("git diff <base>...HEAD", "git add -N .", "git diff HEAD", "make quality") and a complete, ready-to-use subagent prompt template with placeholders, matching the score-5 anchor.

5 / 5

Workflow Clarity

Clear numbered sequence (steps 1–5) with explicit validation checkpoints — the verdict→action table is a feedback loop ("needs-attention" → fix then open; "risky" → do NOT open, wait for user) and "Run make quality before pushing" is a final gate — matching the score-5 anchor and satisfying the destructive-operation validation requirement.

5 / 5

Progressive Disclosure

No bundle files exist and the body is organized into clear, well-signaled sections (When this gate applies, Steps, Subagent Launch, After Commit); the ~45-line inlined subagent prompt is justified as the runtime-filled core but is substantial content that could otherwise live one level deep, keeping it at score-4 rather than score-5.

4 / 5

Total

18

/

20

Passed

Description

92%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 is strong, specific, and complete, clearly stating both what the gate does and when to trigger it with concrete, framework-anchored trigger phrases. Its only mild weakness is trigger-term breadth, where a couple of natural user phrasings are missing.

DimensionReasoningScore

Specificity

Lists multiple concrete actions — "launches a subagent that checks implementation quality, multi-file consistency, and known constraint violations, then rates the change as safe/needs-attention/risky" — with comprehensive coverage of what the gate does; fits the score-5 anchor and exceeds score-4's 'minor gaps'.

5 / 5

Completeness

Explicitly answers both what ("Pre-PR code review gate … launches a subagent … rates the change") and when ("Run before opening a pull request", "Required when the PR's branch diff touches …") with concrete trigger phrases, matching the score-5 anchor.

5 / 5

Trigger Term Quality

Good natural-term coverage with "code review", "pull request", and "PR" alongside concrete infra triggers, but a few common phrasings a user might say (e.g. "review my changes", "check this diff") are absent, keeping it just below comprehensive.

4 / 5

Distinctiveness Conflict Risk

Clear niche tied to a specific framework via "veomni/", "BaseTrainer", "FSDP2", and "ops dispatch"; distinct triggers and minimal overlap with generic review skills, matching the score-5 anchor.

5 / 5

Total

19

/

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.

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
ByteDance-Seed/VeOmni
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.