@bike4mind/cli 0.2.23 → 0.2.24-cli-file-read-offset.18478

package/dist/index.js

@@ -3489,8 +3489,16 @@ CORE BEHAVIOR:
 
 FOR SOFTWARE ENGINEERING TASKS:
 When requested to perform tasks like fixing bugs, adding features, refactoring, or explaining code, follow this sequence:
-1. **Understand:** Think about the user's request and the relevant codebase context. Use '${TOOL_GREP_SEARCH}' and '${TOOL_GLOB_FILES}' search tools extensively (in parallel if independent) to understand file structures, existing code patterns, and conventions. Use '${TOOL_FILE_READ}' to understand context and validate any assumptions you may have. If you need to read multiple files, you should make multiple parallel calls to '${TOOL_FILE_READ}'.
-                   When the task involves **complex refactoring, codebase exploration or system-wide analysis**, your **first and primary action** must be to delegate to the '${EXPLORE_SUBAGENT_TYPE}' agent using the '${TOOL_SUBAGENT_DELEGATE}' tool. Use it to build a comprehensive understanding of the code, its structure, and dependencies. For **simple, targeted searches** (like finding a specific function name, file path, or variable declaration), you should use '${TOOL_GREP_SEARCH}' or '${TOOL_GLOB_FILES}' directly.
+1. **Understand:** Think about the user's request and the relevant codebase context. Use '${TOOL_GREP_SEARCH}' and '${TOOL_GLOB_FILES}' search tools extensively (in parallel if independent) to understand file structures, existing code patterns, and conventions. Use '${TOOL_FILE_READ}' to understand context and validate any assumptions you may have.
+
+   IMPORTANT FILE READING RULES:
+   - Read each file ONCE and refer to it in conversation history instead of re-reading
+   - Read files COMPLETELY by default (without offset/limit parameters)
+   - Only use offset/limit for files that are too large to fit in context (thousands of lines)
+   - If you need to read multiple DIFFERENT files, make multiple parallel calls to '${TOOL_FILE_READ}'
+   - NEVER make multiple calls to read the SAME file with different offsets unless it's truly too large
+
+   When the task involves **complex refactoring, codebase exploration or system-wide analysis**, your **first and primary action** must be to delegate to the '${EXPLORE_SUBAGENT_TYPE}' agent using the '${TOOL_SUBAGENT_DELEGATE}' tool. Use it to build a comprehensive understanding of the code, its structure, and dependencies. For **simple, targeted searches** (like finding a specific function name, file path, or variable declaration), you should use '${TOOL_GREP_SEARCH}' or '${TOOL_GLOB_FILES}' directly.
 2. **Plan:** Build a coherent and grounded (based on the understanding in step 1) plan for how you intend to resolve the user's task. Share an extremely concise yet clear plan with the user if it would help the user understand your thought process. As part of the plan, you should use an iterative development process that includes writing unit tests to verify your changes. Use output logs or debug statements as part of this process to arrive at a solution.
   If '${EXPLORE_SUBAGENT_TYPE}' subagent was used, do not ignore the output of the agent, you must use it as the foundation of your plan. For complex tasks, break them down into smaller, manageable subtasks and use the \`${TOOL_WRITE_TODOS}\` tool to track your progress. Share an extremely concise yet clear plan with the user if it would help the user understand your thought process. As part of the plan, you should use an iterative development process that includes writing unit tests to verify your changes. Use output logs or debug statements as part of this process to arrive at a solution.
 3. **Implement:** Use the available tools (e.g., '${TOOL_EDIT_LOCAL_FILE}', '${TOOL_CREATE_FILE}', '${TOOL_BASH_EXECUTE}' ...) to act on the plan, strictly adhering to the project's established conventions.
@@ -5585,15 +5593,6 @@ var llmAnalysisResponseSchema = z136.object({
   suggestedTags: z136.array(z136.string()).default([])
 });
 
-// ../../b4m-core/packages/services/dist/src/llm/ChatCompletion.js
-import { z as z138 } from "zod";
-
-// ../../b4m-core/packages/services/dist/src/llm/ChatCompletionInvoke.js
-import { fromZodError } from "zod-validation-error";
-
-// ../../b4m-core/packages/services/dist/src/llm/tools/toolManager.js
-var BUILT_IN_TOOL_SET = new Set(b4mLLMTools.options);
-
 // ../../b4m-core/packages/services/dist/src/llm/tools/ToolCacheManager.js
 var DEFAULT_CONFIG = {
   ttl: 5 * 60 * 1e3,
@@ -8859,7 +8858,7 @@ import { existsSync as existsSync3, statSync as statSync4 } from "fs";
 import path7 from "path";
 var MAX_FILE_SIZE2 = 10 * 1024 * 1024;
 async function readFileContent(params) {
-  const { path: filePath, encoding = "utf-8", maxLines } = params;
+  const { path: filePath, encoding = "utf-8", offset = 0, limit } = params;
   const normalizedPath = path7.normalize(filePath);
   const resolvedPath = path7.resolve(process.cwd(), normalizedPath);
   const cwd = path7.resolve(process.cwd());
@@ -8881,16 +8880,39 @@ async function readFileContent(params) {
     throw new Error(`File appears to be binary. Use encoding 'base64' to read binary files, or specify a different encoding.`);
   }
   const content = await fs8.readFile(resolvedPath, encoding);
-  if (maxLines && typeof content === "string") {
+  if (typeof content === "string") {
     const lines = content.split("\n");
-    if (lines.length > maxLines) {
-      const truncatedContent = lines.slice(0, maxLines).join("\n");
-      return `${truncatedContent}
+    const totalLines = lines.length;
+    if (offset < 0) {
+      throw new Error(`Invalid offset: ${offset}. Offset must be 0 or greater.`);
+    }
+    if (offset >= totalLines) {
+      return `No content to show. File has ${totalLines} lines, but offset is ${offset}.
+(offset is 0-based, so valid range is 0-${Math.max(0, totalLines - 1)})`;
+    }
+    if (limit !== void 0 && limit > 0) {
+      const endLine = Math.min(offset + limit, totalLines);
+      const paginatedContent = lines.slice(offset, endLine).join("\n");
+      if (endLine < totalLines) {
+        const nextOffset = endLine;
+        return `${paginatedContent}
+
+... Showing lines ${offset + 1}-${endLine} of ${totalLines} total lines (${stats.size} bytes total).
+To read more, use offset: ${nextOffset}`;
+      }
+      return `${paginatedContent}
+
+... Showing lines ${offset + 1}-${endLine} of ${totalLines} total lines (${stats.size} bytes total). End of file reached.`;
+    }
+    if (offset > 0) {
+      const paginatedContent = lines.slice(offset).join("\n");
+      return `${paginatedContent}
 
-... (truncated: ${lines.length - maxLines} more lines, ${stats.size} bytes total)`;
+... Showing lines ${offset + 1}-${totalLines} of ${totalLines} total lines (${stats.size} bytes total).`;
     }
+    return content;
   }
-  return typeof content === "string" ? content : `[Binary content, ${stats.size} bytes, base64 encoded]
+  return `[Binary content, ${stats.size} bytes, base64 encoded]
 ${content}`;
 }
 async function checkIfBinary(filePath) {
@@ -8927,7 +8949,7 @@ var fileReadTool = {
     },
     toolSchema: {
       name: "file_read",
-      description: "Read the contents of a file from the local filesystem. Supports text files with various encodings. Files are restricted to the current working directory and subdirectories for security.",
+      description: "Read the contents of a file from the local filesystem. Supports text files with various encodings. Files are restricted to the current working directory and subdirectories for security. IMPORTANT: Read files completely by default (without offset/limit). Only use offset/limit for extremely large files (thousands of lines) that exceed context limits. Never re-read the same file multiple times - refer to previous reads in conversation history instead.",
       parameters: {
         type: "object",
         properties: {
@@ -8940,9 +8962,13 @@ var fileReadTool = {
             description: "File encoding (default: utf-8). Use base64 for binary files.",
             enum: ["utf-8", "ascii", "base64"]
           },
-          maxLines: {
+          offset: {
             type: "number",
-            description: "Maximum number of lines to return (optional). Useful for previewing large files. Returns full file by default."
+            description: "OPTIONAL: For text files, the 0-based line number to start reading from. Only use for extremely large files (thousands of lines) that cannot fit in context. Default behavior is to read the entire file, which is preferred for most cases."
+          },
+          limit: {
+            type: "number",
+            description: "OPTIONAL: Maximum number of lines to read from offset. Only use for extremely large files (thousands of lines) that cannot fit in context. Default behavior is to read the entire file, which is preferred for most cases."
           }
         },
         required: ["path"]
@@ -10097,16 +10123,8 @@ var generateMcpTools = async (mcpData) => {
 import throttle2 from "lodash/throttle.js";
 
 // ../../b4m-core/packages/services/dist/src/llm/ChatCompletionFeatures.js
+import { z as z138 } from "zod";
 import uniq4 from "lodash/uniq.js";
-
-// ../../b4m-core/packages/services/dist/src/llm/StatusManager.js
-import throttle from "lodash/throttle.js";
-
-// ../../b4m-core/packages/services/dist/src/llm/ChatCompletionProcess.js
-var DISABLE_SERVER_THROTTLING = process.env.DISABLE_SERVER_THROTTLING === "true";
-var questSaveMutex = new Mutex();
-
-// ../../b4m-core/packages/services/dist/src/llm/ChatCompletion.js
 var QuestStartBodySchema = z138.object({
   userId: z138.string(),
   sessionId: z138.string(),
@@ -10148,6 +10166,19 @@ var QuestStartBodySchema = z138.object({
   timezone: z138.string().optional()
 });
 
+// ../../b4m-core/packages/services/dist/src/llm/StatusManager.js
+import throttle from "lodash/throttle.js";
+
+// ../../b4m-core/packages/services/dist/src/llm/ChatCompletionProcess.js
+var DISABLE_SERVER_THROTTLING = process.env.DISABLE_SERVER_THROTTLING === "true";
+var questSaveMutex = new Mutex();
+
+// ../../b4m-core/packages/services/dist/src/llm/ChatCompletionInvoke.js
+import { fromZodError } from "zod-validation-error";
+
+// ../../b4m-core/packages/services/dist/src/llm/tools/toolManager.js
+var BUILT_IN_TOOL_SET = new Set(b4mLLMTools.options);
+
 // ../../b4m-core/packages/services/dist/src/llm/ImageGeneration.js
 import axios8 from "axios";
 import { fileTypeFromBuffer as fileTypeFromBuffer4 } from "file-type";
@@ -12513,7 +12544,7 @@ import { isAxiosError as isAxiosError2 } from "axios";
 // package.json
 var package_default = {
   name: "@bike4mind/cli",
-  version: "0.2.23",
+  version: "0.2.24-cli-file-read-offset.18478+9538467fd",
   type: "module",
   description: "Interactive CLI tool for Bike4Mind with ReAct agents",
   license: "UNLICENSED",
@@ -12621,10 +12652,10 @@ var package_default = {
   },
   devDependencies: {
     "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.47.0",
-    "@bike4mind/mcp": "1.27.0",
-    "@bike4mind/services": "2.45.0",
-    "@bike4mind/utils": "2.3.2",
+    "@bike4mind/common": "2.47.1-cli-file-read-offset.18478+9538467fd",
+    "@bike4mind/mcp": "1.27.1-cli-file-read-offset.18478+9538467fd",
+    "@bike4mind/services": "2.45.1-cli-file-read-offset.18478+9538467fd",
+    "@bike4mind/utils": "2.3.3-cli-file-read-offset.18478+9538467fd",
     "@types/better-sqlite3": "^7.6.13",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
@@ -12641,7 +12672,7 @@ var package_default = {
   optionalDependencies: {
     "@vscode/ripgrep": "^1.17.0"
   },
-  gitHead: "ad52049ef04e434ab45416bed8e4754b4b232c0d"
+  gitHead: "9538467fd3acaaf1c31be03ff723e206139a8469"
 };
 
 // src/config/constants.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bike4mind/cli",
-  "version": "0.2.23",
+  "version": "0.2.24-cli-file-read-offset.18478+9538467fd",
   "type": "module",
   "description": "Interactive CLI tool for Bike4Mind with ReAct agents",
   "license": "UNLICENSED",
@@ -108,10 +108,10 @@
   },
   "devDependencies": {
     "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.47.0",
-    "@bike4mind/mcp": "1.27.0",
-    "@bike4mind/services": "2.45.0",
-    "@bike4mind/utils": "2.3.2",
+    "@bike4mind/common": "2.47.1-cli-file-read-offset.18478+9538467fd",
+    "@bike4mind/mcp": "1.27.1-cli-file-read-offset.18478+9538467fd",
+    "@bike4mind/services": "2.45.1-cli-file-read-offset.18478+9538467fd",
+    "@bike4mind/utils": "2.3.3-cli-file-read-offset.18478+9538467fd",
     "@types/better-sqlite3": "^7.6.13",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
@@ -128,5 +128,5 @@
   "optionalDependencies": {
     "@vscode/ripgrep": "^1.17.0"
   },
-  "gitHead": "ad52049ef04e434ab45416bed8e4754b4b232c0d"
+  "gitHead": "9538467fd3acaaf1c31be03ff723e206139a8469"
 }