fix: use argparse entrypoint in top-level launcher (#3874)
The ./hermes convenience script still used the legacy Fire-based cli.main wrapper, which doesn't support subcommands (gateway, cron, doctor, etc.). The installed 'hermes' command already uses hermes_cli.main:main (argparse) — this aligns the launcher. Salvaged from PR #2009 by gito369.
This commit is contained in:
11
hermes
11
hermes
@@ -1,12 +1,11 @@
|
|||||||
#!/usr/bin/env python3
|
#!/usr/bin/env python3
|
||||||
"""
|
"""
|
||||||
Hermes Agent CLI Launcher
|
Hermes Agent CLI launcher.
|
||||||
|
|
||||||
This is a convenience wrapper to launch the Hermes CLI.
|
This wrapper should behave like the installed `hermes` command, including
|
||||||
Usage: ./hermes [options]
|
subcommands such as `gateway`, `cron`, and `doctor`.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
from cli import main
|
from hermes_cli.main import main
|
||||||
import fire
|
main()
|
||||||
fire.Fire(main)
|
|
||||||
|
|||||||
42
tests/hermes_cli/test_launcher.py
Normal file
42
tests/hermes_cli/test_launcher.py
Normal file
@@ -0,0 +1,42 @@
|
|||||||
|
"""Tests for the top-level `./hermes` launcher script."""
|
||||||
|
|
||||||
|
import runpy
|
||||||
|
import sys
|
||||||
|
import types
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
|
||||||
|
def test_launcher_delegates_to_argparse_entrypoint(monkeypatch):
|
||||||
|
"""`./hermes` should use `hermes_cli.main`, not the legacy Fire wrapper."""
|
||||||
|
launcher_path = Path(__file__).resolve().parents[2] / "hermes"
|
||||||
|
called = []
|
||||||
|
|
||||||
|
fake_main_module = types.ModuleType("hermes_cli.main")
|
||||||
|
|
||||||
|
def fake_main():
|
||||||
|
called.append("hermes_cli.main")
|
||||||
|
|
||||||
|
fake_main_module.main = fake_main
|
||||||
|
monkeypatch.setitem(sys.modules, "hermes_cli.main", fake_main_module)
|
||||||
|
|
||||||
|
fake_cli_module = types.ModuleType("cli")
|
||||||
|
|
||||||
|
def legacy_cli_main(*args, **kwargs):
|
||||||
|
raise AssertionError("launcher should not import cli.main")
|
||||||
|
|
||||||
|
fake_cli_module.main = legacy_cli_main
|
||||||
|
monkeypatch.setitem(sys.modules, "cli", fake_cli_module)
|
||||||
|
|
||||||
|
fake_fire_module = types.ModuleType("fire")
|
||||||
|
|
||||||
|
def legacy_fire(*args, **kwargs):
|
||||||
|
raise AssertionError("launcher should not invoke fire.Fire")
|
||||||
|
|
||||||
|
fake_fire_module.Fire = legacy_fire
|
||||||
|
monkeypatch.setitem(sys.modules, "fire", fake_fire_module)
|
||||||
|
|
||||||
|
monkeypatch.setattr(sys, "argv", [str(launcher_path), "gateway", "status"])
|
||||||
|
|
||||||
|
runpy.run_path(str(launcher_path), run_name="__main__")
|
||||||
|
|
||||||
|
assert called == ["hermes_cli.main"]
|
||||||
Reference in New Issue
Block a user