[BUG] /autoserver reload clears server configurations, causing infinite "Command not found" loop

[hongshaoluobotou]

Description

After executing /autoserver reload, the plugin loses server-specific configurations (specifically stop-command), causing an infinite loop of failed stop attempts that spams the console with RuntimeException: Command not found errors.

Item	Version/Details
Proxy Software	Velocity
Plugin	AutoServer
Related Software	MCSManager Panel (daemon API)

Steps to Reproduce

1. Configure a server (e.g., warwall) with stop-command in config.yml
2. Start the Velocity proxy with AutoServer loaded
3. Execute /autoserver reload in console
4. Wait for the auto-shutdown timer to trigger (or trigger it manually)
5. Observe console spam with Stop command not found for warwall

Expected Behavior

/autoserver reload should reload configuration without losing existing server definitions
If a server configuration is invalid, the plugin should log a single warning and skip that server
Failed stop attempts should not retry in a tight loop

Actual Behavior

[10:55:20 INFO] [autoserver]: Configuration reloaded.
[10:56:35 ERROR] [autoserver]: Stop command not found for warwall
[10:56:35 ERROR] [autoserver]: Failed to stop server: java.lang.RuntimeException: Command not found
[10:56:35 ERROR] [autoserver]: error: java.lang.RuntimeException: Command not found
[10:56:35 INFO] [autoserver]: Attempting to stop server: warwall
[10:56:35 ERROR] [autoserver]: Stop command not found for warwall
... (repeats 20+ times per second)

Logs

Before reload (working):

[10:39:16 INFO] [autoserver]: Running start command for warwall server. "curl -s -X GET 'http://localhost:18700/api/protected_instance/open?daemonId=...'"
[10:39:34 INFO] [autoserver]: Server warwall is online. Moving queued players...

After reload (broken):

[10:56:35 ERROR] [autoserver]: Stop command not found for warwall
[10:56:35 ERROR] [autoserver]: Failed to stop server: java.lang.RuntimeException: Command not found
[10:56:36 INFO] [autoserver]: Attempting to stop server: warwall
... (infinite loop)

Impact

latest.log

Console spam: ~20+ error messages per second
CPU usage: Wasted cycles on failed retry loop
Operational risk: Unable to gracefully stop servers after config reload
Data integrity: Potential race conditions if stop commands are intermittently missing

By the way, when setting autoShutdownDelay = 0 (intending to disable auto-shutdown), the plugin still attempts to stop servers after /autoserver reload.

[hongshaoluobotou]

before reload

# AutoServer Configuration File
# Author: Artificial-720
# For issues or suggestions, please submit them at: https://github.com/Artificial-720/AutoServer/issues

# Should AutoServer check for updates?
checkForUpdates = false

# Time (in minutes) between maintenance task executions.
# You generally don't need to change this value.
maintenanceInterval = 5

# Number of requests allowed per user per minute.
# Limits how frequently a user can initiate a new request.
startRateLimit = 2

[messages]
# Prefix added to all messages displayed to the player.
prefix = "[<green>AutoServer<reset>] "

# Message displayed to the player when they attempt to connect to a server that is currently offline.
# %serverName% will be replaced with the name of the backend server.
starting = "<red> %serverName% <reset> 当前已离线。 服务器正在启动，稍后将自动连接。"

# Message displayed to the player if the server fails to start or cannot be connected to.
# %serverName% will be replaced with the name of the backend server.
failed = "<red> 无法连接到 %serverName% <reset> 请稍后重试或者汇报此问题给服主"

# Message displayed to the player when the server is ready, indicating that they will be connected shortly.
# %serverName% will be replaced with the name of the backend server.
notify = "服务器 %serverName% 已就绪，您将在5秒后连接..."

# Message shown to users when they exceed the rate limit.
# %time% will be replaced with the number of seconds remaining.
startRateLimitExceeded = "请求过于频繁，请在%time%喵后重试。"

# 在此处定义服务器。
[servers]
# 为每个需要管理的服务器创建一个配置表
# 表名必须与 velocity.toml 文件中定义的服务器名称一致
# 将 "example" 替换为 velocity.toml 中的服务器名称
#[servers.example]
# 在本地启动服务器的命令。
#start = ""
# 在本地停止服务器的命令。
#stop = ""
# 指定服务器是远程的（true）还是本地的（false）（默认值：false）。
#remote = false
# 远程服务器监听启动命令的端口号（默认值：8080）。
#port = 8080
# 尝试连接玩家前的等待时间（秒）（默认值：60）。
# 对于远程服务器，由于网络延迟，通常需要更长的延迟时间。
#startupDelay = 60
# 验证服务器是否成功关闭前的等待时间（秒）。
#shutdownDelay = 5
# 服务器运行目录的路径。
# 如果未提供，则使用与 Velocity 相同的目录
#workingDirectory = ""
# 控制是否保留开头和结尾的引号，
# 默认在非 Windows 系统上会移除引号，除非显式设置为 `true`。（默认值：null）
#preserveQuotes = false
# 指定远程服务器是否对消息使用安全验证（默认值：true）。
# 注意：这需要将 Velocity 的 forwarding.secret 中的确切值复制到远程配置中
#security = true
# 自动关闭服务器前的等待时间（秒）。
# 设置为 0 可禁用自动关闭功能。
#autoShutdownDelay = 1800

[servers.warwall]
# 启动命令 - 调用 MCSManager API 启动实例
start = "curl -s -X GET 'http://localhost:18700/api/protected_instance/open?daemonId=0285ddb5621f47b0a16dbfbb03684f34&uuid=2b790fd53dfa4c76b53cf5903eea9714&apikey=***********************************' -H 'X-Requested-With: XMLHttpRequest'"
# 停止命令 - 调用 MCSManager API 停止实例
stop = "curl -s -X GET 'http://localhost:18700/api/protected_instance/stop?daemonId=0285ddb5621f47b0a16dbfbb03684f34&uuid=2b790fd53dfa4c76b53cf5903eea9714&apikey=***********************************' -H 'X-Requested-With: XMLHttpRequest'"
startupDelay = 10
shutdownDelay = 5
autoShutdownDelay = 60

# Except for the uuid, everything is the same
[servers.manhunt2]
...

# Except for the uuid, everything is the same
[servers.lobby]
...

# more server config ...

# Example Configuration for a REMOTE server named "creative".
[servers.creative]
# No local start and stop commands since it is a remote server.
remote = true
# Port the remote server listens on for start command (default: 8080).
port = 8080
# Time in seconds to wait before attempting to connect a player.
# For a remote server probably want a longer delay
startupDelay = 120
# Enable security for the connection with this backend server.
# If not provided the default is true.
security = true


# Configure logging here
[logging]
# Logging level: possible values - TRACE, DEBUG, INFO, WARN, ERROR
level = "INFO"

edit and reload

# AutoServer Configuration File
# Author: Artificial-720
# For issues or suggestions, please submit them at: https://github.com/Artificial-720/AutoServer/issues

# Should AutoServer check for updates?
checkForUpdates = false

# Time (in minutes) between maintenance task executions.
# You generally don't need to change this value.
maintenanceInterval = 5

# Number of requests allowed per user per minute.
# Limits how frequently a user can initiate a new request.
startRateLimit = 2

[messages]
# Prefix added to all messages displayed to the player.
prefix = "[<green>AutoServer<reset>] "

# Message displayed to the player when they attempt to connect to a server that is currently offline.
# %serverName% will be replaced with the name of the backend server.
starting = "<red> %serverName% <reset> 当前已离线。 服务器正在启动，稍后将自动连接。"

# Message displayed to the player if the server fails to start or cannot be connected to.
# %serverName% will be replaced with the name of the backend server.
failed = "<red> 无法连接到 %serverName% <reset> 请稍后重试或者汇报此问题给服主"

# Message displayed to the player when the server is ready, indicating that they will be connected shortly.
# %serverName% will be replaced with the name of the backend server.
notify = "服务器 %serverName% 已就绪，您将在5秒后连接..."

# Message shown to users when they exceed the rate limit.
# %time% will be replaced with the number of seconds remaining.
startRateLimitExceeded = "请求过于频繁，请在%time%喵后重试。"

# Define servers here.
[servers]
# For each server you want to have managed create a table for it
# Table name must match server name located in velocity.toml file
# You would replace "example" with server name from velocity.toml
#[servers.example]
# Command to start the server locally.
#start = ""
# Command to stop the server locally.
#stop = ""
# Specifies whether the server is remote (true) or local (false) (default: false).
#remote = false
# Port number on which the remote server listens for the start command (default: 8080).
#port = 8080
# Time in seconds to wait before attempting to connect a player. (default: 60)
# For a remote server, a longer delay is often necessary due to network latency.
#startupDelay = 60
# Time in seconds to wait before verifying whether the server has successfully shut down.
#shutdownDelay = 5
# Path to the directory where the server runs.
# If not provided uses same directory as Velocity
#workingDirectory = ""
# Controls whether leading and trailing quotes are preserved,
# with quotes being removed by default on non-Windows systems unless explicitly set to `true`. (default: null)
#preserveQuotes = false
# Specifies whether the REMOTE server should use security on message (default: true).
# Note this requires copying the exact value from Velocity's forwarding.secret to the remote config
#security = true
# Duration (in seconds) to wait before automatically shutting down the server.
# Set to 0 to disable automatically shutting down the server.
#autoShutdownDelay = 1800


# Example Configuration for a REMOTE server named "creative".
[servers.creative]
# No local start and stop commands since it is a remote server.
remote = true
# Port the remote server listens on for start command (default: 8080).
port = 8080
# Time in seconds to wait before attempting to connect a player.
# For a remote server probably want a longer delay
startupDelay = 120
# Enable security for the connection with this backend server.
# If not provided the default is true.
security = true


# Configure logging here
[logging]
# Logging level: possible values - TRACE, DEBUG, INFO, WARN, ERROR
level = "INFO"