fix: HIGH quality-debt batch — logic bugs, exit codes, path typo, arg guards

.agents/scripts/comfy-cli-helper.sh

@@ -111,6 +111,10 @@ cmd_setup() {
 	while [[ $# -gt 0 ]]; do
 		case "$1" in
 		--path)
+			if [[ $# -lt 2 ]]; then
+				log_error "--path requires a value"
+				return 1
+			fi
 			path="$2"
 			shift 2
 			;;
@@ -146,10 +150,18 @@ cmd_launch() {
 	while [[ $# -gt 0 ]]; do
 		case "$1" in
 		--port)
+			if [[ $# -lt 2 ]]; then
+				log_error "--port requires a value"
+				return 1
+			fi
 			port="$2"
 			shift 2
 			;;
 		--listen)
+			if [[ $# -lt 2 ]]; then
+				log_error "--listen requires a value"
+				return 1
+			fi
 			listen="$2"
 			shift 2
 			;;
@@ -239,6 +251,10 @@ cmd_snapshot_save() {
 	while [[ $# -gt 0 ]]; do
 		case "$1" in
 		--output)
+			if [[ $# -lt 2 ]]; then
+				log_error "--output requires a value"
+				return 1
+			fi
 			output="$2"
 			shift 2
 			;;

.agents/scripts/eeat-score-helper.sh

@@ -810,7 +810,7 @@ do_analyze() {
 	local urls=()
 	if [[ "$input_file" == *.json ]]; then
 		# JSON format - extract URLs with status 200
-		mapfile -t urls < <(jq -r '.[] | select(.status_code == 200) | .url' "$input_file" 2>/dev/null || echo "")
+		mapfile -t urls < <(jq -r '.[] | select(.status_code == 200) | .url' "$input_file" 2>/dev/null)
 	elif [[ "$input_file" == *.csv ]]; then
 		# CSV format - extract URLs from first column where status is 200
 		mapfile -t urls < <(tail -n +2 "$input_file" | awk -F',' '$2 == "200" || $2 == 200 {gsub(/"/, "", $1); print $1}')

.agents/scripts/email-batch-convert-helper.sh

@@ -98,6 +98,11 @@ cmd_convert() {
 		fi
 	done < <(find "$input_dir" -type f \( -name "*.eml" -o -name "*.msg" \) -print0 2>/dev/null)
 
+	if [[ "$count" -gt 0 && "$success" -eq 0 ]]; then
+		print_error "Conversion complete: ${success}/${count} succeeded, ${failed} failed"
+		print_error "All conversions failed"
+		return 1
+	fi
 	print_success "Conversion complete: ${success}/${count} succeeded, ${failed} failed"
 	return 0
 }