Skip to content

fix(wheel): include hermes_cli subpackages in wheel build - #34505

Closed
arshkumarsingh wants to merge 1 commit into
NousResearch:mainfrom
arshkumarsingh:fix/wheel-hermes-cli-subpackages
Closed

fix(wheel): include hermes_cli subpackages in wheel build#34505
arshkumarsingh wants to merge 1 commit into
NousResearch:mainfrom
arshkumarsingh:fix/wheel-hermes-cli-subpackages

Conversation

@arshkumarsingh

Copy link
Copy Markdown

Summary

Add "hermes_cli.*" to [tool.setuptools.packages.find].include to fix the v0.15.1 wheel missing dashboard_auth/ and proxy/ subpackages.

Root Cause

[tool.setuptools.packages.find].include listed "hermes_cli" but was missing "hermes_cli.*". Every other package in the list (agent.*, tools.*, gateway.*, tui_gateway.*, plugins.*, providers.*) had its subpackage glob — only hermes_cli was missing it. This caused hermes_cli/dashboard_auth/ and hermes_cli/proxy/ to be excluded from the built wheel.

Impact

The dashboard subprocess crashes with:

File "hermes_cli/web_server.py", line 4822
    from hermes_cli.dashboard_auth.routes import router
ModuleNotFoundError: No module named "hermes_cli.dashboard_auth"

Only affects wheel installs (e.g. uv pip install hermes-agent from 0.14.x). The Docker :latest image uses an editable install so it bypasses the bug.

Changes

One line in pyproject.toml: added "hermes_cli.*" to the include list.

Verification

Built wheel with uv build --wheel and confirmed the RECORD now contains:

hermes_cli/dashboard_auth/__init__.py
hermes_cli/dashboard_auth/routes.py
hermes_cli/dashboard_auth/middleware.py
hermes_cli/dashboard_auth/base.py
hermes_cli/dashboard_auth/cookies.py
hermes_cli/dashboard_auth/login_page.py
hermes_cli/dashboard_auth/prefix.py
hermes_cli/dashboard_auth/public_paths.py
hermes_cli/dashboard_auth/registry.py
hermes_cli/dashboard_auth/ws_tickets.py
hermes_cli/dashboard_auth/audit.py
hermes_cli/proxy/__init__.py
hermes_cli/proxy/cli.py
hermes_cli/proxy/server.py
hermes_cli/proxy/adapters/__init__.py
hermes_cli/proxy/adapters/base.py
hermes_cli/proxy/adapters/nous_portal.py
hermes_cli/proxy/adapters/xai.py

(Alongside the existing plugins/dashboard_auth/nous/ plugin directory — these are separate things.)

Fixes #34346

dashboard_auth and proxy subpackages were excluded from the v0.15.1
wheel because `hermes_cli.*` was missing from `[tool.setuptools.packages.find].include`
while every other package had its `.*` glob. This caused the dashboard
to crash with ModuleNotFoundError when the wheel was installed.

Fixes NousResearch#34346
@arshkumarsingh
arshkumarsingh requested a review from a team May 29, 2026 08:49
@alt-glitch alt-glitch added type/bug Something isn't working P1 High — major feature broken, no workaround comp/cli CLI entry point, hermes_cli/, setup wizard labels May 29, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #27728 (omnibus packaging fix that adds hermes_cli.* plus acp_adapter.*, MANIFEST.in grafts, Homebrew formula). Previous duplicates: #27726, #28060, #28231, #28820 — all closed in favor of #27728.

@arshkumarsingh

Copy link
Copy Markdown
Author

Duplicate of #27728 which is the comprehensive fix (adds hermes_cli., acp_adapter., MANIFEST.in grafts, Homebrew formula, and invariant tests). Closing in favor of that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P1 High — major feature broken, no workaround type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

v0.15.1 wheel installs hermes_cli/dashboard_auth/ to wrong path → dashboard ModuleNotFoundError

2 participants