nimcode 0.1.18__tar.gz → 0.1.21__tar.gz

{nimcode-0.1.18 → nimcode-0.1.21}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nimcode
-Version: 0.1.18
+Version: 0.1.21
 Summary: A standalone, robust coding agent for NVIDIA NIM models.
 Author: Autonomous Agent
 Requires-Dist: httpx>=0.27.0

{nimcode-0.1.18 → nimcode-0.1.21}/setup.py

@@ -2,7 +2,7 @@ from setuptools import setup, find_packages
 
 setup(
     name="nimcode",
-    version="0.1.18",
+    version="0.1.21",
     packages=find_packages(where="src"),
     package_dir={"": "src"},
     install_requires=[

{nimcode-0.1.18 → nimcode-0.1.21}/src/nimcode/cli.py

@@ -14,15 +14,20 @@ def run_login():
     console.print("Get your API key from [bold underline blue]https://build.nvidia.com/[/bold underline blue]")
     
     import getpass
-    import keyring
+    from .config import save_global_setting
     
-    api_key = getpass.getpass("Enter your NVIDIA NIM API Key: ")
+    try:
+        api_key = getpass.getpass("Enter your NVIDIA NIM API Key: ")
+    except (EOFError, Exception):
+        console.print("[red]Could not prompt for API key. Please set NIM_API_KEY environment variable or run 'nimcode login' in a regular terminal.[/red]")
+        sys.exit(1)
+        
     if not api_key.strip():
         console.print("[red]API Key cannot be empty.[/red]")
         return
         
-    keyring.set_password("nimcode", "api_key", api_key.strip())
-    console.print("[green][OK] API Key saved securely to OS Keyring[/green]")
+    save_global_setting("api_key", api_key.strip())
+    console.print("[green][OK] API Key saved securely to settings[/green]")
 
 def run_doctor():
     console.print("[bold cyan]NimCode Doctor[/bold cyan] - Diagnostics")
@@ -69,7 +74,7 @@ def main():
         elif sys.argv[1] == "install-hook":
             install_hook()
             return
-        elif sys.argv[1] == "login":
+        elif sys.argv[1] == "login" or sys.argv[1] == "/login":
             run_login()
             return
         
@@ -86,13 +91,12 @@ def main():
     
     settings = load_settings()
     
-    import keyring
-    final_key = args.api_key or os.environ.get("NIM_API_KEY") or keyring.get_password("nimcode", "api_key") or settings.get("api_key")
+    final_key = args.api_key or os.environ.get("NIM_API_KEY") or settings.get("api_key")
     if not final_key:
         console.print("[yellow]No API Key found. Let's get you set up![/yellow]")
         run_login()
         settings = load_settings()
-        final_key = keyring.get_password("nimcode", "api_key") or settings.get("api_key")
+        final_key = settings.get("api_key")
         if not final_key:
             console.print("[bold red]API Key is required to use NimCode. Exiting.[/bold red]")
             sys.exit(1)

{nimcode-0.1.18 → nimcode-0.1.21}/src/nimcode/config.py

@@ -22,20 +22,7 @@ def load_settings() -> Dict[str, Any]:
         except Exception as e:
             logger.error(f"Failed to load global settings: {e}")
             
-    # Scrub legacy api_key if found to ensure it uses keyring
-    if "api_key" in settings:
-        import keyring
-        try:
-            if not keyring.get_password("nimcode", "api_key"):
-                keyring.set_password("nimcode", "api_key", settings["api_key"])
-            
-            # Remove from file
-            del settings["api_key"]
-            with open(global_path, "w", encoding="utf-8") as f:
-                json.dump(settings, f, indent=4)
-        except Exception:
-            pass
-
+    # Keyring caused 30-second hangs on Windows, so we no longer scrub api_key from settings.
     # Local settings
     local_path = os.path.join(os.getcwd(), ".nimcode", "settings.json")
     if os.path.exists(local_path):

{nimcode-0.1.18 → nimcode-0.1.21}/src/nimcode.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nimcode
-Version: 0.1.18
+Version: 0.1.21
 Summary: A standalone, robust coding agent for NVIDIA NIM models.
 Author: Autonomous Agent
 Requires-Dist: httpx>=0.27.0

{nimcode-0.1.18 → nimcode-0.1.21}/tests/test_cli.py

@@ -6,10 +6,10 @@ from nimcode.cli import main, run_login, run_doctor, install_hook
 
 def test_run_login():
     with patch("getpass.getpass", return_value="my_key"):
-        with patch("keyring.set_password") as mock_save:
+        with patch("nimcode.config.save_global_setting") as mock_save:
             with patch("nimcode.cli.console.print") as mock_print:
                 run_login()
-                mock_save.assert_called_with("nimcode", "api_key", "my_key")
+                mock_save.assert_called_with("api_key", "my_key")
                 
 def test_run_login_empty():
     with patch("getpass.getpass", return_value="   "):
@@ -67,10 +67,11 @@ def test_main_install_hook():
             mock_hook.assert_called_once()
 
 def test_main_login():
-    with patch("sys.argv", ["nimcode", "login"]):
+    with patch("sys.argv", ["nimcode", "/login"]):
         with patch("nimcode.cli.run_login") as mock_log:
-            main()
-            mock_log.assert_called_once()
+            with patch("sys.exit"):
+                main()
+                mock_log.assert_called_once()
 
 def test_main_no_key():
     with patch("sys.argv", ["nimcode", "my prompt"]):
@@ -79,7 +80,7 @@ def test_main_no_key():
             with patch("os.environ.get", side_effect=lambda k, d=None: None if k == "NIM_API_KEY" else real_env_get(k, d)):
                 with patch("nimcode.cli.run_login"):
                     with patch("sys.stdin.isatty", return_value=True):
-                        with patch("keyring.get_password", return_value=None):
+                        with patch("builtins.input", side_effect=EOFError):
                             with pytest.raises(SystemExit):
                                 main()
 
@@ -91,8 +92,9 @@ def test_main_with_key():
                 mock_agent_class.return_value = mock_agent
                 with patch("nimcode.cli.asyncio.run") as mock_run:
                     with patch("sys.stdin.isatty", return_value=True):
-                        with patch("keyring.get_password", return_value=None):
-                            with patch("rich.prompt.Prompt.ask", return_value="1"):
+                        with patch("rich.prompt.Prompt.ask", return_value="1"):
+                            with patch("nimcode.nim_client.NimClient") as mock_client:
+                                mock_client.return_value.get_available_models.return_value = []
                                 main()
                                 assert mock_run.called
 
@@ -103,8 +105,9 @@ def test_main_piped_input():
                 with patch("sys.stdin.isatty", return_value=False):
                     with patch("sys.stdin.read", return_value="piped"):
                         with patch("nimcode.cli.asyncio.run") as mock_run:
-                            with patch("keyring.get_password", return_value=None):
-                                with patch("rich.prompt.Prompt.ask", return_value="1"):
+                            with patch("rich.prompt.Prompt.ask", return_value="1"):
+                                with patch("nimcode.nim_client.NimClient") as mock_client:
+                                    mock_client.return_value.get_available_models.return_value = []
                                     main()
                                     assert mock_run.called
 
@@ -114,7 +117,8 @@ def test_main_repl():
             with patch("nimcode.agent.Agent") as mock_agent_class:
                 with patch("sys.stdin.isatty", return_value=True):
                     with patch("nimcode.cli.asyncio.run") as mock_run:
-                        with patch("keyring.get_password", return_value=None):
-                            with patch("rich.prompt.Prompt.ask", return_value="1"):
+                        with patch("rich.prompt.Prompt.ask", return_value="1"):
+                            with patch("nimcode.nim_client.NimClient") as mock_client:
+                                mock_client.return_value.get_available_models.return_value = []
                                 main()
                                 assert mock_run.called