Skip to content

[eas-cli] Deduplicate simulator --type flag options - #4321

Open
expo-tuft[bot] wants to merge 1 commit into
mainfrom
tchayen/simulator-type-flag-duplicate
Open

[eas-cli] Deduplicate simulator --type flag options#4321
expo-tuft[bot] wants to merge 1 commit into
mainfrom
tchayen/simulator-type-flag-duplicate

Conversation

@expo-tuft

@expo-tuft expo-tuft Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Why

eas simulator --help lists web-preview-only twice in the --type options.

How

DeviceRunSessionType.ServeSim and DeviceRunSessionType.WebPreviewOnly both map to web-preview-only, and the flag took its options straight from that record's values. A deduped DEVICE_RUN_SESSION_TYPE_FLAG_OPTIONS now feeds both eas simulator and eas simulator:list.

Test Plan

eas simulator --help and eas simulator:list --help each show web-preview-only once. Passing --type web-preview-only still resolves to the WebPreviewOnly enum, which the existing mapping test covers.

@expo-tuft
expo-tuft Bot force-pushed the tchayen/simulator-type-flag-duplicate branch from 00d0985 to 685dd66 Compare August 31, 2026 20:02
@github-actions

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.

@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.39%. Comparing base (4b1e5a7) to head (2aa55b2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4321      +/-   ##
==========================================
+ Coverage   65.38%   65.39%   +0.01%     
==========================================
  Files        1058     1058              
  Lines       49090    49091       +1     
  Branches    10353    10353              
==========================================
+ Hits        32095    32096       +1     
  Misses      16879    16879              
  Partials      116      116              

☔ 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.

@tchayen
tchayen requested a review from szdziedzic August 31, 2026 20:22
@expo-tuft
expo-tuft Bot force-pushed the tchayen/simulator-type-flag-duplicate branch from 685dd66 to 9b8033d Compare August 31, 2026 20:26
Co-authored-by: tchayen <5769281+tchayen@users.noreply.github.com>
@expo-tuft
expo-tuft Bot force-pushed the tchayen/simulator-type-flag-duplicate branch from 9b8033d to 2aa55b2 Compare August 31, 2026 20:44
@github-actions

Copy link
Copy Markdown

✅ Thank you for adding the changelog entry!

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.

0 participants