chore(expo-router): reduce usage of expo-constants (#26834)

# Why - reduce chance of issues like this expo/router#918 - Using globals directly is the future. A lot less bundling and resolving. # Test Plan Manually ran this in a dev client, web, and expo go for ios. --------- Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
expo · Feb 1, 2024 · 12d93da · 12d93da
1 parent b362564
commit 12d93da
Show file tree

Hide file tree

Showing 24 changed files with 58 additions and 103 deletions.
diff --git a/packages/@expo/cli/CHANGELOG.md b/packages/@expo/cli/CHANGELOG.md
@@ -24,6 +24,7 @@
 
 ### 💡 Others
 
+- Change server log tag. ([#26834](https://github.com/expo/expo/pull/26834) by [@EvanBacon](https://github.com/EvanBacon))
 - Eagerly perform iOS system checks to speed up iOS simulator launches. ([#26746](https://github.com/expo/expo/pull/26746) by [@EvanBacon](https://github.com/EvanBacon))
 - Improve warning for favicon missing during web export. ([#26733](https://github.com/expo/expo/pull/26733) by [@EvanBacon](https://github.com/EvanBacon))
 

diff --git a/packages/@expo/cli/src/start/server/metro/MetroTerminalReporter.ts b/packages/@expo/cli/src/start/server/metro/MetroTerminalReporter.ts
@@ -238,7 +238,7 @@ function getEnvironmentForBuildDetails(bundleDetails?: BundleDetails | null): st
   // Expo CLI will pass `customTransformOptions.environment = 'node'` when bundling for the server.
   const env = bundleDetails?.customTransformOptions?.environment ?? null;
   if (env === 'node') {
-    return `${chalk.bold(' λ ')} `;
+    return chalk.bold('λ') + ' ';
   }
 
   return '';

diff --git a/...xpo/cli/src/start/server/metro/__tests__/__snapshots__/MetroTerminalReporter-test.ts.snap b/...xpo/cli/src/start/server/metro/__tests__/__snapshots__/MetroTerminalReporter-test.ts.snap
@@ -1,3 +1,3 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`_getBundleStatusMessage should format standard progress for a server invocation 1`] = `" λ  ./index.js ▓▓▓▓▓▓▓▓░░░░░░░░ 50.0% ( 50/100)"`;
+exports[`_getBundleStatusMessage should format standard progress for a server invocation 1`] = `"λ ./index.js ▓▓▓▓▓▓▓▓░░░░░░░░ 50.0% ( 50/100)"`;
diff --git a/packages/expo-router/CHANGELOG.md b/packages/expo-router/CHANGELOG.md
@@ -17,6 +17,7 @@
 
 ### 💡 Others
 
+- Reduce usage of `expo-constants`. ([#26834](https://github.com/expo/expo/pull/26834) by [@EvanBacon](https://github.com/EvanBacon))
 - Deprecate `useRootNavigation` in favor of `useNavigationContainerRef`. ([#26529](https://github.com/expo/expo/pull/26529) by [@EvanBacon](https://github.com/EvanBacon))
 - Remove duplicate context mocking functions ([#26651](https://github.com/expo/expo/pull/26651) by [@marklawlor](https://github.com/marklawlor))
 

diff --git a/packages/expo-router/build/fork/extractPathFromURL.d.ts.map b/packages/expo-router/build/fork/extractPathFromURL.d.ts.map
diff --git a/packages/expo-router/build/fork/extractPathFromURL.js b/packages/expo-router/build/fork/extractPathFromURL.js
diff --git a/packages/expo-router/build/fork/extractPathFromURL.js.map b/packages/expo-router/build/fork/extractPathFromURL.js.map
diff --git a/packages/expo-router/build/head/ExpoHeadModule.d.ts b/packages/expo-router/build/head/ExpoHeadModule.d.ts
diff --git a/packages/expo-router/build/head/ExpoHeadModule.d.ts.map b/packages/expo-router/build/head/ExpoHeadModule.d.ts.map
diff --git a/packages/expo-router/build/head/ExpoHeadModule.js b/packages/expo-router/build/head/ExpoHeadModule.js