-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Revert CODEX_ACP_BRIDGE_VERSION and fix main-process log persistence #1367
Copy link
Copy link
Closed
Description
Problem
- PR fix(acp): bump bridge versions and fix Windows UTF-8 encoding for npx #1316 bumped
CODEX_ACP_BRIDGE_VERSIONfrom0.9.5to0.10.0. Users have reported issues with the new version. - Main-process
console.logoutput was not being persisted to electron-log file transport. src/index.tscontained too much business logic (~1060 lines), making it hard to maintain.
Solution
- Revert
CODEX_ACP_BRIDGE_VERSIONback to0.9.5insrc/types/acpTypes.ts - Redirect main-process console to electron-log via
Object.assign(console, log.functions) - Extract business code from
src/index.tsinto dedicated modules:src/process/tray.ts— system tray logicsrc/process/deepLink.ts— deep link parsing and handlingsrc/process/webuiConfig.ts— WebUI config resolutionsrc/process/mainWindowLifecycle.ts— main window reference binding and show/focus helpers- Move DevTools IPC providers to
src/process/bridge/applicationBridge.ts
Note: CLAUDE_ACP_BRIDGE_VERSION (0.21.0) is unaffected and remains unchanged.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels