Describe the bug

list_commits does not interpret theperPage 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 in6f7458a

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

Expected vs actual behavior

TheperPage parameter should be correctly respected.

Logs

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