tiennm99/ccs
1
0
Fork 0
mirror of https://github.com/tiennm99/ccs.git synced 2026-07-30 12:22:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(cliproxy): filter undefined config values to preserve defaults

Browse Source 
Object spread with {port: undefined} was overwriting DEFAULT_CONFIG.port,
causing "Using existing CLIProxy on port undefined" after upgrade.

Root cause: profileInfo.port is undefined for hardcoded CLIProxy profiles
(gemini, codex, agy, qwen). When passed to execClaudeWithCLIProxy(), the
undefined value replaced the default port 8317.

Fix: Filter out undefined values before merging with DEFAULT_CONFIG.
This commit is contained in:
kaitranntt
2026-01-01 01:38:19 -05:00
parent d28e926ed4
commit 4c35e8a39e
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/cliproxy/cliproxy-executor.ts
+5 -1
View File
@@ -121,7 +121,11 @@ export async function execClaudeWithCLIProxy(
args: string[], args: string[],
config: Partial<ExecutorConfig> = {} config: Partial<ExecutorConfig> = {}
): Promise<void> { ): Promise<void> {
const cfg = { ...DEFAULT_CONFIG, ...config }; // Filter out undefined values to prevent overwriting defaults
const filteredConfig = Object.fromEntries(
Object.entries(config).filter(([, v]) => v !== undefined)
) as Partial<ExecutorConfig>;
const cfg = { ...DEFAULT_CONFIG, ...filteredConfig };
const verbose = cfg.verbose || args.includes('--verbose') || args.includes('-v'); const verbose = cfg.verbose || args.includes('--verbose') || args.includes('-v');
const log = (msg: string) => { const log = (msg: string) => {