Skip to content

Commit ee41661

Browse files
fix: ensure proper return values in custom properties functions
1 parent 02e09c3 commit ee41661

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

pkg/github/custom_properties.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ func GetRepositoryCustomProperties(getClient GetClientFn, t translations.Transla
5252
return mcp.NewToolResultError(err.Error()), nil
5353
}
5454

55-
return MarshalledTextResult(props)
55+
return MarshalledTextResult(props), nil
5656
}
5757
}
5858

@@ -124,7 +124,7 @@ func GetOrganizationCustomProperties(getClient GetClientFn, t translations.Trans
124124
if err != nil {
125125
return mcp.NewToolResultError(err.Error()), nil
126126
}
127-
return MarshalledTextResult("Organization custom properties for "+org, props)
127+
return MarshalledTextResult(props), nil
128128
}
129129
}
130130

@@ -191,7 +191,7 @@ func GetEnterpriseCustomProperties(getClient GetClientFn, t translations.Transla
191191
if err != nil {
192192
return mcp.NewToolResultError(err.Error()), nil
193193
}
194-
return MarshalledTextResult(props)
194+
return MarshalledTextResult(props), nil
195195
}
196196
}
197197

pkg/github/tools.go

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -62,12 +62,7 @@ func DefaultToolsetGroup(readOnly bool, getClient GetClientFn, getGQLClient GetG
6262
AddWriteTools(func() server.ServerTool {
6363
tool, handler := CreateOrUpdateRepositoryCustomProperties(getClient, t)
6464
return toolsets.NewServerTool(tool, handler)
65-
AddReadTools(
66-
toolsets.NewServerTool(GetRepositoryCustomProperties(getClient, t)),
67-
).
68-
AddWriteTools(
69-
toolsets.NewServerTool(CreateOrUpdateRepositoryCustomProperties(getClient, t)),
70-
)
65+
}())
7166
issues := toolsets.NewToolset("issues", "GitHub Issues related tools").
7267
AddReadTools(
7368
toolsets.NewServerTool(GetIssue(getClient, t)),

0 commit comments

Comments
 (0)