refactor(errortracking): clarify suppress log + test-stub comment

Address two of three Greptile review notes on PR #569:

1. The "skipping autocapture" log message previously read as if the
   throwable's outermost class was always the matched ignoredType. A
   cause-chain match would report e.g. `java.lang.RuntimeException` as
   suppressed even when only `JavascriptException` was in the ignore
   list. Reworded to surface that "the throwable or a cause in its
   chain" matched.

2. The `ReactNativeJavascriptExceptionStub` doc comment claimed it was
   "named the same way" as the real RN class — it isn't; the stub's
   JVM name lives under the test class. Restated the actual design:
   the stub's own FQCN is what tests register as the ignored value,
   exercising the same code path without needing RN on the classpath.

Greptile's third note (parameterising the new tests) is intentionally
deferred — the individual test names ("skips capture when…", "still
captures when…", etc.) document distinct behaviours and read more
clearly than a `@ParameterizedTest` table would. Happy to revisit if
the maintainer prefers the table form.

Author: tsushanth

posthog/src/main/java/com/posthog/errortracking/PostHogErrorTrackingAutoCaptureIntegration.kt

@@ -86,8 +86,14 @@ public class PostHogErrorTrackingAutoCaptureIntegration : PostHogIntegration, Th
                 postHog.captureException(PostHogThrowable(throwable, thread))
                 postHog.flush()
             } else {
+                // The match may have been against a cause anywhere in the chain,
+                // not necessarily the outermost throwable, so we deliberately
+                // don't pin the log to `throwable.javaClass.name` — that would
+                // mislead anyone looking at the logs and seeing e.g. a bare
+                // RuntimeException reported as suppressed when their ignore
+                // list only mentions com.facebook.react.common.JavascriptException.
                 config.logger.log(
-                    "Skipping autocapture for ignored exception type: ${throwable.javaClass.name}",
+                    "Skipping autocapture: ${throwable.javaClass.name} (or a cause in its chain) matches ignoredExceptionTypes",
                 )
             }
         }

posthog/src/test/java/com/posthog/errortracking/PostHogErrorTrackingAutoCaptureIntegrationTest.kt

@@ -228,8 +228,12 @@ internal class PostHogErrorTrackingAutoCaptureIntegrationTest {
     /**
      * Local stand-in for `com.facebook.react.common.JavascriptException`. The real type
      * lives in React Native, which isn't (and shouldn't be) a test dependency of the
-     * SDK. The ignored-exception filter is purely class-name based, so a local class
-     * named the same way exercises the same code path without dragging in RN.
+     * SDK. The ignored-exception filter is purely class-name based: each test that
+     * uses this stub registers its own JVM name
+     * (`...PostHogErrorTrackingAutoCaptureIntegrationTest$ReactNativeJavascriptExceptionStub`)
+     * as the ignored FQCN. That exercises the matching logic without requiring RN on
+     * the classpath. The stub name intentionally differs from the real RN class name —
+     * the production code path is identical either way.
      */
     private class ReactNativeJavascriptExceptionStub(message: String) : RuntimeException(message)