Skip to content

chore(repo): add flutter-version-bump skill for v9 - #2902

Merged
VelikovPetar merged 1 commit into
v9from
chore/FLU-705_port-flutter-version-bump-skill-v9
Aug 18, 2026
Merged

chore(repo): add flutter-version-bump skill for v9#2902
VelikovPetar merged 1 commit into
v9from
chore/FLU-705_port-flutter-version-bump-skill-v9

Conversation

@VelikovPetar

Copy link
Copy Markdown
Contributor

Submit a pull request

Linear: FLU-705

Github Issue: #

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

Ports the flutter-version-bump skill from master to the v9 branch, adapted to v9's own infrastructure instead of a blind copy:

  • Floor policy: v9's minimum (Flutter 3.27.4 / Dart 3.6.2) is frozen, not "latest stable − 1" — a floor raise is framed as an exceptional maintainer decision, including the repo-wide tall-style reformat that crossing Dart 3.7 would trigger.
  • CI facts corrected for v9: the "3.x" auto-adopt pin lives in stream_flutter_workflow.yml's env (plus update_goldens.yml and the pana action) rather than a shared setup-flutter action; update_goldens.yml takes no target input; package_analysis runs the non-golden test suite on the floor, not just lib/ analysis; the melos.yaml shared dependency catalog must move together with package pubspecs on dependency floor bumps (precedent: chore(repo): support Flutter 3.47 #2895).
  • v9-specific traps: stale .dart_tool/package_config.json flipping dart format to the tall style; the committed seven-entry analyzer.exclude block; minimal error-floor Gradle/Kotlin bumps as the branch convention.

The skill was verified by replaying the Flutter 3.47 adoption: a throwaway worktree at the commit before #2895, with 3.27.4 and 3.47.0 side by side, executing every locally-runnable step and scoring output against #2895's actual diff. The replay rediscovered everything mechanically discoverable in that PR (golden drift, the two Gradle/Kotlin floor violations, the analyzer.exclude injection) and caught five defects that are fixed in this version: a broken failure-set normalization in the test A/B, a stale-lockfile blind spot (alchemist 0.11.0 fails to compile on 3.47, masquerading as 100+ test regressions), a misstated Android precedent, undocumented macOS attribution distortions, and the beta canary running against the default branch only.

Test instructions: none — documentation-only change, not exercised by CI.

Screenshots / Videos

No UI changes.

🤖 Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 8f3e9215-9d9d-4d85-ae07-c0d0c56b2a6d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@VelikovPetar
VelikovPetar marked this pull request as ready for review August 17, 2026 11:14
@VelikovPetar VelikovPetar changed the title chore(repo): add flutter-version-bump skill for v9 chore(repo): add flutter-version-bump skill for v9 Aug 17, 2026
@VelikovPetar
VelikovPetar merged commit 7c056b9 into v9 Aug 18, 2026
28 checks passed
@VelikovPetar
VelikovPetar deleted the chore/FLU-705_port-flutter-version-bump-skill-v9 branch August 18, 2026 07:08
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