mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-04-25 00:51:20 +00:00
fix: add missing shutil import for Matrix E2EE setup
Cherry-picked from PR #5136 by thakoreh. setup_gateway() uses shutil.which('uv') at line 2126 but shutil was never imported at module level, causing NameError during Matrix E2EE auto-install. Adds top-level import and regression test.
This commit is contained in:
parent
70f798043b
commit
914f7461dc
2 changed files with 32 additions and 0 deletions
31
tests/hermes_cli/test_setup_matrix_e2ee.py
Normal file
31
tests/hermes_cli/test_setup_matrix_e2ee.py
Normal file
|
|
@ -0,0 +1,31 @@
|
|||
"""Test that setup.py has shutil available for Matrix E2EE auto-install."""
|
||||
import ast
|
||||
|
||||
import pytest
|
||||
|
||||
|
||||
def _parse_setup_imports():
|
||||
"""Parse setup.py and return top-level import names."""
|
||||
with open("hermes_cli/setup.py") as f:
|
||||
tree = ast.parse(f.read())
|
||||
names = set()
|
||||
for node in ast.walk(tree):
|
||||
if isinstance(node, ast.Import):
|
||||
for alias in node.names:
|
||||
names.add(alias.name)
|
||||
elif isinstance(node, ast.ImportFrom):
|
||||
for alias in node.names:
|
||||
names.add(alias.name)
|
||||
return names
|
||||
|
||||
|
||||
class TestSetupShutilImport:
|
||||
def test_shutil_imported_at_module_level(self):
|
||||
"""shutil must be imported at module level so setup_gateway can use it
|
||||
for the matrix-nio auto-install path (line ~2126)."""
|
||||
names = _parse_setup_imports()
|
||||
assert "shutil" in names, (
|
||||
"shutil is not imported at the top of hermes_cli/setup.py. "
|
||||
"This causes a NameError when the Matrix E2EE auto-install "
|
||||
"tries to call shutil.which('uv')."
|
||||
)
|
||||
Loading…
Add table
Add a link
Reference in a new issue