Remove JQ

Author: noelsaw1

CHANGELOG.md

@@ -25,6 +25,14 @@ 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
+
 ### Changed
 - **Documentation** - Enhanced `dist/TEMPLATES/README.md` with context and background
   - Added "What Are Templates?" section explaining the concept and purpose

dist/PATTERN-LIBRARY.json

@@ -1,6 +1,6 @@
 {
   "version": "1.0.0",
-  "generated": "2026-01-10T04:15:12Z",
+  "generated": "2026-01-10T04:42:33Z",
   "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:15:12 UTC
+**Last Updated:** 2026-01-10 04:42:33 UTC
 
 ---
 
@@ -117,6 +117,6 @@
 
 ---
 
-**Generated:** 2026-01-10 04:15:12 UTC
+**Generated:** 2026-01-10 04:42:33 UTC
 **Version:** 1.0.0
 **Tool:** Pattern Library Manager

dist/tests/run-fixture-tests.sh

@@ -123,9 +123,10 @@ run_test() {
   tmp_output=$(mktemp)
 
   # Debug: Show command being run
-  echo -e "  ${BLUE}[DEBUG] Running: $BIN_DIR/check-performance.sh --paths \"$fixture_file\" --no-log${NC}"
+  echo -e "  ${BLUE}[DEBUG] Running: $BIN_DIR/check-performance.sh --format text --paths \"$fixture_file\" --no-log${NC}"
 
-  "$BIN_DIR/check-performance.sh" --paths "$fixture_file" --no-log > "$tmp_output" 2>&1 || true
+  # 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
 
   # Strip ANSI color codes for parsing (using perl for reliability)
   local clean_output
@@ -136,23 +137,14 @@ run_test() {
   tail -20 "$tmp_output" | perl -pe 's/\e\[[0-9;]*m//g' | sed 's/^/    /'
   echo ""
 
-  # Extract counts from JSON output using jq
-  # Note: check-performance.sh defaults to JSON format, so we parse JSON
+  # Extract counts from text output (no jq dependency)
   local actual_errors
   local actual_warnings
 
-  # 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
+  # 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}"
 
   # Default to 0 if not found
   actual_errors=${actual_errors:-0}