[Performance] Memoize isWsl

packages/cli-kit/src/public/node/system.test.ts

@@ -393,3 +393,15 @@ describe('readStdinString', () => {
     await expect(got).rejects.toThrow('Stdin input exceeded the maximum allowed size.')
   })
 })
+
+describe('isWsl', () => {
+  test('memoizes the result', async () => {
+    // When
+    const firstCall = system.isWsl()
+    const secondCall = system.isWsl()
+
+    // Then
+    expect(firstCall).toBe(secondCall)
+    await expect(firstCall).resolves.toBe(await secondCall)
+  })
+})

packages/cli-kit/src/public/node/system.ts

@@ -98,9 +98,9 @@ export interface CaptureOutputResult {
  * @example
  * ```typescript
  * const result = await captureOutputWithExitCode('ls', ['-la'])
- * if (result.exitCode !== 0) \{
+ * if (result.exitCode !== 0) {
  *   console.error('Command failed:', result.stderr)
- * \}
+ * }
  * ```
  */
 export async function captureOutputWithExitCode(
@@ -351,14 +351,23 @@ export function isCI(): boolean {
   return isTruthy(process.env.CI)
 }
 
+/**
+ * Memoized value for the WSL check.
+ */
+let memoizedIsWsl: Promise<boolean> | undefined
+
 /**
  * Check if the current environment is a WSL environment.
  *
  * @returns True if the current environment is a WSL environment.
  */
-export async function isWsl(): Promise<boolean> {
-  const wsl = await import('is-wsl')
-  return wsl.default
+export function isWsl(): Promise<boolean> {
+  // Memoize the promise to avoid redundant dynamic imports and filesystem/environment checks.
+  // This is checked frequently during analytics generation.
+  return (memoizedIsWsl ??= (async () => {
+    const wsl = await import('is-wsl')
+    return wsl.default
+  })())
 }
 
 /**