fix(#834): KeyError 'missing_vars' crashes CLI startup
Some checks failed
Contributor Attribution Check / check-attribution (pull_request) Failing after 21s
Docker Build and Publish / build-and-push (pull_request) Has been skipped
Supply Chain Audit / Scan PR for supply chain risks (pull_request) Successful in 20s
Tests / e2e (pull_request) Successful in 2m19s
Tests / test (pull_request) Failing after 1h3m51s
Some checks failed
Contributor Attribution Check / check-attribution (pull_request) Failing after 21s
Docker Build and Publish / build-and-push (pull_request) Has been skipped
Supply Chain Audit / Scan PR for supply chain risks (pull_request) Successful in 20s
Tests / e2e (pull_request) Successful in 2m19s
Tests / test (pull_request) Failing after 1h3m51s
Fix _show_tool_availability_warnings() to use 'env_vars' key instead of 'missing_vars' which doesn't exist in registry output. Closes #834
This commit is contained in:
4
cli.py
4
cli.py
@@ -3611,7 +3611,7 @@ class HermesCLI:
|
||||
available, unavailable = check_tool_availability()
|
||||
|
||||
# Filter to only those missing API keys (not system deps)
|
||||
api_key_missing = [u for u in unavailable if u["missing_vars"]]
|
||||
api_key_missing = [u for u in unavailable if u.get("env_vars")]
|
||||
|
||||
if api_key_missing:
|
||||
self.console.print()
|
||||
@@ -3620,7 +3620,7 @@ class HermesCLI:
|
||||
tools_str = ", ".join(item["tools"][:2]) # Show first 2 tools
|
||||
if len(item["tools"]) > 2:
|
||||
tools_str += f", +{len(item['tools'])-2} more"
|
||||
self.console.print(f" [dim]• {item['name']}[/] [dim italic]({', '.join(item['missing_vars'])})[/]")
|
||||
self.console.print(f" [dim]• {item['name']}[/] [dim italic]({', '.join(item['env_vars'])})[/]")
|
||||
self.console.print("[dim] Run 'hermes setup' to configure[/]")
|
||||
except Exception:
|
||||
pass # Don't crash on import errors
|
||||
|
||||
Reference in New Issue
Block a user