@@ -53,9 +53,12 @@ func ListDiscussions(getGQLClient GetGQLClientFn, t translations.TranslationHelp
5353 // Get pagination parameters and convert to GraphQL format
5454 unifiedPagination , err := OptionalUnifiedPaginationParams (request )
5555 if err != nil {
56- return mcp .NewToolResultError (err .Error ()), nil
56+ return nil , err
57+ }
58+ pagination , err := unifiedPagination .ToGraphQLParams ()
59+ if err != nil {
60+ return nil , err
5761 }
58- pagination := unifiedPagination .ToGraphQLParams ()
5962
6063 client , err := getGQLClient (ctx )
6164 if err != nil {
@@ -276,15 +279,18 @@ func GetDiscussionComments(getGQLClient GetGQLClientFn, t translations.Translati
276279 // Get unified pagination parameters and convert to GraphQL format
277280 unifiedPagination , err := OptionalUnifiedPaginationParams (request )
278281 if err != nil {
279- return mcp . NewToolResultError ( err . Error ()), nil
282+ return nil , err
280283 }
281284
282285 // Check if pagination parameters were explicitly provided
283- _ , pageProvided := request .Params . Arguments .( map [ string ] interface {} )["page" ]
284- _ , perPageProvided := request .Params . Arguments .( map [ string ] interface {} )["perPage" ]
286+ _ , pageProvided := request .GetArguments ( )["page" ]
287+ _ , perPageProvided := request .GetArguments ( )["perPage" ]
285288 paginationExplicit := pageProvided || perPageProvided
286289
287- paginationParams := unifiedPagination .ToGraphQLParams ()
290+ paginationParams , err := unifiedPagination .ToGraphQLParams ()
291+ if err != nil {
292+ return nil , err
293+ }
288294
289295 // Use default of 100 if pagination was not explicitly provided
290296 if ! paginationExplicit {
@@ -366,15 +372,18 @@ func ListDiscussionCategories(getGQLClient GetGQLClientFn, t translations.Transl
366372 // Get unified pagination parameters and convert to GraphQL format
367373 unifiedPagination , err := OptionalUnifiedPaginationParams (request )
368374 if err != nil {
369- return mcp . NewToolResultError ( err . Error ()), nil
375+ return nil , err
370376 }
371377
372378 // Check if pagination parameters were explicitly provided
373- _ , pageProvided := request .Params . Arguments .( map [ string ] interface {} )["page" ]
374- _ , perPageProvided := request .Params . Arguments .( map [ string ] interface {} )["perPage" ]
379+ _ , pageProvided := request .GetArguments ( )["page" ]
380+ _ , perPageProvided := request .GetArguments ( )["perPage" ]
375381 paginationExplicit := pageProvided || perPageProvided
376382
377- pagination := unifiedPagination .ToGraphQLParams ()
383+ pagination , err := unifiedPagination .ToGraphQLParams ()
384+ if err != nil {
385+ return nil , err
386+ }
378387
379388 // Use default of 100 if pagination was not explicitly provided
380389 if ! paginationExplicit {
0 commit comments