From 89c5414af0d344accb6632161a93c62243e85d69 Mon Sep 17 00:00:00 2001 From: Ali Shahkar Date: Fri, 10 Apr 2026 15:30:30 -0700 Subject: [PATCH] fix(doctor): respect preferred_engine/engine.default when reporting default model (#204) * fix(cloud-router): prevent local HF org models from being misrouted to OpenRouter get_provider() previously routed any "/" model to OpenRouter, which caused locally-served models (e.g. mlx-community/Qwen3.5-27B-4bit-DWQ) to be sent to the cloud instead of localhost:8080. Adds _LOCAL_HF_ORGS module-level allowlist covering mlx-community/, bartowski/, unsloth/, and lmstudio-community/. Returns None early for these before the openrouter "/" check. * fix(doctor): respect preferred_engine/engine.default when reporting default model _check_default_model() iterated engines in sorted alphabetical order, causing engines like 'lemonade' to be reported as the host for the default model even when 'mlx' was configured as preferred_engine and engine.default. Now checks the configured engine first, then falls back to alphabetical scan. Co-Authored-By: Claude Sonnet 4.6 --------- Co-authored-by: Claude Sonnet 4.6 --- src/openjarvis/cli/doctor_cmd.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/openjarvis/cli/doctor_cmd.py b/src/openjarvis/cli/doctor_cmd.py index 15a66f33..07f35c2e 100644 --- a/src/openjarvis/cli/doctor_cmd.py +++ b/src/openjarvis/cli/doctor_cmd.py @@ -170,7 +170,13 @@ def _check_default_model() -> CheckResult: from openjarvis.core.registry import EngineRegistry from openjarvis.engine import _discovery - for key in sorted(EngineRegistry.keys()): + preferred = config.intelligence.preferred_engine or config.engine.default + check_order = [] + if preferred: + check_order.append(preferred) + check_order += [k for k in sorted(EngineRegistry.keys()) if k != preferred] + + for key in check_order: try: engine = _discovery._make_engine(key, config) if engine.health():