onbuzz 4.9.13 → 4.10.0

package/src/tools/fileContentReplaceTool.js

@@ -216,7 +216,7 @@ LIMITATIONS:
       this.logger?.error('Failed to parse file-content-replace parameters', {
         error: error.message
       });
-      throw new Error(`Parameter parsing failed: ${error.message}`);
+      throw new Error(`Parameter parsing failed: ${error.message}`, { cause: error });
     }
   }
 
@@ -442,7 +442,7 @@ LIMITATIONS:
     if (!firstFile.replacements || !Array.isArray(firstFile.replacements) || firstFile.replacements.length === 0) {
       return false;
     }
-    return firstFile.replacements[0].hasOwnProperty('mode');
+    return Object.prototype.hasOwnProperty.call(firstFile.replacements[0], 'mode');
   }
 
   /**
@@ -607,7 +607,7 @@ LIMITATIONS:
     // Check file exists
     try {
       await fs.access(resolvedPath);
-    } catch (error) {
+    } catch {
       throw new Error(`File not found: ${filePath}`);
     }
 
@@ -642,8 +642,7 @@ LIMITATIONS:
         content,
         replacement.oldContent,
         replacement.newContent,
-        replacement.linesLimit,
-        replacement.mode
+        replacement.linesLimit
       );
 
       content = result.newContent;
@@ -715,10 +714,9 @@ LIMITATIONS:
    * @param {string} oldContent - Content to replace
    * @param {string} newContent - Replacement content
    * @param {string|null} linesLimit - Line limit specification
-   * @param {string} mode - Trim mode
    * @returns {Object} Result with newContent and count
    */
-  async applyReplacement(content, oldContent, newContent, linesLimit, mode) {
+  async applyReplacement(content, oldContent, newContent, linesLimit) {
     if (!linesLimit) {
       // Replace in entire file (simple string replace, not regex)
       const count = this.countOccurrences(content, oldContent);

package/src/tools/fileSystemTool.js

@@ -13,11 +13,8 @@ import TagParser from '../utilities/tagParser.js';
 import DirectoryAccessManager from '../utilities/directoryAccessManager.js';
 import fs from 'fs/promises';
 import path from 'path';
-import crypto from 'crypto';
 
 import {
-  TOOL_STATUS,
-  FILE_EXTENSIONS,
   SYSTEM_DEFAULTS
 } from '../utilities/constants.js';
 import { validateForToolResponse } from '../utilities/structuredFileValidator.js';
@@ -250,7 +247,7 @@ SECURITY:
       return params;
       
     } catch (error) {
-      throw new Error(`Failed to parse filesystem parameters: ${error.message}`);
+      throw new Error(`Failed to parse filesystem parameters: ${error.message}`, { cause: error });
     }
   }
 
@@ -455,7 +452,7 @@ SECURITY:
       throw new Error(`Parameter validation failed: ${errorMsg}`);
     }
 
-    const { projectDir, agentId, directoryAccess } = context;
+    const { projectDir, directoryAccess } = context;
 
     // Per-agent config overrides (agent.toolConfig.filesystem) merged
     // with this tool's global defaults. Applied per-action at the top of
@@ -473,10 +470,6 @@ SECURITY:
     const effectiveAllowedExt = (Array.isArray(effectiveFs.allowedExtensions) && effectiveFs.allowedExtensions.length > 0)
       ? effectiveFs.allowedExtensions.map(e => String(e).toLowerCase())
       : null;
-    const effectiveMaxFileSize = (typeof effectiveFs.maxFileSize === 'number' && effectiveFs.maxFileSize > 0)
-      ? effectiveFs.maxFileSize
-      : this.maxFileSize;
-
     const _checkExt = (filePath) => {
       if (!filePath) return null;
       const ext = (filePath.match(/\.[^./\\]+$/)?.[0] || '').toLowerCase();
@@ -713,7 +706,7 @@ SECURITY:
       };
       
     } catch (error) {
-      throw new Error(`Failed to read file ${filePath}: ${error.message}`);
+      throw new Error(`Failed to read file ${filePath}: ${error.message}`, { cause: error });
     }
   }
 
@@ -834,7 +827,7 @@ SECURITY:
       return result;
       
     } catch (error) {
-      throw new Error(`Failed to write file ${fullPath}: ${error.message}`);
+      throw new Error(`Failed to write file ${fullPath}: ${error.message}`, { cause: error });
     }
   }
 
@@ -903,7 +896,7 @@ SECURITY:
       };
       
     } catch (error) {
-      throw new Error(`Failed to append to file ${fullPath}: ${error.message}`);
+      throw new Error(`Failed to append to file ${fullPath}: ${error.message}`, { cause: error });
     }
   }
 
@@ -945,7 +938,7 @@ SECURITY:
       };
       
     } catch (error) {
-      throw new Error(`Failed to delete file ${fullPath}: ${error.message}`);
+      throw new Error(`Failed to delete file ${fullPath}: ${error.message}`, { cause: error });
     }
   }
 
@@ -998,7 +991,7 @@ SECURITY:
       };
       
     } catch (error) {
-      throw new Error(`Failed to copy ${fullSourcePath} to ${fullDestPath}: ${error.message}`);
+      throw new Error(`Failed to copy ${fullSourcePath} to ${fullDestPath}: ${error.message}`, { cause: error });
     }
   }
 
@@ -1047,7 +1040,7 @@ SECURITY:
       };
 
     } catch (error) {
-      throw new Error(`Failed to move ${sourcePath} to ${destPath}: ${error.message}`);
+      throw new Error(`Failed to move ${sourcePath} to ${destPath}: ${error.message}`, { cause: error });
     }
   }
 
@@ -1079,7 +1072,7 @@ SECURITY:
       };
 
     } catch (error) {
-      throw new Error(`Failed to create directory ${directory}: ${error.message}`);
+      throw new Error(`Failed to create directory ${directory}: ${error.message}`, { cause: error });
     }
   }
 
@@ -1130,7 +1123,7 @@ SECURITY:
       };
 
     } catch (error) {
-      throw new Error(`Failed to list directory ${directory}: ${error.message}`);
+      throw new Error(`Failed to list directory ${directory}: ${error.message}`, { cause: error });
     }
   }
 
@@ -1176,7 +1169,7 @@ SECURITY:
         };
       }
 
-      throw new Error(`Failed to check existence of ${filePath}: ${error.message}`);
+      throw new Error(`Failed to check existence of ${filePath}: ${error.message}`, { cause: error });
     }
   }
 
@@ -1217,7 +1210,7 @@ SECURITY:
       };
 
     } catch (error) {
-      throw new Error(`Failed to get stats for ${filePath}: ${error.message}`);
+      throw new Error(`Failed to get stats for ${filePath}: ${error.message}`, { cause: error });
     }
   }