@rama_nigg/open-cursor 2.3.5 → 2.3.8

package/README.md

@@ -27,6 +27,9 @@ Add to `~/.config/opencode/opencode.json`:
     "cursor-acp": {
       "name": "Cursor ACP",
       "npm": "@ai-sdk/openai-compatible",
+      "options": {
+        "baseURL": "http://127.0.0.1:32124/v1"
+      },
       "models": {
         "cursor-acp/auto": { "name": "Auto" },
         "cursor-acp/composer-1.5": { "name": "Composer 1.5" },

package/dist/index.js

@@ -17431,30 +17431,32 @@ function createToolLoopGuard(messages, maxRepeat) {
         const successFingerprint = `${toolCall.function.name}|values:${valueSignature}|success`;
         const repeatCount = (counts.get(successFingerprint) ?? 0) + 1;
         counts.set(successFingerprint, repeatCount);
+        const isExplorationTool = EXPLORATION_TOOLS.has(toolCall.function.name.toLowerCase());
+        const effectiveMaxRepeat = isExplorationTool ? maxRepeat * EXPLORATION_LIMIT_MULTIPLIER : maxRepeat;
         const coarseSuccessFingerprint = deriveSuccessCoarseFingerprint(toolCall.function.name, toolCall.function.arguments);
         const coarseRepeatCount = coarseSuccessFingerprint ? (coarseCounts.get(coarseSuccessFingerprint) ?? 0) + 1 : 0;
         if (coarseSuccessFingerprint) {
           coarseCounts.set(coarseSuccessFingerprint, coarseRepeatCount);
         }
-        const coarseTriggered = coarseSuccessFingerprint ? coarseRepeatCount > maxRepeat : false;
+        const coarseTriggered = coarseSuccessFingerprint ? coarseRepeatCount > effectiveMaxRepeat : false;
         return {
           fingerprint: coarseTriggered ? coarseSuccessFingerprint : successFingerprint,
           repeatCount: coarseTriggered ? coarseRepeatCount : repeatCount,
-          maxRepeat,
+          maxRepeat: effectiveMaxRepeat,
           errorClass,
-          triggered: repeatCount > maxRepeat || coarseTriggered,
+          triggered: repeatCount > effectiveMaxRepeat || coarseTriggered,
           tracked: true
         };
       }
       const strictFingerprint = `${toolCall.function.name}|${argShape}|${errorClass}`;
       const coarseFingerprint = `${toolCall.function.name}|${errorClass}`;
-      return evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, counts, coarseCounts, maxRepeat, coarseMaxRepeat);
+      return evaluateWithFingerprints(toolCall.function.name, errorClass, strictFingerprint, coarseFingerprint, counts, coarseCounts, maxRepeat, coarseMaxRepeat);
     },
     evaluateValidation(toolCall, validationSignature) {
       const normalizedSignature = normalizeValidationSignature(validationSignature);
       const strictFingerprint = `${toolCall.function.name}|schema:${normalizedSignature}|validation`;
       const coarseFingerprint = `${toolCall.function.name}|validation`;
-      return evaluateWithFingerprints("validation", strictFingerprint, coarseFingerprint, validationCounts, validationCoarseCounts, maxRepeat, coarseMaxRepeat);
+      return evaluateWithFingerprints(toolCall.function.name, "validation", strictFingerprint, coarseFingerprint, validationCounts, validationCoarseCounts, maxRepeat, coarseMaxRepeat);
     },
     resetFingerprint(fingerprint) {
       counts.delete(fingerprint);
@@ -17685,7 +17687,7 @@ function normalizeValidationSignature(signature) {
   const normalized = signature.trim().toLowerCase();
   return normalized.length > 0 ? normalized : "invalid";
 }
-function evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat, coarseMaxRepeat) {
+function evaluateWithFingerprints(toolName, errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat, coarseMaxRepeat) {
   if (errorClass === "success") {
     return {
       fingerprint: strictFingerprint,
@@ -17698,9 +17700,20 @@ function evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerpri
   }
   const strictRepeatCount = (strictCounts.get(strictFingerprint) ?? 0) + 1;
   strictCounts.set(strictFingerprint, strictRepeatCount);
+  const strictTriggered = strictRepeatCount > maxRepeat;
+  const isExplorationTool = EXPLORATION_TOOLS.has(toolName.toLowerCase());
+  if (isExplorationTool) {
+    return {
+      fingerprint: strictFingerprint,
+      repeatCount: strictRepeatCount,
+      maxRepeat,
+      errorClass,
+      triggered: strictTriggered,
+      tracked: true
+    };
+  }
   const coarseRepeatCount = (coarseCounts.get(coarseFingerprint) ?? 0) + 1;
   coarseCounts.set(coarseFingerprint, coarseRepeatCount);
-  const strictTriggered = strictRepeatCount > maxRepeat;
   const coarseTriggered = coarseRepeatCount > coarseMaxRepeat;
   const preferCoarseFingerprint = coarseTriggered && !strictTriggered;
   return {
@@ -17791,7 +17804,7 @@ function containsAny(text, patterns) {
 function isRecord4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-var UNKNOWN_AS_SUCCESS_TOOLS, COARSE_LIMIT_MULTIPLIER = 3;
+var UNKNOWN_AS_SUCCESS_TOOLS, EXPLORATION_TOOLS, COARSE_LIMIT_MULTIPLIER = 3, EXPLORATION_LIMIT_MULTIPLIER = 5;
 var init_tool_loop_guard = __esm(() => {
   UNKNOWN_AS_SUCCESS_TOOLS = new Set([
     "bash",
@@ -17803,9 +17816,19 @@ var init_tool_loop_guard = __esm(() => {
     "ls",
     "glob",
     "stat",
-    "webfetch",
     "mkdir",
-    "rm"
+    "rm",
+    "webfetch",
+    "semsearch",
+    "readlints"
+  ]);
+  EXPLORATION_TOOLS = new Set([
+    "read",
+    "grep",
+    "glob",
+    "ls",
+    "stat",
+    "semsearch"
   ]);
 });
 

package/dist/plugin-entry.js

@@ -17431,30 +17431,32 @@ function createToolLoopGuard(messages, maxRepeat) {
         const successFingerprint = `${toolCall.function.name}|values:${valueSignature}|success`;
         const repeatCount = (counts.get(successFingerprint) ?? 0) + 1;
         counts.set(successFingerprint, repeatCount);
+        const isExplorationTool = EXPLORATION_TOOLS.has(toolCall.function.name.toLowerCase());
+        const effectiveMaxRepeat = isExplorationTool ? maxRepeat * EXPLORATION_LIMIT_MULTIPLIER : maxRepeat;
         const coarseSuccessFingerprint = deriveSuccessCoarseFingerprint(toolCall.function.name, toolCall.function.arguments);
         const coarseRepeatCount = coarseSuccessFingerprint ? (coarseCounts.get(coarseSuccessFingerprint) ?? 0) + 1 : 0;
         if (coarseSuccessFingerprint) {
           coarseCounts.set(coarseSuccessFingerprint, coarseRepeatCount);
         }
-        const coarseTriggered = coarseSuccessFingerprint ? coarseRepeatCount > maxRepeat : false;
+        const coarseTriggered = coarseSuccessFingerprint ? coarseRepeatCount > effectiveMaxRepeat : false;
         return {
           fingerprint: coarseTriggered ? coarseSuccessFingerprint : successFingerprint,
           repeatCount: coarseTriggered ? coarseRepeatCount : repeatCount,
-          maxRepeat,
+          maxRepeat: effectiveMaxRepeat,
           errorClass,
-          triggered: repeatCount > maxRepeat || coarseTriggered,
+          triggered: repeatCount > effectiveMaxRepeat || coarseTriggered,
           tracked: true
         };
       }
       const strictFingerprint = `${toolCall.function.name}|${argShape}|${errorClass}`;
       const coarseFingerprint = `${toolCall.function.name}|${errorClass}`;
-      return evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, counts, coarseCounts, maxRepeat, coarseMaxRepeat);
+      return evaluateWithFingerprints(toolCall.function.name, errorClass, strictFingerprint, coarseFingerprint, counts, coarseCounts, maxRepeat, coarseMaxRepeat);
     },
     evaluateValidation(toolCall, validationSignature) {
       const normalizedSignature = normalizeValidationSignature(validationSignature);
       const strictFingerprint = `${toolCall.function.name}|schema:${normalizedSignature}|validation`;
       const coarseFingerprint = `${toolCall.function.name}|validation`;
-      return evaluateWithFingerprints("validation", strictFingerprint, coarseFingerprint, validationCounts, validationCoarseCounts, maxRepeat, coarseMaxRepeat);
+      return evaluateWithFingerprints(toolCall.function.name, "validation", strictFingerprint, coarseFingerprint, validationCounts, validationCoarseCounts, maxRepeat, coarseMaxRepeat);
     },
     resetFingerprint(fingerprint) {
       counts.delete(fingerprint);
@@ -17685,7 +17687,7 @@ function normalizeValidationSignature(signature) {
   const normalized = signature.trim().toLowerCase();
   return normalized.length > 0 ? normalized : "invalid";
 }
-function evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat, coarseMaxRepeat) {
+function evaluateWithFingerprints(toolName, errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat, coarseMaxRepeat) {
   if (errorClass === "success") {
     return {
       fingerprint: strictFingerprint,
@@ -17698,9 +17700,20 @@ function evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerpri
   }
   const strictRepeatCount = (strictCounts.get(strictFingerprint) ?? 0) + 1;
   strictCounts.set(strictFingerprint, strictRepeatCount);
+  const strictTriggered = strictRepeatCount > maxRepeat;
+  const isExplorationTool = EXPLORATION_TOOLS.has(toolName.toLowerCase());
+  if (isExplorationTool) {
+    return {
+      fingerprint: strictFingerprint,
+      repeatCount: strictRepeatCount,
+      maxRepeat,
+      errorClass,
+      triggered: strictTriggered,
+      tracked: true
+    };
+  }
   const coarseRepeatCount = (coarseCounts.get(coarseFingerprint) ?? 0) + 1;
   coarseCounts.set(coarseFingerprint, coarseRepeatCount);
-  const strictTriggered = strictRepeatCount > maxRepeat;
   const coarseTriggered = coarseRepeatCount > coarseMaxRepeat;
   const preferCoarseFingerprint = coarseTriggered && !strictTriggered;
   return {
@@ -17791,7 +17804,7 @@ function containsAny(text, patterns) {
 function isRecord4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-var UNKNOWN_AS_SUCCESS_TOOLS, COARSE_LIMIT_MULTIPLIER = 3;
+var UNKNOWN_AS_SUCCESS_TOOLS, EXPLORATION_TOOLS, COARSE_LIMIT_MULTIPLIER = 3, EXPLORATION_LIMIT_MULTIPLIER = 5;
 var init_tool_loop_guard = __esm(() => {
   UNKNOWN_AS_SUCCESS_TOOLS = new Set([
     "bash",
@@ -17803,9 +17816,19 @@ var init_tool_loop_guard = __esm(() => {
     "ls",
     "glob",
     "stat",
-    "webfetch",
     "mkdir",
-    "rm"
+    "rm",
+    "webfetch",
+    "semsearch",
+    "readlints"
+  ]);
+  EXPLORATION_TOOLS = new Set([
+    "read",
+    "grep",
+    "glob",
+    "ls",
+    "stat",
+    "semsearch"
   ]);
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rama_nigg/open-cursor",
-  "version": "2.3.5",
+  "version": "2.3.8",
   "description": "No prompt limits. No broken streams. Full thinking + tool support. Your Cursor subscription, properly integrated.",
   "type": "module",
   "main": "dist/plugin-entry.js",

package/src/provider/tool-loop-guard.ts

@@ -10,6 +10,7 @@ type ToolLoopErrorClass =
   | "unknown";
 
 const UNKNOWN_AS_SUCCESS_TOOLS = new Set([
+  // Core filesystem tools
   "bash",
   "shell",
   "read",
@@ -19,9 +20,27 @@ const UNKNOWN_AS_SUCCESS_TOOLS = new Set([
   "ls",
   "glob",
   "stat",
-  "webfetch",
   "mkdir",
   "rm",
+  // Web/network tools
+  "webfetch",
+  // cursor-agent specific tools (passthrough, but should not trigger loop guard)
+  // Discovered via tests/experiments/ harness - see docs/cursor-agent-tools.md
+  "semsearch",   // semantic code search
+  "readlints",   // lint/diagnostic reader
+]);
+
+// Exploratory tools that commonly iterate over many files/patterns.
+// These are exempt from COARSE fingerprint tracking (tool|errorClass) to allow
+// legitimate multi-file exploration. Strict fingerprints (tool|args|errorClass)
+// still apply to catch identical repeated failures.
+const EXPLORATION_TOOLS = new Set([
+  "read",
+  "grep",
+  "glob",
+  "ls",
+  "stat",
+  "semsearch",
 ]);
 
 export interface ToolLoopGuardDecision {
@@ -56,6 +75,7 @@ export function parseToolLoopMaxRepeat(
 // to allow legitimate exploration across different files/targets while still
 // catching spray-and-pray patterns.
 const COARSE_LIMIT_MULTIPLIER = 3;
+const EXPLORATION_LIMIT_MULTIPLIER = 5;
 
 export function createToolLoopGuard(
   messages: Array<unknown>,
@@ -94,6 +114,16 @@ export function createToolLoopGuard(
         const repeatCount = (counts.get(successFingerprint) ?? 0) + 1;
         counts.set(successFingerprint, repeatCount);
 
+        // Exploration tools (read, grep, glob, etc.) get a higher limit because
+        // re-reading the same file across turns is legitimate behavior (verifying
+        // edits, checking state, etc.). Use 5x multiplier for these tools.
+        const isExplorationTool = EXPLORATION_TOOLS.has(
+          toolCall.function.name.toLowerCase(),
+        );
+        const effectiveMaxRepeat = isExplorationTool
+          ? maxRepeat * EXPLORATION_LIMIT_MULTIPLIER
+          : maxRepeat;
+
         // Some tools (notably edit/write) can get stuck in "successful" loops where
         // the model keeps re-issuing the same operation with slightly different
         // content (e.g. trailing newline differences). Track a coarse signature for
@@ -110,14 +140,14 @@ export function createToolLoopGuard(
           coarseCounts.set(coarseSuccessFingerprint, coarseRepeatCount);
         }
         const coarseTriggered = coarseSuccessFingerprint
-          ? coarseRepeatCount > maxRepeat
+          ? coarseRepeatCount > effectiveMaxRepeat
           : false;
         return {
           fingerprint: coarseTriggered ? coarseSuccessFingerprint! : successFingerprint,
           repeatCount: coarseTriggered ? coarseRepeatCount : repeatCount,
-          maxRepeat,
+          maxRepeat: effectiveMaxRepeat,
           errorClass,
-          triggered: repeatCount > maxRepeat || coarseTriggered,
+          triggered: repeatCount > effectiveMaxRepeat || coarseTriggered,
           tracked: true,
         };
       }
@@ -125,6 +155,7 @@ export function createToolLoopGuard(
       const coarseFingerprint = `${toolCall.function.name}|${errorClass}`;
 
       return evaluateWithFingerprints(
+        toolCall.function.name,
         errorClass,
         strictFingerprint,
         coarseFingerprint,
@@ -140,6 +171,7 @@ export function createToolLoopGuard(
       const strictFingerprint = `${toolCall.function.name}|schema:${normalizedSignature}|validation`;
       const coarseFingerprint = `${toolCall.function.name}|validation`;
       return evaluateWithFingerprints(
+        toolCall.function.name,
         "validation",
         strictFingerprint,
         coarseFingerprint,
@@ -456,6 +488,7 @@ function normalizeValidationSignature(signature: string): string {
 }
 
 function evaluateWithFingerprints(
+  toolName: string,
   errorClass: ToolLoopErrorClass,
   strictFingerprint: string,
   coarseFingerprint: string,
@@ -477,9 +510,22 @@ function evaluateWithFingerprints(
 
   const strictRepeatCount = (strictCounts.get(strictFingerprint) ?? 0) + 1;
   strictCounts.set(strictFingerprint, strictRepeatCount);
+  const strictTriggered = strictRepeatCount > maxRepeat;
+
+  const isExplorationTool = EXPLORATION_TOOLS.has(toolName.toLowerCase());
+  if (isExplorationTool) {
+    return {
+      fingerprint: strictFingerprint,
+      repeatCount: strictRepeatCount,
+      maxRepeat,
+      errorClass,
+      triggered: strictTriggered,
+      tracked: true,
+    };
+  }
+
   const coarseRepeatCount = (coarseCounts.get(coarseFingerprint) ?? 0) + 1;
   coarseCounts.set(coarseFingerprint, coarseRepeatCount);
-  const strictTriggered = strictRepeatCount > maxRepeat;
   const coarseTriggered = coarseRepeatCount > coarseMaxRepeat;
   const preferCoarseFingerprint = coarseTriggered && !strictTriggered;
   return {