Add JQ again

Author: noelsaw1

.github/workflows/ci.yml

@@ -128,6 +128,9 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
+      - name: Install dependencies
+        run: sudo apt-get update && sudo apt-get install -y jq
+
       - name: Make scripts executable
         run: |
           chmod +x ./dist/bin/check-performance.sh

CHANGELOG.md

@@ -25,13 +25,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Fixes "command not found" errors when running tests in CI environment
   - **Impact:** CI tests now run successfully on pull requests
 
-- **Test Suite** - Fixed output format parsing to work across macOS and Ubuntu
-  - Changed `run_test()` function to explicitly request `--format text` for consistent parsing
-  - Removed `jq` dependency from text output parsing (was causing failures in CI)
-  - Simplified parsing logic to use only text format for fixture validation tests
-  - JSON format tests already explicitly use `--format json` (no changes needed)
-  - **Root Cause:** Script was getting JSON output by default but trying to parse as text
-  - **Impact:** Tests now pass consistently in both local (macOS) and CI (Ubuntu) environments
+- **GitHub Actions** - Fixed test suite JSON parsing by installing `jq` dependency
+  - Added `jq` installation step to CI workflow: `sudo apt-get install -y jq`
+  - Test script uses JSON parsing as primary method (with text fallback)
+  - **Root Cause:** `jq` was missing in Ubuntu CI environment, causing JSON parsing to fail
+  - **Architecture:** Script defaults to JSON output → parses with `jq` → falls back to text if `jq` unavailable
+  - **Impact:** Tests now parse JSON correctly in CI environment (9/10 tests passing)
 
 ### Changed
 - **Documentation** - Enhanced `dist/TEMPLATES/README.md` with context and background

dist/PATTERN-LIBRARY.json

@@ -1,6 +1,6 @@
 {
   "version": "1.0.0",
-  "generated": "2026-01-10T04:42:33Z",
+  "generated": "2026-01-10T04:47:43Z",
   "summary": {
     "total_patterns": 29,
     "enabled": 29,

dist/PATTERN-LIBRARY.md

@@ -1,7 +1,7 @@
 # Pattern Library Registry
 
 **Auto-generated by Pattern Library Manager**
-**Last Updated:** 2026-01-10 04:42:33 UTC
+**Last Updated:** 2026-01-10 04:47:43 UTC
 
 ---
 
@@ -117,6 +117,6 @@
 
 ---
 
-**Generated:** 2026-01-10 04:42:33 UTC
+**Generated:** 2026-01-10 04:47:43 UTC
 **Version:** 1.0.0
 **Tool:** Pattern Library Manager

dist/tests/run-fixture-tests.sh

@@ -123,10 +123,9 @@ run_test() {
   tmp_output=$(mktemp)
 
   # Debug: Show command being run
-  echo -e "  ${BLUE}[DEBUG] Running: $BIN_DIR/check-performance.sh --format text --paths \"$fixture_file\" --no-log${NC}"
+  echo -e "  ${BLUE}[DEBUG] Running: $BIN_DIR/check-performance.sh --paths \"$fixture_file\" --no-log${NC}"
 
-  # Explicitly request text format for consistent parsing across all environments
-  "$BIN_DIR/check-performance.sh" --format text --paths "$fixture_file" --no-log > "$tmp_output" 2>&1 || true
+  "$BIN_DIR/check-performance.sh" --paths "$fixture_file" --no-log > "$tmp_output" 2>&1 || true
 
   # Strip ANSI color codes for parsing (using perl for reliability)
   local clean_output
@@ -137,14 +136,23 @@ run_test() {
   tail -20 "$tmp_output" | perl -pe 's/\e\[[0-9;]*m//g' | sed 's/^/    /'
   echo ""
 
-  # Extract counts from text output (no jq dependency)
+  # Extract counts from JSON output using jq
+  # Note: check-performance.sh defaults to JSON format, so we parse JSON
   local actual_errors
   local actual_warnings
 
-  # Parse text format output
-  actual_errors=$(echo "$clean_output" | grep -E "^[[:space:]]*Errors:" | grep -oE '[0-9]+' | head -1)
-  actual_warnings=$(echo "$clean_output" | grep -E "^[[:space:]]*Warnings:" | grep -oE '[0-9]+' | head -1)
-  echo -e "  ${BLUE}[DEBUG] Parsed text output${NC}"
+  # Try to parse as JSON first (default format)
+  if echo "$clean_output" | jq empty 2>/dev/null; then
+    # Valid JSON - extract from summary
+    actual_errors=$(echo "$clean_output" | jq -r '.summary.total_errors // 0' 2>/dev/null)
+    actual_warnings=$(echo "$clean_output" | jq -r '.summary.total_warnings // 0' 2>/dev/null)
+    echo -e "  ${BLUE}[DEBUG] Parsed JSON output${NC}"
+  else
+    # Fallback to text format parsing (legacy)
+    actual_errors=$(echo "$clean_output" | grep -E "^[[:space:]]*Errors:" | grep -oE '[0-9]+' | head -1)
+    actual_warnings=$(echo "$clean_output" | grep -E "^[[:space:]]*Warnings:" | grep -oE '[0-9]+' | head -1)
+    echo -e "  ${BLUE}[DEBUG] Parsed text output (fallback)${NC}"
+  fi
 
   # Default to 0 if not found
   actual_errors=${actual_errors:-0}