modelcontextprotocol/inspector
Watch
1
0
Fork
You've already forked inspector
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

Improve content compatibility messages for structured content matching

Browse source 
Co-authored-by: me <me@kentcdodds.com>
This commit is contained in:
Cursor Agent 2025-07-10 14:22:31 +00:00
commit 1756b010d8
2 changed files with 12 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

6
client/src/components/ToolResults.tsx
View file

@ -27,7 +27,7 @@ const checkContentCompatibility = (
if (textBlocks.length === 0) { if (textBlocks.length === 0) {
return { return {
isCompatible: false, isCompatible: false,
message: "No text blocks found", message: "No text blocks to match structured content",
}; };
} }
@ -46,7 +46,7 @@ const checkContentCompatibility = (
if (isEqual) { if (isEqual) {
return { return {
isCompatible: true, isCompatible: true,
message: `Matching JSON found${textBlocks.length > 1 ? " (multiple blocks)" : ""}${unstructuredContent.length > textBlocks.length ? " + other content" : ""}`, message: `Structured content matches text block${textBlocks.length > 1 ? " (multiple blocks)" : ""}${unstructuredContent.length > textBlocks.length ? " + other content" : ""}`,
}; };
} }
} catch { } catch {
@ -57,7 +57,7 @@ const checkContentCompatibility = (
return { return {
isCompatible: false, isCompatible: false,
message: "No matching JSON found", message: "No text block matches structured content",
}; };
}; };

12
client/src/components/__tests__/ToolsTab.test.tsx
View file

@ -354,7 +354,9 @@ describe("ToolsTab", () => {
}); });
// Should show compatibility result // Should show compatibility result
expect(screen.getByText(/matching json/i)).toBeInTheDocument(); expect(
screen.getByText(/structured content matches/i),
).toBeInTheDocument();
}); });
it("should accept multiple content blocks with structured output", () => { it("should accept multiple content blocks with structured output", () => {
@ -374,7 +376,9 @@ describe("ToolsTab", () => {
}); });
// Should show compatible result with multiple blocks // Should show compatible result with multiple blocks
expect(screen.getByText(/matching json.*multiple/i)).toBeInTheDocument(); expect(
screen.getByText(/structured content matches.*multiple/i),
).toBeInTheDocument();
}); });
it("should accept mixed content types with structured output", () => { it("should accept mixed content types with structured output", () => {
@ -445,7 +449,9 @@ describe("ToolsTab", () => {
// Should not show any compatibility messages // Should not show any compatibility messages
expect( expect(
screen.queryByText(/matching json|no text blocks|no matching/i), screen.queryByText(
/structured content matches|no text blocks|no.*matches/i,
),
).not.toBeInTheDocument(); ).not.toBeInTheDocument();
}); });
}); });