Full-stack Code Review

VerifiedSafe

Perform static analysis and line-by-line review of full-stack code, detect correctness, performance, and quality issues, and generate a report.

Sby Skills Guide Bot
DevelopmentAdvanced
7/25/2026
Claude Code
#code-review#full-stack#rust#react#typescript

Recommended for

Automated summary

Performs a full-stack code review of a Rust/React application, including static analysis, edge case checks, and a prioritized report.

Strengths

  • Structured priority-based approach (P0/P1/P2) to guide fixes
  • Includes Rust-specific checks (unwrap, Mutex, Tauri) and React-specific ones (hooks, events)
  • Offers a step to enrich internal rules by analyzing error patterns

Limitations

  • Requires a Rust and Node.js environment for static analysis
  • Relies on predefined rules in .claude/rules/ whose quality affects detection
  • Does not automatically cover unit or integration tests
When to use it

For rigorous quality assurance before merging on a Rust/React Tauri codebase.

When not to use it

For projects without Rust/React components or when a quick informal review is sufficient.

Security analysis

Safe
Quality score85/100

The skill uses only static analysis commands (npx tsc, cargo check) and git operations for diffs; there are no destructive actions, exfiltration, or execution of arbitrary user-provided code. The npx whitelisting is restricted to the safe tsc compiler in the given instructions.

No concerns found

How this analysis is made

Examples

Review recent commits
Review the code changes from the last 3 commits in this full-stack Tauri project.
Review a specific directory
Do a full-stack code review on the src/ directory, covering both Rust and React code.
Review current branch diff
Run a full-stack code review on the current branch diff compared to main.

description: 代码审查(全栈) allowed-tools: Bash(git:, cargo:, npm:, npx:), Read, Glob, Grep argument-hint: [HEAD~N | file | directory]

全栈代码审查

$ARGUMENTS


1. 确定审查范围

根据参数确定要审查的代码:

  • 无参数:git diff 当前分支相对 main 的变更
  • HEAD~N:最近 N 次提交的变更
  • 文件/目录路径:指定范围

2. 执行静态分析

npx tsc --noEmit
cd src-tauri && cargo check

3. 逐文件审查(借鉴 Bugbot 策略)

核心方法论见 .claude/rules/code-review-strategy.md

对每个变更的函数/代码块,先做边界情况扫描:

[ ] 输入为空/null/undefined 时行为是否正确?
[ ] 错误路径是否会阻止后续执行(而非静默继续)?
[ ] 返回值的所有可能形态是否都被消费方处理?
[ ] 外部调用失败时是否有正确的错误传播?
[ ] 并发/重复执行时是否安全?
[ ] 状态变更是否会影响其他依赖方?

然后按优先级分类:

正确性(P0)

Rust 端:

  • 逻辑错误、边界情况(空值、零值、空数组)
  • unwrap() panic 风险
  • SQL 参数化(禁止字符串拼接)
  • Mutex 锁持有时间(禁止跨 await)
  • Tauri 命令是否在 lib.rs 中注册

React 端:

  • TypeScript 类型安全(是否有 any、类型断言)
  • React Hook 使用规则(条件调用、依赖数组)
  • Tauri Event 监听是否在 cleanup 中取消
  • 安全漏洞(硬编码凭证、XSS)

性能(P1)

  • Rust: 不必要的 clone()、大数据复制
  • React: 组件内定义对象/数组导致子组件重渲染
  • 数据库:缺少索引、N+1 查询
  • useEffect 依赖是否正确

代码质量(P2)

  • 文件是否超过 400 行
  • 命名是否清晰
  • 是否符合架构(依赖方向)
  • 是否有重复代码可以提取

4. 生成审查报告

以表格形式输出发现的问题和建议,按优先级排列:

  • P0:必须修复
  • P1:建议修复
  • P2:可选优化

5. 经验沉淀

问题修复后,回顾本次发现的问题,判断是否有可泛化的规律性教训应写入 .claude/rules/:

  • 对照 architecture.md、rust-patterns.md、react-patterns.md、security.md 等现有规则
  • 如果某类问题反复出现但 rules 中未覆盖 → 新增规则条目
  • 如果某条现有规则表述不够具体导致遗漏 → 补充细化
  • 输出一张"现有 rules 覆盖 vs 缺失"对照表,标注需要新增/强化的条目
  • 确认后更新对应 rules 文件

From the file to a first result

The download contains exactly the content displayed on this page. This copy may differ from the latest version in the source repository.

View the source and supporting files

This download contains SKILL.md only. Any scripts, references, templates and dependencies are not included. Get the required files from the source, preserving their directory structure, before continuing.

1. Download the file

Download SKILL.md

From the root of a test project, save the file as .skill-downloads/full-stack-code-review-mrx3xxva/SKILL.md, or use the command below. This folder is for inspecting the skill before installation.

Command for macOS/Linux or WSL: requires a POSIX shell and curl. On Windows without WSL, use the download button and create the folder manually.

mkdir -p '.skill-downloads/full-stack-code-review-mrx3xxva' && curl --fail --show-error --location 'https://skills-guides.com/api/skills/full-stack-code-review-mrx3xxva/download' --output '.skill-downloads/full-stack-code-review-mrx3xxva/SKILL.md.part' && mv '.skill-downloads/full-stack-code-review-mrx3xxva/SKILL.md.part' '.skill-downloads/full-stack-code-review-mrx3xxva/SKILL.md'

2. Check prerequisites

Read SKILL.md before using it. If it references scripts or external services, inspect them and prepare the required access in your test environment.

Tools declared by the author (check availability in your agent): Bash(git:*, cargo:*, npm:*, npx:*), Read, Glob, Grep

No recognized prerequisites section in this file. This does not mean it is self-contained: libraries, tools and access requirements may be mentioned elsewhere in the content or source repository.

The platforms on this page are catalog indications. The paths below follow tool documentation; they do not prove this skill has been run in those tools. No minimum tool version is confirmed for this listing.

3. Load the skill

This file does not provide a usable name/description header for this native procedure. It remains available to download and inspect; use the manual method below.

Manual method or another tool: attach SKILL.md to a conversation, or open it in an agent that can read files. Include any required supporting resources. No universal folder is recognized by every tool.

4. Try a first task

Use a short, synthetic input: an API endpoint, a small document or an interface brief. In a new conversation, attach the file or give the agent its path, then adapt this request:

Read .skill-downloads/full-stack-code-review-mrx3xxva/SKILL.md and apply its instructions to this example: [describe a test input]. Identify any missing files, dependencies or access. Produce the result requested by the skill without inventing data or running unreviewed scripts.

Expected result: output matching the skill instructions, or a precise list of missing prerequisites. A read confirmation alone does not validate a first use. If an external resource is unavailable, that path remains untested.

Related skills