fix: respect Honcho env var fallback in doctor and honcho status
hermes doctor and hermes honcho status warned 'Honcho config not found' whenever ~/.honcho/config.json was absent, even though HONCHO_API_KEY in .env resolves a working config via HonchoClientConfig.from_global_config() -> from_env(). Both now check hcfg.api_key/base_url before warning. Co-authored-by: oxngon <98992931+oxngon@users.noreply.github.com>
This commit is contained in:
parent
fa8fd513ea
commit
e2cc24e331
@ -2040,7 +2040,15 @@ def run_doctor(args):
|
|||||||
_honcho_cfg_path = resolve_config_path()
|
_honcho_cfg_path = resolve_config_path()
|
||||||
|
|
||||||
if not _honcho_cfg_path.exists():
|
if not _honcho_cfg_path.exists():
|
||||||
check_warn("Honcho config not found", "run: hermes memory setup")
|
# Config file missing — but env var fallback may have resolved it.
|
||||||
|
# Only warn if the config didn't actually resolve from env vars.
|
||||||
|
if hcfg.api_key or hcfg.base_url:
|
||||||
|
check_ok(
|
||||||
|
"Honcho configured via environment variables",
|
||||||
|
f"config file {_honcho_cfg_path} not found, using HONCHO_API_KEY env var",
|
||||||
|
)
|
||||||
|
else:
|
||||||
|
check_warn("Honcho config not found", "run: hermes memory setup")
|
||||||
elif not hcfg.enabled:
|
elif not hcfg.enabled:
|
||||||
check_info(f"Honcho disabled (set enabled: true in {_honcho_cfg_path} to activate)")
|
check_info(f"Honcho disabled (set enabled: true in {_honcho_cfg_path} to activate)")
|
||||||
elif not (hcfg.api_key or hcfg.base_url):
|
elif not (hcfg.api_key or hcfg.base_url):
|
||||||
|
|||||||
@ -878,9 +878,21 @@ def cmd_status(args) -> None:
|
|||||||
write_path = _local_config_path()
|
write_path = _local_config_path()
|
||||||
|
|
||||||
if not cfg:
|
if not cfg:
|
||||||
print(f" No Honcho config found at {active_path}")
|
# Config file missing — try env var fallback before giving up.
|
||||||
print(" Run 'hermes honcho setup' to configure.\n")
|
try:
|
||||||
return
|
from plugins.memory.honcho.client import HonchoClientConfig
|
||||||
|
_env_cfg = HonchoClientConfig.from_global_config(host=_host_key())
|
||||||
|
if _env_cfg.api_key or _env_cfg.base_url:
|
||||||
|
# Env var fallback worked — use that config instead.
|
||||||
|
cfg = {"apiKey": _env_cfg.api_key, "enabled": _env_cfg.enabled}
|
||||||
|
else:
|
||||||
|
print(f" No Honcho config found at {active_path}")
|
||||||
|
print(" Run 'hermes honcho setup' to configure.\n")
|
||||||
|
return
|
||||||
|
except Exception:
|
||||||
|
print(f" No Honcho config found at {active_path}")
|
||||||
|
print(" Run 'hermes honcho setup' to configure.\n")
|
||||||
|
return
|
||||||
|
|
||||||
try:
|
try:
|
||||||
from plugins.memory.honcho.client import HonchoClientConfig, get_honcho_client
|
from plugins.memory.honcho.client import HonchoClientConfig, get_honcho_client
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user