refactorai-cli 0.3.3__tar.gz → 0.3.4__tar.gz

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: refactorai-cli
-Version: 0.3.3
+Version: 0.3.4
 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.0.3
+Requires-Dist: refactorai-core>=3.0.4
 
 # refactorai-cli
 
@@ -54,7 +54,7 @@ python -m twine upload ./refactorai-cli/dist/*
 ## Install test (local)
 
 ```bash
-python -m pip install ./refactorai-cli/dist/refactorai_cli-0.3.3-py3-none-any.whl
+python -m pip install ./refactorai-cli/dist/refactorai_cli-0.3.4-py3-none-any.whl
 refactor --version
 ```
 

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/README.md

@@ -42,7 +42,7 @@ python -m twine upload ./refactorai-cli/dist/*
 ## Install test (local)
 
 ```bash
-python -m pip install ./refactorai-cli/dist/refactorai_cli-0.3.3-py3-none-any.whl
+python -m pip install ./refactorai-cli/dist/refactorai_cli-0.3.4-py3-none-any.whl
 refactor --version
 ```
 

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "refactorai-cli"
-version = "0.3.3"
+version = "0.3.4"
 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.0.3",
+  "refactorai-core>=3.0.4",
 ]
 
 [project.scripts]

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/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.3"
+__version__ = "0.3.4"

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/refactorai_cli/cloud_rr.py

@@ -23,6 +23,9 @@ this design fixes. ``tests/test_cloud_rr_client_boundary.py`` enforces this.
 from __future__ import annotations
 
 import hashlib
+import re
+import shlex
+import shutil
 import subprocess
 from pathlib import Path
 from typing import Callable
@@ -48,11 +51,33 @@ from refactor_core.refactor_requests import (
     save_request,
 )
 from refactor_core.store import create_run
-from refactor_core.testcmd import resolve_test_command
+from refactor_core.testcmd import _SKIP_DIRS, discover_test_command, resolve_test_command
 
 from refactorai_cli.auth import ensure_authenticated
 from refactorai_cli.settings import platform_url
 
+_ENV_ASSIGN_RE = re.compile(r"^[A-Za-z_][A-Za-z0-9_]*=.*$")
+
+# Manifest/config files that reveal how a project runs its tests. Sent (truncated)
+# to the server for agentic test-command discovery when deterministic detection
+# finds nothing.
+_MANIFEST_NAMES = (
+    "package.json",
+    "pyproject.toml",
+    "setup.cfg",
+    "setup.py",
+    "pytest.ini",
+    "tox.ini",
+    "noxfile.py",
+    "requirements.txt",
+    "Makefile",
+    "makefile",
+    "go.mod",
+    "Cargo.toml",
+    "build.gradle",
+    "pom.xml",
+)
+
 
 class CloudRRError(RuntimeError):
     """Raised when the cloud RR session cannot proceed."""
@@ -98,6 +123,170 @@ def _run_test(project_root: Path, test_command: str | None) -> tuple[bool, str]:
     return False, (proc.stdout or proc.stderr or "")[-500:]
 
 
+def _client_preflight_command(project_root: Path, command: str | None) -> bool:
+    """Return True when ``command`` looks runnable on this host.
+
+    Composite/shell commands (``&&``, subshells, pipes) run under ``shell=True``
+    and can't be reduced to one binary, so we trust them and let the real exit
+    code gate. Simple commands are validated by checking the leading binary is on
+    PATH — this guards against running (and charging for) a server-suggested
+    command whose toolchain isn't installed.
+    """
+    if not command or not str(command).strip():
+        return False
+    if any(op in command for op in ("&&", "||", ";", "|", "(", ")")):
+        return True
+    try:
+        tokens = shlex.split(command, posix=True)
+    except ValueError:
+        return False
+    while tokens and _ENV_ASSIGN_RE.match(tokens[0]):
+        tokens.pop(0)
+    if not tokens:
+        return False
+    binary = tokens[0]
+    if "/" in binary:
+        return Path(binary).expanduser().exists()
+    return shutil.which(binary) is not None
+
+
+def _collect_test_manifests(
+    project_root: Path, *, max_files: int = 24, max_bytes: int = 4000
+) -> list[dict]:
+    """Gather manifest snippets (repo root + immediate subdirs) for discovery."""
+    root = Path(project_root)
+    docs: list[dict] = []
+    dirs: list[Path] = [root]
+    try:
+        for child in sorted(root.iterdir(), key=lambda p: p.name):
+            if child.is_dir() and child.name not in _SKIP_DIRS and not child.name.startswith("."):
+                dirs.append(child)
+    except OSError:
+        return docs
+
+    for directory in dirs:
+        for name in _MANIFEST_NAMES:
+            path = directory / name
+            if not path.is_file():
+                continue
+            try:
+                content = path.read_text(encoding="utf-8", errors="replace")[:max_bytes]
+            except OSError:
+                continue
+            rel = path.relative_to(root)
+            docs.append({"path": str(rel), "content": content})
+            if len(docs) >= max_files:
+                return docs
+        tests_dir = directory / "tests"
+        if tests_dir.is_dir():
+            try:
+                names = sorted(p.name for p in tests_dir.iterdir() if p.is_file())[:40]
+            except OSError:
+                names = []
+            rel = tests_dir.relative_to(root)
+            docs.append({"path": f"{rel}/ (listing)", "content": "\n".join(names)})
+            if len(docs) >= max_files:
+                return docs
+    return docs
+
+
+def _discover_test_command_via_cloud(
+    *,
+    project_root: Path,
+    constitution,
+    execution_mode: str,
+    auth_ctx,
+    model_hint: str | None,
+    apply_byok_payload: Callable[[dict, object], None] | None,
+) -> str:
+    """Ask the server to infer a test command from the project's manifests.
+
+    Best-effort: any transport/parse/auth issue returns ``""`` so the caller
+    falls back to the existing defer-when-unverifiable behavior.
+    """
+    documents = _collect_test_manifests(project_root)
+    if not documents:
+        return ""
+    payload = {
+        "stage": "rr_discover_testcmd",
+        "constitution_hash": constitution.content_hash,
+        "constitution_body": getattr(constitution, "body", "") or "",
+        "model_hint": model_hint,
+        "documents": documents,
+        "execution_mode": execution_mode,
+    }
+    if execution_mode == "cloud_byok" and apply_byok_payload is not None:
+        apply_byok_payload(payload, constitution)
+    idem = hashlib.sha256(
+        "|".join(
+            [auth_ctx.account_id, execution_mode, constitution.content_hash, "rr_discover_testcmd"]
+        ).encode("utf-8")
+    ).hexdigest()
+    try:
+        resp = httpx.post(
+            f"{platform_url()}/v1/inference/refactor",
+            headers={
+                "Authorization": f"Bearer {auth_ctx.key.key}",
+                "Content-Type": "application/json",
+                "Idempotency-Key": idem,
+            },
+            json=payload,
+            timeout=45.0,
+        )
+    except httpx.HTTPError:
+        return ""
+    if resp.status_code != 200:
+        return ""
+    try:
+        data = resp.json() if resp.content else {}
+    except ValueError:
+        return ""
+    return str((data or {}).get("test_command", "") or "").strip()
+
+
+def _resolve_cloud_test_command(
+    *,
+    project_root: Path,
+    config: dict,
+    constitution,
+    execution_mode: str,
+    auth_ctx,
+    model_hint: str | None,
+    apply_byok_payload: Callable[[dict, object], None] | None,
+) -> tuple[str | None, str]:
+    """Resolve a test command for a cloud run; return ``(command, note)``.
+
+    Layered resolution so the developer usually needs no config:
+    1. configured ``verification.tests.command`` or root-marker detection,
+    2. deterministic monorepo discovery (subproject test setups),
+    3. agentic discovery (server infers from manifests), preflight-validated.
+    """
+    configured = resolve_test_command(project_root, config)
+    if configured:
+        return configured, ""
+
+    discovered = discover_test_command(project_root)
+    if discovered:
+        return discovered, f"auto-detected test command: {discovered}"
+
+    suggested = _discover_test_command_via_cloud(
+        project_root=project_root,
+        constitution=constitution,
+        execution_mode=execution_mode,
+        auth_ctx=auth_ctx,
+        model_hint=model_hint,
+        apply_byok_payload=apply_byok_payload,
+    )
+    if suggested and _client_preflight_command(project_root, suggested):
+        return suggested, f"server-suggested test command: {suggested}"
+    if suggested:
+        return None, (
+            f"server suggested test command '{suggested}' but its toolchain was not "
+            "found on PATH; skipped."
+        )
+    return None, ""
+
+
 class _Session:
     """Transport for the RR session endpoints with status-code error mapping."""
 
@@ -200,7 +389,15 @@ def run_cloud_refactor_requests(
 
     execution_mode = str(constitution.get_setting("mode", "cloud_managed") or "cloud_managed")
     auth_ctx = ensure_authenticated(project_root)
-    test_command = resolve_test_command(project_root, config)
+    test_command, testcmd_note = _resolve_cloud_test_command(
+        project_root=project_root,
+        config=config,
+        constitution=constitution,
+        execution_mode=execution_mode,
+        auth_ctx=auth_ctx,
+        model_hint=model_hint,
+        apply_byok_payload=apply_byok_payload,
+    )
 
     if requests is None:
         requests = pending_requests(project_root)
@@ -217,6 +414,8 @@ def run_cloud_refactor_requests(
     outcomes: list[dict] = []
     counts = {"applied": 0, "denied": 0, "failed": 0, "deferred": 0}
     notes: list[str] = []
+    if testcmd_note:
+        notes.append(testcmd_note)
     stopped = False
     stop_reason = ""
     model_id = resolved_model

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/refactorai_cli/commands/run_cmds.py

@@ -1956,7 +1956,7 @@ def code(
         help="Skip interactive issue approvals in agentic code mode.",
     ),
     on_rr_failure: str = typer.Option(
-        "stop",
+        "continue",
         "--on-rr-failure",
         help="RR failure behavior: stop | continue | defer.",
     ),
@@ -2227,7 +2227,7 @@ def requests(
         help="Auto-approve retry-deferred proposals.",
     ),
     on_rr_failure: str = typer.Option(
-        "stop",
+        "continue",
         "--on-rr-failure",
         help="RR failure behavior for retry: stop | continue | defer.",
     ),
@@ -3484,7 +3484,7 @@ def _report_structural_readiness(project_root: Path, config) -> None:
     if not isinstance(structural, dict):
         structural = {}
     eq_mode = str(structural.get("equivalence_mode", "strict") or "strict")
-    rr_policy = str(structural.get("on_rr_failure", "stop") or "stop")
+    rr_policy = str(structural.get("on_rr_failure", "continue") or "continue")
     rewrite_mode = str(structural.get("reference_rewrite", "auto") or "auto")
     langs = structural.get("languages", [])
     rollout = str(structural.get("rollout", "default") or "default")

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/refactorai_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: refactorai-cli
-Version: 0.3.3
+Version: 0.3.4
 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.0.3
+Requires-Dist: refactorai-core>=3.0.4
 
 # refactorai-cli
 
@@ -54,7 +54,7 @@ python -m twine upload ./refactorai-cli/dist/*
 ## Install test (local)
 
 ```bash
-python -m pip install ./refactorai-cli/dist/refactorai_cli-0.3.3-py3-none-any.whl
+python -m pip install ./refactorai-cli/dist/refactorai_cli-0.3.4-py3-none-any.whl
 refactor --version
 ```
 

{refactorai_cli-0.3.3 → refactorai_cli-0.3.4}/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.0.3
+refactorai-core>=3.0.4