Remove pagination for now

Author: JoannaaKL

README.md

@@ -664,8 +664,6 @@ The following sets of tools are available (all are on by default):
   - `project_number`: The project's number (number, required)
 
 - **list_projects** - List projects
-  - `after`: Cursor for items after (forward pagination) (string, optional)
-  - `before`: Cursor for items before (backwards pagination) (string, optional)
   - `owner`: If owner_type == user it is the handle for the GitHub user account. If owner_type == organization it is the name of the organization. The name is not case sensitive. (string, required)
   - `owner_type`: Owner type (string, required)
   - `per_page`: Number of results per page (max 100, default: 30) (number, optional)

pkg/github/__toolsnaps__/list_projects.snap

@@ -6,14 +6,6 @@
   "description": "List Projects for a user or organization",
   "inputSchema": {
     "properties": {
-      "after": {
-        "description": "Cursor for items after (forward pagination)",
-        "type": "string"
-      },
-      "before": {
-        "description": "Cursor for items before (backwards pagination)",
-        "type": "string"
-      },
       "owner": {
         "description": "If owner_type == user it is the handle for the GitHub user account. If owner_type == organization it is the name of the organization. The name is not case sensitive.",
         "type": "string"

pkg/github/projects.go

@@ -24,8 +24,6 @@ func ListProjects(getClient GetClientFn, t translations.TranslationHelperFunc) (
 			mcp.WithString("owner_type", mcp.Required(), mcp.Description("Owner type"), mcp.Enum("user", "organization")),
 			mcp.WithString("owner", mcp.Required(), mcp.Description("If owner_type == user it is the handle for the GitHub user account. If owner_type == organization it is the name of the organization. The name is not case sensitive.")),
 			mcp.WithString("query", mcp.Description("Filter projects by a search query (matches title and description)")),
-			mcp.WithString("before", mcp.Description("Cursor for items before (backwards pagination)")),
-			mcp.WithString("after", mcp.Description("Cursor for items after (forward pagination)")),
 			mcp.WithNumber("per_page", mcp.Description("Number of results per page (max 100, default: 30)")),
 		), func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 			owner, err := RequiredParam[string](req, "owner")
@@ -40,15 +38,6 @@ func ListProjects(getClient GetClientFn, t translations.TranslationHelperFunc) (
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
-
-			beforeCursor, err := OptionalParam[string](req, "before")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
-			afterCursor, err := OptionalParam[string](req, "after")
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
-			}
 			perPage, err := OptionalIntParamWithDefault(req, "per_page", 30)
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
@@ -68,12 +57,7 @@ func ListProjects(getClient GetClientFn, t translations.TranslationHelperFunc) (
 			projects := []github.ProjectV2{}
 
 			opts := ListProjectsOptions{PerPage: perPage}
-			if afterCursor != "" {
-				opts.After = afterCursor
-			}
-			if beforeCursor != "" {
-				opts.Before = beforeCursor
-			}
+
 			if queryStr != "" {
 				opts.Query = queryStr
 			}
@@ -183,12 +167,6 @@ func GetProject(getClient GetClientFn, t translations.TranslationHelperFunc) (to
 }
 
 type ListProjectsOptions struct {
-	// A cursor, as given in the Link header. If specified, the query only searches for events before this cursor.
-	Before string `url:"before,omitempty"`
-
-	// A cursor, as given in the Link header. If specified, the query only searches for events after this cursor.
-	After string `url:"after,omitempty"`
-
 	// For paginated result sets, the number of results to include per page.
 	PerPage int `url:"per_page,omitempty"`
 

pkg/github/projects_test.go

@@ -25,8 +25,6 @@ func Test_ListProjects(t *testing.T) {
 	assert.Contains(t, tool.InputSchema.Properties, "owner")
 	assert.Contains(t, tool.InputSchema.Properties, "owner_type")
 	assert.Contains(t, tool.InputSchema.Properties, "query")
-	assert.Contains(t, tool.InputSchema.Properties, "before")
-	assert.Contains(t, tool.InputSchema.Properties, "after")
 	assert.Contains(t, tool.InputSchema.Properties, "per_page")
 	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "owner_type"})
 
@@ -80,7 +78,7 @@ func Test_ListProjects(t *testing.T) {
 					http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 						q := r.URL.Query()
 						// Assert query params present
-						if q.Get("after") == "cursor123" && q.Get("per_page") == "50" && q.Get("q") == "roadmap" {
+						if q.Get("per_page") == "50" && q.Get("q") == "roadmap" {
 							w.WriteHeader(http.StatusOK)
 							_, _ = w.Write(mock.MustMarshal(orgProjects))
 							return
@@ -93,7 +91,6 @@ func Test_ListProjects(t *testing.T) {
 			requestArgs: map[string]interface{}{
 				"owner":      "octo-org",
 				"owner_type": "organization",
-				"after":      "cursor123",
 				"per_page":   float64(50),
 				"query":      "roadmap",
 			},