(janitor/dead-code): remove unused ownerScopeFromRouteScope helper

[kilo-code-bot]

What was deleted

ownerScopeFromRouteScope in packages/mcp-gateway/src/routes.ts, and its re-export from packages/mcp-gateway/src/index.ts. This helper mapped GatewayRouteScope (user/org) to GatewayOwnerScope (personal/organization).

Evidence of unreachability

• Repo-wide grep for ownerScopeFromRouteScope across apps/, services/, packages/, dev/, scripts/, tools/ only matched its own definition and the barrel re-export — zero call sites.
• The one place in the codebase that needs this exact mapping, apps/web/src/lib/mcp-gateway/route-params.ts (parseScopedRouteParams), already hand-inlines the 'personal'/'organization' logic instead of calling this helper, confirming it was never adopted.
• No test in packages/mcp-gateway/src/index.test.ts (the package's only test file) references it.
• git log shows the function was introduced once, in the original PR that added the whole mcp-gateway OAuth runtime gateway (feat(mcp-gateway): implement OAuth runtime gateway #3717), and has never been touched or used since.

Dynamic and configured entry points checked

• packages/mcp-gateway is "private": true with exports: { ".": "./src/index.ts" } — not published to npm, no external SDK contract.
• Confirmed no reference in the Kilo-Org/kilocode extension repo (the primary external API consumer) via a fresh clone and grep.
• Not a Next.js file-route, not referenced by any Cloudflare wrangler.jsonc, binding, queue, cron, alarm, or Durable Object.
• No CLI/bin entry, script, or CI job references the symbol name.

Contract and compatibility considerations

Purely an internal, unused, unpublished helper function — no persisted data, migration, or public contract involves it. GatewayRouteScope and GatewayOwnerScope themselves are untouched and still exported/used elsewhere.

Validation performed

• packages/mcp-gateway: pnpm test → 44/44 passing (baseline and post-change).
• packages/mcp-gateway: pnpm typecheck (tsgo --noEmit) → clean (baseline and post-change).
• packages/mcp-gateway: pnpm lint (oxlint) → 0 warnings/errors.
• Repo-wide grep post-deletion confirms zero remaining references anywhere.
• pnpm format:changed → no formatting diffs; git diff --check clean.
• apps/web full typecheck (which builds @kilocode/trpc first) exceeded the 2-minute command budget for this environment and was not completed; the removed symbol has no references in apps/web per the repo-wide grep above, so this is not expected to affect that build.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)

• packages/mcp-gateway/src/index.ts
• packages/mcp-gateway/src/routes.ts

---

_{Reviewed by grok-4.6 · Input: 26.8K · Output: 2.9K · Cached: 116.1K}

_{Review guidance: REVIEW.md from base branch main}