From efc4100640a638e946669a152db1fac9a11abd18 Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 26 Jun 2026 13:20:49 +0000 Subject: [PATCH] fix dispatcher hyphen bug in 4 more services setup.sh's run_service() looks up install_ using the literal hyphenated registered name, not an underscore-converted one. borg-backup, calibre-web, gaming-backup, and stirling-pdf all used underscored function names and were therefore uninstallable ("has no install_"). Same fix already applied to ai-gpu/ai-stack; this closes out the rest of the repo-wide audit. Co-Authored-By: Claude Sonnet 4.6 Claude-Session: https://claude.ai/code/session_01Nb2vJ8W7bHKx1JXVvpCraH --- services/borg-backup.sh | 4 ++-- services/calibre-web.sh | 4 ++-- services/gaming-backup.sh | 4 ++-- services/stirling-pdf.sh | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/services/borg-backup.sh b/services/borg-backup.sh index 912c8a3..fb24d5e 100644 --- a/services/borg-backup.sh +++ b/services/borg-backup.sh @@ -207,7 +207,7 @@ fi register_service borg-backup backup "Encrypted backup of all Docker services via Borg" -install_borg_backup() { +install_borg-backup() { require_docker || return 1 local DIR="$DOCKER_DIR/borg-backup" @@ -660,4 +660,4 @@ SVCEOF } # Run immediately when executed directly (deferred until after function definition) -[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_borg_backup +[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_borg-backup diff --git a/services/calibre-web.sh b/services/calibre-web.sh index f55bae1..8ac3643 100644 --- a/services/calibre-web.sh +++ b/services/calibre-web.sh @@ -181,7 +181,7 @@ fi register_service calibre-web media "Ebook library web UI with metadata editing (Calibre-Web)" 8083 -install_calibre_web() { +install_calibre-web() { require_docker || return 1 log_info "Installing Calibre-Web..." @@ -295,4 +295,4 @@ MD } # Run immediately when executed directly (deferred until after function definition) -[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_calibre_web +[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_calibre-web diff --git a/services/gaming-backup.sh b/services/gaming-backup.sh index 473963a..c25e6df 100644 --- a/services/gaming-backup.sh +++ b/services/gaming-backup.sh @@ -89,7 +89,7 @@ fi register_service gaming-backup backup "Gaming saves backup (Minecraft worlds, emulator saves, Steam)" -install_gaming_backup() { +install_gaming-backup() { log_info "Setting up gaming saves backup (Kopia)..." # ── Repo-conventional paths ────────────────────────────────────────────── @@ -497,4 +497,4 @@ UNITEOF } # Run immediately when executed directly (deferred until after function definition) -[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_gaming_backup +[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_gaming-backup diff --git a/services/stirling-pdf.sh b/services/stirling-pdf.sh index 0bbeb23..2d1e9ef 100644 --- a/services/stirling-pdf.sh +++ b/services/stirling-pdf.sh @@ -181,7 +181,7 @@ fi register_service stirling-pdf utilities "PDF toolkit — merge, split, compress, OCR (Stirling PDF)" 8070 -install_stirling_pdf() { +install_stirling-pdf() { require_docker || return 1 log_info "Installing Stirling PDF..." @@ -313,4 +313,4 @@ MD } # Run immediately when executed directly (deferred until after function definition) -[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_stirling_pdf +[[ "${_RUN_STANDALONE:-0}" == 1 ]] && install_stirling-pdf