vhost-import: address code review feedback

Copilot · PDowney · web-flow · commit 705d7dfd3f48 · 2026-04-10T22:51:31.000Z
Agent-Logs-Url: https://github.com/EngineScript/EngineScript/sessions/a63d09ad-f51e-4c06-b402-f5a643060c12 Co-authored-by: PDowney <11467177+PDowney@users.noreply.github.com>
diff --git a/scripts/functions/vhost/vhost-import.sh b/scripts/functions/vhost/vhost-import.sh
@@ -68,7 +68,7 @@ validate_zip_archive_paths() {
 # Returns true (0) when a directory is absent or contains no files/subdirs
 is_directory_absent_or_empty() {
     local dir="$1"
-    [[ ! -d "$dir" ]] || [[ -z "$(find "$dir" -mindepth 1 -print -quit 2>/dev/null)" ]]
+    [[ ! -d "$dir" ]] || [[ -z "$(find "$dir" -mindepth 1 -print -quit)" ]]
 }
 
 # --- Instructions for Preparing Files ---
@@ -156,6 +156,8 @@ extract_prefix_from_db() {
     # Look for CREATE TABLE or INSERT INTO lines with common tables (_options or _users)
     # Capture the table name between backticks/quotes, then strip _options/_users to derive prefix
     local search_pattern="(CREATE TABLE|INSERT INTO)[[:space:]]+(\`|\")[^\`\"]+_(options|users)(\`|\")"
+    # NOTE: [^\`\"]+ intentionally allows hyphens and other characters valid in MySQL table names,
+    # broadening the previous [a-zA-Z0-9_]+ which would miss prefixes like "my-site_".
     local table_name=""
 
     # Use zgrep for .gz, grep for .sql.
