tighten content filtering logic

kerobbi · kerobbi · commit 807bb923fce5 · 2025-10-06T17:02:26.000+01:00
diff --git a/pkg/github/repositories.go b/pkg/github/repositories.go
@@ -601,7 +601,13 @@ func GetFileContents(getClient GetClientFn, getRawClient raw.GetRawClientFn, t t
 						}
 					}
 
-					if strings.HasPrefix(contentType, "application") || strings.HasPrefix(contentType, "text") {
+					// Determine if content is text or binary
+					isTextContent := strings.HasPrefix(contentType, "text/") ||
+						contentType == "application/json" ||
+						strings.HasSuffix(contentType, "+json") ||
+						strings.HasSuffix(contentType, "+xml")
+
+					if isTextContent {
 						result := mcp.TextResourceContents{
 							URI:      resourceURI,
 							Text:     string(body),

Original file line number	Diff line number	Diff line change
`@@ -601,7 +601,13 @@ func GetFileContents(getClient GetClientFn, getRawClient raw.GetRawClientFn, t t`
`601`	`601`	`}`
`602`	`602`	`}`
`603`	`603`
`604`		`- if strings.HasPrefix(contentType, "application") \|\| strings.HasPrefix(contentType, "text") {`
	`604`	`+ // Determine if content is text or binary`
	`605`	`+ isTextContent := strings.HasPrefix(contentType, "text/") \|\|`
	`606`	`+ contentType == "application/json" \|\|`
	`607`	`+ strings.HasSuffix(contentType, "+json") \|\|`
	`608`	`+ strings.HasSuffix(contentType, "+xml")`
	`609`	`+`
	`610`	`+ if isTextContent {`
`605`	`611`	`result := mcp.TextResourceContents{`
`606`	`612`	`URI: resourceURI,`
`607`	`613`	`Text: string(body),`