Buildengine phase1 - #386
Open
kanhadhiwar53-hash wants to merge 2 commits into
Open
kanhadhiwar53-hash wants to merge 2 commits into
kanhadhiwar53-hash wants to merge 2 commits into
Conversation
added 2 commits
September 22, 2026 08:06
…2/zipalign/apksigner) - BuildManager: async orchestrator (build/clean/rebuild/cancel), streams BuildLogEvent per stage - steps/: EcjCompileStep, KotlincCompileStep, D8DexStep, Aapt2Step, ApkAssembleStep, ZipAlignStep, ApkSignStep — each parses tool diagnostics into BuildError(file, line, message) for editor click-to-line - classpath/ClasspathCollector: libs/*.jar + libs/*.aar -> compile/dex classpath, with content-hash AAR exploding cache - tools/BuildToolsLocator + ToolsProvisioner: locates bundled tools under <filesDir>/build-tools/, first-run checksummed download of the core package (not bundled in APK, not on app-open) so no forced huge SDK download; Kotlin toolchain is a separate optional package - See buildengine/PLAN.md for full roadmap, requirement mapping, and the exact existing files (GradleBuildService, RunTasksViewModel, etc.) that Phase 3 needs to touch to wire this in.
…t model)
Scans src/main/{java,kotlin,res,assets} + AndroidManifest.xml + libs/*.jar
+ libs/*.aar directly and produces the same AndroidModule/GradleProject/
WorkspaceImpl shape WorkspaceModelBuilder builds from a Gradle sync today —
so Java/Kotlin completion, indexing and XML resource resolution keep
working once Gradle sync is gone, not just APK building.
Reuses tooling:builder-model-impl's Default* classes (plain data holders
around AGP builder-model interfaces, no Gradle runtime dependency) instead
of inventing new model types, so existing consumers of the project model
are untouched. Updated buildengine/PLAN.md: keep tooling:builder-model-impl
in Phase 4's removal list, added Phase 2 status + known limitations
(single-module only, no transitive Maven deps, ProjectSyncIssues shape
unverified against the real jar, not yet wired into ProjectManagerImpl).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.