Skip to content

feat: add contributor-pr-description skill - #190

Merged
reidbaker merged 3 commits into
flutter:mainfrom
reidbaker:feature/pr-description-skill
Jul 21, 2026
Merged

feat: add contributor-pr-description skill#190
reidbaker merged 3 commits into
flutter:mainfrom
reidbaker:feature/pr-description-skill

Conversation

@reidbaker-agent

@reidbaker-agent reidbaker-agent commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR introduces the contributor-pr-description skill to guide contributors and AI agents on how to author well-structured pull request descriptions.

Motivation and Context

Previously, agents (like myself) and contributors could open PRs with extremely brief or poorly formatted descriptions, omitting crucial context. Furthermore, there was an issue where agents opened PRs using the incorrect GitHub identity, leading to authorization errors. This skill adds a PR template and an Identity Verification step to ensure identities are checked before pushing and that PR bodies are informative.

Related Issues

N/A

What changed

  • Created .agents/skills/contributor-pr-description/SKILL.md containing the PR description guidelines, template, and identity verification checklist.
  • Un-ignored the contributor-pr-description directory in .agents/skills/.gitignore.
  • Configured dart_skills_lint.yaml to enforce the prevent-skills-sh-publishing: error rule for this new internal skill.

Testing Instructions

  • Review the SKILL.md content for clarity and accuracy.
  • Verify that dart run dart_skills_lint -d .agents/skills passes locally, confirming the skill is valid and properly marked as internal.

@reidbaker
reidbaker self-requested a review July 21, 2026 18:07

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new agent skill, contributor-pr-description, which provides guidelines and a template for drafting pull request descriptions, and updates the .gitignore file to un-ignore this new skill. The feedback points out that internal skills must include internal: true in their frontmatter to prevent accidental publication to the global registry, which is a valid and helpful correction.

@reidbaker
reidbaker merged commit 0de9658 into flutter:main Jul 21, 2026
13 checks passed
@reidbaker
reidbaker deleted the feature/pr-description-skill branch July 21, 2026 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants