[Gradle] Run DependencyVersionChecker before accessing Android DSL extensions in FlutterPlugin.apply (#194918)

## Description

Part of [#180137](https://github.com/flutter/flutter/issues/180137),
[#166550](https://github.com/flutter/flutter/issues/166550).

I broke the check in PR 7/11
([#193693](https://github.com/flutter/flutter/pull/193693)) in the new
DSL migration for users that were on AGP 8.11. This fixes the mistake
and adds a test. This will likely need to be cherry picked.

<details>
<summary>Agent authored details</summary>

### Related PR

- Follow-up PR:
[#195006](https://github.com/flutter/flutter/pull/195006). Because
`#194918` is a cherry-pick candidate, it only replaces the
`OUT_OF_SUPPORT_RANGE_PROPERTY` check in `FlutterPlugin.apply` with a
typed `catch (e: DependencyValidationException)`. Removing
`OUT_OF_SUPPORT_RANGE_PROPERTY` itself—which touches 7 property setters,
a task action, and 14 test assertions across
`DependencyVersionChecker.kt` and `DependencyVersionCheckerTest.kt`—is
split into [#195006](https://github.com/flutter/flutter/pull/195006) so
this cherry-pick stays minimal.

### Background

When [#193813](https://github.com/flutter/flutter/pull/193813)
(`ce6e766ad1c`) raised the minimum supported AGP version in
`DependencyVersionChecker` to `9.1.1` and bumped the compile-time
`agpVersion` in `packages/flutter_tools/gradle/build.gradle.kts` from
`8.11.1` to `9.1.1`, the Kotlin compiler began emitting AGP 9 binary
method signatures for calls against `ApplicationExtension` (such as
`ApplicationExtension.getDefaultConfig()Lcom/android/build/api/dsl/ApplicationDefaultConfig;`,
whereas in AGP 8.x `CommonExtension.getDefaultConfig()` returned
`DefaultConfig` without a covariant bridge on `ApplicationExtension`).

Because `FlutterPlugin.apply` invoked `addFlutterTasks(project)` (which
calls `configureAbis` -> `configureAbiWithoutSplits` ->
`extension.defaultConfig.ndk` and `DslVersionCodes.from(appExtension)`
introduced in [#193693](https://github.com/flutter/flutter/pull/193693)
/ [#192488](https://github.com/flutter/flutter/pull/192488)) before
running `DependencyVersionChecker.checkDependencyVersions(project)`,
building an AGP 8.x project crashed during `addFlutterTasks` with:

```text
java.lang.NoSuchMethodError: 'com.android.build.api.dsl.ApplicationDefaultConfig com.android.build.api.dsl.ApplicationExtension.getDefaultConfig()'
```

before reaching
`DependencyVersionChecker.checkDependencyVersions(project)`, preventing
the user from seeing the actionable minimum AGP version error message
and upgrade guidance.

### Changes

1. **`FlutterPlugin.kt`**:
- Moves `FlutterPluginUtils.detectApplyingKotlinGradlePlugin(project)`
and the `shouldSkipDependencyChecks` /
`DependencyVersionChecker.checkDependencyVersions(project)` block to the
beginning of `FlutterPlugin.apply(project)` (immediately after
`this.project = project`), before `addFlutterTasks(project)` and all
`ApplicationExtension` / Android DSL accesses.
- Replaces the `OUT_OF_SUPPORT_RANGE_PROPERTY` extra-property check in
`FlutterPlugin.apply` with a typed `catch (e:
DependencyValidationException)` that rethrows
`DependencyValidationException` directly and logs any other `Exception`,
removing the comment that described `OUT_OF_SUPPORT_RANGE_PROPERTY`.
2. **`FlutterPluginTest.kt`**:
- Adds `FlutterPlugin apply checks dependency versions before
configuring project`, verifying via `verifyOrder` that
`FlutterPluginUtils.detectApplyingKotlinGradlePlugin(project)` and
`DependencyVersionChecker.checkDependencyVersions(project)` run before
task registration, `AndroidComponentsExtension` callbacks
(`finalizeDsl`, `onVariants`), and `ApplicationExtension` DSL
configuration (`defaultConfig`, `buildTypes`).
- Adds `FlutterPlugin apply rethrows DependencyValidationException and
logs other exceptions`, verifying both exception branches without
relying on the extra property, and stubs supported Gradle and AGP
versions in `setupTestProjectEnvironment` /
`setupMockComponentsExtension`.
3. **`packages/flutter_tools/gradle/AGENTS.md`**:
- Updates the `Matrix Verification` bullet on line 40 to reference the
supported AGP and Gradle versions defined in
`DependencyVersionChecker.kt`.

### Verification

- **Unit test negative/positive verification**:
- Moving `DependencyVersionChecker` after `addFlutterTasks` or moving
`detectApplyingKotlinGradlePlugin` after `DependencyVersionChecker`
fails `FlutterPlugin apply checks dependency versions before configuring
project`.
- Reverting the typed `catch (e: DependencyValidationException)` to the
property check fails `FlutterPlugin apply rethrows
DependencyValidationException and logs other exceptions`.
  - `./gradlew test` with the fix applied: all tests pass.
- **End-to-end reproduction with AGP 8.11.1 and Gradle 9.3.1**:
- Before fix: failed at `FlutterPlugin.configureAbiWithoutSplits` with
`java.lang.NoSuchMethodError:
'com.android.build.api.dsl.ApplicationDefaultConfig
com.android.build.api.dsl.ApplicationExtension.getDefaultConfig()'`.
- After fix: fails cleanly with `Error: Your project's Android Gradle
Plugin version (Android Gradle Plugin version 8.11.1) is lower than
Flutter's minimum supported version of Android Gradle Plugin version
9.1.1. Please upgrade your Android Gradle Plugin version.`
- **Formatting**:
- `ktlint 1.5.0` with
`--editorconfig=dev/bots/test/analyze-test-input/.editorconfig
--baseline=dev/bots/test/analyze-test-input/ktlint-baseline.xml` passed
with zero findings.

</details>

---

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [AI contribution guidelines] and understand my
responsibilities, or I am not using AI tools.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant in-code documentation (doc comments with
`///`).
- [x] If this PR introduces a new feature or capability, I created and
linked a website documentation issue or PR in [flutter/website] (or
verified none is needed).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
[AI contribution guidelines]:
https://github.com/flutter/flutter/blob/main/CONTRIBUTING.ai.md
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
[test-exempt]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[flutter/website]: https://github.com/flutter/website
[breaking change policy]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
[Discord]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
[Data Driven Fixes]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
3 files changed
tree: 2db5d4c73e7641f330661da99b7cb562448e972a
  1. .agents/
  2. .claude/
  3. .gemini/
  4. .github/
  5. .vscode/
  6. agent-artifacts/
  7. bin/
  8. buildtools/
  9. dev/
  10. docs/
  11. engine/
  12. examples/
  13. packages/
  14. third_party/
  15. .autoroller-preupload.sh
  16. .ci.yaml
  17. .direnv
  18. .gitattributes
  19. .gitignore
  20. analysis_options.yaml
  21. analysis_options_common.yaml
  22. AUTHORS
  23. CHANGELOG.md
  24. CODE_OF_CONDUCT.md
  25. CODEOWNERS
  26. CONTRIBUTING.ai.md
  27. CONTRIBUTING.md
  28. dartdoc_options.yaml
  29. DEPS
  30. flutter_console.bat
  31. LICENSE
  32. PATENT_GRANT
  33. pubspec.lock
  34. pubspec.yaml
  35. README.md
  36. TESTOWNERS
README.md

Flutter CI Status Discord badge Twitter handle BlueSky badge LFX Health Score CII Best Practices SLSA 1

Flutter is Google's SDK for crafting beautiful, fast user experiences for mobile, web, and desktop from a single codebase. Flutter works with existing code, is used by developers and organizations around the world, and is free and open source.

Documentation

For release and other announcements, join the flutter-announce mailing list. Our documentation also tracks breaking changes across releases.

Terms of service

The Flutter tool may occasionally download resources from Google servers. By downloading or using the Flutter SDK, you agree to the Google Terms of Service: https://policies.google.com/terms

For example, when installed from GitHub (as opposed to from a prepackaged archive), the Flutter tool will download the Dart SDK from Google servers immediately when first run, as it is used to execute the flutter tool itself. This will also occur when Flutter is upgraded (e.g. by running the flutter upgrade command).

About Flutter

We think Flutter will help you create beautiful, fast apps, with a productive, extensible and open development model, whether you're targeting iOS or Android, web, Windows, macOS, Linux or embedding it as the UI toolkit for a platform of your choice.

Beautiful user experiences

We want to enable designers to deliver their full creative vision without being forced to water it down due to limitations of the underlying framework. Flutter‘s layered architecture gives you control over every pixel on the screen and its powerful compositing capabilities let you overlay and animate graphics, video, text, and controls without limitation. Flutter includes a full set of widgets that deliver pixel-perfect experiences whether you’re building for iOS (Cupertino) or other platforms (Material), along with support for customizing or creating entirely new visual components.

Fast results

Flutter is fast. It's powered by hardware-accelerated 2D graphics libraries like Skia (which underpins Chrome and Android) and Impeller. We architected Flutter to support glitch-free, jank-free graphics at the native speed of your device.

Flutter code is powered by the world-class Dart programming language, which enables compilation to 32-bit and 64-bit ARM machine code for iOS and Android, JavaScript and WebAssembly for the web, as well as Intel x64 and ARM for desktop devices.

Productive development

Flutter offers stateful hot reload, allowing you to make changes to your code and see the results instantly without restarting your app or losing its state.

Hot reload animation

Extensible and open model

Flutter works with any development tool (or none at all), and also includes editor plug-ins for both Visual Studio Code and IntelliJ / Android Studio. Flutter provides tens of thousands of packages to speed your development, regardless of your target platform. And accessing other native code is easy, with support for both FFI (on Android, on iOS, on macOS, on Linux, and on Windows) as well as platform-specific APIs.

Flutter is a fully open-source project, and we welcome contributions. Information on how to get started can be found in our contributor guide.