Skip to content

Commit 8efe6ec

Browse files
Align error wording with repo convention
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 3c916d2 commit 8efe6ec

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

pkg/github/server.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ func NewServer(version, name, title string, opts *mcp.ServerOptions) *mcp.Server
205205
func CompletionsHandler(getClient GetClientFn) func(ctx context.Context, req *mcp.CompleteRequest) (*mcp.CompleteResult, error) {
206206
return func(ctx context.Context, req *mcp.CompleteRequest) (*mcp.CompleteResult, error) {
207207
if req == nil || req.Params == nil || req.Params.Ref == nil {
208-
return nil, fmt.Errorf("invalid request: missing ref parameter")
208+
return nil, fmt.Errorf("missing required parameter: ref")
209209
}
210210
switch req.Params.Ref.Type {
211211
case "ref/resource":

pkg/github/server_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ func TestCompletionsHandler_RejectsMissingRef(t *testing.T) {
369369
result, err := handler(context.Background(), tc.req)
370370
require.Error(t, err)
371371
assert.Nil(t, result)
372-
assert.Contains(t, err.Error(), "missing ref parameter")
372+
assert.Contains(t, err.Error(), "missing required parameter: ref")
373373
})
374374
}
375375
}

0 commit comments

Comments
 (0)