菜单
Expert code reviewer with deep knowledge of software engineering best practices, design patterns, and code quality standards. Use this skill when reviewing code changes, pull requests, or conducting code quality assessments.
默认使用会先检查来源的 Prompt；你也可以切换为直接命令，或下载本地副本。
决定是否安装前，请先阅读 SKILL.md，以及 SkillsMP 当前展示的配套文件。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
直接命令不会经过审查 Prompt；运行前请先检查来源。
npx skills add https://github.com/laurenceputra/goal-portfolio-viewer --skill code-review命令会保持在同一行。复制前请横向滚动并检查完整内容。
想先保存到本地？可下载 SkillsMP 当前能够提供的文件。
QA engineer with expertise in software testing methodologies, test design, and quality assurance practices. Use this skill when planning tests, writing test cases, or improving test coverage and quality.
Combine security scanning and threat modeling for changes involving data handling, API interception, sync, storage, authentication, or encryption.
Required workflow for handling review findings with classification, focused fixes, verification, fresh-context subagent review, and PR artifact updates.
正在显示 SKILL.md
基于 SOC 职业分类
| name | code-review |
| description | Expert code reviewer with deep knowledge of software engineering best practices, design patterns, and code quality standards. Use this skill when reviewing code changes, pull requests, or conducting code quality assessments. |
| license | MIT |
| tags | ["code-review","quality","best-practices"] |
| allowed-tools | ["bash","git","markdown"] |
| metadata | {"author":"laurenceputra","version":"1.0.0"} |
Provide structured, actionable review feedback focused on correctness, security, performance, and maintainability.
Use this gate to prevent symptom-only fixes and require evidence-backed closure.
debugging-assistant.qa-testing and maps failures to tests and outcomes.eslint-disable added for convenienceMark as blocking if:
When blocked for ambiguity, require a human decision record only when ambiguity remains after applying the Spec-Clarity Gate (canonical definition in .agents/agent-instructions.md).
Reference: debugging-assistant escalation policy and qa-testing verification matrix.
Number.isFinite(), preserve null vs 0, round only at display.innerHTML, avoid inline event handlers, use textContent or DOM nodes.