diff --git a/.gitignore b/.gitignore index e77e9dec..bde1cf45 100644 --- a/.gitignore +++ b/.gitignore @@ -1,6 +1,7 @@ .idea .vscode .DS_Store +.claude/ node_modules dist .env diff --git a/clients/web/.storybook/preview.tsx b/clients/web/.storybook/preview.tsx index 7c856228..29527c4b 100644 --- a/clients/web/.storybook/preview.tsx +++ b/clients/web/.storybook/preview.tsx @@ -1,5 +1,5 @@ import type { Preview } from '@storybook/react-vite' -import { MantineProvider, useMantineColorScheme } from '@mantine/core' +import { MantineProvider, useMantineColorScheme, type CSSVariablesResolver } from '@mantine/core' import { Notifications } from '@mantine/notifications' import '@mantine/core/styles.css' import '@mantine/notifications/styles.css' @@ -24,6 +24,14 @@ function ColorSchemeWrapper({ return <>{children} } +const resolver: CSSVariablesResolver = () => ({ + variables: {}, + light: {}, + dark: { + '--mantine-color-body': 'var(--mantine-color-dark-9)', + }, +}) + const preview: Preview = { globalTypes: { colorScheme: { @@ -46,7 +54,7 @@ const preview: Preview = { (Story, context) => { const isFullscreen = context.parameters?.layout === 'fullscreen' return ( - + {isFullscreen ? ( diff --git a/clients/web/src/App.css b/clients/web/src/App.css index 673797e9..0c064dd1 100644 --- a/clients/web/src/App.css +++ b/clients/web/src/App.css @@ -26,6 +26,9 @@ --inspector-surface-card: var(--mantine-color-gray-0); --inspector-surface-code: var(--mantine-color-white); + /* ── Inputs ────────────────────────────────────────────── */ + --inspector-input-background: var(--mantine-color-white); + /* ── Borders ───────────────────────────────────────────── */ --inspector-border-default: var(--mantine-color-default-border); --inspector-border-subtle: var(--mantine-color-gray-3); @@ -34,7 +37,7 @@ --inspector-status-connected: var(--mantine-color-green-6); --inspector-status-connecting: var(--mantine-color-yellow-6); --inspector-status-disconnected: var(--mantine-color-gray-5); - --inspector-status-failed: var(--mantine-color-red-6); + --inspector-status-error: var(--mantine-color-red-6); /* ── Log levels (RFC 5424) ─────────────────────────────── */ --inspector-log-debug: var(--mantine-color-gray-6); @@ -46,15 +49,6 @@ --inspector-log-alert: var(--mantine-color-red-9); --inspector-log-emergency: var(--mantine-color-red-9); - /* ── Annotations ───────────────────────────────────────── */ - --inspector-annotation-destructive: var(--mantine-color-red-6); - --inspector-annotation-long-run: var(--mantine-color-yellow-6); - --inspector-annotation-read-only: var(--mantine-color-dark-6); - --inspector-annotation-audience: var(--mantine-color-gray-6); - --inspector-annotation-priority-high: var(--mantine-color-orange-6); - --inspector-annotation-priority-medium: var(--mantine-color-yellow-6); - --inspector-annotation-priority-low: var(--mantine-color-green-6); - /* ── Task status ───────────────────────────────────────── */ --inspector-task-waiting: var(--mantine-color-gray-5); --inspector-task-running: var(--mantine-color-blue-6); @@ -71,7 +65,8 @@ [data-mantine-color-scheme="dark"] { --inspector-surface-card: var(--mantine-color-gray-9); --inspector-surface-code: var(--mantine-color-dark-9); - --inspector-border-subtle: var(--mantine-color-dark-4); + --inspector-border-subtle: var(--mantine-color-gray-4); + --inspector-input-background: var(--mantine-color-dark-9); } /* ── Animations ──────────────────────────────────────────── */ @@ -108,11 +103,43 @@ /* ── List item hover ────────────────────────────────────── */ .list-item:hover { - background-color: var(--mantine-color-gray-1); + background-color: var(--mantine-primary-color-light); } [data-mantine-color-scheme="dark"] .list-item:hover { - background-color: var(--mantine-color-dark-5); + background-color: var(--mantine-primary-color-light); +} + +/* ── Select dropdown / option (dark mode) ──────────────────────── */ + +[data-mantine-color-scheme="dark"] .mantine-Select-dropdown { + background-color: var(--mantine-color-gray-8); +} + +[data-mantine-color-scheme="dark"] .mantine-Select-option:hover { + background-color: var(--mantine-primary-color-light); +} + +/* ── Menu dropdown / option (dark mode) ──────────────────────── */ + +[data-mantine-color-scheme="dark"] .mantine-Menu-dropdown { + background-color: var(--mantine-color-gray-8); +} + +[data-mantine-color-scheme="dark"] .mantine-Menu-item:hover { + background-color: var(--mantine-primary-color-light); +} + +/* ── Segmented control label (dark mode) ──────────────────────── */ + +[data-mantine-color-scheme="dark"] .mantine-SegmentedControl-indicator { + background-color: var(--mantine-primary-color-light); +} + +/* ── Accordion button hover ─────────────────────────────── */ + +.mantine-Accordion-control:hover { + background-color: var(--mantine-primary-color-light-hover); } /* ── Grid alignment ─────────────────────────────────────── */ diff --git a/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.stories.tsx b/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.stories.tsx index 469abc1b..4186bc2b 100644 --- a/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.stories.tsx +++ b/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.stories.tsx @@ -11,42 +11,63 @@ type Story = StoryObj; export const Audience: Story = { args: { - label: "user", - variant: "audience", + facet: "audience", + value: ["user"], + }, +}; + +export const AudienceMultiple: Story = { + args: { + facet: "audience", + value: ["user", "assistant"], }, }; export const ReadOnly: Story = { args: { - label: "read-only", - variant: "readOnly", + facet: "readOnlyHint", + value: true, }, }; export const Destructive: Story = { args: { - label: "destructive", - variant: "destructive", + facet: "destructiveHint", + value: true, }, }; -export const LongRun: Story = { +export const Idempotent: Story = { args: { - label: "long-run", - variant: "longRun", + facet: "idempotentHint", + value: true, + }, +}; + +export const OpenWorld: Story = { + args: { + facet: "openWorldHint", + value: true, + }, +}; + +export const LongRunning: Story = { + args: { + facet: "longRunHint", + value: true, }, }; export const PriorityHigh: Story = { args: { - label: "priority: high", - variant: "priority", + facet: "priority", + value: 0.9, }, }; -export const Default: Story = { +export const PriorityLow: Story = { args: { - label: "custom", - variant: "default", + facet: "priority", + value: 0.2, }, }; diff --git a/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.tsx b/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.tsx index 14b8ee29..9f1f08f9 100644 --- a/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.tsx +++ b/clients/web/src/components/elements/AnnotationBadge/AnnotationBadge.tsx @@ -1,37 +1,63 @@ import { Badge, useComputedColorScheme } from "@mantine/core"; +import type { Role } from "@modelcontextprotocol/sdk/types.js"; -export type AnnotationVariant = +export type AnnotationFacet = | "audience" - | "readOnly" - | "destructive" - | "longRun" | "priority" - | "default"; + | "readOnlyHint" + | "destructiveHint" + | "idempotentHint" + | "openWorldHint" + | "longRunHint"; export interface AnnotationBadgeProps { - label: string; - variant?: AnnotationVariant; + facet: AnnotationFacet; + value: Role[] | number | boolean; } -const colorMap: Record = { +const colorMap: Record = { audience: "blue", - readOnly: "green", - destructive: "red", - longRun: "yellow", priority: "orange", - default: "gray", + readOnlyHint: "green", + destructiveHint: "red", + idempotentHint: "teal", + openWorldHint: "grape", + longRunHint: "yellow", }; -export function AnnotationBadge({ - label, - variant = "default", -}: AnnotationBadgeProps) { +function formatLabel( + facet: AnnotationFacet, + value: Role[] | number | boolean, +): string { + switch (facet) { + case "audience": + return `audience: ${(value as Role[]).join(", ")}`; + case "priority": { + const n = value as number; + if (n >= 0.7) return "priority: high"; + if (n >= 0.4) return "priority: medium"; + return "priority: low"; + } + case "readOnlyHint": + return "read-only"; + case "destructiveHint": + return "destructive"; + case "idempotentHint": + return "idempotent"; + case "openWorldHint": + return "open-world"; + case "longRunHint": + return "long-running"; + } +} + +export function AnnotationBadge({ facet, value }: AnnotationBadgeProps) { const colorScheme = useComputedColorScheme(); - const color = colorMap[variant]; + const color = colorMap[facet]; const textColor = colorScheme === "dark" ? "black" : "white"; return ( - {label} + {formatLabel(facet, value)} ); } diff --git a/clients/web/src/components/elements/CapabilityItem/CapabilityItem.stories.tsx b/clients/web/src/components/elements/CapabilityItem/CapabilityItem.stories.tsx index 19d26ae6..1c7fb425 100644 --- a/clients/web/src/components/elements/CapabilityItem/CapabilityItem.stories.tsx +++ b/clients/web/src/components/elements/CapabilityItem/CapabilityItem.stories.tsx @@ -11,14 +11,14 @@ type Story = StoryObj; export const Supported: Story = { args: { - name: "Tools", + capability: "tools", supported: true, }, }; export const SupportedWithCount: Story = { args: { - name: "Tools", + capability: "tools", supported: true, count: 4, }, @@ -26,7 +26,7 @@ export const SupportedWithCount: Story = { export const NotSupported: Story = { args: { - name: "Completions", + capability: "completions", supported: false, }, }; diff --git a/clients/web/src/components/elements/CapabilityItem/CapabilityItem.tsx b/clients/web/src/components/elements/CapabilityItem/CapabilityItem.tsx index 2a2130ef..e4a1ba63 100644 --- a/clients/web/src/components/elements/CapabilityItem/CapabilityItem.tsx +++ b/clients/web/src/components/elements/CapabilityItem/CapabilityItem.tsx @@ -1,16 +1,36 @@ import { Group, Text } from "@mantine/core"; +import type { + ClientCapabilities, + ServerCapabilities, +} from "@modelcontextprotocol/sdk/types.js"; + +export type CapabilityKey = keyof ServerCapabilities | keyof ClientCapabilities; export interface CapabilityItemProps { - name: string; + capability: CapabilityKey; supported: boolean; count?: number; } +const displayLabel: Record = { + tools: "Tools", + resources: "Resources", + prompts: "Prompts", + logging: "Logging", + completions: "Completions", + tasks: "Tasks", + experimental: "Experimental", + roots: "Roots", + sampling: "Sampling", + elicitation: "Elicitation", +}; + export function CapabilityItem({ - name, + capability, supported, count, }: CapabilityItemProps) { + const name = displayLabel[capability] ?? String(capability); const label = count != null ? `${name} (${count})` : name; return ( diff --git a/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.stories.tsx b/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.stories.tsx index eda50990..faa765db 100644 --- a/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.stories.tsx +++ b/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.stories.tsx @@ -6,7 +6,8 @@ const meta: Meta = { title: "Elements/ConnectionToggle", component: ConnectionToggle, args: { - onChange: fn(), + onConnect: fn(), + onDisconnect: fn(), }, }; @@ -15,32 +16,31 @@ type Story = StoryObj; export const Connected: Story = { args: { - checked: true, - loading: false, - disabled: false, + status: "connected", }, }; export const Disconnected: Story = { args: { - checked: false, - loading: false, - disabled: false, + status: "disconnected", }, }; -export const Loading: Story = { +export const Connecting: Story = { args: { - checked: false, - loading: true, - disabled: false, + status: "connecting", }, }; export const Disabled: Story = { args: { - checked: false, - loading: false, + status: "disconnected", disabled: true, }, }; + +export const Error: Story = { + args: { + status: "error", + }, +}; diff --git a/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.tsx b/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.tsx index 486dca61..5dcb6265 100644 --- a/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.tsx +++ b/clients/web/src/components/elements/ConnectionToggle/ConnectionToggle.tsx @@ -1,24 +1,34 @@ import { Switch } from "@mantine/core"; +import type { ConnectionStatus } from "@inspector/core/mcp/types.js"; export interface ConnectionToggleProps { - checked: boolean; - loading: boolean; - disabled: boolean; - onChange: (checked: boolean) => void; + status: ConnectionStatus; + disabled?: boolean; + onConnect: () => void; + onDisconnect: () => void; } export function ConnectionToggle({ - checked, - loading, - disabled, - onChange, + status, + disabled = false, + onConnect, + onDisconnect, }: ConnectionToggleProps) { + const isConnected = status === "connected"; + const isConnecting = status === "connecting"; + return ( onChange(event.currentTarget.checked)} + checked={isConnected || isConnecting} + disabled={disabled || isConnecting} + onChange={(event) => { + if (event.currentTarget.checked) { + onConnect(); + } else { + onDisconnect(); + } + }} /> ); } diff --git a/clients/web/src/components/elements/ContentViewer/ContentViewer.stories.tsx b/clients/web/src/components/elements/ContentViewer/ContentViewer.stories.tsx index a9dea061..4427bdd3 100644 --- a/clients/web/src/components/elements/ContentViewer/ContentViewer.stories.tsx +++ b/clients/web/src/components/elements/ContentViewer/ContentViewer.stories.tsx @@ -11,77 +11,66 @@ type Story = StoryObj; export const PlainText: Story = { args: { - type: "text", - content: "Hello, world!\nThis is plain text.", + block: { type: "text", text: "Hello, world!\nThis is plain text." }, }, }; export const JsonContent: Story = { args: { - type: "json", - content: - '{"name":"my-app","version":"1.0.0","description":"Sample configuration file"}', + block: { + type: "text", + text: '{"name":"my-app","version":"1.0.0","description":"Sample configuration file"}', + }, }, }; export const ImagePreview: Story = { args: { - type: "image", - content: - "iVBORw0KGgoAAAANSUhEUgAAAGQAAABkCAYAAABw4pVUAAABWUlEQVR4nO3YsQnAMAwAQe+/dLKCRZpD+UA6cVh8p/Oc81z9t1/eJ+/oD/ybVxDMKwjmFQTzCoJ5BcG8gmBeQTCvIJg3mLQX2eIVBPMKgnkFwbyCYF5BMK8gmFcQzCsI5hUE8zqdYF5BMK8gmFcQzCsI5hUE8wqCeQXBvIJgXkEwbzBpL7LFKwjmFQTzCoJ5BcG8gmBeQTCvIJhXEMwrCOZ1OsG8gmBeQTCvIJhXEMwrCOYVBPMKgnkFwbyCYN5g0l5ki1cQzCsI5hUE8wqCeQXBvIJgXkEwryCYVxDM63SCeQXBvIJgXkEwryCYVxDMKwjmFQTzCoJ5BcG8waS9yBavIJhXEMwrCOYVBPMKgnkFwbyCYF5BMK8gmNfpBPMKgnkFwbyCYF5BMK8gmFcQzCsI5hUE8wqCeYNJe5EtXkEwryCYVxDMKwjmFQTzCoJ5BcG8gmBeQTDvBaFNwZ0T5gfxAAAAAElFTkSuQmCC", - mimeType: "image/png", + block: { + type: "image", + data: "iVBORw0KGgoAAAANSUhEUgAAAGQAAABkCAYAAABw4pVUAAABWUlEQVR4nO3YsQnAMAwAQe+/dLKCRZpD+UA6cVh8p/Oc81z9t1/eJ+/oD/ybVxDMKwjmFQTzCoJ5BcG8gmBeQTCvIJg3mLQX2eIVBPMKgnkFwbyCYF5BMK8gmFcQzCsI5hUE8zqdYF5BMK8gmFcQzCsI5hUE8wqCeQXBvIJgXkEwbzBpL7LFKwjmFQTzCoJ5BcG8gmBeQTCvIJhXEMwrCOZ1OsG8gmBeQTCvIJhXEMwrCOYVBPMKgnkFwbyCYN5g0l5ki1cQzCsI5hUE8wqCeQXBvIJgXkEwryCYVxDM63SCeQXBvIJgXkEwryCYVxDMKwjmFQTzCoJ5BcG8waS9yBavIJhXEMwrCOYVBPMKgnkFwbyCYF5BMK8gmNfpBPMKgnkFwbyCYF5BMK8gmFcQzCsI5hUE8wqCeYNJe5EtXkEwryCYVxDMKwjmFQTzCoJ5BcG8gmBeQTDvBaFNwZ0T5gfxAAAAAElFTkSuQmCC", + mimeType: "image/png", + }, }, }; -export const LongContent: Story = { +export const LongJson: Story = { args: { - type: "json", - content: JSON.stringify({ - name: "my-app", - version: "2.5.0", - description: "A comprehensive application configuration", - author: "Inspector Team", - license: "MIT", - repository: "https://github.com/example/my-app", - dependencies: { - react: "^18.0.0", - typescript: "^5.0.0", - mantine: "^7.0.0", - }, - devDependencies: { - vitest: "^1.0.0", - storybook: "^8.0.0", - eslint: "^9.0.0", - }, - scripts: { - dev: "vite", - build: "tsc && vite build", - test: "vitest", - lint: "eslint .", - }, - settings: { - theme: "dark", - language: "en", - notifications: true, - maxRetries: 3, - }, - }), + block: { + type: "text", + text: JSON.stringify({ + name: "my-app", + version: "2.5.0", + description: "A comprehensive application configuration", + dependencies: { + react: "^18.0.0", + typescript: "^5.0.0", + }, + scripts: { + dev: "vite", + build: "tsc && vite build", + }, + }), + }, }, }; export const WithCopyButton: Story = { args: { - type: "text", - content: "Hello, world!\nThis text can be copied.", + block: { + type: "text", + text: "Hello, world!\nThis text can be copied.", + }, copyable: true, }, }; export const JsonWithCopy: Story = { args: { - type: "json", - content: - '{"name":"my-app","version":"1.0.0","description":"Sample configuration file"}', + block: { + type: "text", + text: '{"name":"my-app","version":"1.0.0","description":"Sample configuration file"}', + }, copyable: true, }, }; diff --git a/clients/web/src/components/elements/ContentViewer/ContentViewer.tsx b/clients/web/src/components/elements/ContentViewer/ContentViewer.tsx index d9b6bb02..a5710e32 100644 --- a/clients/web/src/components/elements/ContentViewer/ContentViewer.tsx +++ b/clients/web/src/components/elements/ContentViewer/ContentViewer.tsx @@ -1,10 +1,9 @@ -import { Code, Flex, Image, Stack } from "@mantine/core"; +import { Code, Flex, Image, Stack, Text } from "@mantine/core"; +import type { ContentBlock } from "@modelcontextprotocol/sdk/types.js"; import { CopyButton } from "../CopyButton/CopyButton"; export interface ContentViewerProps { - type: "text" | "json" | "image" | "audio"; - content: string; - mimeType?: string; + block: ContentBlock; copyable?: boolean; } @@ -16,12 +15,14 @@ function formatJson(content: string): string { } } -function buildDataUri(mimeType: string, content: string): string { - return `data:${mimeType};base64,${content}`; +function isJsonText(block: ContentBlock): boolean { + if (block.type !== "text") return false; + const trimmed = block.text.trimStart(); + return trimmed.startsWith("{") || trimmed.startsWith("["); } -function formatContent(type: "text" | "json", content: string): string { - return type === "json" ? formatJson(content) : content; +function buildDataUri(mimeType: string, data: string): string { + return `data:${mimeType};base64,${data}`; } const ContentWrapper = Flex.withProps({ @@ -41,36 +42,61 @@ const PreviewImage = Image.withProps({ radius: "md", }); -export function ContentViewer({ - type, - content, - mimeType, - copyable = false, -}: ContentViewerProps) { - const showCopy = copyable && (type === "text" || type === "json"); - - return ( - - {(type === "json" || type === "text") && ( - - - {formatContent(type, content)} - - {showCopy && ( - - - - )} - - )} - {type === "image" && ( - - )} - {type === "audio" && ( - - )} - - ); +export function ContentViewer({ block, copyable = false }: ContentViewerProps) { + switch (block.type) { + case "text": { + const isJson = isJsonText(block); + const displayText = isJson ? formatJson(block.text) : block.text; + return ( + + + + {displayText} + + {copyable && ( + + + + )} + + + ); + } + case "image": + return ( + + + + ); + case "audio": + return ( + + + + ); + case "resource": + return ( + + + + {"text" in block.resource + ? block.resource.text + : `[blob: ${block.resource.uri}]`} + + + + ); + case "resource_link": + return ( + + + {block.name ?? block.uri} + + + ); + default: + return null; + } } diff --git a/clients/web/src/components/elements/CopyButton/CopyButton.tsx b/clients/web/src/components/elements/CopyButton/CopyButton.tsx index 487cab9f..98f27549 100644 --- a/clients/web/src/components/elements/CopyButton/CopyButton.tsx +++ b/clients/web/src/components/elements/CopyButton/CopyButton.tsx @@ -15,8 +15,9 @@ export function CopyButton({ value }: CopyButtonProps) { {copied ? "\u2713" : "\u2398"} diff --git a/clients/web/src/components/elements/InlineError/InlineError.stories.tsx b/clients/web/src/components/elements/InlineError/InlineError.stories.tsx index 60dbac4d..c3784109 100644 --- a/clients/web/src/components/elements/InlineError/InlineError.stories.tsx +++ b/clients/web/src/components/elements/InlineError/InlineError.stories.tsx @@ -11,13 +11,13 @@ type Story = StoryObj; export const ShortError: Story = { args: { - message: "Connection timeout after 20s", + error: { message: "Connection timeout after 20s" }, }, }; export const WithRetryCount: Story = { args: { - message: "Connection failed", + error: { message: "Connection failed" }, retryCount: 3, maxRetries: 5, }, @@ -25,24 +25,26 @@ export const WithRetryCount: Story = { export const WithDetails: Story = { args: { - message: "Connection refused", - details: - "Error: ECONNREFUSED 127.0.0.1:3000\n at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1494:16)", + error: { + message: "Connection refused", + data: "Error: ECONNREFUSED 127.0.0.1:3000\n at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1494:16)", + }, }, }; export const WithDocLink: Story = { args: { - message: "Connection failed", + error: { message: "Connection failed" }, docLink: "https://example.com/troubleshooting", }, }; export const FullError: Story = { args: { - message: "Connection refused", - details: - "Error: ECONNREFUSED 127.0.0.1:3000\n at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1494:16)", + error: { + message: "Connection refused", + data: "Error: ECONNREFUSED 127.0.0.1:3000\n at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1494:16)", + }, retryCount: 3, maxRetries: 5, docLink: "https://example.com/troubleshooting", diff --git a/clients/web/src/components/elements/InlineError/InlineError.tsx b/clients/web/src/components/elements/InlineError/InlineError.tsx index 5b232178..7d86deb3 100644 --- a/clients/web/src/components/elements/InlineError/InlineError.tsx +++ b/clients/web/src/components/elements/InlineError/InlineError.tsx @@ -2,8 +2,7 @@ import { Alert, Button, Collapse, Group, Stack, Text } from "@mantine/core"; import { useState } from "react"; export interface InlineErrorProps { - message: string; - details?: string; + error: { message: string; data?: unknown }; retryCount?: number; maxRetries?: number; docLink?: string; @@ -49,14 +48,20 @@ function formatRetryLabel(retryCount: number, maxRetries?: number): string { : `Retry attempt ${retryCount}`; } +function formatDetails(data: unknown): string { + if (typeof data === "string") return data; + return JSON.stringify(data, null, 2); +} + export function InlineError({ - message, - details, + error, retryCount, maxRetries, docLink, }: InlineErrorProps) { const [expanded, setExpanded] = useState(false); + const details = + error.data !== undefined ? formatDetails(error.data) : undefined; const hasExpandable = details !== undefined || docLink !== undefined; return ( @@ -64,7 +69,7 @@ export function InlineError({ - {message} + {error.message} {retryCount !== undefined && ( {formatRetryLabel(retryCount, maxRetries)} )} diff --git a/clients/web/src/components/elements/ListChangedIndicator/ListChangedIndicator.tsx b/clients/web/src/components/elements/ListChangedIndicator/ListChangedIndicator.tsx index a9704b11..1f3365b4 100644 --- a/clients/web/src/components/elements/ListChangedIndicator/ListChangedIndicator.tsx +++ b/clients/web/src/components/elements/ListChangedIndicator/ListChangedIndicator.tsx @@ -19,7 +19,7 @@ const UpdateLabel = Text.withProps({ const RefreshButton = Button.withProps({ size: "sm", - variant: "light", + variant: "subtle", }); export function ListChangedIndicator({ diff --git a/clients/web/src/components/elements/ListToggle/ListToggle.tsx b/clients/web/src/components/elements/ListToggle/ListToggle.tsx index 27964ddd..7d4ec106 100644 --- a/clients/web/src/components/elements/ListToggle/ListToggle.tsx +++ b/clients/web/src/components/elements/ListToggle/ListToggle.tsx @@ -23,7 +23,7 @@ export function ListToggle({ } return ( - ); diff --git a/clients/web/src/components/elements/LogEntry/LogEntry.stories.tsx b/clients/web/src/components/elements/LogEntry/LogEntry.stories.tsx index eb2d3711..9dc8205d 100644 --- a/clients/web/src/components/elements/LogEntry/LogEntry.stories.tsx +++ b/clients/web/src/components/elements/LogEntry/LogEntry.stories.tsx @@ -11,73 +11,70 @@ type Story = StoryObj; export const Debug: Story = { args: { - timestamp: "2026-03-17T10:00:00Z", - level: "debug", - message: "Initializing connection pool", + entry: { + receivedAt: new Date("2026-03-29T20:18:20.000Z"), + params: { level: "debug", data: "Initializing connection pool" }, + }, }, }; export const Info: Story = { args: { - timestamp: "2026-03-17T10:00:01Z", - level: "info", - message: "Server started on port 3000", - }, -}; - -export const Notice: Story = { - args: { - timestamp: "2026-03-17T10:00:02Z", - level: "notice", - message: "Configuration reloaded successfully", + entry: { + receivedAt: new Date("2026-03-29T20:18:21.000Z"), + params: { level: "info", data: "Server started on port 3000" }, + }, }, }; export const Warning: Story = { args: { - timestamp: "2026-03-17T10:00:03Z", - level: "warning", - message: "Memory usage above 80%", + entry: { + receivedAt: new Date("2026-03-29T20:18:22.000Z"), + params: { + level: "warning", + data: "Connection pool nearing capacity", + logger: "pool", + }, + }, }, }; export const Error: Story = { args: { - timestamp: "2026-03-17T10:00:04Z", - level: "error", - message: "Failed to connect to database", - }, -}; - -export const Critical: Story = { - args: { - timestamp: "2026-03-17T10:00:05Z", - level: "critical", - message: "Disk space critically low", - }, -}; - -export const Alert: Story = { - args: { - timestamp: "2026-03-17T10:00:06Z", - level: "alert", - message: "Service degradation detected", + entry: { + receivedAt: new Date("2026-03-29T20:18:23.000Z"), + params: { + level: "error", + data: "Failed to connect to database", + logger: "db", + }, + }, }, }; export const Emergency: Story = { args: { - timestamp: "2026-03-17T10:00:07Z", - level: "emergency", - message: "System is unusable", + entry: { + receivedAt: new Date("2026-03-29T20:18:24.000Z"), + params: { + level: "emergency", + data: "System is unusable", + logger: "kernel", + }, + }, }, }; -export const WithLogger: Story = { +export const WithJsonData: Story = { args: { - timestamp: "2026-03-17T10:00:08Z", - level: "info", - message: "Tool execution completed", - logger: "mcp-server", + entry: { + receivedAt: new Date("2026-03-29T20:18:25.000Z"), + params: { + level: "info", + data: { key: "value", count: 42 }, + logger: "api", + }, + }, }, }; diff --git a/clients/web/src/components/elements/LogEntry/LogEntry.tsx b/clients/web/src/components/elements/LogEntry/LogEntry.tsx index 73d66915..9f2c1e8e 100644 --- a/clients/web/src/components/elements/LogEntry/LogEntry.tsx +++ b/clients/web/src/components/elements/LogEntry/LogEntry.tsx @@ -1,24 +1,20 @@ import { Group, Text } from "@mantine/core"; +import type { + LoggingLevel, + LoggingMessageNotification, +} from "@modelcontextprotocol/sdk/types.js"; import { LogLevelBadge } from "../LogLevelBadge/LogLevelBadge"; -export type LogLevel = - | "debug" - | "info" - | "notice" - | "warning" - | "error" - | "critical" - | "alert" - | "emergency"; - -export interface LogEntryProps { - timestamp: string; - level: LogLevel; - message: string; - logger?: string; +export interface LogEntryData { + receivedAt: Date; + params: LoggingMessageNotification["params"]; } -const levelMessageColor: Record = { +export interface LogEntryProps { + entry: LogEntryData; +} + +const levelMessageColor: Record = { debug: "dimmed", info: "blue", notice: undefined, @@ -29,10 +25,20 @@ const levelMessageColor: Record = { emergency: "red", }; +function formatTimestamp(date: Date): string { + return date.toLocaleTimeString(); +} + function formatLogger(logger: string): string { return `[${logger}]`; } +function formatData(data: unknown): string { + if (data === undefined || data === null) return ""; + if (typeof data === "string") return data; + return JSON.stringify(data); +} + const TimestampText = Text.withProps({ size: "sm", ff: "monospace", @@ -45,13 +51,16 @@ const LoggerText = Text.withProps({ c: "dimmed", }); -export function LogEntry({ timestamp, level, message, logger }: LogEntryProps) { +export function LogEntry({ entry }: LogEntryProps) { + const { receivedAt, params } = entry; + const message = formatData(params.data); + return ( - {timestamp} - - {logger && {formatLogger(logger)}} - + {formatTimestamp(receivedAt)} + + {params.logger && {formatLogger(params.logger)}} + {message} diff --git a/clients/web/src/components/elements/LogLevelBadge/LogLevelBadge.tsx b/clients/web/src/components/elements/LogLevelBadge/LogLevelBadge.tsx index 3839b7f2..12168ae0 100644 --- a/clients/web/src/components/elements/LogLevelBadge/LogLevelBadge.tsx +++ b/clients/web/src/components/elements/LogLevelBadge/LogLevelBadge.tsx @@ -1,11 +1,11 @@ import { Badge, useComputedColorScheme } from "@mantine/core"; -import type { LogLevel } from "../LogEntry/LogEntry"; +import type { LoggingLevel } from "@modelcontextprotocol/sdk/types.js"; export interface LogLevelBadgeProps { - level: LogLevel; + level: LoggingLevel; } -const levelColor: Record = { +const levelColor: Record = { debug: "gray", info: "blue", notice: "teal", @@ -16,7 +16,7 @@ const levelColor: Record = { emergency: "red", }; -const boldLevels: Set = new Set(["alert", "emergency"]); +const boldLevels: Set = new Set(["alert", "emergency"]); export function LogLevelBadge({ level }: LogLevelBadgeProps) { const colorScheme = useComputedColorScheme(); diff --git a/clients/web/src/components/elements/MessageBubble/MessageBubble.stories.tsx b/clients/web/src/components/elements/MessageBubble/MessageBubble.stories.tsx index dd6b1608..44c75fde 100644 --- a/clients/web/src/components/elements/MessageBubble/MessageBubble.stories.tsx +++ b/clients/web/src/components/elements/MessageBubble/MessageBubble.stories.tsx @@ -12,28 +12,39 @@ type Story = StoryObj; export const UserText: Story = { args: { index: 0, - role: "user", - content: "Hello, my name is John and I like cats", + message: { + role: "user", + content: { type: "text", text: "Hello, my name is John and I like cats" }, + }, }, }; export const AssistantText: Story = { args: { index: 1, - role: "assistant", - content: - "Nice to meet you, John! It's wonderful that you enjoy cats. They're such fascinating and independent creatures. Do you have any cats of your own?", + message: { + role: "assistant", + content: { + type: "text", + text: "Nice to meet you, John! It's wonderful that you enjoy cats. They're such fascinating and independent creatures. Do you have any cats of your own?", + }, + }, }, }; export const UserWithImage: Story = { args: { index: 0, - role: "user", - content: "Check this out", - imageContent: { - data: "iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mNk+M9QDwADhgGAWjR9awAAAABJRU5ErkJggg==", - mimeType: "image/png", + message: { + role: "user", + content: [ + { type: "text", text: "Check this out" }, + { + type: "image", + data: "iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mNk+M9QDwADhgGAWjR9awAAAABJRU5ErkJggg==", + mimeType: "image/png", + }, + ], }, }, }; @@ -41,8 +52,12 @@ export const UserWithImage: Story = { export const LongMessage: Story = { args: { index: 0, - role: "user", - content: - "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum. Sed ut perspiciatis unde omnis iste natus error sit voluptatem accusantium doloremque laudantium, totam rem aperiam, eaque ipsa quae ab illo inventore veritatis et quasi architecto beatae vitae dicta sunt explicabo.", + message: { + role: "user", + content: { + type: "text", + text: "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur.", + }, + }, }, }; diff --git a/clients/web/src/components/elements/MessageBubble/MessageBubble.tsx b/clients/web/src/components/elements/MessageBubble/MessageBubble.tsx index ee24a4f1..68465ed6 100644 --- a/clients/web/src/components/elements/MessageBubble/MessageBubble.tsx +++ b/clients/web/src/components/elements/MessageBubble/MessageBubble.tsx @@ -1,12 +1,13 @@ import { Group, Image, Paper, Stack, Text } from "@mantine/core"; +import type { + PromptMessage, + SamplingMessage, +} from "@modelcontextprotocol/sdk/types.js"; import { CopyButton } from "../CopyButton/CopyButton"; export interface MessageBubbleProps { index: number; - role: "user" | "assistant"; - content: string; - imageContent?: { data: string; mimeType: string }; - audioContent?: { data: string; mimeType: string }; + message: SamplingMessage | PromptMessage; } function buildDataUri(mimeType: string, data: string): string { @@ -21,6 +22,53 @@ function formatQuotedContent(content: string): string { return `"${content}"`; } +interface ContentBlockRendered { + text: string; + imageUri?: string; + audioUri?: string; + audioMime?: string; +} + +function extractContent( + message: SamplingMessage | PromptMessage, +): ContentBlockRendered { + const content = message.content; + const blocks = Array.isArray(content) ? content : [content]; + let text = ""; + let imageUri: string | undefined; + let audioUri: string | undefined; + let audioMime: string | undefined; + + for (const block of blocks) { + switch (block.type) { + case "text": + text += block.text; + break; + case "image": + imageUri = buildDataUri(block.mimeType, block.data); + break; + case "audio": + audioUri = buildDataUri(block.mimeType, block.data); + audioMime = block.mimeType; + break; + case "resource": + text += + "text" in block.resource + ? block.resource.text + : `[resource: ${block.resource.uri}]`; + break; + case "resource_link": + text += `[resource: ${block.uri}]`; + break; + default: + text += `[${block.type}]`; + break; + } + } + + return { text, imageUri, audioUri, audioMime }; +} + const BubbleContainer = Paper.withProps({ p: "md", radius: "md", @@ -38,32 +86,21 @@ const PreviewImage = Image.withProps({ mt: "xs", }); -export function MessageBubble({ - index, - role, - content, - imageContent, - audioContent, -}: MessageBubbleProps) { +export function MessageBubble({ index, message }: MessageBubbleProps) { + const { text, imageUri, audioUri, audioMime } = extractContent(message); + return ( - {formatRoleLabel(index, role)} - + {formatRoleLabel(index, message.role)} + {text && } - {formatQuotedContent(content)} - {imageContent && ( - - )} - {audioContent && ( + {text && {formatQuotedContent(text)}} + {imageUri && } + {audioUri && ( )} diff --git a/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.stories.tsx b/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.stories.tsx index 0e8aa20f..914b866b 100644 --- a/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.stories.tsx +++ b/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.stories.tsx @@ -11,29 +11,47 @@ type Story = StoryObj; export const ZeroPercent: Story = { args: { - progress: 0, + params: { progress: 0, total: 100 }, }, }; export const HalfComplete: Story = { args: { - progress: 50, - description: "Processing...", + params: { + progress: 50, + total: 100, + message: "Processing...", + }, }, }; export const NearComplete: Story = { args: { - progress: 95, - description: "Almost done", + params: { + progress: 95, + total: 100, + message: "Almost done", + }, elapsed: "1m 30s", }, }; export const Complete: Story = { args: { - progress: 100, - description: "Done", + params: { + progress: 100, + total: 100, + message: "Done", + }, elapsed: "2m 15s", }, }; + +export const WithoutTotal: Story = { + args: { + params: { + progress: 42, + message: "Working...", + }, + }, +}; diff --git a/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.tsx b/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.tsx index 2783fd8e..1e03791d 100644 --- a/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.tsx +++ b/clients/web/src/components/elements/ProgressDisplay/ProgressDisplay.tsx @@ -1,13 +1,23 @@ import { Group, Progress, Stack, Text } from "@mantine/core"; +import type { ProgressNotification } from "@modelcontextprotocol/sdk/types.js"; export interface ProgressDisplayProps { - progress: number; - description?: string; + params: Pick< + ProgressNotification["params"], + "progress" | "total" | "message" + >; elapsed?: string; } -function formatPercent(progress: number): string { - return `${progress}%`; +function computePercent(progress: number, total?: number): number { + if (total != null && total > 0) { + return Math.round((progress / total) * 100); + } + return progress; +} + +function formatPercent(percent: number): string { + return `${percent}%`; } const ProgressLabel = Text.withProps({ @@ -20,18 +30,16 @@ const ElapsedText = Text.withProps({ c: "dimmed", }); -export function ProgressDisplay({ - progress, - description, - elapsed, -}: ProgressDisplayProps) { +export function ProgressDisplay({ params, elapsed }: ProgressDisplayProps) { + const percent = computePercent(params.progress, params.total); + return ( - {description && {description}} - {formatPercent(progress)} + {params.message && {params.message}} + {formatPercent(percent)} - + {elapsed && {elapsed}} ); diff --git a/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.stories.tsx b/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.stories.tsx index 87385cf1..2a4ef092 100644 --- a/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.stories.tsx +++ b/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.stories.tsx @@ -28,15 +28,15 @@ export const Disconnected: Story = { }, }; -export const Failed: Story = { +export const Error: Story = { args: { - status: "failed", + status: "error", }, }; -export const FailedWithRetries: Story = { +export const ErrorWithRetries: Story = { args: { - status: "failed", + status: "error", retryCount: 3, }, }; diff --git a/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.tsx b/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.tsx index b9ace767..58f65cfc 100644 --- a/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.tsx +++ b/clients/web/src/components/elements/ServerStatusIndicator/ServerStatusIndicator.tsx @@ -1,20 +1,21 @@ import { Group, Paper, Text } from "@mantine/core"; +import type { ConnectionStatus } from "@inspector/core/mcp/types.js"; export interface ServerStatusIndicatorProps { - status: "connected" | "connecting" | "disconnected" | "failed"; + status: ConnectionStatus; latencyMs?: number; retryCount?: number; } -const statusColorVar: Record = { +const statusColorVar: Record = { connected: "var(--inspector-status-connected)", connecting: "var(--inspector-status-connecting)", disconnected: "var(--inspector-status-disconnected)", - failed: "var(--inspector-status-failed)", + error: "var(--inspector-status-error)", }; function getLabel( - status: ServerStatusIndicatorProps["status"], + status: ConnectionStatus, latencyMs?: number, retryCount?: number, ): string { @@ -25,8 +26,8 @@ function getLabel( return "Connecting..."; case "disconnected": return "Disconnected"; - case "failed": - return retryCount != null ? `Failed (${retryCount})` : "Failed"; + case "error": + return retryCount != null ? `Error (${retryCount})` : "Error"; } } diff --git a/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.stories.tsx b/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.stories.tsx index 2cdd88ef..b5c18a23 100644 --- a/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.stories.tsx +++ b/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.stories.tsx @@ -9,8 +9,8 @@ const meta: Meta = { export default meta; type Story = StoryObj; -export const Waiting: Story = { args: { status: "waiting" } }; -export const Running: Story = { args: { status: "running" } }; +export const Working: Story = { args: { status: "working" } }; +export const InputRequired: Story = { args: { status: "input_required" } }; export const Completed: Story = { args: { status: "completed" } }; export const Failed: Story = { args: { status: "failed" } }; export const Cancelled: Story = { args: { status: "cancelled" } }; diff --git a/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.tsx b/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.tsx index 49c4d786..549b1ff3 100644 --- a/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.tsx +++ b/clients/web/src/components/elements/TaskStatusBadge/TaskStatusBadge.tsx @@ -1,16 +1,24 @@ import { Badge, useComputedColorScheme } from "@mantine/core"; -import type { TaskStatus } from "../../groups/TaskCard/TaskCard"; +import type { TaskStatus } from "@modelcontextprotocol/sdk/types.js"; export interface TaskStatusBadgeProps { status: TaskStatus; } const statusColor: Record = { - waiting: "gray", - running: "blue", + working: "blue", + input_required: "yellow", completed: "green", failed: "red", - cancelled: "yellow", + cancelled: "gray", +}; + +const statusLabel: Record = { + working: "working", + input_required: "input required", + completed: "completed", + failed: "failed", + cancelled: "cancelled", }; export function TaskStatusBadge({ status }: TaskStatusBadgeProps) { @@ -19,7 +27,7 @@ export function TaskStatusBadge({ status }: TaskStatusBadgeProps) { return ( - {status} + {statusLabel[status]} ); } diff --git a/clients/web/src/components/elements/TransportBadge/TransportBadge.stories.tsx b/clients/web/src/components/elements/TransportBadge/TransportBadge.stories.tsx index 290c51b0..6e710864 100644 --- a/clients/web/src/components/elements/TransportBadge/TransportBadge.stories.tsx +++ b/clients/web/src/components/elements/TransportBadge/TransportBadge.stories.tsx @@ -15,8 +15,14 @@ export const Stdio: Story = { }, }; -export const Http: Story = { +export const Sse: Story = { args: { - transport: "http", + transport: "sse", + }, +}; + +export const StreamableHttp: Story = { + args: { + transport: "streamable-http", }, }; diff --git a/clients/web/src/components/elements/TransportBadge/TransportBadge.tsx b/clients/web/src/components/elements/TransportBadge/TransportBadge.tsx index d581f002..129d88e1 100644 --- a/clients/web/src/components/elements/TransportBadge/TransportBadge.tsx +++ b/clients/web/src/components/elements/TransportBadge/TransportBadge.tsx @@ -1,13 +1,20 @@ import { Badge } from "@mantine/core"; +import type { ServerType } from "@inspector/core/mcp/types.js"; export interface TransportBadgeProps { - transport: "stdio" | "http"; + transport: ServerType; } +const transportLabel: Record = { + stdio: "STDIO", + sse: "SSE", + "streamable-http": "HTTP", +}; + export function TransportBadge({ transport }: TransportBadgeProps) { return ( - {transport.toUpperCase()} + {transportLabel[transport]} ); } diff --git a/clients/web/src/components/groups/HistoryControls/HistoryControls.tsx b/clients/web/src/components/groups/HistoryControls/HistoryControls.tsx index dacb874b..086e6526 100644 --- a/clients/web/src/components/groups/HistoryControls/HistoryControls.tsx +++ b/clients/web/src/components/groups/HistoryControls/HistoryControls.tsx @@ -37,7 +37,7 @@ export function HistoryControls({