chore: upgrade to flutter 3.32.0 (#1286)

* chore: upgrade to flutter 3.32.0

* run tool/generate_bundles.sh

* fix: lints

* fix: format

* fix: generate test optimizer bundle

* test: first round of test fixes

* test: very_good_cli build tests passing

* test: add missing cli tests

* fix: e2e test stubs

* chore(deps): bump pana version

* chore: regenerate spdx license

* chore: mason make spdx_license

* chore: remove erroneous file

* fix: spdx license generation

* fix: spdx license generation

* chore: update dart cli bundle

* chore: update flutter plugin bundle

* chore: run generators

* attempt to fix pana workflow

* fix: change pana minimum score

---------

Co-authored-by: Alejandro Santiago <[email protected]>

Author: marcossevilla

.github/workflows/e2e.yaml

@@ -27,7 +27,7 @@ jobs:
         flutter-version:
           # The version of Flutter to use should use the minimum Dart SDK version supported by the package,
           # refer to https://docs.flutter.dev/development/tools/sdk/releases.
-          - "3.24.0"
+          - "3.32.0"
           - "3.x"
         test:
           # E2E tests for the test command

.github/workflows/spdx_license.yaml

@@ -36,7 +36,7 @@ jobs:
       - name: 🎯 Setup Dart
         uses: dart-lang/setup-dart@v1
         with:
-          sdk: 3.5.0
+          sdk: 3.8.0
 
       - name: 📦 Install Dependencies
         run: dart pub get
@@ -71,7 +71,7 @@ jobs:
       - name: 🎯 Setup Dart
         uses: dart-lang/setup-dart@v1
         with:
-          sdk: 3.5.0
+          sdk: 3.8.0
 
       - name: 📦 Install Dependencies
         run: dart pub get

.github/workflows/test_optimizer.yaml

@@ -22,7 +22,7 @@ jobs:
   build_hooks:
     uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/dart_package.yml@v1
     with:
-      dart_sdk: 3.5.0
+      dart_sdk: 3.8.0
       working_directory: bricks/test_optimizer/hooks
 
   verify_bundle:

.github/workflows/very_good_cli.yaml

@@ -25,7 +25,7 @@ jobs:
 
       - uses: subosito/[email protected]
         with:
-          flutter-version: 3.24.0
+          flutter-version: 3.32.0
 
       - name: Install Dependencies
         run: flutter pub get
@@ -50,21 +50,7 @@ jobs:
           exclude: "**/*.gen.dart"
 
   pana:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - uses: subosito/[email protected]
-
-      - name: Install Dependencies
-        run: |
-          flutter packages get
-          flutter pub global activate pana
-
-      - name: Verify Pub Score
-        run: |
-          PANA=$(pana . --no-warning); PANA_SCORE=$(echo $PANA | sed -n "s/.*Points: \([0-9]*\)\/\([0-9]*\)./\1\/\2/p")
-          echo "score: $PANA_SCORE"
-          IFS='/'; read -a SCORE_ARR <<< "$PANA_SCORE"; SCORE=SCORE_ARR[0]; TOTAL=SCORE_ARR[1]
-          if (( $SCORE < $TOTAL )); then echo "minimum score not met!"; exit 1; fi
+    uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/pana.yml@v1
+    with:
+      min_score: 150 # Update minimum score to 160 once we have the CLI up to date.
+      pana_version: 0.22.21

analysis_options.yaml

@@ -1,4 +1,4 @@
-include: package:very_good_analysis/analysis_options.5.1.0.yaml
+include: package:very_good_analysis/analysis_options.yaml
 analyzer:
   exclude:
     - "**/version.dart"

bin/very_good.dart

@@ -12,6 +12,8 @@ Future<void> main(List<String> args) async {
 /// exited already. This is useful to prevent Future chains from proceeding
 /// after you've decided to exit.
 Future<void> _flushThenExit(int status) {
-  return Future.wait<void>([stdout.close(), stderr.close()])
-      .then<void>((_) => exit(status));
+  return Future.wait<void>([
+    stdout.close(),
+    stderr.close(),
+  ]).then<void>((_) => exit(status));
 }

bricks/test_optimizer/hooks/analysis_options.yaml

@@ -1,4 +1,4 @@
-include: package:very_good_analysis/analysis_options.7.0.0.yaml
+include: package:very_good_analysis/analysis_options.yaml
 linter:
   rules:
     public_member_api_docs: false

bricks/test_optimizer/hooks/lib/pre_gen.dart

@@ -34,8 +34,9 @@ Future<void> run(HookContext context) async {
   final testIdentifierTable = <Map<String, String>>[];
   for (final entity
       in testDir.listSync(recursive: true).where((entity) => entity.isTest)) {
-    final relativePath =
-        path.relative(entity.path, from: testDir.path).replaceAll(r'\', '/');
+    final relativePath = path
+        .relative(entity.path, from: testDir.path)
+        .replaceAll(r'\', '/');
     testIdentifierTable.add({
       'path': relativePath,
       'identifier': identifierGenerator.next(),

bricks/test_optimizer/hooks/pubspec.yaml

@@ -2,7 +2,7 @@ name: hooks
 publish_to: none
 
 environment:
-  sdk: ^3.5.0
+  sdk: ^3.8.0
 
 dependencies:
   mason: ^0.1.0
@@ -13,4 +13,4 @@ dependencies:
 dev_dependencies:
   mocktail: ^1.0.0
   test: ^1.25.0
-  very_good_analysis: ^7.0.0
+  very_good_analysis: ^9.0.0

bricks/test_optimizer/hooks/test/dart_identifier_generator_test.dart

@@ -34,14 +34,12 @@ void main() {
           ids.add(id);
         }
 
-        expect(
-          ids.where((id) => _dartReservedKeywords.contains(id)),
-          isEmpty,
-        );
+        expect(ids.where((id) => _dartReservedKeywords.contains(id)), isEmpty);
         expect(
           ids.every((id) {
             final idStart = id.codeUnitAt(0);
-            final isAlphabetic = (idStart >= 65 && idStart <= 90) ||
+            final isAlphabetic =
+                (idStart >= 65 && idStart <= 90) ||
                 (idStart >= 97 && idStart <= 122);
             final isUnderscore = idStart == 95;
             final isDollarSign = idStart == 36;