[tool] Migrate SymbolizeCommand to modular dependency injection by bkonyi · Pull Request #190744 · flutter/flutter

bkonyi · 2026-08-07T20:58:27Z

Summary

Part 5 of the modular dependency injection migration.

Migrates SymbolizeCommand to explicit constructor dependency injection via ToolContext, removing ambient access from FlutterCommand and adding hermetic unit tests.

Part of #47161

gemini-code-assist

Code Review

This pull request refactors dependency injection in flutter_tools by introducing structured context containers (ToolContext, AndroidContext, AppleContext) and a bootstrapping mechanism (ToolDependencies) to manage tool dependencies instead of relying on global state. Commands such as CleanCommand, ConfigCommand, and SymbolizeCommand are updated to receive their dependencies via these contexts, and tests are refactored to use hermetic testWithoutContext setups. Feedback on the changes suggests using the local finalShutdownHooks reference during file system initialization to prevent duplicate hook instances, and utilizing finalSystemClock.now() instead of DateTime.now() to maintain a mockable time source.

…ners

…ired non-nullable contexts

# Conflicts: # packages/flutter_tools/lib/src/context/tool_dependencies.dart # packages/flutter_tools/test/general.shard/context/dependency_injection_test.dart

…ection

… globals fallback

…ctly in FlutterCommandRunner

…config_test.dart

gemini-code-assist

Code Review

This pull request refactors ConfigCommand and SymbolizeCommand to explicitly receive and resolve their dependencies via ToolContext rather than relying on global variables. Consequently, associated unit tests have been migrated from testUsingContext to testWithoutContext using injected fakes. Feedback on the changes includes a suggestion to simplify the createConfigCommand helper in config_test.dart by defaulting to setup instance variables, and a recommendation to correct a test assertion in symbolize_test.dart to ensure it validates the actual injection of ToolContext rather than just checking the command name.

… symbolize_test.dart

# Conflicts: # packages/flutter_tools/test/commands.shard/hermetic/config_test.dart

loic-sharma

LGTM

flutter/flutter@8d8c1aa...c978386 2026-08-30 [email protected] Roll Skia from ce359c7fbfe6 to 588b550a4dd8 (1 revision) (flutter/flutter#192040) 2026-08-30 [email protected] Roll Skia from 3ae8e3d1e335 to ce359c7fbfe6 (1 revision) (flutter/flutter#192037) 2026-08-30 [email protected] Roll Fuchsia Linux SDK from J123CwfDPYyVDpkMl... to o9DikEfRoFyVK3LKD... (flutter/flutter#192031) 2026-08-29 [email protected] Make the AbsorbPointer sample show the difference from IgnorePointer (flutter/flutter#188940) 2026-08-29 [email protected] Roll Dart SDK from 391ff83cd1d3 to 76abdf94faa8 (1 revision) (flutter/flutter#192021) 2026-08-29 [email protected] Roll Skia from b392fb672d13 to 3ae8e3d1e335 (1 revision) (flutter/flutter#192020) 2026-08-29 [email protected] Roll Skia from 0b4249b26b38 to b392fb672d13 (6 revisions) (flutter/flutter#192016) 2026-08-29 [email protected] Fixes for texture-backed gradients (flutter/flutter#191925) 2026-08-29 [email protected] Roll Dart SDK from ae63a6eca3f1 to 391ff83cd1d3 (3 revisions) (flutter/flutter#192010) 2026-08-29 [email protected] Fix Android license detection for cmdline-tools 23.0+ (flutter/flutter#191554) 2026-08-29 [email protected] [flutter_tools] Pause web server requests until initial compilation finishes (flutter/flutter#191990) 2026-08-28 [email protected] fix(workflow): Mac Verify Binaries needs `Linux mac_android_aot_engine` (flutter/flutter#191999) 2026-08-28 [email protected] Roll Fuchsia Linux SDK from dDxH8G5cDBP602eyG... to J123CwfDPYyVDpkMl... (flutter/flutter#191984) 2026-08-28 [email protected] Update scheduled workflow for more tasks, add goldctl update (flutter/flutter#191991) 2026-08-28 [email protected] Update no-main-prs to leave a comment (flutter/flutter#191973) 2026-08-28 [email protected] [AGP 9.1.0 Migration #4] Replace plugin build-type copy with initWith on public DSL (flutter/flutter#191606) 2026-08-28 [email protected] Roll Skia from fe0b9a5261f9 to 0b4249b26b38 (1 revision) (flutter/flutter#191974) 2026-08-28 [email protected] [UIScene] Migrate dev/benchmarks/macrobenchmarks to UIScene lifecycle (flutter/flutter#191648) 2026-08-28 [email protected] Uiscene migrate remaining apps (flutter/flutter#191652) 2026-08-28 [email protected] [tool] Migrate SymbolizeCommand to modular dependency injection (flutter/flutter#190744) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

…r#190751) ## Summary Part 6 of the modular dependency injection migration. Migrates `ChannelCommand` to explicit constructor dependency injection via `ToolContext`, removing ambient access from `FlutterCommand` and adding hermetic unit tests. Stacked on flutter#190744 Compare: di/04-symbolize...di/05-channel Part of flutter#47161

…r#12693) flutter/flutter@8d8c1aa...c978386 2026-08-30 [email protected] Roll Skia from ce359c7fbfe6 to 588b550a4dd8 (1 revision) (flutter/flutter#192040) 2026-08-30 [email protected] Roll Skia from 3ae8e3d1e335 to ce359c7fbfe6 (1 revision) (flutter/flutter#192037) 2026-08-30 [email protected] Roll Fuchsia Linux SDK from J123CwfDPYyVDpkMl... to o9DikEfRoFyVK3LKD... (flutter/flutter#192031) 2026-08-29 [email protected] Make the AbsorbPointer sample show the difference from IgnorePointer (flutter/flutter#188940) 2026-08-29 [email protected] Roll Dart SDK from 391ff83cd1d3 to 76abdf94faa8 (1 revision) (flutter/flutter#192021) 2026-08-29 [email protected] Roll Skia from b392fb672d13 to 3ae8e3d1e335 (1 revision) (flutter/flutter#192020) 2026-08-29 [email protected] Roll Skia from 0b4249b26b38 to b392fb672d13 (6 revisions) (flutter/flutter#192016) 2026-08-29 [email protected] Fixes for texture-backed gradients (flutter/flutter#191925) 2026-08-29 [email protected] Roll Dart SDK from ae63a6eca3f1 to 391ff83cd1d3 (3 revisions) (flutter/flutter#192010) 2026-08-29 [email protected] Fix Android license detection for cmdline-tools 23.0+ (flutter/flutter#191554) 2026-08-29 [email protected] [flutter_tools] Pause web server requests until initial compilation finishes (flutter/flutter#191990) 2026-08-28 [email protected] fix(workflow): Mac Verify Binaries needs `Linux mac_android_aot_engine` (flutter/flutter#191999) 2026-08-28 [email protected] Roll Fuchsia Linux SDK from dDxH8G5cDBP602eyG... to J123CwfDPYyVDpkMl... (flutter/flutter#191984) 2026-08-28 [email protected] Update scheduled workflow for more tasks, add goldctl update (flutter/flutter#191991) 2026-08-28 [email protected] Update no-main-prs to leave a comment (flutter/flutter#191973) 2026-08-28 [email protected] [AGP 9.1.0 Migration flutter#4] Replace plugin build-type copy with initWith on public DSL (flutter/flutter#191606) 2026-08-28 [email protected] Roll Skia from fe0b9a5261f9 to 0b4249b26b38 (1 revision) (flutter/flutter#191974) 2026-08-28 [email protected] [UIScene] Migrate dev/benchmarks/macrobenchmarks to UIScene lifecycle (flutter/flutter#191648) 2026-08-28 [email protected] Uiscene migrate remaining apps (flutter/flutter#191652) 2026-08-28 [email protected] [tool] Migrate SymbolizeCommand to modular dependency injection (flutter/flutter#190744) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

bkonyi requested review from a team as code owners August 7, 2026 20:58

bkonyi requested review from reidbaker and removed request for a team August 7, 2026 20:58

flutter-dashboard Bot added the CICD Run CI/CD label Aug 7, 2026

github-actions Bot added tool Affects the "flutter" command-line tool. See also t: labels. team-android Owned by Android platform team team-ios Owned by iOS platform team team-macos Owned by the macOS platform team labels Aug 7, 2026

gemini-code-assist Bot reviewed Aug 7, 2026

View reviewed changes

Comment thread packages/flutter_tools/lib/src/context/tool_dependencies.dart Outdated

Comment thread packages/flutter_tools/lib/src/context/tool_dependencies.dart Outdated

bkonyi force-pushed the di/04-symbolize branch from c5eb54a to fbe46f0 Compare August 7, 2026 21:32

bkonyi mentioned this pull request Aug 7, 2026

[tool] Migrate DoctorCommand to modular dependency injection #190758

Merged

bkonyi force-pushed the di/04-symbolize branch from fbe46f0 to 0d7e85e Compare August 8, 2026 00:35

bkonyi marked this pull request as draft August 8, 2026 16:38

bkonyi removed request for a team and reidbaker August 8, 2026 16:41

bkonyi force-pushed the di/04-symbolize branch from 0d7e85e to 5a81601 Compare August 11, 2026 14:47

[tool] Define modular dependency injection containers and bootstrapper

85aa4f1

bkonyi force-pushed the di/04-symbolize branch from 5a81601 to d4ac95a Compare August 11, 2026 15:31

bkonyi added 2 commits August 11, 2026 17:52

[tool] Address review comments on modular dependency injection contai…

341b711

…ners

[tool] Wire dependency injection into runner and executable with requ…

5533bb8

…ired non-nullable contexts

bkonyi force-pushed the di/04-symbolize branch 2 times, most recently from 21b5ea2 to 2c011ef Compare August 11, 2026 19:48

bkonyi added 4 commits August 12, 2026 17:19

Merge remote-tracking branch 'origin/master' into di/02-runner-bootstrap

0125d94

# Conflicts: # packages/flutter_tools/lib/src/context/tool_dependencies.dart # packages/flutter_tools/test/general.shard/context/dependency_injection_test.dart

[tool] Address review feedback on FlutterCommandRunner dependency inj…

cd90309

…ection

[tool] Require ToolDependencies on FlutterCommandRunner and eliminate…

9f24ddb

… globals fallback

[tool] Remove unused getters and store analytics and toolContext dire…

15d4ff6

…ctly in FlutterCommandRunner

bkonyi force-pushed the di/04-symbolize branch from 2c011ef to 0339484 Compare August 12, 2026 18:16

bkonyi added 6 commits August 27, 2026 21:02

[tool] Make featureFlags public in ConfigCommand

16c0f18

Merge branch 'di/03-config' into di/04-symbolize

b43c3e2

[tool] Store non-nullable _toolContext in ConfigCommand

5b2972c

Merge branch 'di/03-config' into di/04-symbolize

4f45dfb

[tool] Remove unused analytics parameter from createConfigCommand in …

caac485

…config_test.dart

Merge branch 'di/03-config' into di/04-symbolize

4d39d30

bkonyi marked this pull request as ready for review August 28, 2026 14:43

gemini-code-assist Bot reviewed Aug 28, 2026

View reviewed changes

Comment thread packages/flutter_tools/test/commands.shard/hermetic/config_test.dart Outdated

Comment thread packages/flutter_tools/test/commands.shard/hermetic/symbolize_test.dart

bkonyi added 4 commits August 28, 2026 14:52

[tool] Default createConfigCommand to test fixtures in config_test.dart

a35837d

Merge branch 'di/03-config' into di/04-symbolize

eb7b9dd

[tool] Explicitly override toolContext getter and assert injection in…

4f0fef1

… symbolize_test.dart

Merge remote-tracking branch 'upstream/master' into di/04-symbolize

82c6d03

# Conflicts: # packages/flutter_tools/test/commands.shard/hermetic/config_test.dart

bkonyi requested review from chingjun and loic-sharma August 28, 2026 16:36

loic-sharma approved these changes Aug 28, 2026

View reviewed changes

bkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 28, 2026

auto-submit Bot added this pull request to the merge queue Aug 28, 2026

bkonyi mentioned this pull request Aug 28, 2026

[tool] Migrate ChannelCommand to modular dependency injection #190751

Merged

Merged via the queue into flutter:master with commit 4301cf5 Aug 28, 2026
23 checks passed

flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 28, 2026

Navigation Menu

Repository navigation

[tool] Migrate SymbolizeCommand to modular dependency injection - #190744

[tool] Migrate SymbolizeCommand to modular dependency injection#190744
auto-submit[bot] merged 63 commits into
flutter:masterfrom
bkonyi:di/04-symbolize

bkonyi commented Aug 7, 2026 •

edited

Loading

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

Uh oh!

Uh oh!

loic-sharma left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

bkonyi commented Aug 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

loic-sharma left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

bkonyi commented Aug 7, 2026 •

edited

Loading