fix(android): skip explicit Kotlin plugin when AGP registers the kotlin extension

[gabrieldonadel]

Problem

Android Gradle Plugin 9 ships built-in Kotlin support and enables it by default, so
AGP registers the kotlin extension itself. When a library also applies org.jetbrains.kotlin.android
explicitly, the two collide and configuration fails before anything compiles. AGP
words it two ways, both the same problem:

> Failed to apply plugin 'org.jetbrains.kotlin.android'.
   > Cannot add extension with name 'kotlin', as there is an extension already registered with that name.

> The 'org.jetbrains.kotlin.android' plugin is no longer required for Kotlin support since AGP 9.0.

The apply is unconditional in this module, so on an AGP 9 project this cannot be built
at all. There is no consumer-side workaround short of patching the file — setting
android.builtInKotlin=false project-wide just to build one dependency is not a
reasonable ask, and that escape hatch is removed in AGP 10.

Change

Apply the plugin only when nothing has registered the kotlin extension yet:

if (project.extensions.findByName('kotlin') == null) {
    apply plugin: 'org.jetbrains.kotlin.android'
}

Files changed:

• android/build.gradle

This tests the condition that actually fails, so there is no AGP version table to
keep in sync, and it covers AGP 10 — where the android.builtInKotlin opt-out is
removed — without a special case.

AGP	android.builtInKotlin	kotlin extension	explicit apply
8.x	unset or false	absent	yes (unchanged)
9.x	unset or true	registered by AGP	no
9.x	false	absent	yes
10+	n/a (removed)	registered by AGP	no

The guard sits after apply plugin: 'com.android.library' in every file it touches,
so AGP has already registered its extensions by the time it runs. I checked that
ordering per file rather than assuming it.

What I verified, and what I did not

• Verified end to end on a real Expo SDK 58 / React Native 0.87 project with AGP
  9.2.1 and Gradle 9.4.1: :app:assembleDebug succeeds both with
  -Pandroid.newDsl=true -Pandroid.builtInKotlin=true and with both flags off.
• Confirmed both branches actually execute rather than one path always winning: with
  the flags off, compileDebugKotlin runs from the explicitly applied plugin; with
  them on the build completes without it.
• Every changed file passes a Groovy Phases.CONVERSION syntax check.
• Not run: this repo's own CI or example app.

Where this came from

A sweep of 500 popular React Native libraries against the AGP 9 defaults. 152 failed
with the new DSL enabled, and 144 of those failed on exactly this collision — by
far the most common blocker. Affects @mattermost/react-native-paste-input here.

The same guard shape was accepted in
RevenueCat/react-native-purchases#1934,
at that maintainer's suggestion.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 94c12487-d8ef-4add-a096-8456cdcea825

📥 Commits

Reviewing files that changed from the base of the PR and between 671378e and dae2479.

📒 Files selected for processing (1)

• android/build.gradle

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

---

📝 Walkthrough

Walkthrough

The Android build script now checks for an existing kotlin extension before it applies org.jetbrains.kotlin.android. This prevents duplicate Kotlin plugin application when AGP 9 already registers the extension.

Changes

Kotlin plugin compatibility

Layer / File(s)	Summary
Conditional Kotlin plugin application android/build.gradle	The script applies org.jetbrains.kotlin.android only when the kotlin extension is absent.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to dae24

The Android build now avoids reapplying Kotlin when AGP provides it while retaining existing behavior where it does not. The change is ready to merge.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely identifies the main change: skipping the explicit Kotlin plugin when AGP registers the kotlin extension.
Description check	✅ Passed	The description directly explains the AGP 9 conflict, the conditional plugin application, supported configurations, and verification performed.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}