@modootoday/datalab-extension-mcp 1.4.1 → 1.4.3

package/dist/cli.js

@@ -3,7 +3,7 @@ import {
   dispatchCli,
   runAdapter,
   runDaemon
-} from "./chunk-DDJ2JWP5.js";
+} from "./chunk-CBHNMIS3.js";
 import "./chunk-MLKGABMK.js";
 
 // src/cli.ts
@@ -15,7 +15,7 @@ function log(message) {
 `);
 }
 async function runInstaller(sub, argv) {
-  const { createNodeIo, runInstall, runUninstall } = await import("./dist-Y7SH44SY.js");
+  const { createNodeIo, runInstall, runUninstall } = await import("./dist-WMNHKGZR.js");
   const opts = { version: VERSION };
   for (let i = 0; i < argv.length; i += 1) {
     const flag = argv[i];
@@ -41,9 +41,9 @@ async function runInstaller(sub, argv) {
 async function main() {
   await dispatchCli(process.argv, {
     install: (sub, argv) => runInstaller(sub, argv),
-    // Run the inlined daemon in this process. `runDaemon` reads the pairing
-    // token and extension id from the environment and binds the loopback port;
-    // if another daemon already owns it, it exits 0 (success by proxy).
+    // Run the inlined daemon in this process. It reads the pairing token and
+    // extension id from the environment and binds the loopback port; if another
+    // daemon already owns it, it exits 0 — the daemon wanted already exists.
     serve: () => {
       runDaemon(process.env);
     },

package/dist/{dist-Y7SH44SY.js → dist-WMNHKGZR.js}

@@ -87,9 +87,9 @@ var CLI_HOSTS = [
     tier: 1,
     displayName: "Claude Code",
     bin: "claude",
-    // --scope user is mandatory: the default "local" scope binds the server to
-    // whatever directory the user happened to paste the command in, which
-    // looks like a success and then silently does nothing everywhere else.
+    // 🔴 User scope is mandatory: the default local scope binds the server to
+    // whatever directory the command was pasted in, which reads as success and
+    // then does nothing everywhere else.
     buildAddArgs(opts) {
       const args = ["mcp", "add", SERVER_NAME, "--scope", "user"];
       for (const pair of envPairs(opts)) {
@@ -107,8 +107,8 @@ var CLI_HOSTS = [
     tier: 1,
     displayName: "ChatGPT \uB370\uC2A4\uD06C\uD1B1 / Codex",
     bin: "codex",
-    // The Codex CLI and the ChatGPT desktop app share one config, so a single
-    // registration covers both — worth saying out loud in the output.
+    // The CLI and the desktop app share one config, so a single registration
+    // covers both — worth saying out loud in the output.
     note: CODEX_CHATGPT_NOTE,
     buildAddArgs(opts) {
       const args = ["mcp", "add", SERVER_NAME];
@@ -127,7 +127,7 @@ var CLI_HOSTS = [
     tier: 1,
     displayName: "Gemini CLI",
     bin: "gemini",
-    // -s user for the same cwd-binding reason as Claude Code's --scope user.
+    // User scope, for the same cwd-binding reason as the entry above.
     buildAddArgs(opts) {
       const args = ["mcp", "add", "-s", "user"];
       for (const pair of envPairs(opts)) {
@@ -224,7 +224,8 @@ var FILE_HOSTS = [
     displayName: "Claude Desktop",
     configKey: "mcpServers",
     entryKind: "stdio",
-    // 🔴 리눅스는 공식 경로가 없어 Tier 3 가 맡는다 — 여기서 추측하면 false-success.
+    // 🔴 Linux has no documented path, so a snippet host covers it; guessing
+    // one here would manufacture a false success.
     configPath(io) {
       return resolveHostPath(io, {
         kind: "appData",
@@ -233,10 +234,9 @@ var FILE_HOSTS = [
       });
     },
     /**
-     * 🔴 Store(MSIX) 설치판이면 `%APPDATA%\Claude` 가 아니라 패키지 컨테이너 안을
-     * 쓴다. 두 설치 경로(exe / Store)가 완전히 다른 위치를 쓰는데 앱은 어느 쪽인지
-     * 말해 주지 않는다 — 컨테이너가 실제로 있으면 그쪽이 앱이 읽는 곳이므로 그쪽을
-     * 택한다. 없으면 null 을 돌려 기존 `%APPDATA%` 경로로 되돌아간다.
+     * 🔴 A Store install writes inside its package container instead. The two
+     * install kinds use entirely different locations and the app does not say
+     * which it is, so an existing container wins and its absence falls back.
      */
     async resolveConfigPath(io) {
       const dir = await msixRoamingDir(io, "Claude_", "Claude");
@@ -251,8 +251,8 @@ var FILE_HOSTS = [
     tier: 2,
     displayName: "Cursor",
     configKey: "mcpServers",
-    // Cursor speaks local HTTP MCP, so it connects to the daemon directly
-    // (url entry) — no adapter process spawned for it.
+    // Speaks local HTTP MCP, so it connects to the daemon directly and no
+    // adapter process is spawned for it.
     entryKind: "url",
     configPath(io) {
       return resolveHostPath(io, {
@@ -262,10 +262,9 @@ var FILE_HOSTS = [
     }
   },
   {
-    // Windsurf's config is documented strict JSON with an `mcpServers` map, so
-    // it graduates from a printed snippet to a safe single-key merge. The
-    // hygiene floor still refuses (and falls back to a snippet) if a given
-    // install turns out to carry comments, so the promotion cannot corrupt.
+    // Documented strict JSON, so a single-key merge is safe. The hygiene floor
+    // still refuses and falls back to a snippet if an install turns out to
+    // carry comments.
     id: "windsurf",
     tier: 2,
     displayName: "Windsurf",
@@ -282,10 +281,8 @@ var FILE_HOSTS = [
     }
   },
   {
-    // GitHub Copilot CLI — 표준 `mcpServers` + {command,args,env}(공식 문서).
-    // 🔴 경로는 `~/.copilot/mcp-config.json` 이지만 `COPILOT_HOME` 이 설정돼 있으면
-    // 그 아래다. override 를 무시하면 사용자가 옮겨 둔 설정 대신 엉뚱한 곳에 써서
-    // "성공했다는데 안 보인다" 가 된다.
+    // 🔴 A home override env var wins when set. Ignoring it would write beside
+    // the config the user moved, reporting success the app never sees.
     id: "copilot-cli",
     tier: 2,
     displayName: "GitHub Copilot CLI",
@@ -303,8 +300,7 @@ var FILE_HOSTS = [
     }
   },
   {
-    // JetBrains Junie — documented strict-JSON user config, home-relative on
-    // every OS.
+    // Documented strict-JSON user config, home-relative on every OS.
     id: "junie",
     tier: 2,
     displayName: "JetBrains Junie",
@@ -318,7 +314,7 @@ var FILE_HOSTS = [
     }
   },
   {
-    // Kiro — documented strict-JSON user settings, home-relative on every OS.
+    // Documented strict-JSON user settings, home-relative on every OS.
     id: "kiro",
     tier: 2,
     displayName: "Kiro",
@@ -382,7 +378,6 @@ function zedConfigPath(io) {
   return resolveHostPath(io, {
     kind: "xdg",
     segments: ["zed", "settings.json"],
-    // Zed 문서: Windows 만 `%APPDATA%\Zed` (대문자), 그 외는 `~/.config/zed`.
     win32Segments: ["Zed", "settings.json"]
   });
 }
@@ -469,8 +464,8 @@ var SNIPPET_HOSTS = [
     },
     reason: "\uC8FC\uC11D\uC774 \uC788\uB294 \uC124\uC815 \uD30C\uC77C\uC774\uB77C \uC790\uB3D9 \uC218\uC815\uD558\uC9C0 \uC54A\uC544\uC694.",
     pasteWhere: '\uC544\uB798 \uB0B4\uC6A9\uC744 \uC124\uC815 \uD30C\uC77C\uC758 "servers" \uD56D\uBAA9\uC5D0 \uC9C1\uC811 \uBD99\uC5EC\uB123\uC5B4 \uC8FC\uC138\uC694.',
-    // 🔴 프로필을 함께 말한다. 기본 프로필이 아니면 mcp.json 이 User/profiles/<id>/
-    // 아래에 따로 있어서, 위에 적힌 경로에 붙여넣어도 그 프로필에는 반영되지 않는다.
+    // 🔴 The hint names the profile too: a non-default profile keeps its own
+    // config elsewhere, so pasting into the path above would not reach it.
     createHint: "\uD30C\uC77C\uC774 \uC5C6\uC73C\uBA74 VS Code \uC5D0\uC11C \uBA85\uB839 \uD314\uB808\uD2B8(Ctrl+Shift+P) \u2192 'MCP: Open User Configuration' \uC744 \uC2E4\uD589\uD558\uBA74 \uB9CC\uB4E4\uC5B4\uC838\uC694. \uAE30\uBCF8 \uD504\uB85C\uD544\uC774 \uC544\uB2C8\uBA74 \uADF8 \uD504\uB85C\uD544\uC758 mcp.json \uC774 User/profiles \uC544\uB798\uC5D0 \uB530\uB85C \uC788\uC73C\uB2C8, \uADF8 \uBA85\uB839\uC73C\uB85C \uC5F4\uB9B0 \uD30C\uC77C\uC5D0 \uBD99\uC5EC\uB123\uC5B4 \uC8FC\uC138\uC694.",
     bootstrapWhenAbsent: true,
     buildSnippet(opts, platform) {
@@ -489,9 +484,9 @@ var SNIPPET_HOSTS = [
     },
     reason: "\uB3C4\uAD6C \uC804\uCCB4 \uC124\uC815 \uD30C\uC77C\uC774\uB77C \uC790\uB3D9 \uC218\uC815\uD558\uC9C0 \uC54A\uC544\uC694.",
     pasteWhere: '\uC544\uB798 \uB0B4\uC6A9\uC744 \uC124\uC815 \uD30C\uC77C\uC758 "mcp" \uD56D\uBAA9\uC5D0 \uC9C1\uC811 \uBD99\uC5EC\uB123\uC5B4 \uC8FC\uC138\uC694.',
-    // 🔴 만들지 않는다. 이 파일은 OpenCode **전체 설정**이고, 같은 폴더에
-    // `opencode.jsonc` 를 쓰고 있을 수 있다 — 그때 우리가 `opencode.json` 을 만들면
-    // 설정 파일이 둘이 되고 어느 쪽이 이기는지 사용자가 알 수 없다.
+    // 🔴 Never created for this host: the file is the tool's whole config and a
+    // sibling dialect may already hold it, so creating ours would leave two
+    // config files with no way for the user to know which wins.
     createHint: "\uC124\uC815 \uD30C\uC77C\uC740 opencode.json \uB610\uB294 opencode.jsonc \uC608\uC694. \uC4F0\uACE0 \uC788\uB294 \uCABD\uC5D0 \uBD99\uC5EC\uB123\uC5B4 \uC8FC\uC138\uC694.",
     buildSnippet(opts) {
       return opencodeSnippet(opts);
@@ -518,10 +513,8 @@ var SNIPPET_HOSTS = [
     }
   },
   {
-    // Cline — 🔴 벤더 문서가 명시하는 경로는 `~/.cline/mcp.json` 하나다(키 mcpServers).
-    // 예전에는 VS Code 확장의 globalStorage 경로를 썼는데 그건 서드파티 표에만 있었고
-    // 벤더 문서는 "확장이 쓰는 설정 파일" 이라고만 하고 경로를 주지 않는다. 확인되지
-    // 않은 경로를 안내하면 붙여넣어도 아무 일이 없고, 사용자는 우리 안내를 의심한다.
+    // 🔴 One documented path only. An unverified path from a third-party table
+    // would have the user paste into a file nothing reads.
     id: "cline",
     tier: 3,
     displayName: "Cline",
@@ -538,9 +531,9 @@ var SNIPPET_HOSTS = [
     }
   },
   {
-    // LM Studio — same `mcpServers` shape ("Cursor notation"). The exact path
-    // is only medium-confidence, so it stays a snippet: a wrong detect just
-    // means no card (safe), where a wrong auto-write would be a false success.
+    // Same entry shape, but the exact path is only medium-confidence, so it
+    // stays a snippet: a wrong detection costs a missing card, while a wrong
+    // auto-write would be a false success.
     id: "lmstudio",
     tier: 3,
     displayName: "LM Studio",
@@ -557,8 +550,7 @@ var SNIPPET_HOSTS = [
     }
   },
   {
-    // Warp terminal — same `mcpServers` shape at `~/.warp/.mcp.json`
-    // (medium-confidence path → snippet, not auto-write).
+    // Same entry shape, medium-confidence path — snippet, not auto-write.
     id: "warp",
     tier: 3,
     displayName: "Warp",
@@ -587,8 +579,8 @@ var SNIPPET_HOSTS = [
     detectedPath(io) {
       return joinPath(io, io.homedir(), ".config", "Claude");
     },
-    // Guessing a path here would manufacture false successes that only
-    // surface after a restart — the worst possible failure mode.
+    // 🔴 Guessing a path here would manufacture a false success that only
+    // surfaces after a restart.
     reason: "\uB9AC\uB205\uC2A4\uC6A9 \uACF5\uC2DD \uC124\uC815 \uD30C\uC77C \uACBD\uB85C\uAC00 \uD655\uC778\uB418\uC9C0 \uC54A\uC544 \uC790\uB3D9 \uC218\uC815\uD558\uC9C0 \uC54A\uC544\uC694.",
     pasteWhere: "Claude Desktop \uACF5\uC2DD \uBB38\uC11C\uAC00 \uC548\uB0B4\uD558\uB294 \uC124\uC815 \uD30C\uC77C(claude_desktop_config.json)\uC5D0 \uC544\uB798 \uB0B4\uC6A9\uC744 \uC9C1\uC811 \uBD99\uC5EC\uB123\uC5B4 \uC8FC\uC138\uC694.",
     buildSnippet(opts, platform) {
@@ -613,7 +605,7 @@ var SNIPPET_HOSTS = [
     },
     reason: "Codex \uBA85\uB839\uC5B4(CLI)\uAC00 \uC124\uCE58\uB418\uC5B4 \uC788\uC9C0 \uC54A\uC544 TOML \uC124\uC815 \uD30C\uC77C\uC744 \uC790\uB3D9 \uC218\uC815\uD558\uC9C0 \uC54A\uC544\uC694.",
     pasteWhere: "\uC544\uB798 \uB0B4\uC6A9\uC744 \uC704 \uC124\uC815 \uD30C\uC77C\uC5D0 \uC9C1\uC811 \uBD99\uC5EC\uB123\uC5B4 \uC8FC\uC138\uC694.",
-    // 데스크톱 앱·Codex CLI·IDE 확장이 이 파일 하나를 공유한다(OpenAI 공식 문서).
+    // The desktop app, the CLI, and the IDE extension share this one file.
     createHint: "\uD30C\uC77C\uC774 \uC5C6\uC73C\uBA74 ChatGPT \uB370\uC2A4\uD06C\uD1B1\uC5D0\uC11C \uC124\uC815 \u2192 MCP \uC11C\uBC84 \u2192 \uC11C\uBC84 \uCD94\uAC00\uB97C \uD55C \uBC88 \uD558\uBA74 \uB9CC\uB4E4\uC5B4\uC838\uC694. \uD3F4\uB354\uAC00 \uC5C6\uC73C\uBA74 \uC9C1\uC811 \uB9CC\uB4E4\uC5B4\uB3C4 \uB3FC\uC694.",
     bootstrapWhenAbsent: true,
     buildSnippet(opts) {

package/dist/index.d.ts

@@ -2,19 +2,13 @@ import { Server } from '@modelcontextprotocol/sdk/server/index.js';
 import { Transport } from '@modelcontextprotocol/sdk/shared/transport.js';
 
 /**
- * The thin stdio adapter — a stdio↔daemon proxy that owns nothing.
+ * The thin stdio adapter — a stdio-to-daemon proxy that owns nothing.
  *
- * An MCP host spawns this bin; it speaks MCP over stdio to that host and
- * forwards the two methods it supports (`tools/list`, `tools/call`) to the
- * background daemon over loopback HTTP. It binds no port and holds no bridge:
- * the daemon owns 127.0.0.1:8765 and the panel connection, and many adapters
- * (one per host) share that single daemon. This is why the earlier
- * one-server-per-host model was replaced — every host raced for the same port
- * and only one survived.
- *
- * Everything with an edge (the readiness probe, the child spawn, the HTTP
- * fetch) is injectable, so the whole proxy is unit-testable with neither a real
- * socket nor a real daemon.
+ * An MCP host spawns this bin; it speaks MCP over stdio and forwards its two
+ * methods to the background daemon over loopback HTTP. It binds no port and
+ * holds no bridge, so many adapters (one per host) share one daemon. Every
+ * edge — readiness probe, child spawn, HTTP fetch — is injectable, which is
+ * what makes the proxy testable without a real socket or daemon.
  */
 
 /** The `fetch` shape this module needs — injectable so tests need no real HTTP. */
@@ -39,18 +33,17 @@ interface EnsureRunningDeps {
     /** Injected sleep so the ready-poll never waits real milliseconds in tests. */
     sleep?: (ms: number) => Promise<void>;
     /**
-     * Absolute path the spawner runs. Defaults to this bin (`process.argv[1]`):
-     * running THIS file with the `serve` subcommand starts the inlined daemon.
+     * Absolute path the spawner runs. Defaults to this bin, which starts the
+     * inlined daemon when run with the serve subcommand.
      */
     daemonEntry?: string;
     log?: (message: string) => void;
     attempts?: number;
     intervalMs?: number;
     /**
-     * This adapter's own version. When set, a running daemon OLDER than this is
-     * replaced (it is a stale one still holding the port after an update); a daemon
-     * at this version or newer is left alone, so an older adapter never downgrades
-     * a newer daemon. Unset ⇒ no reconciliation (the daemon is used as-is).
+     * This adapter's own version. When set, a running daemon older than this is
+     * replaced; one at this version or newer is left alone, so an older adapter
+     * never downgrades a newer daemon. Unset means no reconciliation at all.
      */
     selfVersion?: string;
     /** Pairing token authorising the shutdown. Defaults to `DATALAB_MCP_TOKEN`. */
@@ -61,17 +54,12 @@ interface EnsureRunningDeps {
     shutdown?: (base: string, token: string) => Promise<boolean>;
 }
 /**
- * Ensure a daemon is up before the adapter starts proxying.
- *
- * Fast path first: if a daemon already owns the port, we are done — we only
- * needed to know it exists, so the probe socket is closed immediately and the
- * real calls go over HTTP. Otherwise spawn THIS bin as `serve` (which runs the
- * inlined daemon) and poll until it answers. Racing is fine: whichever daemon
- * binds first wins and the losers exit 0 on EADDRINUSE.
+ * Ensure a daemon is up before the adapter starts proxying. A daemon already
+ * on the port ends it; otherwise spawn this bin as serve and poll. Racing is
+ * fine — whichever binds first wins and the losers exit 0 on EADDRINUSE.
  *
- * If it never comes up inside the budget we log and return rather than throw —
- * the panel-status card will show disconnected, which is a far better outcome
- * than crashing the host's whole MCP session over a daemon that is slow to boot.
+ * Exceeding the budget logs and returns rather than throws: showing
+ * disconnected beats crashing the host's whole MCP session over a slow boot.
  */
 declare function ensureDaemonRunning(deps?: EnsureRunningDeps): Promise<void>;
 interface AdapterServerDeps {
@@ -85,20 +73,15 @@ interface AdapterServerDeps {
     version?: string;
 }
 /**
- * Build the low-level MCP `Server` whose two handlers proxy to the daemon.
- *
- * The low-level `Server` (not `McpServer`) is the honest fit: we own no tools
- * and declare no static schema — the catalog is discovered at runtime from the
- * daemon and changes while running. The request-handler form matches that, and
- * it is the same shape the daemon it replaced used, so the host sees no
- * difference across the swap.
+ * Build the low-level MCP Server whose two handlers proxy to the daemon. The
+ * low-level form fits because we own no tools and declare no static schema —
+ * the catalog is discovered from the daemon and changes while running.
  */
 declare function createAdapterServer(deps?: AdapterServerDeps): Server;
 /**
- * Read the daemon's `/mcp/notifications` SSE stream, calling `onEvent` with each
- * `data:` payload. Injectable so the subscription loop is testable without a
- * real stream. Heartbeat comments (`: hb`) are not data lines, so they are
- * skipped here and never reach `onEvent`.
+ * Read the daemon's notification stream, invoking the callback with each data
+ * payload. Injectable so the subscription loop is testable without a real
+ * stream. Heartbeat comment lines are not data and never reach the callback.
  */
 type SubscribeImpl = (url: string, onEvent: (data: string) => void, signal: AbortSignal) => Promise<void>;
 interface RunAdapterDeps extends AdapterServerDeps {
@@ -110,12 +93,9 @@ interface RunAdapterDeps extends AdapterServerDeps {
     subscribe?: SubscribeImpl;
 }
 /**
- * Run the adapter: make sure a daemon exists, then serve MCP over stdio.
- *
- * The two legs are separate on purpose — `ensureDaemonRunning` deals with the
- * socket/spawn edges and `createAdapterServer` with the request handlers — so
- * each is testable alone. This function is the only place that touches the real
- * stdio transport, which is why it stays a thin composition.
+ * Run the adapter: make sure a daemon exists, then serve MCP over stdio. The
+ * two legs stay separate so each is testable alone, and this is the only place
+ * that touches the real stdio transport.
  */
 declare function runAdapter(deps?: RunAdapterDeps): Promise<void>;
 interface CliHandlers {
@@ -124,11 +104,9 @@ interface CliHandlers {
     adapter: () => Promise<void>;
 }
 /**
- * Route the process argv to one of the three subcommand handlers.
- *
- * Extracted from `main()` so the routing decision is testable with injected
- * handlers, without spawning a daemon or connecting a transport. The default
- * (no recognised subcommand) is the adapter, which is what an MCP host spawns.
+ * Route argv to one of the three subcommand handlers. Kept out of the entry so
+ * the routing is testable with injected handlers. The default, with no
+ * recognised subcommand, is the adapter — what an MCP host spawns.
  */
 declare function dispatchCli(argv: readonly string[], handlers: CliHandlers): Promise<void>;
 

package/dist/index.js

@@ -3,7 +3,7 @@ import {
   dispatchCli,
   ensureDaemonRunning,
   runAdapter
-} from "./chunk-DDJ2JWP5.js";
+} from "./chunk-CBHNMIS3.js";
 import "./chunk-MLKGABMK.js";
 export {
   createAdapterServer,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@modootoday/datalab-extension-mcp",
-  "version": "1.4.1",
+  "version": "1.4.3",
   "private": false,
   "type": "module",
   "main": "./dist/index.js",