Address code review feedback - improve error handling and use bash built-ins

Co-authored-by: jmiller-moko <230051081+jmiller-moko@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2026-01-04 01:26:21 +00:00
parent d89744df28
commit 2534fd3cae
2 changed files with 7 additions and 2 deletions

View File

@@ -85,7 +85,7 @@ validate_prefix() {
validate_version() {
local v="$1"
if ! printf '%s' "$v" | grep -Eq '^[0-9]{2}\.[0-9]{2}\.[0-9]{2}$'; then
if [[ ! "$v" =~ ^[0-9]{2}\.[0-9]{2}\.[0-9]{2}$ ]]; then
die "Invalid version format: $v (expected NN.NN.NN)"
fi
}

View File

@@ -52,7 +52,12 @@ validate_yaml_syntax() {
python3 - "$file" <<'PYEOF'
import sys
import yaml
try:
import yaml
except ModuleNotFoundError:
print("WARNING: PyYAML module not installed. Install with: pip3 install pyyaml")
sys.exit(0)
file_path = sys.argv[1]