SuperAPI AI 工具集

code-review

测试与质量

查看项目

更新于 10-16 00:31

技能介绍

用途:审查 microsoft/aspire 仓库的 GitHub 拉取请求,仅识别问题,包括 bug、安全风险、正确性错误、性能回退、系统边界处缺失的错误处理,以及仓库约定违规;不对风格偏好发表评论。\n\n使用流程:先解析用户请求中的 PR 编号或完整 URL,仓库默认 microsoft/aspire;未给出编号时可检查当前分支是否有开放 PR:gh pr view --json number,title,headRefName 2>/dev/null。第一步(阻塞步骤)必须先在本地确保 PR 分支可用,否则不得获取 diff 或文件列表。检查当前分支可用:gh pr view <number> --repo microsoft/aspire --json headRefName --jq '.headRefName' 和 git branch --show-current;不一致时由用户选择:检出分支(推荐,必要时先 git status --porcelain、git stash push -m "auto-stash before PR review of #<number>",再 gh pr checkout <number> --repo microsoft/aspire),或仅用 GitHub diff 审查(质量可能下降)。\n\n第二步收集上下文:使用 mcp_github_pull_request_read 的 get、get_files、get_diff、get_review_comments 方法获取标题、描述、基础分支、作者、变更文件、完整 diff 与已有评论,避免重复;MCP 不可用时回退到 gh CLI。第三步按路径分组变更(Hosting、Dashboard、Integrations/Components、CLI、Tests、Deployment、Build/Infra、API 文件、Extension、Docs/Config),api 目录下的 src//api/.cs 不应手工编辑。\n\n第四步审查代码并阅读周边上下文,进行影响分析与测试覆盖评估:将变更代码路径映射到可能回退的行为,并对照测试改动;对条件测试选择(Conditional Test Selection)规则检查触发映射与 run_* 接线;依据变更类型判断所需覆盖(单元/集成、CLI 端到端、Dashboard Playwright、部署端到端、VS Code 扩展 E2E),纯视觉样式变更不要求自动化断言。\n\n应报告的问题类别:逻辑错误与空引用、缺失 await、竞态与资源释放、注入与凭证泄露等安全问题、相对 PR 描述或既有契约的正确性错误、ATS 表面破坏性变更、被弱化的不变量、公共 API 边界缺失的输入校验、热路径分配与阻塞式异步调用、并发问题、时序耦合与初始化安全。资料未提供收费、价格、平台或效果信息。