Skip to content

test: Verify Size Analysis threshold (do not merge) - #6895

Closed
antonis wants to merge 1 commit into
mainfrom
antonis/size-analysis-threshold-test
Closed

antonis wants to merge 1 commit into
mainfrom
antonis/size-analysis-threshold-test

Conversation

@antonis

@antonis antonis commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Warning

Do not merge. Throwaway PR to verify the Sentry Size Analysis threshold config.

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Adds a ~480 KB incompressible noise PNG (samples/react-native/src/assets/size-analysis-test.png) and renders it in the sample app's Playground tab, so Metro bundles it into both the iOS and Android release builds uploaded by size-analysis.yml.

💡 Motivation and Context

Size Analysis status check and PR comment rules (absolute diff > 50 KB) were configured for the sentry-react-native project. This PR intentionally exceeds the threshold to confirm:

  • the Size Analysis check from Sentry fails (instead of neutral/success), and
  • a size PR comment is posted.

This is a prerequisite for replacing the binarySizeTest in the metrics job (#6890).

💚 How did you test it?

CI only — the ready-to-merge label triggers the Size Analysis workflow.

📝 Checklist

  • I added tests to verify changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.
  • No breaking changes.

🔮 Next steps

Close without merging once the check result is verified.

🤖 Generated with Claude Code

Do not merge. Adds a ~480 KB incompressible PNG to the RN sample app to
check that the Sentry Size Analysis status check fails and posts a PR
comment when the 50 KB threshold is exceeded.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@antonis antonis added the ready-to-merge Triggers the full CI test suite label Oct 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


  • test: Verify Size Analysis threshold (do not merge) by antonis in #6895

🤖 This preview updates automatically when you update the PR.

1 similar comment
@github-actions

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


  • test: Verify Size Analysis threshold (do not merge) by antonis in #6895

🤖 This preview updates automatically when you update the PR.

@sentry

sentry Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Size Analysis

❌ 4 Failed Size Checks

iOS Builds

Name Configuration Version Download Size Install Size
sentryreactnativesample (iOS)
io.sentry.reactnative.sample
Release 8.31.0 (116) 11.7 MB (+480.9 KB) 34.8 MB (+487.4 KB)

Android Builds

Name Configuration Version Download Size Uncompressed Size
Sentry RN (Android)
io.sentry.reactnative.sample
Release 8.31.0 (111) 15.1 MB (+480.5 KB) 40.9 MB (+481.0 KB)
4 Failed Checks

io.sentry.reactnative.sample | Release (iOS)

  • Install Size (Absolute Diff) > 50.0 KB — Main App
  • Download Size (Absolute Diff) > 50.0 KB — Main App
    io.sentry.reactnative.sample | Release (Android)
  • Install Size (Absolute Diff) > 50.0 KB — Main App
  • Download Size (Absolute Diff) > 50.0 KB — Main App

Configure sentry-react-native status check rules

@sentry

sentry Bot commented Oct 15, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
Sentry RN io.sentry.reactnative.sample 8.31.0 (111) Release

⚙️ sentry-react-native Build Distribution Settings

@antonis

antonis commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

Verification done: Size Analysis check failed (4 rules > 50 KB) and posted a PR comment as expected. Closing without merging.

@antonis antonis closed this Oct 15, 2026
@antonis
antonis deleted the antonis/size-analysis-threshold-test branch October 15, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Triggers the full CI test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant