Repository navigation
docs: add the Kotlin migration playbook and tracker - #846
Conversation
📦 SDK Size Impact ReportWhat the SDK adds to a minified release APK. Measured against an empty baseline app. Unlike the Rokt kit, android-core ships no Compose and no resources, so there is nothing here that a host app would already provide. mParticle Core SDK
➡️ SDK size impact change is minimal. Raw measurementsTarget branch: {"baseline_dex_bytes": 0, "baseline_download_bytes": 2512, "baseline_install_bytes": 7528, "core_dex_bytes": 218884, "core_download_bytes": 122937, "core_install_bytes": 130190}This PR: {"baseline_dex_bytes": 0, "baseline_download_bytes": 2513, "baseline_install_bytes": 7529, "core_dex_bytes": 218884, "core_download_bytes": 122936, "core_install_bytes": 130189} |
a5d2c6f to
362c34e
Compare
PR SummaryLow Risk Overview Introduces Introduces Reviewed by Cursor Bugbot for commit 5fcb2ae. Bugbot is set up for automated code reviews on this repo. Configure here. |
362c34e to
7b3e149
Compare
7b3e149 to
4d16a95
Compare
ec5da3a to
5fcb2ae
Compare
jamesnrokt
left a comment
There was a problem hiding this comment.
One flag is we need a deeper test before any releases following this process
5fcb2ae to
c2e7df5
Compare
Binary compatibility
Compares the release AAR against the same artifact at the last version published to Maven Central; R8-renamed classes and members are excluded (see the script docstring). Full japicmp HTML/XML reports are attached as the |
jamesnrokt
left a comment
There was a problem hiding this comment.
Approving and reran checks - Happy to approve again if checks need a proper fix
Adds docs/kotlin-migration/PLAYBOOK.md, which sets out the rules for converting android-core and android-kit-base internals from Java to Kotlin without changing the published API: what stays Java and why, the compatibility tiers and the check that proves each, the conversion rules, the per-PR checklist, risk classes and merge rules, and the ProGuard protocol. Adds docs/kotlin-migration/TRACKER.md with every planned pull request by phase and stack, and points AGENTS.md at both. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
c2e7df5 to
b20a64d
Compare
Summary
Adds the playbook and tracker for the Java to Kotlin migration of
android-coreandandroid-kit-baseinternals, and pointsAGENTS.mdat them.docs/kotlin-migration/PLAYBOOK.md: the two governing rules (no breaking change;mainis always releasable), what converts and what stays Java and why, the compatibility tiers and the check that proves each, the conversion rules that keep the compiled API identical, the per-PR checklist, risk classes and merge rules, the ProGuard protocol, rollback, and the steps to add when a release is cut while migration work is unreleased.docs/kotlin-migration/TRACKER.md: every planned pull request by phase and stack, with line counts, existing-test counts and risk class, plus the QA checkpoints.AGENTS.md: a short pointer to both documents.Dependencies
The documents refer to tooling introduced by the other Phase 0 pull requests (
apiCheckandscripts/check_api_dump.py,scripts/api_compat_report.py,settings-compat.gradle). This pull request should merge after them.🤖 Generated with Claude Code