Remove the AGP 9 opt-outs once react-native-test-app supports them
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 45/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 静か
- 技術スタック
- android, kotlin, react-native
- 領域
- build-system, mobile
調査の方向性
apps/test-app/android/gradle.properties と packages/host/android/build.gradle から始め、次に、引用されている react-native-test-app の Android Gradle ファイルと 2 つの移行ブロッカーを確認します。host モジュールの新しい DSL に関する変更を検証し、その built-in-Kotlin に対する条件付き動作を維持したうえで、Android テストアプリの構成およびインストールのチェックを実行します。完了の条件は、両方のオプトアウトが削除され、upstream のブロッカーが解決された後もテストアプリの構成とインストールが成功することです。
索引モデルが issue の本文から書いたものです。
説明
The test app sets both AGP 9 opt-outs in apps/test-app/android/gradle.properties:
android.builtInKotlin=false
android.newDsl=false
This issue tracks removing them. Both are blocked on react-native-test-app, not on us.
Correction. An earlier revision of this issue said we were "already off the sibling flag" and running built-in Kotlin natively. That was wrong — we tried it, the Android build failed, and #372 had to put
android.builtInKotlin=falseback. Details in blocker 1.
Deadline
Per Android's DSL/API migration timeline, AGP 10.0 (estimated late 2026) removes both opt-outs, along with the legacy APIs. Once React Native pins an AGP 10 these flags stop being a choice — the build either has migrated or it doesn't configure. That is plausibly one or two React Native minors away, so this is a when, not an if.
Blocker 1: built-in Kotlin — sources added via java.srcDirs are not compiled
react-native-test-app's app module adds its version-specific sources with main.java.srcDirs += [...]. The Kotlin plugin adds those directories to the Kotlin source set; AGP's built-in Kotlin only picks up the standard ones. Building with the flag unset (react-native-test-app 5.4.8, React Native 0.88 nightly, AGP 9.2.1) fails :app:compileReleaseKotlin:
e: ComponentActivity.kt:33:9 Unresolved reference 'ComponentActivityDelegate'
e: MainActivity.kt:145:9 Unresolved reference 'testApp'
e: MainActivity.kt:225:50 Unresolved reference 'reactHost'
e: MainActivity.kt:135:17 Unresolved reference 'canUseCamera'
e: MainActivity.kt:25:45 Unresolved reference 'ComponentBottomSheetDialogFragment'
Every unresolved symbol lives in an added directory — src/reactactivitydelegate-0.75/java, src/reactapplication-0.76/java, src/reacthost-0.76/java, src/camera/java. Files under the standard src/main/java compile fine. Their useBuiltInKotlin gate avoids applying the Kotlin plugin on top of built-in Kotlin, but the module still needs that plugin to compile at all.
Likely fix upstream: add the same directories to main.kotlin.srcDirs.
Our own half is already done. react-native-node-api applies the Kotlin plugin only when built-in Kotlin is unavailable, so the published library works under either regime no matter what the harness needs. That shipped in #372 and waits on nothing.
Blocker 2: the new DSL
Both react-native-test-app modules still use the old DSL, permalinked at 5.4.8:
| Location | Old DSL |
|---|---|
android/app/build.gradle#L77 |
compileSdkVersion |
android/app/build.gradle#L88 |
kotlinOptions (gated on !useBuiltInKotlin) |
android/app/build.gradle#L95-L96 |
minSdkVersion, targetSdkVersion |
android/app/build.gradle#L161 |
lintOptions |
android/support/build.gradle#L16 |
compileSdkVersion |
android/support/build.gradle#L19-L20 |
minSdkVersion, targetSdkVersion |
packages/host/android/build.gradle is old DSL too, and this half is ours. It matters beyond our CI: it is the published library, so it has to be new-DSL clean before consumers can migrate — the same shape as the built-in Kotlin problem that file already solves.
#L52,#L69— script-styleapply plugin:instead of aplugins {}block#L101—compileSdkVersion#L109-L110—minSdkVersion,targetSdkVersion#L154—lintOptions
Ordering
The flags are coupled one way: builtInKotlin=false requires newDsl=false, because the Kotlin plugin is incompatible with the new DSL. So blocker 1 has to be resolved before blocker 2 can be — though each piece of work is independently useful, and our own module can move to the new DSL at any time.
Nothing upstream tracks either blocker yet. Whether to file it, and the paste-ready text, is #390.
Done when
Both lines are gone from apps/test-app/android/gradle.properties and the Android test app still configures and installs.
Introduced by #372. Links into this repo point at main and are accurate as of that PR merging.
- 主要言語
- TypeScript
- スター
- 188
- フォーク
- 10
- 平均マージ
- 2日 17時間
- マージ済み PR(30日)
- 3
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
callstackincubator/react-native-node-api のほかの issue
-
Android 🤖
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects オープンAndroid 🤖 good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
callstackincubator/react-native-node-api#163 · コメント 1 件 ·
-
enhancement good first issue gyp-to-cmake
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
callstackincubator/react-native-node-api#161 · コメント 3 件 · リアクション 1 件 ·
-
enhancement Host 🏡
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
Automatable good first issue
難易度 3/5 1〜2日 初心者へのやさしさ 15/100
callstackincubator/react-native-node-api#424 · コメント 1 件 ·
callstackincubator/react-native-node-api の issue をすべて見る
似ている issue
-
blocklist removal
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
MetaMask/eth-phishing-detect#296544 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
pastelsky/bundlephobia#1122 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
category/development priority/P2 scope/file-operations scope/testing type/enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
Enatega Customer and Rider app: Add-ons price is not visible to customer after order is placed. オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100