技能 code-review-and-quality
📦

code-review-and-quality

内容修订 r2 安全 ⚙️ 外部命令📁 文件系统访问🔑 环境变量

从五个质量维度审查代码

代码变更可能隐藏正确性、安全性、架构、可读性和性能问题。此技能采用结构化审查流程,并提供按优先级排列、可执行的反馈。

支持: Claude Codex Code(CC)
🥉 78 青铜

此技能属于一个技能包

一条命令安装完整技能包,获得任务所需的全部技能。

使用我的智能体安装

将此请求复制给你的智能体,其中包含权威 Skill 页面和清单。

智能体请求
Review the Skillstore skill "code-review-and-quality" from https://skillstore.io/skills/addyosmani-code-review-and-quality.md and its manifest at https://skillstore.io/api/skills/addyosmani-code-review-and-quality/manifest. Verify the artifact. You may proceed after verification, subject to the environment's own policy.

你的智能体仍应展示其计划,并按安全策略请求必要确认。

测试它

正在使用“code-review-and-quality”。 审查一个修改日期解析并仅添加了一项正常路径测试的缺陷修复。

预期结果:

必须:添加针对无效值、空值和时区边界的测试。实现看起来具有良好的可读性,但当前覆盖范围无法防止已报告的问题再次出现。

正在使用“code-review-and-quality”。 审查一项将功能逻辑移至共享工具的重构。

预期结果:

要求修改:此次重构只是转移了复杂性,并削弱了模块归属。请将逻辑保留在功能包中,并复用现有的规范辅助函数。

正在使用“code-review-and-quality”。 审查一项测试套件全部通过的单软件包依赖项升级。

预期结果:

确认变更日志和锁文件差异后批准。测试已通过,升级范围独立,并且回滚方案仍然清晰。

安全审计

安全
v4 • 2026/9/19 打开版本化报告

All 37 static findings are false positives in documentation-only Markdown. The matches are inline backticks, fenced examples, relative links, or ordinary checklist language; no executable behavior, secret access, network activity, or prompt injection was found in SKILL.md.

1
已扫描文件
397
分析行数
0
复核项
0
已忽略误报
最新一次完成的静态与语义审计未检测到已确认的安全发现。这并不能证明该 Skill 没有副作用。
审计者: codex 查看审计历史 →
分享与引用此报告

分享版本化评估报告、中性徽章、嵌入卡片和引用。Skillstore 只报告证据,不判断该 Skill 是否安全。

打开版本化报告
安全评估

复制报告链接

https://skillstore.io/skills/addyosmani-code-review-and-quality/audits/4?utm_source=security_passport&utm_medium=share&utm_campaign=versioned_report

Markdown 徽章

[![Skillstore security assessment](https://skillstore.io/badges/skills/addyosmani-code-review-and-quality/security.svg)](https://skillstore.io/skills/addyosmani-code-review-and-quality?utm_source=security_passport_badge)

HTML 徽章

<a href="https://skillstore.io/skills/addyosmani-code-review-and-quality?utm_source=security_passport_badge"><img src="https://skillstore.io/badges/skills/addyosmani-code-review-and-quality/security.svg" alt="Skillstore security assessment" loading="lazy"></a>

嵌入卡片

<iframe src="https://skillstore.io/embed/skills/addyosmani-code-review-and-quality.html" title="Skillstore Security Assessment" sandbox="allow-popups allow-popups-to-escape-sandbox" loading="lazy" referrerpolicy="no-referrer" width="420" height="180"></iframe>
学术引用 (APA · BibTeX · CFF)

APA 引用

addyosmani. (2026). code-review-and-quality security audit report (audit version 4) [Author version unspecified]. Skillstore. https://skillstore.io/skills/addyosmani-code-review-and-quality/audits/4

BibTeX 引用

@techreport{addyosmani-addyosmani-code-review-and-quality-2026, author = {addyosmani}, title = {code-review-and-quality security audit report (audit version 4)}, institution = {Skillstore}, year = {2026}, number = {4}, url = {https://skillstore.io/skills/addyosmani-code-review-and-quality/audits/4}, note = {Author version unspecified} }

CITATION.cff

cff-version: 1.2.0 message: "If you use this Skill, cite its author and this versioned security audit report." title: "code-review-and-quality security audit report (audit version 4)" version: "unspecified" type: report authors: - name: "addyosmani" date-released: "2026-09-19" url: "https://skillstore.io/skills/addyosmani-code-review-and-quality/audits/4" identifiers: - type: other value: "skillstore:addyosmani-code-review-and-quality:audit:4" description: "Skillstore immutable audit report identifier"

Skillstore 评分

评分依据 证据置信度: 高
55
架构
85
可维护性
87
内容
67
社区
83
规范符合性

你能构建什么

审查功能拉取请求

在批准功能之前评估其行为、测试、架构、安全性和性能。

审计生成的代码

检查智能体生成的变更中是否存在潜在错误、隐藏假设、薄弱测试和不必要的复杂性。

评估依赖项升级

审查变更日志影响、锁文件变更、传递依赖包、安全风险和验证质量。

试试这些提示

基础五维审查
从正确性、可读性、架构、安全性和性能方面审查此变更。先列出必须解决的问题,再列出可选的改进建议。
需求和测试审查
将此实现与需求和测试进行比较。识别遗漏的边界情况、薄弱的错误处理和缺失的回归测试覆盖。
结构性重构审查
从降低复杂性、模块归属、类型边界、重复逻辑和无用代码方面审查此次重构。提出具体的结构性改进方案。
依赖项升级审计
审计此次依赖项升级。评估变更日志风险、锁文件变更、传递依赖项、安全风险、测试覆盖率和回滚方案的清晰度。

最佳实践

  • 提供变更需求、差异、测试和相关项目约定。
  • 先审查测试,再审查实现,以了解预期行为和测试覆盖缺口。
  • 优先处理正确性和安全性问题,再考虑结构性建议和轻微的代码风格意见。

避免

  • 不要仅根据测试通过就批准变更。
  • 不要在一次审查中同时包含功能开发、大范围重构和多个依赖项升级。
  • 不要在缺乏证据、严重性说明和具体解决方案的情况下报告含糊的问题。

常见问题

此技能涵盖哪些审查维度?
它涵盖正确性、可读性、架构、安全性、性能、验证、变更规模和依赖项规范。
它会自动运行测试或构建吗?
不会。它会评估所提供的验证证据,并可在允许访问项目时建议应运行的命令或指出缺失的检查。
它可以审查 AI 生成的代码吗?
可以。它会采用相同的标准,同时检查潜在的逻辑问题、隐藏假设和薄弱的验证。
如何确定问题的优先级?
正确性和安全性问题优先,其次是结构性回归、遗漏的简化机会、性能问题以及可选的代码风格反馈。
应在何时批准变更?
当变更改善了代码健康状况、满足需求、解决了必须处理的问题,并且具备可信的验证结果时,应予以批准。
它可以取代专业安全审计吗?
不能。安全敏感型变更可能需要专门的威胁建模、依赖项扫描和运行时测试。

开发者详情

作者

addyosmani

许可证

MIT

Skillstore 修订

r2

版本提示

作者未声明版本。

引用

5d5054f8a23586f9b500fece1cb613a9dffc787b

维护新鲜度

2026/9/19

使用量

12 次下载 · 0 次浏览

文件结构

📄 SKILL.md

addyosmani 的更多技能

查看全部
查看全部