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

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