Skip to content

Move endroid/qr-code to require and exclude from app bundles#90

Merged
shanerbaner82 merged 1 commit intomainfrom
fix/exclude-qr-code-from-bundle
Apr 10, 2026
Merged

Move endroid/qr-code to require and exclude from app bundles#90
shanerbaner82 merged 1 commit intomainfrom
fix/exclude-qr-code-from-bundle

Conversation

@shanerbaner82
Copy link
Copy Markdown
Contributor

@shanerbaner82 shanerbaner82 commented Apr 10, 2026

Summary

  • Moves endroid/qr-code from require-dev to require (needed at runtime by Jump dev server for QR code generation)
  • Upgrades to v6 (^6.1.3) and updates Builder API usage in router.php
  • Excludes vendor/endroid from all app bundle build paths to prevent ~15MB bloat:
    • JumpCommand.php — Jump ZIP bundle
    • PreparesBuild.php — Android native:run/native:package
    • BuildIosAppCommand.php — iOS native:run/native:package

Test plan

  • Run native:jump and verify QR code renders in browser
  • Run native:run android and confirm bundle size is reduced
  • Run native:run ios and confirm bundle size is reduced
  • Verify vendor/endroid is not present in built app bundles

🤖 Generated with Claude Code

The QR code package is needed at runtime for the Jump dev server but
should not be shipped in mobile app bundles (~15MB bloat). Moves it
from require-dev to require, upgrades to v6, updates the Builder API
usage, and adds vendor/endroid to exclusion lists in all build paths.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@shanerbaner82 shanerbaner82 merged commit adbf8f7 into main Apr 10, 2026
0 of 5 checks passed
@shanerbaner82 shanerbaner82 deleted the fix/exclude-qr-code-from-bundle branch April 10, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant