Oleksandr Redko (8ef75cef) at 18 Mar 18:30

Oleksandr Redko (f152d5f0) at 18 Mar 18:26

• Use assert/require instead of t.Error/t.Fatal
• Simplify with testParam and testBodyJSON

Updates #2206

Oleksandr Redko (cf98213f) at 18 Mar 18:16

Oleksandr Redko (58b8a54c) at 08 Mar 18:30

Fixed

Oleksandr Redko (76c6adc3) at 08 Mar 18:11

Oleksandr Redko (00bce983) at 08 Mar 18:04

Updates #2206

Oleksandr Redko (bfefbfbb) at 08 Mar 17:59

Not needed since !2812 (merged)

Description

Add content inside the structuredContent MCP result, so the Claude correctly identifies tool output.

Fix the error in VS Code Copilot Chat: Failed to validate tool mcp_local_gitlab__glab_api: Error: tool parameters array type must have items. Please open a Github issue for the MCP server or extension which provides this tool.

Related Issues

Resolves #8104

How has this been tested?

Tested in VS Code with GitHub Copilot Chat 0.36.2.

Version: 1.108.2
Commit: c9d77990917f3102ada88be140d28b038d1dd7c7
Date: 2026-01-21T13:52:09.270Z
Electron: 39.2.7
ElectronBuildId: 12953945
Chromium: 142.0.7444.235
Node.js: 22.21.1
V8: 14.2.231.21-electron.0
OS: Darwin arm64 25.2.0

1. make build
2. Add .vscode/mcp.json:

{
    "servers": {
        "Local GitLab MCP": {
            "type": "stdio",
            "command": "/Users/alexandear/src/gitlab.com/gitlab-org/cli/bin/glab",
            "args": [
                "mcp",
                "serve"
            ]
        },
    },
    "inputs": []
}

3. Start Local GitLab MCP.
4. Open Copilot Chat and prompt: "Give me the list of the last 5 PRs? Use glab mcp".

Before this PR:

Failed to validate tool mcp_local_gitlab__glab_api: Error: tool parameters array type must have items. Please open a Github issue for the MCP server or extension which provides this tool

After:

Screenshots (if appropriate):

@PatrickRice You're welcome. I'm the one responsible for implementing the revive checks. We're planning to move all package-related checks out of var-naming into a new rule called package-naming. The var-naming rule should only cover variables and functions, as it originally did in golint.

Oleksandr Redko (98b6eb0c) at 12 Feb 19:55

Oleksandr Redko (02240118) at 12 Feb 19:54

Oleksandr Redko (962568e7) at 12 Feb 19:53

Oleksandr Redko (ce9b5a45) at 12 Feb 19:53

@PatrickRice to fix the pipeline, disable the skip-package-name-collision-with-go-std check:

diff --git a/.golangci.yml b/.golangci.yml
index 2df0746b..fb7a72ae 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -65,6 +65,10 @@ linters:
         - name: unused-parameter
         - name: use-errors-new
         - name: var-naming
+          arguments:
+            - []
+            - []
+            - - skip-package-name-collision-with-go-std: true
     staticcheck:
       checks:
         - ST1020

This check warns on the testing package.

Oleksandr Redko (ce9b5a45) at 12 Feb 19:47

Oleksandr Redko (14b27de1) at 10 Feb 12:22