feat(kilo-mcp): remote MCP worker with hybrid catalog search, tRPC calls, and OAuth 2.1 sign-in (part 1/1) - #6030
Open
iscekic wants to merge 1 commit into
Open
feat(kilo-mcp): remote MCP worker with hybrid catalog search, tRPC calls, and OAuth 2.1 sign-in (part 1/1)#6030iscekic wants to merge 1 commit into
iscekic wants to merge 1 commit into
Conversation
iscekic
marked this pull request as draft
September 9, 2026 23:16
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (12 files)
Previous Review Summary (commit 513bd2f)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 513bd2f)Status: 2 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
SUGGESTION
Files Reviewed (32 files)
Reviewed by grok-4.6 · Input: 68.9K · Output: 12.5K · Cached: 533.6K Review guidance: REVIEW.md from base branch |
…lls, and OAuth 2.1 sign-in (part 1/1) #6030
iscekic
force-pushed
the
kwf/kilo-remote-mcp-c204-l1
branch
from
September 10, 2026 00:58
cda4506 to
5df3ac4
Compare
iscekic
marked this pull request as ready for review
September 10, 2026 01:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changelog for users
searchandcall.searchturns a plain-language task into matching Kilo endpoints: exact path and keyword matches rank first.searchstill answers from keyword matching instead of failing.callexecutes a Kilo API query as the signed-in user. It accepts only catalog paths and validates arguments against the published schema before sending anything.[truncated]marker.admin.*,debug.*, andtest.*, each with a search-friendly summary authors can hand-edit.Changelog for maintainers
apps/web/src/scripts/mcp-catalogwalks the liverootRouter(queries only, denylistadmin/debug/test) and keeps committed summaries byte-for-byte. Missing summaries are generated per router file via OpenRouter or Anthropic; a corrupt committed catalog fails loudly, and an incomplete catalog is never written.services/kilo-mcp/catalog.jsonto same-repo PR branches. Fork PRs fail with acatalog.patchartifact and a comment, including when GitHub withholds the LLM secret. The main-push job re-dumps and upserts Vectorize; PR jobs never touch it.scripts/kilo-mcp-catalog.test.mjsasserts this wiring.@cf/baai/bge-base-en-v1.5(768 dimensions, cosine) with vector id = procedure path; only the search query is embedded per request. Production deploys from main keep the bundled catalog and the index in lockstep.callrejects unknown paths and schema-invalid input before any upstream request. It forwards over apps/web's tRPC GET transport with the verified identity's Kilo credential and the org header from the token claims.x-kilocode-organizationidmirrors the apps/web constant — keep them in sync./authorizepairing with consent page, a single-use status relay, and a membership-validated org picker./tokenenforces single-use codes, S256 PKCE, and rotating hashed refresh tokens. State lives in one Durable Object (KiloMcpOAuthStore, DO SQLite, migration tag v1, 6-hour purge alarm)./mcpaccepts only this worker's HS256 tokens (issuer, audience, expiry, and jti registry checked). A deployment missingMCP_TOKEN_SECRETor the Durable Object binding refuses POSTs with 503 instead of forwarding unverified bearers.E2E proof (stack-wide)
/home/igor_kilocode_ai/.local/share/kwf/sections/kilo-remote-mcp-c204/e2e-web/prior/e10-device-auth.png
/home/igor_kilocode_ai/.local/share/kwf/sections/kilo-remote-mcp-c204/e2e-web/prior/e10-authorize.png
/home/igor_kilocode_ai/.local/share/kwf/sections/kilo-remote-mcp-c204/e2e-web/prior/e11-expired-pairing.png
Owner request for the stack
Owner manual verification
The workflow skipped these checks. Owner verification is pending; these checks did not pass automatically.
PR stack (merge bottom to top)
kwf/kilo-remote-mcp-c204-l1) ← this PR