Android build.gradle: ARCore dep + SDK/Kotlin bumps (replicates upstream #33)

[iamEtornam]

Replicates upstream PR hlefe#33 by @navakanth.

Change (net of 9 commits): rewrites android/build.gradle — removes the buildscript/allprojects blocks, adds com.google.ar:core:1.52.0, bumps compileSdk/targetSdk to 36 and kotlin-stdlib to 2.1.10, adds packagingOptions { jniLibs { useLegacyPackaging = false } }.

⚠️ Review note: the net diff also drops androidx.constraintlayout:constraintlayout (used by the hand-motion guide layout) and removes the AGP classpath while raising compileSdk to 36 — this may break the build. Flagged for review.

Original author: navakanth. Upstream PR: hlefe#33

[iamEtornam]

Review — VERDICT: ⚠️ CONCERNS — do not merge as-is without a host-toolchain check

Replicates upstream hlefe#33 (rewrites android/build.gradle). Gradle can't be built here (no example app); reviewed by static read.

• ✅ constraintlayout drop is SAFE — despite my initial worry. grep -rniE constraintlayout android/src = 0 hits; sceneform_hand_layout.xml uses FrameLayout, HandMotionView extends ImageView. Unused → correct cleanup.
• ✅ Removing buildscript{}/allprojects{} is fine for a Flutter plugin module — AGP/Kotlin classpath & repos come from the host app; self-declaring them is redundant.
• ⚠️ compileSdk/targetSdk 34 → 36 is a conditional break. API 36 needs AGP ≥ 8.6 (rec. 8.9+). The removed buildscript had pinned AGP 8.3.2 — with the pin gone, AGP comes from the host; if it's still 8.3.2 the build fails: "compileSdk 36 requires Android Gradle plugin 8.6.0 or higher."
• ⚠️ Hardcoded kotlin-stdlib:2.1.10 while the Kotlin plugin version now comes from the host (previously pinned 1.9.10). A host on Kotlin < 2.1 will error/warn on incompatible stdlib metadata.
• 🔸 packagingOptions{ jniLibs{ useLegacyPackaging=false } } valid & harmless (already the default; packagingOptions is the deprecated-but-working alias). Adding explicit com.google.ar:core:1.52.0 is benign (already transitive via arsceneview).

Bottom line: internally coherent and follows modern plugin conventions, but it raises the required host toolchain (AGP ≥ 8.6, Kotlin ≥ 2.1). Against this repo's prior baseline (AGP 8.3.2 / Kotlin 1.9.10) it will fail to build until the consuming app upgrades. Merge only after confirming your app's Android toolchain, or split out the SDK-36/Kotlin-2.1 bumps.

[gemini-code-assist]

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.