Skip to content

fix(stack-monitor): real version signal for litellm/llama.cpp/comfyui + ROLLING status#49

Merged
AlienWalker1995 merged 1 commit into
mainfrom
fix/stack-monitor-version-audit
Jun 23, 2026
Merged

fix(stack-monitor): real version signal for litellm/llama.cpp/comfyui + ROLLING status#49
AlienWalker1995 merged 1 commit into
mainfrom
fix/stack-monitor-version-audit

Conversation

@AlienWalker1995

Copy link
Copy Markdown
Owner

Problem

The daily GitHub Monitor repo-news cron (scripts/stack_monitor.py) gave no usable signal for several services:

  • LiteLLM has no version pin anywhere (model-gateway is FROM ghcr.io/berriai/litellm:main-stable, a rolling tag) → fell into classify_severity's no-digit branch → silent MEDIUM / "Version format unknown" every run.
  • llama.cpp (server-cuda rolling tag + TurboQuant fork) → same dead end.
  • ComfyUI was compared against a hardcoded v0.20.1 while the box actually runs 0.17.0.

Fix

  • resolve_current_version() reads ComfyUI live from data/comfyui-storage/ComfyUI/comfyui_version.py and LiteLLM live from the running model-gateway container.
  • New ROLLING severity for non-semver pins — surfaces them for manual "rebuild to update" review instead of a misleading MEDIUM.
  • run_cmd hardened to UTF-8/errors=replace (crashed on non-UTF-8 release notes on a cp1252 host, leaving stdout=None).
  • Dropped stale PINNED hardcodes for ComfyUI/LiteLLM.

Validation

Full E2E run now reports real deltas: LiteLLM 1.82.3 → v1.89.2, ComfyUI 0.17.0 → v0.25.1, llama.cpp server-cuda → b9724 (ROLLING), Hermes CRITICAL (security note). tests/test_stack_monitor_versions.py — 7 new tests, all 14 pass.

Note: the companion U+200D fix (cron prompt sanitization) lives in gitignored data/ and is already applied to the live scheduler.

🤖 Generated with Claude Code

… + ROLLING status

The daily GitHub Monitor cron reported "version format unknown" for every
rolling-tag / source-built service and compared ComfyUI against a stale
hardcoded pin (v0.20.1 while the box runs 0.17.0).

- resolve_current_version(): read ComfyUI live from comfyui_version.py and
  LiteLLM live from the running model-gateway container (it has no version
  pin anywhere — model-gateway is FROM litellm:main-stable).
- New ROLLING severity for non-semver pins (llama.cpp server-cuda) so they
  surface for manual review instead of a misleading MEDIUM "version unknown".
- Harden run_cmd with UTF-8/errors=replace (crashed on non-UTF-8 release
  notes on a cp1252 host, leaving stdout=None).
- Drop the stale ComfyUI/LiteLLM hardcodes from PINNED.

Tests: tests/test_stack_monitor_versions.py (7 new, all pass).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@AlienWalker1995 AlienWalker1995 force-pushed the fix/stack-monitor-version-audit branch from a88efd4 to 6b76faa Compare June 23, 2026 17:34
@AlienWalker1995 AlienWalker1995 merged commit dd3860e into main Jun 23, 2026
5 checks passed
@AlienWalker1995 AlienWalker1995 deleted the fix/stack-monitor-version-audit branch June 23, 2026 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant