fix(proguard): disable shrinking for Compose animation classes (#5116)

app/proguard-rules.pro

@@ -48,14 +48,8 @@
 # curves, transition specs, Animatable internals) which can cause animations to
 # silently snap in release builds.
 #
-# -keep prevents class merging (EnterTransition/ExitTransition into *Impl,
-#   VectorizedSpringSpec/TweenSpec elimination, etc.).
-# allowshrinking lets R8 remove genuinely unreachable classes (e.g.
-#   SharedTransition APIs, RepeatableSpec — unused by this app). Verified via
-#   dex analysis: 278 classes survive in release vs 139 without this rule;
-#   all actively used classes (AnimatedVisibility, Crossfade, SpringSpec,
-#   TweenSpec, EnterTransition, ExitTransition, etc.) are preserved.
-# allowobfuscation is moot (-dontobfuscate is set above) but explicit for
-#   clarity.
-# The ** wildcard is recursive and covers animation.core.* sub-packages.
--keep,allowshrinking,allowobfuscation class androidx.compose.animation.** { *; }
+# We use a full -keep here without allowshrinking/allowobfuscation. While it
+# might keep some unused transition APIs, R8's aggressive shrinking is known
+# to incorrectly remove internal states or merging empty transitions (like None)
+# causing AnimatedVisibility and others to snap.
+-keep class androidx.compose.animation.** { *; }

desktop/README.md

@@ -32,7 +32,7 @@ Release builds use ProGuard for tree-shaking (unused code removal), significantl
 - `proguard-rules.pro` — Keep-rules for reflection/JNI-sensitive dependencies (Koin, kotlinx-serialization, Wire protobuf, Room KMP `androidx.room3`, Ktor, Kable BLE, Coil, SQLite JNI, Compose Multiplatform resources) and an anti-merge rule for Compose animation classes.
 
 **Key rules:**
-- **Compose animation anti-merge** (`-keep,allowshrinking,allowobfuscation class androidx.compose.animation.** { *; }`) — Prevents ProGuard's optimizer from merging animation class hierarchies (e.g. `EnterTransition`/`ExitTransition` into `*Impl`), which causes animations to silently snap. Same rule as Android.
+- **Compose animation anti-merge** (`-keep class androidx.compose.animation.** { *; }`) — Prevents ProGuard's optimizer from incorrectly tree-shaking or merging animation class hierarchies (e.g. `EnterTransition`/`ExitTransition` into `*Impl`), which causes animations to silently snap. Same rule as Android.
 - **Room KMP** — Uses `androidx.room3` package path (Room KMP 3.x).
 
 **Troubleshooting ProGuard issues:**

desktop/proguard-rules.pro

@@ -150,10 +150,9 @@
 # ---- Compose Animation (anti-merge) ----------------------------------------
 
 # Prevent ProGuard from merging animation spec class hierarchies (same issue
-# as R8 on Android — EnterTransition/ExitTransition merged into *Impl,
-# VectorizedSpringSpec/TweenSpec eliminated). allowshrinking lets ProGuard
-# remove genuinely unreachable classes.
--keep,allowshrinking,allowobfuscation class androidx.compose.animation.** { *; }
+# as R8 on Android). We use a full keep to prevent incorrect tree-shaking
+# of internal transitions.
+-keep class androidx.compose.animation.** { *; }
 
 # ---- AboutLibraries ---------------------------------------------------------