String-to-bool normalization treats "false" as true

[erensunerr]

Affected file: sk/microsoft-agent-framework/skills/azure-architecture-autopilot/scripts/cli.py

What I hit

I passed in a service payload with {"private": "false"} and the generated output still flagged it as private.

I’m thinking this comes from svc["private"] = bool(svc["private"]) where any non-empty string is cast to True.

Repro

1. Pass {"private": "false"} into the service normalization path.
2. Inspect output and confirm private remains True.

Expected

String booleans should map semantically ("true", "false", etc.), not by truthy casting.

Actual

"false" and other non-empty values are interpreted as true.

Suggested fix

Implement explicit string normalization for true/false inputs before assignment.

[erensunerr]

Btw I’m working on a free project called Skillfully, and it helps skill authors like you get direct feedback from the users' coding agents.

It takes <30s to start tracking your skill; it will help you avoid this situation in the future :))

[aaronpowell]

If there are improvements that can be made to skill files (or any files), do send through a PR to address them.