Skip to content

Conversation

@burmudar
Copy link
Contributor

  • basic sse resposne parsing
  • remove zero / nil arguments before making the request

Test plan

Tested with go run ./cmd/src mcp commit-search -repos "sourcegraph/sourcegraph" -count 1 | tail -n 1 | jq

{
  "jsonrpc": "2.0",
  "id": 1,
  "result": {
    "content": [
      {
        "type": "text",
        "text": "{\"query\":\"type:commit repo:sourcegraph/sourcegraph count:1\",\"totalCount\":1,\"commits\":[{\"repository\":\"github.com/sourcegraph/sourcegraph\",\"commit\":\"7cf5c82\",\"author\":\"Erik Seliger \\u003cerikseliger@me.com\\u003e\",\"date\":\"2025-11-28\",\"title\":\"Remove enable-simple-search (#8064)\",\"message\":\"Remove enable-simple-search (#8064)\\n\\nNot polished, never moved out of experimental, not implemented in\\nSvelte.\"}],\"limitHit\":true}"
      }
    ],
    "structuredContent": {
      "commits": [
        {
          "author": "Erik Seliger <erikseliger@me.com>",
          "commit": "7cf5c82",
          "date": "2025-11-28",
          "message": "Remove enable-simple-search (#8064)\n\nNot polished, never moved out of experimental, not implemented in\nSvelte.",
          "repository": "github.com/sourcegraph/sourcegraph",
          "title": "Remove enable-simple-search (#8064)"
        }
      ],
      "limitHit": true,
      "query": "type:commit repo:sourcegraph/sourcegraph count:1",
      "totalCount": 1
    }
  }
}

@burmudar
Copy link
Contributor Author

burmudar commented Nov 28, 2025

- basic sse resposne parsing
- remove zero / nil arguments before making the request
@burmudar burmudar force-pushed the wb/mcp-flagset-inputschema branch from c93e6a0 to 619a3a0 Compare December 2, 2025 09:46
@burmudar burmudar force-pushed the wb/mcp-http-tool-call branch from 09ac776 to e94f2f9 Compare December 2, 2025 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants