fix(e2e): Address PR review feedback

- Use nullish coalescing for httpSpans length check to avoid TypeError
  when spans is undefined
- Document maestro retry envelope contamination limitation

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: antonis

samples/react-native/e2e/tests/captureSpaceflightNewsScreenTransaction/captureSpaceflightNewsScreenTransaction.test.ts

@@ -143,6 +143,6 @@ describe('Capture Spaceflight News Screen Transaction', () => {
     const httpSpans = spans?.filter(
       span => span.data?.['sentry.op'] === 'http.client',
     );
-    expect(httpSpans!.length).toBeGreaterThanOrEqual(1);
+    expect(httpSpans?.length ?? 0).toBeGreaterThanOrEqual(1);
   });
 });

samples/react-native/e2e/utils/maestro.ts

@@ -25,6 +25,11 @@ const runMaestro = (test: string): Promise<void> => {
 /**
  * Run a Maestro test with retries to handle transient app crashes on slow CI VMs.
  *
+ * Note: Retries happen at the Maestro flow level. If a failed attempt sends partial
+ * envelopes to the mock server before crashing, they will accumulate across retries.
+ * In practice, crashes occur on app launch before any SDK transactions are sent,
+ * so this does not cause issues with test assertions.
+ *
  * @param test - The path to the Maestro test file relative to the `e2e` directory.
  * @returns A promise that resolves when the test passes.
  */