Fix lint issues

Author: JoannaaKL

pkg/github/discussions.go

@@ -69,7 +69,7 @@ func ListDiscussions(getGQLClient GetGQLClientFn, t translations.TranslationHelp
 			mcp.WithDescription(t("TOOL_LIST_DISCUSSIONS_DESCRIPTION", "List discussions for a repository")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{
 				Title:        t("TOOL_LIST_DISCUSSIONS_USER_TITLE", "List discussions"),
-				ReadOnlyHint: toBoolPtr(true),
+				ReadOnlyHint: ToBoolPtr(true),
 			}),
 			mcp.WithString("owner",
 				mcp.Required(),
@@ -232,7 +232,7 @@ func GetDiscussion(getGQLClient GetGQLClientFn, t translations.TranslationHelper
 			mcp.WithDescription(t("TOOL_GET_DISCUSSION_DESCRIPTION", "Get a specific discussion by ID")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{
 				Title:        t("TOOL_GET_DISCUSSION_USER_TITLE", "Get discussion"),
-				ReadOnlyHint: toBoolPtr(true),
+				ReadOnlyHint: ToBoolPtr(true),
 			}),
 			mcp.WithString("owner",
 				mcp.Required(),
@@ -303,7 +303,7 @@ func GetDiscussionComments(getGQLClient GetGQLClientFn, t translations.Translati
 			mcp.WithDescription(t("TOOL_GET_DISCUSSION_COMMENTS_DESCRIPTION", "Get comments from a discussion")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{
 				Title:        t("TOOL_GET_DISCUSSION_COMMENTS_USER_TITLE", "Get discussion comments"),
-				ReadOnlyHint: toBoolPtr(true),
+				ReadOnlyHint: ToBoolPtr(true),
 			}),
 			mcp.WithString("owner", mcp.Required(), mcp.Description("Repository owner")),
 			mcp.WithString("repo", mcp.Required(), mcp.Description("Repository name")),
@@ -363,7 +363,7 @@ func ListDiscussionCategories(getGQLClient GetGQLClientFn, t translations.Transl
 			mcp.WithDescription(t("TOOL_LIST_DISCUSSION_CATEGORIES_DESCRIPTION", "List discussion categories with their id and name, for a repository")),
 			mcp.WithToolAnnotation(mcp.ToolAnnotation{
 				Title:        t("TOOL_LIST_DISCUSSION_CATEGORIES_USER_TITLE", "List discussion categories"),
-				ReadOnlyHint: toBoolPtr(true),
+				ReadOnlyHint: ToBoolPtr(true),
 			}),
 			mcp.WithString("owner",
 				mcp.Required(),

pkg/github/discussions_test.go

@@ -148,7 +148,7 @@ func Test_ListDiscussions(t *testing.T) {
 		reqParams   map[string]interface{}
 		response    githubv4mock.GQLResponse
 		expectError bool
-		expectedIds []int64
+		expectedIDs []int64
 		errContains string
 		catMatcher  githubv4mock.Matcher
 	}{
@@ -161,7 +161,7 @@ func Test_ListDiscussions(t *testing.T) {
 			},
 			response:    mockResponseListAll,
 			expectError: false,
-			expectedIds: []int64{1, 2, 3},
+			expectedIDs: []int64{1, 2, 3},
 			catMatcher:  catMatcher,
 		},
 		{
@@ -186,7 +186,7 @@ func Test_ListDiscussions(t *testing.T) {
 			},
 			response:    mockResponseCategory,
 			expectError: false,
-			expectedIds: []int64{1},
+			expectedIDs: []int64{1},
 			catMatcher:  catMatcher,
 		},
 		{
@@ -199,7 +199,7 @@ func Test_ListDiscussions(t *testing.T) {
 			},
 			response:    mockResponseListAll,
 			expectError: false,
-			expectedIds: []int64{2, 3},
+			expectedIDs: []int64{2, 3},
 			catMatcher:  catMatcher,
 		},
 		{
@@ -282,16 +282,16 @@ func Test_ListDiscussions(t *testing.T) {
 			err = json.Unmarshal([]byte(text), &returnedDiscussions)
 			require.NoError(t, err)
 
-			assert.Len(t, returnedDiscussions, len(tc.expectedIds), "Expected %d discussions, got %d", len(tc.expectedIds), len(returnedDiscussions))
+			assert.Len(t, returnedDiscussions, len(tc.expectedIDs), "Expected %d discussions, got %d", len(tc.expectedIDs), len(returnedDiscussions))
 
 			// If no discussions are expected, skip further checks
-			if len(tc.expectedIds) == 0 {
+			if len(tc.expectedIDs) == 0 {
 				return
 			}
 
 			// Create a map of expected IDs for easier checking
 			expectedIDMap := make(map[int64]bool)
-			for _, id := range tc.expectedIds {
+			for _, id := range tc.expectedIDs {
 				expectedIDMap[id] = true
 			}