chore(deps): bump to Flutter 3.32.0

lishaduck · lishaduck · commit c00cc802d873 · 2025-05-23T09:20:12.000-05:00
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -24,13 +24,9 @@ permissions:
 jobs:
   setup:
     name: Setup
-    needs: []
     timeout-minutes: 5
     runs-on: ubuntu-latest
 
-    strategy:
-      fail-fast: false
-
     steps:
       - name: 📚 Git checkout
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
@@ -268,9 +264,9 @@ jobs:
         continue-on-error: true
       - name: 📊 Upload code coverage
         if: ${{ !cancelled() }}
-        uses: codecov/codecov-action@b9fd7d16f6d7d1b5d2bec1a2887e65ceed900238 # v4.6.0
+        uses: codecov/codecov-action@18283e04ce6e62d37312384ff67231eb8fd56d24 # v5.4.3
         with:
-          token: ${{ secrets.CODECOV_TOKEN }}
+          use_oidc: true
           files: packages/app/coverage/lcov.info
           fail_ci_if_error: true
           verbose: true
@@ -299,6 +295,8 @@ jobs:
           cache: true
       - name: 🌐 Disable analytics
         run: flutter --disable-analytics
+      - name: 📦 Install dependencies
+        run: dart pub get --enforce-lockfile
       - name: ✨ Verify formatting
         run: dart format . --output=none --set-exit-if-changed
 
@@ -344,7 +342,6 @@ jobs:
 
   spell-check:
     name: Check Spelling
-    needs: []
     runs-on: ubuntu-latest
 
     steps:
@@ -370,7 +367,6 @@ jobs:
 
   link-check:
     name: Check Links
-    needs: []
     timeout-minutes: 2
     runs-on: ubuntu-latest
 
@@ -384,7 +380,6 @@ jobs:
 
   markdownlint:
     name: Lint Markdown
-    needs: []
     timeout-minutes: 4
     runs-on: ubuntu-latest
 
@@ -405,3 +400,26 @@ jobs:
           # Surprisingly, the default is to only lint the project root.
           globs: |
             **/*.md
+
+  zizmor:
+    name: Run zizmor
+    timeout-minutes: 1
+    runs-on: ubuntu-latest
+
+    permissions:
+      contents: read
+      security-events: write
+      actions: read
+
+    steps:
+      - name: 📚 Git Checkout
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+        with:
+          submodules: recursive
+          clean: true
+          persist-credentials: false
+          set-safe-directory: true
+      - name: 🌈 Run zizmor
+        uses: zizmorcore/zizmor-action@2520132f44b3ed84916048d32e5c7153fc739fe7 # v0.0.3
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
diff --git a/packages/_analysis_options/pubspec.yaml b/packages/_analysis_options/pubspec.yaml
@@ -7,8 +7,8 @@ issue_tracker: https://github.com/PHS-TSA/nexus/issues
 repository: https://github.com/PHS-TSA/nexus.git
 
 environment:
-  sdk: ^3.7.0
+  sdk: ^3.8.0
 resolution: workspace
 
 dependencies:
-  very_good_analysis: ^7.0.0
+  very_good_analysis: ^8.0.0-rc.1
diff --git a/packages/app/lib/src/features/home/presentation/home/feed.dart b/packages/app/lib/src/features/home/presentation/home/feed.dart
@@ -89,21 +89,20 @@ class Feed extends ConsumerWidget {
                     ),
                   ),
 
-                AsyncValue(:final value, hasValue: true) when value != null =>
-                  GestureDetector(
-                    onTap: () async {
-                      await context.router.push(PostViewRoute(id: value.id.id));
-                    },
-                    child: Card(
-                      margin: const EdgeInsets.all(4),
-                      child: ProviderScope(
-                        overrides: [
-                          currentPostProvider.overrideWithValue(value),
-                        ],
-                        child: Post(key: ValueKey(value)),
-                      ),
+                // Intentional, we don’t want to match null.
+                // ignore: async_value_nullable_pattern
+                AsyncValue(:final value?, hasValue: true) => GestureDetector(
+                  onTap: () async {
+                    await context.router.push(PostViewRoute(id: value.id.id));
+                  },
+                  child: Card(
+                    margin: const EdgeInsets.all(4),
+                    child: ProviderScope(
+                      overrides: [currentPostProvider.overrideWithValue(value)],
+                      child: Post(key: ValueKey(value)),
                     ),
                   ),
+                ),
 
                 // If we have none, return a placeholder.
                 AsyncValue(hasValue: true) when index == 0 => const Center(
diff --git a/packages/app/lib/src/features/home/presentation/home/post_view_page.dart b/packages/app/lib/src/features/home/presentation/home/post_view_page.dart
@@ -48,23 +48,22 @@ class PostViewPage extends ConsumerWidget {
         ),
       ),
       floatingActionButton: Consumer(
-        builder:
-            (context, ref, _) => FloatingActionButton(
-              onPressed: () async {
-                final post = ref.read(postServiceProvider(_postId)).valueOrNull;
-                if (post == null) {
-                  return;
-                }
-
-                await showDialog<void>(
-                  context: context,
-                  builder: (context) => CreateComment(post: post),
-                );
-
-                ref.invalidate(singlePostProvider(_postId));
-              },
-              child: const Icon(Icons.add_comment),
-            ),
+        builder: (context, ref, _) => FloatingActionButton(
+          onPressed: () async {
+            final post = ref.read(postServiceProvider(_postId)).valueOrNull;
+            if (post == null) {
+              return;
+            }
+
+            await showDialog<void>(
+              context: context,
+              builder: (context) => CreateComment(post: post),
+            );
+
+            ref.invalidate(singlePostProvider(_postId));
+          },
+          child: const Icon(Icons.add_comment),
+        ),
       ),
       body: RefreshIndicator(
         onRefresh: () async {
@@ -92,6 +91,8 @@ class PostViewPage extends ConsumerWidget {
                             style: const TextStyle(color: Colors.red),
                           ),
                         ),
+                      // Intentional, we don’t want to match null.
+                      // ignore: async_value_nullable_pattern
                       AsyncValue(:final value?, hasValue: true) => Column(
                         children: [
                           ProviderScope(
diff --git a/packages/app/pubspec.yaml b/packages/app/pubspec.yaml
@@ -23,7 +23,7 @@ issue_tracker: https://github.com/PHS-TSA/nexus/issues
 repository: https://github.com/PHS-TSA/nexus.git
 
 environment:
-  sdk: ^3.7.0
+  sdk: ^3.8.0
 resolution: workspace
 
 # Dependencies specify other packages that your package needs in order to work.
@@ -34,61 +34,61 @@ resolution: workspace
 # versions available, run `flutter pub outdated`.
 dependency_overrides:
   flutter_web_auth_2: 4.1.0 # Supports WASM.
-  pana: 0.22.18
+  pana: 0.22.21
 
 dependencies:
-  appwrite: ^14.0.0
-  auto_route: ^10.0.0
+  appwrite: ^16.1.0
+  auto_route: ^10.0.1
   # The following adds the Cupertino Icons font to your application.
   # Use with the CupertinoIcons class for iOS style icons.
   cupertino_icons: ^1.0.8 # Only required if you use Cupertino (iOS style) icons
   dio: ^5.8.0+1
   envied: ^1.1.1
-  fast_immutable_collections: ^11.0.3
+  fast_immutable_collections: ^11.0.4
   flutter:
     sdk: flutter
   flutter_hooks: ^0.21.2
   flutter_image_compress: ^2.4.0
   flutter_localizations: # Required to enable localization
     sdk: flutter
-  flutter_map: ^8.1.0
+  flutter_map: ^8.1.1
   flutter_map_cancellable_tile_provider: ^3.1.0
-  flutter_native_splash: ^2.4.5
+  flutter_native_splash: ^2.4.6
   flutter_web_plugins:
     sdk: flutter
   freezed_annotation: ^3.0.0
-  geolocator: ^13.0.2
+  geolocator: ^14.0.0
   hooks_riverpod: ^2.6.1
   image_picker: ^1.1.2
-  intl: ^0.19.0
+  intl: ^0.20.2
   json_annotation: ^4.9.0
   latlong2: ^0.9.1
   meta: ^1.16.0
   os_detect: ^2.0.3
   path_provider: ^2.1.5
   riverpod_annotation: ^2.6.1
-  shared_preferences: ^2.5.2
+  shared_preferences: ^2.5.3
   timeago_flutter: ^3.7.0
   url_launcher: ^6.3.1
   window_manager: ^0.4.3
 
 dev_dependencies:
   _analysis_options: 0.0.0
-  auto_route_generator: ^10.0.0
+  auto_route_generator: ^10.0.1
   build_runner: ^2.4.15
   build_version: ^2.1.1
   checks: ^0.3.0
   custom_lint: ^0.7.5
   dhttpd: ^4.1.0
   envied_generator: ^1.1.1
-  flutter_checks: ^0.1.1
-  flutter_gen_runner: ^5.9.0
+  flutter_checks: ^0.1.2
+  flutter_gen_runner: ^5.10.0
   flutter_launcher_icons: ^0.14.3
   flutter_test: # Required for a Flutter project that includes tests
     sdk: flutter
-  freezed: ^3.0.2
+  freezed: ^3.0.6
   freezed_lint: ^0.0.9
-  json_serializable: ^6.9.4
+  json_serializable: ^6.9.5
   mocktail: ^1.0.4
   riverpod_generator: ^2.6.5
   riverpod_lint: ^2.6.5
@@ -105,7 +105,7 @@ flutter:
   uses-material-design: true # Required if you use the Material icon font
 
   # Enable generation of localized strings from arb files.
-  generate: false
+  generate: true
 
   # To add assets to your application, add an assets section, like this:
   assets: # Lists assets such as image files
diff --git a/packages/app/test/src/app/router_test.dart b/packages/app/test/src/app/router_test.dart
@@ -40,33 +40,31 @@ void main() {
         final routerSubscription = container.listen(routerProvider, (_, _) {});
         final tested = routerSubscription.read();
 
-        final mapRoute = tested.routes[0].children?.routes.toList()[0];
+        final mapRoute = tested.routes[0].children?.toList()[0];
         check(mapRoute?.path).equals('');
       });
       test('should be correct for MapInfoRoute.', () {
         final container = createContainer();
         final routerSubscription = container.listen(routerProvider, (_, _) {});
         final tested = routerSubscription.read();
 
-        final sampleItemDetailsRoute =
-            tested.routes[0].children?.routes.toList()[1];
+        final sampleItemDetailsRoute = tested.routes[0].children?.toList()[1];
         check(sampleItemDetailsRoute?.path).equals('info');
       });
       test('should be correct for SettingsRoute.', () {
         final container = createContainer();
         final routerSubscription = container.listen(routerProvider, (_, _) {});
         final tested = routerSubscription.read();
 
-        final settingsRoute = tested.routes[0].children?.routes.toList()[2];
+        final settingsRoute = tested.routes[0].children?.toList()[2];
         check(settingsRoute?.path).equals('settings');
       });
       test("should be correct for ShellRoute('Feed')", () {
         final container = createContainer();
         final routerSubscription = container.listen(routerProvider, (_, _) {});
         final tested = routerSubscription.read();
 
-        final sampleItemDetailsRoute =
-            tested.routes[0].children?.routes.toList()[3];
+        final sampleItemDetailsRoute = tested.routes[0].children?.toList()[3];
         check(sampleItemDetailsRoute?.path).equals('local');
       });
       test('should allow logging in', () {
diff --git a/pubspec.lock b/pubspec.lock
diff --git a/pubspec.yaml b/pubspec.yaml
