Agent skill

Reviewing Devtools PRs

by flutter in flutter/devtools

DevTools repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns.

BSD-3-ClauseAuto-check passedDevelopment

Install Reviewing Devtools PRs

skills CLI
$ npx skills add flutter/devtools --skill reviewing-devtools-prs -a claude-code

Project install by default; add -g for ~/.claude/skills/.

GitHub CLI
$ gh skill install flutter/devtools reviewing-devtools-prs --agent claude-code

Project scope by default; add --scope user for a personal install. Needs GitHub CLI 2.90.0 or later (public preview).

Manual copy
$ git clone --depth 1 https://github.com/flutter/devtools.git skills-src && mkdir -p .claude/skills && cp -r skills-src/.agents/skills/reviewing-devtools-prs .claude/skills/reviewing-devtools-prs && rm -rf skills-src

Use ~/.claude/skills/ instead of .claude/skills for a personal install. The folder must contain SKILL.md.

Claude Code skills documentation · loads skills from .claude/skills/

Facts

Skill name
reviewing-devtools-prs
GitHub stars
1.7k
Token cost
~638 tokens
SKILL.md length
242 words
Files
1
Skills in repo
10
Repo updated
First seen
Licence
BSD-3-Clause

At a glance

DevTools repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns.

  • Works in 9 steps: Listener & Resource Disposals → Helper Widgets over Helper Methods → Reuse Shared Components & Test Helpers → …
  • Reviewing pull requests in the flutter/devtools repository
  • SKILL.md covers References & Style Guidelines and Common Review Patterns
  • Instructions only: no scripts, shell commands, URLs or credentials in SKILL.md

What it does

Reviewing Devtools PRs is an agent skill from flutter/devtools. DevTools repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns. Use when reviewing pull requests in the flutter/devtools repository.

Its SKILL.md is about 640 tokens, which your agent loads only when the skill is triggered. It is a single SKILL.md file with no bundled scripts.

It sits in Development, covering Pull requests and Cross-platform mobile apps. It works with Flutter. The repository describes itself as: Developer tools for Flutter & Dart. The licence is BSD-3-Clause.

When your agent uses it

  • Reviewing pull requests in the flutter/devtools repository
  • Tasks that involve Pull requests
  • Tasks that involve Cross-platform mobile apps

Example prompts

  • “Use the reviewing-devtools-prs skill to devtool repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns”
  • “/reviewing-devtools-prs”

Workflow steps

9 steps, taken from the first numbered list in SKILL.md.

  1. Listener & Resource Disposals
  2. Helper Widgets over Helper Methods
  3. Reuse Shared Components & Test Helpers
  4. TODO Formatting
  5. Async & Unawaited Futures
  6. Feature Flags
  7. Test File Structure & PR Scope
  8. Constant Scoping
  9. Release Notes Scope (NEXT_RELEASE_NOTES.md)

What it can do on your machine

Read from SKILL.md and the folder at commit e25ba13. It shows what the files ask for, not the result of running them.

  • Tool permissions

    Pre-approves nothing: there is no allowed-tools line, so your agent's usual permission prompts apply.

    From allowed-tools in the SKILL.md frontmatter.

  • Runs code

    No scripts in the folder and no shell commands in SKILL.md.

    From the folder's file list and the shell code blocks in SKILL.md.

  • Network

    No URLs in SKILL.md.

    From URLs in SKILL.md, links to its own repository left out.

  • Credentials

    Names no API keys, tokens, secrets or passwords.

    From names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.

Context cost

Reviewing Devtools PRs loads about 638 tokens when it runs. Until then it costs about 51 tokens; SKILL.md has 242 words of instructions outside code blocks.

Always · name and description, kept in context so the agent knows when to use it
~51
When it runs · the whole SKILL.md, loaded when a task matches
~638

Estimates: characters ÷ 4, the usual rule of thumb; real counts depend on the model's tokenizer. Scripts and assets cost tokens only if the agent reads them.

Safety

Auto-check passed

The automated check found no risky patterns in SKILL.md.

Automated static check — not a guarantee. Review scripts before installing. It scans the text of SKILL.md for risky patterns (piping downloads into a shell, reading credential files, hidden Unicode, destructive commands); files beside SKILL.md are not scanned.

SKILL.md

The full file from flutter/devtools at commit e25ba13, republished under its BSD-3-Clause licence (© flutter). 242 words, ~638 tokens.

Download SKILL.mdSave it as .claude/skills/reviewing-devtools-prs/SKILL.md (or your agent's skills folder).
name
reviewing-devtools-prs
description
DevTools repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns. Use when reviewing pull requests in the flutter/devtools repository.

Reviewing DevTools Pull Requests

Extends reviewing-prs for pull requests in flutter/devtools. Follow reviewing-prs for GitHub CLI data retrieval and the strict user approval workflow.

References & Style Guidelines

Read and enforce:

  • Style Guide: styleguide.md (severity tags [MUST-FIX], [CONCERN], [NIT], zero-formatting policy, copyright headers, DRY rules, magic values)
  • Repository Constraints: AGENTS.md
  • Code Style: STYLE.md

Common Review Patterns

  1. Listener & Resource Disposals:

    • Ensure controller and notifier listeners use addAutoDisposeListener(...).
  2. Helper Widgets over Helper Methods:

    • Prefer small composable Widget classes over helper methods returning Widget (_buildFoo()).
  3. Reuse Shared Components & Test Helpers:

    • Use standard shared widgets (e.g. CenteredMessage) and test mocks (e.g. mockConnectedApp) instead of re-creating them inline.
  4. TODO Formatting:

    • Link TODOs to a GitHub issue or LDAP: // TODO(https://github.com/flutter/devtools/issues/<issue_number>): <description>.
  5. Async & Unawaited Futures:

    • Audit unawaited futures and suggest unawaited(...) or safeUnawaited(...) where appropriate.
  6. Feature Flags:

    • Default feature flags to false with explicit test expectations in feature_flags_test.dart.
  7. Test File Structure & PR Scope:

    • Place test fakes/helpers below main().
    • Ask authors to revert unrelated file changes or commented-out test code.
  8. Constant Scoping:

    • Keep single-use constants local to the component, but extract user-facing UI strings into shared constants when used across multiple places.
  9. Release Notes Scope (NEXT_RELEASE_NOTES.md):

    • Release notes are strictly for end-user facing changes (e.g. Inspector, Memory UI/UX). Internal tools (dt / devtools_tool), CI, and refactors are NOT user-facing.
    • Request removing release notes added for developer tools like dt, or suggest a * <Description>. [#<PR_NUMBER>](https://github.com/flutter/devtools/pull/<PR_NUMBER>) entry via adding-release-notes if a user-facing PR lacks one.

© flutter, BSD-3-Clause. Rendered from Markdown: HTML in the file is shown as text, images as links, and headings moved down two levels. Raw file

Files

Just SKILL.md in .agents/skills/reviewing-devtools-prs of flutter/devtools.

Open the folder on GitHubat commit e25ba13

Compare with similar skills

Reviewing Devtools PRs next to the 5 skills that share the most tags, products or categories with it. Stars are the repository's; “used in” counts other GitHub owners with a copy.

Reviewing Devtools PRs compared with similar skills
SkillStarsUsed inTokensAuto-checkLicenceRepo updated
Reviewing Devtools PRs this skillflutter/devtools1.7k—~638Automated safety check: PassBSD-3-Clause
Review PRhyochan/flutter_calendar_carousel860—~1.4kAutomated safety check: PassMIT
Frb Issue To Green PRfzyzcjy/flutter_rust_bridge5.4k—~1.8kAutomated safety check: PassMIT
Review PRhyochan/flutter_calendar_carousel860—~172Automated safety check: PassMIT
Code Review Skillawesome-skills/code-review-skill2.1k—~2.8kAutomated safety check: NotesMIT
Grix Code Reviewaskie/grix153—~799Automated safety check: PassCustom licence

Similar skills

  • Review PR

    hyochan/flutter_calendar_carousel

    Inspect and finish fluttercalendarcarousel pull requests by fixing valid review findings, replying and resolving threads, diagnosing CI, using review-self when automation is unavailable, and polling…

    860 GitHub stars~1.4k tokensUpdated 16 days ago
    DevelopmentAuto-check passed
  • Frb Issue To Green PR

    fzyzcjy/flutter_rust_bridge

    A skill your agent uses when implementing a GitHub issue, bug fix, or feature in flutterrustbridge end-to-end: develop the change, add regression coverage, prepare and open a PR, and use the user's…

    5.4k GitHub stars~1.8k tokensUpdated yesterday
    DevelopmentAuto-check passed
  • Review PR

    hyochan/flutter_calendar_carousel

    Inspect and finish fluttercalendarcarousel pull requests by fixing valid review findings, resolving threads, diagnosing CI, and polling every five minutes until the exact head is clean.

    860 GitHub stars~172 tokensUpdated 16 days ago
    DevelopmentAuto-check passed
  • Code Review Skill

    awesome-skills/code-review-skill

    Provides comprehensive code review guidance for React 19, Vue 3, Angular 17+, Svelte 5, Rust, TypeScript, Java, Java 8, PHP, Ruby, Rails, Python, Django, FastAPI, Go, C/.NET, Kotlin, Swift, Dart…

    2.1k GitHub stars~2.8k tokensUpdated 29 days ago
    DevelopmentAuto-check: notes
  • Audit Grix diffs and pull requests for correctness, regressions, security, lifecycle safety, and cross-component contract consistency.

    153 GitHub stars~799 tokensUpdated 5 days ago
    DevelopmentAuto-check passed
  • Frb Prepare PR

    fzyzcjy/flutter_rust_bridge

    A skill your agent uses when about to create a PR or push changes in flutterrustbridge to ensure code generation is up to date

    5.4k GitHub stars~998 tokensUpdated yesterday
    MobileAuto-check passed

More from flutter/devtools

All 10 skills in this repo
  • Adding Release Notes

    flutter/devtools

    Adds user-facing change descriptions to DevTools release notes.

    1.7k GitHub stars~1k tokensUpdated today
    Auto-check passed
  • Adding Copyright Headers

    flutter/devtools

    Adds copyright headers to files based on file type. An agent skill from flutter/devtools.

    1.7k GitHub stars~640 tokensUpdated today
    Auto-check passed
  • Guidelines and step-by-step workflow for debugging DevTools extensions locally, including stub mode, fixed-port launching, browser auto-opening, URL query parameters, target app connection, and…

    1.7k GitHub stars~839 tokensUpdated today
    Auto-check passed
  • Reviewing PRs

    flutter/devtools

    General workflow for fetching, inspecting, reviewing GitHub Pull Requests using the gh CLI, drafting user-aligned review comments, and securing approval before posting.

    1.7k GitHub stars~570 tokensUpdated today
    Auto-check passed
  • Closing Obsolete Issues

    flutter/devtools

    Find and close obsolete, stale, or not reproducible issues in the flutter/devtools repository.

    1.7k GitHub stars~949 tokensUpdated today
    Auto-check passed
  • Adding Changelog Entries

    flutter/devtools

    A skill your agent uses when documenting changes in CHANGELOG.md files for devtoolsshared, devtoolsappshared, or devtoolsextensions.

    1.7k GitHub stars~482 tokensUpdated today
    Auto-check passed

Works with

Questions about Reviewing Devtools PRs

What does Reviewing Devtools PRs do?

DevTools repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns. Reviewing Devtools PRs is an agent skill from flutter/devtools. DevTools repository-specific PR review workflow enforcing DevTools style guidelines and common review patterns.

When should I use Reviewing Devtools PRs?

Reviewing Devtools PRs fits situations like: reviewing pull requests in the flutter/devtools repository; tasks that involve Pull requests; tasks that involve Cross-platform mobile apps.

How do I install Reviewing Devtools PRs in Claude Code?

Run `npx skills add flutter/devtools --skill reviewing-devtools-prs -a claude-code`. Or copy the skill folder (.agents/skills/reviewing-devtools-prs in flutter/devtools) into .claude/skills/reviewing-devtools-prs in your project. Claude Code loads it when a task matches its description.

How do I install Reviewing Devtools PRs in Codex?

Run `npx skills add flutter/devtools --skill reviewing-devtools-prs -a codex`. Or copy the skill folder (.agents/skills/reviewing-devtools-prs in flutter/devtools) into .agents/skills/reviewing-devtools-prs in your project. Codex loads it when a task matches its description.

Can I use Reviewing Devtools PRs in Cursor, Gemini CLI or GitHub Copilot?

Cursor, Gemini CLI, GitHub Copilot and OpenCode also load SKILL.md folders. With the skills CLI, run `npx skills add flutter/devtools --skill reviewing-devtools-prs -a cursor` (or -a gemini-cli, github-copilot or opencode for the others). To copy it by hand, put the folder in .cursor/skills/reviewing-devtools-prs, .gemini/skills/reviewing-devtools-prs, .github/skills/reviewing-devtools-prs and .opencode/skills/reviewing-devtools-prs in your project.

What does Reviewing Devtools PRs need to run?

SKILL.md names no scripts, command-line tools or credentials: Reviewing Devtools PRs is instructions for the agent only.

Does Reviewing Devtools PRs access the network?

SKILL.md contains no URLs. Any network use would come from the scripts or tools the agent runs. This is read from the text; nothing was executed.

Is Reviewing Devtools PRs safe to install?

Our automated static check of SKILL.md found no risky patterns, such as piping downloads into a shell, reading credential files or hidden Unicode. It is not a guarantee. Review the folder before installing.

What licence does Reviewing Devtools PRs use?

Reviewing Devtools PRs is published under the BSD-3-Clause licence (the repository's licence). It allows redistribution, so the full SKILL.md is shown on this page.

How many tokens does Reviewing Devtools PRs use?

About 638 tokens (SKILL.md is roughly 2.6k characters). Agents keep only the skill's name and description in context until a task matches; then they load SKILL.md in full.

What are the alternatives to Reviewing Devtools PRs?

Skills that share tags, products or a category with Reviewing Devtools PRs: Review PR (hyochan/flutter_calendar_carousel, 860 stars), Frb Issue To Green PR (fzyzcjy/flutter_rust_bridge, 5.4k stars), Review PR (hyochan/flutter_calendar_carousel, 860 stars) and Code Review Skill (awesome-skills/code-review-skill, 2.1k stars). The comparison table on this page puts their stars, adoption, token cost, safety result and licence side by side.

Who maintains Reviewing Devtools PRs?

flutter (a GitHub organization) maintains it in flutter/devtools, which has 1,731 GitHub stars. The repository holds 10 skills in this directory. The repository was last updated on October 6, 2026.

Source: flutter/devtools on GitHub. Facts on this page come from the repository at the commit we read; the author's words are quoted as theirs.