fix: MCP instruction expression brace syntax

[topher-lo]

Motivation

• MCP clients expect examples to use ${{ ... }} expression syntax, but the instruction string was an f-string which unescaped doubled braces and caused examples to appear as ${ ... }, so the instructions were misleading.

Description

• Remove the f prefix from the _MCP_INSTRUCTIONS string in tracecat/mcp/server.py so literal ${{ ... }} placeholders are preserved in the server-provided instructions.

Testing

• Ran uv run ruff check tracecat/mcp/server.py and it completed successfully.

---

Codex Task

---

Summary by cubic

Fix MCP server instructions so ${{ ... }} placeholders render correctly for clients. Also removes a redundant unit test tied to specific instruction text.

• Bug Fixes
  • Switched _MCP_INSTRUCTIONS to a plain triple-quoted string in tracecat/mcp/server.py to preserve literal ${{ ... }}.
  • Removed redundant MCP instructions test in tests/unit/test_mcp_server.py.

^{Written for commit 17460cd. Summary will update on new commits.}

[zeropath-ai]

✅ No security or compliance issues detected. Reviewed everything up to 17460cd.

Security Overview

• 🔎 Scanned files: 2 changed file(s)
• 🔗 Scan Link: https://zeropath.com/app/repositories/00dffd6c-8834-4dc9-b6d8-b44cd1622986?scanId=c0f9360d-9b39-45ed-918c-342ef94ff72c&codeScanTypes=PrScan&tab=issues

Detected Code Changes

Change Type	Relevant files
Refactor	► tests/unit/test_mcp_server.py     Remove unused test function ► tracecat/mcp/server.py     Simplify _MCP_INSTRUCTIONS string formatting

[cubic-dev-ai]

No issues found across 1 file

Confidence score: 5/5

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

[blacksmith-sh]

Found 1 test failure on Blacksmith runners:

Failure

Test	View Logs
test_large_collection_regressions/test_scatter_gather_massive_payload_50x2mb_e2e	View Logs