From 95994bbc568a61297e10bbea0d0c96f3d2148b07 Mon Sep 17 00:00:00 2001 From: HexLab98 Date: Fri, 26 Jun 2026 14:37:02 +0700 Subject: [PATCH] fix(windows): repair missing hermes.exe after pip install (#52931) On Windows, uv pip install -e . can register hermes.exe in package metadata while the launcher never lands on disk. Detect missing [project.scripts] shims and reinstall entry points under the existing quarantine path in hermes update and install.ps1. --- hermes_cli/main.py | 91 +++++++++++++++++++++++++++++++++++++++++++++ scripts/install.ps1 | 42 +++++++++++++++++++++ 2 files changed, 133 insertions(+) diff --git a/hermes_cli/main.py b/hermes_cli/main.py index e6445afd766..d67c85718a6 100644 --- a/hermes_cli/main.py +++ b/hermes_cli/main.py @@ -7666,6 +7666,97 @@ def _install_python_dependencies_with_optional_fallback( # missing, then re-verify so the failure surfaces here instead of # downstream. _verify_core_dependencies_installed(install_cmd_prefix, env=env, group=group) + _verify_console_scripts_installed(install_cmd_prefix, env=env) + + +def _load_console_script_names() -> list[str]: + """Return ``[project.scripts]`` entry-point names from pyproject.toml.""" + try: + import tomllib # Python 3.11+ + except ImportError: # pragma: no cover + return [] + + pyproject = PROJECT_ROOT / "pyproject.toml" + if not pyproject.is_file(): + return [] + + try: + with open(pyproject, "rb") as f: + data = tomllib.load(f) + scripts = data.get("project", {}).get("scripts", {}) or {} + return [str(name) for name in scripts if name] + except Exception as e: + logger.debug("console script verification: failed to read pyproject.toml: %s", e) + return [] + + +def _verify_console_scripts_installed( + install_cmd_prefix: list[str], + *, + env: dict[str, str] | None = None, +) -> None: + """Ensure every declared console_script shim exists on disk after install. + + On Windows, ``uv pip install -e .`` can register ``hermes.exe`` in the + wheel RECORD while the file never lands on disk — typically when the live + ``hermes.exe`` shim is locked during ``hermes update``, or when uv/distlib + skips a launcher write. The symptom is ``hermes-agent.exe`` and + ``hermes-acp.exe`` present but ``hermes.exe`` missing, so ``hermes`` drops + off PATH even though the install reported success (issue #52931). + + If any shim is missing we reinstall with ``--reinstall -e .`` under the + same quarantine dance as the primary install path, then re-check. + """ + if not _is_windows(): + return + + scripts_dir = _venv_scripts_dir() + if scripts_dir is None: + return + + names = _load_console_script_names() + if not names: + return + + def _missing() -> list[str]: + return [ + name + for name in names + if not (scripts_dir / f"{name}.exe").is_file() + ] + + missing = _missing() + if not missing: + return + + print( + f" ⚠ Verification: {len(missing)} console script(s) missing on disk: " + f"{', '.join(missing)}" + ) + print(" → Reinstalling entry points with --reinstall...") + + try: + _run_quarantined_install( + install_cmd_prefix + ["install", "--reinstall", "-e", "."], + env=env, + scripts_dir=scripts_dir, + ) + except subprocess.CalledProcessError as e: + logger.warning("console script verification: repair install failed: %s", e) + print( + " ⚠ Entry point repair failed; try `hermes update --force` after " + "closing other hermes processes." + ) + return + + still_missing = _missing() + if still_missing: + print( + f" ⚠ Still missing after repair: {', '.join(still_missing)}. " + "Workaround: python -m hermes_cli.main " + ) + else: + print(" ✓ All console entry points restored") def _verify_core_dependencies_installed( diff --git a/scripts/install.ps1 b/scripts/install.ps1 index 86bf987ef37..fde9d4363d4 100644 --- a/scripts/install.ps1 +++ b/scripts/install.ps1 @@ -1854,6 +1854,48 @@ except Exception: Write-Success "Baseline imports verified in venv" } + if (-not $NoVenv) { + # uv on Windows can register hermes.exe in dist-info/RECORD but fail to + # materialise the .exe (file lock during self-update, distlib edge case). + # Catch it here so a fresh install/update does not finish with a broken + # `hermes` command while hermes-agent.exe / hermes-acp.exe exist + $scriptsDir = Join-Path $InstallDir "venv\Scripts" + $pythonExe = Join-Path $scriptsDir "python.exe" + if ((Test-Path $scriptsDir) -and (Test-Path $pythonExe)) { + $scriptNames = & $pythonExe -c @" +import tomllib +with open('pyproject.toml', 'rb') as fh: + scripts = tomllib.load(fh).get('project', {}).get('scripts', {}) or {} +print(','.join(scripts)) +"@ 2>$null + if ($LASTEXITCODE -eq 0 -and $scriptNames) { + $expected = @($scriptNames.Trim().Split(',') | Where-Object { $_ }) + $missing = @() + foreach ($name in $expected) { + $exe = Join-Path $scriptsDir "$name.exe" + if (-not (Test-Path $exe)) { $missing += "$name.exe" } + } + if ($missing.Count -gt 0) { + Write-Warn "Console entry point(s) missing: $($missing -join ', ')" + Write-Info "Reinstalling entry points..." + $env:UV_PROJECT_ENVIRONMENT = "$InstallDir\venv" + Invoke-NativeWithRelaxedErrorAction { & $UvCmd pip install --reinstall -e . } + $stillMissing = @() + foreach ($name in $expected) { + $exe = Join-Path $scriptsDir "$name.exe" + if (-not (Test-Path $exe)) { $stillMissing += "$name.exe" } + } + if ($stillMissing.Count -gt 0) { + Write-Warn "Entry points still missing after repair: $($stillMissing -join ', ')" + Write-Info "Workaround: `"$pythonExe`" -m hermes_cli.main " + } else { + Write-Success "Console entry points restored" + } + } + } + } + } + # Verify the dashboard deps specifically -- they're the most common thing # users hit and lazy-import errors from `hermes dashboard` are confusing. # If tier 1 failed (the common case), [web] was still picked up by tiers