@morphllm/morphsdk 0.2.76 → 0.2.81

package/dist/tools/warp_grep/index.js

@@ -10,29 +10,29 @@ import {
   executeToolCall,
   executeWarpGrep,
   formatResult
-} from "../../chunk-XREWLWLA.js";
+} from "../../chunk-4VMIPD4G.js";
 import {
   runWarpGrep
-} from "../../chunk-RRWN62VU.js";
+} from "../../chunk-B3UC7UVA.js";
 import {
   RemoteCommandsProvider
-} from "../../chunk-26QXQFLZ.js";
+} from "../../chunk-PHJQ4N3T.js";
 import {
   normalizeFinishFiles,
   readFinishFiles,
   toolGrep,
   toolListDirectory,
   toolRead
-} from "../../chunk-DCIUCDWJ.js";
+} from "../../chunk-467MCW5R.js";
 import "../../chunk-APP75CBN.js";
 import "../../chunk-5QRN3JNB.js";
 import {
   SYSTEM_PROMPT,
   getSystemPrompt
-} from "../../chunk-Q5AHGIQO.js";
+} from "../../chunk-FMLHRJDF.js";
 import {
   LocalRipgrepProvider
-} from "../../chunk-HBIDSKIQ.js";
+} from "../../chunk-4NI25AKV.js";
 import "../../chunk-G2RSY56Q.js";
 import {
   fixPathRepetition

package/dist/tools/warp_grep/openai.cjs

@@ -319,6 +319,8 @@ I think I have a rough idea, but this is my last turn so I must call the finish
 No commentary outside \`<think>\`. No explanations after tool calls.
 </output_format>
 
+use as all 8 tool calls to be optimal
+
 <finishing_requirements>
 When calling \`finish\`:
 - Include the import section (typically lines 1-20) of each file
@@ -606,10 +608,25 @@ async function toolGrep(provider, args) {
 }
 
 // tools/warp_grep/agent/tools/read.ts
+function isValidRange(start, end) {
+  return typeof start === "number" && typeof end === "number" && Number.isFinite(start) && Number.isFinite(end) && start > 0 && end >= start;
+}
 async function toolRead(provider, args) {
-  if (args.lines && args.lines.length > 0) {
+  if (args.lines && Array.isArray(args.lines) && args.lines.length > 0) {
+    const validRanges = [];
+    for (const range of args.lines) {
+      if (Array.isArray(range) && range.length >= 2 && isValidRange(range[0], range[1])) {
+        validRanges.push([range[0], range[1]]);
+      }
+    }
+    if (validRanges.length === 0) {
+      const res2 = await provider.read({ path: args.path });
+      if (res2.error) return res2.error;
+      if (!res2.lines.length) return "(empty file)";
+      return res2.lines.join("\n");
+    }
     const chunks = [];
-    for (const [start, end] of args.lines) {
+    for (const [start, end] of validRanges) {
       const res2 = await provider.read({ path: args.path, start, end });
       if (res2.error) return res2.error;
       chunks.push(res2.lines.join("\n"));
@@ -645,14 +662,28 @@ async function toolListDirectory(provider, args) {
 }
 
 // tools/warp_grep/agent/tools/finish.ts
+function isValidRange2(range) {
+  return Array.isArray(range) && range.length >= 2 && typeof range[0] === "number" && typeof range[1] === "number" && Number.isFinite(range[0]) && Number.isFinite(range[1]) && range[0] > 0 && range[1] >= range[0];
+}
+function extractValidRanges(lines) {
+  if (!Array.isArray(lines)) return null;
+  const valid = [];
+  for (const range of lines) {
+    if (isValidRange2(range)) {
+      valid.push([range[0], range[1]]);
+    }
+  }
+  return valid.length > 0 ? valid : null;
+}
 async function readFinishFiles(repoRoot, files, reader) {
   const out = [];
   for (const f of files) {
-    if (f.lines === "*") {
+    const validRanges = f.lines === "*" ? null : extractValidRanges(f.lines);
+    if (f.lines === "*" || !validRanges) {
       const lines = await reader(f.path);
       out.push({ path: f.path, ranges: "*", content: lines.join("\n") });
     } else {
-      const ranges = mergeRanges(f.lines);
+      const ranges = mergeRanges(validRanges);
       const chunks = [];
       for (const [s, e] of ranges) {
         const lines = await reader(f.path, s, e);
@@ -1060,7 +1091,7 @@ async function runWarpGrep(config) {
   }
   const parts = ["Relevant context found:"];
   for (const f of finishMeta.files) {
-    const ranges = f.lines === "*" ? "*" : f.lines.map(([s, e]) => `${s}-${e}`).join(", ");
+    const ranges = f.lines === "*" ? "*" : Array.isArray(f.lines) ? f.lines.map(([s, e]) => `${s}-${e}`).join(", ") : "*";
     parts.push(`- ${f.path}: ${ranges}`);
   }
   const payload = parts.join("\n");
@@ -1420,11 +1451,19 @@ Details: ${res.stderr}` : ""}`
       };
     }
     const total = lines.length;
-    let s = params.start ?? 1;
-    let e = Math.min(params.end ?? total, total);
-    if (s > total && total > 0) {
-      s = 1;
-      e = total;
+    const rawStart = params.start;
+    const rawEnd = params.end;
+    let s = 1;
+    let e = total;
+    const startValid = rawStart === void 0 || Number.isFinite(rawStart) && rawStart > 0;
+    const endValid = rawEnd === void 0 || Number.isFinite(rawEnd) && rawEnd > 0;
+    if (startValid && endValid) {
+      s = rawStart ?? 1;
+      e = Math.min(rawEnd ?? total, total);
+      if (s > total && total > 0 || s > e) {
+        s = 1;
+        e = total;
+      }
     }
     const out = [];
     for (let i = s; i <= e; i += 1) {
@@ -1600,8 +1639,13 @@ var RemoteCommandsProvider = class {
    * Read file and add line numbers
    */
   async read(params) {
-    const start = params.start ?? 1;
-    const end = params.end ?? 1e6;
+    const rawStart = params.start;
+    const rawEnd = params.end;
+    const startValid = rawStart === void 0 || Number.isFinite(rawStart) && rawStart > 0;
+    const endValid = rawEnd === void 0 || Number.isFinite(rawEnd) && rawEnd > 0;
+    const rangeValid = startValid && endValid && (rawStart === void 0 || rawEnd === void 0 || rawStart <= rawEnd);
+    const start = rangeValid && rawStart !== void 0 ? rawStart : 1;
+    const end = rangeValid && rawEnd !== void 0 ? rawEnd : 1e6;
     try {
       const stdout = await this.commands.read(params.path, start, end);
       const contentLines = (stdout || "").split("\n");