Skip to content

ADFA-3604: Fix 2 more R8 shrink bugs (Gson templates, JDI debugger) #1610

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
davidschachterADFA merged 3 commits into from
Aug 1, 2026
+25 −7
Merged

ADFA-3604: Fix 2 more R8 shrink bugs (Gson templates, JDI debugger) #1610

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d8b7f3f
ADFA-3604: Fix Gson model classes stripped by R8 (6th shrink bug)
davidschachterADFA Aug 1, 2026
74e2752
ADFA-3604: Fix JDI debugger connector stripped by R8 (7th shrink bug)
davidschachterADFA Aug 1, 2026
b4e28fe
ADFA-3604: Fix comment positional reference in proguard-rules.pro
davidschachterADFA Aug 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 25 additions & 7 deletions app/proguard-rules.pro
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,10 +112,12 @@
}
-keep class com.itsaky.androidide.utils.DialogUtils { public <methods>; }

# APK Metadata
-keep class com.itsaky.androidide.models.ApkMetadata { *; }
-keep class com.itsaky.androidide.models.ArtifactType { *; }
-keep class com.itsaky.androidide.models.MetadataElement { *; }
# Gson model classes deserialized only via reflection (gson.fromJson(...,
# X::class.java)), same "R8 strips the unreachable constructor" issue as
# templates.impl.zip below. Covers OpenedFilesCache/OpenedFile (no prior
# rule) as well as the APK metadata classes already listed individually.
-keep class com.itsaky.androidide.models.** { *; }
-keep class com.itsaky.androidide.lsp.debug.model.** { *; }

# Parcelable
-keepclassmembers class * implements android.os.Parcelable {
Expand Down Expand Up @@ -164,6 +166,13 @@
-keep class * implements com.google.gson.JsonSerializer
-keep class * implements com.google.gson.JsonDeserializer

# Gson model classes: nothing calls `new TemplatesIndex(...)` directly --
# only gson.fromJson(..., TemplatesIndex::class.java) does, via reflection.
# With no traceable constructor call, R8 strips the constructor and Gson's
# runtime then reports the class as abstract ("Failed to load template
# archive ... Abstract classes can't be instantiated!").
-keep class com.itsaky.androidide.templates.impl.zip.** { *; }

-keepclassmembers,allowobfuscation class * {
@com.google.gson.annotations.SerializedName <fields>;
}
Expand Down Expand Up @@ -197,9 +206,18 @@
-keep class com.itsaky.androidide.plugins.** { *; }
-keep interface com.itsaky.androidide.plugins.** { *; }

## Initial rules to enable when R8 is shrinking to address exceptions
#-keep class com.sun.tools.jdi.** { *; }
#-keep class com.sun.jdi.** { *; }
## ADFA-3604: JDI's SocketAttachingConnector/SocketListeningConnector are
## loaded via ServiceLoader (META-INF/services), which R8 can't trace, so it
## stripped their no-arg constructors. This surfaced as
## "ServiceConfigurationError: Provider ... could not be instantiated" ->
## "java.lang.Error: no Connectors loaded" from VirtualMachineManagerImpl,
## which the app then reports to the user as a generic "Network access
## error" (the debug-connect failure handler always appends a network
## suggestion regardless of cause) even though this has nothing to do with
## network permissions. This is exactly the exceptions these rules were
## anticipating -- enabling them now that shrinking is genuinely on.
-keep class com.sun.tools.jdi.** { *; }
-keep class com.sun.jdi.** { *; }

## R8 Kotlin metadata workaround for Kotlin 2.3.0 compatibility
## Suppresses D8 errors when parsing kotlin metadata for StopWatch inline functions
Expand Down
Loading