reviewing-prs

作者: flutter

使用gh CLI获取、检查、审查GitHub拉取请求、起草与用户对齐的审查评论并在……之前获得批准的一般工作流程…

npx skills add https://github.com/flutter/devtools --skill reviewing-prs

Reviewing Pull Requests

This skill outlines the workflow for inspecting GitHub Pull Requests using the gh CLI, drafting review feedback, and securing user approval before posting review comments.

Approval Safeguard (Strict Requirement)

[!IMPORTANT] NEVER post comments or reviews to GitHub without explicit prior user approval. Always present draft review comments to the user in natural language first. Only execute write commands (gh pr comment, gh pr review) after the user approves.

Workflow

1. Request Information via GitHub CLI

  • PR Details:
    gh pr view <pr-number> --repo <owner/repo> --json title,body,author,state,headRefName,baseRefName,comments,reviews,files
    
  • Code Diff:
    gh pr diff <pr-number> --repo <owner/repo>
    
  • Existing Inline Comments:
    gh api repos/<owner/repo>/pulls/<pr-number>/comments
    
  • CI / Status Checks:
    gh pr checks <pr-number> --repo <owner/repo>
    

2. Inspect Context & Prior Feedback

  • Read the PR description, linked issues, and full diff.
  • Verify whether existing bot or human comments have already been addressed in subsequent commits.

3. Draft Review Comments

  • Keep comments direct, concise, and focused on code quality and correctness.
  • Approvals: Keep comments concise (LGTM or A couple comments but lgtm.). Avoid fluffy praise or re-summarizing the PR.
  • Actionable Feedback: Reference specific files, line numbers, and rationale when leaving suggestions.

4. Present Draft & Post Only Upon User Approval

[!IMPORTANT] NEVER post comments or reviews to GitHub without explicit prior user approval.

  1. Present the drafted review comments to the user in your response window.
  2. Ask for confirmation: "Would you like me to submit this review to GitHub?"
  3. Once explicitly approved by the user, post the review:
    gh pr review <pr-number> --repo <owner/repo> --comment --body "<approved review text>"
    

来自 flutter 的更多技能

dart-modern-features
flutter
为了找到现代化的候选方案:
flutter-fix-layout-issues
flutter
使用Dart和Flutter MCP工具修复Flutter布局错误(溢出、无界约束)。适用于处理“RenderFlex溢出”、“垂直…
adding-release-notes
flutter
在 DevTools 发布说明中添加面向用户的变更描述。用于在 NEXT_RELEASE_NOTES.md 文件中记录改进、修复或新功能。
reviewing-devtools-prs
flutter
DevTools 仓库专属的PR审查工作流程,执行DevTools风格指南和常见审查模式。在审查……中的拉取请求时使用。
dart-use-primary-constructors
flutter
帮助用户在Dart中编写语法和语义正确的主构造函数,并迁移/使用新的构造函数语法、空体分号语法等。
code-documentation
flutter
编写有效代码文档的指南,包括docstrings、JSDoc、dartdoc和实现注释。在编写新代码、添加…时使用此技能。
api-review
flutter
对照规范的API设计指南审查指定代码。当用户要求进行API审查或根据API设计规范检查代码时,请使用此技能…
flutter-accessibility
flutter
在Flutter应用中实现WCAG 2和EN 301 549无障碍标准及自适应布局。在移动端、网页端和桌面端强制实施语义标注、点击目标尺寸(最小48x48 dp)和文本对比度(小文本4.5:1,大文本3:1)。提供网页语义初始化、交互式组件包裹、基于屏幕尺寸的布局切换以及键盘/鼠标输入处理的决策逻辑。包含通过FocusTraversalGroup实现的焦点遍历管理,以及...