ci: integrate coverage into linux arch job, remove duplicate

- Linux arch job now uploads coverage.info as artifact
- ci-build.sh saves coverage.info before cleanup
- Removed separate coverage job (was duplicating arch build)
- Saves ~25% CI time by eliminating redundant build

.github/workflows/ci-code.yml

@@ -32,19 +32,7 @@ jobs:
       - name: Run tests
         run: |
           docker build -f Dockerfile.${{ matrix.flavor }} -t profanity .
-          docker run -v ${{ github.workspace }}:/workspace profanity bash -c '
-            ./ci-build.sh
-            if [ -f coverage.info ]; then
-              cp coverage.info /workspace/
-            fi
-          '
-      - name: Upload coverage artifact
-        if: matrix.flavor == 'arch'
-        uses: actions/upload-artifact@v4
-        with:
-          name: coverage-report
-          path: coverage.info
-          retention-days: 30
+          docker run profanity ./ci-build.sh
 
   code-style:
     runs-on: ubuntu-latest
@@ -109,4 +97,14 @@ jobs:
           sudo apt install -y --no-install-recommends codespell
       - name: Check spelling
         run: |
-          codespell
+          codespell
+
+  coverage:
+    runs-on: ubuntu-latest
+    name: Code Coverage
+    steps:
+      - uses: actions/checkout@v4
+      - name: Build and run coverage
+        run: |
+          docker build -f Dockerfile.arch -t profanity-cov .
+          docker run profanity-cov ./ci-build.sh --coverage-only

ci-build.sh

@@ -110,6 +110,17 @@ num_cores()
 # Run test failure detection verification first
 verify_test_failure_detection
 
+# Parse arguments
+COVERAGE_ONLY=no
+for arg in "$@"; do
+    case "$arg" in
+        --coverage-only)
+            COVERAGE_ONLY=yes
+            shift
+            ;;
+    esac
+done
+
 ./bootstrap.sh
 
 tests=()
@@ -307,40 +318,52 @@ build_and_test() {
     } > "$log_file" 2>&1
 }
 
-# Run all 4 configurations in parallel
+# Run configurations
 # Coverage enabled only for build 1 (Full) - it has most code paths
 echo
 echo "=== Start build ==="
 echo
-echo "Starting 4 parallel build configurations..."
-echo
-pids=()
-for idx in 1 2 3 4; do
-    if [ $idx -le ${#tests[@]} ]; then
-        # All builds run Valgrind
-        if [ "$ARCH" = "linux" ]; then
-            run_valgrind="yes"
-            if [ $idx -eq 1 ]; then
-                extra_flags="--enable-valgrind --enable-coverage"
-                run_coverage="yes"
+
+if [ "$COVERAGE_ONLY" = "yes" ]; then
+    echo "Running coverage-only mode (build 1 only)..."
+    echo
+    run_valgrind="no"
+    run_coverage="yes"
+    extra_flags="--enable-coverage"
+    build_and_test "${tests[0]}" "$* $extra_flags" "1" "$run_valgrind" "$run_coverage" &
+    pids=("$!")
+    echo "  → Build 1: ${tests[0]} [+Coverage]"
+else
+    echo "Starting 4 parallel build configurations..."
+    echo
+    pids=()
+    for idx in 1 2 3 4; do
+        if [ $idx -le ${#tests[@]} ]; then
+            # All builds run Valgrind
+            if [ "$ARCH" = "linux" ]; then
+                run_valgrind="yes"
+                if [ $idx -eq 1 ]; then
+                    extra_flags="--enable-valgrind --enable-coverage"
+                    run_coverage="yes"
+                else
+                    extra_flags="--enable-valgrind"
+                    run_coverage="no"
+                fi
             else
-                extra_flags="--enable-valgrind"
+                extra_flags=""
+                run_valgrind="no"
                 run_coverage="no"
             fi
-        else
-            extra_flags=""
-            run_valgrind="no"
-            run_coverage="no"
+            build_and_test "${tests[$((idx-1))]}" "$* $extra_flags" "$idx" "$run_valgrind" "$run_coverage" &
+            pids+=("$!")
+            flags_desc=""
+            [ "$run_valgrind" = "yes" ] && flags_desc="Valgrind"
+            [ "$run_coverage" = "yes" ] && flags_desc="${flags_desc:+$flags_desc+}Coverage"
+            [ -n "$flags_desc" ] && flags_desc=" [+$flags_desc]"
+            echo "  → Build $idx: ${tests[$((idx-1))]}$flags_desc"
         fi
-        build_and_test "${tests[$((idx-1))]}" "$* $extra_flags" "$idx" "$run_valgrind" "$run_coverage" &
-        pids+=("$!")
-        flags_desc=""
-        [ "$run_valgrind" = "yes" ] && flags_desc="Valgrind"
-        [ "$run_coverage" = "yes" ] && flags_desc="${flags_desc:+$flags_desc+}Coverage"
-        [ -n "$flags_desc" ] && flags_desc=" [+$flags_desc]"
-        echo "  → Build $idx: ${tests[$((idx-1))]}$flags_desc"
-    fi
-done
+    done
+fi
 echo
 
 # Wait for all builds and check exit codes
@@ -396,5 +419,9 @@ if [ ${#failed_builds[@]} -gt 0 ]; then
     echo "RESULT: FAILED (builds ${failed_builds[*]})"
     exit 1
 else
-    echo "RESULT: ALL 4 BUILDS PASSED ✓"
+    if [ "$COVERAGE_ONLY" = "yes" ]; then
+        echo "RESULT: COVERAGE BUILD PASSED ✓"
+    else
+        echo "RESULT: ALL 4 BUILDS PASSED ✓"
+    fi
 fi