improve response/feedback loop with better error messages (#1414)

tonytrg · web-flow · commit 88a594e76bb5 · 2025-11-17T10:19:42.000+01:00
diff --git a/pkg/github/repositories.go b/pkg/github/repositories.go
@@ -560,7 +560,7 @@ func GetFileContents(getClient GetClientFn, getRawClient raw.GetRawClientFn, t t
 					), nil
 				}
 				if fileContent == nil || fileContent.SHA == nil {
-					return mcp.NewToolResultError("file content SHA is nil"), nil
+					return mcp.NewToolResultError("file content SHA is nil, if a directory was requested, path parameters should end with a trailing slash '/'"), nil
 				}
 				fileSHA = *fileContent.SHA
 

Original file line number	Diff line number	Diff line change
`@@ -560,7 +560,7 @@ func GetFileContents(getClient GetClientFn, getRawClient raw.GetRawClientFn, t t`
`560`	`560`	`), nil`
`561`	`561`	`}`
`562`	`562`	`if fileContent == nil \|\| fileContent.SHA == nil {`
`563`		`- return mcp.NewToolResultError("file content SHA is nil"), nil`
	`563`	`+ return mcp.NewToolResultError("file content SHA is nil, if a directory was requested, path parameters should end with a trailing slash '/'"), nil`
`564`	`564`	`}`
`565`	`565`	`fileSHA = *fileContent.SHA`
`566`	`566`