failproofai 1.0.4-beta.1 → 1.0.4-beta.2

package/src/hooks/resolve-transcript-path.ts

@@ -31,8 +31,6 @@ import { findCodexTranscript } from "../../lib/codex-sessions";
 import { findCopilotTranscript } from "../../lib/copilot-sessions";
 import { findCursorTranscript } from "../../lib/cursor-sessions";
 import { findPiTranscript } from "../../lib/pi-sessions";
-import { findGrokTranscript } from "../../lib/grok-sessions";
-import { findQwenTranscript } from "../../lib/qwen-sessions";
 import { findFactoryTranscript } from "../../lib/factory-sessions";
 import { findAntigravityTranscript } from "../../lib/antigravity-sessions";
 import type { IntegrationType } from "./types";
@@ -76,14 +74,6 @@ export function resolveTranscriptPath(
       // its live-hook payload carries no transcript file, so hand back a virtual
       // path (like devin) — audit/download read the DB directly.
       return `goose-db://${sessionId}`;
-    case "grok":
-      // grok writes real JSONL at ~/.grok/sessions/<percent-encoded-cwd>/
-      // <sessionId>/chat_history.jsonl.
-      return findGrokTranscript(sessionId) ?? undefined;
-    case "qwen":
-      // qwen writes real JSONL at ~/.qwen/projects/<encoded-cwd>/chats/
-      // <sessionId>.jsonl (note the extra `chats/` level vs Claude/Factory).
-      return findQwenTranscript(sessionId) ?? undefined;
     case "opencode":
       return `opencode-db://${sessionId}`;
     case "hermes":

package/src/hooks/tool-name-canonicalize.ts

@@ -25,14 +25,7 @@ import {
   ANTIGRAVITY_TOOL_INPUT_MAP,
   GOOSE_TOOL_MAP,
   GOOSE_TOOL_INPUT_MAP,
-  GROK_TOOL_MAP,
-  GROK_TOOL_INPUT_MAP,
-  QWEN_TOOL_MAP,
-  ORI_TOOL_MAP,
-  ORI_TOOL_INPUT_MAP,
-  CLINE_TOOL_MAP,
 } from "./types";
-import { canonicalizeClineToolInput, applyPatchFilePaths } from "./batch-expand";
 
 /**
  * Canonicalize a per-CLI tool name to the Claude PascalCase form that builtin
@@ -63,18 +56,6 @@ export function canonicalizeToolName(
   // Goose: shell→Bash, write/edit/view→file ops, todo__todo_write→TodoWrite, …
   // (verified live against goose v1.43.0). Handles bare + `<ext>__<tool>` names.
   if (cli === "goose") return GOOSE_TOOL_MAP[raw] ?? raw;
-  // grok: run_terminal_command→Bash, write/read_file/search_replace→file ops, …
-  // (every entry observed on the wire against grok 1.0.3).
-  if (cli === "grok") return GROK_TOOL_MAP[raw] ?? raw;
-  // qwen: run_shell_command→Bash, write_file/read_file/edit→file ops, …
-  // (verified live against qwen-code 0.21.12).
-  if (cli === "qwen") return QWEN_TOOL_MAP[raw] ?? raw;
-  // Ori: tool ids arrive lowercase (bash/read/write/edit/glob/grep) — all six
-  // captured at the live approval gate on ori 0.12.0+68f9a36.
-  if (cli === "ori") return ORI_TOOL_MAP[raw] ?? raw;
-  // Cline: run_commands/read_files/search_codebase/apply_patch — all four
-  // observed live on cline v3.0.60's PreToolUse payload.
-  if (cli === "cline") return CLINE_TOOL_MAP[raw] ?? raw;
   return raw;
 }
 
@@ -114,26 +95,6 @@ export function canonicalizeToolInput(
   // Goose file tools (write/edit/view) deliver the path as `path`, read_image as
   // `source`; map to `file_path` so path builtins fire (verified goose v1.43.0).
   else if (cli === "goose") perToolMap = GOOSE_TOOL_INPUT_MAP[toolName];
-  // grok's read_file delivers `target_file` and list_dir `target_directory`;
-  // every other grok tool is already canonical. Without the Read entry a live
-  // `.env` read walks past block-env-files (verified grok 1.0.3). qwen needs no
-  // entry at all — all six of its tools deliver canonical keys.
-  else if (cli === "grok") perToolMap = GROK_TOOL_INPUT_MAP[toolName];
-  // Ori: read/write deliver the path as `path`; bash/glob/grep are already
-  // canonical. `edit` is special — its path lives INSIDE the patch blob and is
-  // derived below rather than renamed, because there is no key to rename.
-  else if (cli === "ori") {
-    if (toolName === "Edit") return canonicalizeOriEditInput(rawInput as Record<string, unknown>);
-    perToolMap = ORI_TOOL_INPUT_MAP[toolName];
-  }
-  // Cline has NO key-rename map: every tool is batch- or blob-shaped, so this
-  // is a value-SHAPE transform. It is the SAFETY NET for the paths that do not
-  // fan out (PostToolUse, audit replay, fail-closed shaping); PreToolUse
-  // enforces through expandBatchToolInput instead, because a collapse cannot
-  // make an anchored regex match every element.
-  else if (cli === "cline") {
-    return canonicalizeClineToolInput(toolName, rawInput as Record<string, unknown>);
-  }
   if (!perToolMap) return rawInput;
   const out: Record<string, unknown> = {};
   for (const [k, v] of Object.entries(rawInput as Record<string, unknown>)) {
@@ -141,28 +102,3 @@ export function canonicalizeToolInput(
   }
   return out;
 }
-
-/**
- * Recover the file path(s) from an OpenAI apply_patch blob.
- *
- * Kept as the ori-facing NAME so existing callers and
- * __tests__/hooks/ori-canonicalize.test.ts keep working, but the implementation
- * now lives in batch-expand.ts and is shared: ori's `edit` and cline's
- * `apply_patch` carry byte-identical blobs, and one implementation is what
- * stops a third CLI adding a third regex.
- *
- * NOTE for a follow-up: `splitApplyPatch` (same module) now returns per-file
- * old/new text as well, which closes ori's documented multi-file KNOWN GAP —
- * and a worse one the gap comment understates, that ori's Edit sets NO
- * old_string/new_string at all, so block-secrets-write can never fire on it.
- * Deliberately NOT changed here: ori's behaviour stays as shipped in this PR.
- */
-export const oriPatchFilePaths = applyPatchFilePaths;
-
-function canonicalizeOriEditInput(input: Record<string, unknown>): Record<string, unknown> {
-  const patch = input.patch;
-  if (typeof patch !== "string") return input;
-  const paths = oriPatchFilePaths(patch);
-  if (paths.length === 0) return input;
-  return { ...input, file_path: paths[0], ori_patch_files: paths };
-}