@fusengine/harness 0.1.91 → 0.1.92

package/src/runtime/lifecycle/aipilot/dispatch-aipilot.ts

@@ -90,7 +90,13 @@ export async function dispatchAipilot(event: string, payload: Record<string, unk
   if (event === "SubagentStop") return onSubagentStop(payload, cwd, home, id);
   // Stop too: Codex emits no SessionEnd, so its ai-pilot hooks.json wires Stop here as the sole analytics-flush trigger — reusing the SessionEnd handler verbatim (codex-plugins/docs/reference/hooks.md).
   if (event === "SessionEnd" || event === "Stop") { await cacheAnalyticsSave(home, now); return ""; }
-  if (event === "PreToolUse") return docCacheGate(payload, cwd, now, home, id);
+  // "BeforeMCPExecution" is a Cursor-only lifecycle literal (produced solely
+  // by cursorEventContract in adapters/cursor/events.ts); asyncScopeStdout
+  // forwards every non-Cursor id's `hook_event_name` RAW and unvalidated, so
+  // gating on the literal alone would let a claude-code/codex payload that
+  // happens to carry this exact string reach docCacheGate — the guard must
+  // be structural (`id === "cursor"`), never event-name-only.
+  if (event === "PreToolUse" || (id === "cursor" && event === "BeforeMCPExecution")) return docCacheGate(payload, cwd, now, home, id);
   return null;
 }
 

package/src/runtime/lifecycle/failure-lesson.ts

@@ -5,8 +5,12 @@
  * matches the failure message — reusing the PreToolUse {@link lessonFor} index and
  * its cooldown (idempotent under the ~11-process fan-out). Fail-open throughout.
  *
- * Claude-Code-only: no equivalent `PostToolUseFailure` hook exists on Codex or
- * Hermes, so this handler is never reached through those adapters.
+ * Claude-Code-only in the sense that no equivalent `PostToolUseFailure` hook
+ * exists on Codex or Hermes — but Cursor's own `postToolUseFailure` DOES
+ * arrive here too, via `lifecycle-bridge.ts`'s `lifecycleStdout` translating
+ * the wire event name and forwarding to `dispatchLifecycle`'s
+ * `"PostToolUseFailure"` case (see `handle.ts`'s `cursorRawPayloadProjection`
+ * for how `data.tool_name` below arrives already canonicalized on Cursor).
  * @packageDocumentation
  */
 import { homedir } from "node:os";

package/src/runtime/lifecycle/rules-root.ts

@@ -2,8 +2,15 @@
  * Dynamic rules-plugin root resolution. The historical `CLAUDE_PLUGIN_ROOT ??
  * cwd` chain only worked when the harness exported the plugin root — Kimi
  * injects `KIMI_PLUGIN_ROOT` instead, and a bare cwd fallback never held a
- * `rules/` dir. Resolution order (first hit wins):
- * 1. `CLAUDE_PLUGIN_ROOT` (claude-code/codex plugin-declared hooks);
+ * `rules/` dir.
+ *
+ * `id === "cursor"` is resolved by a SEPARATE branch (`resolveCursorPluginRoot`)
+ * before any of the below, because Cursor's env contract differs from the
+ * other harnesses (see `../../adapters/cursor/plugin-root.ts`) — it is never
+ * folded into the shared switch. For every other id, resolution order (first
+ * hit wins, unchanged):
+ * 1. `CLAUDE_PLUGIN_ROOT` (read for ALL non-cursor ids, historical quirk —
+ *    frozen by non-regression tests, do not "fix" without an explicit ask);
  * 2. `KIMI_PLUGIN_ROOT` (kimi plugin-declared hooks);
  * 3. Per-harness install probe (claude marketplace, codex versioned cache,
  *    kimi managed plugins) — first `<plugin>/rules` dir whose plugin folder
@@ -14,6 +21,7 @@ import { existsSync, readdirSync } from "node:fs";
 import { homedir } from "node:os";
 import { join } from "node:path";
 import { maxSemver } from "../../util/semver";
+import { resolveCursorPluginRoot } from "../../adapters/cursor/plugin-root";
 
 /** Immediate child dir names of `dir`, or [] when unreadable. */
 function children(dir: string): string[] {
@@ -72,6 +80,14 @@ export function resolveRulesRoot(
   cwd: string,
   env: Record<string, string | undefined> = process.env,
 ): string {
+  if (id === "cursor") {
+    const result = resolveCursorPluginRoot(env, cwd);
+    if (result.root) return result.root;
+    process.stderr.write(
+      `[fuse-harness] cursor: no plugin root proven (checked: ${result.checked.join("; ")}); rules root falls back to ${cwd}\n`,
+    );
+    return cwd;
+  }
   if (env.CLAUDE_PLUGIN_ROOT) return env.CLAUDE_PLUGIN_ROOT;
   if (env.KIMI_PLUGIN_ROOT) return env.KIMI_PLUGIN_ROOT;
   const home = env.HOME ?? homedir();