methodproof 0.7.18__tar.gz → 0.7.20__tar.gz

{methodproof-0.7.18 → methodproof-0.7.20}/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [0.7.20] — 2026-04-11
+
+### Fixed
+- **TUI init layout** — toggle rows now use `height: auto` so long capture descriptions don't clip; `width: 1fr` on `.row-label` overrides the 24-char truncation from base CSS; Switch widgets widened from 4→6 for correct rendering; Full Spectrum status no longer cut off at 1 line
+- **TUI consent layout** — Switch widgets widened from 4→6 (same fix as init)
+- **TUI status markup** — `_token_expiry()` was returning Rich markup strings into `Text.append()`, which treats input as plain text; tags now rendered literally (e.g. `[#d93326]expires soon[/#d93326]`). Fixed to return `(text, style)` tuple and apply styles at call site
+- **Session complete timeout** — `mp push` was using 15s for the final `/complete` call, which times out on large sessions while the server drains the ingest queue and materializes stats. Raised to 90s. Timeout is now a configurable parameter on internal request helpers
+
 ## [0.7.6] — 2026-04-08
 
 ### Added

{methodproof-0.7.18 → methodproof-0.7.20}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: methodproof
-Version: 0.7.18
+Version: 0.7.20
 Summary: See how you code. Capture and visualize your engineering process.
 License-Expression: Apache-2.0
 License-File: LICENSE

{methodproof-0.7.18 → methodproof-0.7.20}/methodproof/__init__.py

@@ -1,3 +1,3 @@
 """MethodProof — see how you code."""
 
-__version__ = "0.7.18"
+__version__ = "0.7.19"

{methodproof-0.7.18 → methodproof-0.7.20}/methodproof/cli.py

@@ -1398,7 +1398,6 @@ def cmd_log(args: argparse.Namespace) -> None:
             return
         print(f"Found {len(empty)} empty session{'s' if len(empty) != 1 else ''}:")
         for s in empty:
-            from datetime import datetime, UTC
             dt = datetime.fromtimestamp(s["created_at"], tz=UTC).strftime("%Y-%m-%d %H:%M")
             print(f"  {s['id'][:8]}  {dt}  {s.get('watch_dir', '?')}")
         answer = input(f"\nDelete {len(empty)} empty session{'s' if len(empty) != 1 else ''}? [y/N]: ").strip().lower()
@@ -1646,13 +1645,15 @@ def cmd_login(args: argparse.Namespace) -> None:
     api = args.api_url or cfg["api_url"]
 
     if cfg.get("token") and not getattr(args, "force", False):
-        current = cfg.get("email") or cfg.get("account_id", "")[:8] or "an account"
-        print(f"Already logged in as {current}.")
-        answer = input("  Switch accounts? [y/N]: ").strip().lower()
-        if answer not in ("y", "yes"):
-            return
-        # Stash current profile before switching
-        config.save_active_profile(cfg)
+        if getattr(args, "add", False):
+            config.save_active_profile(cfg)
+        else:
+            current = cfg.get("email") or cfg.get("account_id", "")[:8] or "an account"
+            print(f"Already logged in as {current}.")
+            answer = input("  Switch accounts? [y/N]: ").strip().lower()
+            if answer not in ("y", "yes"):
+                return
+            config.save_active_profile(cfg)
 
     # Start device auth flow
     result = _request("POST", "/auth/cli/start", api, "")
@@ -1661,6 +1662,9 @@ def cmd_login(args: argparse.Namespace) -> None:
     user_code = result.get("user_code", "")
     verification_url = result.get("verification_url", "")
 
+    if getattr(args, "add", False):
+        auth_url += ("&" if "?" in auth_url else "?") + "mode=add"
+
     print(f"\nOpening browser to sign in...\n")
     print(f"  {auth_url}\n")
     if user_code and verification_url:
@@ -1692,11 +1696,13 @@ def cmd_login(args: argparse.Namespace) -> None:
                     _setup_master_key(cfg)
                 from methodproof.sync import sync_research_consent
                 sync_research_consent(cfg["token"], cfg["api_url"])
-                label = cfg.get("email") or cfg.get("account_id", "")[:8]
-                profiles = cfg.get("profiles", {})
-                n = len(profiles)
-                print(f"Logged in as {label}. {n} account{'s' if n != 1 else ''} on this device.")
-                print("  Quick-swap: `mp switch`")
+                from methodproof.tui.login_success import run as show_success
+                show_success(
+                    display_name=cfg.get("email") or cfg.get("account_id", "")[:8],
+                    email=cfg.get("email", ""),
+                    account_count=len(cfg.get("profiles", {})),
+                    added=getattr(args, "add", False),
+                )
                 return
         except Exception:
             pass
@@ -2134,6 +2140,7 @@ def main() -> None:
     _add_ui_flags(s_status)
     l = sub.add_parser("login", help="Connect to platform")
     l.add_argument("--api-url")
+    l.add_argument("--add", action="store_true", help="Add another account to this device")
     l.add_argument("--force", "-f", action="store_true", help="Skip switch-account prompt")
     l.add_argument("--no-key", action="store_true", help="Skip master key generation (test accounts)")
     sub.add_parser("logout", help="Clear login credentials (keeps consent and sessions)")

{methodproof-0.7.18 → methodproof-0.7.20}/methodproof/tui/consent.py

@@ -53,7 +53,7 @@ _CSS = BASE_CSS + f"""
 }}
 .toggle-row Switch {{
     margin: 0 1 0 0;
-    width: 4;
+    width: 6;
 }}
 .pro-row .row-label {{
     color: {PURPLE};

{methodproof-0.7.18 → methodproof-0.7.20}/methodproof/tui/init.py

@@ -34,14 +34,16 @@ PrefsScreen {{
     margin: 0 0 1 0;
 }}
 .toggle-row {{
-    height: 2;
+    height: auto;
     align: left middle;
+    margin: 0 0 1 0;
 }}
 .toggle-row Switch {{
     margin: 0 1 0 0;
-    width: 4;
+    width: 6;
 }}
 .row-label {{
+    width: 1fr;
     color: {TEXT};
 }}
 .pro-row .row-label {{
@@ -55,16 +57,17 @@ PrefsScreen {{
     width: 5;
 }}
 .opt-row {{
-    height: 2;
+    height: auto;
     align: left middle;
+    margin: 0 0 1 0;
 }}
 .opt-row Switch {{
     margin: 0 1 0 0;
-    width: 4;
+    width: 6;
 }}
 #fs-status {{
     color: {DIM};
-    height: 1;
+    height: auto;
     margin: 1 0 0 0;
 }}
 #fs-status.full {{

methodproof-0.7.20/methodproof/tui/login_success.py

@@ -0,0 +1,41 @@
+"""Rich success panel shown after mp login / mp login --add."""
+from __future__ import annotations
+
+from rich.console import Console
+from rich.panel import Panel
+from rich.text import Text
+
+from methodproof.tui.theme import BORDER, DIM, GOLD, GREEN, TEXT
+
+_CONSOLE = Console()
+
+
+def run(display_name: str, email: str, account_count: int, added: bool) -> None:
+    """Render post-login success panel. Always shown regardless of ui_mode."""
+    body = Text()
+    body.append("✓  ", style=f"bold {GREEN}")
+
+    if added:
+        body.append("You have successfully added ", style=TEXT)
+        body.append(display_name, style=f"bold {TEXT}")
+        body.append(" authentication to your CLI.", style=TEXT)
+    else:
+        body.append("Signed in as ", style=TEXT)
+        body.append(display_name, style=f"bold {TEXT}")
+        body.append(".", style=TEXT)
+
+    if email and email != display_name:
+        body.append(f"\n   {email}", style=DIM)
+
+    n = account_count
+    body.append(
+        f"\n\n   {n} account{'s' if n != 1 else ''} on this device  ·  ",
+        style=DIM,
+    )
+    body.append("mp switch", style=f"bold {GOLD}")
+    body.append(" to swap", style=DIM)
+
+    title = f"[{GOLD}]  {'Added to CLI' if added else 'Logged In'}  [/{GOLD}]"
+    _CONSOLE.print()
+    _CONSOLE.print(Panel(body, title=title, border_style=BORDER, padding=(0, 1)))
+    _CONSOLE.print()

{methodproof-0.7.18 → methodproof-0.7.20}/methodproof/tui/status.py

@@ -19,15 +19,16 @@ def _tier_color(tier: str) -> str:
     return {"free": DIM, "basic": TEXT, "pro": GOLD, "team": PURPLE}.get(tier.lower(), TEXT)
 
 
-def _token_expiry(cfg: dict) -> str:
+def _token_expiry(cfg: dict) -> tuple[str, str]:
+    """Return (text, style) for token expiry display."""
     last_auth = cfg.get("last_auth_at", 0)
     if not last_auth:
-        return "unknown"
+        return "unknown", DIM
     age_h = (time.time() - last_auth) / 3600
     remaining_h = max(0, 24 - age_h)
     if remaining_h < 1:
-        return f"[{RED}]expires soon[/{RED}]"
-    return f"[{DIM}]expires in {int(remaining_h)}h[/{DIM}]"
+        return "expires soon", RED
+    return f"expires in {int(remaining_h)}h", DIM
 
 
 def run(cfg: dict) -> None:
@@ -52,7 +53,8 @@ def run(cfg: dict) -> None:
     acct_lines.append("  ·  ", style=DIM)
     acct_lines.append(tier, style=_tier_color(tier))
     acct_lines.append("\n")
-    acct_lines.append(_token_expiry(cfg), style=DIM)
+    expiry_text, expiry_style = _token_expiry(cfg)
+    acct_lines.append(expiry_text, style=expiry_style)
 
     console.print(Panel(
         acct_lines,

{methodproof-0.7.18 → methodproof-0.7.20}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "methodproof"
-version = "0.7.18"
+version = "0.7.20"
 description = "See how you code. Capture and visualize your engineering process."
 requires-python = ">=3.11"
 dependencies = ["watchdog>=4.0", "websocket-client>=1.7", "cryptography>=43.0", "keyring>=25.0", "textual>=0.59", "rich>=13.7"]

{methodproof-0.7.18 → methodproof-0.7.20}/tests/conftest.py

@@ -27,7 +27,8 @@ def isolate_fs(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> Path:
     monkeypatch.setattr(config, "CONFIG", mp_dir / "config.json")
     monkeypatch.setattr(config, "DB_PATH", mp_dir / "methodproof.db")
     monkeypatch.setattr(config, "CMD_LOG", mp_dir / "commands.jsonl")
-    (mp_dir / "config.json").write_text(json.dumps(dict(config._DEFAULTS), indent=2))
+    defaults = {**config._DEFAULTS, "ui_mode": False}
+    (mp_dir / "config.json").write_text(json.dumps(defaults, indent=2))
     monkeypatch.setattr(store, "_conn", None)
     store.init_db()
     return tmp_path

{methodproof-0.7.18 → methodproof-0.7.20}/tests/test_cli_auth.py

@@ -215,7 +215,7 @@ def test_login_happy_path(mock_req, mock_sleep, mock_browser, mock_key, mock_con
     assert cfg["account_id"] == "new-user"
     mock_browser.assert_called_once()
     out = capsys.readouterr().out
-    assert "Logged in" in out
+    assert "Logged In" in out
 
 
 @patch("methodproof.sync._request")

{methodproof-0.7.18 → methodproof-0.7.20}/tests/test_cli_share.py

@@ -98,7 +98,7 @@ def test_review_shows_breakdown(make_session, cli_args, capsys):
     out = capsys.readouterr().out
     assert "5" in out
     assert "events" in out
-    assert "fields:" in out
+    assert "fields" in out
 
 
 # ── cmd_view ──

{methodproof-0.7.18 → methodproof-0.7.20}/tests/test_cli_start.py

@@ -427,21 +427,11 @@ def test_consent_detailed_redaction_toggle():
 # ── cmd_init ──
 
 
-@patch("methodproof.integrity.has_keypair", return_value=True)
-@patch("methodproof.sync.sync_research_consent")
-@patch("methodproof.hooks.wrappers.install", return_value=[])
-@patch("methodproof.mcp.register_with_claude", return_value="registered")
-@patch("methodproof.hooks.install.install", return_value="installed")
-@patch("methodproof.hooks.openclaw_install.install", return_value="installed")
-@patch("methodproof.hook.install", return_value="hook installed")
-def test_init_first_run(mock_hook, mock_oc, mock_claude, mock_mcp, mock_wrap, mock_consent,
-                         mock_keypair, cli_args, capsys):
-    # input() calls: auto-update, alias, local AI ports
-    with patch("methodproof.cli._run_consent", return_value=config.load()) as mock_run_consent, \
-         patch("builtins.input", side_effect=["N", "N", "N"]):
-        cli.cmd_init(cli_args())
-    mock_run_consent.assert_called_once()
-    mock_hook.assert_called_once()
+@patch("methodproof.tui.init.run")
+def test_init_first_run(mock_tui_init, cli_args):
+    # First run: consent_acknowledged=False forces TUI regardless of ui_mode
+    cli.cmd_init(cli_args())
+    mock_tui_init.assert_called_once()
 
 
 @patch("methodproof.integrity.has_keypair", return_value=True)
@@ -450,7 +440,7 @@ def test_init_already_configured(mock_hook, mock_keypair, cli_args, capsys):
     cfg = config.load()
     cfg["consent_acknowledged"] = True
     config.save(cfg)
-    # input() calls: auto-update, alias, local AI ports
-    with patch("builtins.input", side_effect=["N", "N", "N"]):
+    # input() calls: auto-update, alias, ui-mode, local AI ports
+    with patch("builtins.input", side_effect=["N", "N", "N", "N"]):
         cli.cmd_init(cli_args())
     mock_hook.assert_called_once()

methodproof-0.7.18/.code-review-graph/.gitignore

@@ -1,3 +0,0 @@
-# Auto-generated by code-review-graph — do not commit database files.
-# The graph.db contains absolute paths and code structure metadata.
-*