Skip to content

Commit 1dcba51

Browse files
Elaine YINclaude
authored andcommitted
fix(config): validate MINIMAX_TIMEOUT env var to prevent NaN crash
Non-numeric values like "abc" produced NaN → AbortSignal.timeout(NaN) threw TypeError. Now invalid values are silently ignored with fallback to 300s. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 920273d commit 1dcba51

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

src/config/loader.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,10 @@ export function loadConfig(flags: GlobalFlags): Config {
5050
flags.output || process.env.MINIMAX_OUTPUT || file.output,
5151
);
5252

53-
const timeout = flags.timeout
54-
?? (process.env.MINIMAX_TIMEOUT ? Number(process.env.MINIMAX_TIMEOUT) : undefined)
55-
?? file.timeout
56-
?? 300;
53+
const envTimeout = process.env.MINIMAX_TIMEOUT ? Number(process.env.MINIMAX_TIMEOUT) : undefined;
54+
const validEnvTimeout = envTimeout !== undefined && Number.isFinite(envTimeout) && envTimeout > 0
55+
? envTimeout : undefined;
56+
const timeout = flags.timeout ?? validEnvTimeout ?? file.timeout ?? 300;
5757

5858
return {
5959
apiKey,

0 commit comments

Comments
 (0)