fix: workaround for ale+playerbots

2026-01-13 09:07:20 +00:00 · 2026-01-02 02:53:47 -05:00
11 changed files with 92 additions and 367 deletions
--- a/.env.template
+++ b/.env.template
@@ -14,7 +14,7 @@ COMPOSE_OVERRIDE_WORLDSERVER_DEBUG_LOGGING_ENABLED=0
 # Project name
 # =====================
 # Customize this to match your deployment slug (used for container names/tags)
-COMPOSE_PROJECT_NAME=azerothcore-realmmaster
+COMPOSE_PROJECT_NAME=azerothcore-stack

 # =====================
 # Storage & Timezone
@@ -96,15 +96,15 @@ AC_DB_IMPORT_IMAGE=acore/ac-wotlk-db-import:master
 AC_AUTHSERVER_IMAGE=acore/ac-wotlk-authserver:master
 AC_WORLDSERVER_IMAGE=acore/ac-wotlk-worldserver:master
 # Services (Playerbots)
-AC_AUTHSERVER_IMAGE_PLAYERBOTS=${COMPOSE_PROJECT_NAME}:authserver-playerbots
-AC_WORLDSERVER_IMAGE_PLAYERBOTS=${COMPOSE_PROJECT_NAME}:worldserver-playerbots
+AC_AUTHSERVER_IMAGE_PLAYERBOTS=azerothcore-realmmaster:authserver-playerbots
+AC_WORLDSERVER_IMAGE_PLAYERBOTS=azerothcore-realmmaster:worldserver-playerbots
 # Services (Module Build Tags)
 # Images used during module compilation and tagging
-AC_AUTHSERVER_IMAGE_MODULES=${COMPOSE_PROJECT_NAME}:authserver-modules-latest
-AC_WORLDSERVER_IMAGE_MODULES=${COMPOSE_PROJECT_NAME}:worldserver-modules-latest
+AC_AUTHSERVER_IMAGE_MODULES=azerothcore-realmmaster:authserver-modules-latest
+AC_WORLDSERVER_IMAGE_MODULES=azerothcore-realmmaster:worldserver-modules-latest
 # Client Data
 AC_CLIENT_DATA_IMAGE=acore/ac-wotlk-client-data:master
-AC_CLIENT_DATA_IMAGE_PLAYERBOTS=${COMPOSE_PROJECT_NAME}:client-data-playerbots
+AC_CLIENT_DATA_IMAGE_PLAYERBOTS=uprightbass360/azerothcore-wotlk-playerbots:client-data-Playerbot
 # Build artifacts
 DOCKER_IMAGE_TAG=master
 AC_AUTHSERVER_IMAGE_BASE=acore/ac-wotlk-authserver
--- a/.gitignore
+++ b/.gitignore
@@ -13,7 +13,6 @@ images/
 node_modules/
 .mcp*/
 scripts/__pycache__/*
-scripts/bash/__pycache__/*
 scripts/python/__pycache__/*
 .env
 package-lock.json
--- a/changelog.sh
+++ b/changelog.sh
@@ -99,36 +99,7 @@ done
 # Get last build time from container metadata
 get_last_build_time() {
  local containers=("ac-worldserver" "ac-authserver")
-  local images=()
-
-  # Require COMPOSE_PROJECT_NAME to be set
-  if [[ -z "${COMPOSE_PROJECT_NAME:-}" ]]; then
-    warn "COMPOSE_PROJECT_NAME not set in environment"
-    return 1
-  fi
-
-  # Use actual image names from environment
-  # Detect variant to check appropriate images
-  if [[ "${STACK_IMAGE_MODE:-standard}" == "playerbots" ]] || [[ "${MODULE_PLAYERBOTS:-0}" == "1" ]] || [[ "${PLAYERBOT_ENABLED:-0}" == "1" ]] || [[ "${STACK_SOURCE_VARIANT:-}" == "playerbots" ]]; then
-    if [[ -z "${AC_WORLDSERVER_IMAGE_PLAYERBOTS:-}" ]] || [[ -z "${AC_AUTHSERVER_IMAGE_PLAYERBOTS:-}" ]]; then
-      warn "Playerbots mode detected but AC_WORLDSERVER_IMAGE_PLAYERBOTS or AC_AUTHSERVER_IMAGE_PLAYERBOTS not set"
-      return 1
-    fi
-    images=(
-      "${AC_WORLDSERVER_IMAGE_PLAYERBOTS}"
-      "${AC_AUTHSERVER_IMAGE_PLAYERBOTS}"
-    )
-  else
-    if [[ -z "${AC_WORLDSERVER_IMAGE:-}" ]] || [[ -z "${AC_AUTHSERVER_IMAGE:-}" ]]; then
-      warn "Standard mode detected but AC_WORLDSERVER_IMAGE or AC_AUTHSERVER_IMAGE not set"
-      return 1
-    fi
-    images=(
-      "${AC_WORLDSERVER_IMAGE}"
-      "${AC_AUTHSERVER_IMAGE}"
-    )
-  fi
-
+  local images=("azerothcore-stack:worldserver-playerbots" "azerothcore-stack:authserver-playerbots")
  local latest_date=""

  # Try to get build timestamp from containers and images
@@ -172,7 +143,7 @@ if [[ -n "$SINCE_DATE" ]]; then
  DATE_DESC="since $SINCE_DATE"
 else
  # Try to use last build time as default
-  LAST_BUILD_DATE=$(get_last_build_time 2>/dev/null) || LAST_BUILD_DATE=""
+  LAST_BUILD_DATE=$(get_last_build_time)

  if [[ -n "$LAST_BUILD_DATE" ]]; then
    SINCE_OPTION="--since=$LAST_BUILD_DATE"
@@ -223,17 +194,11 @@ detect_source_config() {
    $VERBOSE && log "Switched to playerbots variant" >&2
  fi

-  # Repository URLs from environment (required)
-  local standard_repo="${ACORE_REPO_STANDARD}"
-  local standard_branch="${ACORE_BRANCH_STANDARD}"
-  local playerbots_repo="${ACORE_REPO_PLAYERBOTS}"
-  local playerbots_branch="${ACORE_BRANCH_PLAYERBOTS}"
-
-  if [[ -z "$standard_repo" ]] || [[ -z "$standard_branch" ]] || [[ -z "$playerbots_repo" ]] || [[ -z "$playerbots_branch" ]]; then
-    warn "Repository configuration missing from environment"
-    warn "Required: ACORE_REPO_STANDARD, ACORE_BRANCH_STANDARD, ACORE_REPO_PLAYERBOTS, ACORE_BRANCH_PLAYERBOTS"
-    return 1
-  fi
+  # Repository URLs from environment or defaults
+  local standard_repo="${ACORE_REPO_STANDARD:-https://github.com/azerothcore/azerothcore-wotlk.git}"
+  local standard_branch="${ACORE_BRANCH_STANDARD:-master}"
+  local playerbots_repo="${ACORE_REPO_PLAYERBOTS:-https://github.com/mod-playerbots/azerothcore-wotlk.git}"
+  local playerbots_branch="${ACORE_BRANCH_PLAYERBOTS:-Playerbot}"

  if [[ "$variant" == "playerbots" ]]; then
    echo "$playerbots_repo|$playerbots_branch|$LOCAL_STORAGE_ROOT/source/azerothcore-playerbots"
--- a/cleanup.sh
+++ b/cleanup.sh
@@ -146,6 +146,8 @@ sanitize_project_name(){
  project_name::sanitize "$1"
 }

+PROJECT_IMAGE_PREFIX="$(sanitize_project_name "${COMPOSE_PROJECT_NAME:-$DEFAULT_PROJECT_NAME}")"
+
 remove_storage_dir(){
  local path="$1"
  if [ -d "$path" ]; then
@@ -221,7 +223,8 @@ nuclear_cleanup() {

  # Remove project images (server/tool images typical to this project)
  execute_command "Remove acore images" "docker images --format '{{.Repository}}:{{.Tag}}' | grep -E '^acore/' | xargs -r docker rmi"
-  execute_command "Remove project-specific images" "docker images --format '{{.Repository}}:{{.Tag}}' | grep -E \"^${PROJECT_NAME}:\" | xargs -r docker rmi"
+  execute_command "Remove local project images" "docker images --format '{{.Repository}}:{{.Tag}}' | grep -E '^${PROJECT_IMAGE_PREFIX}:' | xargs -r docker rmi"
+  execute_command "Remove legacy playerbots images" "docker images --format '{{.Repository}}:{{.Tag}}' | grep -E '^uprightbass360/azerothcore-wotlk-playerbots' | xargs -r docker rmi"
  execute_command "Remove tool images" "docker images --format '{{.Repository}}:{{.Tag}}' | grep -E 'phpmyadmin|uprightbass360/keira3' | xargs -r docker rmi"

  # Storage cleanup (preserve backups if requested)
--- a/docs/PHASE1_INTEGRATION_TEST_SUMMARY.md
+++ b/docs/PHASE1_INTEGRATION_TEST_SUMMARY.md
@@ -142,7 +142,7 @@ MODULES_ENABLED="mod-playerbots mod-aoe-loot ..."
 **What Gets Built:**
 - AzerothCore with playerbots branch
 - 93 modules compiled and integrated in this run (current manifest: 348 total / 221 supported)
- Custom Docker images: `${COMPOSE_PROJECT_NAME}:worldserver-modules-latest` etc.
+- Custom Docker images: `acore-compose:worldserver-modules-latest` etc.

 ### Deployment Status: READY TO DEPLOY 🚀

--- a/scripts/bash/manage-modules.sh
+++ b/scripts/bash/manage-modules.sh
@@ -174,7 +174,18 @@ install_enabled_modules(){
      continue
    fi
    if [ -d "$dir/.git" ]; then
-      info "$dir already present; skipping clone"
+      info "$dir already present; checking for updates"
+      (cd "$dir" && git fetch origin >/dev/null 2>&1 || warn "Failed to fetch updates for $dir")
+      local current_branch
+      current_branch=$(cd "$dir" && git rev-parse --abbrev-ref HEAD 2>/dev/null || echo "master")
+      if (cd "$dir" && git pull origin "$current_branch" 2>&1 | grep -q "Already up to date"); then
+        info "$dir is already up to date"
+      else
+        ok "$dir updated from remote"
+      fi
+      if [ -n "$ref" ]; then
+        (cd "$dir" && git checkout "$ref") || warn "Unable to checkout ref $ref for $dir"
+      fi
    elif [ -d "$dir" ]; then
      warn "$dir exists but is not a git repository; leaving in place"
    else
--- a/scripts/bash/statusjson.sh
+++ b/scripts/bash/statusjson.sh
@@ -4,7 +4,6 @@ import os
 import re
 import socket
 import subprocess
-import sys
 import time
 from pathlib import Path

@@ -469,14 +468,8 @@ def docker_stats():

 def main():
    env = load_env()
-    project = read_env(env, "COMPOSE_PROJECT_NAME")
-    if not project:
-        print(json.dumps({"error": "COMPOSE_PROJECT_NAME not set in environment"}), file=sys.stderr)
-        sys.exit(1)
-    network = read_env(env, "NETWORK_NAME")
-    if not network:
-        print(json.dumps({"error": "NETWORK_NAME not set in environment"}), file=sys.stderr)
-        sys.exit(1)
+    project = read_env(env, "COMPOSE_PROJECT_NAME", "acore-compose")
+    network = read_env(env, "NETWORK_NAME", "azerothcore")

    services = [
        ("ac-mysql", "MySQL"),
--- a/scripts/bash/validate-env.sh
+++ b/scripts/bash/validate-env.sh
@@ -1,301 +0,0 @@
-#!/bin/bash
-# Validate environment configuration for AzerothCore RealmMaster
-# Usage: ./scripts/bash/validate-env.sh [--strict] [--quiet]
-
-set -euo pipefail
-
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-PROJECT_ROOT="$(cd "$SCRIPT_DIR/../.." && pwd)"
-ENV_FILE="$PROJECT_ROOT/.env"
-TEMPLATE_FILE="$PROJECT_ROOT/.env.template"
-
-# Colors
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[1;33m'
-BLUE='\033[0;34m'
-NC='\033[0m'
-
-# Flags
-STRICT_MODE=false
-QUIET_MODE=false
-EXIT_CODE=0
-
-# Parse arguments
-while [[ $# -gt 0 ]]; do
-  case "$1" in
-    --strict)
-      STRICT_MODE=true
-      shift
-      ;;
-    --quiet)
-      QUIET_MODE=true
-      shift
-      ;;
-    -h|--help)
-      cat <<EOF
-Usage: $0 [OPTIONS]
-
-Validates environment configuration for required variables.
-
-OPTIONS:
-  --strict    Fail on missing optional variables
-  --quiet     Only show errors, suppress info/success messages
-  -h, --help  Show this help
-
-EXIT CODES:
-  0 - All required variables present
-  1 - Missing required variables
-  2 - Missing optional variables (only in --strict mode)
-
-REQUIRED VARIABLES:
-  Project Configuration:
-    COMPOSE_PROJECT_NAME       - Project name for containers/images
-    NETWORK_NAME              - Docker network name
-
-  Repository Configuration:
-    ACORE_REPO_STANDARD       - Standard AzerothCore repository URL
-    ACORE_BRANCH_STANDARD     - Standard AzerothCore branch name
-    ACORE_REPO_PLAYERBOTS     - Playerbots repository URL
-    ACORE_BRANCH_PLAYERBOTS   - Playerbots branch name
-
-  Storage Paths:
-    STORAGE_PATH              - Main storage path
-    STORAGE_PATH_LOCAL        - Local storage path
-
-  Database Configuration:
-    MYSQL_ROOT_PASSWORD       - MySQL root password
-    MYSQL_USER                - MySQL user (typically root)
-    MYSQL_PORT                - MySQL port (typically 3306)
-    MYSQL_HOST                - MySQL hostname
-    DB_AUTH_NAME              - Auth database name
-    DB_WORLD_NAME             - World database name
-    DB_CHARACTERS_NAME        - Characters database name
-    DB_PLAYERBOTS_NAME        - Playerbots database name
-
-  Container Configuration:
-    CONTAINER_MYSQL           - MySQL container name
-    CONTAINER_USER            - Container user (format: uid:gid)
-
-OPTIONAL VARIABLES (checked with --strict):
-  MySQL Performance:
-    MYSQL_INNODB_BUFFER_POOL_SIZE - InnoDB buffer pool size
-    MYSQL_INNODB_LOG_FILE_SIZE    - InnoDB log file size
-    MYSQL_INNODB_REDO_LOG_CAPACITY - InnoDB redo log capacity
-
-  Database Connection:
-    DB_RECONNECT_SECONDS      - Database reconnection delay
-    DB_RECONNECT_ATTEMPTS     - Database reconnection attempts
-
-  Build Configuration:
-    MODULES_REBUILD_SOURCE_PATH - Path to source for module builds
-
-  Backup Configuration:
-    BACKUP_PATH               - Backup storage path
-    BACKUP_RETENTION_DAYS     - Daily backup retention
-    BACKUP_RETENTION_HOURS    - Hourly backup retention
-
-  Image Configuration:
-    AC_AUTHSERVER_IMAGE       - Auth server Docker image
-    AC_WORLDSERVER_IMAGE      - World server Docker image
-    AC_DB_IMPORT_IMAGE        - Database import Docker image
-
-EXAMPLES:
-  $0                  # Basic validation
-  $0 --strict         # Strict validation (check optional vars)
-  $0 --quiet          # Only show errors
-EOF
-      exit 0
-      ;;
-    *)
-      echo -e "${RED}Unknown option: $1${NC}" >&2
-      exit 1
-      ;;
-  esac
-done
-
-log_info() {
-  $QUIET_MODE || echo -e "${BLUE}ℹ️  $*${NC}"
-}
-
-log_success() {
-  $QUIET_MODE || echo -e "${GREEN}✅ $*${NC}"
-}
-
-log_warning() {
-  echo -e "${YELLOW}⚠️  $*${NC}" >&2
-}
-
-log_error() {
-  echo -e "${RED}❌ $*${NC}" >&2
-}
-
-# Load environment
-load_env() {
-  local file="$1"
-  if [[ ! -f "$file" ]]; then
-    return 1
-  fi
-
-  set -a
-  # shellcheck disable=SC1090
-  source "$file" 2>/dev/null || return 1
-  set +a
-  return 0
-}
-
-# Check if variable is set and non-empty
-check_var() {
-  local var_name="$1"
-  local var_value="${!var_name:-}"
-
-  if [[ -z "$var_value" ]]; then
-    return 1
-  fi
-  return 0
-}
-
-# Validate required variables
-validate_required() {
-  local missing=()
-
-  local required_vars=(
-    # Project Configuration
-    "COMPOSE_PROJECT_NAME"
-    "NETWORK_NAME"
-    # Repository Configuration
-    "ACORE_REPO_STANDARD"
-    "ACORE_BRANCH_STANDARD"
-    "ACORE_REPO_PLAYERBOTS"
-    "ACORE_BRANCH_PLAYERBOTS"
-    # Storage Paths
-    "STORAGE_PATH"
-    "STORAGE_PATH_LOCAL"
-    # Database Configuration
-    "MYSQL_ROOT_PASSWORD"
-    "MYSQL_USER"
-    "MYSQL_PORT"
-    "MYSQL_HOST"
-    "DB_AUTH_NAME"
-    "DB_WORLD_NAME"
-    "DB_CHARACTERS_NAME"
-    "DB_PLAYERBOTS_NAME"
-    # Container Configuration
-    "CONTAINER_MYSQL"
-    "CONTAINER_USER"
-  )
-
-  log_info "Checking required variables..."
-
-  for var in "${required_vars[@]}"; do
-    if check_var "$var"; then
-      log_success "$var=${!var}"
-    else
-      log_error "$var is not set"
-      missing+=("$var")
-    fi
-  done
-
-  if [[ ${#missing[@]} -gt 0 ]]; then
-    log_error "Missing required variables: ${missing[*]}"
-    return 1
-  fi
-
-  log_success "All required variables are set"
-  return 0
-}
-
-# Validate optional variables (strict mode)
-validate_optional() {
-  local missing=()
-
-  local optional_vars=(
-    # MySQL Performance Tuning
-    "MYSQL_INNODB_BUFFER_POOL_SIZE"
-    "MYSQL_INNODB_LOG_FILE_SIZE"
-    "MYSQL_INNODB_REDO_LOG_CAPACITY"
-    # Database Connection Settings
-    "DB_RECONNECT_SECONDS"
-    "DB_RECONNECT_ATTEMPTS"
-    # Build Configuration
-    "MODULES_REBUILD_SOURCE_PATH"
-    # Backup Configuration
-    "BACKUP_PATH"
-    "BACKUP_RETENTION_DAYS"
-    "BACKUP_RETENTION_HOURS"
-    # Image Configuration
-    "AC_AUTHSERVER_IMAGE"
-    "AC_WORLDSERVER_IMAGE"
-    "AC_DB_IMPORT_IMAGE"
-  )
-
-  log_info "Checking optional variables..."
-
-  for var in "${optional_vars[@]}"; do
-    if check_var "$var"; then
-      log_success "$var is set"
-    else
-      log_warning "$var is not set (using default)"
-      missing+=("$var")
-    fi
-  done
-
-  if [[ ${#missing[@]} -gt 0 ]]; then
-    log_warning "Optional variables not set: ${missing[*]}"
-    return 2
-  fi
-
-  log_success "All optional variables are set"
-  return 0
-}
-
-# Main validation
-main() {
-  log_info "Validating environment configuration..."
-  echo ""
-
-  # Check if .env exists
-  if [[ ! -f "$ENV_FILE" ]]; then
-    log_error ".env file not found at $ENV_FILE"
-    log_info "Copy .env.template to .env and configure it:"
-    log_info "  cp $TEMPLATE_FILE $ENV_FILE"
-    exit 1
-  fi
-
-  # Load environment
-  if ! load_env "$ENV_FILE"; then
-    log_error "Failed to load $ENV_FILE"
-    exit 1
-  fi
-
-  log_success "Loaded environment from $ENV_FILE"
-  echo ""
-
-  # Validate required variables
-  if ! validate_required; then
-    EXIT_CODE=1
-  fi
-
-  echo ""
-
-  # Validate optional variables if strict mode
-  if $STRICT_MODE; then
-    if ! validate_optional; then
-      [[ $EXIT_CODE -eq 0 ]] && EXIT_CODE=2
-    fi
-    echo ""
-  fi
-
-  # Final summary
-  if [[ $EXIT_CODE -eq 0 ]]; then
-    log_success "Environment validation passed ✨"
-  elif [[ $EXIT_CODE -eq 1 ]]; then
-    log_error "Environment validation failed (missing required variables)"
-  elif [[ $EXIT_CODE -eq 2 ]]; then
-    log_warning "Environment validation passed with warnings (missing optional variables)"
-  fi
-
-  exit $EXIT_CODE
-}
-
-main "$@"
--- a/scripts/go/go.mod
+++ b/scripts/go/go.mod
@@ -1,4 +1,4 @@
-module azerothcore-realmmaster/statusdash
+module acore-compose/statusdash

 go 1.22

--- a/scripts/hooks/mod-ale-patches
+++ b/scripts/hooks/mod-ale-patches
@@ -1,5 +1,6 @@
 #!/bin/bash
 # Module-specific hook for mod-ale compatibility patches
+# NOTE: These patches are primarily needed for the AzerothCore playerbots fork
 set -e

 # Hook environment
@@ -7,12 +8,34 @@ MODULE_KEY="${MODULE_KEY:-}"
 MODULE_DIR="${MODULE_DIR:-}"
 MODULE_NAME="${MODULE_NAME:-}"

+# Detect if we're building with playerbots fork
+IS_PLAYERBOTS_FORK=0
+if [ -n "$MODULE_DIR" ]; then
+    # Check if the parent directory structure indicates playerbots fork
+    if echo "$MODULE_DIR" | grep -q "azerothcore-playerbots"; then
+        IS_PLAYERBOTS_FORK=1
+    fi
+fi
+
+# Feature flags (set to 0 to disable specific patches)
+APPLY_MOVEPATH_PATCH="${APPLY_MOVEPATH_PATCH:-0}"  # Disabled by default - appears unnecessary
+# SendTrainerList patch: auto-detect based on fork, but can be overridden
+if [ -z "${APPLY_SENDTRAINERLIST_PATCH:-}" ]; then
+    APPLY_SENDTRAINERLIST_PATCH="$IS_PLAYERBOTS_FORK"  # Only needed for playerbots fork
+else
+    APPLY_SENDTRAINERLIST_PATCH="${APPLY_SENDTRAINERLIST_PATCH}"
+fi
+
 if [ -z "$MODULE_DIR" ] || [ ! -d "$MODULE_DIR" ]; then
    echo "❌ mod-ale-patches: Invalid module directory: $MODULE_DIR"
    exit 2
 fi

-echo "🔧 mod-ale-patches: Applying compatibility fixes to $MODULE_NAME"
+if [ "$IS_PLAYERBOTS_FORK" = "1" ]; then
+    echo "🔧 mod-ale-patches: Applying playerbots fork compatibility fixes to $MODULE_NAME"
+else
+    echo "🔧 mod-ale-patches: Checking compatibility fixes for $MODULE_NAME"
+fi

 # Apply MovePath compatibility patch
 apply_movepath_patch() {
@@ -37,10 +60,42 @@ apply_movepath_patch() {
    fi
 }

+# Apply SendTrainerList compatibility patch
+apply_sendtrainerlist_patch() {
+    local target_file="$MODULE_DIR/src/LuaEngine/methods/PlayerMethods.h"
+
+    if [ ! -f "$target_file" ]; then
+        echo "   ⚠️  SendTrainerList patch target file missing: $target_file"
+        return 1
+    fi
+
+    # Check if the buggy code exists (without GET_GUID())
+    if grep -q 'player->GetSession()->SendTrainerList(obj);' "$target_file"; then
+        # Apply the fix by adding ->GET_GUID()
+        if sed -i 's/player->GetSession()->SendTrainerList(obj);/player->GetSession()->SendTrainerList(obj->GET_GUID());/' "$target_file"; then
+            echo "   ✅ Applied SendTrainerList compatibility fix"
+            return 0
+        else
+            echo "   ❌ Failed to apply SendTrainerList compatibility fix"
+            return 2
+        fi
+    else
+        echo "   ✅ SendTrainerList compatibility fix already present"
+        return 0
+    fi
+}
+
 # Apply all patches
 patch_count=0
-if apply_movepath_patch; then
-    patch_count=$((patch_count + 1))
+if [ "$APPLY_MOVEPATH_PATCH" = "1" ]; then
+    if apply_movepath_patch; then
+        patch_count=$((patch_count + 1))
+    fi
+fi
+if [ "$APPLY_SENDTRAINERLIST_PATCH" = "1" ]; then
+    if apply_sendtrainerlist_patch; then
+        patch_count=$((patch_count + 1))
+    fi
 fi

 if [ $patch_count -eq 0 ]; then
--- a/scripts/python/update_module_manifest.py
+++ b/scripts/python/update_module_manifest.py
@@ -46,7 +46,7 @@ CATEGORY_BY_TYPE = {
    "data": "data",
    "cpp": "uncategorized",
 }
-USER_AGENT = "azerothcore-realmmaster-module-manifest"
+USER_AGENT = "acore-compose-module-manifest"


 def parse_args(argv: Sequence[str]) -> argparse.Namespace: