refactor(env): sort functions alphabetically in env modules

Author: jdalton

src/env/rewire.ts

@@ -51,6 +51,13 @@ if (isVitestEnv && !globalThis[sharedOverridesSymbol]) {
 const sharedOverrides: Map<string, string | undefined> | undefined =
   globalThis[sharedOverridesSymbol]
 
+/**
+ * Clear a specific environment variable override.
+ */
+export function clearEnv(key: string): void {
+  sharedOverrides?.delete(key)
+}
+
 /**
  * Get an environment variable value, checking overrides first.
  *
@@ -77,6 +84,14 @@ export function getEnvValue(key: string): string | undefined {
   return process.env[key]
 }
 
+/**
+ * Check if an environment variable has been overridden.
+ */
+export function hasOverride(key: string): boolean {
+  const isolatedOverrides = isolatedOverridesStorage.getStore()
+  return !!(isolatedOverrides?.has(key) || sharedOverrides?.has(key))
+}
+
 /**
  * Check if an environment variable exists (has a key), checking overrides first.
  *
@@ -103,6 +118,23 @@ export function isInEnv(key: string): boolean {
   return hasOwn(process.env, key)
 }
 
+/**
+ * Clear all environment variable overrides.
+ * Useful in afterEach hooks to ensure clean test state.
+ *
+ * @example
+ * ```typescript
+ * import { resetEnv } from './rewire'
+ *
+ * afterEach(() => {
+ *   resetEnv()
+ * })
+ * ```
+ */
+export function resetEnv(): void {
+  sharedOverrides?.clear()
+}
+
 /**
  * Set an environment variable override for testing.
  * This does not modify process.env, only affects env getters.
@@ -132,38 +164,6 @@ export function setEnv(key: string, value: string | undefined): void {
   sharedOverrides?.set(key, value)
 }
 
-/**
- * Clear a specific environment variable override.
- */
-export function clearEnv(key: string): void {
-  sharedOverrides?.delete(key)
-}
-
-/**
- * Clear all environment variable overrides.
- * Useful in afterEach hooks to ensure clean test state.
- *
- * @example
- * ```typescript
- * import { resetEnv } from './rewire'
- *
- * afterEach(() => {
- *   resetEnv()
- * })
- * ```
- */
-export function resetEnv(): void {
-  sharedOverrides?.clear()
-}
-
-/**
- * Check if an environment variable has been overridden.
- */
-export function hasOverride(key: string): boolean {
-  const isolatedOverrides = isolatedOverridesStorage.getStore()
-  return !!(isolatedOverrides?.has(key) || sharedOverrides?.has(key))
-}
-
 /**
  * Run code with environment overrides in an isolated AsyncLocalStorage context.
  * Creates true context isolation - overrides don't leak to concurrent code.

src/env/socket-cli.ts

@@ -76,6 +76,28 @@ export function getSocketCliApiToken(): string | undefined {
   )
 }
 
+/**
+ * Bootstrap cache directory path.
+ * Set by bootstrap wrappers to pass dlx cache location to CLI.
+ *
+ * @returns Bootstrap cache directory or undefined
+ */
+/*@__NO_SIDE_EFFECTS__*/
+export function getSocketCliBootstrapCacheDir(): string | undefined {
+  return getEnvValue('SOCKET_CLI_BOOTSTRAP_CACHE_DIR')
+}
+
+/**
+ * Bootstrap package spec (e.g., @socketsecurity/cli@^2.0.11).
+ * Set by bootstrap wrappers (SEA/smol/npm) to pass package spec to CLI.
+ *
+ * @returns Bootstrap package spec or undefined
+ */
+/*@__NO_SIDE_EFFECTS__*/
+export function getSocketCliBootstrapSpec(): string | undefined {
+  return getEnvValue('SOCKET_CLI_BOOTSTRAP_SPEC')
+}
+
 /**
  * Socket CLI configuration file path (alternative name).
  *
@@ -96,6 +118,21 @@ export function getSocketCliFix(): string | undefined {
   return getEnvValue('SOCKET_CLI_FIX')
 }
 
+/**
+ * Socket CLI GitHub authentication token.
+ * Checks SOCKET_CLI_GITHUB_TOKEN, SOCKET_SECURITY_GITHUB_PAT, then falls back to GITHUB_TOKEN.
+ *
+ * @returns GitHub token or undefined
+ */
+/*@__NO_SIDE_EFFECTS__*/
+export function getSocketCliGithubToken(): string | undefined {
+  return (
+    getEnvValue('SOCKET_CLI_GITHUB_TOKEN') ||
+    getEnvValue('SOCKET_SECURITY_GITHUB_PAT') ||
+    getEnvValue('GITHUB_TOKEN')
+  )
+}
+
 /**
  * Whether to skip Socket CLI API token requirement (alternative name).
  *
@@ -136,40 +173,3 @@ export function getSocketCliOrgSlug(): string | undefined {
 export function getSocketCliViewAllRisks(): boolean {
   return envAsBoolean(getEnvValue('SOCKET_CLI_VIEW_ALL_RISKS'))
 }
-
-/**
- * Socket CLI GitHub authentication token.
- * Checks SOCKET_CLI_GITHUB_TOKEN, SOCKET_SECURITY_GITHUB_PAT, then falls back to GITHUB_TOKEN.
- *
- * @returns GitHub token or undefined
- */
-/*@__NO_SIDE_EFFECTS__*/
-export function getSocketCliGithubToken(): string | undefined {
-  return (
-    getEnvValue('SOCKET_CLI_GITHUB_TOKEN') ||
-    getEnvValue('SOCKET_SECURITY_GITHUB_PAT') ||
-    getEnvValue('GITHUB_TOKEN')
-  )
-}
-
-/**
- * Bootstrap package spec (e.g., @socketsecurity/cli@^2.0.11).
- * Set by bootstrap wrappers (SEA/smol/npm) to pass package spec to CLI.
- *
- * @returns Bootstrap package spec or undefined
- */
-/*@__NO_SIDE_EFFECTS__*/
-export function getSocketCliBootstrapSpec(): string | undefined {
-  return getEnvValue('SOCKET_CLI_BOOTSTRAP_SPEC')
-}
-
-/**
- * Bootstrap cache directory path.
- * Set by bootstrap wrappers to pass dlx cache location to CLI.
- *
- * @returns Bootstrap cache directory or undefined
- */
-/*@__NO_SIDE_EFFECTS__*/
-export function getSocketCliBootstrapCacheDir(): string | undefined {
-  return getEnvValue('SOCKET_CLI_BOOTSTRAP_CACHE_DIR')
-}

src/env/temp-dir.ts

@@ -5,15 +5,6 @@
 
 import { getEnvValue } from './rewire'
 
-/**
- * TMPDIR environment variable.
- * Unix/macOS temporary directory path.
- */
-/*@__NO_SIDE_EFFECTS__*/
-export function getTmpdir(): string | undefined {
-  return getEnvValue('TMPDIR')
-}
-
 /**
  * TEMP environment variable.
  * Windows temporary directory path.
@@ -31,3 +22,12 @@ export function getTemp(): string | undefined {
 export function getTmp(): string | undefined {
   return getEnvValue('TMP')
 }
+
+/**
+ * TMPDIR environment variable.
+ * Unix/macOS temporary directory path.
+ */
+/*@__NO_SIDE_EFFECTS__*/
+export function getTmpdir(): string | undefined {
+  return getEnvValue('TMPDIR')
+}

src/env/windows.ts

@@ -14,6 +14,15 @@ export function getAppdata(): string | undefined {
   return getEnvValue('APPDATA')
 }
 
+/**
+ * COMSPEC environment variable.
+ * Points to the Windows command processor (typically cmd.exe).
+ */
+/*@__NO_SIDE_EFFECTS__*/
+export function getComspec(): string | undefined {
+  return getEnvValue('COMSPEC')
+}
+
 /**
  * LOCALAPPDATA environment variable.
  * Points to the Local Application Data directory on Windows.
@@ -31,12 +40,3 @@ export function getLocalappdata(): string | undefined {
 export function getUserprofile(): string | undefined {
   return getEnvValue('USERPROFILE')
 }
-
-/**
- * COMSPEC environment variable.
- * Points to the Windows command processor (typically cmd.exe).
- */
-/*@__NO_SIDE_EFFECTS__*/
-export function getComspec(): string | undefined {
-  return getEnvValue('COMSPEC')
-}