mcp-cli-skill 0.8.0__tar.gz → 0.8.2__tar.gz

mcp_cli_skill-0.8.2/.github/workflows/publish.yml

@@ -0,0 +1,40 @@
+# Tests on every push/PR; a `v*` tag also publishes to PyPI via Trusted Publishing (OIDC, no tokens).
+name: ci
+
+on:
+  push:
+    branches: [master]
+    tags: ["v*"]
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python: ["3.10", "3.13"]
+    steps:
+      - uses: actions/checkout@v5
+      - uses: actions/setup-python@v6
+        with:
+          python-version: ${{ matrix.python }}
+      - run: python -m unittest discover tests -v
+
+  publish:
+    if: startsWith(github.ref, 'refs/tags/v')
+    needs: test
+    runs-on: ubuntu-latest
+    environment: pypi
+    permissions:
+      id-token: write  # PyPI Trusted Publishing
+    steps:
+      - uses: actions/checkout@v5
+      - uses: actions/setup-python@v6
+        with:
+          python-version: "3.13"
+      - name: Tag must match __version__
+        run: |
+          v=$(sed -n 's/^__version__ = "\(.*\)"/\1/p' src/mcp_cli_skill/__init__.py)
+          test "v$v" = "$GITHUB_REF_NAME" || { echo "tag $GITHUB_REF_NAME != __version__ $v"; exit 1; }
+      - run: python -m pip install build && python -m build
+      - uses: pypa/gh-action-pypi-publish@release/v1

{mcp_cli_skill-0.8.0 → mcp_cli_skill-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: mcp-cli-skill
-Version: 0.8.0
+Version: 0.8.2
 Summary: Call any MCP server tool from the command line with shell composition support
 Project-URL: Homepage, https://github.com/wise-toddler/mcp-cli-skill
 Project-URL: Repository, https://github.com/wise-toddler/mcp-cli-skill
@@ -65,6 +65,13 @@ mcp-call emergent --tools     # bearer token is sent and refreshed automatically
 mcp-call --logout emergent    # forget the local tokens (no server-side revocation)
 ```
 
+Multiple accounts on one server: add it under one name per account, and log in with `--no-browser`, which only prints the URL so you can open it in the browser profile signed in to that account (the default browser may silently approve as whoever is already signed in):
+
+```bash
+mcp-call --add-http emergent-work https://mcp.emergent.sh/
+mcp-call --login emergent-work --no-browser
+```
+
 Tokens live in `~/.mcp-cli/tokens.json` (mode 0600), bound to the server's URL. A call to a server that needs login exits with code 4 and says which `--login` to run. `--login` waits up to `MCP_CALL_LOGIN_TIMEOUT` seconds (default 300) for the browser; over SSH it prints the URL instead of opening a browser. A static `Authorization` header (`--header`) always wins over OAuth.
 
 ## Shell completion

{mcp_cli_skill-0.8.0 → mcp_cli_skill-0.8.2}/README.md

@@ -49,6 +49,13 @@ mcp-call emergent --tools     # bearer token is sent and refreshed automatically
 mcp-call --logout emergent    # forget the local tokens (no server-side revocation)
 ```
 
+Multiple accounts on one server: add it under one name per account, and log in with `--no-browser`, which only prints the URL so you can open it in the browser profile signed in to that account (the default browser may silently approve as whoever is already signed in):
+
+```bash
+mcp-call --add-http emergent-work https://mcp.emergent.sh/
+mcp-call --login emergent-work --no-browser
+```
+
 Tokens live in `~/.mcp-cli/tokens.json` (mode 0600), bound to the server's URL. A call to a server that needs login exits with code 4 and says which `--login` to run. `--login` waits up to `MCP_CALL_LOGIN_TIMEOUT` seconds (default 300) for the browser; over SSH it prints the URL instead of opening a browser. A static `Authorization` header (`--header`) always wins over OAuth.
 
 ## Shell completion

{mcp_cli_skill-0.8.0 → mcp_cli_skill-0.8.2}/SKILL.md

@@ -14,7 +14,7 @@ Always use `mcp-call` as the command, never `uvx mcp-cli-skill`.
 
 ```bash
 mcp-call --servers                                    # list configured servers
-mcp-call <server> --tools                             # list tools (human-readable)
+mcp-call <server> --tools                             # list tools (human-readable; --list / --help work too)
 mcp-call <server> --discover                          # list tools as JSON with schemas
 mcp-call <server> <tool> --schema                     # show tool's input schema as JSON
 mcp-call <server> <tool> --key=value ...              # call a tool
@@ -23,7 +23,7 @@ echo '{}' | mcp-call <server> <tool>                  # call with stdin JSON
 mcp-call --add <name> <cmd> [args] [--env K=V ...]    # add stdio server
 mcp-call --add-http <name> <url> [--header 'K: V' ...] # add HTTP server (auth via --header/-H)
 mcp-call --remove <name>                              # remove server
-mcp-call --login <name>                               # OAuth browser login (the user runs this, not you)
+mcp-call --login <name> [--no-browser]                # OAuth browser login (the user runs this, not you)
 mcp-call --logout <name>                              # forget stored OAuth tokens
 mcp-call --sync                                       # re-sync from Claude configs
 mcp-call --completion <bash|zsh|fish>                 # print shell completion hook
@@ -42,6 +42,8 @@ mcp-call --remove myredash
 mcp-call --sync
 ```
 
+An unknown option or tool name exits 2 with a "Did you mean" hint and is never sent to the server — use the hint instead of guessing another name.
+
 If a call exits 4 with "needs interactive browser login", ask the user to run the `mcp-call --login <name>` command it prints; don't run it yourself.
 
 ## Examples

{mcp_cli_skill-0.8.0 → mcp_cli_skill-0.8.2}/scripts/mcp_call.py

@@ -1,6 +1,7 @@
 #!/usr/bin/env python3
 """Call any MCP server tool from CLI with --flag=value args."""
 import base64
+import difflib
 import hashlib
 import http.server
 import json
@@ -189,7 +190,7 @@ def parse_args():
         print("       mcp-call --add <name> <command> [args...] [--env KEY=VAL ...]", file=sys.stderr)
         print("       mcp-call --add-http <name> <url>", file=sys.stderr)
         print("       mcp-call --remove <name>", file=sys.stderr)
-        print("       mcp-call --login <name>             (OAuth browser login for an HTTP server)", file=sys.stderr)
+        print("       mcp-call --login <name> [--no-browser]  (OAuth browser login; --no-browser only prints the URL)", file=sys.stderr)
         print("       mcp-call --logout <name>            (forget stored OAuth tokens)", file=sys.stderr)
         print("       mcp-call --sync", file=sys.stderr)
         print("       mcp-call --completion <bash|zsh|fish>", file=sys.stderr)
@@ -231,7 +232,7 @@ def parse_args():
         if len(args) < 2:
             print(f"Usage: mcp-call {args[0]} <name>", file=sys.stderr)
             sys.exit(1)
-        return f"__{args[0][2:]}__", args[1], {}
+        return f"__{args[0][2:]}__", args[1], {"no_browser": "--no-browser" in args[2:]}
     if args[0] == "--sync":
         return "__sync__", None, {}
     if args[0] == "--completion":
@@ -245,11 +246,18 @@ def parse_args():
     server = args[0]
     # `mcp-call <server> --login|--logout` alias the top-level flags
     if len(args) > 1 and args[1] in ("--login", "--logout"):
-        return f"__{args[1][2:]}__", server, {}
-    if len(args) < 2 or args[1] == "--tools":
+        return f"__{args[1][2:]}__", server, {"no_browser": "--no-browser" in args[2:]}
+    # --list / --help / -h are what people (and agents) guess for "list tools"
+    if len(args) < 2 or args[1] in ("--tools", "--list", "--help", "-h"):
         return server, "__tools__", {}
     if args[1] == "--discover":
         return server, "__discover__", {}
+    if args[1].startswith("-"):
+        # MCP tool names never start with "-": a mistyped option, so never send it to the server
+        close = difflib.get_close_matches(args[1], ["--tools", "--discover", "--login", "--logout"], n=1)
+        hint = f" Did you mean {close[0]}?" if close else ""
+        print(f"Error: unknown option '{args[1]}'.{hint} List tools with: mcp-call {server} --tools", file=sys.stderr)
+        sys.exit(2)
 
     tool = args[1]
     tool_args = {}
@@ -669,7 +677,7 @@ def oauth_logout(name):
     return name in _tokens_read() and _tokens_put(name, None) is not None
 
 
-def oauth_login(name, url):
+def oauth_login(name, url, open_browser=True):
     """Interactive browser login: discover, register client (DCR), PKCE authorize, store tokens."""
     # fail fast instead of racing a second login; held until this function returns
     lock = _flock(os.path.join(CONFIG_DIR, "login-" + re.sub(r"[^A-Za-z0-9._-]", "_", name) + ".lock"), 0)
@@ -773,10 +781,14 @@ def oauth_login(name, url):
             params["scope"] = scope
         auth = meta["authorization_endpoint"]
         auth_url = auth + ("&" if urllib.parse.urlsplit(auth).query else "?") + urllib.parse.urlencode(params)
-        print(f"Opening the browser to log in to {name}. If it doesn't open, visit:\n{auth_url}", file=sys.stderr)
+        if open_browser:
+            print(f"Opening the browser to log in to {name}. If it doesn't open, visit:\n{auth_url}", file=sys.stderr)
+        else:
+            # pick the account: open it in whichever browser profile is signed in to it
+            print(f"Open this URL in the browser profile you want to log in to {name} with:\n{auth_url}", file=sys.stderr)
         # over SSH / headless Linux, webbrowser falls back to a blocking text browser
         headless = sys.platform.startswith("linux") and not (os.environ.get("DISPLAY") or os.environ.get("WAYLAND_DISPLAY"))
-        if not os.environ.get("SSH_CONNECTION") and not headless:
+        if open_browser and not os.environ.get("SSH_CONNECTION") and not headless:
             webbrowser.open(auth_url)
         if not done.wait(LOGIN_TIMEOUT):
             _die(f"Error: no callback received in {LOGIN_TIMEOUT}s. If the browser showed an error, run `mcp-call --logout {name}` and retry.")
@@ -1280,6 +1292,14 @@ def do_completion():
 
 # --- Main ---
 
+def _unknown_tool(server_name, tool_name, names):
+    """Exit 2 with close-match suggestions for a tool name the server doesn't have."""
+    close = difflib.get_close_matches(tool_name, names, n=3, cutoff=0.4)
+    hint = f" Did you mean: {', '.join(close)}?" if close else ""
+    print(f"Error: unknown tool '{tool_name}' for {server_name}.{hint} List tools with: mcp-call {server_name} --tools", file=sys.stderr)
+    sys.exit(2)
+
+
 def run_server(config, tool_name, tool_args, server_name=""):
     """Route to HTTP or stdio transport."""
     # tool discovery commands
@@ -1297,18 +1317,21 @@ def run_server(config, tool_name, tool_args, server_name=""):
                 if t["name"] == target:
                     print(json.dumps(t.get("inputSchema", {}), indent=2))
                     return
-            print(f"Error: tool '{target}' not found", file=sys.stderr)
-            sys.exit(1)
+            _unknown_tool(server_name, target, [t["name"] for t in tools])
         elif tool_name == "__help__":
             target = tool_args["_tool"]
             for t in tools:
                 if t["name"] == target:
                     _print_tool_help(server_name or "<server>", t)
                     return
-            print(f"Error: tool '{target}' not found", file=sys.stderr)
-            sys.exit(1)
+            _unknown_tool(server_name, target, [t["name"] for t in tools])
         return
-    # tool calls
+    # tool calls: a name missing from the cached list is a typo or a newly added tool,
+    # so re-fetch once (refreshes the cache) and only refuse if the server lacks it too
+    if server_name and tool_name not in [t.get("name") for t in _cache_read(server_name)]:
+        names = [t.get("name") for t in fetch_tools(config, server_name)]
+        if names and tool_name not in names:  # empty list = server can't list tools; let the call decide
+            _unknown_tool(server_name, tool_name, names)
     if is_http(config):
         http_call_tool(config["url"], tool_name, tool_args, config.get("headers"), server_name)
     else:
@@ -1356,7 +1379,7 @@ def main():
             print(f"Error: '{tool_name}' has a static Authorization header; re-add it without one to use --login.", file=sys.stderr)
             sys.exit(1)
         try:
-            oauth_login(tool_name, _expand_env(cfg["url"]))
+            oauth_login(tool_name, _expand_env(cfg["url"]), open_browser=not tool_args.get("no_browser"))
         except KeyboardInterrupt:
             print("\nLogin cancelled.", file=sys.stderr)
             sys.exit(130)

{mcp_cli_skill-0.8.0 → mcp_cli_skill-0.8.2}/src/mcp_cli_skill/__init__.py

@@ -1,2 +1,2 @@
 """MCP CLI - Call any MCP server tool from the command line."""
-__version__ = "0.8.0"
+__version__ = "0.8.2"

{mcp_cli_skill-0.8.0 → mcp_cli_skill-0.8.2}/src/mcp_cli_skill/cli.py

@@ -1,6 +1,7 @@
 #!/usr/bin/env python3
 """Call any MCP server tool from CLI with --flag=value args."""
 import base64
+import difflib
 import hashlib
 import http.server
 import json
@@ -189,7 +190,7 @@ def parse_args():
         print("       mcp-call --add <name> <command> [args...] [--env KEY=VAL ...]", file=sys.stderr)
         print("       mcp-call --add-http <name> <url>", file=sys.stderr)
         print("       mcp-call --remove <name>", file=sys.stderr)
-        print("       mcp-call --login <name>             (OAuth browser login for an HTTP server)", file=sys.stderr)
+        print("       mcp-call --login <name> [--no-browser]  (OAuth browser login; --no-browser only prints the URL)", file=sys.stderr)
         print("       mcp-call --logout <name>            (forget stored OAuth tokens)", file=sys.stderr)
         print("       mcp-call --sync", file=sys.stderr)
         print("       mcp-call --completion <bash|zsh|fish>", file=sys.stderr)
@@ -231,7 +232,7 @@ def parse_args():
         if len(args) < 2:
             print(f"Usage: mcp-call {args[0]} <name>", file=sys.stderr)
             sys.exit(1)
-        return f"__{args[0][2:]}__", args[1], {}
+        return f"__{args[0][2:]}__", args[1], {"no_browser": "--no-browser" in args[2:]}
     if args[0] == "--sync":
         return "__sync__", None, {}
     if args[0] == "--completion":
@@ -245,11 +246,18 @@ def parse_args():
     server = args[0]
     # `mcp-call <server> --login|--logout` alias the top-level flags
     if len(args) > 1 and args[1] in ("--login", "--logout"):
-        return f"__{args[1][2:]}__", server, {}
-    if len(args) < 2 or args[1] == "--tools":
+        return f"__{args[1][2:]}__", server, {"no_browser": "--no-browser" in args[2:]}
+    # --list / --help / -h are what people (and agents) guess for "list tools"
+    if len(args) < 2 or args[1] in ("--tools", "--list", "--help", "-h"):
         return server, "__tools__", {}
     if args[1] == "--discover":
         return server, "__discover__", {}
+    if args[1].startswith("-"):
+        # MCP tool names never start with "-": a mistyped option, so never send it to the server
+        close = difflib.get_close_matches(args[1], ["--tools", "--discover", "--login", "--logout"], n=1)
+        hint = f" Did you mean {close[0]}?" if close else ""
+        print(f"Error: unknown option '{args[1]}'.{hint} List tools with: mcp-call {server} --tools", file=sys.stderr)
+        sys.exit(2)
 
     tool = args[1]
     tool_args = {}
@@ -669,7 +677,7 @@ def oauth_logout(name):
     return name in _tokens_read() and _tokens_put(name, None) is not None
 
 
-def oauth_login(name, url):
+def oauth_login(name, url, open_browser=True):
     """Interactive browser login: discover, register client (DCR), PKCE authorize, store tokens."""
     # fail fast instead of racing a second login; held until this function returns
     lock = _flock(os.path.join(CONFIG_DIR, "login-" + re.sub(r"[^A-Za-z0-9._-]", "_", name) + ".lock"), 0)
@@ -773,10 +781,14 @@ def oauth_login(name, url):
             params["scope"] = scope
         auth = meta["authorization_endpoint"]
         auth_url = auth + ("&" if urllib.parse.urlsplit(auth).query else "?") + urllib.parse.urlencode(params)
-        print(f"Opening the browser to log in to {name}. If it doesn't open, visit:\n{auth_url}", file=sys.stderr)
+        if open_browser:
+            print(f"Opening the browser to log in to {name}. If it doesn't open, visit:\n{auth_url}", file=sys.stderr)
+        else:
+            # pick the account: open it in whichever browser profile is signed in to it
+            print(f"Open this URL in the browser profile you want to log in to {name} with:\n{auth_url}", file=sys.stderr)
         # over SSH / headless Linux, webbrowser falls back to a blocking text browser
         headless = sys.platform.startswith("linux") and not (os.environ.get("DISPLAY") or os.environ.get("WAYLAND_DISPLAY"))
-        if not os.environ.get("SSH_CONNECTION") and not headless:
+        if open_browser and not os.environ.get("SSH_CONNECTION") and not headless:
             webbrowser.open(auth_url)
         if not done.wait(LOGIN_TIMEOUT):
             _die(f"Error: no callback received in {LOGIN_TIMEOUT}s. If the browser showed an error, run `mcp-call --logout {name}` and retry.")
@@ -1280,6 +1292,14 @@ def do_completion():
 
 # --- Main ---
 
+def _unknown_tool(server_name, tool_name, names):
+    """Exit 2 with close-match suggestions for a tool name the server doesn't have."""
+    close = difflib.get_close_matches(tool_name, names, n=3, cutoff=0.4)
+    hint = f" Did you mean: {', '.join(close)}?" if close else ""
+    print(f"Error: unknown tool '{tool_name}' for {server_name}.{hint} List tools with: mcp-call {server_name} --tools", file=sys.stderr)
+    sys.exit(2)
+
+
 def run_server(config, tool_name, tool_args, server_name=""):
     """Route to HTTP or stdio transport."""
     # tool discovery commands
@@ -1297,18 +1317,21 @@ def run_server(config, tool_name, tool_args, server_name=""):
                 if t["name"] == target:
                     print(json.dumps(t.get("inputSchema", {}), indent=2))
                     return
-            print(f"Error: tool '{target}' not found", file=sys.stderr)
-            sys.exit(1)
+            _unknown_tool(server_name, target, [t["name"] for t in tools])
         elif tool_name == "__help__":
             target = tool_args["_tool"]
             for t in tools:
                 if t["name"] == target:
                     _print_tool_help(server_name or "<server>", t)
                     return
-            print(f"Error: tool '{target}' not found", file=sys.stderr)
-            sys.exit(1)
+            _unknown_tool(server_name, target, [t["name"] for t in tools])
         return
-    # tool calls
+    # tool calls: a name missing from the cached list is a typo or a newly added tool,
+    # so re-fetch once (refreshes the cache) and only refuse if the server lacks it too
+    if server_name and tool_name not in [t.get("name") for t in _cache_read(server_name)]:
+        names = [t.get("name") for t in fetch_tools(config, server_name)]
+        if names and tool_name not in names:  # empty list = server can't list tools; let the call decide
+            _unknown_tool(server_name, tool_name, names)
     if is_http(config):
         http_call_tool(config["url"], tool_name, tool_args, config.get("headers"), server_name)
     else:
@@ -1356,7 +1379,7 @@ def main():
             print(f"Error: '{tool_name}' has a static Authorization header; re-add it without one to use --login.", file=sys.stderr)
             sys.exit(1)
         try:
-            oauth_login(tool_name, _expand_env(cfg["url"]))
+            oauth_login(tool_name, _expand_env(cfg["url"]), open_browser=not tool_args.get("no_browser"))
         except KeyboardInterrupt:
             print("\nLogin cancelled.", file=sys.stderr)
             sys.exit(130)

mcp_cli_skill-0.8.2/tests/test_args.py

@@ -0,0 +1,101 @@
+"""Mistyped options / tool names fail fast with a hint and never reach the server."""
+import json
+import os
+import subprocess
+import sys
+import tempfile
+import textwrap
+import unittest
+
+REPO = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
+
+FAKE_SERVER = textwrap.dedent("""
+    import json, os, sys
+    tools = os.environ.get("FAKE_TOOLS", "echo,slack_channel").split(",")
+    for line in sys.stdin:
+        msg = json.loads(line)
+        if msg.get("id") is None:
+            continue
+        method = msg["method"]
+        if method == "initialize":
+            res = {"protocolVersion": "2024-11-05", "capabilities": {}, "serverInfo": {"name": "fake", "version": "1"}}
+        elif method == "tools/list":
+            res = {"tools": [{"name": n, "inputSchema": {"type": "object", "properties": {}}} for n in tools]}
+        else:  # tools/call: log every name that actually reached the server
+            with open(os.environ["FAKE_LOG"], "a") as f:
+                f.write(msg["params"]["name"] + "\\n")
+            res = {"content": [{"type": "text", "text": "ok " + msg["params"]["name"]}]}
+        sys.stdout.write(json.dumps({"jsonrpc": "2.0", "id": msg["id"], "result": res}) + "\\n")
+        sys.stdout.flush()
+""")
+
+
+class ArgsTest(unittest.TestCase):
+    def setUp(self):
+        self.home = tempfile.mkdtemp()
+        self.log = os.path.join(self.home, "calls.log")
+        script = os.path.join(self.home, "fake_server.py")
+        with open(script, "w") as f:
+            f.write(FAKE_SERVER)
+        os.makedirs(os.path.join(self.home, ".mcp-cli"))
+        with open(os.path.join(self.home, ".mcp-cli", "servers.json"), "w") as f:
+            json.dump({"fake": {"command": sys.executable, "args": [script], "env": {"FAKE_LOG": self.log}}}, f)
+
+    def run_cli(self, *args, tools=None):
+        """Run mcp-call in a subprocess with an isolated HOME; return (exit code, stdout, stderr)."""
+        env = {**os.environ, "HOME": self.home, "PYTHONPATH": os.path.join(REPO, "src"), "NO_COLOR": "1"}
+        if tools:
+            env["FAKE_TOOLS"] = tools
+        p = subprocess.run([sys.executable, "-m", "mcp_cli_skill.cli", *args], env=env, cwd=REPO,
+                           stdin=subprocess.DEVNULL, capture_output=True, text=True, timeout=30)
+        return p.returncode, p.stdout, p.stderr
+
+    def calls(self):
+        if not os.path.exists(self.log):
+            return []
+        with open(self.log) as f:
+            return f.read().split()
+
+    def test_list_help_aliases_show_tools(self):
+        for flag in ("--tools", "--list", "--help", "-h"):
+            code, out, err = self.run_cli("fake", flag)
+            self.assertEqual(code, 0, f"{flag}: {err}")
+            self.assertIn("echo", out, flag)
+        self.assertEqual(self.calls(), [])
+
+    def test_unknown_option_fails_fast(self):
+        code, _, err = self.run_cli("fake", "--list-tools")
+        self.assertEqual(code, 2)
+        self.assertIn("unknown option '--list-tools'", err)
+        self.assertIn("Did you mean --tools?", err)
+        self.assertIn("mcp-call fake --tools", err)
+        self.assertEqual(self.calls(), [])
+
+    def test_unknown_tool_suggests_and_never_calls(self):
+        code, _, err = self.run_cli("fake", "ehco")
+        self.assertEqual(code, 2)
+        self.assertIn("unknown tool 'ehco'", err)
+        self.assertIn("Did you mean: echo", err)
+        self.assertEqual(self.calls(), [])
+
+    def test_known_tool_still_called(self):
+        code, out, err = self.run_cli("fake", "echo")
+        self.assertEqual(code, 0, err)
+        self.assertIn("ok echo", out)
+        self.assertEqual(self.calls(), ["echo"])
+
+    def test_stale_cache_new_tool_still_callable(self):
+        self.assertEqual(self.run_cli("fake", "echo")[0], 0)  # caches echo,slack_channel
+        code, out, err = self.run_cli("fake", "brand_new", tools="echo,slack_channel,brand_new")
+        self.assertEqual(code, 0, err)
+        self.assertIn("ok brand_new", out)
+        self.assertEqual(self.calls(), ["echo", "brand_new"])
+
+    def test_schema_unknown_tool_suggests(self):
+        code, _, err = self.run_cli("fake", "ech", "--schema")
+        self.assertEqual(code, 2)
+        self.assertIn("Did you mean: echo", err)
+
+
+if __name__ == "__main__":
+    unittest.main()

{mcp_cli_skill-0.8.0 → mcp_cli_skill-0.8.2}/tests/test_oauth.py

@@ -201,9 +201,9 @@ class OAuthTest(unittest.TestCase):
         threading.Thread(target=go, daemon=True).start()
         return True
 
-    def run_cli(self, *args):
+    def run_cli(self, *args, err=None):
         """Run mcp-call in-process; return (exit code, stdout, stderr)."""
-        out, err, code = io.StringIO(), io.StringIO(), 0
+        out, err, code = io.StringIO(), err or io.StringIO(), 0
         with mock.patch.object(sys, "argv", ["mcp-call", *args]), mock.patch.object(sys, "stdin", io.StringIO("")), \
                 redirect_stdout(out), redirect_stderr(err):
             try:
@@ -363,6 +363,25 @@ class OAuthTest(unittest.TestCase):
         self.assertEqual(self.fake.registers, 0)
         self.assertFalse(os.path.exists(cli.TOKENS_PATH))
 
+    def test_14_no_browser_prints_url_only(self):
+        opened = []
+        self.browser = opened.append  # webbrowser.open must not be called
+        test = self
+
+        class User(io.StringIO):
+            """Fake user: opens the printed authorize URL in 'another browser profile'."""
+            def write(self, text):
+                for word in text.split():
+                    if "/authorize?" in word:
+                        test.open_in_thread(word)
+                return super().write(text)
+
+        code, out, err = self.run_cli("fake", "--login", "--no-browser", err=User())
+        self.assertEqual(code, 0, err)
+        self.assertEqual(opened, [])
+        self.assertIn("browser profile", err)
+        self.assertIn("Logged in to fake.", out)
+
 
 if __name__ == "__main__":
     unittest.main()