refactorai-cli 0.2.15__tar.gz → 0.2.16__tar.gz

{refactorai_cli-0.2.15 → refactorai_cli-0.2.16}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: refactorai-cli
-Version: 0.2.15
+Version: 0.2.16
 Summary: Local-first CLI for the refactor platform
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{refactorai_cli-0.2.15 → refactorai_cli-0.2.16}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "refactorai-cli"
-version = "0.2.15"
+version = "0.2.16"
 description = "Local-first CLI for the refactor platform"
 readme = "README.md"
 requires-python = ">=3.11"

{refactorai_cli-0.2.15 → refactorai_cli-0.2.16}/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.2.15"
+__version__ = "0.2.16"

{refactorai_cli-0.2.15 → refactorai_cli-0.2.16}/refactorai_cli/commands/run_cmds.py

@@ -50,6 +50,8 @@ from refactor_core.execution.structural_ops import structural_compatibility_matr
 from refactor_core.gate import evaluate_gate
 from refactor_core.refactor_requests import (
     DEFERRED,
+    RequestSuppression,
+    RequestWarning,
     clear_requests,
     create_requests,
     list_requests,
@@ -1555,9 +1557,13 @@ def review(
     _print_findings(artifact.findings)
 
     # A fresh review supersedes prior refactor requests: replace them with one
-    # RR per finding (central-only; nothing is written into the project tree).
-    clear_requests(project_root)
-    requests = create_requests(project_root, artifact.findings, run_id=artifact.run_id)
+    # RR per actionable finding (central-only; nothing is written into project tree).
+    requests = _replace_requests_with_framework_guardrails(
+        project_root=project_root,
+        findings=artifact.findings,
+        run_id=artifact.run_id,
+        artifact=artifact,
+    )
     if requests:
         console.print(
             f"\n[bold]Created {len(requests)} refactor request(s).[/bold] "
@@ -1666,9 +1672,11 @@ def code(
             review_store = create_run(project_root, run_id=review_artifact.run_id)
             review_store.write_artifact(review_artifact, target=target, status="completed")
             _print_findings(review_artifact.findings)
-            clear_requests(project_root)
-            requests = create_requests(
-                project_root, review_artifact.findings, run_id=review_artifact.run_id
+            requests = _replace_requests_with_framework_guardrails(
+                project_root=project_root,
+                findings=review_artifact.findings,
+                run_id=review_artifact.run_id,
+                artifact=review_artifact,
             )
             console.print(f"[bold]Created {len(requests)} refactor request(s).[/bold]")
 
@@ -1848,6 +1856,59 @@ def _format_operation_scope(rr) -> str:
     return "-"
 
 
+def _replace_requests_with_framework_guardrails(
+    *,
+    project_root: Path,
+    findings: list[Finding],
+    run_id: str,
+    artifact: RunArtifact | None = None,
+) -> list:
+    clear_requests(project_root)
+    requests, suppressions, warnings = create_requests(
+        project_root,
+        findings,
+        run_id=run_id,
+        return_diagnostics=True,
+    )
+    _report_framework_suppressions(suppressions, artifact=artifact)
+    _report_framework_warnings(warnings, artifact=artifact)
+    return requests
+
+
+def _report_framework_suppressions(
+    suppressions: list[RequestSuppression], *, artifact: RunArtifact | None = None
+) -> None:
+    if not suppressions:
+        return
+    console.print(
+        f"[yellow]warning:[/yellow] suppressed {len(suppressions)} finding(s) due to framework invariants."
+    )
+    for item in suppressions:
+        console.print(
+            f"[yellow]-[/yellow] {item.location}: {item.message} "
+            f"[dim](framework={item.framework or 'unknown'}, rule={item.rule_id or 'unspecified'})[/dim]"
+        )
+        if artifact is not None:
+            artifact.notes.append(item.note())
+
+
+def _report_framework_warnings(
+    warnings: list[RequestWarning], *, artifact: RunArtifact | None = None
+) -> None:
+    if not warnings:
+        return
+    console.print(
+        f"[yellow]warning:[/yellow] flagged {len(warnings)} finding(s) with framework caution."
+    )
+    for item in warnings:
+        console.print(
+            f"[yellow]-[/yellow] {item.location}: {item.message} "
+            f"[dim](framework={item.framework or 'unknown'}, rule={item.rule_id or 'unspecified'})[/dim]"
+        )
+        if artifact is not None:
+            artifact.notes.append(item.note())
+
+
 def apply(
     force: bool = typer.Option(False, "--force", help="Apply even if the quality gate blocks."),
 ) -> None:

{refactorai_cli-0.2.15 → refactorai_cli-0.2.16}/refactorai_cli/commands/runtime_cmds.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 
 import typer
 from rich.console import Console
+from rich.progress import BarColumn, DownloadColumn, Progress, TaskID, TextColumn, TimeRemainingColumn, TransferSpeedColumn
 
 from refactorai_cli.auth import AuthError, ensure_authenticated
 from refactorai_cli.control_plane import ensure_lease, heartbeat, read_cached_lease, resolve_policy
@@ -62,7 +63,23 @@ def update(
             f"[bold]resolved manifest[/bold]: version={manifest.runtime_version} "
             f"channel={channel}"
         )
-        artifact = download_artifact(manifest.artifact_url)
+        progress_columns = (
+            TextColumn("[progress.description]{task.description}"),
+            BarColumn(),
+            DownloadColumn(),
+            TransferSpeedColumn(),
+            TimeRemainingColumn(),
+        )
+        with Progress(*progress_columns, console=console, transient=True) as progress:
+            task_id: TaskID = progress.add_task("Downloading runtime artifact", total=None)
+
+            def _on_progress(downloaded: int, total: int | None) -> None:
+                update_kwargs = {"completed": int(max(downloaded, 0))}
+                if total is not None and total > 0:
+                    update_kwargs["total"] = total
+                progress.update(task_id, **update_kwargs)
+
+            artifact = download_artifact(manifest.artifact_url, progress_cb=_on_progress)
         console.print(f"[bold]downloaded artifact[/bold]: {len(artifact)} bytes")
         if dry_run:
             # Verification happens in activate_runtime; dry-run still checks

{refactorai_cli-0.2.15 → refactorai_cli-0.2.16}/refactorai_cli/runtime_manager.py

@@ -14,6 +14,7 @@ import platform
 from dataclasses import dataclass
 from datetime import datetime, timezone
 from pathlib import Path
+from typing import Callable
 from urllib.parse import urlparse
 
 import httpx
@@ -153,17 +154,49 @@ def resolve_runtime_manifest(*, channel: str = "stable", timeout: float = 20.0)
     return manifest
 
 
-def download_artifact(url: str, *, timeout: float = 60.0) -> bytes:
+def download_artifact(
+    url: str,
+    *,
+    timeout: float = 60.0,
+    progress_cb: Callable[[int, int | None], None] | None = None,
+) -> bytes:
     headers: dict[str, str] = {}
     if _should_send_runtime_auth(url):
         headers["Authorization"] = _runtime_auth_header()
+    resolved_timeout = max(float(timeout), 300.0)
+    if progress_cb is None:
+        try:
+            response = httpx.get(url, headers=headers, timeout=resolved_timeout)
+        except httpx.HTTPError as exc:
+            raise RuntimeError(f"Could not download runtime artifact: {exc}") from exc
+        if response.status_code >= 400:
+            raise RuntimeError(f"Runtime artifact download failed ({response.status_code})")
+        return response.content
     try:
-        response = httpx.get(url, headers=headers, timeout=max(float(timeout), 300.0))
+        with httpx.stream("GET", url, headers=headers, timeout=resolved_timeout) as response:
+            if response.status_code >= 400:
+                raise RuntimeError(f"Runtime artifact download failed ({response.status_code})")
+            total: int | None = None
+            total_header = str(response.headers.get("Content-Length") or "").strip()
+            if total_header:
+                try:
+                    parsed_total = int(total_header)
+                except ValueError:
+                    parsed_total = -1
+                if parsed_total > 0:
+                    total = parsed_total
+            downloaded = 0
+            chunks: list[bytes] = []
+            progress_cb(downloaded, total)
+            for chunk in response.iter_bytes(chunk_size=64 * 1024):
+                if not chunk:
+                    continue
+                chunks.append(chunk)
+                downloaded += len(chunk)
+                progress_cb(downloaded, total)
     except httpx.HTTPError as exc:
         raise RuntimeError(f"Could not download runtime artifact: {exc}") from exc
-    if response.status_code >= 400:
-        raise RuntimeError(f"Runtime artifact download failed ({response.status_code})")
-    return response.content
+    return b"".join(chunks)
 
 
 def _should_send_runtime_auth(url: str) -> bool:

{refactorai_cli-0.2.15 → refactorai_cli-0.2.16}/refactorai_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: refactorai-cli
-Version: 0.2.15
+Version: 0.2.16
 Summary: Local-first CLI for the refactor platform
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown