Agent Skills › flutter/agent-plugins › definition-of-done

definition-of-done

GitHub

确保涉及md或dart代码的任务完成前执行格式、分析、复杂度、测试等强制检查，并规范Changelog更新及输出验证清单。

tool/dart_skills_lint/.agents/skills/definition-of-done/SKILL.md flutter/agent-plugins

Trigger Scenarios

任务完成前检查请求审查前验证

Install

npx skills add flutter/agent-plugins --skill definition-of-done -g -y

More Options

Non-standard path

npx skills add https://github.com/flutter/agent-plugins/tree/main/tool/dart_skills_lint/.agents/skills/definition-of-done -g -y

Use without installing

npx skills use flutter/agent-plugins@definition-of-done

指定 Agent (Claude Code)

npx skills add flutter/agent-plugins --skill definition-of-done -a claude-code -g -y

安装 repo 全部 skill

npx skills add flutter/agent-plugins --all -g -y

预览 repo 内 skill

npx skills add flutter/agent-plugins --list

SKILL.md

Frontmatter

{
    "name": "definition-of-done",
    "metadata": {
        "internal": true
    },
    "description": "Mandatory checks to run before completing any task that touches md files or dart code in this repository."
}

Definition of Done

Use this skill to ensure that all work meets the repository standards before declaring a task complete or requesting review.

📋 Mandatory Verification Steps

Before stating that a task is complete, you MUST execute and pass the following checks:

Format: Run dart format . to format files, or dart format --output=none --set-exit-if-changed . to check without modifying. Ensure all files are formatted correctly.
Analysis: Run dart analyze --fatal-infos and ensure there are zero issues (including info-level issues).
Metrics: Run dart run cognitive_complexity --fail-threshold <N> lib test, where <N> is the --fail-threshold value configured in .github/workflows/dart_skills_lint_workflow.yaml, and ensure there are zero issues. This checks for cognitive complexity.
Tests: Run dart test and ensure all tests pass successfully.
Skills: If any skill files were modified, run dart run dart_skills_lint -d .agents/skills to ensure they are valid.
Changelog: If the task introduces user-facing CLI flags, package API changes, bug fixes, or user-facing behavioral changes, update CHANGELOG.md.
- Do NOT log internal chores: Do not add entries for internal CI workflows, dev dependency updates/migrations, test refactoring, or repository infrastructure scripts.
- Explicit N/A: If the task is internal-only, leave CHANGELOG.md untouched and output [x] Changelog: (N/A) <reason>.
- Audit all entries against the previously released version (do not document changes to intermediate PR development code or new unreleased APIs as breaking changes).
Temporal: Ensure that code and code comments contain no relative temporal terms (e.g., 'now', 'currently', 'new', 'old', 'existing behavior').
Documentation: Ensure that any relevant documentation is updated.

🚦 Output Formatting

You MUST include a text list of all mandatory verification steps in your final response to the user. Use the exact following format:

Use [x] <Identifier>: <Explanation> if the step was completed.
Use [ ] <Identifier>: <Skipped explanation> if the step was skipped or not applicable.

CRITICAL: Do not just copy the full step description text. You MUST use the exact bolded Identifier from the Mandatory Verification Steps list above, followed by a colon and your short explanation.

Examples:

[x] Format: dart format success.
[x] Analysis: Static clean (0 issues, dart analyze --fatal-infos).
[ ] Skills: Skipped because dart_skills_lint is not installed.
[x] Changelog: (N/A) Not necessary since we're updating internal eval fixtures.
[x] Temporal: no added words.