提供专业的代码审查服务,检查代码质量、安全漏洞、性能问题和最佳实践。支持多种编程语言和框架。
npx skills add https://github.com/bruc3van/agent-skills-guard --skill yao-codereview-hskill
提供全面的代码审查和质量改进建议。
description.SKILL.md lean.references/, templates in templates/.Quick: 快速扫描,关注明显问题。Security: 安全聚焦,深入检查漏洞。Full: 全面审查,含架构和可维护性。| 维度 | 检查项 |
|------|--------|
| 安全 | 注入、认证、授权、敏感数据 |
| 性能 | 算法效率、资源泄露、缓存策略 |
| 可读性 | 命名、注释、结构 |
| 可维护性 | 耦合度、测试覆盖、文档 |
| 规范性 | 编码规范、提交规范 |
## 审查摘要
- 代码行数: X
- 发现问题: Y (严重: a, 警告: b, 建议: c)
## 严重问题
[问题列表]
## 警告
[问题列表]
## 改进建议
[建议列表]
## 修复示例
[代码示例]
审查时参考以下最佳实践模板:
# -*- coding: utf-8 -*-
"""
项目配置模块
Author: Code Quality Team
Maintainer: [email protected]
Version: 1.0.0
"""
import os
from dataclasses import dataclass
@dataclass
class Config:
"""应用配置类"""
debug: bool = False
log_level: str = "INFO"
# 更多配置项...
/**
* @fileoverview 模块说明
* @author Code Quality Team
* @contributor 姚金刚 (yaojingang)
* @license MIT
*/
'use strict';
// 模块实现...
对于遗留代码,建议:
自动化工具处理规范性检查,人工审查关注:
Expert in secure backend coding practices specializing in input validation, authentication, and API security. Use PROACTIVELY for backend security implementations or security code reviews.
Expert in secure mobile coding practices specializing in input validation, WebView security, and mobile-specific security patterns. Use PROACTIVELY for mobile security implementations or mobile security code reviews.
Expert in secure frontend coding practices specializing in XSS prevention, output sanitization, and client-side security patterns. Use PROACTIVELY for frontend security implementations or client-side security code reviews.
Security hardening reviewer for GitHub Actions workflow files (.github/workflows/*.yml). Reasons about the Actions threat model that pattern matchers and general code linters miss — untrusted-input script injection, privileged triggers running fork code, mutable action references, and over-scoped tokens. Use this skill when asked to review, audit, harden, or secure a GitHub Actions workflow, when writing a new workflow, or for any request like "is this workflow safe?", "review my CI for security issues", "why is pull_request_target dangerous here?", "pin my actions", or "lock down GITHUB_TOKEN permissions". Covers script injection via ${{ }} interpolation, pull_request_target / workflow_run privilege escalation, SHA-pinning of third-party actions, least-privilege permissions, GITHUB_ENV/GITHUB_OUTPUT injection, secret exposure, OIDC over long-lived credentials, and self-hosted runner exposure on public repositories.
Review a PR, or a PR linked to an issue, for security risks. Check nine categories and report PASS, WARNING, or FAIL. Use when reviewing code for vulnerabilities, secrets, injection, authorization bypasses, or unsafe configuration. Trigger keywords - security review, code review, appsec, vulnerability assessment, security audit, review PR security.
>- Run Semgrep static analysis scan on a codebase using parallel subagents. Supports two scan modes — "run all" (full ruleset coverage) and "important only" (high-confidence security vulnerabilities). Automatically detects and uses Semgrep Pro for cross-file taint analysis when available. Use when asked to scan code for vulnerabilities, run a security audit with Semgrep, find bugs, or perform static analysis. Spawns parallel workers for multi-language codebases.
Expands one confirmed or suspected vulnerability into a Trailmark graph neighborhood of variant candidates by finding sibling functions, shared callers and callees, common sensitive sinks, common entrypoint paths, interface implementations, override relationships, type/reference neighbors, and structurally similar nodes. Use after one issue is found to seed variant-analysis, semgrep-rule-creator, static-analysis, or manual review with graph-derived candidate locations.
Apply modern web development best practices for security, compatibility, and code quality. Use when asked to "apply best practices", "security audit", "modernize code", "code quality review", or "check for vulnerabilities". Do NOT use for accessibility (use web-accessibility), SEO (use seo), performance (use core-web-vitals), or comprehensive multi-area audits (use web-quality-audit).
Take bruc3van/yao-codereview-hskill from the repository into ~/.claude/skills for personal
use, or into .claude/skills inside a project.
The agent identifies a skill by the name field in its header. Two skills with the
same name cannot sit side by side — one of them will be ignored.