Skip to content

Query must include 'is:issue' or 'is:pull-request' on search_issues #462

Description

@raffaeler

Describe the bug

Every time the search_issues in invoked I get this "error" (Query must include 'is:issue' or 'is:pull-request').
Anyway, the request goes on and the answer contains the results.

ModelContextProtocol.Client.McpClient[145193555]
      stdio-docker request failed for method 'tools/call': failed to search issues: GET https://api.github.com/search/issues?order=desc&page=1&per_page=1&q=AssemblyLoadContext+repo%3Adotnet%2Fruntime&sort=comments: 422 Query must include 'is:issue' or 'is:pull-request' [] (-32603).

A clear and concise description of what the bug is.

Affected version

{"name":"github-mcp-server","version":"v0.4.0"}' capabilities: '{"logging":{},"resources":{"subscribe":true,"listChanged":true},"tools":{"listChanged":true}}'

Steps to reproduce the behavior

I can see the error just because I log all the traffic, otherwise standard MCP clients normally don't bother about errors that are not blocking.
The query is "AssemblyLoadContext" on the dotnet/runtime repository.
While the repo may be not the best value passed to the mcp server (it's the LLM creating it), the error does not depend on it.

Expected vs actual behavior

The error should not occur even if it is not blocking.

Activity

  1. changed the title [-]search_issues[/-] [+]Query must include 'is:issue' or 'is:pull-request' on `search_issues`[/+] on Jun 1, 2025
  2. SamMorrowDrums commented on Jun 7, 2025

    @SamMorrowDrums
    Collaborator

    I am actually planning on splitting the tool in two, and doing it as separate tools.

    I have an open PR for doing it for search users to split orgs and users. I'm not 100% about merging it, but the same PR to split issues and PRs search and put them into the appropriate toolsets would definitely get merged.

    We need that in feel.

  3. tonytrg commented on Jun 17, 2025

    @tonytrg
    Contributor

    see #334

  4. self-assigned this
    on Jun 25, 2025
  5. raffaeler commented on Jun 26, 2025

    @raffaeler
    Author

    @SamMorrowDrums out of curiousity, are you still willing to split issues from PRs?
    From the semantical perspective, I believe that splitting would be less error-prone for the LLM as it has to choose between two different functions.

  6. JoannaaKL commented on Jun 26, 2025

    @JoannaaKL
    Collaborator

    @raffaeler yes - in the latest pr we split the search query into two and gave better tool description to avoid the confusion.

  7. raffaeler commented on Jun 26, 2025

    @raffaeler
    Author

    Thanks @JoannaaKL I didn´t see that, great job!

  8. SamMorrowDrums commented on Jun 26, 2025

    @SamMorrowDrums
    Collaborator

    It will be in our next release, and is on main already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions