Update documentation and fix linter issues

Author: MattBabbage

README.md

@@ -539,14 +539,14 @@ The following sets of tools are available (all are on by default):
   - `repo`: Repository name (string, required)
 
 - **list_issues** - List issues
-  - `direction`: Sort direction (string, optional)
+  - `after`: Cursor for pagination. Use the endCursor from the previous page's PageInfo for GraphQL APIs. (string, optional)
+  - `direction`: Order direction. If provided, the 'orderBy' also needs to be provided. (string, optional)
   - `labels`: Filter by labels (string[], optional)
+  - `orderBy`: Order issues by field. If provided, the 'direction' also needs to be provided. (string, optional)
   - `owner`: Repository owner (string, required)
-  - `page`: Page number for pagination (min 1) (number, optional)
   - `perPage`: Results per page for pagination (min 1, max 100) (number, optional)
   - `repo`: Repository name (string, required)
   - `since`: Filter by date (ISO 8601 timestamp) (string, optional)
-  - `sort`: Sort order (string, optional)
   - `state`: Filter by state (string, optional)
 
 - **list_sub_issues** - List sub-issues

pkg/github/discussions_test.go

@@ -484,7 +484,7 @@ func Test_GetDiscussion(t *testing.T) {
 	assert.ElementsMatch(t, toolDef.InputSchema.Required, []string{"owner", "repo", "discussionNumber"})
 
 	// Use exact string query that matches implementation output
-    qGetDiscussion := "query($discussionNumber:Int!$owner:String!$repo:String!){repository(owner: $owner, name: $repo){discussion(number: $discussionNumber){number,title,body,createdAt,url,category{name}}}}"
+	qGetDiscussion := "query($discussionNumber:Int!$owner:String!$repo:String!){repository(owner: $owner, name: $repo){discussion(number: $discussionNumber){number,title,body,createdAt,url,category{name}}}}"
 
 	vars := map[string]interface{}{
 		"owner":            "owner",

pkg/github/issues.go

@@ -34,7 +34,7 @@ type IssueFragment struct {
 	Labels    struct {
 		Nodes []struct {
 			Name        githubv4.String
-			Id          githubv4.String
+			ID          githubv4.String
 			Description githubv4.String
 		}
 	} `graphql:"labels(first: 10)"`
@@ -102,14 +102,16 @@ func (q *ListIssuesQueryTypeWithLabelsWithSince) GetIssueFragment() IssueQueryFr
 }
 
 func getIssueQueryType(hasLabels bool, hasSince bool) any {
-	if hasLabels && hasSince {
+	switch {
+	case hasLabels && hasSince:
 		return &ListIssuesQueryTypeWithLabelsWithSince{}
-	} else if hasLabels {
+	case hasLabels:
 		return &ListIssuesQueryTypeWithLabels{}
-	} else if hasSince {
+	case hasSince:
 		return &ListIssuesQueryWithSince{}
+	default:
+		return &ListIssuesQuery{}
 	}
-	return &ListIssuesQuery{}
 }
 
 func fragmentToIssue(fragment IssueFragment) *github.Issue {
@@ -118,7 +120,7 @@ func fragmentToIssue(fragment IssueFragment) *github.Issue {
 	for _, labelNode := range fragment.Labels.Nodes {
 		foundLabels = append(foundLabels, &github.Label{
 			Name:        github.Ptr(string(labelNode.Name)),
-			NodeID:      github.Ptr(string(labelNode.Id)),
+			NodeID:      github.Ptr(string(labelNode.ID)),
 			Description: github.Ptr(string(labelNode.Description)),
 		})
 	}
@@ -900,7 +902,7 @@ func ListIssues(getGQLClient GetGQLClientFn, t translations.TranslationHelperFun
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
-			//If the state has a value, cast into an array of strings
+			// If the state has a value, cast into an array of strings
 			var states []githubv4.IssueState
 			if state != "" {
 				states = append(states, githubv4.IssueState(state))
@@ -914,7 +916,7 @@ func ListIssues(getGQLClient GetGQLClientFn, t translations.TranslationHelperFun
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
-			//If labels is empty, default to nil for gql query
+			// If labels is empty, default to nil for gql query
 			if len(labels) == 0 {
 				labels = nil
 			}
@@ -923,7 +925,7 @@ func ListIssues(getGQLClient GetGQLClientFn, t translations.TranslationHelperFun
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
-			//If orderBy is empty, default to CREATED_AT
+			// If orderBy is empty, default to CREATED_AT
 			if orderBy == "" {
 				orderBy = "CREATED_AT"
 			}
@@ -932,7 +934,7 @@ func ListIssues(getGQLClient GetGQLClientFn, t translations.TranslationHelperFun
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
-			//If direction is empty, default to DESC
+			// If direction is empty, default to DESC
 			if direction == "" {
 				direction = "DESC"
 			}