feat(drawbridge): tool error enricher v1.2.0 (P0)

CHANGELOG.md

@@ -2,6 +2,23 @@
 
 All notable changes to Drawbridge are documented here. Follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [1.2.0] — 2026-04-09
+
+### Added
+- **Tool error enricher** — three-hook system that intercepts MCP tool failures, classifies by category/severity, and appends structured recovery guidance to the LLM context window
+- Circuit breaker blocking tool calls after 3 consecutive failures per tool
+- Three-layer template cascade: tool-specific > category > global fallback
+- Sensitive parameter redaction and 800-char enrichment cap
+- Tool name normalization for OpenClaw MCP server prefixes
+- Counter compensation in `tool_result_persist` for content-based error detection
+- 178 extension tests (was 68)
+
+### Fixed
+- Migrated all hook registrations from legacy `api.registerHook()` to typed `api.on()` — hooks were registered but never dispatched in live runtime
+- Changed `gateway:stop` hook name to `gateway_stop` (typed hook system uses underscores)
+- Added `EAI_AGAIN` to server_unreachable error pattern
+- Synchronous handler invariant: `tool_result_persist` no longer returns a Promise
+
 ## [1.1.1] — 2026-04-02
 
 ### Fixed

extensions/drawbridge/__tests__/helpers.ts

@@ -11,7 +11,20 @@ import type { ResolvedConfig } from "../src/config.js";
 import { resolveConfig } from "../src/config.js";
 import type { PluginState } from "../src/pipeline-factory.js";
 import { LogSink } from "../src/audit-sink.js";
-import type { HookContext, BeforeDispatchContext, BeforeDispatchEvent, MessageReceivedEvent, MessageSendingEvent, LlmOutputEvent } from "../src/types/openclaw.js";
+import type {
+  HookContext,
+  BeforeDispatchContext,
+  BeforeDispatchEvent,
+  MessageReceivedEvent,
+  MessageSendingEvent,
+  LlmOutputEvent,
+  ToolResultPersistEvent,
+  ToolResultPersistContext,
+  AfterToolCallEvent,
+  AfterToolCallContext,
+  BeforeToolCallEvent,
+  BeforeToolCallContext,
+} from "../src/types/openclaw.js";
 
 // ---------------------------------------------------------------------------
 // Mock ClawMoat engine
@@ -196,3 +209,68 @@ export function makeLlmOutputEvent(overrides?: Partial<LlmOutputEvent>): LlmOutp
     ...overrides,
   };
 }
+
+// ---------------------------------------------------------------------------
+// Tool error enricher event/context builders
+// ---------------------------------------------------------------------------
+
+export function makeToolResultPersistEvent(
+  overrides?: Partial<ToolResultPersistEvent> & { message?: Record<string, unknown> },
+): ToolResultPersistEvent {
+  return {
+    message: {
+      isError: true,
+      content: [{ type: "text", text: "Request timeout" }],
+    },
+    isSynthetic: false,
+    ...overrides,
+  };
+}
+
+export function makeToolResultPersistCtx(
+  overrides?: Partial<ToolResultPersistContext>,
+): ToolResultPersistContext {
+  return {
+    sessionKey: "test-session-key",
+    toolName: "memory_search",
+    ...overrides,
+  };
+}
+
+export function makeAfterToolCallEvent(
+  overrides?: Partial<AfterToolCallEvent>,
+): AfterToolCallEvent {
+  return {
+    toolName: "memory_search",
+    params: { query: "test query", namespace: "personal" },
+    error: "Request timeout",
+    ...overrides,
+  };
+}
+
+export function makeAfterToolCallCtx(
+  overrides?: Partial<AfterToolCallContext>,
+): AfterToolCallContext {
+  return {
+    sessionKey: "test-session-key",
+    ...overrides,
+  };
+}
+
+export function makeBeforeToolCallEvent(
+  overrides?: Partial<BeforeToolCallEvent>,
+): BeforeToolCallEvent {
+  return {
+    toolName: "memory_search",
+    ...overrides,
+  };
+}
+
+export function makeBeforeToolCallCtx(
+  overrides?: Partial<BeforeToolCallContext>,
+): BeforeToolCallContext {
+  return {
+    sessionKey: "test-session-key",
+    ...overrides,
+  };
+}