CV - Switch to the Unbundled ML Kit

[avestaadfa]

No description provided.

[coderabbitai]

📝 Walkthrough

Release Notes - CV: Switch to the Unbundled ML Kit

Changes

• Updated ML Kit text recognition dependency from 16.0.0 to 16.0.0-beta6
• Added ML Kit dependency metadata declaration in AndroidManifest.xml for OCR module support

Risks & Best Practices Considerations

• ⚠️Beta Dependency Risk: Migrating to a beta version (16.0.0-beta6) of ML Kit may introduce instability or breaking changes in future updates. Consider planning for additional testing and potential future migration costs when this moves to a stable release.
• Unbundled ML Kit approach reduces APK size by allowing on-demand delivery, improving app performance
• Ensure the OCR meta-data declaration is correctly recognized by the Google Play system for proper dependency resolution

Recommendations

• Thoroughly test OCR functionality across various device configurations and Android API levels
• Monitor for official stable release of this ML Kit version and plan migration accordingly
• Verify that users without OCR dependency pre-installed can successfully request and install it through Google Play services

Walkthrough

Updated the ML Kit text-recognition dependency from version 16.0.0 to 16.0.0-beta6 and added a meta-data declaration in the Android manifest to specify ML Kit vision OCR dependencies.

Changes

Cohort / File(s)	Summary
Dependency Management cv-image-to-xml/build.gradle.kts	Updated ML Kit text-recognition dependency to 16.0.0-beta6.
Manifest Configuration cv-image-to-xml/src/main/AndroidManifest.xml	Added meta-data entry declaring ML Kit vision OCR dependencies within the application tag.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• jatezzz

Poem

🐰 With whiskers twitching and eyes so keen,
We've upgraded the text we've seen!
Beta six brings vision bright,
And metadata shines with light—
OCR now hops with delight! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to assess whether any description relates to the changeset.	Add a pull request description explaining the rationale for switching to the Unbundled ML Kit and any migration steps or breaking changes involved.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: switching to the Unbundled ML Kit, which aligns with the dependency update from standard to beta version and the manifest configuration change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/ADFA-2823-CV-Switch-to-the-Unbundled-MLKit-experimental

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@cv-image-to-xml/build.gradle.kts`:
- Line 45: Replace the outdated dependency declaration
implementation("com.google.mlkit:text-recognition:16.0.0-beta6") with the
current stable ML Kit artifact or the Play Services-backed artifact: either
upgrade to implementation("com.google.mlkit:text-recognition:16.0.1") or, if you
require the Play services-backed model, replace it with
implementation("com.google.android.gms:play-services-mlkit-text-recognition:19.0.1");
update the build.gradle.kts dependency entry accordingly and verify the project
sync/build.