Skip to content

refactor: Add keep rules for java types#949

Merged
tonidero merged 3 commits into
mainfrom
add-keep-rules-java-types
Jun 11, 2026
Merged

refactor: Add keep rules for java types#949
tonidero merged 3 commits into
mainfrom
add-keep-rules-java-types

Conversation

@tonidero

@tonidero tonidero commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

We're working to narrow down our consumer rules in RevenueCat/purchases-android#3557. However, this makes it not find our code anymore in Unity when minifying since nothing is directly referencing the Java types from Unity. This adds some Keep annotations that from my tests avoid the issue. This does mean we will need to remember to mark all java types as @Keep but it was easier than trying to add proguard rules to be used by our customers when importing our plugins.

Note this won't do anything until the purchases-android changes actually get merged

@tonidero tonidero added the pr:other Changes to our CI configuration files and scripts label Jun 10, 2026
@tonidero
tonidero marked this pull request as ready for review June 10, 2026 11:43
@tonidero
tonidero requested a review from a team as a code owner June 10, 2026 11:43

@ajpallares ajpallares left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

This does mean we will need to remember to mark all java types as @keep

Is there any way we can enforce this in our PRs?

@tonidero

Copy link
Copy Markdown
Contributor Author

Added a check with a bash script in a separate PR: #953

@tonidero
tonidero enabled auto-merge (squash) June 11, 2026 10:41
@tonidero
tonidero merged commit 35dde0b into main Jun 11, 2026
10 of 11 checks passed
@tonidero
tonidero deleted the add-keep-rules-java-types branch June 11, 2026 10:54
RCGitBot added a commit that referenced this pull request Jun 18, 2026
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* Add presentedOfferingContext support to custom paywall impression
events (#959) via Rick (@rickvdl)
* feat: Add SetAppsFlyerConversionData (#956) via Toni Rico (@tonidero)
### 📦 Dependency Updates
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 18.15.1 (#958) via
RevenueCat Git Bot (@RCGitBot)
* [Android
10.9.1](https://github.com/RevenueCat/purchases-android/releases/tag/10.9.1)
* [Android
10.9.0](https://github.com/RevenueCat/purchases-android/releases/tag/10.9.0)
* [iOS
5.78.0](https://github.com/RevenueCat/purchases-ios/releases/tag/5.78.0)
* [iOS
5.77.0](https://github.com/RevenueCat/purchases-ios/releases/tag/5.77.0)
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 18.15.0 (#955) via
RevenueCat Git Bot (@RCGitBot)
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 18.14.1 (#954) via
RevenueCat Git Bot (@RCGitBot)

### 🔄 Other Changes
* chore: Add 3-day Dependabot cooldown, excluding fastlane plugin (#961)
via Toni Rico (@tonidero)
* Bump fastlane-plugin-revenuecat_internal from `ce6a7ef` to `70bf5c7`
(#957) via dependabot[bot] (@dependabot[bot])
* ci: Enforce @keep on public Android bridge types (#953) via Toni Rico
(@tonidero)
* refactor: Add keep rules for java types (#949) via Toni Rico
(@tonidero)
* Bump fastlane from 2.236.0 to 2.236.1 (#951) via dependabot[bot]
(@dependabot[bot])

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Routine release PR, but it pins newer native billing SDKs (iOS/Android
via hybrid-common), which can affect purchase and attribution behavior
in production builds.
> 
> **Overview**
> Automatic **9.3.0** release: bumps Unity SDK version everywhere it is
reported (`.version`, `RevenueCat` / `RevenueCatUI` `package.json`,
Android `PLUGIN_VERSION`, iOS `platformFlavorVersion`) and records
**9.3.0** in `CHANGELOG.md`, `CHANGELOG.latest.md`, and `VERSIONS.md`.
> 
> The release notes capture what ships with this tag:
**purchases-hybrid-common 18.15.1** (iOS **5.78.0**, Android
**10.9.1**), plus recently merged SDK
features—**`presentedOfferingContext` on custom paywall impression
events** and **`SetAppsFlyerConversionData`**—along with repo
maintenance (Dependabot cooldown, Android `@Keep` / ProGuard keep rules,
fastlane bumps). No additional runtime code changes appear in this diff
beyond the version strings.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
234e38b. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:other Changes to our CI configuration files and scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants