list_commits returns 30 commits despite perPage set to 1

[bocytko]

Describe the bug

list_commits does not interpret the perPage parameter correctly and returns 30 commits (default).

list_commits({
  "owner": "X",
  "repo": "Y",
  "perPage": 1
})

Maybe it's a mismatch between

github-mcp-server/pkg/github/repositories.go

Line 35 in 6f7458a

mcp.WithNumber("perPage",

and

github-mcp-server/pkg/github/repositories.go

Line 56 in 6f7458a

perPage, err := optionalIntParamWithDefault(request, "per_page", 30)

as a result of changes from #90.

Affected version

Please run docker run -i --rm ghcr.io/github/github-mcp-server ./github-mcp-server --version and paste the output below

server version v0.1.0 (b89336793c5bc9b9abdd5100d876babbc1031f5d) 2025-04-04T15:38:21Z

Steps to reproduce the behavior

1. Agent config:

    agent = Agent(
        name="Assistant",
        instructions=f"Answer questions about Git repositories.",
        mcp_servers=[mcp_server],
    )

2. User input: Summarize the last change in the repository X/Y
3. LLM translated the input into the function call:

list_commits({
  "owner": "X",
  "repo": "Y",
  "perPage": 1
})

4. The output from the tool call contains 30 commits, which is the default set in

   github-mcp-server/pkg/github/repositories.go

   Line 56 in 6f7458a

   perPage, err := optionalIntParamWithDefault(request, "per_page", 30)

Expected vs actual behavior

The perPage parameter should be correctly respected.

Logs

$ wc -c output.json
  179574 output.json

$ cat output.json | jq -r .text | jq '.[].sha' | wc -l
      30

[vanshavenger]

It should be perPage, A similar PR was merged yesterday, Same issue in search_repositories

[AlexanderYastrebov]

A similar PR was merged yesterday, Same issue in search_repositories

#129

[vanshavenger]

A similar PR was merged yesterday, Same issue in search_repositories

#129

Yess
, ig there can be more references and One single PR can resolve it

[vanshavenger]

@williammartin
for all the such cases we should move it to "perPage" (Golang Convenction)
and Maka const file for these keywords