docs(kmp): fix debug-symbols Gradle syntax to match wrapped SDK version - #45
Conversation
Both the debug-symbols page and the KMP Agent Skill instructed the
nested-block NDK syntax (bugsee { ndk { enabled.set(true) } }) and,
in the skill file, bugseeGradle = "4+". Per the Gradle plugin's own
release notes, that syntax and the 4.x plugin line are new in the
4.x/SDK-7.x generation and explicitly not backward-compatible with
SDK 6.x. The KMP SDK's own release notes say it currently wraps
native Android SDK 6.0.2, which requires the 3.x plugin and the
boolean form ndk(true) — following the old instructions as written
throws a Kotlin type-mismatch build error.
Updated both files to the 3.x plugin line and ndk(true), and added a
note pointing readers to check the KMP release notes for which native
SDK generation is wrapped before picking a plugin line, since this is
expected to change as KMP tracks newer native SDK releases.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Review follow-up: the previous revision told readers to set
bugseeGradle = "3.x.x". That pairing is not what the KMP repo actually
ships — the KMP 0.1.1 sample (composeApp/build.gradle.kts) uses plugin
4.0.0-beta4 with the boolean ndk(true) form, so "3.x" is an unverified
recommendation.
The boolean-vs-nested split is a plugin-line property, not an SDK-line
one: ndk(true) was valid on 3.x *and* on 4.0.0-beta4..beta9, and the
nested ndk { enabled.set(true) } block only landed in 4.0.0-beta10.
Keep ndk(true) (correct for the native SDK 6.0.2 that released KMP
wraps), restore the neutral "x.y.z" version placeholder, and document
both syntaxes so readers can adapt when KMP moves to the 7.x line.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Change-Id: I61747fac34e7a086231e90e496ac3ff8db1a554a
|
Validated against Confirmed: // composeApp/build.gradle.kts @ v0.1.1
bugsee {
appToken("...")
ndk(true) // upload NDK debug symbols
}The current plugin has no What I changed: the Heads-up on churn: Also confirmed editing cspell clean (288 files, 0 issues). LGTM to merge with the follow-up. 🤖 Generated with Claude Code |
krassx
left a comment
There was a problem hiding this comment.
Validated against source (see review comment above). Approving.
Summary
docs/sdk/kmp/debug-symbols.mdand the KMP Agent Skill file both instructedbugsee { ndk { enabled.set(true) } }(nested-block syntax), and the skill file additionally suggestedbugseeGradle = "4+".gradle-plugin/releases.mdx, that nested-block NDK syntax and the 4.x plugin line are both new in the 4.x/SDK-7.x generation and explicitly not backward-compatible with SDK 6.x.sdk/kmp/release-notes.md) say the current KMP release wraps native Android SDK 6.0.2, which requires the 3.x plugin and the old boolean formndk(true).inferred type is () -> Unit but Boolean was expected).Changes
ndk(true)and the 3.x plugin line.Test plan
npx cspellon both changed files — 0 issuesnpm run build— succeeds (two unrelated pre-existing warnings on/sdk/dotnet/network/and/sdk/xamarin/network/are fixed by docs(dotnet,xamarin): fix raw HTML leftover breaking network.md rendering #42, not touched here)🤖 Generated with Claude Code
https://claude.ai/code/session_01QnXDhVitCVmMNafPRvKUat