ci: enforce repo-wide dart format + blame-ignore the reformat

Author: Alspb

.git-blame-ignore-revs

@@ -0,0 +1,9 @@
+# Revisions for `git blame` to skip (large, mechanical changes with no logic impact).
+#
+# Enable locally once with:
+#   git config blame.ignoreRevsFile .git-blame-ignore-revs
+#
+# GitHub honors this file automatically. List full 40-char commit SHAs only.
+
+# style: apply Dart 3.12 format (tall style) — repo-wide dart_format, no logic changes
+10e0814088ae3b12f0cca4673f885e2828eaaa7a

.github/workflows/checks.yml

@@ -16,6 +16,10 @@ jobs:
       - name: Set up Flutter
         uses: subosito/flutter-action@v2
         with:
+          # Pinned so a future Flutter/Dart release can't change formatter output
+          # and turn the format check below red on unrelated PRs. Bump deliberately
+          # alongside a repo-wide reformat (see .git-blame-ignore-revs).
+          flutter-version: 3.44.1
           channel: stable
           cache: true
 
@@ -34,9 +38,8 @@ jobs:
       - name: Perform Flutter Analysis
         run: flutter analyze
 
-      # TODO: Restore Dart formatting check, disabled due to an issue https://github.com/singerdmx/flutter-quill/actions/runs/13355109619/job/37296760819.
-      # - name: Check Dart code formatting
-      #   run: dart format --set-exit-if-changed .
+      - name: Check Dart code formatting
+        run: dart format --set-exit-if-changed .
 
       - name: Preview Dart proposed changes
         run: dart fix --dry-run

scripts/before_push.dart

@@ -1,6 +1,5 @@
 // ignore_for_file: avoid_print
 
-import 'dart:convert';
 import 'dart:io';
 
 void main() async {
@@ -12,21 +11,12 @@ void main() async {
 
   await runCommand('dart', ['fix', '--apply']);
 
-  // Format only files that are modified but not yet committed, to avoid
-  // reformatting the entire repository (e.g. after a Dart SDK formatter
-  // update changes the default formatting style).
-  final changedDartFiles = await uncommittedDartFiles();
-  if (changedDartFiles.isEmpty) {
-    print('No modified Dart files to format; skipping `dart format`.');
-  } else {
-    await runCommand('dart', ['format', ...changedDartFiles]);
-
-    await runCommand('dart', [
-      'format',
-      '--set-exit-if-changed',
-      ...changedDartFiles,
-    ]);
-  }
+  // Format the whole repository, then assert it is fully formatted (mirrors the
+  // CI `dart format --set-exit-if-changed .` check). The repo is kept in canonical
+  // style by a one-off reformat + the pinned-version CI check, so this no longer
+  // needs to scope to changed files only.
+  await runCommand('dart', ['format', '.']);
+  await runCommand('dart', ['format', '--set-exit-if-changed', '.']);
 
   await runCommand('flutter', [
     'build',
@@ -44,29 +34,6 @@ void main() async {
   print('Checks completed.');
 }
 
-/// Returns the repo-relative paths of Dart files that are modified but not yet
-/// committed: tracked changes (staged or unstaged) relative to `HEAD`, plus
-/// untracked files. Deleted files are excluded so they aren't passed to
-/// `dart format`.
-Future<List<String>> uncommittedDartFiles() async {
-  final tracked = await Process.run('git', ['diff', '--name-only', 'HEAD']);
-  final untracked = await Process.run('git', [
-    'ls-files',
-    '--others',
-    '--exclude-standard',
-  ]);
-
-  final paths = <String>{
-    ...const LineSplitter().convert(tracked.stdout as String),
-    ...const LineSplitter().convert(untracked.stdout as String),
-  };
-
-  return paths
-      .where((path) => path.endsWith('.dart'))
-      .where((path) => File(path).existsSync())
-      .toList();
-}
-
 Future<void> runCommand(
   String executable,
   List<String> arguments, {