Repository navigation
Conversation
Verified MCP Router's storage format (SQLite at userData/mcprouter.db, lossy JSON export, bearer tokens OS-encrypted) and specified `mcpproxy import --from mcp-router [--path] [--dry-run]`: snapshot read, mapping onto mcpServers, quarantine always set, secrets via keyring with explicit confirmation, idempotent re-runs, reconnect advice, closed-enum import_source telemetry counter, optional Web UI entry. No implementation. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Deploying mcpproxy-docs with
|
| Latest commit: |
62f2255
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://7a38ffa2.mcpproxy-docs.pages.dev |
| Branch Preview URL: | https://claude-spec-mcp-router-impor.mcpproxy-docs.pages.dev |
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Contributor
📦 Build ArtifactsWorkflow Run: View Run Available Artifacts
How to DownloadOption 1: GitHub Web UI (easiest)
Option 2: GitHub CLI gh run download 40173268097 --repo smart-mcp-proxy/mcpproxy-go
|
…r-import-7ae3fb # Conflicts: # ROADMAP.md
…r-import-7ae3fb # Conflicts: # ROADMAP.md
This branch has not been deployed
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.
Summary
Experiment E4 (Q4 2026 promotion strategy), phase 1: spec only, no implementation. Adds
specs/120-mcp-router-import/(research, spec, clarifications, plan, data model, contracts, quickstart, tasks) formcpproxy import --from mcp-router [--path <file>] [--dry-run]. MCP Router (2,135 stars) was archived 2026-09-18 with "new adoption is not recommended".(Spec number: first drafted as 119, renumbered to 120 on 2026-10-25 because
119-scan-default-trustlanded on main; 121 is already a remote branch. Telemetry schema is v16 on main now, so the planned bump is the next free version, 17, re-checked at implementation time.)Format check verdict: importable
Source:
mcp-router/mcp-router@7d6cd4e5e7e73ef54b938c29d14e0cab64850fae(final, v0.6.4). Full detail with file paths and per-file SHAs inresearch.md.mcprouter.dbin ElectronuserData(macOS~/Library/Application Support/MCP Router, Windows%APPDATA%\MCP Router; no official Linux build). Default workspace usesmcprouter.db; extra local workspaces useworkspaces/<id>/database.db. WAL mode.serverstable (command,args,env,server_typelocal|remote|remote-streamable,remote_url,bearer_token,disabled,auto_start,tool_permissions,input_params,project_id, ...). Columns were added by migrations, so old DBs lack some.envis plaintext;bearer_tokenis encrypted by ElectronsafeStorage(mcpr-secret:v1:prefix). The importer never decrypts it. Remote auth is bearer-only, no OAuth.{"mcpServers":{name:{command,args,env}}}(Claude-Desktop shaped, soupstream importalready parses it) but is lossy: remote servers become empty-command stdio entries; disabled flags, tool toggles, tokens and other workspaces are dropped..dxtimport only; no.mcpb, no DXT export.mcp-routerentry (npx -y @mcp_router/cli@latest connect,MCPR_TOKEN) into Claude Desktop, Cursor, Windsurf, VS Code, Codex, Cline. The importer detects them read-only and printsmcpproxy connect <client>(Cline has no MCPProxy connector).Repo hygiene note: the cloned MCP Router repo's
CLAUDE.mdcontains instructions addressed to AI agents; treated as data, not followed.What the spec decides
importcommand;upstream importuntouched (its daemon mode dropsEnabledand swallows add errors: recorded as a follow-up).os.Rootcontainment, descriptor identity checks, bounded retries), plus the export file as a secondary input.--yes,--secrets-inlinenamed opt-in); output built from a value-free DTO with fixed masks; source${...}text is never given destination-secret semantics; deterministic hashed keyring refs, restore-on-rollback, no rollback on uncertain adds.import_source_counts(the next schema version), recorded by the daemon through a service-owned, event-time-gated call; endpointPOST /api/v1/servers/import/report. Target: >= 25 runs by day 60.Decisions to look at (made under the zero-interruption policy)
modernc.org/sqlite(pure Go). Measured +6.9 MB unstripped on a hello-world (currentmcpproxyis 46.9 MB). Task T003 re-measures stripped release builds on 3 platforms and falls back to export-file-only behind a build tag if any delta exceeds 10 MB.internal/secretrefuses keyring writes unlessMCPPROXY_KEYRING_WRITE=1. MCP Router users are mostly macOS with API keys in env, so a plain-terminal first run stops with a message naming the exact remedy. The importer does not override the gate. Funnel impact is a known risk; follow-up listed.RefName's collision-suffixing is deliberately not used (idempotent repair needs deterministic refs).Cross-model review (opencode
github-copilot/gpt-6.1-sol, high; transcripts in the shared notes folder)GetServersread-back, name-independent keyring comparison, URL masking)Not re-reviewed after the last round's fixes, and
tasks.mditself was not sent to Sol (two rounds per document, well under the 10-round cap). Findings were verified against the repo before applying (for example the placeholder precedence and the macOS keyring gate were confirmed in source).Next
Implementation per
tasks.md: PR-A read-only command (T001-T025), PR-B apply + secrets + idempotency, PR-C telemetry v15, PR-D report/export/docs, PR-E optional Web UI. The marketing migration page (other session) should wait forhandoff-website.md(task T059) so it matches the shipped flags.🤖 Generated with Claude Code