Skip to content

Commit 8ffa1fc

Browse files
authored
Merge pull request #46 from MiniMax-AI-Dev/fix/validate-timeout-env
fix(config): validate MINIMAX_TIMEOUT env var to prevent NaN crash
2 parents 0c73f45 + 1dcba51 commit 8ffa1fc

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)