fix: resolve issue #177379 by using lazy buildDirectory.dir() API in build.gradle template#187127
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates Gradle build files and templates to use the lazy DirectoryProperty.dir(...) API instead of resolving the build directory eagerly. It also adds a unit test to verify this template structure. The review feedback suggests avoiding direct usage of dart:io and io.File in flutter_tools tests, recommending the use of package:file/file.dart and package:file/local.dart instead to maintain test hermeticity.
reidbaker
left a comment
There was a problem hiding this comment.
The tests test the wrong thing but the fix seems reasonable.
Blocking because we do not want to verify that we use the new behavior. We want a test that shows that the bug is resolved. That will likely involve creating an aab that is exhibits to this bug and verifying that assets are present.
…API in build.gradle template
…ies and using lazy directory providers - Update deferred component Gradle template and test configurations to use lazy directory providers (project.layout.buildDirectory.dir(...)) instead of eager GString string coercion, preventing incorrect build path evaluation. - Configure dynamic feature subprojects to explicitly depend on flutterCompileTask for their assets and native libraries merging tasks (mergeVariantAssets and mergeVariantJniLibFolders), resolving parallel build race conditions during clean builds.
220c0d1 to
c6ac19f
Compare
gmackall
left a comment
There was a problem hiding this comment.
I wasn't at the bug bash but was this verified to fix the linked issue? Based on the test it would seem like yes?
If so lgtm
So, triage bot correctly identified the fix - the lazy buildDirectory is correct. I verified this manually: letanssang/test_dynamic_module#3 I had to verify it manually because the tests do not correctly repro the issue. I even removed the FlutterPlugin change, because that was added with the test. The repro test passes on master - currently trying to come up with a better test. |
|
Ok, I figured out why the test pass on master and the issue exists. The test actually is setup correctly - and DIFFERENTLY - from the template: The configuration uses the The template does not use the .get() call. So, I believe the test being added still is somewhat valuable - even if it passes on master. The important part is the test now matches the actual template. |
|
@reidbaker - Can you take a look at the test and the above comments and see if it fits your requested changes? |
reidbaker
left a comment
There was a problem hiding this comment.
Thanks for removing the change detector tests
Roll Flutter from 2ba5420a7049 to 1bdf4af29076 (43 revisions) flutter/flutter@2ba5420...1bdf4af 2026-06-05 [email protected] Roll Packages from 03352b5 to 61bdbb4 (5 revisions) (flutter/flutter#187612) 2026-06-05 [email protected] Roll Skia from 6e003d7f69c8 to a47a9a2c8ae5 (1 revision) (flutter/flutter#187610) 2026-06-05 [email protected] Roll Dart SDK from aad8be4ce307 to 6a9a0efe66eb (10 revisions) (flutter/flutter#187609) 2026-06-05 [email protected] Roll Skia from 494f1bf55f51 to 6e003d7f69c8 (2 revisions) (flutter/flutter#187607) 2026-06-05 [email protected] Roll Fuchsia Linux SDK from ZE1Jy9CtVVi-tjBAE... to N_LiSaBSUsE2LDZgG... (flutter/flutter#187597) 2026-06-05 [email protected] Roll Skia from 59556fdb8c33 to 494f1bf55f51 (2 revisions) (flutter/flutter#187596) 2026-06-04 [email protected] Roll Skia from 8eb107046fd5 to 59556fdb8c33 (1 revision) (flutter/flutter#187590) 2026-06-04 [email protected] Remove `embedded_android_views_integration_test.dart` (flutter/flutter#187465) 2026-06-04 [email protected] Remove Material imports from rendering editable tests (flutter/flutter#186951) 2026-06-04 [email protected] [Impeller] Wait for the Vulkan device to become idle before destroying Vulkan objects in the AHBSwapchainImplVK destructor (flutter/flutter#187477) 2026-06-04 [email protected] [iOS] Eliminate unnecessary redeclaration of FlutterDisplayLink (flutter/flutter#187557) 2026-06-04 [email protected] Roll Skia from 928ded2a31af to 8eb107046fd5 (1 revision) (flutter/flutter#187583) 2026-06-04 [email protected] Log stdout in adb.dart (flutter/flutter#187531) 2026-06-04 [email protected] Clarify RouterDelegate popRoute bubbling (flutter/flutter#186875) 2026-06-04 [email protected] Roll Skia from 928ded2a31af to 8eb107046fd5 (1 revision) (flutter/flutter#187584) 2026-06-04 [email protected] Add updating-android-sdk agent skill for rolling Android SDK in CIPD (flutter/flutter#187576) 2026-06-04 [email protected] Fixing alignment issue (flutter/flutter#187518) 2026-06-04 [email protected] [Material Cross Imports] Clean up Material Divider usages (flutter/flutter#187300) 2026-06-04 [email protected] Roll Skia from cecc0e0da9ae to 928ded2a31af (6 revisions) (flutter/flutter#187574) 2026-06-04 [email protected] Use swift demangle to verify internal Swift symbols (flutter/flutter#186835) 2026-06-04 [email protected] Add android 37 platform and build tools to script for android cipd bundle creation (flutter/flutter#187571) 2026-06-04 [email protected] [Impeller] Increase the precision of the IPSampleWithTileModeOES coords parameter to match the input coordinates in the tiled_texture_fill_external shader (flutter/flutter#187545) 2026-06-04 [email protected] Roll Packages from b11504f to 03352b5 (4 revisions) (flutter/flutter#187569) 2026-06-04 [email protected] Roll Dart SDK to Dart 3.13 beta2 (flutter/flutter#187555) 2026-06-04 [email protected] Roll Skia from 611e3f8ceb93 to cecc0e0da9ae (1 revision) (flutter/flutter#187562) 2026-06-04 [email protected] Add step to bootstrap Flutter tool in coverage workflow (flutter/flutter#187199) 2026-06-04 [email protected] Roll Skia from 4fdb859c8da7 to 611e3f8ceb93 (4 revisions) (flutter/flutter#187554) 2026-06-04 [email protected] Roll Skia from 0020aae33f63 to 4fdb859c8da7 (2 revisions) (flutter/flutter#187552) 2026-06-04 [email protected] Roll Fuchsia Linux SDK from ap7MhLX4TdpWRrLS_... to ZE1Jy9CtVVi-tjBAE... (flutter/flutter#187550) 2026-06-04 [email protected] Add vector_math to package issue template (flutter/flutter#187536) 2026-06-04 [email protected] Manual roll Dart SDK from d39850bf4a01 to 3b70b98fa7c0 (flutter/flutter#187519) 2026-06-04 [email protected] Roll Skia from d625048c853a to 0020aae33f63 (20 revisions) (flutter/flutter#187539) 2026-06-04 [email protected] [Impeller][Windows] fix black screen on OpenGL fallback (flutter/flutter#187288) 2026-06-04 [email protected] Fix unintentionally joined path contours (flutter/flutter#187522) 2026-06-03 [email protected] fix: resolve issue #177379 by using lazy buildDirectory.dir() API in build.gradle template (flutter/flutter#187127) 2026-06-03 [email protected] Add a skill for flake analysis (flutter/flutter#187530) 2026-06-03 [email protected] adds linux impeller project flag (flutter/flutter#186982) 2026-06-03 [email protected] Add support for stylus buttons (flutter/flutter#183369) 2026-06-03 [email protected] Prevent Cubic transform from looping on out-of-range input (flutter/flutter#185875) 2026-06-03 [email protected] [Impeller] Reland: Allow attaching specific texture mip levels and slices (flutter/flutter#187470) 2026-06-03 [email protected] [skia] Update image deserial proc (flutter/flutter#185041) 2026-06-03 [email protected] docs: update Impeller advanced blend docs for framebuffer fetch (flutter/flutter#185457) 2026-06-03 [email protected] Handle#6537 fifth grouped tests (flutter/flutter#183720) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: ...
…r#11849) Roll Flutter from 2ba5420a7049 to 1bdf4af29076 (43 revisions) flutter/flutter@2ba5420...1bdf4af 2026-06-05 [email protected] Roll Packages from 03352b5 to 61bdbb4 (5 revisions) (flutter/flutter#187612) 2026-06-05 [email protected] Roll Skia from 6e003d7f69c8 to a47a9a2c8ae5 (1 revision) (flutter/flutter#187610) 2026-06-05 [email protected] Roll Dart SDK from aad8be4ce307 to 6a9a0efe66eb (10 revisions) (flutter/flutter#187609) 2026-06-05 [email protected] Roll Skia from 494f1bf55f51 to 6e003d7f69c8 (2 revisions) (flutter/flutter#187607) 2026-06-05 [email protected] Roll Fuchsia Linux SDK from ZE1Jy9CtVVi-tjBAE... to N_LiSaBSUsE2LDZgG... (flutter/flutter#187597) 2026-06-05 [email protected] Roll Skia from 59556fdb8c33 to 494f1bf55f51 (2 revisions) (flutter/flutter#187596) 2026-06-04 [email protected] Roll Skia from 8eb107046fd5 to 59556fdb8c33 (1 revision) (flutter/flutter#187590) 2026-06-04 [email protected] Remove `embedded_android_views_integration_test.dart` (flutter/flutter#187465) 2026-06-04 [email protected] Remove Material imports from rendering editable tests (flutter/flutter#186951) 2026-06-04 [email protected] [Impeller] Wait for the Vulkan device to become idle before destroying Vulkan objects in the AHBSwapchainImplVK destructor (flutter/flutter#187477) 2026-06-04 [email protected] [iOS] Eliminate unnecessary redeclaration of FlutterDisplayLink (flutter/flutter#187557) 2026-06-04 [email protected] Roll Skia from 928ded2a31af to 8eb107046fd5 (1 revision) (flutter/flutter#187583) 2026-06-04 [email protected] Log stdout in adb.dart (flutter/flutter#187531) 2026-06-04 [email protected] Clarify RouterDelegate popRoute bubbling (flutter/flutter#186875) 2026-06-04 [email protected] Roll Skia from 928ded2a31af to 8eb107046fd5 (1 revision) (flutter/flutter#187584) 2026-06-04 [email protected] Add updating-android-sdk agent skill for rolling Android SDK in CIPD (flutter/flutter#187576) 2026-06-04 [email protected] Fixing alignment issue (flutter/flutter#187518) 2026-06-04 [email protected] [Material Cross Imports] Clean up Material Divider usages (flutter/flutter#187300) 2026-06-04 [email protected] Roll Skia from cecc0e0da9ae to 928ded2a31af (6 revisions) (flutter/flutter#187574) 2026-06-04 [email protected] Use swift demangle to verify internal Swift symbols (flutter/flutter#186835) 2026-06-04 [email protected] Add android 37 platform and build tools to script for android cipd bundle creation (flutter/flutter#187571) 2026-06-04 [email protected] [Impeller] Increase the precision of the IPSampleWithTileModeOES coords parameter to match the input coordinates in the tiled_texture_fill_external shader (flutter/flutter#187545) 2026-06-04 [email protected] Roll Packages from b11504f to 03352b5 (4 revisions) (flutter/flutter#187569) 2026-06-04 [email protected] Roll Dart SDK to Dart 3.13 beta2 (flutter/flutter#187555) 2026-06-04 [email protected] Roll Skia from 611e3f8ceb93 to cecc0e0da9ae (1 revision) (flutter/flutter#187562) 2026-06-04 [email protected] Add step to bootstrap Flutter tool in coverage workflow (flutter/flutter#187199) 2026-06-04 [email protected] Roll Skia from 4fdb859c8da7 to 611e3f8ceb93 (4 revisions) (flutter/flutter#187554) 2026-06-04 [email protected] Roll Skia from 0020aae33f63 to 4fdb859c8da7 (2 revisions) (flutter/flutter#187552) 2026-06-04 [email protected] Roll Fuchsia Linux SDK from ap7MhLX4TdpWRrLS_... to ZE1Jy9CtVVi-tjBAE... (flutter/flutter#187550) 2026-06-04 [email protected] Add vector_math to package issue template (flutter/flutter#187536) 2026-06-04 [email protected] Manual roll Dart SDK from d39850bf4a01 to 3b70b98fa7c0 (flutter/flutter#187519) 2026-06-04 [email protected] Roll Skia from d625048c853a to 0020aae33f63 (20 revisions) (flutter/flutter#187539) 2026-06-04 [email protected] [Impeller][Windows] fix black screen on OpenGL fallback (flutter/flutter#187288) 2026-06-04 [email protected] Fix unintentionally joined path contours (flutter/flutter#187522) 2026-06-03 [email protected] fix: resolve issue #177379 by using lazy buildDirectory.dir() API in build.gradle template (flutter/flutter#187127) 2026-06-03 [email protected] Add a skill for flake analysis (flutter/flutter#187530) 2026-06-03 [email protected] adds linux impeller project flag (flutter/flutter#186982) 2026-06-03 [email protected] Add support for stylus buttons (flutter/flutter#183369) 2026-06-03 [email protected] Prevent Cubic transform from looping on out-of-range input (flutter/flutter#185875) 2026-06-03 [email protected] [Impeller] Reland: Allow attaching specific texture mip levels and slices (flutter/flutter#187470) 2026-06-03 [email protected] [skia] Update image deserial proc (flutter/flutter#185041) 2026-06-03 [email protected] docs: update Impeller advanced blend docs for framebuffer fetch (flutter/flutter#185457) 2026-06-03 [email protected] Handle#6537 fifth grouped tests (flutter/flutter#183720) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: ...
…API in build.gradle template (flutter#187127) # Executive Triage & Resolution Report: Issue flutter#177379 A comprehensive report on the end-to-end lifecycle of reproducing and resolving GitHub issue [flutter#177379](flutter#177379). --- ## 1. Executive Summary - **GitHub Issue**: `flutter#177379` - **Problem Description**: When building an Android App Bundle (`.aab`) with deferred components, the Dart code split works, but all declared assets/JNI libraries in the dynamic feature modules are missing from the final `.aab`. - **Triage Diagnosis**: The root cause is a property coercion bug in the Gradle templates where `project.layout.buildDirectory` (which is a lazy `DirectoryProperty` in Gradle 7.x/8.x+) was evaluated inside double-quoted Groovy GStrings (`"${project.layout.buildDirectory}/..."`). Groovy coerced this property into a debug string descriptor (e.g. `"property(org.gradle.api.file.Directory, ...)"`), pointing Gradle to an invalid path. Consequently, Gradle silently ignored and skipped packaging the assets and JNI library directories. - **Resolution Details**: The template and affected integration test configurations were updated to use the native lazy Gradle subdirectory API: `project.layout.buildDirectory.dir(...)`. A new unit test has been added to ensure this template maintains correct lazy API usage and prevents regression. - **Status**: All unit and integration tests pass successfully on `triage-issue-177379` branch. The changes have been fully approved by peer reviewer agents and committed. --- ## 2. Root Cause Analysis & Technical Deep Dive In Gradle 7.x/8.x+, `project.layout.buildDirectory` represents a `DirectoryProperty`. It is designed to be evaluated lazily to allow late modification of build output pathways. ### The Problematic Code (Before) ```groovy sourceSets { applicationVariants.all { variant -> main.assets.srcDirs += "${project.layout.buildDirectory}/intermediates/flutter/${variant.name}/deferred_assets" main.jniLibs.srcDirs += "${project.layout.buildDirectory}/intermediates/flutter/${variant.name}/deferred_libs" } } ``` When Groovy parses `"${project.layout.buildDirectory}/..."`, it calls `.toString()` on the `DirectoryProperty` during GString evaluation. Instead of resolving to the build path (e.g. `build/`), it coerces to `property(org.gradle.api.file.Directory, property(org.gradle.api.file.Directory, ...))/intermediates/flutter/release/deferred_assets`. Since Gradle does not find this directory, it silently skips it, leading to empty dynamic feature asset bundles. ### The Solution (After) ```groovy sourceSets { applicationVariants.all { variant -> main.assets.srcDirs += project.layout.buildDirectory.dir("intermediates/flutter/${variant.name}/deferred_assets") main.jniLibs.srcDirs += project.layout.buildDirectory.dir("intermediates/flutter/${variant.name}/deferred_libs") } } ``` By calling `.dir(...)`, we obtain a provider of `Directory`, which is evaluated correctly during the execution phase of Gradle when assets and JNI libraries are gathered. Fixes: flutter#177379 ## 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 documentation (doc comments with `///`). - [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.
Executive Triage & Resolution Report: Issue #177379
A comprehensive report on the end-to-end lifecycle of reproducing and resolving GitHub issue flutter/flutter#177379.
1. Executive Summary
flutter/flutter#177379.aab) with deferred components, the Dart code split works, but all declared assets/JNI libraries in the dynamic feature modules are missing from the final.aab.project.layout.buildDirectory(which is a lazyDirectoryPropertyin Gradle 7.x/8.x+) was evaluated inside double-quoted Groovy GStrings ("${project.layout.buildDirectory}/..."). Groovy coerced this property into a debug string descriptor (e.g."property(org.gradle.api.file.Directory, ...)"), pointing Gradle to an invalid path. Consequently, Gradle silently ignored and skipped packaging the assets and JNI library directories.project.layout.buildDirectory.dir(...). A new unit test has been added to ensure this template maintains correct lazy API usage and prevents regression.triage-issue-177379branch. The changes have been fully approved by peer reviewer agents and committed.2. Root Cause Analysis & Technical Deep Dive
In Gradle 7.x/8.x+,
project.layout.buildDirectoryrepresents aDirectoryProperty. It is designed to be evaluated lazily to allow late modification of build output pathways.The Problematic Code (Before)
sourceSets { applicationVariants.all { variant -> main.assets.srcDirs += "${project.layout.buildDirectory}/intermediates/flutter/${variant.name}/deferred_assets" main.jniLibs.srcDirs += "${project.layout.buildDirectory}/intermediates/flutter/${variant.name}/deferred_libs" } }When Groovy parses
"${project.layout.buildDirectory}/...", it calls.toString()on theDirectoryPropertyduring GString evaluation. Instead of resolving to the build path (e.g.build/), it coerces toproperty(org.gradle.api.file.Directory, property(org.gradle.api.file.Directory, ...))/intermediates/flutter/release/deferred_assets. Since Gradle does not find this directory, it silently skips it, leading to empty dynamic feature asset bundles.The Solution (After)
sourceSets { applicationVariants.all { variant -> main.assets.srcDirs += project.layout.buildDirectory.dir("intermediates/flutter/${variant.name}/deferred_assets") main.jniLibs.srcDirs += project.layout.buildDirectory.dir("intermediates/flutter/${variant.name}/deferred_libs") } }By calling
.dir(...), we obtain a provider ofDirectory, which is evaluated correctly during the execution phase of Gradle when assets and JNI libraries are gathered.Fixes: #177379
Pre-launch Checklist
///).