refactorai-cli 0.3.9__tar.gz → 0.3.12__tar.gz

{refactorai_cli-0.3.9 → refactorai_cli-0.3.12}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: refactorai-cli
-Version: 0.3.9
+Version: 0.3.12
 Summary: Local-first CLI for the refactor platform
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
@@ -8,7 +8,7 @@ Requires-Dist: typer>=0.12.0
 Requires-Dist: httpx>=0.27.0
 Requires-Dist: rich>=13.7.0
 Requires-Dist: PyYAML>=6.0.1
-Requires-Dist: refactorai-core>=3.1.0
+Requires-Dist: refactorai-core>=3.1.1
 
 # refactorai-cli
 

{refactorai_cli-0.3.9 → refactorai_cli-0.3.12}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "refactorai-cli"
-version = "0.3.9"
+version = "0.3.12"
 description = "Local-first CLI for the refactor platform"
 readme = "README.md"
 requires-python = ">=3.11"
@@ -12,7 +12,7 @@ dependencies = [
   "httpx>=0.27.0",
   "rich>=13.7.0",
   "PyYAML>=6.0.1",
-  "refactorai-core>=3.1.0",
+  "refactorai-core>=3.1.1",
 ]
 
 [project.scripts]

{refactorai_cli-0.3.9 → refactorai_cli-0.3.12}/refactorai_cli/__init__.py

@@ -5,4 +5,4 @@ the shared `refactor_core` pipeline from a project folder while staying
 authenticated to the hosted platform via a developer key.
 """
 
-__version__ = "0.3.9"
+__version__ = "0.3.12"

{refactorai_cli-0.3.9 → refactorai_cli-0.3.12}/refactorai_cli/cloud_rr.py

@@ -22,6 +22,8 @@ this design fixes. ``tests/test_cloud_rr_client_boundary.py`` enforces this.
 
 from __future__ import annotations
 
+from collections import deque
+import contextlib
 import hashlib
 import re
 import shlex
@@ -135,6 +137,55 @@ def _run_test(
     return False, (proc.stdout or proc.stderr or "")[-500:]
 
 
+def _run_command_with_progress(
+    project_root: Path,
+    command: str,
+    *,
+    env: dict | None = None,
+    progress_cb: Callable[[dict], None] | None = None,
+    label: str = "",
+    timeout: int = 1800,
+) -> tuple[bool, str]:
+    """Run a host command and stream output lines as progress events.
+
+    Used for long-running dependency setup steps (e.g. ``npm install``) so the
+    CLI does not appear stuck while the process is active.
+    """
+    try:
+        proc = subprocess.Popen(
+            command,
+            shell=True,
+            cwd=str(project_root),
+            stdout=subprocess.PIPE,
+            stderr=subprocess.STDOUT,
+            text=True,
+            bufsize=1,
+            env=env,
+        )
+    except (subprocess.SubprocessError, OSError) as exc:
+        return False, f"command error: {exc}"
+
+    tail: deque[str] = deque(maxlen=30)
+    try:
+        if proc.stdout is not None:
+            for raw in iter(proc.stdout.readline, ""):
+                line = raw.rstrip()
+                if not line:
+                    continue
+                tail.append(line)
+                detail = f"{label}: {line}" if label else line
+                _emit(progress_cb, phase="rr", event="toolchain_provisioning", detail=detail[:240])
+        code = proc.wait(timeout=timeout)
+    except subprocess.TimeoutExpired:
+        proc.kill()
+        with contextlib.suppress(Exception):
+            proc.wait(timeout=2)
+        return False, f"command timed out after {timeout}s: {command}"
+    if code == 0:
+        return True, "ok"
+    return False, ("\n".join(tail))[-700:]
+
+
 def _leading_binary(command: str | None) -> str | None:
     """Return the leading executable of a simple command, or None.
 
@@ -193,6 +244,21 @@ def _managed_toolchain_enabled(config: dict) -> bool:
     return True
 
 
+def _detected_is_provisionable(command: str | None) -> bool:
+    """True when ``command``'s leading binary maps to a managed runtime.
+
+    Lets the resolver keep a detected command (e.g. ``npm test``) whose runtime
+    is missing from the host PATH but which the managed toolchain can download,
+    instead of discarding it and deferring the run.
+    """
+    from refactor_core.host_toolchain import binary_to_runtime
+
+    binary = _leading_binary(command)
+    if not binary:
+        return False
+    return binary_to_runtime(binary) is not None
+
+
 def _provision_toolchain(
     project_root: Path,
     test_command: str | None,
@@ -251,16 +317,72 @@ def _restore_dependencies(
     present. Failures are non-fatal: the test run surfaces the real error.
     """
     if runtime == "node" and (project_root / "package.json").is_file():
-        _emit(progress_cb, phase="rr", event="toolchain_provisioning", detail="npm install")
-        _run_test(project_root, "npm install", env=env)
+        _emit(
+            progress_cb,
+            phase="rr",
+            event="toolchain_provisioning",
+            detail="step 1/1: npm install (streaming output below)",
+        )
+        ok, detail = _run_command_with_progress(
+            project_root,
+            "npm install",
+            env=env,
+            progress_cb=progress_cb,
+            label="npm",
+            timeout=1800,
+        )
+        if not ok:
+            _emit(
+                progress_cb,
+                phase="rr",
+                event="toolchain_provisioning",
+                detail=f"npm install finished with errors: {detail}",
+            )
     elif runtime == "python":
         if (project_root / "requirements.txt").is_file():
-            _emit(progress_cb, phase="rr", event="toolchain_provisioning",
-                  detail="pip install -r requirements.txt")
-            _run_test(project_root, "python -m pip install -r requirements.txt", env=env)
+            _emit(
+                progress_cb,
+                phase="rr",
+                event="toolchain_provisioning",
+                detail="step 1/1: pip install -r requirements.txt (streaming output below)",
+            )
+            ok, detail = _run_command_with_progress(
+                project_root,
+                "python -m pip install -r requirements.txt",
+                env=env,
+                progress_cb=progress_cb,
+                label="pip",
+                timeout=1800,
+            )
+            if not ok:
+                _emit(
+                    progress_cb,
+                    phase="rr",
+                    event="toolchain_provisioning",
+                    detail=f"pip install finished with errors: {detail}",
+                )
         elif (project_root / "pyproject.toml").is_file() or (project_root / "setup.py").is_file():
-            _emit(progress_cb, phase="rr", event="toolchain_provisioning", detail="pip install -e .")
-            _run_test(project_root, "python -m pip install -e .", env=env)
+            _emit(
+                progress_cb,
+                phase="rr",
+                event="toolchain_provisioning",
+                detail="step 1/1: pip install -e . (streaming output below)",
+            )
+            ok, detail = _run_command_with_progress(
+                project_root,
+                "python -m pip install -e .",
+                env=env,
+                progress_cb=progress_cb,
+                label="pip",
+                timeout=1800,
+            )
+            if not ok:
+                _emit(
+                    progress_cb,
+                    phase="rr",
+                    event="toolchain_provisioning",
+                    detail=f"pip install finished with errors: {detail}",
+                )
 
 
 def _collect_test_manifests(
@@ -396,13 +518,22 @@ def _resolve_cloud_test_command(
             "toolchain or update verification.tests.command."
         )
 
-    # 2) Root-marker / monorepo detection: only usable if runnable on the host.
+    # 2) Root-marker / monorepo detection. Usable if runnable on the host OR if
+    #    its runtime can be provisioned by the managed toolchain (the caller then
+    #    downloads it before the baseline). Only when neither holds do we fall
+    #    through to server discovery, so we never lock onto a command that is
+    #    truly un-runnable (e.g. a Tier-3 binary with no managed runtime).
     detected = resolve_test_command(project_root, config) or discover_test_command(project_root)
     if detected:
         if _client_preflight_command(project_root, detected):
             return detected, f"auto-detected test command: {detected}"
-        # Detected but not runnable on the host -> fall through to server
-        # discovery rather than committing to an un-runnable command.
+        if _managed_toolchain_enabled(config) and _detected_is_provisionable(detected):
+            return detected, (
+                f"auto-detected test command: {detected} "
+                "(runtime not on host PATH; will provision a managed toolchain)"
+            )
+        # Detected but neither runnable nor provisionable -> fall through to
+        # server discovery rather than committing to an un-runnable command.
         detected_note = (
             f"auto-detected test command '{detected}' but its toolchain was not "
             "found on the host PATH; trying server discovery."
@@ -545,6 +676,18 @@ def run_cloud_refactor_requests(
         apply_byok_payload=apply_byok_payload,
     )
 
+    if requests is None:
+        requests = pending_requests(project_root)
+
+    _emit(
+        progress_cb,
+        phase="rr",
+        event="run_start",
+        total_requests=len(requests),
+        test_command=test_command or "",
+        execution_env="cloud",
+    )
+
     # Managed host toolchain: when the resolved command's runtime is missing from
     # the host PATH, download a pinned runtime into the central store and run
     # tests with it (no container, no system changes). ``toolchain_env`` is None
@@ -557,18 +700,12 @@ def run_cloud_refactor_requests(
         progress_cb=progress_cb,
     )
 
-    if requests is None:
-        requests = pending_requests(project_root)
-
     run_store = create_run(project_root)
     run_id = run_store.run_id
     resolved_model = model_hint or str(
         constitution.get_setting("model_id", "managed-default") or "managed-default"
     )
 
-    _emit(progress_cb, phase="rr", event="run_start", total_requests=len(requests),
-          test_command=test_command or "", execution_env="cloud")
-
     outcomes: list[dict] = []
     counts = {"applied": 0, "denied": 0, "failed": 0, "deferred": 0}
     notes: list[str] = []

{refactorai_cli-0.3.9 → refactorai_cli-0.3.12}/refactorai_cli/commands/run_cmds.py

@@ -464,8 +464,10 @@ def _toolchain_consent_prompt(*, auto_approve: bool) -> Callable[[dict], bool]:
     """First-run consent for downloading a managed language runtime.
 
     Managed toolchains are downloaded into the central store (``~/.refactor``),
-    never system paths, so this is low-risk; we still confirm once per run before
-    fetching. ``--auto-approve`` and non-interactive terminals proceed silently.
+    never system paths. To avoid an apparent "stuck" run when Rich status
+    spinners are active, interactive runs default to approve after showing an
+    informational message (no blocking prompt). ``--auto-approve`` and
+    non-interactive terminals proceed silently.
     """
 
     def _ask(info: dict) -> bool:
@@ -474,11 +476,10 @@ def _toolchain_consent_prompt(*, auto_approve: bool) -> Callable[[dict], bool]:
         if auto_approve or not sys.stdin.isatty():
             return True
         console.print(
-            f"[dim]'{binary}' was not found on your PATH. refactor can download a "
-            f"pinned {runtime} runtime into ~/.refactor (no system changes, no container).[/dim]"
+            f"[dim]{binary} not found on your PATH. "
+            f"Refactor is downloading a pinned {runtime} runtime for testing.[/dim]"
         )
-        raw = typer.prompt("Download managed toolchain? (Y/n)", default="Y")
-        return raw.strip().lower() in {"", "y", "yes"}
+        return True
 
     return _ask
 

{refactorai_cli-0.3.9 → refactorai_cli-0.3.12}/refactorai_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: refactorai-cli
-Version: 0.3.9
+Version: 0.3.12
 Summary: Local-first CLI for the refactor platform
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
@@ -8,7 +8,7 @@ Requires-Dist: typer>=0.12.0
 Requires-Dist: httpx>=0.27.0
 Requires-Dist: rich>=13.7.0
 Requires-Dist: PyYAML>=6.0.1
-Requires-Dist: refactorai-core>=3.1.0
+Requires-Dist: refactorai-core>=3.1.1
 
 # refactorai-cli
 

{refactorai_cli-0.3.9 → refactorai_cli-0.3.12}/refactorai_cli.egg-info/requires.txt

@@ -2,4 +2,4 @@ typer>=0.12.0
 httpx>=0.27.0
 rich>=13.7.0
 PyYAML>=6.0.1
-refactorai-core>=3.1.0
+refactorai-core>=3.1.1