electron-rare/Aperant
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(windows): add platform detection for terminal profile commands

Browse Source 
- Use cmd.exe syntax (set/%) on Windows
- Use bash syntax (export/$) on Unix/macOS

Fixes #51

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
AndyMik90
2025-12-20 12:34:45 +01:00
parent cdda3ff277
commit f0a6a0a0af
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
auto-claude-ui/src/main/ipc-handlers/terminal-handlers.ts
+9 -3
View File
@@ -327,13 +327,19 @@ export function registerTerminalHandlers(
await new Promise(resolve => setTimeout(resolve, 500));
// Build the login command with the profile's config dir
// Use export to ensure the variable persists, then run setup-token
// Use platform-specific syntax for environment variables
let loginCommand: string;
if (!profile.isDefault && profile.configDir) {
// Use export and run in subshell to ensure CLAUDE_CONFIG_DIR is properly set
// SECURITY: Use escapeShellArg to prevent command injection via configDir
const escapedConfigDir = escapeShellArg(profile.configDir);
loginCommand = `export CLAUDE_CONFIG_DIR=${escapedConfigDir} && echo "Config dir: $CLAUDE_CONFIG_DIR" && claude setup-token`;
if (process.platform === 'win32') {
// Windows cmd.exe syntax: set "VAR=value" with %VAR% for expansion
loginCommand = `set "CLAUDE_CONFIG_DIR=${profile.configDir}" && echo Config dir: %CLAUDE_CONFIG_DIR% && claude setup-token`;
} else {
// Unix/Mac bash/zsh syntax: export VAR=value with $VAR for expansion
loginCommand = `export CLAUDE_CONFIG_DIR=${escapedConfigDir} && echo "Config dir: $CLAUDE_CONFIG_DIR" && claude setup-token`;
}
} else {
loginCommand = 'claude setup-token';
}