Skip to content

fix(mcp): remove MCP tool namespaces#2791

Merged
daryllimyt merged 1 commit into
mainfrom
codex/remove-mcp-namespaces
May 30, 2026
Merged

fix(mcp): remove MCP tool namespaces#2791
daryllimyt merged 1 commit into
mainfrom
codex/remove-mcp-namespaces

Conversation

@topher-lo
Copy link
Copy Markdown
Contributor

@topher-lo topher-lo commented May 30, 2026

Summary

  • Register Tracecat MCP tools directly on the main server without FastMCP namespace mounts.
  • Regenerate the Tracecat MCP tools docs as a flat list with unprefixed tool names.
  • Update Tracecat automation skill guidance and authoring evals/tests to use unprefixed MCP tool names.

Validation

  • uv run ruff check tracecat/mcp/server.py scripts/generate_mcp_docs.py scripts/evals/tracecat_authoring/run_local.py tests/unit/test_tracecat_authoring_eval.py tests/unit/test_mcp_server.py
  • uv run ruff format --check tracecat/mcp/server.py scripts/generate_mcp_docs.py scripts/evals/tracecat_authoring/run_local.py tests/unit/test_tracecat_authoring_eval.py tests/unit/test_mcp_server.py
  • uv run pytest tests/unit/test_tracecat_authoring_eval.py tests/unit/test_mcp_server.py -q
  • Commit hooks: ruff, gitleaks, frontend client generation, basedpyright, and MCP docs generation passed.

Summary by cubic

Removes MCP tool namespaces and registers all tools directly on the main mcp server with unprefixed names. Updates docs, skills, evals, and tests to the new flat naming.

  • Refactors

    • Removed FastMCP namespace mounts and _TOOL_NAMESPACE_BY_NAME; tools are now registered via mcp.tool with simple names.
    • Regenerated MCP tool docs as a flat list; scripts/generate_mcp_docs.py now detects mcp.tool decorators.
    • Updated skill guidance and authoring evals/tests to use unprefixed tool names; added a canonical tool name set in the eval runner.
    • Adjusted unit tests to assert tool presence via attributes instead of namespace mapping.
  • Migration

    • Update clients to call unprefixed tools. Rule: strip <namespace>_ from the name.
      • Examples: workspaces_list_workspaces -> list_workspaces, workflows_edit_workflow -> edit_workflow, tables_create_column_index -> create_column_index.
    • No schema changes; only tool names changed.

Written for commit 655b619. Summary will update on new commits.

Review in cubic

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented May 30, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
tracecat 🟢 Ready View Preview May 30, 2026, 2:51 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@topher-lo topher-lo added docs Improvements or additions to documentation fix Bug fix mcp Tracecat MCP Connector tests Changes to unit and integration tests labels May 30, 2026 — with ChatGPT Codex Connector
@topher-lo topher-lo marked this pull request as ready for review May 30, 2026 02:52
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 9 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@zeropath-ai
Copy link
Copy Markdown

zeropath-ai Bot commented May 30, 2026

No security or compliance issues detected. Reviewed everything up to 655b619.

Security Overview
Detected Code Changes
Change Type Relevant files
Refactor ► .agents/skills/tracecat-automation-best-practices/SKILL.md
    Update workflow tool names to remove workflows_ prefix
► docs/snippets/mcp-tools.mdx
    Update workflow tool names to remove workflows_ prefix
► scripts/evals/tracecat_authoring/README.md
    Update workspaces_list_workspaces to list_workspaces
► scripts/evals/tracecat_authoring/cases.yaml
    Update workflows_get_workflow to get_workflow
► scripts/evals/tracecat_authoring/run_local.py
    Update tool names to remove workflows_ prefix
    Add TRACECAT_MCP_TOOL_NAMES set for validation
► scripts/generate_mcp_docs.py
    Remove namespace handling and _TOOL_NAMESPACE_BY_NAME extraction
► tests/unit/test_mcp_server.py
    Update tests for removed namespace mapping
► tests/unit/test_tracecat_authoring_eval.py
    Update tool name in test
Enhancement ► .agents/skills/tracecat-automation-best-practices/SKILL.md
    Update action names in documentation for consistency

Copy link
Copy Markdown
Contributor

@daryllimyt daryllimyt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@daryllimyt daryllimyt merged commit 026056c into main May 30, 2026
18 checks passed
@daryllimyt daryllimyt deleted the codex/remove-mcp-namespaces branch May 30, 2026 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation fix Bug fix mcp Tracecat MCP Connector tests Changes to unit and integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants