fix(snapshot): Fix compile errors in generated sidecar metadata code

- Remove SentrySnapshotHandler that used java.awt.image.BufferedImage
  and javax.imageio.ImageIO which are unavailable in Android test
  classpath
- Use SnapshotVerifier pointed at sentry output dir instead, which
  writes images with predictable names (not content hashes)
- Fix methodName access: use preview.methodName (ComposablePreview)
  instead of info.methodName (AndroidPreviewInfo doesn't have it)
- Fix declaringClass: it's a non-nullable String on ComposablePreview
- Write sidecar filenames matching SnapshotVerifier's naming pattern
  (Paparazzi_Preview_Test_{screenshotId_lowercased})
- Clean sentry output dir before each test run to ensure fresh writes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: runningcode

plugin-build/src/main/kotlin/io/sentry/android/gradle/snapshot/GenerateSnapshotTestsTask.kt

@@ -99,7 +99,6 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.unit.dp
 import app.cash.paparazzi.DeviceConfig
-import app.cash.paparazzi.HtmlReportWriter
 import app.cash.paparazzi.Paparazzi
 import app.cash.paparazzi.Snapshot
 import app.cash.paparazzi.SnapshotHandler
@@ -108,9 +107,8 @@ import app.cash.paparazzi.TestName
 import app.cash.paparazzi.detectEnvironment
 import com.android.ide.common.rendering.api.SessionParams
 import com.android.resources.*
-import java.awt.image.BufferedImage
 import java.io.File
-import javax.imageio.ImageIO
+import java.util.Locale
 import kotlin.math.ceil
 import org.junit.Rule
 import org.junit.Test
@@ -208,29 +206,6 @@ private class TestNameOverrideHandler(
     }
 }
 
-private class SentrySnapshotHandler(
-    private val outputDir: File,
-    private val delegate: SnapshotHandler,
-) : SnapshotHandler {
-    override fun newFrameHandler(
-        snapshot: Snapshot,
-        frameCount: Int,
-        fps: Int,
-    ): SnapshotHandler.FrameHandler {
-        val delegateHandler = delegate.newFrameHandler(snapshot, frameCount, fps)
-        return object : SnapshotHandler.FrameHandler {
-            override fun handle(image: BufferedImage) {
-                delegateHandler.handle(image)
-                val name = snapshot.name ?: return
-                outputDir.mkdirs()
-                ImageIO.write(image, "png", File(outputDir, "${'$'}{name}.png"))
-            }
-            override fun close() = delegateHandler.close()
-        }
-    }
-    override fun close() = delegate.close()
-}
-
 private object PaparazziPreviewRule {
     const val UNDEFINED_API_LEVEL = -1
     const val MAX_API_LEVEL = 36
@@ -243,7 +218,7 @@ private object PaparazziPreviewRule {
         }
         val tolerance = 0.0
         val sentryOutputDir = File(
-            System.getProperty("sentry.snapshot.output", "build/sentry-snapshots/images")
+            System.getProperty("sentry.snapshot.output", "build/sentry-snapshots")
         )
         return Paparazzi(
             environment = detectEnvironment().copy(compileSdkVersion = previewApiLevel),
@@ -255,14 +230,11 @@ private object PaparazziPreviewRule {
                 previewInfo.widthDp > 0 && previewInfo.heightDp > 0 -> SessionParams.RenderingMode.FULL_EXPAND
                 else -> SessionParams.RenderingMode.SHRINK
             },
-            snapshotHandler = SentrySnapshotHandler(
-                outputDir = sentryOutputDir,
-                delegate = TestNameOverrideHandler(
-                    when (System.getProperty("paparazzi.test.verify")?.toBoolean() == true) {
-                        true -> SnapshotVerifier(maxPercentDifference = tolerance)
-                        false -> HtmlReportWriter(maxPercentDifference = tolerance)
-                    }
-                ),
+            snapshotHandler = TestNameOverrideHandler(
+                SnapshotVerifier(
+                    maxPercentDifference = tolerance,
+                    rootDirectory = sentryOutputDir,
+                )
             ),
             maxPercentDifference = tolerance,
         )
@@ -368,29 +340,26 @@ class $CLASS_NAME(
         screenshotId: String,
         preview: ComposablePreview<AndroidPreviewInfo>,
     ) {
-        val outputDir = File(
-            System.getProperty("sentry.snapshot.output", "build/sentry-snapshots/images")
+        val sentryRoot = File(
+            System.getProperty("sentry.snapshot.output", "build/sentry-snapshots")
         )
-        outputDir.mkdirs()
+        val imagesDir = File(sentryRoot, "images")
+        imagesDir.mkdirs()
         val info = preview.previewInfo
         val q = '"'
         val parts = mutableListOf<String>()
         parts.add(q + "display_name" + q + ": " + q + escapeJson(screenshotId) + q)
         parts.add(q + "image_file_name" + q + ": " + q + escapeJson(screenshotId) + q)
         val group = info.group
-        if (!group.isNullOrBlank()) {
+        if (group.isNotBlank()) {
             parts.add(q + "group" + q + ": " + q + escapeJson(group) + q)
         }
-        val declaringClass = preview.declaringClass
-        if (declaringClass != null) {
-            parts.add(q + "className" + q + ": " + q + escapeJson(declaringClass) + q)
-        }
-        val methodName = info.methodName
-        if (methodName != null) {
-            parts.add(q + "methodName" + q + ": " + q + escapeJson(methodName) + q)
-        }
+        parts.add(q + "className" + q + ": " + q + escapeJson(preview.declaringClass) + q)
+        parts.add(q + "methodName" + q + ": " + q + escapeJson(preview.methodName) + q)
         val json = parts.joinToString(",\n  ", prefix = "{\n  ", postfix = "\n}")
-        File(outputDir, "${'$'}{screenshotId}.json").writeText(json)
+        val sidecarName = "Paparazzi_Preview_Test_" +
+            screenshotId.lowercase(Locale.US).replace("\\s".toRegex(), "_")
+        File(imagesDir, "${'$'}{sidecarName}.json").writeText(json)
     }
 
     private fun escapeJson(s: String): String =

plugin-build/src/main/kotlin/io/sentry/android/gradle/snapshot/SentrySnapshotPlugin.kt

@@ -4,6 +4,7 @@ import com.android.build.api.variant.ApplicationAndroidComponentsExtension
 import com.android.build.api.variant.HostTestBuilder
 import com.android.build.gradle.BaseExtension
 import io.sentry.android.gradle.util.AgpVersions
+import java.io.File
 import kotlin.jvm.java
 import org.gradle.api.Plugin
 import org.gradle.api.Project
@@ -30,12 +31,18 @@ class SentrySnapshotPlugin : Plugin<Project> {
       val androidComponents =
         project.extensions.getByType(ApplicationAndroidComponentsExtension::class.java)
 
-      val sentrySnapshotOutputDir = project.layout.buildDirectory.dir("sentry-snapshots/images")
+      val sentrySnapshotRootDir = project.layout.buildDirectory.dir("sentry-snapshots")
       project.tasks.withType(Test::class.java).configureEach { testTask ->
         testTask.systemProperty(
           "sentry.snapshot.output",
-          sentrySnapshotOutputDir.get().asFile.absolutePath,
+          sentrySnapshotRootDir.get().asFile.absolutePath,
         )
+        testTask.doFirst {
+          val imagesDir = File(sentrySnapshotRootDir.get().asFile, "images")
+          if (imagesDir.exists()) {
+            imagesDir.deleteRecursively()
+          }
+        }
       }
 
       androidComponents.onVariants { variant ->