fix(config): treat whitespace-only numeric env values as unset in envNumber

Author: waleedlatif1

apps/sim/lib/billing/sandbox-pricing.test.ts

@@ -70,6 +70,9 @@ describe('sandbox pricing', () => {
 
       setEnv({ COST_MULTIPLIER: '-2' })
       expect(createSandboxPricing('e2b').multiplier).toBe(1)
+
+      setEnv({ COST_MULTIPLIER: '   ' })
+      expect(createSandboxPricing('e2b').multiplier).toBe(1)
     })
   })
 })

apps/sim/lib/core/config/env.test.ts

@@ -12,4 +12,11 @@ describe('envNumber', () => {
     expect(envNumber('5.5', 1, { min: 1, integer: true })).toBe(1)
     expect(envNumber(5.5, 1, { min: 1, integer: true })).toBe(1)
   })
+
+  it('treats whitespace-only values as unset instead of coercing them to 0', () => {
+    expect(envNumber('   ', 1)).toBe(1)
+    expect(envNumber('', 1)).toBe(1)
+    expect(envNumber(' 1.1 ', 1)).toBe(1.1)
+    expect(envNumber('0', 1)).toBe(0)
+  })
 })

apps/sim/lib/core/config/env.ts

@@ -813,7 +813,7 @@ export function envNumber(
   ) {
     return value
   }
-  if (value === undefined || value === null || value === '') return fallback
+  if (value === undefined || value === null || String(value).trim() === '') return fallback
   const parsed = Number(value)
   return Number.isFinite(parsed) && parsed >= min && (!options.integer || Number.isInteger(parsed))
     ? parsed

packages/testing/src/mocks/env.mock.ts

@@ -136,7 +136,7 @@ export function envNumberImpl(
   ) {
     return value
   }
-  if (value === undefined || value === null || value === '') return fallback
+  if (value === undefined || value === null || String(value).trim() === '') return fallback
   const parsed = Number(value)
   return Number.isFinite(parsed) && parsed >= min && (!options.integer || Number.isInteger(parsed))
     ? parsed