@rama_nigg/open-cursor 2.3.4 → 2.3.6

package/README.md

@@ -7,11 +7,6 @@
 
 No prompt limits. No broken streams. Full thinking + tool support in OpenCode. Your Cursor subscription, properly integrated.
 
-## Prerequisites
-
-Required: [OpenCode](https://opencode.ai/) + [cursor-agent](https://cursor.com/) (`curl -fsSL https://cursor.com/install | bash && cursor-agent login`)
-Optional: [Bun](https://bun.sh/) (Options B-F), [Go 1.21+](https://go.dev/) (Option D)
-
 ## Installation
 
 ### Option A — One-line installer
@@ -20,7 +15,8 @@ Optional: [Bun](https://bun.sh/) (Options B-F), [Go 1.21+](https://go.dev/) (Opt
 curl -fsSL https://raw.githubusercontent.com/Nomadcxx/opencode-cursor/main/install.sh | bash
 ```
 
-### Option B — Add to opencode.json
+<details>
+<summary><b>Option B</b> — Add to opencode.json</summary>
 
 Add to `~/.config/opencode/opencode.json`:
 
@@ -31,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" },
@@ -72,8 +71,10 @@ Add to `~/.config/opencode/opencode.json`:
 ```
 
 > Update models anytime: `cursor-agent models`
+</details>
 
-### Option C — npm global + CLI
+<details>
+<summary><b>Option C</b> — npm global + CLI</summary>
 
 ```bash
 npm install -g @rama_nigg/open-cursor
@@ -81,22 +82,28 @@ open-cursor install
 ```
 
 Upgrade: `npm update -g @rama_nigg/open-cursor`
+</details>
 
-### Option D — Go TUI installer
+<details>
+<summary><b>Option D</b> — Go TUI installer</summary>
 
 ```bash
 git clone https://github.com/Nomadcxx/opencode-cursor.git
 cd opencode-cursor
 go build -o ./installer ./cmd/installer && ./installer
 ```
+</details>
 
-### Option E — LLM paste
+<details>
+<summary><b>Option E</b> — LLM paste</summary>
 
 ```
 Install open-cursor for OpenCode: edit ~/.config/opencode/opencode.json, add "@rama_nigg/open-cursor@latest" to "plugin", add a "cursor-acp" provider with npm "@ai-sdk/openai-compatible" and models from `cursor-agent models` prefixed with "cursor-acp/". Auth: `cursor-agent login`. Verify: `opencode models | grep cursor-acp`.
 ```
+</details>
 
-### Option F — Manual (from source)
+<details>
+<summary><b>Option F</b> — Manual (from source)</summary>
 
 ```bash
 git clone https://github.com/Nomadcxx/opencode-cursor.git && cd opencode-cursor
@@ -106,6 +113,7 @@ ln -sf $(pwd)/dist/plugin-entry.js ~/.config/opencode/plugin/cursor-acp.js
 ```
 
 Add `"cursor-acp"` to the `plugin` array and reuse the provider block from Option B.
+</details>
 
 ## Authentication
 
@@ -170,6 +178,23 @@ Default mode: `CURSOR_ACP_TOOL_LOOP_MODE=opencode`. Legacy `proxy-exec` still av
 
 Debug logging: `CURSOR_ACP_LOG_LEVEL=debug opencode run "your prompt" --model cursor-acp/auto`
 
+## Roadmap
+
+```mermaid
+flowchart LR
+    P1[/Stabilise/] --> P2[/MCP Server/] --> P3[/Simplify/] --> P4[/ACP + MCP/]
+    
+    style P1 fill:#264653,stroke:#1d3557,color:#fff
+    style P2 fill:#264653,stroke:#1d3557,color:#fff
+    style P3 fill:#495057,stroke:#343a40,color:#adb5bd
+    style P4 fill:#495057,stroke:#343a40,color:#adb5bd
+```
+
+[X] **Stabilise** — Clean up dead code, fix test isolation  
+[ ] **MCP Server** — Expose OpenCode tools via stdio transport  
+[ ] **Simplify** — Rip out serialisation layers  
+[ ] **ACP + MCP** — Structured protocols end-to-end
+
 ## License
 
 BSD-3-Clause

package/dist/index.js

@@ -17408,6 +17408,7 @@ function parseToolLoopMaxRepeat(value) {
   return { value: Math.floor(parsed), valid: true };
 }
 function createToolLoopGuard(messages, maxRepeat) {
+  const coarseMaxRepeat = maxRepeat * COARSE_LIMIT_MULTIPLIER;
   const {
     byCallId,
     latest,
@@ -17447,13 +17448,13 @@ function createToolLoopGuard(messages, maxRepeat) {
       }
       const strictFingerprint = `${toolCall.function.name}|${argShape}|${errorClass}`;
       const coarseFingerprint = `${toolCall.function.name}|${errorClass}`;
-      return evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, counts, coarseCounts, maxRepeat);
+      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);
+      return evaluateWithFingerprints(toolCall.function.name, "validation", strictFingerprint, coarseFingerprint, validationCounts, validationCoarseCounts, maxRepeat, coarseMaxRepeat);
     },
     resetFingerprint(fingerprint) {
       counts.delete(fingerprint);
@@ -17684,7 +17685,7 @@ function normalizeValidationSignature(signature) {
   const normalized = signature.trim().toLowerCase();
   return normalized.length > 0 ? normalized : "invalid";
 }
-function evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat) {
+function evaluateWithFingerprints(toolName, errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat, coarseMaxRepeat) {
   if (errorClass === "success") {
     return {
       fingerprint: strictFingerprint,
@@ -17697,15 +17698,26 @@ 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 > maxRepeat;
+  const coarseTriggered = coarseRepeatCount > coarseMaxRepeat;
   const preferCoarseFingerprint = coarseTriggered && !strictTriggered;
   return {
     fingerprint: preferCoarseFingerprint ? coarseFingerprint : strictFingerprint,
     repeatCount: preferCoarseFingerprint ? coarseRepeatCount : strictRepeatCount,
-    maxRepeat,
+    maxRepeat: preferCoarseFingerprint ? coarseMaxRepeat : maxRepeat,
     errorClass,
     triggered: strictTriggered || coarseTriggered,
     tracked: true
@@ -17790,7 +17802,7 @@ function containsAny(text, patterns) {
 function isRecord4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-var UNKNOWN_AS_SUCCESS_TOOLS;
+var UNKNOWN_AS_SUCCESS_TOOLS, EXPLORATION_TOOLS, COARSE_LIMIT_MULTIPLIER = 3;
 var init_tool_loop_guard = __esm(() => {
   UNKNOWN_AS_SUCCESS_TOOLS = new Set([
     "bash",
@@ -17802,9 +17814,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

@@ -17408,6 +17408,7 @@ function parseToolLoopMaxRepeat(value) {
   return { value: Math.floor(parsed), valid: true };
 }
 function createToolLoopGuard(messages, maxRepeat) {
+  const coarseMaxRepeat = maxRepeat * COARSE_LIMIT_MULTIPLIER;
   const {
     byCallId,
     latest,
@@ -17447,13 +17448,13 @@ function createToolLoopGuard(messages, maxRepeat) {
       }
       const strictFingerprint = `${toolCall.function.name}|${argShape}|${errorClass}`;
       const coarseFingerprint = `${toolCall.function.name}|${errorClass}`;
-      return evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, counts, coarseCounts, maxRepeat);
+      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);
+      return evaluateWithFingerprints(toolCall.function.name, "validation", strictFingerprint, coarseFingerprint, validationCounts, validationCoarseCounts, maxRepeat, coarseMaxRepeat);
     },
     resetFingerprint(fingerprint) {
       counts.delete(fingerprint);
@@ -17684,7 +17685,7 @@ function normalizeValidationSignature(signature) {
   const normalized = signature.trim().toLowerCase();
   return normalized.length > 0 ? normalized : "invalid";
 }
-function evaluateWithFingerprints(errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat) {
+function evaluateWithFingerprints(toolName, errorClass, strictFingerprint, coarseFingerprint, strictCounts, coarseCounts, maxRepeat, coarseMaxRepeat) {
   if (errorClass === "success") {
     return {
       fingerprint: strictFingerprint,
@@ -17697,15 +17698,26 @@ 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 > maxRepeat;
+  const coarseTriggered = coarseRepeatCount > coarseMaxRepeat;
   const preferCoarseFingerprint = coarseTriggered && !strictTriggered;
   return {
     fingerprint: preferCoarseFingerprint ? coarseFingerprint : strictFingerprint,
     repeatCount: preferCoarseFingerprint ? coarseRepeatCount : strictRepeatCount,
-    maxRepeat,
+    maxRepeat: preferCoarseFingerprint ? coarseMaxRepeat : maxRepeat,
     errorClass,
     triggered: strictTriggered || coarseTriggered,
     tracked: true
@@ -17790,7 +17802,7 @@ function containsAny(text, patterns) {
 function isRecord4(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-var UNKNOWN_AS_SUCCESS_TOOLS;
+var UNKNOWN_AS_SUCCESS_TOOLS, EXPLORATION_TOOLS, COARSE_LIMIT_MULTIPLIER = 3;
 var init_tool_loop_guard = __esm(() => {
   UNKNOWN_AS_SUCCESS_TOOLS = new Set([
     "bash",
@@ -17802,9 +17814,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.4",
+  "version": "2.3.6",
   "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 {
@@ -52,10 +71,16 @@ export function parseToolLoopMaxRepeat(
   return { value: Math.floor(parsed), valid: true };
 }
 
+// Coarse fingerprint (tool|errorClass without args) uses a higher multiplier
+// to allow legitimate exploration across different files/targets while still
+// catching spray-and-pray patterns.
+const COARSE_LIMIT_MULTIPLIER = 3;
+
 export function createToolLoopGuard(
   messages: Array<unknown>,
   maxRepeat: number,
 ): ToolLoopGuard {
+  const coarseMaxRepeat = maxRepeat * COARSE_LIMIT_MULTIPLIER;
   const {
     byCallId,
     latest,
@@ -119,12 +144,14 @@ export function createToolLoopGuard(
       const coarseFingerprint = `${toolCall.function.name}|${errorClass}`;
 
       return evaluateWithFingerprints(
+        toolCall.function.name,
         errorClass,
         strictFingerprint,
         coarseFingerprint,
         counts,
         coarseCounts,
         maxRepeat,
+        coarseMaxRepeat,
       );
     },
 
@@ -133,12 +160,14 @@ 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,
         validationCounts,
         validationCoarseCounts,
         maxRepeat,
+        coarseMaxRepeat,
       );
     },
 
@@ -448,12 +477,14 @@ function normalizeValidationSignature(signature: string): string {
 }
 
 function evaluateWithFingerprints(
+  toolName: string,
   errorClass: ToolLoopErrorClass,
   strictFingerprint: string,
   coarseFingerprint: string,
   strictCounts: Map<string, number>,
   coarseCounts: Map<string, number>,
   maxRepeat: number,
+  coarseMaxRepeat: number,
 ): ToolLoopGuardDecision {
   if (errorClass === "success") {
     return {
@@ -468,15 +499,28 @@ 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 > maxRepeat;
+  const coarseTriggered = coarseRepeatCount > coarseMaxRepeat;
   const preferCoarseFingerprint = coarseTriggered && !strictTriggered;
   return {
     fingerprint: preferCoarseFingerprint ? coarseFingerprint : strictFingerprint,
     repeatCount: preferCoarseFingerprint ? coarseRepeatCount : strictRepeatCount,
-    maxRepeat,
+    maxRepeat: preferCoarseFingerprint ? coarseMaxRepeat : maxRepeat,
     errorClass,
     triggered: strictTriggered || coarseTriggered,
     tracked: true,