@rama_nigg/open-cursor 2.3.6 → 2.3.9

package/README.md

@@ -154,7 +154,7 @@ flowchart TB
 Default mode: `CURSOR_ACP_TOOL_LOOP_MODE=opencode`. Legacy `proxy-exec` still available. Details: [docs/architecture/runtime-tool-loop.md](docs/architecture/runtime-tool-loop.md).
 
 ## Alternatives
-
+THERE is currently not a single perfect plugin for cursor in opencode, my advice is stick with what is the LEAST worst option for you.
 |                   |        open-cursor         | [yet-another-opencode-cursor-auth](https://github.com/Yukaii/yet-another-opencode-cursor-auth) | [opencode-cursor-auth](https://github.com/POSO-PocketSolutions/opencode-cursor-auth) | [cursor-opencode-auth](https://github.com/R44VC0RP/cursor-opencode-auth) |
 | ----------------- | :------------------------: | :--------------------------------------------------------------------------------------------: | :----------------------------------------------------------------------------------: | :----------------------------------------------------------------------: |
 | **Architecture**  | HTTP proxy via cursor-agent |                                       Direct Connect-RPC                                       |                             HTTP proxy via cursor-agent                              |                       Direct Connect-RPC/protobuf                        |

package/dist/index.js

@@ -17431,18 +17431,20 @@ 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
         };
       }
@@ -17802,7 +17804,7 @@ function containsAny(text, patterns) {
 function isRecord4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-var UNKNOWN_AS_SUCCESS_TOOLS, EXPLORATION_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",
@@ -17826,7 +17828,9 @@ var init_tool_loop_guard = __esm(() => {
     "glob",
     "ls",
     "stat",
-    "semsearch"
+    "semsearch",
+    "bash",
+    "shell"
   ]);
 });
 

package/dist/plugin-entry.js

@@ -17431,18 +17431,20 @@ 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
         };
       }
@@ -17802,7 +17804,7 @@ function containsAny(text, patterns) {
 function isRecord4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-var UNKNOWN_AS_SUCCESS_TOOLS, EXPLORATION_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",
@@ -17826,7 +17828,9 @@ var init_tool_loop_guard = __esm(() => {
     "glob",
     "ls",
     "stat",
-    "semsearch"
+    "semsearch",
+    "bash",
+    "shell"
   ]);
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rama_nigg/open-cursor",
-  "version": "2.3.6",
+  "version": "2.3.9",
   "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

@@ -41,6 +41,8 @@ const EXPLORATION_TOOLS = new Set([
   "ls",
   "stat",
   "semsearch",
+  "bash",
+  "shell",
 ]);
 
 export interface ToolLoopGuardDecision {
@@ -75,6 +77,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>,
@@ -113,6 +116,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
@@ -129,14 +142,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,
         };
       }