@tonyclaw/agent-inspector 2.0.40 → 2.0.42

package/src/components/proxy-viewer/LogEntry.tsx

@@ -1,6 +1,6 @@
 import { AlertTriangle, GitCompareArrows } from "lucide-react";
 import { Suspense, type JSX } from "react";
-import { useEffect, useMemo, useRef, useState, memo } from "react";
+import { useCallback, useEffect, useMemo, useRef, useState, memo } from "react";
 import type { CapturedLog } from "../../contracts";
 import type { TimeDisplayFormat } from "../../lib/runtimeConfig";
 import { stripClaudeCodeBillingHeader } from "../../proxy/claudeCodeStrip";
@@ -308,9 +308,12 @@ export const LogEntry = memo(function ({
     },
   });
 
-  useEffect(() => {
-    if (!expanded) return;
-    setActiveTab(anatomySegments !== null ? "anatomy" : "request");
+  const handleToggleExpanded = useCallback(() => {
+    const nextExpanded = !expanded;
+    if (nextExpanded) {
+      setActiveTab(anatomySegments !== null ? "anatomy" : "request");
+    }
+    setExpanded(nextExpanded);
   }, [anatomySegments, expanded]);
 
   useEffect(() => {
@@ -336,7 +339,7 @@ export const LogEntry = memo(function ({
           messageCount={requestAnalysis.messageCount}
           toolCount={requestAnalysis.toolCount}
           expanded={expanded}
-          onToggle={() => setExpanded(!expanded)}
+          onToggle={handleToggleExpanded}
           cacheTrend={cacheTrend}
           slowResponseThresholdSeconds={slowResponseThresholdSeconds}
           timeDisplayFormat={timeDisplayFormat}

package/src/components/proxy-viewer/useCopyFeedback.ts

@@ -1,4 +1,5 @@
 import { useCallback, useEffect, useRef, useState, type MouseEvent } from "react";
+import { copyTextToClipboard } from "../../lib/clipboard";
 
 /**
  * Clipboard write with a 2s "Copied!" indicator. Returns null-safe
@@ -23,7 +24,8 @@ export function useCopyFeedback(text: string | null): {
     (event: MouseEvent) => {
       event.stopPropagation();
       if (text === null) return;
-      void window.navigator.clipboard.writeText(text).then(() => {
+      void copyTextToClipboard(text).then((success) => {
+        if (!success) return;
         setCopied(true);
         if (timerRef.current !== null) clearTimeout(timerRef.current);
         timerRef.current = setTimeout(() => setCopied(false), 2000);

package/src/components/ui/json-viewer.tsx

@@ -1,6 +1,7 @@
 import { Check, ChevronDown, ChevronRight, ChevronsDown, Copy } from "lucide-react";
 import { type JSX, memo, useEffect, useMemo, useState } from "react";
 import ReactMarkdown from "react-markdown";
+import { copyTextToClipboard } from "../../lib/clipboard";
 import { cn } from "../../lib/utils";
 import { Button } from "./button";
 import { Tooltip, TooltipContent, TooltipProvider, TooltipTrigger } from "./tooltip";
@@ -134,7 +135,8 @@ function CopyValueButton({ value }: { value: JsonValue }): JSX.Element {
 
   function handleCopy(e: React.MouseEvent): void {
     e.stopPropagation();
-    void window.navigator.clipboard.writeText(JSON.stringify(value, null, 2)).then(() => {
+    void copyTextToClipboard(JSON.stringify(value, null, 2)).then((success) => {
+      if (!success) return;
       setCopied(true);
       setTimeout(() => {
         setCopied(false);
@@ -147,6 +149,7 @@ function CopyValueButton({ value }: { value: JsonValue }): JSX.Element {
       type="button"
       onClick={handleCopy}
       className="inline-flex items-center gap-1 text-[10px] text-muted-foreground hover:text-foreground transition-colors"
+      aria-label={copied ? "Copied JSON" : "Copy JSON"}
       title="Copy JSON"
     >
       {copied ? <Check className="size-3 text-green-500" /> : <Copy className="size-3" />}
@@ -159,7 +162,8 @@ function CopyButton({ value }: { value: JsonValue }): JSX.Element {
 
   function handleCopy(e: React.MouseEvent): void {
     e.stopPropagation();
-    void window.navigator.clipboard.writeText(JSON.stringify(value, null, 2)).then(() => {
+    void copyTextToClipboard(JSON.stringify(value, null, 2)).then((success) => {
+      if (!success) return;
       setCopied(true);
       setTimeout(() => {
         setCopied(false);
@@ -172,6 +176,7 @@ function CopyButton({ value }: { value: JsonValue }): JSX.Element {
       type="button"
       onClick={handleCopy}
       className="opacity-0 group-hover/row:opacity-100 hover:bg-muted p-0.5 rounded transition-opacity shrink-0"
+      aria-label={copied ? "Copied JSON value" : "Copy JSON value"}
       title="Copy to clipboard"
     >
       {copied ? (

package/src/lib/clipboard.ts

@@ -0,0 +1,67 @@
+function restoreSelection(selection: Selection | null, ranges: readonly Range[]): void {
+  if (selection === null) return;
+  selection.removeAllRanges();
+  for (const range of ranges) {
+    selection.addRange(range);
+  }
+}
+
+function readSelectionRanges(selection: Selection | null): Range[] {
+  if (selection === null) return [];
+  const ranges: Range[] = [];
+  for (let index = 0; index < selection.rangeCount; index++) {
+    ranges.push(selection.getRangeAt(index));
+  }
+  return ranges;
+}
+
+function copyWithTextareaFallback(text: string): boolean {
+  if (typeof document === "undefined") return false;
+
+  const activeElement = document.activeElement;
+  const selection = document.getSelection();
+  const ranges = readSelectionRanges(selection);
+  const textarea = document.createElement("textarea");
+
+  textarea.value = text;
+  textarea.setAttribute("readonly", "");
+  textarea.style.position = "fixed";
+  textarea.style.top = "0";
+  textarea.style.left = "-9999px";
+  textarea.style.opacity = "0";
+
+  document.body.appendChild(textarea);
+  textarea.focus();
+  textarea.select();
+  textarea.setSelectionRange(0, text.length);
+
+  let copied = false;
+  try {
+    copied = document.execCommand("copy");
+  } catch {
+    // Keep the false default.
+  } finally {
+    document.body.removeChild(textarea);
+    restoreSelection(selection, ranges);
+    if (typeof HTMLElement !== "undefined" && activeElement instanceof HTMLElement) {
+      activeElement.focus({ preventScroll: true });
+    }
+  }
+
+  return copied;
+}
+
+export async function copyTextToClipboard(text: string): Promise<boolean> {
+  const clipboard = globalThis.navigator?.clipboard;
+  if (clipboard !== undefined) {
+    try {
+      await clipboard.writeText(text);
+      return true;
+    } catch {
+      // Fall back below. Some embedded browsers expose Clipboard API but reject
+      // writes outside a secure context or without an explicit permission grant.
+    }
+  }
+
+  return copyWithTextareaFallback(text);
+}

package/src/lib/runContract.ts

@@ -76,6 +76,7 @@ export const EvidenceReportSchema = JenkinsReportSchema;
 export const InspectorRunSchema = z.object({
   id: z.string(),
   sessionId: z.string(),
+  groupId: z.string().nullable().default(null),
   title: z.string(),
   task: z.string().nullable(),
   project: z.string().nullable(),
@@ -92,6 +93,7 @@ export const CreateInspectorRunInputSchema = z
   .object({
     runId: z.string().min(1).optional(),
     sessionId: z.string().min(1).optional(),
+    groupId: z.string().min(1).nullable().optional(),
     title: z.string().min(1).optional(),
     task: z.string().nullable().optional(),
     project: z.string().nullable().optional(),
@@ -104,6 +106,7 @@ export const CreateInspectorRunInputSchema = z
 
 export const UpdateInspectorRunInputSchema = z.object({
   sessionId: z.string().min(1).optional(),
+  groupId: z.string().min(1).nullable().optional(),
   title: z.string().min(1).optional(),
   task: z.string().nullable().optional(),
   project: z.string().nullable().optional(),

package/src/mcp/currentContext.ts

@@ -0,0 +1,65 @@
+import type { JsonValue } from "../contracts";
+
+export type InspectorCurrentContext = {
+  groupId: string | null;
+  runId: string | null;
+  sessionId: string | null;
+  model: string | null;
+  provider: string | null;
+  agent: string | null;
+  project: string | null;
+  task: string | null;
+  metadata: Record<string, JsonValue>;
+  updatedAt: string;
+};
+
+export type InspectorCurrentContextPatch = Partial<
+  Omit<InspectorCurrentContext, "metadata" | "updatedAt">
+> & {
+  metadata?: Record<string, JsonValue>;
+};
+
+function envString(name: string): string | null {
+  const value = process.env[name];
+  if (value === undefined) return null;
+  const trimmed = value.trim();
+  return trimmed.length === 0 ? null : trimmed;
+}
+
+function initialContext(): InspectorCurrentContext {
+  return {
+    groupId: envString("INSPECTOR_GROUP_ID"),
+    runId: envString("INSPECTOR_RUN_ID"),
+    sessionId: envString("INSPECTOR_SESSION_ID"),
+    model: envString("INSPECTOR_MODEL"),
+    provider: envString("INSPECTOR_PROVIDER"),
+    agent: envString("INSPECTOR_AGENT"),
+    project: envString("INSPECTOR_PROJECT"),
+    task: envString("INSPECTOR_TASK"),
+    metadata: {},
+    updatedAt: new Date().toISOString(),
+  };
+}
+
+let currentContext = initialContext();
+
+export function getCurrentContext(): InspectorCurrentContext {
+  return currentContext;
+}
+
+export function setCurrentContext(patch: InspectorCurrentContextPatch): InspectorCurrentContext {
+  currentContext = {
+    ...currentContext,
+    ...patch,
+    metadata:
+      patch.metadata === undefined
+        ? currentContext.metadata
+        : { ...currentContext.metadata, ...patch.metadata },
+    updatedAt: new Date().toISOString(),
+  };
+  return currentContext;
+}
+
+export function _resetCurrentContextForTests(): void {
+  currentContext = initialContext();
+}

package/src/mcp/mode.ts

@@ -0,0 +1,56 @@
+export type McpWriteMode = "enabled" | "readonly";
+
+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"]);
+  if (writesFlag === false) return false;
+
+  return true;
+}
+
+export function getMcpWriteMode(): McpWriteMode {
+  return isMcpWriteEnabled() ? "enabled" : "readonly";
+}
+
+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. Set AGENT_INSPECTOR_MCP_READONLY=1 to expose the same catalog while blocking mutations."
+      : "MCP is in readonly mode. Read tools/resources still work, while write/action tools return an isError result.",
+  };
+}