[waleedlatif1]

Summary

• A Slack source limited to selected channels runs one search per allowed channel, each scoped with in:<#ID>, and drops the status notes of the empty ones. When the query also had an in:#name filter, every empty per-channel search spent an extra rate-limited RTS channel lookup on a note that was then thrown away. A query that already carries an in:<#ID> scope now gets no lookup
• The channel-name filter now reads names in any script, including combining marks, and drops a trailing period, so in:#general. looks up general
• "No channel named #x is visible to this account" becomes "No channel named #x can be searched here", which is also true for a channel the source settings exclude
• The logged-keys TSDoc in the MCP argument logging now names the real bound on a rejection's size, the route's request body cap

Type of Change

• Improvement

Testing

• New tests in providers.test.ts (no lookup on a channel-scoped search; channel names with trailing punctuation, accented and combining-mark scripts) fail on the old code and pass on the new
• lib/sim-search/live/** and lib/knowledge/mcp/** suites pass (524 tests); biome clean

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing (new tests pass the test-audit authoring gate)
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Nov 13, 2026 10:56pm UTC

[cubic-dev-ai]

No issues found across 3 files

Confidence score: 5/5

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

_{Turn on auto-fix | Re-trigger cubic}

[greptile-apps]

Confidence Score: 5/5

[Medium impact] This PR appears safe to merge.

Summary

This PR avoids extra Slack channel lookups and expands channel-name parsing.

• Slack search skips extra channel lookups and reads channel names more fully.
• MCP logging notes point to the request body cap.

_{Reviews (1) · Last reviewed commit: "improvement(search): skip the Slack chan..." · Reviewed by Greptile}