Add a skills for adding changelog entries and updating package versions#9781
Merged
Conversation
kenzieschmoll
requested review from
elliette and
srawlins
and removed request for
a team
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request introduces two new AI skill definitions for managing changelog entries and package versions within the DevTools repository. The review identified that both new Markdown files are missing the required copyright headers. Additionally, there are concerns regarding the consistency of changelog header levels and version suffixes compared to existing packages, as well as a need for a rule to prevent published packages from depending on unpublished ones.
srawlins
approved these changes
Apr 17, 2026
jakemac53
reviewed
Apr 17, 2026
|
|
||
| When adding changelog entries to published packages (`devtools_shared`, `devtools_app_shared`, and `devtools_extensions`), follow these rules: | ||
|
|
||
| - **Check Publication State**: Never add an entry to an already published version (e.g. `# 12.1.0`). You MUST check which versions are published on pub.dev (e.g., at `https://pub.dev/packages/<package_name>/versions`) before determining section headers! If a version is not yet published, it should be suffixed with the next numeric `-wip` tag (e.g., `## 0.5.1-wip`) or follow the existing convention in the file, such as `(not released)`. You should also match the existing header level (H1 or H2) used for versions in that file. Avoid generic placeholder headers like `# WIP`! |
Contributor
There was a problem hiding this comment.
Should we direct the agent to https://pub.dev/api/packages/<package> instead? Not sure if this is better or worse honestly. cc @jonasfj is there a way to filter that response to only the latest version?
Member
Author
There was a problem hiding this comment.
Will address as a follow up pending @jonasfj's response.
Co-authored-by: Jacob MacDonald <jakemac@google.com>
Co-authored-by: Jacob MacDonald <jakemac@google.com>
rishika0212
pushed a commit
to rishika0212/devtools
that referenced
this pull request
Apr 21, 2026
…ns (flutter#9781) * Add a skill for adding changelog entries to DevTools packages * Add skill for updating package versions that is used from the changelog skill * add exception for copyright requirement * address comments * Update .agents/skills/adding-changelog-entries/SKILL.md Co-authored-by: Jacob MacDonald <jakemac@google.com> * Update .agents/skills/adding-changelog-entries/SKILL.md Co-authored-by: Jacob MacDonald <jakemac@google.com> * whitespace --------- Co-authored-by: Jacob MacDonald <jakemac@google.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds skills to update package version numbers and add Changelog entries for published devtools packages: devtools_shared, devtools_app_shared, devtools_extensions.
Tests: not required for skills content. Skills validated through dart_skills_lint
Fixes #9778, #9780
Pre-launch Checklist
General checklist
///).Issues checklist
contributions-welcomeorgood-first-issuelabel.contributions-welcomeorgood-first-issuelabel. I understand this means my PR might take longer to be reviewed.Tests checklist
AI-tooling checklist
Feature-change checklist
release-notes-not-requiredlabel or left a comment requesting the label be added.packages/devtools_app/release_notes/NEXT_RELEASE_NOTES.md.If you need help, consider asking for help on Discord.