From eb7dc53d2efc7679ea81e175200cc52a0ca4175d Mon Sep 17 00:00:00 2001
From: tanzhenxin <tanzhenxing1987@gmail.com>
Date: Tue, 20 Jan 2026 09:36:28 +0800
Subject: [PATCH] fix(cli): isolate modelConfigUtils tests from system env vars

Use a clean process.env object instead of shallow-copying the original
environment. This prevents test failures when system has auth-related
env vars (e.g., OPENAI_API_KEY) that would interfere with test assertions.
---
 packages/cli/src/utils/modelConfigUtils.test.ts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/packages/cli/src/utils/modelConfigUtils.test.ts b/packages/cli/src/utils/modelConfigUtils.test.ts
index 0bf0acdcb..28812e8d4 100644
--- a/packages/cli/src/utils/modelConfigUtils.test.ts
+++ b/packages/cli/src/utils/modelConfigUtils.test.ts
@@ -31,7 +31,8 @@ describe('modelConfigUtils', () => {
 
     beforeEach(() => {
       vi.resetModules();
-      process.env = { ...originalEnv };
+      // Start with a clean env - getAuthTypeFromEnv only checks auth-related vars
+      process.env = {};
     });
 
     afterEach(() => {