feat(dart): implement bump

[brianquinlan]

librarian bump works differently for Dart compared to other languages:

1. bump for Dart does not work on single libraries
2. bump for Dart uses analysis of the package interface as a heuristic to determine how much to bump the version
3. bump also updates CHANGELOG.md and dependency constraints

Expected publishing workflow:

$ librarian update # update all SDK descriptions
$ librarian generate --all # regenerate all packages
$ git commit …
$ librarian bump --all # inc the version number of changed packages
$ git commit …
$ librarian publish # publish packages that were updated
$ librarian tag # create a git tag for each published package

Refactors:

1. Moved FormatTagName and HasChangesIn to internal/librarian/bump.go from internal/git/git.go

Repository Package Release Algorithm (excluding edge cases)

1. Find the dependencies of all packages in the repository.
2. Order the packages topologically so that no package appears before any of its dependencies.
3. For each package in order:
   • a. Check if any files have changed since the last release.
   • b. Use dart-apitool to determine the recommended next version.
   • c. If the version should be updated:
     • Update the version in pubspec.yaml.
     • Update CHANGELOG.md.
     • Update the dependencies: section of each dependent package.
     • Update the version for the package in cfg.Default.Dart.Packages if the package has an entry there.

[brianquinlan]

/gemini review

[gemini-code-assist]

Code Review

This pull request introduces support for bumping Dart packages in the workspace, utilizing dart-apitool to recommend version updates, updating pubspec.yaml and CHANGELOG.md files, and managing dependencies. It also refactors common git operations (FormatTagName and HasChangesIn) into the internal/git package. The review feedback highlights a bug in the manual version increment logic that prevents file updates and changelog generation, as well as a potential file collision issue when creating temporary reports in os.TempDir().

[brianquinlan]

/gemini review

[gemini-code-assist]

Code Review

This pull request introduces support for bumping Dart packages, adding dependency sorting, pubspec updates, and changelog generation, alongside refactoring git helper functions into the git package. The review feedback highlights several critical issues: potential corruption of nested dependencies in pubspec.yaml due to line-by-line replacement, a potential crash in recommendedVersion when a package is not yet published, a security/collision risk from using a predictable temporary file path, and incorrect matching of nested version fields. Additionally, a style improvement was suggested to remove an unnecessary else block after a return statement.

[brianquinlan]

/gemini review

[gemini-code-assist]

Code Review

This pull request introduces support for bumping Dart packages in the workspace. It adds a new internal/librarian/dart package that implements the bumping logic, including dependency sorting, version recommendations using dart-apitool, and updating pubspec.yaml and CHANGELOG.md. It also refactors common Git helper functions into internal/git. The review feedback highlights several improvement opportunities: ensuring the version: field update in pubspec.yaml only matches top-level keys, avoiding indented success flows after error checks, removing an unused newDeps map, and using os.CreateTemp to safely generate unique temporary files.

[brianquinlan]

I'm away all next week so there is no hurry for this review.