Merge pull request modelcontextprotocol#300 from kavinkumar807/fix-tool-error-is-not-highlighted-for-sample-llm-tool-call-rejection

cliffhall · web-flow · commit 25cc0f69fdb4 · 2025-04-14T17:53:07.000-04:00
fix: missing condition for lengthy strings
diff --git a/client/src/components/JsonView.tsx b/client/src/components/JsonView.tsx
@@ -227,7 +227,7 @@ const JsonNode = memo(
             )}
             <pre
               className={clsx(
-                typeStyleMap.string,
+                isError ? typeStyleMap.error : typeStyleMap.string,
                 "break-all whitespace-pre-wrap",
               )}
             >

Original file line number	Diff line number	Diff line change
`@@ -227,7 +227,7 @@ const JsonNode = memo(`
`227`	`227`	`)}`
`228`	`228`	`<pre`
`229`	`229`	`className={clsx(`
`230`		`- typeStyleMap.string,`
	`230`	`+ isError ? typeStyleMap.error : typeStyleMap.string,`
`231`	`231`	`"break-all whitespace-pre-wrap",`
`232`	`232`	`)}`
`233`	`233`	`>`