From 2231c315d0ca498e1ee77804a111eb875a9196c1 Mon Sep 17 00:00:00 2001 From: Neha Mandge Date: Wed, 6 May 2026 05:21:18 +0530 Subject: [PATCH] fix: implement semantic string-to-bool normalization in cli.py (#1466) (#1623) --- skills/azure-architecture-autopilot/scripts/cli.py | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/skills/azure-architecture-autopilot/scripts/cli.py b/skills/azure-architecture-autopilot/scripts/cli.py index 982f2d7f..bf3fc8c0 100644 --- a/skills/azure-architecture-autopilot/scripts/cli.py +++ b/skills/azure-architecture-autopilot/scripts/cli.py @@ -137,7 +137,15 @@ def _normalize_services(services): if isinstance(svc.get("details"), str): svc["details"] = [svc["details"]] if isinstance(svc.get("private"), str): - svc["private"] = bool(svc["private"]) + val = svc["private"].lower() + if val in ("true", "1", "yes", "on"): + svc["private"] = True + elif val in ("false", "0", "no", "off"): + svc["private"] = False + else: + # Log warning for invalid values + print(f"WARNING: Invalid boolean value '{svc['private']}' for 'private' field. Defaulting to False.", file=sys.stderr) + svc["private"] = False return services