localcode 0.3.70__tar.gz → 0.3.71__tar.gz

{localcode-0.3.70/src/localcode.egg-info → localcode-0.3.71}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: localcode
-Version: 0.3.70
+Version: 0.3.71
 Summary: High-performance AI coding on consumer hardware.
 Author: LocalCode contributors
 License-Expression: Apache-2.0

{localcode-0.3.70 → localcode-0.3.71}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "localcode"
-version = "0.3.70"
+version = "0.3.71"
 description = "High-performance AI coding on consumer hardware."
 readme = "README.md"
 requires-python = ">=3.10"

{localcode-0.3.70 → localcode-0.3.71}/src/localcode/__init__.py

@@ -8,4 +8,4 @@ from importlib.metadata import PackageNotFoundError, version as _pkg_version
 try:
     __version__ = _pkg_version("localcode")
 except PackageNotFoundError:  # not installed (e.g. raw source checkout)
-    __version__ = "0.3.70"
+    __version__ = "0.3.71"

{localcode-0.3.70 → localcode-0.3.71}/src/localcode/mcp/__init__.py

@@ -51,13 +51,7 @@ is re-exported here so `from localcode.mcp import ...` is unchanged.
 from __future__ import annotations
 
 from ._bridge import _CONNECT_TIMEOUT, _DEFAULT_TIMEOUT, _EventLoopThread
-from ._config import (
-    MCP_CONFIG_PATH,
-    _config_path,
-    add_mcp_server,
-    load_mcp_config,
-    remove_mcp_server,
-)
+from ._config import MCP_CONFIG_PATH, _config_path, load_mcp_config
 from ._transports import _InMemoryTokenStorage, _build_oauth_provider
 from .client import (
     MCPClient,
@@ -78,8 +72,6 @@ __all__ = [
     # config
     "MCP_CONFIG_PATH",
     "load_mcp_config",
-    "add_mcp_server",
-    "remove_mcp_server",
     # client
     "MCPClient",
     # registry

localcode-0.3.71/src/localcode/mcp/_config.py

@@ -0,0 +1,33 @@
+"""Config loading for the MCP package.
+
+Reads `~/.localcode/mcp.json` (key `mcpServers`), honoring LOCALCODE_HOME
+like the rest of the app.
+"""
+from __future__ import annotations
+
+import json
+import os
+from pathlib import Path
+
+
+MCP_CONFIG_PATH = Path.home() / ".localcode" / "mcp.json"
+
+
+def _config_path() -> Path:
+    """Resolve the mcp.json path, honoring LOCALCODE_HOME like the rest of
+    the app (config.get_home_dir). Falls back to ~/.localcode."""
+    override = os.environ.get("LOCALCODE_HOME")
+    if override:
+        return Path(override).expanduser() / "mcp.json"
+    return MCP_CONFIG_PATH
+
+
+def load_mcp_config() -> dict[str, dict]:
+    """Read ~/.localcode/mcp.json. Returns the `mcpServers` dict."""
+    path = _config_path()
+    if not path.is_file():
+        return {}
+    try:
+        return json.loads(path.read_text()).get("mcpServers", {})
+    except Exception:
+        return {}

{localcode-0.3.70 → localcode-0.3.71}/src/localcode/tui/screens/chat.py

@@ -2987,16 +2987,12 @@ class ChatScreen(Screen):
         parts = text.strip().split()
         sub = parts[1] if len(parts) >= 2 else None
 
-        # Bare /mcp opens the interactive management screen (status glyphs,
-        # expandable tools, r-reload, d-disconnect). Subcommands keep the text
-        # path below for muscle memory / scripting (/mcp reload).
-        if sub is None:
-            from .mcp_screen import MCPScreen
-            self.app.push_screen(MCPScreen())
-            return
-
-        # Every MCP call below can spawn subprocesses / touch the async bridge;
-        # a bad server config must surface as a message, never crash the TUI.
+        # Bare /mcp prints an inline server list, the same shape as /skills —
+        # no full-screen screen, no add form. Servers are added by editing
+        # ~/.localcode/mcp.json (like skills are added by dropping a file), then
+        # `/mcp reload`. Every MCP call below can spawn subprocesses / touch the
+        # async bridge; a bad server config must surface as a message, never
+        # crash the TUI.
         try:
             if sub == "reload":
                 shutdown_all()

{localcode-0.3.70 → localcode-0.3.71/src/localcode.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: localcode
-Version: 0.3.70
+Version: 0.3.71
 Summary: High-performance AI coding on consumer hardware.
 Author: LocalCode contributors
 License-Expression: Apache-2.0

{localcode-0.3.70 → localcode-0.3.71}/src/localcode.egg-info/SOURCES.txt

@@ -1167,8 +1167,6 @@ src/localcode/tui/paste_collapse.py
 src/localcode/tui/screens/__init__.py
 src/localcode/tui/screens/chat.py
 src/localcode/tui/screens/confirm.py
-src/localcode/tui/screens/mcp_add.py
-src/localcode/tui/screens/mcp_screen.py
 src/localcode/tui/screens/mode_picker.py
 src/localcode/tui/screens/model_picker.py
 src/localcode/tui/screens/setup.py
@@ -1229,8 +1227,6 @@ tests/test_info_markup.py
 tests/test_inspect_symbol.py
 tests/test_kv_cap.py
 tests/test_loop_recovery_ladder.py
-tests/test_mcp_add.py
-tests/test_mcp_screen.py
 tests/test_model_arg_resolution.py
 tests/test_model_config.py
 tests/test_model_verification.py

localcode-0.3.70/src/localcode/mcp/_config.py

@@ -1,79 +0,0 @@
-"""Config loading for the MCP package.
-
-Reads `~/.localcode/mcp.json` (key `mcpServers`), honoring LOCALCODE_HOME
-like the rest of the app.
-"""
-from __future__ import annotations
-
-import json
-import os
-from pathlib import Path
-
-
-MCP_CONFIG_PATH = Path.home() / ".localcode" / "mcp.json"
-
-
-def _config_path() -> Path:
-    """Resolve the mcp.json path, honoring LOCALCODE_HOME like the rest of
-    the app (config.get_home_dir). Falls back to ~/.localcode."""
-    override = os.environ.get("LOCALCODE_HOME")
-    if override:
-        return Path(override).expanduser() / "mcp.json"
-    return MCP_CONFIG_PATH
-
-
-def load_mcp_config() -> dict[str, dict]:
-    """Read ~/.localcode/mcp.json. Returns the `mcpServers` dict."""
-    path = _config_path()
-    if not path.is_file():
-        return {}
-    try:
-        return json.loads(path.read_text()).get("mcpServers", {})
-    except Exception:
-        return {}
-
-
-def _read_full_config(path: Path) -> dict:
-    """The whole mcp.json document (not just `mcpServers`), so a write preserves
-    any other keys the user keeps in the file. Empty dict if missing/invalid."""
-    if not path.is_file():
-        return {}
-    try:
-        data = json.loads(path.read_text())
-        return data if isinstance(data, dict) else {}
-    except Exception:
-        return {}
-
-
-def add_mcp_server(name: str, entry: dict) -> Path:
-    """Add (or replace) one server under `mcpServers` and write mcp.json back,
-    creating the file and its parent directory if needed. Returns the path.
-
-    This is what the in-TUI "add server" form calls so a user never has to
-    hand-edit the JSON. The written shape is exactly what ``load_mcp_config``
-    reads and ``connect_all`` consumes.
-    """
-    path = _config_path()
-    data = _read_full_config(path)
-    servers = data.get("mcpServers")
-    if not isinstance(servers, dict):
-        servers = {}
-    servers[name] = entry
-    data["mcpServers"] = servers
-    path.parent.mkdir(parents=True, exist_ok=True)
-    path.write_text(json.dumps(data, indent=2) + "\n")
-    return path
-
-
-def remove_mcp_server(name: str) -> bool:
-    """Drop one server from mcp.json. True if it was there, False otherwise."""
-    path = _config_path()
-    data = _read_full_config(path)
-    servers = data.get("mcpServers")
-    if not isinstance(servers, dict) or name not in servers:
-        return False
-    del servers[name]
-    data["mcpServers"] = servers
-    path.parent.mkdir(parents=True, exist_ok=True)
-    path.write_text(json.dumps(data, indent=2) + "\n")
-    return True

localcode-0.3.70/src/localcode/tui/screens/mcp_add.py

@@ -1,163 +0,0 @@
-"""Add-an-MCP-server modal — the in-TUI answer to `claude mcp add`.
-
-localcode has no CLI subcommands; everything happens in the TUI. So adding an
-MCP server is a small keyboard form here, not a shell command. It writes
-``~/.localcode/mcp.json`` for the user (via ``mcp.add_mcp_server``) so nobody
-has to hand-edit JSON.
-
-The common case is the one Codex/Claude make trivial: "use this MCP" + a URL.
-So the URL/command field comes first, and:
-
-  * a value starting with ``http://`` / ``https://`` becomes a remote
-    streamable-HTTP server (``{"transport": "http", "url": ...}``);
-  * anything else is a stdio server (``{"command": ..., "args": [...]}``),
-    with args split shell-style.
-
-Name is optional: left blank, it is derived from the URL host (``api.
-githubcopilot.com`` -> ``githubcopilot``) or the command/package basename.
-
-Resolves to ``{"name": str, "entry": dict}`` on save, or ``None`` on cancel.
-"""
-from __future__ import annotations
-
-import shlex
-from urllib.parse import urlparse
-
-from textual.app import ComposeResult
-from textual.binding import Binding
-from textual.containers import Vertical
-from textual.screen import ModalScreen
-from textual.widgets import Input, Static
-
-from ...theme import C
-
-
-def _derive_name_from_url(url: str) -> str:
-    """A short, sane server name from a URL host, e.g.
-    https://api.githubcopilot.com/mcp/ -> 'githubcopilot'."""
-    host = (urlparse(url).hostname or "").lower()
-    if not host:
-        return ""
-    labels = [p for p in host.split(".") if p]
-    # Drop leading noise and a trailing TLD so the meaningful label is left.
-    while labels and labels[0] in ("api", "mcp", "www", "app"):
-        labels.pop(0)
-    if len(labels) > 1:
-        labels = labels[:-1]  # strip TLD (.com/.dev/.ai/...)
-    return labels[0] if labels else host
-
-
-def _derive_name_from_command(command: str, args: list[str]) -> str:
-    """A name for a stdio server. Runners like npx/uvx carry the real name in
-    args (@scope/server), so prefer a package-looking arg over the runner."""
-    for a in args:
-        if a.startswith("-"):
-            continue
-        if "/" in a or a.startswith("@"):
-            base = a.rsplit("/", 1)[-1]  # @scope/server-foo -> server-foo
-            if base:
-                return base
-    base = command.rsplit("/", 1)[-1]
-    return base or command
-
-
-class AddMCPServerScreen(ModalScreen[dict | None]):
-    """Modal form to add one MCP server. Resolves to {name, entry} or None."""
-
-    BINDINGS = [
-        Binding("escape", "cancel", "Cancel", show=False),
-    ]
-
-    DEFAULT_CSS = """
-    AddMCPServerScreen {
-        align: center middle;
-    }
-    AddMCPServerScreen > #add-box {
-        width: 68;
-        max-width: 92%;
-        height: auto;
-        padding: 1 2;
-        border: round $accent;
-        background: $surface;
-    }
-    AddMCPServerScreen #add-title {
-        text-style: bold;
-        padding-bottom: 1;
-    }
-    AddMCPServerScreen #add-help {
-        color: $text-muted;
-        padding-bottom: 1;
-    }
-    AddMCPServerScreen Input {
-        margin-bottom: 1;
-    }
-    AddMCPServerScreen #add-hint {
-        color: $text-muted;
-        padding-top: 1;
-    }
-    """
-
-    def compose(self) -> ComposeResult:
-        with Vertical(id="add-box"):
-            yield Static("Add MCP server", id="add-title")
-            yield Static(
-                "Paste a URL for a remote server, or a command for a local one.",
-                id="add-help",
-            )
-            yield Input(
-                placeholder="https://… URL, or a command like: npx -y @scope/server /path",
-                id="add-target",
-            )
-            yield Input(placeholder="name (optional)", id="add-name")
-            yield Static(
-                "Enter save  ·  Tab next field  ·  Esc cancel", id="add-hint"
-            )
-
-    def on_mount(self) -> None:
-        self.query_one("#add-target", Input).focus()
-
-    # ── build / validate ────────────────────────────────────────────
-
-    def _build(self) -> dict | None:
-        target = self.query_one("#add-target", Input).value.strip()
-        name = self.query_one("#add-name", Input).value.strip()
-        if not target:
-            return None
-
-        if target.lower().startswith(("http://", "https://")):
-            entry: dict = {"transport": "http", "url": target}
-            if not name:
-                name = _derive_name_from_url(target)
-        else:
-            parts = shlex.split(target)
-            if not parts:
-                return None
-            command, args = parts[0], parts[1:]
-            entry = {"command": command}
-            if args:
-                entry["args"] = args
-            if not name:
-                name = _derive_name_from_command(command, args)
-
-        if not name:
-            return None
-        return {"name": name, "entry": entry}
-
-    def _flash(self, msg: str) -> None:
-        try:
-            self.query_one("#add-hint", Static).update(f"[{C.error}]{msg}[/]")
-        except Exception:
-            pass
-
-    # ── actions ─────────────────────────────────────────────────────
-
-    def on_input_submitted(self, event: Input.Submitted) -> None:
-        # Enter in either field tries to save.
-        result = self._build()
-        if result is None:
-            self._flash("Enter a URL or a command first.")
-            return
-        self.dismiss(result)
-
-    def action_cancel(self) -> None:
-        self.dismiss(None)