chatcc-agent 0.5.3 → 0.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chatcc-agent",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "CCLink Agent - bridges Claude Code CLI with instant messaging",
   "bin": {
     "chatcc": "src/cli.js"

package/src/file-service.js

@@ -3,6 +3,7 @@ const path = require('path');
 const os = require('os');
 const crypto = require('crypto');
 const { BLOCKED_ROOTS } = require('./security');
+const logUploader = require('./log-uploader');
 
 const FILE_TREE_MAX_DEPTH = 3;
 const FILE_READ_MAX_LINES = 100;
@@ -32,6 +33,10 @@ const SENSITIVE_FILE_NAMES = new Set([
 ]);
 const HOME_DIR = os.homedir();
 
+function fileLog(level, msg, extra) {
+  logUploader.collect(level, msg, { tag: 'FileService', module: 'file-service', ...(extra || {}) });
+}
+
 function isBinary(filePath) {
   const ext = path.extname(filePath).toLowerCase();
   if (BINARY_EXTENSIONS.has(ext)) return true;
@@ -177,7 +182,10 @@ class FileService {
   handlePermissionResponse(data) {
     const { request_id, approved, remember } = data;
     const pending = this._pendingPermissions.get(request_id);
-    if (!pending) return;
+    if (!pending) {
+      fileLog('WARN', 'permission response without pending request', { request_id });
+      return;
+    }
 
     clearTimeout(pending.timer);
     this._pendingPermissions.delete(request_id);
@@ -193,12 +201,14 @@ class FileService {
     const depth = Math.min(Math.max(1, data.depth || FILE_TREE_MAX_DEPTH), FILE_TREE_MAX_DEPTH);
     const resolved = resolvePath(dirPath);
     if (!resolved) {
+      fileLog('WARN', 'file tree invalid path', { from, request_id, path: dirPath });
       await this._reply(from, 'file_tree_response', { request_id, error: 'Invalid path' });
       return;
     }
 
     const safePath = await this.checkPermission(from, resolved, 'list', sessionConstraint);
     if (!safePath) {
+      fileLog('WARN', 'file tree access denied', { from, request_id, path: dirPath });
       await this._reply(from, 'file_tree_response', { request_id, error: 'Access denied' });
       return;
     }
@@ -221,7 +231,13 @@ class FileService {
         });
       } catch (sendErr) {
         // Payload too large — retry with minimal info
-        console.error('[FileService] Tree reply failed (likely too large), sending names only:', sendErr.message, sendErr.stack);
+        fileLog('WARN', 'file tree reply too large; retrying light tree', {
+          from,
+          request_id,
+          path: dirPath,
+          msg: sendErr.message,
+          stack: sendErr.stack,
+        });
         const lightTree = { name: tree.name, type: 'directory', children: tree.children.map(c => ({ name: c.name, type: c.type })) };
         await this._reply(from, 'file_tree_response', {
           request_id,
@@ -236,7 +252,13 @@ class FileService {
           error: e.message,
         });
       } catch (replyErr) {
-        console.error('[FileService] Error reply also failed:', replyErr.message, replyErr.stack);
+        fileLog('ERROR', 'file tree error reply failed', {
+          from,
+          request_id,
+          path: dirPath,
+          msg: replyErr.message,
+          stack: replyErr.stack,
+        });
       }
     }
   }
@@ -290,12 +312,14 @@ class FileService {
     const { request_id, path: filePath, start_line = 1, end_line } = data;
     const resolved = resolvePath(filePath);
     if (!resolved) {
+      fileLog('WARN', 'file read invalid path', { from, request_id, path: filePath });
       await this._reply(from, 'file_read_response', { request_id, error: 'Invalid path' });
       return;
     }
 
     const safePath = await this.checkPermission(from, resolved, 'read', sessionConstraint);
     if (!safePath) {
+      fileLog('WARN', 'file read access denied', { from, request_id, path: filePath });
       await this._reply(from, 'file_read_response', { request_id, error: 'Access denied' });
       return;
     }
@@ -354,6 +378,13 @@ class FileService {
         language: ext || 'text',
       });
     } catch (e) {
+      fileLog('ERROR', 'file read failed', {
+        from,
+        request_id,
+        path: filePath,
+        msg: e.message,
+        stack: e.stack,
+      });
       await this._reply(from, 'file_read_response', {
         request_id,
         error: e.message,
@@ -368,6 +399,7 @@ class FileService {
     if (!name || typeof name !== 'string' || name.length > 255 ||
         name.includes('/') || name.includes('\0') ||
         name === '.' || name === '..') {
+      fileLog('WARN', 'file mkdir invalid folder name', { from, request_id, path: parentPath, name });
       await this._reply(from, 'file_mkdir_response', {
         request_id,
         error: 'Invalid folder name',
@@ -377,12 +409,14 @@ class FileService {
 
     const resolved = resolvePath(parentPath);
     if (!resolved) {
+      fileLog('WARN', 'file mkdir invalid parent path', { from, request_id, path: parentPath, name });
       await this._reply(from, 'file_mkdir_response', { request_id, error: 'Invalid parent path' });
       return;
     }
 
     const safePath = await this.checkPermission(from, resolved, 'mkdir', sessionConstraint);
     if (!safePath) {
+      fileLog('WARN', 'file mkdir access denied', { from, request_id, path: parentPath, name });
       await this._reply(from, 'file_mkdir_response', { request_id, error: 'Access denied' });
       return;
     }
@@ -401,13 +435,21 @@ class FileService {
       }
 
       fs.mkdirSync(fullPath, { recursive: false });
-      console.log(`[FileService] mkdir: ${fullPath} (by ${from})`);
+      fileLog('INFO', 'file mkdir created', { from, request_id, path: fullPath, name });
       await this._reply(from, 'file_mkdir_response', {
         request_id,
         path: fullPath,
         name,
       });
     } catch (e) {
+      fileLog('ERROR', 'file mkdir failed', {
+        from,
+        request_id,
+        path: parentPath,
+        name,
+        msg: e.message,
+        stack: e.stack,
+      });
       await this._reply(from, 'file_mkdir_response', { request_id, error: e.message });
     }
   }
@@ -416,12 +458,14 @@ class FileService {
     const { request_id, pattern, path: searchPath } = data;
     const resolved = resolvePath(searchPath);
     if (!resolved) {
+      fileLog('WARN', 'file search invalid path', { from, request_id, path: searchPath, pattern });
       await this._reply(from, 'file_search_response', { request_id, error: 'Invalid path' });
       return;
     }
 
     const safePath = await this.checkPermission(from, resolved, 'search', sessionConstraint);
     if (!safePath) {
+      fileLog('WARN', 'file search access denied', { from, request_id, path: searchPath, pattern });
       await this._reply(from, 'file_search_response', { request_id, error: 'Access denied' });
       return;
     }
@@ -436,6 +480,14 @@ class FileService {
         })),
       });
     } catch (e) {
+      fileLog('ERROR', 'file search failed', {
+        from,
+        request_id,
+        path: searchPath,
+        pattern,
+        msg: e.message,
+        stack: e.stack,
+      });
       await this._reply(from, 'file_search_response', {
         request_id,
         error: e.message,