proteworks 0.1.1__tar.gz → 0.1.2__tar.gz

{proteworks-0.1.1 → proteworks-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: proteworks
-Version: 0.1.1
+Version: 0.1.2
 Summary: Sync files with Proteworks using file-browser paths.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{proteworks-0.1.1 → proteworks-0.1.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "proteworks"
-version = "0.1.1"
+version = "0.1.2"
 description = "Sync files with Proteworks using file-browser paths."
 readme = "README.md"
 requires-python = ">=3.11"

{proteworks-0.1.1 → proteworks-0.1.2}/src/proteworks/auth.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 
 import base64
 import hashlib
+import os
 import secrets
 import webbrowser
 from http.server import BaseHTTPRequestHandler, HTTPServer
@@ -65,6 +66,18 @@ def loopback_redirect_uri(port: int = LOOPBACK_PORT) -> str:
     return f"http://127.0.0.1:{port}{LOOPBACK_REDIRECT_PATH}"
 
 
+def open_browser(url: str) -> bool:
+    """Open the sign-in page. The browser's own startup errors stay hidden."""
+    saved_stderr = os.dup(2)
+    try:
+        with open(os.devnull, "wb") as discarded:
+            os.dup2(discarded.fileno(), 2)
+            return bool(webbrowser.open(url))
+    finally:
+        os.dup2(saved_stderr, 2)
+        os.close(saved_stderr)
+
+
 def login(settings: Settings, token_store: TokenStore, stderr) -> None:
     """Open a browser, receive the loopback redirect, and save the access token."""
     verifier, challenge = pkce_pair()
@@ -77,9 +90,8 @@ def login(settings: Settings, token_store: TokenStore, stderr) -> None:
         ) from exc
     redirect_uri = loopback_redirect_uri()
     url = authorize_url(settings, redirect_uri, state, challenge)
-    print(f"Auth0 must allow this callback URL: {redirect_uri}", file=stderr)
     print("Opening a browser to sign in.", file=stderr)
-    opened = webbrowser.open(url)
+    opened = open_browser(url)
     if not opened:
         print("Open this URL to sign in:", file=stderr)
         print(url, file=stderr)

{proteworks-0.1.1 → proteworks-0.1.2}/src/proteworks.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: proteworks
-Version: 0.1.1
+Version: 0.1.2
 Summary: Sync files with Proteworks using file-browser paths.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{proteworks-0.1.1 → proteworks-0.1.2}/tests/test_cli.py

@@ -5,12 +5,15 @@ from __future__ import annotations
 import hashlib
 import json
 import mimetypes
+import os
+import subprocess
+import sys
 import threading
 from http.server import BaseHTTPRequestHandler, ThreadingHTTPServer
 from pathlib import Path
 from urllib.parse import parse_qs, urlparse
 
-from proteworks.auth import loopback_redirect_uri
+from proteworks.auth import loopback_redirect_uri, open_browser
 from proteworks.cli import main
 from proteworks.config import Settings
 from proteworks.push import _iter_local_files
@@ -220,6 +223,31 @@ def test_login_callback_url_is_fixed():
     assert loopback_redirect_uri() == "http://127.0.0.1:8765/callback"
 
 
+def test_open_browser_hides_child_stderr(monkeypatch):
+    def noisy_open(_url: str) -> bool:
+        subprocess.run(
+            [sys.executable, "-c", "import os; os.write(2, b'zygote broken pipe\\n')"],
+            check=False,
+        )
+        return True
+
+    monkeypatch.setattr("proteworks.auth.webbrowser.open", noisy_open)
+    read_fd, write_fd = os.pipe()
+    saved = os.dup(2)
+    os.dup2(write_fd, 2)
+    os.close(write_fd)
+    try:
+        assert open_browser("https://example.test/authorize") is True
+        os.write(2, b"still open\n")
+    finally:
+        os.dup2(saved, 2)
+        os.close(saved)
+    captured = os.read(read_fd, 1024)
+    os.close(read_fd)
+    assert b"zygote" not in captured
+    assert b"still open" in captured
+
+
 def test_root_help_lists_commands(capsys):
     try:
         main(["--help"])