@wrongstack/plugins 1.0.13 → 1.0.14

package/README.md

@@ -699,7 +699,7 @@ once you're confident in the rewrite.
 ### 22 — `loop-breaker`
 
 **Tools**: `loop_breaker_status`
-**Hooks**: `PreToolUse`
+**Hooks**: `PreToolUse`, `PostToolUse`, `UserPromptSubmit`
 
 Detects repeated tool calls before they execute. It tracks exact-repeat
 streaks and A-B-A-B oscillations, then either blocks the call or injects a
@@ -711,6 +711,7 @@ warning depending on `mode`.
     "loop-breaker": {
       "enabled": true,
       "mode": "block",        // "block" | "warn"
+      "maxSteps": 0,          // unlimited; set a positive value to opt into a cap
       "repeatThreshold": 3,
       "oscillationThreshold": 2
     }

package/dist/index.js

@@ -12465,29 +12465,70 @@ var llm_cache_default = plugin34;
 // src/loop-breaker/index.ts
 import { execFile as execFile8 } from "node:child_process";
 import { isAbsolute as isAbsolute15, relative as relative15 } from "node:path";
-var state32 = {
-  lastFingerprint: null,
-  streak: 0,
-  recent: [],
-  pendingBlockReason: null,
-  lastDiffFingerprint: null,
-  noDiffStreak: 0,
-  lastErrorFingerprint: null,
-  repeatedErrorStreak: 0,
-  invocations: 0,
-  postInvocations: 0,
-  warnings: 0,
-  blocks: 0,
-  oscillationsDetected: 0,
-  stepBudgetBlocks: 0,
-  noDiffWarnings: 0,
-  noDiffBlocks: 0,
-  repeatedErrorWarnings: 0,
-  repeatedErrorBlocks: 0,
-  hookUnregister: null
-};
+var FALLBACK_SESSION = "__default__";
+var MAX_TRACKED_RUNS = 32;
+function createRunState() {
+  return {
+    lastFingerprint: null,
+    streak: 0,
+    recent: [],
+    pendingBlockReason: null,
+    lastDiffFingerprint: null,
+    noDiffStreak: 0,
+    lastErrorFingerprint: null,
+    repeatedErrorStreak: 0,
+    invocations: 0,
+    postInvocations: 0,
+    warnings: 0,
+    blocks: 0,
+    oscillationsDetected: 0,
+    stepBudgetBlocks: 0,
+    noDiffWarnings: 0,
+    noDiffBlocks: 0,
+    repeatedErrorWarnings: 0,
+    repeatedErrorBlocks: 0
+  };
+}
+var state32 = { runs: /* @__PURE__ */ new Map(), hookUnregister: null };
+function sessionKey(sessionId) {
+  return sessionId || FALLBACK_SESSION;
+}
+function storeRun(key, current) {
+  state32.runs.set(key, current);
+  if (state32.runs.size <= MAX_TRACKED_RUNS) return;
+  const oldest = state32.runs.keys().next().value;
+  if (oldest !== void 0 && oldest !== key) state32.runs.delete(oldest);
+}
+function runState(sessionId) {
+  const key = sessionKey(sessionId);
+  let current = state32.runs.get(key);
+  if (!current) {
+    current = createRunState();
+    storeRun(key, current);
+  }
+  return current;
+}
+function resetRun(sessionId) {
+  storeRun(sessionKey(sessionId), createRunState());
+}
+function aggregateRuns() {
+  const total = createRunState();
+  for (const current of state32.runs.values()) {
+    total.invocations += current.invocations;
+    total.postInvocations += current.postInvocations;
+    total.warnings += current.warnings;
+    total.blocks += current.blocks;
+    total.oscillationsDetected += current.oscillationsDetected;
+    total.stepBudgetBlocks += current.stepBudgetBlocks;
+    total.noDiffWarnings += current.noDiffWarnings;
+    total.noDiffBlocks += current.noDiffBlocks;
+    total.repeatedErrorWarnings += current.repeatedErrorWarnings;
+    total.repeatedErrorBlocks += current.repeatedErrorBlocks;
+  }
+  return total;
+}
 var DEFAULTS30 = {
-  enabled: true,
+  enabled: false,
   // Documented contract (feature matrix, plugin description): warn, then
   // block. A warn-only default meant an agent stuck re-issuing the same call
   // was never actually stopped unless the user happened to set any option.
@@ -12495,7 +12536,7 @@ var DEFAULTS30 = {
   warnAfter: 3,
   blockAfter: 5,
   oscillationWindow: 8,
-  maxSteps: 200,
+  maxSteps: 0,
   noDiffWarnAfter: 6,
   noDiffBlockAfter: 10,
   repeatedErrorWarnAfter: 2,
@@ -12521,7 +12562,7 @@ function readConfig30(raw) {
   const rawOsc = r["oscillationWindow"] ?? r["oscillation_window"] ?? r["window"];
   const rawMaxSteps = r["maxSteps"] ?? r["max_steps"] ?? r["stepLimit"] ?? r["step_limit"];
   return {
-    enabled: r["enabled"] !== false,
+    enabled: r["enabled"] === true,
     mode,
     warnAfter,
     blockAfter,
@@ -12634,7 +12675,11 @@ var plugin35 = {
   configSchema: {
     type: "object",
     properties: {
-      enabled: { type: "boolean", default: true, description: "Master switch." },
+      enabled: {
+        type: "boolean",
+        default: false,
+        description: "Opt-in master switch; disabled unless explicitly enabled."
+      },
       mode: {
         type: "string",
         enum: ["warn", "block"],
@@ -12662,8 +12707,8 @@ var plugin35 = {
       maxSteps: {
         type: "number",
         minimum: 0,
-        default: 200,
-        description: "Maximum tool steps before blocking; 0 disables the step budget."
+        default: 0,
+        description: "Optional maximum tool steps before blocking; 0 keeps runs unlimited."
       },
       noDiffWarnAfter: {
         type: "number",
@@ -12698,24 +12743,7 @@ var plugin35 = {
     }
   },
   setup(api) {
-    state32.lastFingerprint = null;
-    state32.streak = 0;
-    state32.recent = [];
-    state32.pendingBlockReason = null;
-    state32.lastDiffFingerprint = null;
-    state32.noDiffStreak = 0;
-    state32.lastErrorFingerprint = null;
-    state32.repeatedErrorStreak = 0;
-    state32.invocations = 0;
-    state32.postInvocations = 0;
-    state32.warnings = 0;
-    state32.blocks = 0;
-    state32.oscillationsDetected = 0;
-    state32.stepBudgetBlocks = 0;
-    state32.noDiffWarnings = 0;
-    state32.noDiffBlocks = 0;
-    state32.repeatedErrorWarnings = 0;
-    state32.repeatedErrorBlocks = 0;
+    state32.runs.clear();
     if (state32.hookUnregister) {
       try {
         state32.hookUnregister();
@@ -12728,56 +12756,57 @@ var plugin35 = {
       if (!cfg.enabled) return;
       const toolName = input.toolName ?? "unknown";
       if (ignoreToolsSet.has(toolName)) return;
-      state32.invocations += 1;
-      if (state32.pendingBlockReason && cfg.mode === "block") {
-        const reason = state32.pendingBlockReason;
-        state32.pendingBlockReason = null;
-        state32.blocks += 1;
+      const current = runState(input.sessionId);
+      if (current.pendingBlockReason && cfg.mode === "block") {
+        const reason = current.pendingBlockReason;
+        current.pendingBlockReason = null;
+        current.blocks += 1;
         api.metrics.counter("blocks");
         return { decision: "block", reason };
       }
-      if (cfg.maxSteps > 0 && state32.invocations > cfg.maxSteps && cfg.mode === "block") {
-        state32.blocks += 1;
-        state32.stepBudgetBlocks += 1;
+      if (cfg.maxSteps > 0 && current.invocations >= cfg.maxSteps && cfg.mode === "block") {
+        current.blocks += 1;
+        current.stepBudgetBlocks += 1;
         api.metrics.counter("blocks");
         api.metrics.counter("step_budget_blocks");
         return {
           decision: "block",
-          reason: `loop-breaker: step budget exceeded (${state32.invocations}/${cfg.maxSteps} tool calls). Stop and report what has been tried instead of continuing an unbounded loop.`
+          reason: `loop-breaker: step budget exceeded (${current.invocations}/${cfg.maxSteps} tool calls). Stop and report what has been tried instead of continuing an unbounded loop.`
         };
       }
+      current.invocations += 1;
       const fp = fingerprint(toolName, input.toolInput);
-      if (fp === state32.lastFingerprint) {
-        state32.streak += 1;
+      if (fp === current.lastFingerprint) {
+        current.streak += 1;
       } else {
-        state32.lastFingerprint = fp;
-        state32.streak = 1;
+        current.lastFingerprint = fp;
+        current.streak = 1;
       }
-      state32.recent.push(fp);
-      if (state32.recent.length > Math.max(cfg.oscillationWindow, 16)) {
-        state32.recent.splice(0, state32.recent.length - Math.max(cfg.oscillationWindow, 16));
+      current.recent.push(fp);
+      if (current.recent.length > Math.max(cfg.oscillationWindow, 16)) {
+        current.recent.splice(0, current.recent.length - Math.max(cfg.oscillationWindow, 16));
       }
-      if (state32.streak >= cfg.blockAfter && cfg.mode === "block") {
-        state32.blocks += 1;
+      if (current.streak >= cfg.blockAfter && cfg.mode === "block") {
+        current.blocks += 1;
         api.metrics.counter("blocks");
         return {
           decision: "block",
-          reason: `loop-breaker: "${toolName}" has been called ${state32.streak} times in a row with identical input. This looks like a runaway loop. Change the input, try a different tool, or explain to the user why repetition is needed. (Disable this guard via config.extensions["loop-breaker"].enabled = false.)`
+          reason: `loop-breaker: "${toolName}" has been called ${current.streak} times in a row with identical input. This looks like a runaway loop. Change the input, try a different tool, or explain to the user why repetition is needed. (Disable this guard via config.extensions["loop-breaker"].enabled = false.)`
         };
       }
-      if (state32.streak >= cfg.warnAfter) {
-        state32.warnings += 1;
+      if (current.streak >= cfg.warnAfter) {
+        current.warnings += 1;
         api.metrics.counter("warnings");
-        const remaining = cfg.mode === "block" ? cfg.blockAfter - state32.streak : null;
+        const remaining = cfg.mode === "block" ? cfg.blockAfter - current.streak : null;
         return {
           decision: "allow",
-          additionalContext: `loop-breaker: "${toolName}" repeated ${state32.streak}x with identical input.` + (remaining !== null && remaining > 0 ? ` It will be BLOCKED after ${remaining} more identical call(s).` : "") + " If the previous result was not what you needed, change the approach instead of retrying."
+          additionalContext: `loop-breaker: "${toolName}" repeated ${current.streak}x with identical input.` + (remaining !== null && remaining > 0 ? ` It will be BLOCKED after ${remaining} more identical call(s).` : "") + " If the previous result was not what you needed, change the approach instead of retrying."
         };
       }
-      if (isOscillating(state32.recent, cfg.oscillationWindow)) {
-        state32.oscillationsDetected += 1;
+      if (isOscillating(current.recent, cfg.oscillationWindow)) {
+        current.oscillationsDetected += 1;
         api.metrics.counter("oscillations");
-        state32.recent = [];
+        current.recent = [];
         return {
           decision: "allow",
           additionalContext: `loop-breaker: the last ${cfg.oscillationWindow} tool calls alternate between two identical calls (A-B-A-B pattern). You appear to be undoing and redoing the same work. Step back and pick a single approach.`
@@ -12789,32 +12818,33 @@ var plugin35 = {
       if (!cfg.enabled) return;
       const toolName = input.toolName ?? "unknown";
       if (ignoreToolsSet.has(toolName)) return;
-      state32.postInvocations += 1;
+      const current = runState(input.sessionId);
+      current.postInvocations += 1;
       if (input.toolResult?.isError) {
         const errorFingerprint = normalizeError(input.toolResult.content);
-        if (errorFingerprint && errorFingerprint === state32.lastErrorFingerprint) {
-          state32.repeatedErrorStreak += 1;
+        if (errorFingerprint && errorFingerprint === current.lastErrorFingerprint) {
+          current.repeatedErrorStreak += 1;
         } else {
-          state32.lastErrorFingerprint = errorFingerprint;
-          state32.repeatedErrorStreak = errorFingerprint ? 1 : 0;
+          current.lastErrorFingerprint = errorFingerprint;
+          current.repeatedErrorStreak = errorFingerprint ? 1 : 0;
         }
-        state32.noDiffStreak = 0;
-        if (cfg.repeatedErrorBlockAfter > 0 && state32.repeatedErrorStreak >= cfg.repeatedErrorBlockAfter && cfg.mode === "block") {
-          state32.repeatedErrorBlocks += 1;
-          state32.pendingBlockReason = `loop-breaker: the same tool error repeated ${state32.repeatedErrorStreak} times. The next tool call is blocked so you can stop, summarize the repeated failure, and change approach.`;
+        current.noDiffStreak = 0;
+        if (cfg.repeatedErrorBlockAfter > 0 && current.repeatedErrorStreak >= cfg.repeatedErrorBlockAfter && cfg.mode === "block") {
+          current.repeatedErrorBlocks += 1;
+          current.pendingBlockReason = `loop-breaker: the same tool error repeated ${current.repeatedErrorStreak} times. The next tool call is blocked so you can stop, summarize the repeated failure, and change approach.`;
         }
-        if (state32.repeatedErrorStreak >= cfg.repeatedErrorWarnAfter) {
-          state32.repeatedErrorWarnings += 1;
+        if (current.repeatedErrorStreak >= cfg.repeatedErrorWarnAfter) {
+          current.repeatedErrorWarnings += 1;
           api.metrics.counter("repeated_error_warnings");
           return {
             decision: "allow",
-            additionalContext: `loop-breaker: same error repeated ${state32.repeatedErrorStreak}x. Do not retry the same command/tool unchanged; inspect the root cause or ask for help.`
+            additionalContext: `loop-breaker: same error repeated ${current.repeatedErrorStreak}x. Do not retry the same command/tool unchanged; inspect the root cause or ask for help.`
           };
         }
         return;
       }
-      state32.lastErrorFingerprint = null;
-      state32.repeatedErrorStreak = 0;
+      current.lastErrorFingerprint = null;
+      current.repeatedErrorStreak = 0;
       if (!MUTATING_TOOLS.has(toolName)) return;
       const toolInput = input.toolInput ?? {};
       const rawTarget = toolInput["path"] ?? toolInput["TargetFile"] ?? toolInput["filePath"] ?? toolInput["targetFile"] ?? toolInput["file_path"] ?? toolInput["destination"] ?? toolInput["file"];
@@ -12826,22 +12856,22 @@ var plugin35 = {
         runtime.signal
       );
       if (diffFingerprint === null) return;
-      if (diffFingerprint === state32.lastDiffFingerprint) {
-        state32.noDiffStreak += 1;
+      if (diffFingerprint === current.lastDiffFingerprint) {
+        current.noDiffStreak += 1;
       } else {
-        state32.lastDiffFingerprint = diffFingerprint;
-        state32.noDiffStreak = 0;
+        current.lastDiffFingerprint = diffFingerprint;
+        current.noDiffStreak = 0;
       }
-      if (cfg.noDiffBlockAfter > 0 && state32.noDiffStreak >= cfg.noDiffBlockAfter && cfg.mode === "block") {
-        state32.noDiffBlocks += 1;
-        state32.pendingBlockReason = `loop-breaker: no diff was produced in the last ${state32.noDiffStreak} mutating step(s). The next tool call is blocked because continued edits are not changing the working tree.`;
+      if (cfg.noDiffBlockAfter > 0 && current.noDiffStreak >= cfg.noDiffBlockAfter && cfg.mode === "block") {
+        current.noDiffBlocks += 1;
+        current.pendingBlockReason = `loop-breaker: no diff was produced in the last ${current.noDiffStreak} mutating step(s). The next tool call is blocked because continued edits are not changing the working tree.`;
       }
-      if (state32.noDiffStreak >= cfg.noDiffWarnAfter) {
-        state32.noDiffWarnings += 1;
+      if (current.noDiffStreak >= cfg.noDiffWarnAfter) {
+        current.noDiffWarnings += 1;
         api.metrics.counter("no_diff_warnings");
         return {
           decision: "allow",
-          additionalContext: `loop-breaker: no diff has changed for ${state32.noDiffStreak} mutating step(s). Stop repeating edits; read the file/status, explain why no change is happening, or choose a different approach.`
+          additionalContext: `loop-breaker: no diff has changed for ${current.noDiffStreak} mutating step(s). Stop repeating edits; read the file/status, explain why no change is happening, or choose a different approach.`
         };
       }
       return;
@@ -12856,9 +12886,18 @@ var plugin35 = {
       timeoutMs: 2e3,
       failurePolicy: "open"
     });
+    const unregisterPrompt = api.registerHook(
+      "UserPromptSubmit",
+      void 0,
+      ((input) => {
+        resetRun(input.sessionId);
+      }),
+      { name: "loop-breaker-turn-reset", failurePolicy: "open" }
+    );
     state32.hookUnregister = () => {
       unregisterPre();
       unregisterPost();
+      unregisterPrompt();
     };
     api.tools.register({
       name: "loop_breaker_status",
@@ -12867,7 +12906,8 @@ var plugin35 = {
       permission: "auto",
       category: "Diagnostics",
       mutating: false,
-      async execute() {
+      async execute(_input, ctx) {
+        const current = runState(ctx?.session?.id);
         return {
           ok: true,
           enabled: cfg.enabled,
@@ -12881,20 +12921,20 @@ var plugin35 = {
           repeatedErrorWarnAfter: cfg.repeatedErrorWarnAfter,
           repeatedErrorBlockAfter: cfg.repeatedErrorBlockAfter,
           ignoreTools: cfg.ignoreTools,
-          currentStreak: state32.streak,
-          noDiffStreak: state32.noDiffStreak,
-          repeatedErrorStreak: state32.repeatedErrorStreak,
+          currentStreak: current.streak,
+          noDiffStreak: current.noDiffStreak,
+          repeatedErrorStreak: current.repeatedErrorStreak,
           counters: {
-            invocations: state32.invocations,
-            postInvocations: state32.postInvocations,
-            warnings: state32.warnings,
-            blocks: state32.blocks,
-            oscillationsDetected: state32.oscillationsDetected,
-            stepBudgetBlocks: state32.stepBudgetBlocks,
-            noDiffWarnings: state32.noDiffWarnings,
-            noDiffBlocks: state32.noDiffBlocks,
-            repeatedErrorWarnings: state32.repeatedErrorWarnings,
-            repeatedErrorBlocks: state32.repeatedErrorBlocks
+            invocations: current.invocations,
+            postInvocations: current.postInvocations,
+            warnings: current.warnings,
+            blocks: current.blocks,
+            oscillationsDetected: current.oscillationsDetected,
+            stepBudgetBlocks: current.stepBudgetBlocks,
+            noDiffWarnings: current.noDiffWarnings,
+            noDiffBlocks: current.noDiffBlocks,
+            repeatedErrorWarnings: current.repeatedErrorWarnings,
+            repeatedErrorBlocks: current.repeatedErrorBlocks
           }
         };
       }
@@ -12915,53 +12955,26 @@ var plugin35 = {
       }
       state32.hookUnregister = null;
     }
-    const final = {
-      invocations: state32.invocations,
-      postInvocations: state32.postInvocations,
-      warnings: state32.warnings,
-      blocks: state32.blocks,
-      oscillationsDetected: state32.oscillationsDetected,
-      stepBudgetBlocks: state32.stepBudgetBlocks,
-      noDiffWarnings: state32.noDiffWarnings,
-      noDiffBlocks: state32.noDiffBlocks,
-      repeatedErrorWarnings: state32.repeatedErrorWarnings,
-      repeatedErrorBlocks: state32.repeatedErrorBlocks
-    };
-    state32.lastFingerprint = null;
-    state32.streak = 0;
-    state32.recent = [];
-    state32.pendingBlockReason = null;
-    state32.lastDiffFingerprint = null;
-    state32.noDiffStreak = 0;
-    state32.lastErrorFingerprint = null;
-    state32.repeatedErrorStreak = 0;
-    state32.invocations = 0;
-    state32.postInvocations = 0;
-    state32.warnings = 0;
-    state32.blocks = 0;
-    state32.oscillationsDetected = 0;
-    state32.stepBudgetBlocks = 0;
-    state32.noDiffWarnings = 0;
-    state32.noDiffBlocks = 0;
-    state32.repeatedErrorWarnings = 0;
-    state32.repeatedErrorBlocks = 0;
+    const final = aggregateRuns();
+    state32.runs.clear();
     api.log.info("loop-breaker: teardown complete", { final });
   },
   async health() {
+    const totals = aggregateRuns();
     return {
       ok: true,
-      message: `loop-breaker: ${state32.invocations} call(s) observed, ${state32.warnings + state32.noDiffWarnings + state32.repeatedErrorWarnings} warning(s), ${state32.blocks} block(s), ${state32.oscillationsDetected} oscillation(s)`,
+      message: `loop-breaker: ${totals.invocations} call(s) observed, ${totals.warnings + totals.noDiffWarnings + totals.repeatedErrorWarnings} warning(s), ${totals.blocks} block(s), ${totals.oscillationsDetected} oscillation(s)`,
       counters: {
-        invocations: state32.invocations,
-        postInvocations: state32.postInvocations,
-        warnings: state32.warnings,
-        blocks: state32.blocks,
-        oscillationsDetected: state32.oscillationsDetected,
-        stepBudgetBlocks: state32.stepBudgetBlocks,
-        noDiffWarnings: state32.noDiffWarnings,
-        noDiffBlocks: state32.noDiffBlocks,
-        repeatedErrorWarnings: state32.repeatedErrorWarnings,
-        repeatedErrorBlocks: state32.repeatedErrorBlocks
+        invocations: totals.invocations,
+        postInvocations: totals.postInvocations,
+        warnings: totals.warnings,
+        blocks: totals.blocks,
+        oscillationsDetected: totals.oscillationsDetected,
+        stepBudgetBlocks: totals.stepBudgetBlocks,
+        noDiffWarnings: totals.noDiffWarnings,
+        noDiffBlocks: totals.noDiffBlocks,
+        repeatedErrorWarnings: totals.repeatedErrorWarnings,
+        repeatedErrorBlocks: totals.repeatedErrorBlocks
       }
     };
   }

package/dist/loop-breaker.js

@@ -1,29 +1,70 @@
 // src/loop-breaker/index.ts
 import { execFile } from "node:child_process";
 import { isAbsolute, relative } from "node:path";
-var state = {
-  lastFingerprint: null,
-  streak: 0,
-  recent: [],
-  pendingBlockReason: null,
-  lastDiffFingerprint: null,
-  noDiffStreak: 0,
-  lastErrorFingerprint: null,
-  repeatedErrorStreak: 0,
-  invocations: 0,
-  postInvocations: 0,
-  warnings: 0,
-  blocks: 0,
-  oscillationsDetected: 0,
-  stepBudgetBlocks: 0,
-  noDiffWarnings: 0,
-  noDiffBlocks: 0,
-  repeatedErrorWarnings: 0,
-  repeatedErrorBlocks: 0,
-  hookUnregister: null
-};
+var FALLBACK_SESSION = "__default__";
+var MAX_TRACKED_RUNS = 32;
+function createRunState() {
+  return {
+    lastFingerprint: null,
+    streak: 0,
+    recent: [],
+    pendingBlockReason: null,
+    lastDiffFingerprint: null,
+    noDiffStreak: 0,
+    lastErrorFingerprint: null,
+    repeatedErrorStreak: 0,
+    invocations: 0,
+    postInvocations: 0,
+    warnings: 0,
+    blocks: 0,
+    oscillationsDetected: 0,
+    stepBudgetBlocks: 0,
+    noDiffWarnings: 0,
+    noDiffBlocks: 0,
+    repeatedErrorWarnings: 0,
+    repeatedErrorBlocks: 0
+  };
+}
+var state = { runs: /* @__PURE__ */ new Map(), hookUnregister: null };
+function sessionKey(sessionId) {
+  return sessionId || FALLBACK_SESSION;
+}
+function storeRun(key, current) {
+  state.runs.set(key, current);
+  if (state.runs.size <= MAX_TRACKED_RUNS) return;
+  const oldest = state.runs.keys().next().value;
+  if (oldest !== void 0 && oldest !== key) state.runs.delete(oldest);
+}
+function runState(sessionId) {
+  const key = sessionKey(sessionId);
+  let current = state.runs.get(key);
+  if (!current) {
+    current = createRunState();
+    storeRun(key, current);
+  }
+  return current;
+}
+function resetRun(sessionId) {
+  storeRun(sessionKey(sessionId), createRunState());
+}
+function aggregateRuns() {
+  const total = createRunState();
+  for (const current of state.runs.values()) {
+    total.invocations += current.invocations;
+    total.postInvocations += current.postInvocations;
+    total.warnings += current.warnings;
+    total.blocks += current.blocks;
+    total.oscillationsDetected += current.oscillationsDetected;
+    total.stepBudgetBlocks += current.stepBudgetBlocks;
+    total.noDiffWarnings += current.noDiffWarnings;
+    total.noDiffBlocks += current.noDiffBlocks;
+    total.repeatedErrorWarnings += current.repeatedErrorWarnings;
+    total.repeatedErrorBlocks += current.repeatedErrorBlocks;
+  }
+  return total;
+}
 var DEFAULTS = {
-  enabled: true,
+  enabled: false,
   // Documented contract (feature matrix, plugin description): warn, then
   // block. A warn-only default meant an agent stuck re-issuing the same call
   // was never actually stopped unless the user happened to set any option.
@@ -31,7 +72,7 @@ var DEFAULTS = {
   warnAfter: 3,
   blockAfter: 5,
   oscillationWindow: 8,
-  maxSteps: 200,
+  maxSteps: 0,
   noDiffWarnAfter: 6,
   noDiffBlockAfter: 10,
   repeatedErrorWarnAfter: 2,
@@ -57,7 +98,7 @@ function readConfig(raw) {
   const rawOsc = r["oscillationWindow"] ?? r["oscillation_window"] ?? r["window"];
   const rawMaxSteps = r["maxSteps"] ?? r["max_steps"] ?? r["stepLimit"] ?? r["step_limit"];
   return {
-    enabled: r["enabled"] !== false,
+    enabled: r["enabled"] === true,
     mode,
     warnAfter,
     blockAfter,
@@ -170,7 +211,11 @@ var plugin = {
   configSchema: {
     type: "object",
     properties: {
-      enabled: { type: "boolean", default: true, description: "Master switch." },
+      enabled: {
+        type: "boolean",
+        default: false,
+        description: "Opt-in master switch; disabled unless explicitly enabled."
+      },
       mode: {
         type: "string",
         enum: ["warn", "block"],
@@ -198,8 +243,8 @@ var plugin = {
       maxSteps: {
         type: "number",
         minimum: 0,
-        default: 200,
-        description: "Maximum tool steps before blocking; 0 disables the step budget."
+        default: 0,
+        description: "Optional maximum tool steps before blocking; 0 keeps runs unlimited."
       },
       noDiffWarnAfter: {
         type: "number",
@@ -234,24 +279,7 @@ var plugin = {
     }
   },
   setup(api) {
-    state.lastFingerprint = null;
-    state.streak = 0;
-    state.recent = [];
-    state.pendingBlockReason = null;
-    state.lastDiffFingerprint = null;
-    state.noDiffStreak = 0;
-    state.lastErrorFingerprint = null;
-    state.repeatedErrorStreak = 0;
-    state.invocations = 0;
-    state.postInvocations = 0;
-    state.warnings = 0;
-    state.blocks = 0;
-    state.oscillationsDetected = 0;
-    state.stepBudgetBlocks = 0;
-    state.noDiffWarnings = 0;
-    state.noDiffBlocks = 0;
-    state.repeatedErrorWarnings = 0;
-    state.repeatedErrorBlocks = 0;
+    state.runs.clear();
     if (state.hookUnregister) {
       try {
         state.hookUnregister();
@@ -264,56 +292,57 @@ var plugin = {
       if (!cfg.enabled) return;
       const toolName = input.toolName ?? "unknown";
       if (ignoreToolsSet.has(toolName)) return;
-      state.invocations += 1;
-      if (state.pendingBlockReason && cfg.mode === "block") {
-        const reason = state.pendingBlockReason;
-        state.pendingBlockReason = null;
-        state.blocks += 1;
+      const current = runState(input.sessionId);
+      if (current.pendingBlockReason && cfg.mode === "block") {
+        const reason = current.pendingBlockReason;
+        current.pendingBlockReason = null;
+        current.blocks += 1;
         api.metrics.counter("blocks");
         return { decision: "block", reason };
       }
-      if (cfg.maxSteps > 0 && state.invocations > cfg.maxSteps && cfg.mode === "block") {
-        state.blocks += 1;
-        state.stepBudgetBlocks += 1;
+      if (cfg.maxSteps > 0 && current.invocations >= cfg.maxSteps && cfg.mode === "block") {
+        current.blocks += 1;
+        current.stepBudgetBlocks += 1;
         api.metrics.counter("blocks");
         api.metrics.counter("step_budget_blocks");
         return {
           decision: "block",
-          reason: `loop-breaker: step budget exceeded (${state.invocations}/${cfg.maxSteps} tool calls). Stop and report what has been tried instead of continuing an unbounded loop.`
+          reason: `loop-breaker: step budget exceeded (${current.invocations}/${cfg.maxSteps} tool calls). Stop and report what has been tried instead of continuing an unbounded loop.`
         };
       }
+      current.invocations += 1;
       const fp = fingerprint(toolName, input.toolInput);
-      if (fp === state.lastFingerprint) {
-        state.streak += 1;
+      if (fp === current.lastFingerprint) {
+        current.streak += 1;
       } else {
-        state.lastFingerprint = fp;
-        state.streak = 1;
+        current.lastFingerprint = fp;
+        current.streak = 1;
       }
-      state.recent.push(fp);
-      if (state.recent.length > Math.max(cfg.oscillationWindow, 16)) {
-        state.recent.splice(0, state.recent.length - Math.max(cfg.oscillationWindow, 16));
+      current.recent.push(fp);
+      if (current.recent.length > Math.max(cfg.oscillationWindow, 16)) {
+        current.recent.splice(0, current.recent.length - Math.max(cfg.oscillationWindow, 16));
       }
-      if (state.streak >= cfg.blockAfter && cfg.mode === "block") {
-        state.blocks += 1;
+      if (current.streak >= cfg.blockAfter && cfg.mode === "block") {
+        current.blocks += 1;
         api.metrics.counter("blocks");
         return {
           decision: "block",
-          reason: `loop-breaker: "${toolName}" has been called ${state.streak} times in a row with identical input. This looks like a runaway loop. Change the input, try a different tool, or explain to the user why repetition is needed. (Disable this guard via config.extensions["loop-breaker"].enabled = false.)`
+          reason: `loop-breaker: "${toolName}" has been called ${current.streak} times in a row with identical input. This looks like a runaway loop. Change the input, try a different tool, or explain to the user why repetition is needed. (Disable this guard via config.extensions["loop-breaker"].enabled = false.)`
         };
       }
-      if (state.streak >= cfg.warnAfter) {
-        state.warnings += 1;
+      if (current.streak >= cfg.warnAfter) {
+        current.warnings += 1;
         api.metrics.counter("warnings");
-        const remaining = cfg.mode === "block" ? cfg.blockAfter - state.streak : null;
+        const remaining = cfg.mode === "block" ? cfg.blockAfter - current.streak : null;
         return {
           decision: "allow",
-          additionalContext: `loop-breaker: "${toolName}" repeated ${state.streak}x with identical input.` + (remaining !== null && remaining > 0 ? ` It will be BLOCKED after ${remaining} more identical call(s).` : "") + " If the previous result was not what you needed, change the approach instead of retrying."
+          additionalContext: `loop-breaker: "${toolName}" repeated ${current.streak}x with identical input.` + (remaining !== null && remaining > 0 ? ` It will be BLOCKED after ${remaining} more identical call(s).` : "") + " If the previous result was not what you needed, change the approach instead of retrying."
         };
       }
-      if (isOscillating(state.recent, cfg.oscillationWindow)) {
-        state.oscillationsDetected += 1;
+      if (isOscillating(current.recent, cfg.oscillationWindow)) {
+        current.oscillationsDetected += 1;
         api.metrics.counter("oscillations");
-        state.recent = [];
+        current.recent = [];
         return {
           decision: "allow",
           additionalContext: `loop-breaker: the last ${cfg.oscillationWindow} tool calls alternate between two identical calls (A-B-A-B pattern). You appear to be undoing and redoing the same work. Step back and pick a single approach.`
@@ -325,32 +354,33 @@ var plugin = {
       if (!cfg.enabled) return;
       const toolName = input.toolName ?? "unknown";
       if (ignoreToolsSet.has(toolName)) return;
-      state.postInvocations += 1;
+      const current = runState(input.sessionId);
+      current.postInvocations += 1;
       if (input.toolResult?.isError) {
         const errorFingerprint = normalizeError(input.toolResult.content);
-        if (errorFingerprint && errorFingerprint === state.lastErrorFingerprint) {
-          state.repeatedErrorStreak += 1;
+        if (errorFingerprint && errorFingerprint === current.lastErrorFingerprint) {
+          current.repeatedErrorStreak += 1;
         } else {
-          state.lastErrorFingerprint = errorFingerprint;
-          state.repeatedErrorStreak = errorFingerprint ? 1 : 0;
+          current.lastErrorFingerprint = errorFingerprint;
+          current.repeatedErrorStreak = errorFingerprint ? 1 : 0;
         }
-        state.noDiffStreak = 0;
-        if (cfg.repeatedErrorBlockAfter > 0 && state.repeatedErrorStreak >= cfg.repeatedErrorBlockAfter && cfg.mode === "block") {
-          state.repeatedErrorBlocks += 1;
-          state.pendingBlockReason = `loop-breaker: the same tool error repeated ${state.repeatedErrorStreak} times. The next tool call is blocked so you can stop, summarize the repeated failure, and change approach.`;
+        current.noDiffStreak = 0;
+        if (cfg.repeatedErrorBlockAfter > 0 && current.repeatedErrorStreak >= cfg.repeatedErrorBlockAfter && cfg.mode === "block") {
+          current.repeatedErrorBlocks += 1;
+          current.pendingBlockReason = `loop-breaker: the same tool error repeated ${current.repeatedErrorStreak} times. The next tool call is blocked so you can stop, summarize the repeated failure, and change approach.`;
         }
-        if (state.repeatedErrorStreak >= cfg.repeatedErrorWarnAfter) {
-          state.repeatedErrorWarnings += 1;
+        if (current.repeatedErrorStreak >= cfg.repeatedErrorWarnAfter) {
+          current.repeatedErrorWarnings += 1;
           api.metrics.counter("repeated_error_warnings");
           return {
             decision: "allow",
-            additionalContext: `loop-breaker: same error repeated ${state.repeatedErrorStreak}x. Do not retry the same command/tool unchanged; inspect the root cause or ask for help.`
+            additionalContext: `loop-breaker: same error repeated ${current.repeatedErrorStreak}x. Do not retry the same command/tool unchanged; inspect the root cause or ask for help.`
           };
         }
         return;
       }
-      state.lastErrorFingerprint = null;
-      state.repeatedErrorStreak = 0;
+      current.lastErrorFingerprint = null;
+      current.repeatedErrorStreak = 0;
       if (!MUTATING_TOOLS.has(toolName)) return;
       const toolInput = input.toolInput ?? {};
       const rawTarget = toolInput["path"] ?? toolInput["TargetFile"] ?? toolInput["filePath"] ?? toolInput["targetFile"] ?? toolInput["file_path"] ?? toolInput["destination"] ?? toolInput["file"];
@@ -362,22 +392,22 @@ var plugin = {
         runtime.signal
       );
       if (diffFingerprint === null) return;
-      if (diffFingerprint === state.lastDiffFingerprint) {
-        state.noDiffStreak += 1;
+      if (diffFingerprint === current.lastDiffFingerprint) {
+        current.noDiffStreak += 1;
       } else {
-        state.lastDiffFingerprint = diffFingerprint;
-        state.noDiffStreak = 0;
+        current.lastDiffFingerprint = diffFingerprint;
+        current.noDiffStreak = 0;
       }
-      if (cfg.noDiffBlockAfter > 0 && state.noDiffStreak >= cfg.noDiffBlockAfter && cfg.mode === "block") {
-        state.noDiffBlocks += 1;
-        state.pendingBlockReason = `loop-breaker: no diff was produced in the last ${state.noDiffStreak} mutating step(s). The next tool call is blocked because continued edits are not changing the working tree.`;
+      if (cfg.noDiffBlockAfter > 0 && current.noDiffStreak >= cfg.noDiffBlockAfter && cfg.mode === "block") {
+        current.noDiffBlocks += 1;
+        current.pendingBlockReason = `loop-breaker: no diff was produced in the last ${current.noDiffStreak} mutating step(s). The next tool call is blocked because continued edits are not changing the working tree.`;
       }
-      if (state.noDiffStreak >= cfg.noDiffWarnAfter) {
-        state.noDiffWarnings += 1;
+      if (current.noDiffStreak >= cfg.noDiffWarnAfter) {
+        current.noDiffWarnings += 1;
         api.metrics.counter("no_diff_warnings");
         return {
           decision: "allow",
-          additionalContext: `loop-breaker: no diff has changed for ${state.noDiffStreak} mutating step(s). Stop repeating edits; read the file/status, explain why no change is happening, or choose a different approach.`
+          additionalContext: `loop-breaker: no diff has changed for ${current.noDiffStreak} mutating step(s). Stop repeating edits; read the file/status, explain why no change is happening, or choose a different approach.`
         };
       }
       return;
@@ -392,9 +422,18 @@ var plugin = {
       timeoutMs: 2e3,
       failurePolicy: "open"
     });
+    const unregisterPrompt = api.registerHook(
+      "UserPromptSubmit",
+      void 0,
+      ((input) => {
+        resetRun(input.sessionId);
+      }),
+      { name: "loop-breaker-turn-reset", failurePolicy: "open" }
+    );
     state.hookUnregister = () => {
       unregisterPre();
       unregisterPost();
+      unregisterPrompt();
     };
     api.tools.register({
       name: "loop_breaker_status",
@@ -403,7 +442,8 @@ var plugin = {
       permission: "auto",
       category: "Diagnostics",
       mutating: false,
-      async execute() {
+      async execute(_input, ctx) {
+        const current = runState(ctx?.session?.id);
         return {
           ok: true,
           enabled: cfg.enabled,
@@ -417,20 +457,20 @@ var plugin = {
           repeatedErrorWarnAfter: cfg.repeatedErrorWarnAfter,
           repeatedErrorBlockAfter: cfg.repeatedErrorBlockAfter,
           ignoreTools: cfg.ignoreTools,
-          currentStreak: state.streak,
-          noDiffStreak: state.noDiffStreak,
-          repeatedErrorStreak: state.repeatedErrorStreak,
+          currentStreak: current.streak,
+          noDiffStreak: current.noDiffStreak,
+          repeatedErrorStreak: current.repeatedErrorStreak,
           counters: {
-            invocations: state.invocations,
-            postInvocations: state.postInvocations,
-            warnings: state.warnings,
-            blocks: state.blocks,
-            oscillationsDetected: state.oscillationsDetected,
-            stepBudgetBlocks: state.stepBudgetBlocks,
-            noDiffWarnings: state.noDiffWarnings,
-            noDiffBlocks: state.noDiffBlocks,
-            repeatedErrorWarnings: state.repeatedErrorWarnings,
-            repeatedErrorBlocks: state.repeatedErrorBlocks
+            invocations: current.invocations,
+            postInvocations: current.postInvocations,
+            warnings: current.warnings,
+            blocks: current.blocks,
+            oscillationsDetected: current.oscillationsDetected,
+            stepBudgetBlocks: current.stepBudgetBlocks,
+            noDiffWarnings: current.noDiffWarnings,
+            noDiffBlocks: current.noDiffBlocks,
+            repeatedErrorWarnings: current.repeatedErrorWarnings,
+            repeatedErrorBlocks: current.repeatedErrorBlocks
           }
         };
       }
@@ -451,53 +491,26 @@ var plugin = {
       }
       state.hookUnregister = null;
     }
-    const final = {
-      invocations: state.invocations,
-      postInvocations: state.postInvocations,
-      warnings: state.warnings,
-      blocks: state.blocks,
-      oscillationsDetected: state.oscillationsDetected,
-      stepBudgetBlocks: state.stepBudgetBlocks,
-      noDiffWarnings: state.noDiffWarnings,
-      noDiffBlocks: state.noDiffBlocks,
-      repeatedErrorWarnings: state.repeatedErrorWarnings,
-      repeatedErrorBlocks: state.repeatedErrorBlocks
-    };
-    state.lastFingerprint = null;
-    state.streak = 0;
-    state.recent = [];
-    state.pendingBlockReason = null;
-    state.lastDiffFingerprint = null;
-    state.noDiffStreak = 0;
-    state.lastErrorFingerprint = null;
-    state.repeatedErrorStreak = 0;
-    state.invocations = 0;
-    state.postInvocations = 0;
-    state.warnings = 0;
-    state.blocks = 0;
-    state.oscillationsDetected = 0;
-    state.stepBudgetBlocks = 0;
-    state.noDiffWarnings = 0;
-    state.noDiffBlocks = 0;
-    state.repeatedErrorWarnings = 0;
-    state.repeatedErrorBlocks = 0;
+    const final = aggregateRuns();
+    state.runs.clear();
     api.log.info("loop-breaker: teardown complete", { final });
   },
   async health() {
+    const totals = aggregateRuns();
     return {
       ok: true,
-      message: `loop-breaker: ${state.invocations} call(s) observed, ${state.warnings + state.noDiffWarnings + state.repeatedErrorWarnings} warning(s), ${state.blocks} block(s), ${state.oscillationsDetected} oscillation(s)`,
+      message: `loop-breaker: ${totals.invocations} call(s) observed, ${totals.warnings + totals.noDiffWarnings + totals.repeatedErrorWarnings} warning(s), ${totals.blocks} block(s), ${totals.oscillationsDetected} oscillation(s)`,
       counters: {
-        invocations: state.invocations,
-        postInvocations: state.postInvocations,
-        warnings: state.warnings,
-        blocks: state.blocks,
-        oscillationsDetected: state.oscillationsDetected,
-        stepBudgetBlocks: state.stepBudgetBlocks,
-        noDiffWarnings: state.noDiffWarnings,
-        noDiffBlocks: state.noDiffBlocks,
-        repeatedErrorWarnings: state.repeatedErrorWarnings,
-        repeatedErrorBlocks: state.repeatedErrorBlocks
+        invocations: totals.invocations,
+        postInvocations: totals.postInvocations,
+        warnings: totals.warnings,
+        blocks: totals.blocks,
+        oscillationsDetected: totals.oscillationsDetected,
+        stepBudgetBlocks: totals.stepBudgetBlocks,
+        noDiffWarnings: totals.noDiffWarnings,
+        noDiffBlocks: totals.noDiffBlocks,
+        repeatedErrorWarnings: totals.repeatedErrorWarnings,
+        repeatedErrorBlocks: totals.repeatedErrorBlocks
       }
     };
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wrongstack/plugins",
-  "version": "1.0.13",
+  "version": "1.0.14",
   "description": "Official WrongStack collection of focused plugins for code quality, security, observability, planning, and agent coordination",
   "license": "MIT",
   "author": "ECOSTACK TECHNOLOGY OÜ",
@@ -303,10 +303,10 @@
     "vitest": "^5.0.0"
   },
   "dependencies": {
-    "@wrongstack/core": "1.0.13",
-    "@wrongstack/plugin-sdk": "1.0.13",
-    "@wrongstack/primitives": "1.0.13",
-    "@wrongstack/tools": "1.0.13"
+    "@wrongstack/core": "1.0.14",
+    "@wrongstack/plugin-sdk": "1.0.14",
+    "@wrongstack/primitives": "1.0.14",
+    "@wrongstack/tools": "1.0.14"
   },
   "scripts": {
     "build": "node ../../scripts/build-package.mjs",