chore(extend-app-start): Remove redundant comments and rename reset to clear

Drop comments that restate code or annotate omissions (region markers, getter
javadoc, the reset call-site comment, the ExtendedAppStart/isActive docs). Rename
AppStartExtension.reset() to clear() to match its owner AppStartMetrics.clear().

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

Author: buenaflor

sentry-android-core/api/sentry-android-core.api

@@ -186,13 +186,13 @@ public final class io/sentry/android/core/AppLifecycleIntegration : io/sentry/In
 
 public final class io/sentry/android/core/AppStartExtension : io/sentry/IAppStartExtender {
 	public fun <init> (Lio/sentry/android/core/performance/AppStartMetrics;)V
+	public fun clear ()V
 	public fun extendAppStart ()V
 	public fun finishAppStart ()V
 	public fun finishTransaction (Lio/sentry/SentryDate;)V
 	public fun getExtendedAppStartSpan ()Lio/sentry/ISpan;
 	public fun getExtendedEndTime ()Lio/sentry/SentryDate;
 	public fun isActive ()Z
-	public fun reset ()V
 	public fun setExtendAppStartListener (Lio/sentry/android/core/AppStartExtension$ExtendAppStartListener;)V
 }
 

sentry-android-core/src/main/java/io/sentry/android/core/AppStartExtension.java

@@ -29,9 +29,6 @@
 @ApiStatus.Internal
 public final class AppStartExtension implements IAppStartExtender {
 
-  /**
-   * The standalone App Start transaction and its extended child span, created by the integration.
-   */
   public static final class ExtendedAppStart {
     public final @NotNull ITransaction transaction;
     public final @NotNull ISpan span;
@@ -125,7 +122,6 @@ public void finishAppStart() {
     }
   }
 
-  /** Whether an eagerly-created extension transaction exists and has not finished yet. */
   public boolean isActive() {
     try (final @NotNull ISentryLifecycleToken ignored = lock.acquire()) {
       return extendedTransaction != null && !extendedTransaction.isFinished();
@@ -164,12 +160,7 @@ public void finishTransaction(final @NotNull SentryDate endTimestamp) {
     }
   }
 
-  /**
-   * Resets the per-start state so a stale extension can't affect a later (e.g. warm) app start. The
-   * registered listener is intentionally kept: it is registered once at SDK init and must survive
-   * across app starts.
-   */
-  public void reset() {
+  public void clear() {
     try (final @NotNull ISentryLifecycleToken ignored = lock.acquire()) {
       extendedSpan = null;
       extendedTransaction = null;

sentry-android-core/src/main/java/io/sentry/android/core/performance/AppStartMetrics.java

@@ -283,9 +283,7 @@ public void onAppStartSpansSent() {
     shouldSendStartMeasurements = false;
     contentProviderOnCreates.clear();
     activityLifecycles.clear();
-    // Reset extension state so a stale extended span/txn can't affect a later (e.g. warm) app
-    // start.
-    appStartExtension.reset();
+    appStartExtension.clear();
   }
 
   public boolean shouldSendStartMeasurements(final boolean ignoreForegroundCheck) {
@@ -341,9 +339,6 @@ public long getClassLoadedUptimeMs() {
     return new TimeSpan();
   }
 
-  // region app start extension
-
-  /** The focused component that owns the "extend app start" lifecycle. */
   public @NotNull AppStartExtension getAppStartExtension() {
     return appStartExtension;
   }
@@ -359,8 +354,6 @@ public boolean isAppStartWindowOpen() {
         && !firstDrawDone.get();
   }
 
-  // endregion
-
   @TestOnly
   void setFirstIdle(final long firstIdle) {
     this.firstIdle = firstIdle;
@@ -402,7 +395,7 @@ public void clear() {
     appStartBaggageHeader = null;
     appStartEndTime = null;
     cachedStartInfo = null;
-    appStartExtension.reset();
+    appStartExtension.clear();
   }
 
   public @Nullable ITransactionProfiler getAppStartProfiler() {

sentry-android-core/src/test/java/io/sentry/android/core/AppStartExtensionTest.kt

@@ -196,12 +196,12 @@ class AppStartExtensionTest {
   }
 
   @Test
-  fun `reset clears the extension state`() {
+  fun `clear clears the extension state`() {
     val ext = extension(windowOpen = true)
     ext.registerHandOver()
     ext.extendAppStart()
     assertTrue(ext.isActive)
-    ext.reset()
+    ext.clear()
     assertFalse(ext.isActive)
     assertSame(NoOpSpan.getInstance(), ext.extendedAppStartSpan)
   }