Code Review Assistant

What it does

When your agent uses it

• Requests a code review
• Phrases include take a look at this code
• Is there a problem with this function

Example prompts

• “take a look at this code”
• “review this”
• “is there a problem with this function”
• “/code-review-assistant”

Workflow steps

3 steps, taken from the first numbered list in SKILL.md.

1. 1Issue summary: one or two sentences summarizing the main issues found
2. 2Detailed findings: for each issue, include location, severity, description, and fix suggestion
3. 3Overall assessment: overall quality score and improvement direction

What it can do on your machine

Read from SKILL.md and the folder at commit 7f1ff9b. It shows what the files ask for, not the result of running them.

• Tool permissions

  Pre-approves nothing: there is no allowed-tools line, so your agent's usual permission prompts apply.

  From allowed-tools in the SKILL.md frontmatter.

• Runs code

  No scripts in the folder and no shell commands in SKILL.md.

  From the folder's file list and the shell code blocks in SKILL.md.

• Network

  No URLs in SKILL.md.

  From URLs in SKILL.md, links to its own repository left out.

• Credentials

  Names no API keys, tokens, secrets or passwords.

  From names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.

Context cost

Code Review Assistant loads about 505 tokens when it runs. Until then it costs about 86 tokens; SKILL.md has 147 words of instructions outside code blocks.

Always · name and description, kept in context so the agent knows when to use it

~86

When it runs · the whole SKILL.md, loaded when a task matches

~505

Estimates: characters ÷ 4, the usual rule of thumb; real counts depend on the model's tokenizer. Scripts and assets cost tokens only if the agent reads them.

Safety

Automated static check — not a guarantee. Review scripts before installing. It scans the text of SKILL.md for risky patterns (piping downloads into a shell, reading credential files, hidden Unicode, destructive commands); files beside SKILL.md are not scanned.

SKILL.md

The full file from alibaba/skill-up at commit 7f1ff9b, republished under its Apache-2.0 licence (© alibaba). 147 words, ~505 tokens.

name

code-review-assistant

description

Triggered when the user submits code or requests a code review. Automatically analyzes code quality, identifies potential bugs, security vulnerabilities, and performance issues, and provides improvement suggestions. Trigger phrases include "take a look at this code", "review this", "is there a problem with this function".

## Review Summary

Found 1 critical bug: null pointer dereference risk at line 42.

## Detailed Findings

### Bug #1: null pointer dereference (critical)

- **Location**: `src/handler.go:42`
- **Description**: `user.Profile.Name` is accessed directly when `user.Profile` may be nil
- **Fix suggestion**: add nil check `if user.Profile != nil { ... }`

## Overall Assessment

Code structure is clear but lacks critical null checks. Recommend adding defensive programming practices.

© alibaba, Apache-2.0. Rendered from Markdown: HTML in the file is shown as text, images as links, and headings moved down two levels. Raw file

Files

SKILL.md and 9 other files in e2e/testdata/text-match-rules of alibaba/skill-up.

• SKILL.md
• README.md
• evals/cases/find-null-bug.yaml
• evals/cases/miss-boundary-check.yaml
• evals/eval.yaml
• evals/evals.json
• fixtures/expected/find-null-bug-grading.json
• fixtures/expected/miss-boundary-check-grading.json
• fixtures/judge-inputs/find-null-bug.json
• fixtures/judge-inputs/miss-boundary-check.json

Open the folder on GitHubat commit 7f1ff9b