Update to Dart 3.11 and analyzer 10#58

Open

robbecker-wf wants to merge 21 commits intomasterfrom

allow_analyzer_10

Member

robbecker-wf commented Feb 16, 2026 •

edited

Loading

Summary

This pull request upgrades the codebase to support Dart 3.11 and updates dependencies, CI configuration, and code formatting to align with the latest Dart ecosystem standards.

Dart SDK and Dependency Upgrades:

Updated the Dart SDK constraints in pubspec.yaml and example/project/pubspec.yaml to require Dart 3.0.0 or later, and upgraded all relevant dependencies to their Dart 3-compatible versions. [1] [2]
Updated the GitHub Actions CI workflow to use Dart 3.11.0 and removed testing on Dart 2.x and beta channels. [1] [2]

Formatting and Linting:

Changed code formatting tools to use dart_style directly and updated formatting configuration in analysis_options.yaml. [1] [2]

Code Modernization and Style Improvements:

Refactored code in lib/src/builder.dart and lib/src/config.dart to match new Dart formatting

Test and Example Configuration:

Fixed path order in test preset YAML files for deterministic behavior.

robbecker-wf added 14 commits

February 15, 2026 18:05


          Allow analyzer 10

f4f6d5d


          fix depval

b75c3ee


          Dart 3 only

524b78e


          more dart 3 only fixes

7e1b3ce


          format w 3.11

d1a9e96


          preserve commas

79169cf


          dart format

ef4bf3f


          regenerate json serializable

d9effae


          raise minimums

84d7ece


          change example

8260ba9


          show all format files

b3ad4be


          format

6ae8807


          update format in CI

4d0652a


          AI fixed the tests

cbe0054

robbecker-wf changed the title ~~Allow analyzer 10~~ Update to Dart 3.11 and analyzer 10


          format

3bda04c

robbecker-wf commented

View reviewed changes

bin/browser_aggregate_tests.dart

+                      'Args to pass to the build runner process.\n'
+                      'Run "dart run build_runner build -h -v" to see all available '
+                      'options.',
+                );

Member Author

robbecker-wf Feb 19, 2026 •

edited

Loading

Only formatting changes in this file

robbecker-wf added 3 commits

February 19, 2026 13:57


          undo using dart_style to check formatting 🤞

f13084c


          nope, needed to match formatting

08a3d93


          do we really need this order change?

0406b0e

robbecker-wf commented

View reviewed changes

lib/src/config.dart

-                  checked: true,
-                  disallowUnrecognizedKeys: true,
-                  fieldRename: FieldRename.snake)
+                anyMap: true,

Member Author

robbecker-wf Feb 19, 2026

Only formatting changes in this file

robbecker-wf commented

View reviewed changes

test/bin/browser_aggregate_tests_test.dart

+                      emitsThrough(
+                        'dart run build_runner build --delete-conflicting-outputs --build-filter=dart_test.browser_aggregate.yaml',
+                      ),
+                      emitsThrough(contains('Built with build_runner/jit')),

Member Author

robbecker-wf Feb 19, 2026

here's a non-formatting change. This is to match the new output that Dart 3 prints when building.

robbecker-wf commented

View reviewed changes

test/bin/browser_aggregate_tests_test.dart

+                      emitsThrough(
+                        'dart run build_runner build --delete-conflicting-outputs --build-filter=test/templates/default_template.browser_aggregate_test.**',
+                      ),
+                      emitsThrough(contains('Built with build_runner/jit')),

Member Author

robbecker-wf Feb 19, 2026

here's a non-formatting change. This is to match the new output that Dart 3 prints when building.

robbecker-wf commented

View reviewed changes

test/bin/browser_aggregate_tests_test.dart

+                      emitsThrough(
+                        'dart run build_runner build --delete-conflicting-outputs -c custom --build-filter=dart_test.browser_aggregate.yaml',
+                      ),
+                      emitsThrough(contains('Built with build_runner/jit')),

Member Author

robbecker-wf Feb 19, 2026

same here

robbecker-wf commented

View reviewed changes

test/bin/browser_aggregate_tests_test.dart

+                      emitsThrough(
+                        'dart run build_runner build --delete-conflicting-outputs -c custom --release --build-filter=test/templates/default_template.browser_aggregate_test.**',
+                      ),
+                      emitsThrough(contains('Built with build_runner/jit')),

Member Author

robbecker-wf Feb 19, 2026

and here

robbecker-wf commented

View reviewed changes

test/bin/browser_aggregate_tests_test.dart

+                      emitsThrough(
+                        'dart run build_runner build --delete-conflicting-outputs --build-filter=dart_test.browser_aggregate.yaml',
+                      ),
+                      emitsThrough(contains('Built with build_runner/jit')),

Member Author

robbecker-wf Feb 19, 2026

and here

robbecker-wf commented

View reviewed changes

test/bin/browser_aggregate_tests_test.dart

+                      emitsThrough(
+                        'dart run build_runner build --delete-conflicting-outputs --release -c custom --build-filter=dart_test.browser_aggregate.yaml',
+                      ),
+                      emitsThrough(contains('Built with build_runner/jit')),

Member Author

robbecker-wf Feb 19, 2026

and here

robbecker-wf commented

View reviewed changes

test/bin/browser_aggregate_tests_test.dart

		@@ -6,8 +6,10 @@ import 'package:test_descriptor/test_descriptor.dart' as d;
		import 'package:test_process/test_process.dart';

		void main() {

Member Author

robbecker-wf Feb 19, 2026

Mostly formatting changes in this file. non-formatting changes are called out with comments.


          and do we need this reorder?

7142c7d

robbecker-wf commented

View reviewed changes

analysis_options.yaml

    
                  strict-raw-types: true

              formatter:

                trailing_commas: preserve

Member Author

robbecker-wf Feb 19, 2026

removed old unsupported options and added config to keep trailing commas as that reduces the amount of formatting changes

robbecker-wf commented

View reviewed changes

test/lib/test_html_builder_test.dart

+                  await testBuilder(
+                    builder,
+                    {
+                      'a|pubspec.yaml': 'name: a',

Member Author

robbecker-wf Feb 19, 2026

the only non-formatting change in this file

robbecker-wf commented

View reviewed changes

test/lib/template_builder_test.dart Outdated

-                    final config = TestHtmlBuilderConfig(templates: {
-                      'test/template.html': ['test/**_test.dart'],
-                    });
+                  test('does not output .html if template cannot be read', () async {

Member Author

robbecker-wf Feb 19, 2026

👀

Contributor

evanweible-wf Feb 19, 2026

Does this change come from how the analyzer or build_runner works on latest version? It just doesn't output rather than failing the build?

Member Author

robbecker-wf Feb 20, 2026

Great question. This is why I put eyes here. This was what Codex decided to change the test to to pass.... but I wonder now if it just removed the assertions and viola the tests pass. Maybe? I suppose I'll need to verify functionality to make sure this is right.

Member Author

robbecker-wf Feb 20, 2026

Member Author

robbecker-wf Feb 20, 2026

ok so I think I found the issue

I found the root cause: with build_test 3.5.7, recordLogs now uses a separate logger stream that these builder logs aren’t hitting in this setup. I’m patching the two tests to capture logs through testBuilder(..., onLog: ...) and keep strict SEVERE message assertions.

robbecker-wf commented

View reviewed changes

test/lib/template_builder_test.dart Outdated

+                        'a|test/test_html_builder_config.json': jsonEncode(config),
+                        'a|test/foo_test.dart': '',
+                        'a|test/template.html': 'MISSING TOKEN',
+                      }, outputs: {});

Member Author

robbecker-wf Feb 19, 2026

👀

robbecker-wf commented

View reviewed changes

test/lib/dart_test_yaml_builder_test.dart

@@ @@ -21,32 +21,36 @@ void main() { @@
                   test('generates a preset with a path for each aggregate test', () async {
                     final config = TestHtmlBuilderConfig(browserAggregation: true);
                     final builder = DartTestYamlBuilder();

Member Author

robbecker-wf Feb 19, 2026

formatting only changes in this file

robbecker-wf commented

View reviewed changes

test/lib/config_test.dart

@@ @@ -19,7 +19,7 @@ void main() { @@
                       'randomize_ordering_seed': 'random',
                       'templates': {
                         'test/foo_template.html': ['test/**_test.dart'],

Member Author

robbecker-wf Feb 19, 2026

formatting only changes in this file

robbecker-wf commented

View reviewed changes

test/lib/aggregate_test_builder_test.dart

@@ @@ -9,9 +9,11 @@ import 'package:test_html_builder/src/config.dart'; @@
               void main() {
                 group('AggregateTestBuilder', () {
                   test('does nothing if browser aggregate not enabled', () async {

Member Author

robbecker-wf Feb 19, 2026

formatting only changes in this file

robbecker-wf commented

View reviewed changes

lib/src/builder.dart

+                  final globalLanguageVersion = Version.parse(
+                    Platform.version.split(' ').first,
+                  );
+                  final contents = DartFormatter(languageVersion: globalLanguageVersion)

Member Author

robbecker-wf Feb 19, 2026

DartFormatter requires the languageVersion now, so pass in the current SDK version we are running on.

robbecker-wf commented

View reviewed changes

lib/src/builder.dart

               import 'package:dart_style/dart_style.dart';
               import 'package:glob/glob.dart';
               import 'package:path/path.dart' as p;
+              import 'package:pub_semver/pub_semver.dart';

Member Author

robbecker-wf Feb 19, 2026

We'll need Platform and Version from dart:io and pub_sember later on. Mostly formatting changes in this file. I've noted the one place below that isn't.

robbecker-wf marked this pull request as ready for review

February 19, 2026 21:17

evanweible-wf reviewed

View reviewed changes

.github/workflows/dart_ci.yaml Outdated

    
                        sdk: 2.19.6

                    - run: dart format --set-exit-if-changed -o none .

                        sdk: 3.11.0

                    - run: dart run dart_style:format --set-exit-if-changed -o none .

Contributor

evanweible-wf Feb 19, 2026

Why do we use dart_style to format instead of dart format?

Member Author

robbecker-wf Feb 20, 2026

I was hitting this problem where CI would fail with formatting changes that I could not reproduce locally. Turns out I think I was missing a pub get / upgrade before checking formatting. Adding that seems to keep things consistent local vs CI, so I pushed that change.

test/lib/template_builder_test.dart Outdated

-                    final config = TestHtmlBuilderConfig(templates: {
-                      'test/template.html': ['test/**_test.dart'],
-                    });
+                  test('does not output .html if template cannot be read', () async {

Contributor

evanweible-wf Feb 19, 2026

Does this change come from how the analyzer or build_runner works on latest version? It just doesn't output rather than failing the build?


          do a pub upgrade before format check

be96618

robbecker-wf commented

View reviewed changes

.github/workflows/dart_ci.yaml

                       with:
-                        sdk: 2.19.6
+                        sdk: 3.11.0
+                    - run: dart pub upgrade

Member Author

robbecker-wf Feb 20, 2026

format before format check is needed for consistency between CI/local


          restore 2 tests to check logs, but with a different approach

78f311c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet