fix(deps): exclude dev tooling from all extra

This commit is contained in:
rexdotsh
2026-06-03 21:34:44 +05:30
committed by kshitij
parent 8077e7d2fb
commit ee7948ea6e
3 changed files with 11 additions and 9 deletions

View File

@ -230,7 +230,6 @@ all = [
# where the user is expected to have a toolchain available.
"hermes-agent[cron]",
"hermes-agent[cli]",
"hermes-agent[dev]",
"hermes-agent[pty]",
"hermes-agent[mcp]",
"hermes-agent[homeassistant]",

View File

@ -88,6 +88,17 @@ def test_lazy_installable_extras_excluded_from_all():
)
def test_dev_extra_excluded_from_all():
"""End-user installs should not pull test/lint/debug tooling."""
optional_dependencies = _load_optional_dependencies()
assert "dev" in optional_dependencies
assert not any(
spec == "hermes-agent[dev]"
for spec in optional_dependencies["all"]
)
def test_messaging_extra_includes_qrcode_for_weixin_setup():
optional_dependencies = _load_optional_dependencies()

8
uv.lock generated
View File

@ -1423,20 +1423,13 @@ acp = [
all = [
{ name = "agent-client-protocol" },
{ name = "aiohttp" },
{ name = "debugpy" },
{ name = "fastapi" },
{ name = "google-api-python-client" },
{ name = "google-auth-httplib2" },
{ name = "google-auth-oauthlib" },
{ name = "mcp" },
{ name = "pytest" },
{ name = "pytest-asyncio" },
{ name = "pytest-timeout" },
{ name = "ruff" },
{ name = "setuptools" },
{ name = "simple-term-menu" },
{ name = "starlette" },
{ name = "ty" },
{ name = "uvicorn", extra = ["standard"] },
{ name = "youtube-transcript-api" },
]
@ -1629,7 +1622,6 @@ requires-dist = [
{ name = "hermes-agent", extras = ["cli"], marker = "extra == 'termux'" },
{ name = "hermes-agent", extras = ["cron"], marker = "extra == 'all'" },
{ name = "hermes-agent", extras = ["cron"], marker = "extra == 'termux'" },
{ name = "hermes-agent", extras = ["dev"], marker = "extra == 'all'" },
{ name = "hermes-agent", extras = ["google"], marker = "extra == 'all'" },
{ name = "hermes-agent", extras = ["google"], marker = "extra == 'termux-all'" },
{ name = "hermes-agent", extras = ["homeassistant"], marker = "extra == 'all'" },