refactorai-cli 0.7.13__tar.gz → 0.7.15__tar.gz

{refactorai_cli-0.7.13 → refactorai_cli-0.7.15}/PKG-INFO

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

{refactorai_cli-0.7.13 → refactorai_cli-0.7.15}/pyproject.toml

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

{refactorai_cli-0.7.13 → refactorai_cli-0.7.15}/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.13"
+__version__ = "0.7.15"

{refactorai_cli-0.7.13 → refactorai_cli-0.7.15}/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,
 )
 
@@ -5341,7 +5340,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.13 → refactorai_cli-0.7.15}/refactorai_cli/commands/runtime_proxy_cmds.py

@@ -79,6 +79,30 @@ def _engine_runs_on_server() -> bool:
     return engine_is_remote(_project_execution_mode())
 
 
+def _sync_mode_from_server_if_possible(command_name: str | None = None) -> None:
+    """Best-effort server->local mode sync before routing.
+
+    Fixes the stale-mode chicken-and-egg: when a developer changes the mode in
+    the web UI, the local ``refactor.config`` may still point at the previous
+    variant. If dispatch reads that stale local value first, commands can route
+    to the wrong engine path (for example ``doctor`` going to the runtime path
+    instead of the local CLI cloud path), so the sync never runs and the local
+    mode never updates.
+
+    This helper is intentionally silent and fail-open (offline/auth issues are
+    ignored) so command routing remains robust.
+    """
+    try:
+        from refactorai_cli.mode_sync import sync_execution_variant_from_server
+
+        emit = None
+        if command_name == "doctor":
+            emit = lambda m: console.print(f"[dim]{m}[/dim]")
+        sync_execution_variant_from_server(Path.cwd(), emit=emit)
+    except Exception:
+        return
+
+
 def _active_runtime_artifact() -> Path:
     status = runtime_status()
     version = str(status.get("active_version") or "").strip()
@@ -98,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_missing_module(exc: ModuleNotFoundError) -> None:
+    """Print an actionable message for a missing ``refactor_core`` module.
 
-def _print_core_too_old(exc: ModuleNotFoundError) -> None:
+    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"
@@ -127,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
 
@@ -157,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
@@ -181,6 +244,7 @@ def _dispatch(command_name: str, passthrough_args: list[str]) -> None:
     host-engine variants (``local_model`` / ``local_heuristic``) execute via the
     downloaded runtime artifact.
     """
+    _sync_mode_from_server_if_possible(command_name)
     if _engine_runs_on_server():
         _delegate_to_run_cmds(command_name, passthrough_args)
         return

{refactorai_cli-0.7.13 → refactorai_cli-0.7.15}/refactorai_cli.egg-info/PKG-INFO

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