fix(config): reject partial numeric strings in parseEnvInt

Author: dialupdisaster

src/config.ts

@@ -15,8 +15,9 @@ export type PluginConfig = {
 export function parseEnvInt(key: string, fallback: number): number {
   const raw = process.env[key]
   if (!raw) return fallback
-  const n = parseInt(raw, 10)
-  return Number.isFinite(n) && n > 0 ? n : fallback
+  if (!/^[1-9]\d*$/.test(raw)) return fallback
+  const n = Number(raw)
+  return Number.isSafeInteger(n) ? n : fallback
 }
 
 /**

tests/config.test.ts

@@ -29,7 +29,12 @@ describe("parseEnvInt", () => {
 
   test("returns fallback for float string", () => {
     process.env["TEST_INT"] = "1.5"
-    expect(parseEnvInt("TEST_INT", 42)).toBe(1)
+    expect(parseEnvInt("TEST_INT", 42)).toBe(42)
+  })
+
+  test("returns fallback for partial numeric string", () => {
+    process.env["TEST_INT"] = "5000ms"
+    expect(parseEnvInt("TEST_INT", 42)).toBe(42)
   })
 
   afterEach(() => { delete process.env["TEST_INT"] })