issue: Using disabled_tools always fails

[dionny]

Check Existing Issues

• I have searched the existing issues and discussions.
• I am using the latest version of mcpo.

mcpo Version

0.0.19

Open WebUI Version (if applicable)

No response

Operating System

macOS

Browser (if applicable)

No response

Confirmation

• I have read and followed all instructions in README.md.
• I am using the latest version of both MCPO and Open WebUI.
• I have included the browser console logs.
• I have included the Docker container logs.
• I have listed steps to reproduce the bug in detail.

Expected Behavior

It works

Actual Behavior

It doesn't work

Steps to Reproduce

Use disabled_tools

Logs & Screenshots

ValueError: Server 'mcp-atlassian' must have either 'command' for stdio or 'type' and 'url' for remote servers

Additional Information

No response

[dionny]

The issue is pretty clear here, I can open a PR but I'm not sure what the intention was here.

 # Validate disabledTools
    disabled_tools = server_cfg.get("disabled_tools")
    if disabled_tools is not None:
        if not isinstance(disabled_tools, list):
            raise ValueError(f"Server '{server_name}' 'disabledTools' must be a list")
        for tool_name in disabled_tools:
            if not isinstance(tool_name, str):
                raise ValueError(f"Server '{server_name}' 'disabledTools' must contain only strings")
        raise ValueError(
            f"Server '{server_name}' must have either 'command' for stdio or 'type' and 'url' for remote servers"
        )

The code always raises a ValueError at the end.

[jwhandy3]

Yes, I'm encountering this as well... it will ALWAYS raise the error at the end.

[Baronco]

I have the same problem

[nougad]

Slightly related: The documentation for disabled_tools is confusing because the README.md mentions disabledTools: https://github.com/search?q=repo%3Aopen-webui%2Fmcpo%20disabledTools&type=code but it's actually disabled_tools: https://github.com/search?q=repo%3Aopen-webui%2Fmcpo+disabled_tools&type=code