Skip to content

[eas-cli] Use development mode for Fingerprint - #4246

Open
ramonclaudio wants to merge 3 commits into
@ramonclaudio/dvt-119-eas-cli-env-modefrom
@ramonclaudio/dvt-119-eas-fingerprint-env-mode
Open

[eas-cli] Use development mode for Fingerprint#4246
ramonclaudio wants to merge 3 commits into
@ramonclaudio/dvt-119-eas-cli-env-modefrom
@ramonclaudio/dvt-119-eas-fingerprint-env-mode

Conversation

@ramonclaudio

Copy link
Copy Markdown
Member

Stacked on #4229.

Why

Fingerprint can inherit dotenv vars and NODE_ENV from a parent process, which can make it load different env files for the same project.

How

I updated EAS CLI to remove inherited dotenv vars and use development mode while it creates project fingerprints. We also wait for every Fingerprint call to finish before restoring the original EAS CLI env.

Test Plan

Tests and package checks pass.

@linear-code

linear-code Bot commented Aug 21, 2026

Copy link
Copy Markdown

DVT-119

@github-actions

Copy link
Copy Markdown

✅ Thank you for adding the changelog entry!

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.98%. Comparing base (743749d) to head (14f39e1).

Files with missing lines Patch % Lines
packages/eas-cli/src/fingerprint/cli.ts 86.67% 2 Missing ⚠️
Additional details and impacted files
@@                            Coverage Diff                             @@
##           @ramonclaudio/dvt-119-eas-cli-env-mode    #4246      +/-   ##
==========================================================================
+ Coverage                                   63.88%   63.98%   +0.10%     
==========================================================================
  Files                                        1033     1033              
  Lines                                       47535    47543       +8     
  Branches                                     9999     9999              
==========================================================================
+ Hits                                        30365    30416      +51     
+ Misses                                      17069    17026      -43     
  Partials                                      101      101              

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

@ramonclaudio
ramonclaudio marked this pull request as ready for review August 21, 2026 17:37
@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.

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