ADFA-943: Strip leftover top-level cruft from the release APK

[davidschachterADFA]

Summary

• ADFA-943: the release APK carried a bunch of leftover top-level files not present in upstream AndroidIDE's build.
• Added packaging.resources.excludes entries in app/build.gradle.kts to strip:
  • Raw .proto sources leaked by the protobuf gradle plugin's default resource bundling (project-models, aapt2-proto, profiler, jvm-symbol-models), plus protobuf's own well-known-type sources and edition feature-default descriptors — none of these are read at runtime, only by protoc/codegen.
  • Inert version-metadata properties from httpclient/httpcore, google-http-client, google-auth-library, unbescape, and the shaded OpenTelemetry API in the Kotlin compiler embeddable jar.
  • IntelliJ-platform-only descriptors (intellij.java.frontback.psi[.impl].xml) and the shaded log4j NOTICE from the Kotlin compiler embeddable jar.
  • jgit's OSGi bundle localization file (OSGI-INF/l10n/plugin.properties).
• assets/templates/xml/{drawable,layout,menu}.xml, also flagged in the ticket, turned out to be actively used (ProjectWriter.createMenu/createDrawable/createLayout, wired to the file-tree "new file" action) — left alone.

Test plan

• flox activate -d flox/local -- ./gradlew :app:assembleV8Release --parallel --max-workers=6 builds successfully.
• Unzipped the resulting APK and diffed its top-level entries before/after — every file listed in the ticket is gone; only expected entries remain (dex, manifest, resources.arsc, Kotlin/coroutines metadata, and Firebase/Play Services version metadata used by features CoGo added beyond AndroidIDE).

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes
  • Improved Android packaging to prevent build-time-only metadata and source files from being included in released APK and AAB artifacts.
  • Updated packaged resource exclusions to reduce unnecessary files while preserving required runtime resources.

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

_{Tip: disable this comment in your organization's Code Review settings.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c4d0b7b7-4650-495a-b86d-ee50999a388f

📥 Commits

Reviewing files that changed from the base of the PR and between babd11c and 94c028e.

📒 Files selected for processing (1)

• app/build.gradle.kts

---

📝 Walkthrough

Walkthrough

The Android packaging configuration revises protobuf exclusions and adds exclusions for build-time metadata from several third-party libraries while retaining the public suffix list exclusion.

Changes

Packaging exclusions

Layer / File(s)	Summary
Resource exclusion rules app/build.gradle.kts	The packaging configuration replaces protobuf directory exclusions with raw .proto and descriptor-set exclusions, and adds exclusions for HttpClient, Google HTTP client/auth, Kotlin, unbescape, IntelliJ, jgit, and related metadata files.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• appdevforall/CodeOnTheGo#1569: Updates the same packaging resource exclusion logic for protobuf resources.

Suggested reviewers: jatezzz, jomen-adfa

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main change: removing leftover packaged files from the release APK.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/ADFA-943-cleanup-apk-top-level

---

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.}