External/agent-skills
1
0
Fork 0
mirror of https://github.com/addyosmani/agent-skills.git synced 2026-08-12 18:07:26 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #406 from youjinch/fix/simplify-ignore-test-no-jq

Browse Source 
fix(hooks): make simplify-ignore-test pass on machines without jq
This commit is contained in:
Federico Bartoli
2026-07-18 01:51:43 +02:00
committed by GitHub
parent 5a1b82d644 dbb32a565c
commit aa4e390453
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
hooks/simplify-ignore-test.sh
+7 -1
View File
@@ -238,8 +238,14 @@ assert_eq "HTML suffix preserved" "1" "$(grep -c '\-\->' "$DEST")"
# ── Test 10: JSON parsing error warning ──────────────────────────────────
printf '\nTest 10: Malformed JSON input produces warning\n'
# Without jq the hook's guard exits before parsing input. Assert whichever
# branch this machine can exercise (same pattern as session-start-test.sh).
warning_out=$(echo 'NOT_JSON{{{' | bash hooks/simplify-ignore.sh 2>&1) || true
assert_eq "warning on bad JSON" "1" "$(printf '%s' "$warning_out" | grep -c 'Warning.*failed to parse')"
if command -v jq >/dev/null 2>&1; then
assert_eq "warning on bad JSON" "1" "$(printf '%s' "$warning_out" | grep -c 'Warning.*failed to parse')"
else
assert_eq "missing-jq guard surfaced" "1" "$(printf '%s' "$warning_out" | grep -c 'error: missing jq')"
fi
# ── Summary ──────────────────────────────────────────────────────────────
printf '\n══════════════════════════════════════════\n'