Make PapaSafeTrace isTraceable configurable

Change PapaSafeTrace from delegating to SafeTrace.isTraceable to
accepting isTraceable as a constructor parameter. This gives clients
direct control over whether tracing is enabled, removing the implicit
dependency on SafeTrace.isTraceable, which will be removed soon.

Update WorkflowPapaTracer's default to explicitly pass isTraceable=false to
avoid overhead. Add tests to verify the new configuration capability for both
PapaSafeTrace and WorkflowPapaTracer.

Author: tcmulcahy

workflow-tracing-papa/api/workflow-tracing-papa.api

@@ -1,5 +1,7 @@
 public final class com/squareup/workflow1/tracing/papa/PapaSafeTrace : com/squareup/workflow1/tracing/SafeTraceInterface {
 	public fun <init> ()V
+	public fun <init> (Z)V
+	public synthetic fun <init> (ZILkotlin/jvm/internal/DefaultConstructorMarker;)V
 	public fun beginAsyncSection (Ljava/lang/String;I)V
 	public fun beginSection (Ljava/lang/String;)V
 	public fun endAsyncSection (Ljava/lang/String;I)V

workflow-tracing-papa/src/main/java/com/squareup/workflow1/tracing/papa/PapaSafeTrace.kt

@@ -5,10 +5,13 @@ import papa.SafeTrace
 
 /**
  * Production implementation of [SafeTraceInterface] that delegates to the actual [SafeTrace].
+ *
+ * @param isTraceable Whether tracing is enabled. Clients should configure this directly.
+ *   Defaults to false for backwards compatibility.
  */
-class PapaSafeTrace : SafeTraceInterface {
-  override val isTraceable: Boolean
-    get() = SafeTrace.isTraceable
+class PapaSafeTrace(
+  override val isTraceable: Boolean = false
+) : SafeTraceInterface {
 
   override val isCurrentlyTracing: Boolean
     get() = SafeTrace.isCurrentlyTracing

workflow-tracing-papa/src/main/java/com/squareup/workflow1/tracing/papa/WorkflowPapaTracer.kt

@@ -30,9 +30,11 @@ import kotlin.reflect.KType
  * [WorkflowRuntimeTracer] plugin to add [SafeTraceInterface] traces.
  * By default this uses [PapaSafeTrace] which will use [androidx.tracing.Trace] calls that
  * will be received by the system and included in Perfetto traces.
+ *
+ * @param safeTrace The [SafeTraceInterface] implementation to use for tracing.
  */
 class WorkflowPapaTracer(
-  private val safeTrace: SafeTraceInterface = PapaSafeTrace()
+  private val safeTrace: SafeTraceInterface = PapaSafeTrace(isTraceable = false)
 ) : WorkflowRuntimeTracer() {
 
   private data class NameAndCookie(

workflow-tracing-papa/src/test/java/com/squareup/workflow1/tracing/papa/WorkflowPapaTracerTest.kt

@@ -102,6 +102,22 @@ internal class WorkflowPapaTracerTest {
     assertNotNull(papaTracer)
   }
 
+  @Test
+  fun `PapaSafeTrace can be configured with isTraceable`() {
+    val traceableTrace = PapaSafeTrace(isTraceable = true)
+    assertEquals(true, traceableTrace.isTraceable)
+
+    val nonTraceableTrace = PapaSafeTrace(isTraceable = false)
+    assertEquals(false, nonTraceableTrace.isTraceable)
+  }
+
+  @Test
+  fun `WorkflowPapaTracer can be configured with custom SafeTrace`() {
+    val customTrace = FakeSafeTrace(isTraceable = true)
+    val tracer = WorkflowPapaTracer(safeTrace = customTrace)
+    assertNotNull(tracer)
+  }
+
   @Test
   fun `onPropsChanged delegates to proceed function`() {
     val testWorkflow = TestWorkflow()