@tonyclaw/agent-inspector 4.0.1 → 4.0.3

package/src/mcp/mode.ts

@@ -1,75 +0,0 @@
-export type McpWriteMode = "enabled" | "readonly";
-
-export const MCP_INSTANCE_WRITES_ENV = "AGENT_INSPECTOR_MCP_INSTANCE_WRITES";
-
-function parseFlag(value: string | undefined): boolean | null {
-  if (value === undefined) return null;
-  switch (value.trim().toLowerCase()) {
-    case "1":
-    case "true":
-    case "yes":
-    case "on":
-      return true;
-    case "0":
-    case "false":
-    case "no":
-    case "off":
-      return false;
-    default:
-      return null;
-  }
-}
-
-export function isMcpWriteEnabled(): boolean {
-  const readonlyFlag = parseFlag(process.env["AGENT_INSPECTOR_MCP_READONLY"]);
-  if (readonlyFlag === true) return false;
-
-  const writesFlag = parseFlag(process.env["AGENT_INSPECTOR_MCP_WRITES"]);
-  return writesFlag === true;
-}
-
-export function isMcpInstanceWriteEnabled(): boolean {
-  return process.env[MCP_INSTANCE_WRITES_ENV] === "1";
-}
-
-export function getMcpInstanceModeInfo(): {
-  writeMode: McpWriteMode;
-  writesEnabled: boolean;
-  env: { AGENT_INSPECTOR_MCP_INSTANCE_WRITES: string | null };
-  note: string;
-} {
-  const writesEnabled = isMcpInstanceWriteEnabled();
-  return {
-    writeMode: writesEnabled ? "enabled" : "readonly",
-    writesEnabled,
-    env: {
-      AGENT_INSPECTOR_MCP_INSTANCE_WRITES: process.env[MCP_INSTANCE_WRITES_ENV] ?? null,
-    },
-    note: writesEnabled
-      ? "Named-instance lifecycle tools passed the instance-specific opt-in. General MCP writes must also be enabled."
-      : "Named-instance lifecycle tools require AGENT_INSPECTOR_MCP_INSTANCE_WRITES=1 in addition to general MCP writes.",
-  };
-}
-
-export function getMcpModeInfo(): {
-  writeMode: McpWriteMode;
-  writesEnabled: boolean;
-  env: {
-    AGENT_INSPECTOR_MCP_READONLY: string | null;
-    AGENT_INSPECTOR_MCP_WRITES: string | null;
-  };
-  note: string;
-} {
-  const writesEnabled = isMcpWriteEnabled();
-  return {
-    writeMode: writesEnabled ? "enabled" : "readonly",
-    writesEnabled,
-    env: {
-      AGENT_INSPECTOR_MCP_READONLY: process.env["AGENT_INSPECTOR_MCP_READONLY"] ?? null,
-      AGENT_INSPECTOR_MCP_WRITES: process.env["AGENT_INSPECTOR_MCP_WRITES"] ?? null,
-    },
-    note: writesEnabled
-      ? "MCP write/action tools are enabled explicitly by AGENT_INSPECTOR_MCP_WRITES=1."
-      : "MCP is readonly by default. Read tools/resources still work; set AGENT_INSPECTOR_MCP_WRITES=1 to enable mutations.",
-  };
-}

package/src/mcp/previewExtractor.ts

@@ -1,233 +0,0 @@
-/**
- * Format-parser-driven preview extractor for the MCP `inspector_list_logs`
- * "thick summary" shape. See `openspec/changes/add-mcp-server/design.md`
- * decision D5.
- *
- * Why we re-parse with the format schemas (not slice `rawRequestBody`):
- * a 500-char prefix is almost always JSON envelope + system prompt boilerplate
- * with zero signal about what the user actually asked. The format parsers
- * give us the first text block of the last user message, which is what an
- * agent needs to make a "is this the log I care about?" judgment from
- * `list_logs` alone.
- *
- * Null returns are an expected and frequent outcome; they cover:
- *   - `apiFormat === "unknown"`
- *   - `rawRequestBody` / `responseText` is null
- *   - JSON.parse failure
- *   - schema validation failure
- *   - no qualifying user / assistant message
- *   - the relevant content is non-text (e.g., image-only multimodal block)
- */
-
-import { z } from "zod";
-import type { CapturedLog } from "../contracts";
-import { AnthropicRequestSchema, AnthropicResponseSchema } from "../contracts/anthropic";
-import {
-  OpenAIRequestSchema,
-  OpenAIResponseSchema,
-  OpenAIResponsesRequestSchema,
-  OpenAIResponsesResponseSchema,
-} from "../contracts/openai";
-import { safeGetOwnProperty } from "../lib/objectUtils";
-
-/** Preview length cap, per design D5. */
-const PREVIEW_MAX_CHARS = 500;
-const UnknownArraySchema = z.array(z.unknown());
-
-function truncate(text: string): string {
-  return text.length <= PREVIEW_MAX_CHARS ? text : text.slice(0, PREVIEW_MAX_CHARS);
-}
-
-/**
- * Safely extract the first text from an Anthropic `Message.content` (string
- * or array of blocks). Returns `null` for non-text-only or empty content.
- */
-function firstAnthropicText(
-  content: string | ReadonlyArray<{ type: string; text?: string }>,
-): string | null {
-  if (typeof content === "string") {
-    return content.length > 0 ? content : null;
-  }
-  for (const block of content) {
-    if (block.type === "text" && typeof block.text === "string" && block.text.length > 0) {
-      return block.text;
-    }
-  }
-  return null;
-}
-
-/**
- * Safely extract the first text from an OpenAI message content (string or
- * array of `{type:"text"|"image_url",...}`). Returns `null` for non-text /
- * empty content.
- */
-function firstOpenAIText(
-  content: string | ReadonlyArray<{ type: string; text?: string }> | null | undefined,
-): string | null {
-  if (content === null || content === undefined) return null;
-  if (typeof content === "string") {
-    return content.length > 0 ? content : null;
-  }
-  for (const part of content) {
-    if (part.type === "text" && typeof part.text === "string" && part.text.length > 0) {
-      return part.text;
-    }
-  }
-  return null;
-}
-
-function firstResponsesInputText(input: unknown): string | null {
-  if (typeof input === "string") return input.length > 0 ? input : null;
-  const parsedInput = UnknownArraySchema.safeParse(input);
-  if (!parsedInput.success) return null;
-  for (let i = parsedInput.data.length - 1; i >= 0; i--) {
-    const item = parsedInput.data[i];
-    const role = safeGetOwnProperty(item, "role");
-    if (role !== "user" && role !== undefined) continue;
-    const directText = safeGetOwnProperty(item, "text");
-    if (typeof directText === "string" && directText.length > 0) return directText;
-    const content = safeGetOwnProperty(item, "content");
-    if (typeof content === "string" && content.length > 0) return content;
-    if (Array.isArray(content)) {
-      for (const part of content) {
-        const type = safeGetOwnProperty(part, "type");
-        const text = safeGetOwnProperty(part, "text");
-        if (
-          (type === "input_text" || type === "text") &&
-          typeof text === "string" &&
-          text.length > 0
-        ) {
-          return text;
-        }
-      }
-    }
-  }
-  return null;
-}
-
-function firstResponsesOutputText(response: unknown): string | null {
-  const outputText = safeGetOwnProperty(response, "output_text");
-  if (typeof outputText === "string" && outputText.length > 0) return outputText;
-  const output = safeGetOwnProperty(response, "output");
-  if (!Array.isArray(output)) return null;
-  for (const item of output) {
-    const content = safeGetOwnProperty(item, "content");
-    if (!Array.isArray(content)) continue;
-    for (const part of content) {
-      const type = safeGetOwnProperty(part, "type");
-      const text = safeGetOwnProperty(part, "text");
-      if (
-        (type === "output_text" || type === "text") &&
-        typeof text === "string" &&
-        text.length > 0
-      ) {
-        return text;
-      }
-    }
-  }
-  return null;
-}
-
-/**
- * Extract a 500-char preview of the last `role: "user"` message's first text
- * block. Returns `null` for unknown format, parse failures, or non-text
- * content. See module-level docstring for the full null-case list.
- */
-export function extractLastUserMessagePreview(log: CapturedLog): string | null {
-  if (log.apiFormat === "unknown") return null;
-  if (log.rawRequestBody === null) return null;
-
-  let json: unknown;
-  try {
-    json = JSON.parse(log.rawRequestBody);
-  } catch {
-    return null;
-  }
-
-  if (log.apiFormat === "anthropic") {
-    const parsed = AnthropicRequestSchema.safeParse(json);
-    if (!parsed.success) return null;
-    // Iterate from the end to find the most recent user turn.
-    for (let i = parsed.data.messages.length - 1; i >= 0; i--) {
-      const msg = parsed.data.messages[i];
-      if (msg && msg.role === "user") {
-        const text = firstAnthropicText(msg.content);
-        return text === null ? null : truncate(text);
-      }
-    }
-    return null;
-  }
-
-  if (log.apiFormat === "openai") {
-    const parsed = OpenAIRequestSchema.safeParse(json);
-    if (!parsed.success) {
-      const responsesParsed = OpenAIResponsesRequestSchema.safeParse(json);
-      if (!responsesParsed.success) return null;
-      const text = firstResponsesInputText(responsesParsed.data.input);
-      return text === null ? null : truncate(text);
-    }
-    for (let i = parsed.data.messages.length - 1; i >= 0; i--) {
-      const msg = parsed.data.messages[i];
-      if (msg && msg.role === "user") {
-        const text = firstOpenAIText(msg.content);
-        return text === null ? null : truncate(text);
-      }
-    }
-    return null;
-  }
-
-  return null;
-}
-
-/**
- * Extract a 500-char preview of the assistant's response text. Returns
- * `null` for unknown format, parse failures, missing response, or non-text
- * content.
- */
-export function extractResponsePreview(log: CapturedLog): string | null {
-  if (log.apiFormat === "unknown") return null;
-  if (log.responseText === null) return null;
-
-  let json: unknown;
-  try {
-    json = JSON.parse(log.responseText);
-  } catch {
-    return null;
-  }
-
-  if (log.apiFormat === "anthropic") {
-    const parsed = AnthropicResponseSchema.safeParse(json);
-    if (!parsed.success) return null;
-    for (const block of parsed.data.content) {
-      if (block.type === "text" && typeof block.text === "string" && block.text.length > 0) {
-        return truncate(block.text);
-      }
-    }
-    return null;
-  }
-
-  if (log.apiFormat === "openai") {
-    const parsed = OpenAIResponseSchema.safeParse(json);
-    if (!parsed.success) {
-      const responsesParsed = OpenAIResponsesResponseSchema.safeParse(json);
-      if (!responsesParsed.success) return null;
-      const text = firstResponsesOutputText(responsesParsed.data);
-      return text === null ? null : truncate(text);
-    }
-    // Pick the first choice that actually carries a text-like content.
-    for (const choice of parsed.data.choices) {
-      const msg = choice.message;
-      if (!msg) continue;
-      const direct = firstOpenAIText(msg.content);
-      if (direct !== null) return truncate(direct);
-      // Some MiniMax / Moonshot-style providers stuff the text into
-      // reasoning_content / thinking / think instead of content. We surface
-      // whichever is the first non-empty one in a stable priority order.
-      const fallback = msg.reasoning_content ?? msg.thinking ?? msg.think;
-      if (typeof fallback === "string" && fallback.length > 0) return truncate(fallback);
-    }
-    return null;
-  }
-
-  return null;
-}