Update xml_wellformed.sh

scripts/validate/xml_wellformed.sh

@@ -32,32 +32,22 @@ set -euo pipefail
 
 SRC_DIR="${SRC_DIR:-src}"
 
-json_escape() { python3 - <<'PY' "$1"; import json,sys; print(json.dumps(sys.argv[1])); PY; }
+json_escape() {
-
+  python3 -c 'import json,sys; print(json.dumps(sys.argv[1]))' "$1"
-emit_ok() {
-	local extra="${1:-}"
-	if [ -n "${extra}" ]; then
-		printf '{"status":"ok",%s}
-' "${extra}"
-	else
-		printf '{"status":"ok"}
-'
-	fi
 }
 
-emit_fail() {
+fail() {
-	local msg="$1"
+  local msg="$1"
-	local extra="${2:-}"
+  local extra="${2:-}"
-	if [ -n "${extra}" ]; then
+  if [ -n "${extra}" ]; then
-		printf '{"status":"fail","error":%s,%s}
+    printf '{"status":"fail","error":%s,%s}\n' "$(json_escape "${msg}")" "${extra}"
-' "$(json_escape "${msg}")" "${extra}"
+  else
-	else
+    printf '{"status":"fail","error":%s}\n' "$(json_escape "${msg}")"
-		printf '{"status":"fail","error":%s}
+  fi
-' "$(json_escape "${msg}")"
+  exit 1
-	fi
 }
 
-[ -d "${SRC_DIR}" ] || { emit_fail "src directory missing" "\"src_dir\":$(json_escape "${SRC_DIR}")"; exit 1; }
+[ -d "${SRC_DIR}" ] || fail "src directory missing" "\"src_dir\":$(json_escape "${SRC_DIR}")"
 
 python3 - <<'PY' "${SRC_DIR}"
 import json
@@ -70,16 +60,27 @@ xml_files = sorted([p for p in src.rglob('*.xml') if p.is_file()])
 
 bad = []
 for p in xml_files:
-	try:
+  try:
-		ET.parse(p)
+    ET.parse(p)
-	except Exception as e:
+  except Exception as e:
-		bad.append({"path": str(p), "error": str(e)})
+    bad.append({"path": str(p), "error": str(e)})
 
 if bad:
-	print(json.dumps({"status":"fail","error":"XML parse failed","bad_count":len(bad),"bad":bad[:25]}, ensure_ascii=False))
+  print(json.dumps({
-	sys.exit(1)
+    "status": "fail",
+    "error": "XML parse failed",
+    "src_dir": str(src),
+    "xml_count": len(xml_files),
+    "bad_count": len(bad),
+    "bad": bad[:25],
+  }, ensure_ascii=False))
+  sys.exit(1)
 
-print(json.dumps({"status":"ok","src_dir":str(src),"xml_count":len(xml_files)}, ensure_ascii=False))
+print(json.dumps({
+  "status": "ok",
+  "src_dir": str(src),
+  "xml_count": len(xml_files),
+}, ensure_ascii=False))
 PY
 
 echo "xml_wellformed: ok"