sentinelayer-cli 0.4.5 → 0.6.2

package/src/agents/jules/tools/file-edit.js

@@ -1,180 +1,2 @@
-import fs from "node:fs";
-import path from "node:path";
-import { createHash } from "node:crypto";
-import { PathGuardError, resolveGuardedPath } from "./path-guards.js";
-
-/**
- * String replacement in files with uniqueness enforcement and diff generation.
- * Designed for use inside a worktree — validates path is within allowed directory.
- *
- * @param {object} input
- * @param {string} input.file_path - Absolute path to the file to modify.
- * @param {string} input.old_string - Exact text to replace.
- * @param {string} input.new_string - Replacement text (must differ from old_string).
- * @param {boolean} [input.replace_all] - Replace all occurrences (default: false).
- * @param {string} [input.allowed_root] - Root directory edits are permitted in (worktree guard).
- * @returns {{ filePath, diff, occurrencesFound, occurrencesReplaced, linesChanged }}
- */
-export function fileEdit(input) {
-  if (!input.old_string && input.old_string !== "") {
-    throw new FileEditError("old_string is required.");
-  }
-  if (input.new_string === undefined || input.new_string === null) {
-    throw new FileEditError("new_string is required.");
-  }
-  if (input.old_string === input.new_string) {
-    throw new FileEditError("old_string and new_string must be different.");
-  }
-
-  let filePath;
-  try {
-    const guarded = resolveGuardedPath({
-      filePath: input.file_path,
-      allowedRoot: input.allowed_root || undefined,
-    });
-    filePath = guarded.resolvedPath;
-  } catch (error) {
-    if (error instanceof PathGuardError) {
-      throw new FileEditError(error.message);
-    }
-    if (error instanceof FileEditError) {
-      throw error;
-    }
-    throw new FileEditError(`Cannot access path: ${error.message}`);
-  }
-
-  // Read current content
-  let content;
-  try {
-    content = fs.readFileSync(filePath, "utf-8");
-  } catch (err) {
-    if (err.code === "ENOENT") {
-      throw new FileEditError(`File not found: ${filePath}`);
-    }
-    throw new FileEditError(`Cannot read file: ${err.message}`);
-  }
-
-  // Count occurrences
-  const occurrences = countOccurrences(content, input.old_string);
-  if (occurrences === 0) {
-    throw new FileEditError(
-      `old_string not found in ${filePath}. Verify the exact text including whitespace and indentation.`,
-    );
-  }
-  if (occurrences > 1 && !input.replace_all) {
-    throw new FileEditError(
-      `old_string found ${occurrences} times in ${filePath}. Use replace_all: true to replace all, or provide more surrounding context to make it unique.`,
-    );
-  }
-
-  // Perform replacement
-  const replaceCount = input.replace_all ? occurrences : 1;
-  let newContent;
-  if (input.replace_all) {
-    newContent = content.split(input.old_string).join(input.new_string);
-  } else {
-    const idx = content.indexOf(input.old_string);
-    newContent =
-      content.slice(0, idx) +
-      input.new_string +
-      content.slice(idx + input.old_string.length);
-  }
-
-  // Generate unified diff for display
-  const diff = generateUnifiedDiff(filePath, content, newContent);
-
-  // Count changed lines
-  const oldLines = content.split("\n").length;
-  const newLines = newContent.split("\n").length;
-  const linesChanged = Math.abs(newLines - oldLines) +
-    countDiffLines(content, newContent);
-
-  // Write atomically: temp file + rename
-  const tmpPath = filePath + `.sl-edit-${Date.now()}`;
-  fs.writeFileSync(tmpPath, newContent, "utf-8");
-  fs.renameSync(tmpPath, filePath);
-
-  return {
-    filePath,
-    diff,
-    occurrencesFound: occurrences,
-    occurrencesReplaced: replaceCount,
-    linesChanged,
-    beforeHash: hashContent(content),
-    afterHash: hashContent(newContent),
-  };
-}
-
-function countOccurrences(haystack, needle) {
-  if (!needle) return 0;
-  let count = 0;
-  let idx = 0;
-  while ((idx = haystack.indexOf(needle, idx)) !== -1) {
-    count++;
-    idx += needle.length;
-  }
-  return count;
-}
-
-function generateUnifiedDiff(filePath, oldContent, newContent) {
-  const oldLines = oldContent.split("\n");
-  const newLines = newContent.split("\n");
-  const diffLines = [];
-
-  diffLines.push(`--- a/${path.basename(filePath)}`);
-  diffLines.push(`+++ b/${path.basename(filePath)}`);
-
-  // Simple line-by-line diff (not full Myers — sufficient for review display)
-  const maxLines = Math.max(oldLines.length, newLines.length);
-  let chunkStart = -1;
-  let chunkOld = [];
-  let chunkNew = [];
-
-  for (let i = 0; i < maxLines; i++) {
-    const oldLine = i < oldLines.length ? oldLines[i] : undefined;
-    const newLine = i < newLines.length ? newLines[i] : undefined;
-
-    if (oldLine !== newLine) {
-      if (chunkStart === -1) chunkStart = i;
-      if (oldLine !== undefined) chunkOld.push(`-${oldLine}`);
-      if (newLine !== undefined) chunkNew.push(`+${newLine}`);
-    } else if (chunkStart !== -1) {
-      // Flush chunk
-      diffLines.push(`@@ -${chunkStart + 1},${chunkOld.length} +${chunkStart + 1},${chunkNew.length} @@`);
-      diffLines.push(...chunkOld, ...chunkNew);
-      chunkStart = -1;
-      chunkOld = [];
-      chunkNew = [];
-    }
-  }
-
-  // Flush final chunk
-  if (chunkStart !== -1) {
-    diffLines.push(`@@ -${chunkStart + 1},${chunkOld.length} +${chunkStart + 1},${chunkNew.length} @@`);
-    diffLines.push(...chunkOld, ...chunkNew);
-  }
-
-  return diffLines.join("\n");
-}
-
-function countDiffLines(oldContent, newContent) {
-  const oldLines = oldContent.split("\n");
-  const newLines = newContent.split("\n");
-  let changed = 0;
-  const max = Math.min(oldLines.length, newLines.length);
-  for (let i = 0; i < max; i++) {
-    if (oldLines[i] !== newLines[i]) changed++;
-  }
-  return changed;
-}
-
-function hashContent(content) {
-  return createHash("sha256").update(content).digest("hex").slice(0, 16);
-}
-
-export class FileEditError extends Error {
-  constructor(message) {
-    super(message);
-    this.name = "FileEditError";
-  }
-}
+// Re-export from shared tools. FileEdit is not Jules-specific.
+export { fileEdit, FileEditError } from "../../shared-tools/file-edit.js";

package/src/agents/jules/tools/file-read.js

@@ -1,100 +1,2 @@
-import fs from "node:fs";
-import path from "node:path";
-import { PathGuardError, resolveGuardedPath } from "./path-guards.js";
-
-const MAX_RESULT_CHARS = 5000;
-const BINARY_EXTENSIONS = new Set([
-  ".png", ".jpg", ".jpeg", ".gif", ".webp", ".avif", ".ico", ".svg",
-  ".woff", ".woff2", ".ttf", ".eot", ".otf",
-  ".mp3", ".mp4", ".ogg", ".webm", ".wav",
-  ".zip", ".tar", ".gz", ".br", ".zst",
-  ".pdf", ".wasm", ".node", ".exe", ".dll", ".so", ".dylib",
-]);
-
-/**
- * Read a file with line numbers, offset/limit pagination, and binary detection.
- * Returns { filePath, content, numLines, startLine, totalLines, truncated }.
- */
-export function fileRead(input) {
-  const filePath = resolveAndValidatePath(input.file_path, input.allowed_root);
-  const ext = path.extname(filePath).toLowerCase();
-
-  if (BINARY_EXTENSIONS.has(ext)) {
-    const stat = fs.statSync(filePath);
-    return {
-      filePath,
-      content: `[Binary file: ${ext}, ${stat.size} bytes. Use a specialized viewer.]`,
-      numLines: 0,
-      startLine: 0,
-      totalLines: 0,
-      truncated: false,
-      binary: true,
-    };
-  }
-
-  let raw;
-  try {
-    raw = fs.readFileSync(filePath, "utf-8");
-  } catch (err) {
-    if (err.code === "ENOENT") {
-      throw new FileReadError(`File not found: ${filePath}`);
-    }
-    if (err.code === "EISDIR") {
-      throw new FileReadError(`Path is a directory, not a file: ${filePath}`);
-    }
-    throw new FileReadError(`Cannot read file: ${err.message}`);
-  }
-
-  const allLines = raw.split("\n");
-  const totalLines = allLines.length;
-  const offset = Math.max(0, input.offset ?? 0);
-  const limit = input.limit ?? 2000;
-  const sliced = allLines.slice(offset, offset + limit);
-  const startLine = offset + 1;
-
-  const numbered = sliced.map(
-    (line, i) => `${String(startLine + i).padStart(6)}\t${line}`,
-  );
-  let content = numbered.join("\n");
-  let truncated = false;
-
-  if (content.length > MAX_RESULT_CHARS) {
-    content = content.slice(0, MAX_RESULT_CHARS) + "\n[... truncated]";
-    truncated = true;
-  }
-
-  return {
-    filePath,
-    content,
-    numLines: sliced.length,
-    startLine,
-    totalLines,
-    truncated,
-    binary: false,
-  };
-}
-
-export class FileReadError extends Error {
-  constructor(message) {
-    super(message);
-    this.name = "FileReadError";
-  }
-}
-
-function resolveAndValidatePath(filePath, allowedRoot) {
-  try {
-    const guarded = resolveGuardedPath({
-      filePath,
-      allowedRoot: allowedRoot || undefined,
-    });
-    return guarded.resolvedPath;
-  } catch (error) {
-    if (error instanceof PathGuardError) {
-      throw new FileReadError(error.message);
-    }
-    if (error instanceof FileReadError) {
-      throw error;
-    }
-    throw new FileReadError(`Cannot access path: ${error.message}`);
-  }
-}
+// Re-export from shared tools. FileRead is not Jules-specific.
+export { fileRead, FileReadError } from "../../shared-tools/file-read.js";