refactorai-cli 0.7.14__tar.gz → 0.7.16__tar.gz

{refactorai_cli-0.7.14 → refactorai_cli-0.7.16}/PKG-INFO

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

{refactorai_cli-0.7.14 → refactorai_cli-0.7.16}/pyproject.toml

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

{refactorai_cli-0.7.14 → refactorai_cli-0.7.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.7.14"
+__version__ = "0.7.16"

{refactorai_cli-0.7.14 → refactorai_cli-0.7.16}/refactorai_cli/commands/run_cmds.py

@@ -119,7 +119,6 @@ from refactor_core.execution_variant import (
     is_cloud_execution,
     platform_inference_mode,
     resolve_execution_variant,
-    uses_platform_inference,
     verification_location,
 )
 
@@ -5116,8 +5115,11 @@ def _cloud_doctor(
     resolving/pinging a local provider.
     """
     console.print(f"[bold]project[/bold]: {project_root}")
-    console.print(f"[bold]mode[/bold]: {execution_mode}")
+    # Clarify the two axes to avoid confusion for local_byok/local_managed:
+    # - execution variant: user-facing mode contract
+    # - platform inference mode: wire mode used for server inference endpoint
     console.print(f"[bold]execution variant[/bold]: {execution_variant}")
+    console.print(f"[bold]platform inference mode[/bold]: {execution_mode}")
     _report_verification_environment(project_root, execution_variant, project_config)
     _report_store_binding(project_root)
 
@@ -5136,13 +5138,14 @@ def _cloud_doctor(
     credential_ref = ""
     provider_key = ""
     provider_name = ""
+    mode_label = "local_byok" if execution_variant == "local_byok" else execution_mode
     if execution_mode == "cloud_byok":
         provider_name = str(constitution.get_setting("provider", "") or "").strip()
         provider_key = str(constitution.get_setting("provider_key", "") or "").strip()
         credential_ref = str(constitution.get_setting("credential_ref", "") or "").strip()
         if not provider_name or provider_name.lower() == "cloud":
             console.print(
-                "[red]cloud_byok[/red]: set an explicit `provider` "
+                f"[red]{mode_label}[/red]: set an explicit `provider` "
                 "(for example `provider: openai`)."
             )
             raise typer.Exit(code=1)
@@ -5156,7 +5159,7 @@ def _cloud_doctor(
             console.print(f"[green]credential_ref[/green]: {credential_ref}")
         else:
             console.print(
-                "[red]cloud_byok[/red]: set either `provider_key` (preferred) "
+                f"[red]{mode_label}[/red]: set either `provider_key` (preferred) "
                 "or `credential_ref` in refactor.config."
             )
             if dotenv is not None and not dotenv.found:
@@ -5341,7 +5344,17 @@ def doctor(
     _report_dotenv(dotenv, project_root)
 
     execution_variant = _execution_variant(constitution)
-    if uses_platform_inference(execution_variant):
+    # Route by ENGINE LOCATION (docs/46), matching review/code dispatch. Every
+    # server-engine variant (local_byok / local_managed / cloud_byok /
+    # cloud_managed) runs its engine on the Refactor server, so it is diagnosed
+    # via the platform path and must NEVER import engine-only modules -- those
+    # ship in the private engine wheel and are absent from a public pip/pipx
+    # install. Only the host-engine variants (local_heuristic / local_model)
+    # fall through to the engine-backed local doctor below. Previously this
+    # branched on ``uses_platform_inference`` (False for local_byok), which sent
+    # local_byok down the local path and blew up importing
+    # ``refactor_core.execution`` on a client-only install.
+    if engine_is_remote(execution_variant):
         _cloud_doctor(
             project_root=project_root,
             constitution=constitution,

{refactorai_cli-0.7.14 → refactorai_cli-0.7.16}/refactorai_cli/commands/runtime_proxy_cmds.py

@@ -122,12 +122,50 @@ def _active_runtime_artifact() -> Path:
     return artifact
 
 
+# Subpackages that ship ONLY in the private engine wheel (R21 core split) and are
+# intentionally absent from a public pip/pipx install of ``refactorai-core``.
+_ENGINE_ONLY_PREFIXES = (
+    "refactor_core.execution",
+    "refactor_core.providers",
+    "refactor_core.enginecore",
+    "refactor_core.codeintel",
+)
+
+
+def _is_engine_only_module(exc: ModuleNotFoundError) -> bool:
+    """True when the missing module is an engine-only subpackage (not version skew)."""
+    name = exc.name or ""
+    return any(name == p or name.startswith(p + ".") for p in _ENGINE_ONLY_PREFIXES)
+
+
 def _is_core_too_old(exc: ModuleNotFoundError) -> bool:
-    """True when a missing module points at an outdated installed ``refactor_core``."""
-    return (exc.name or "").startswith("refactor_core")
+    """True when a missing module points at an outdated installed ``refactor_core``.
+
+    Engine-only subpackages are excluded: their absence in a public install is by
+    design, not a version mismatch (see :func:`_print_missing_module`).
+    """
+    return (exc.name or "").startswith("refactor_core") and not _is_engine_only_module(exc)
 
 
-def _print_core_too_old(exc: ModuleNotFoundError) -> None:
+def _print_missing_module(exc: ModuleNotFoundError) -> None:
+    """Print an actionable message for a missing ``refactor_core`` module.
+
+    Distinguishes two very different causes:
+    - engine-only module absent from a public install (a routing/mode issue), vs
+    - a genuinely outdated core (version skew -> upgrade).
+    """
+    if _is_engine_only_module(exc):
+        console.print(
+            "[red]This command needs the local engine, which is not part of the "
+            "public install.[/red]\n"
+            f"Missing engine module: [bold]{exc.name}[/bold]\n"
+            "Only local_heuristic / local_model run the engine on this machine "
+            "(via the runtime artifact). Server-engine modes (local_byok / "
+            "local_managed / cloud_byok / cloud_managed) should not reach this "
+            "path — check the project's execution mode in refactor.config or the "
+            "web UI, then rerun. If the mode is correct, this is a routing bug."
+        )
+        return
     console.print(
         "[red]Installed refactor core is too old for this CLI.[/red]\n"
         f"Missing module: [bold]{exc.name}[/bold]\n"
@@ -151,8 +189,8 @@ def _delegate_to_run_cmds(command_name: str, passthrough_args: list[str]) -> Non
     try:
         from refactorai_cli.commands import run_cmds
     except ModuleNotFoundError as exc:
-        if _is_core_too_old(exc):
-            _print_core_too_old(exc)
+        if (exc.name or "").startswith("refactor_core"):
+            _print_missing_module(exc)
             raise typer.Exit(code=1) from exc
         raise
 
@@ -181,11 +219,12 @@ def _delegate_to_run_cmds(command_name: str, passthrough_args: list[str]) -> Non
         if code:
             raise typer.Exit(code=code) from exc
     except ModuleNotFoundError as exc:
-        # A refactor_core submodule missing at call time (e.g. commit-scoped
-        # review importing refactor_core.intent) means the installed core predates
-        # this CLI. Surface an actionable upgrade hint instead of exiting silently.
-        if _is_core_too_old(exc):
-            _print_core_too_old(exc)
+        # A refactor_core submodule missing at call time can mean two things:
+        # version skew (installed core predates this CLI) or an engine-only module
+        # reached on a client-only install (a routing/mode issue). Surface the
+        # right actionable hint instead of exiting silently.
+        if (exc.name or "").startswith("refactor_core"):
+            _print_missing_module(exc)
         else:
             console.print(f"[red]{command_name} failed:[/red] {exc}")
         raise typer.Exit(code=1) from exc

{refactorai_cli-0.7.14 → refactorai_cli-0.7.16}/refactorai_cli.egg-info/PKG-INFO

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