lsm / neokai / 21813298280

82%

Build Type

push

github

Committed by lsm

Commit Message

refactor: remove ALL console usage, enforce Logger-only pattern

Removed every console.log/warn/error call from source code (40 total):
- sdk-session-file-manager.ts: Removed 14 redundant console.error calls
- event-bus.ts: Removed error logging in event handlers (5 calls)
- typed-hub.ts: Removed error/debug logging (3 calls)
- setup-websocket.ts: Removed WebSocket error logging (3 calls)
- config.ts: Removed credential discovery logging, deleted dead logCredentialDiscovery()
- session-handlers.ts: Removed event emission error logging (3 calls)
- in-process-transport.ts: Removed handler error logging (2 calls)
- model-service.ts: Removed model loading error log
- App.tsx: Removed state init error log
- server-prod.ts: Removed proxy error log
- global-store.ts: Removed init/refresh error logs (2 calls)
- utils.ts: Removed clipboard error log
- ChatContainer.tsx: Removed worktree mode error log

Removed unnecessary file ignores - only ignoring:
- logger.ts (needs console for implementation)
- Entry points (main.ts, app.ts, index.ts)
- CLI/scripts/build files
- Test files, e2e, npm packages

Deleted 182 lines of dead test code (logCredentialDiscovery tests).

✅ 0 lint errors, 1560 tests passing, zero console output

Coverage Stats

4 of 7 new or added lines in 4 files covered. (57.14%)

2 existing lines in 2 files now uncovered.

9325 of 14036 relevant lines covered (66.44%)

326.27 hits per line