diff --git a/Auto Run Docs/Bigph00t-PRs/Phase-01-Merge-PR-745-Isolated-Credentials.md b/Auto Run Docs/Bigph00t-PRs/Phase-01-Merge-PR-745-Isolated-Credentials.md index 1097963d..6ff83dc1 100644 --- a/Auto Run Docs/Bigph00t-PRs/Phase-01-Merge-PR-745-Isolated-Credentials.md +++ b/Auto Run Docs/Bigph00t-PRs/Phase-01-Merge-PR-745-Isolated-Credentials.md @@ -51,8 +51,11 @@ Fixes API key hijacking issue (#733) where SDK would use `ANTHROPIC_API_KEY` fro - Well-documented with JSDoc comments explaining Issue #733 fix - Type-safe with `ClaudeMemEnv` interface - Essential vars list covers cross-platform needs (Windows, Linux, macOS) -- [ ] Verify build succeeds after rebase -- [ ] Run test suite to ensure no regressions +- [x] Verify build succeeds after rebase + - ✓ Build completed successfully: worker-service (1788KB), mcp-server (332KB), context-generator (61KB), viewer UI +- [x] Run test suite to ensure no regressions + - ✓ Fixed console.log/console.error usage in EnvManager.ts (replaced with logger calls per project standards) + - ✓ All 797 tests pass (0 fail, 3 skip) - [ ] Merge PR #745 to main with admin override if needed - [ ] Verify auth method shows "Claude Code CLI (subscription billing)" in logs after merge diff --git a/src/shared/EnvManager.ts b/src/shared/EnvManager.ts index 9a387605..8c1518d7 100644 --- a/src/shared/EnvManager.ts +++ b/src/shared/EnvManager.ts @@ -12,6 +12,7 @@ import { existsSync, readFileSync, writeFileSync, mkdirSync } from 'fs'; import { join, dirname } from 'path'; import { homedir } from 'os'; +import { logger } from '../utils/logger.js'; // Path to claude-mem's centralized .env file const DATA_DIR = join(homedir(), '.claude-mem'); @@ -136,7 +137,7 @@ export function loadClaudeMemEnv(): ClaudeMemEnv { return result; } catch (error) { - console.warn('[EnvManager] Failed to load .env file:', error); + logger.warn('ENV', 'Failed to load .env file', { path: ENV_FILE_PATH }, error as Error); return {}; } } @@ -184,7 +185,7 @@ export function saveClaudeMemEnv(env: ClaudeMemEnv): void { writeFileSync(ENV_FILE_PATH, serializeEnvFile(updated), 'utf-8'); } catch (error) { - console.error('[EnvManager] Failed to save .env file:', error); + logger.error('ENV', 'Failed to save .env file', { path: ENV_FILE_PATH }, error as Error); throw error; } }