Skip to content

[eas-cli] print the preview page link instead of the tunnel url - #4352

Open
gwdp wants to merge 2 commits into
gwdp/build-tools-web-preview-page-logfrom
gwdp/sim-preview-page-link
Open

[eas-cli] print the preview page link instead of the tunnel url#4352
gwdp wants to merge 2 commits into
gwdp/build-tools-web-preview-page-logfrom
gwdp/sim-preview-page-link

Conversation

@gwdp

@gwdp gwdp commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Why

We print the raw ngrok URL, so sharing it means the other person sees an ngrok error if the session is starting or gone.

How

Print the expo.dev preview page instead, in the instructions and in --json. Falls back to the tunnel URL for a host the page cannot address.

Test Plan

  • CI
  • easd simulator:start.. and confirm printed link

@gwdp
gwdp force-pushed the gwdp/sim-preview-page-link branch 3 times, most recently from c78e910 to 0391334 Compare September 4, 2026 23:38
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.52%. Comparing base (a57ff60) to head (b4db19e).

Additional details and impacted files
@@                            Coverage Diff                            @@
##           gwdp/build-tools-web-preview-page-log    #4352      +/-   ##
=========================================================================
+ Coverage                                  65.45%   65.52%   +0.08%     
=========================================================================
  Files                                       1058     1058              
  Lines                                      49005    49018      +13     
  Branches                                   10424    10428       +4     
=========================================================================
+ Hits                                       32070    32113      +43     
+ Misses                                     15518    15489      -29     
+ Partials                                    1417     1416       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gwdp
gwdp force-pushed the gwdp/sim-preview-page-link branch from 0391334 to a25626b Compare September 4, 2026 23:52
@gwdp gwdp changed the title [simulator] print the preview page link instead of the tunnel url [eas-cli] print the preview page link instead of the tunnel url Sep 4, 2026
@gwdp
gwdp force-pushed the gwdp/sim-preview-page-link branch from 9099374 to 0e0e5ad Compare September 5, 2026 01:02
@gwdp
gwdp changed the base branch from main to gwdp/build-tools-web-preview-page-log September 5, 2026 01:02
gwdp added 2 commits September 4, 2026 18:07
The tunnel URL still works, so this is not a breaking change. Tests now cover the rewrite helpers end to end.
@gwdp
gwdp force-pushed the gwdp/sim-preview-page-link branch from 0e0e5ad to b4db19e Compare September 5, 2026 01:07
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

✅ Thank you for adding the changelog entry!

@gwdp
gwdp marked this pull request as ready for review September 5, 2026 01:08
@gwdp
gwdp requested a review from szdziedzic September 5, 2026 01:08
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Subscribed to pull request

File Patterns Mentions
packages/eas-cli/** @douglowder

Generated by CodeMention

Warning: The preamble and epilogue options in commentConfiguration are deprecated. Use template instead.

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