Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 require-ready-label-and-ci

Greptile Summary

This PR fixes a gap where generated MCP clients for non-sniffed specs passed rate=0 to client.New, disabling pacing entirely, while sniffed-spec clients correctly used rate=2. The fix introduces a defaultMCPRateLimit = 2 constant and removes the SpecSource-based branch so all generated MCP clients share the same adaptive-limiter path.

• Template (mcp_tools.go.tmpl): removes the {{- if eq .SpecSource "sniffed"}} branch and replaces both client.New(cfg, 60*time.Second, 2) and client.New(cfg, 60*time.Second, 0) with a single call using the new named constant.
• Test (generator_test.go): renames the helper, adds assertions for the constant value, a NotContains guard against re-introducing rate=0, and a requireGeneratedCompiles call to validate the output compiles.
• Golden files (6 files): all updated consistently to reflect the new constant and call-site change.

Confidence Score: 5/5

Safe to merge. The change is narrowly scoped to the rate-limit argument of client.New in the MCP client factory, with all six golden fixtures updated and the test extended to compile-check the generated output.

The old branching logic left non-sniffed MCP clients with pacing fully disabled. The fix is a one-liner in the template backed by a named constant, the test guards against the old rate=0 call ever reappearing, and all golden files are consistently updated. No credential paths are touched, no behaviour outside the MCP rate-limit path changes.

No files require special attention. The template and its six golden outputs are all in sync.

Important Files Changed

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Generator: Generate MCP tools.go] --> B{SpecSource? — OLD}
    B -->|sniffed| C["client.New(cfg, 60s, 2)\n✅ paced"]
    B -->|other| D["client.New(cfg, 60s, 0)\n❌ unpaced"]

    E[Generator: Generate MCP tools.go] --> F["client.New(cfg, 60s, defaultMCPRateLimit)\ndefaultMCPRateLimit = 2\n✅ always paced — NEW"]

    C --> G[c.NoCache = true]
    D --> G
    F --> H[c.NoCache = true]

_{Reviews (1): Last reviewed commit: "fix(cli): pace generated MCP clients" | Re-trigger Greptile}

Merge Queue Status

• ✅ Entered queue — 2026-06-07 18:49 UTC · Rule: default
• ✅ Checks passed · on draft merge queue: embarking main (015e629), #2805 and #2809 together #2815
• ✅ Merged — 2026-06-07 19:13 UTC · at dfd811b05cf46fff727a95dd378854cd50b6aab9 · squash

This pull request spent 24 minutes 9 seconds in the queue, including 23 minutes 55 seconds running CI.