agentleFS
Sign inStart free
535803710535803710/ai-rd-skills/skills/code-quality-review/SKILL.md

code-quality-review

Skill8 starsChanged 6 months agoMITView on GitHub

审查 TypeScript、JavaScript、TSX 和通用前端逻辑质量，关注命名、类型安全、API 调用、状态管理、工具函数、性能、安全、错误处理和测试遗漏；当用户要求检查代码质量、审查 TS/JS/API/store/utils 时使用。

Create a free account to mount
Files
SKILL.md

Code Quality Review

职责

只审查通用代码逻辑质量，不负责页面视觉规范，也不深入组件拆分。Vue 组件结构交给 vue-component-review，页面 UI 和交互交给 page-review。

检查重点

  • 命名是否表达业务含义
  • 类型是否安全，是否存在不必要的 any、断言或隐式假设
  • API 调用是否处理 loading、错误、取消和重复请求
  • 状态管理是否清晰，是否存在竞态或脏数据
  • 工具函数是否有边界处理和单测
  • 是否有重复逻辑、过深嵌套、复杂条件
  • 权限、安全、数据范围是否存在遗漏
  • 测试是否覆盖核心行为和异常分支

输出要求

  • findings 放在最前面，按 P0/P1/P2/P3 排序
  • 每个问题说明风险和建议，不只说“建议优化”
  • 没有问题时说明残余风险或测试缺口

Download now (ZIP)

  • Other versions23 projects use a version of this.

    Word for word · 0

    None found.

    Edited · 0

    None found.

    Wrote their own version · 23

    1. paiml/aprender★ 137same name
    2. jellydn/my-ai-tools★ 126same name
    3. berntpopp/genefoundry-router★ 6same name
    4. NovruzAliyevDev/skills★ 4same name
    5. VersoXBT/claude-recommended-skills★ 4same name
    6. ming-kang/Skills★ 3same name
    7. damacus/agent-skills★ 2same name
    8. berntpopp/hpo-link★ 1same name
    9. berntpopp/mgi-link★ 1same name
    10. berntpopp/stringdb-link★ 1same name

    13 more not listed

    How matches are found. Word for word: the exact same text. Edited: shares passages with this file (“99% same” means 99% of its five-word phrases also appear here; 10% or more counts). Their own version: similar meaning, or, weakest, the same name. We can’t tell which project had it first. Projects are counted among the 200 most-starred candidates, and mentions are links to the repository we found, up to 50 per site, so both counts are a floor.

  • Used alongsideProjects with it also use dependency-cve-sweep, fleet-standard-adoption and 6 more.

    Other files people use alongside this one.

    Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
    • berntpoppSkill

      dependency-cve-sweep

      Use when handling Dependabot PRs, CodeQL/Trivy alerts, or doing a routine dependency/CVE sweep for this repo.

      Also in 12 projects that use this

    • berntpoppSkill

      fleet-standard-adoption

      Use when bringing this repo into compliance with a GeneFoundry fleet standard (tool-naming, response-envelope, container-hardening, versioning, MCP transport, or logging) or closing its tracking issue.

      Also in 12 projects that use this

    • IgorGanapolskySkill

      ci-failure-triage

      Triage a CI / PR check failure by READING the failure body before forming any hypothesis. Triggered whenever a required check is red, a PR is BLOCKED, a merge won't land, or you're about to call a failure "transient", "flaky", "stale", or "orphaned". Prevents dismissing a real failure (e.g. real CodeQL security alerts) as noise.

      Also in 12 projects that use this

    • Microsoft AzureSkill

      release-readiness

      Complete ODIN release documentation and validation checks before declaring user-visible work complete, preparing a release, or opening the Release-to-main pull request.

      Also in 12 projects that use this

    • SentrySkill

      security-review

      Finds exploitable application security vulnerabilities in code changes. Use for Warden security scans, appsec review, OWASP-style checks, authentication or authorization bugs, injection, XSS, SSRF, path traversal, secrets, unsafe crypto, webhook verification, open redirects, or sensitive data exposure.

      Also in 13 projects that use this

    • AWSSkill

      retro

      Conduct a retrospective on a coding session.

      Also in 3 projects that use this

    • +2 more from berntpopp
  • In this repository17 other files it gives its agents.

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

Check your setup, free