From 8e48a42d04d7a5ea9f73529878d2211fc3377b6f Mon Sep 17 00:00:00 2001
From: Bruno Garcia <bruno@brunogarcia.com>
Date: Sat, 9 Aug 2025 00:56:02 -0400
Subject: [PATCH 1/5] Fix Android binding warnings - treat as errors but
 suppress expected warnings

- Enable TreatWarningsAsErrors for Android bindings project
- Suppress expected/unavoidable Android binding warnings (BG8xxx codes)
- These warnings are caused by Java interface circular dependencies in Sentry SDK
- Warnings are documented with explanations for each type
- New warning types will now break the build for review
- Analyzed 1528 warnings and confirmed they're due to binding limitations
- Solution preserves existing Metadata.xml strategic type removals
---
 .../Sentry.Bindings.Android.csproj             | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
index b8dbd70ea6..1af51aeafe 100644
--- a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
+++ b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
@@ -6,6 +6,24 @@
     <!-- This gets resolved by the DownloadSentryAndroidSdk target -->
     <SentryNativeNdkVersion></SentryNativeNdkVersion>
     <Description>.NET Bindings for the Sentry Android SDK</Description>
+
+    <!-- Treat Android binding generator warnings as errors -->
+    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+    <WarningsAsErrors />
+    
+    <!-- Android binding warnings - these are largely unavoidable due to Java interface circular dependencies -->
+    <!-- Only suppress warnings that are confirmed to be expected/unavoidable -->
+    <!-- BG8801: Invalid parameter types - caused by circular interface dependencies in Sentry Java SDK -->
+    <!-- BG8C01: Invalid base interface - interfaces reference types removed by Metadata.xml -->
+    <!-- BG8701: Invalid return types - return types reference unresolvable interfaces -->
+    <!-- BG8800/BG8700: Unknown/Invalid types - types intentionally removed by Metadata.xml -->
+    <!-- BG8605/BG8606: Missing Java types - Compose UI types not available in our binding context -->
+    <!-- BG8604: Missing ancestor types - nested types whose parents are removed -->
+    <!-- BG8502: Interface invalidation - cascade effect from other binding issues -->
+    <!-- BG8401/BG8400: Field/property conflicts - resolved individually in Metadata.xml where needed -->
+    <NoWarn>$(NoWarn);BG8801;BG8C01;BG8701;BG8800;BG8700;BG8605;BG8606;BG8604;BG8502;BG8401;BG8400;BG8503;BG8C00</NoWarn>
+    
+    <!-- Any NEW Android binding warning types should break the build to ensure they're reviewed -->
   </PropertyGroup>
 
   <!-- Use a separate readme, and don't add the changelog to the nuget. -->

From 36f2d4c57d48b300282fa05641b1962d895d2b53 Mon Sep 17 00:00:00 2001
From: Bruno Garcia <bruno@brunogarcia.com>
Date: Sat, 9 Aug 2025 01:07:17 -0400
Subject: [PATCH 2/5] warn as errors

---
 src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
index 1af51aeafe..f07b82548c 100644
--- a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
+++ b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
@@ -7,10 +7,6 @@
     <SentryNativeNdkVersion></SentryNativeNdkVersion>
     <Description>.NET Bindings for the Sentry Android SDK</Description>
 
-    <!-- Treat Android binding generator warnings as errors -->
-    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
-    <WarningsAsErrors />
-    
     <!-- Android binding warnings - these are largely unavoidable due to Java interface circular dependencies -->
     <!-- Only suppress warnings that are confirmed to be expected/unavoidable -->
     <!-- BG8801: Invalid parameter types - caused by circular interface dependencies in Sentry Java SDK -->

From 6ccdbc5dec7e6960fbc9ac8b04aaf9e3036d6bcb Mon Sep 17 00:00:00 2001
From: Bruno Garcia <bruno@brunogarcia.com>
Date: Sat, 9 Aug 2025 13:35:02 -0400
Subject: [PATCH 3/5] remove misleading comment

---
 src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
index f07b82548c..97327c9236 100644
--- a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
+++ b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
@@ -18,8 +18,6 @@
     <!-- BG8502: Interface invalidation - cascade effect from other binding issues -->
     <!-- BG8401/BG8400: Field/property conflicts - resolved individually in Metadata.xml where needed -->
     <NoWarn>$(NoWarn);BG8801;BG8C01;BG8701;BG8800;BG8700;BG8605;BG8606;BG8604;BG8502;BG8401;BG8400;BG8503;BG8C00</NoWarn>
-    
-    <!-- Any NEW Android binding warning types should break the build to ensure they're reviewed -->
   </PropertyGroup>
 
   <!-- Use a separate readme, and don't add the changelog to the nuget. -->

From 3537c94d6db8b3ee460645916044042805a7d0b2 Mon Sep 17 00:00:00 2001
From: Bruno Garcia <bruno@brunogarcia.com>
Date: Thu, 14 Aug 2025 10:18:01 -0400
Subject: [PATCH 4/5] Update
 src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Stefan Pölz <38893694+Flash0ver@users.noreply.github.com>
---
 src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
index 97327c9236..9fcd95c1fd 100644
--- a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
+++ b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
@@ -17,6 +17,7 @@
     <!-- BG8604: Missing ancestor types - nested types whose parents are removed -->
     <!-- BG8502: Interface invalidation - cascade effect from other binding issues -->
     <!-- BG8401/BG8400: Field/property conflicts - resolved individually in Metadata.xml where needed -->
+    <!-- BG8C00: Base interface not found -->
     <NoWarn>$(NoWarn);BG8801;BG8C01;BG8701;BG8800;BG8700;BG8605;BG8606;BG8604;BG8502;BG8401;BG8400;BG8503;BG8C00</NoWarn>
   </PropertyGroup>
 

From bfc03d9024b28298eed4d7b555b3944758efa246 Mon Sep 17 00:00:00 2001
From: Bruno Garcia <bruno@brunogarcia.com>
Date: Thu, 14 Aug 2025 10:18:07 -0400
Subject: [PATCH 5/5] Update
 src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Stefan Pölz <38893694+Flash0ver@users.noreply.github.com>
---
 src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
index 9fcd95c1fd..632b8ba146 100644
--- a/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
+++ b/src/Sentry.Bindings.Android/Sentry.Bindings.Android.csproj
@@ -15,7 +15,7 @@
     <!-- BG8800/BG8700: Unknown/Invalid types - types intentionally removed by Metadata.xml -->
     <!-- BG8605/BG8606: Missing Java types - Compose UI types not available in our binding context -->
     <!-- BG8604: Missing ancestor types - nested types whose parents are removed -->
-    <!-- BG8502: Interface invalidation - cascade effect from other binding issues -->
+    <!-- BG8502/BG8503: Interface invalidation - cascade effect from other binding issues -->
     <!-- BG8401/BG8400: Field/property conflicts - resolved individually in Metadata.xml where needed -->
     <!-- BG8C00: Base interface not found -->
     <NoWarn>$(NoWarn);BG8801;BG8C01;BG8701;BG8800;BG8700;BG8605;BG8606;BG8604;BG8502;BG8401;BG8400;BG8503;BG8C00</NoWarn>