From 6b9396fbc6d464bc3e3d6d3bb639e70fe5306074 Mon Sep 17 00:00:00 2001
From: kaitranntt <kaitran.ntt@gmail.com>
Date: Thu, 18 Dec 2025 02:54:46 -0500
Subject: [PATCH] feat(cliproxy): set WRITABLE_PATH for log storage in
 ~/.ccs/cliproxy/

- Add getCliproxyWritablePath() helper function
- Set WRITABLE_PATH env var when spawning CLIProxy in both executors
- Logs will now be stored in ~/.ccs/cliproxy/logs/ instead of CWD
- Enables error log viewer to find logs in predictable location

Note: CLIProxyAPI still has hardcoded MaxBackups=0 (unlimited).
Log rotation should be addressed in CLIProxyAPI upstream.
---
 src/cliproxy/cliproxy-executor.ts | 5 +++++
 src/cliproxy/config-generator.ts  | 9 +++++++++
 src/cliproxy/service-manager.ts   | 5 +++++
 3 files changed, 19 insertions(+)

diff --git a/src/cliproxy/cliproxy-executor.ts b/src/cliproxy/cliproxy-executor.ts
index bef6a2fa..1bad1d16 100644
--- a/src/cliproxy/cliproxy-executor.ts
+++ b/src/cliproxy/cliproxy-executor.ts
@@ -24,6 +24,7 @@ import {
   getProviderConfig,
   ensureProviderSettings,
   CLIPROXY_DEFAULT_PORT,
+  getCliproxyWritablePath,
 } from './config-generator';
 import { isAuthenticated } from './auth-handler';
 import { CLIProxyProvider, ExecutorConfig } from './types';
@@ -369,6 +370,10 @@ export async function execClaudeWithCLIProxy(
     proxy = spawn(binaryPath, proxyArgs, {
       stdio: ['ignore', 'ignore', 'ignore'],
       detached: true, // Persist after parent terminal closes
+      env: {
+        ...process.env,
+        WRITABLE_PATH: getCliproxyWritablePath(), // Logs stored in ~/.ccs/cliproxy/logs/
+      },
     });
 
     // Unref so parent process can exit independently
diff --git a/src/cliproxy/config-generator.ts b/src/cliproxy/config-generator.ts
index 3383d313..7f572dba 100644
--- a/src/cliproxy/config-generator.ts
+++ b/src/cliproxy/config-generator.ts
@@ -30,6 +30,15 @@ export const CCS_INTERNAL_API_KEY = 'ccs-internal-managed';
 /** Simple secret key for Control Panel login (user-facing) */
 export const CCS_CONTROL_PANEL_SECRET = 'ccs';
 
+/**
+ * Get CLIProxy writable directory for logs and runtime files.
+ * This directory is set as WRITABLE_PATH env var when spawning CLIProxy.
+ * Logs will be stored in ~/.ccs/cliproxy/logs/
+ */
+export function getCliproxyWritablePath(): string {
+  return path.join(getCcsDir(), 'cliproxy');
+}
+
 /**
  * Config version - bump when config format changes to trigger regeneration
  * v1: Initial config (port, auth-dir, api-keys only)
diff --git a/src/cliproxy/service-manager.ts b/src/cliproxy/service-manager.ts
index a515dcc0..f6977a07 100644
--- a/src/cliproxy/service-manager.ts
+++ b/src/cliproxy/service-manager.ts
@@ -19,6 +19,7 @@ import {
   regenerateConfig,
   configNeedsRegeneration,
   CLIPROXY_DEFAULT_PORT,
+  getCliproxyWritablePath,
 } from './config-generator';
 import { isCliproxyRunning } from './stats-fetcher';
 
@@ -171,6 +172,10 @@ export async function ensureCliproxyService(
   proxyProcess = spawn(binaryPath, proxyArgs, {
     stdio: ['ignore', verbose ? 'pipe' : 'ignore', verbose ? 'pipe' : 'ignore'],
     detached: true, // Allow process to run independently
+    env: {
+      ...process.env,
+      WRITABLE_PATH: getCliproxyWritablePath(), // Logs stored in ~/.ccs/cliproxy/logs/
+    },
   });
 
   // Forward output in verbose mode