fix: initialize all loop variables to prevent unbound variable errors in strict mode

Author: Chemaclass

src/benchmark.sh

@@ -126,7 +126,7 @@ function bashunit::benchmark::print_results() {
     printf '%-40s %6s %6s %10s\n' "Name" "Revs" "Its" "Avg(ms)"
   fi
 
-  local i
+  local i=0
   for i in "${!_BASHUNIT_BENCH_NAMES[@]}"; do
     local name="${_BASHUNIT_BENCH_NAMES[$i]}"
     local revs="${_BASHUNIT_BENCH_REVS[$i]}"

src/colors.sh

@@ -10,7 +10,7 @@ bashunit::sgr() {
   local codes=${1:-0}
   shift
 
-  local c
+  local c=""
   for c in "$@"; do
     codes="$codes;$c"
   done

src/console_results.sh

@@ -351,7 +351,7 @@ function bashunit::console_results::print_error_test() {
 
   if [[ -n "$raw_output" ]] && bashunit::env::is_show_output_on_failure_enabled; then
     line="$line$(printf "    %sOutput:%s\n" "${_BASHUNIT_COLOR_FAINT}" "${_BASHUNIT_COLOR_DEFAULT}")"
-    local output_line
+    local output_line=""
     while IFS= read -r output_line; do
       line="$line$(printf "      %s\n" "$output_line")"
     done <<< "$raw_output"

src/coverage.sh

@@ -416,7 +416,7 @@ function bashunit::coverage::get_hit_lines() {
   # Only count hits that correspond to executable lines
   # This prevents >100% coverage when DEBUG trap fires on non-executable lines
   local count=0
-  local line_num
+  local line_num=0
   for line_num in $hit_lines; do
     local line_content
     line_content=$(sed -n "${line_num}p" "$file" 2>/dev/null) || continue
@@ -437,7 +437,7 @@ function bashunit::coverage::get_line_hits() {
     return
   fi
 
-  local count
+  local count=0
   count=$(grep -c "^${file}:${lineno}$" "$_BASHUNIT_COVERAGE_DATA_FILE" 2>/dev/null) || count=0
   echo "$count"
 }
@@ -562,7 +562,7 @@ function bashunit::coverage::get_function_coverage() {
 
   local executable=0
   local hit=0
-  local lineno
+  local lineno=0
 
   for ((lineno = fn_start; lineno <= fn_end; lineno++)); do
     local line_content
@@ -616,7 +616,7 @@ function bashunit::coverage::report_text() {
   echo "Coverage Report"
   echo "---------------"
 
-  local file
+  local file=""
   while IFS= read -r file; do
     [[ -z "$file" || ! -f "$file" ]] && continue
     has_files=true
@@ -773,7 +773,7 @@ function bashunit::coverage::report_html() {
   local total_hit=0
   local -a file_data=()
   local file_data_count=0
-  local file
+  local file=""
 
   while IFS= read -r file; do
     [[ -z "$file" || ! -f "$file" ]] && continue
@@ -787,7 +787,7 @@ function bashunit::coverage::report_html() {
     total_hit=$((total_hit + hit))
 
     local display_file="${file#"$(pwd)"/}"
-    local safe_filename
+    local safe_filename=""
     safe_filename=$(bashunit::coverage::path_to_filename "$file")
 
     file_data[file_data_count]="$display_file|$hit|$executable|$pct|$safe_filename"; file_data_count=$((file_data_count + 1))
@@ -1192,7 +1192,7 @@ function bashunit::coverage::generate_file_html() {
   # Index: line number, Value: newline-separated list of "test_file:test_function"
   # Using indexed array for Bash 3.0 compatibility (no associative arrays)
   local -a tests_by_line=()
-  local _line_and_test
+  local _line_and_test=""
   while IFS= read -r _line_and_test; do
     [[ -z "$_line_and_test" ]] && continue
     local _tln="${_line_and_test%%|*}"

src/doc.sh

@@ -11,14 +11,14 @@ function bashunit::doc::get_embedded_docs() {
 
 function bashunit::doc::print_asserts() {
   local filter="${1:-}"
-  local line
+  local line=""
   local docstring=""
   local fn=""
   local should_print=0
 
   # Pattern stored in variable for Bash 3.0 compatibility
   local _doc_pattern='^## ([A-Za-z0-9_]+)'
-  local line
+  local line=""
   while IFS='' read -r line || [[ -n "$line" ]]; do
     if [[ $line =~ $_doc_pattern ]]; then
       fn="${BASH_REMATCH[1]}"

src/env.sh

@@ -244,7 +244,7 @@ function bashunit::env::print_verbose() {
 
   local max_length=0
 
-  local key
+  local key=""
   for key in "${keys[@]}"; do
     if (( ${#key} > max_length )); then
       max_length=${#key}

src/globals.sh

@@ -31,7 +31,7 @@ function bashunit::random_str() {
   local length=${1:-6}
   local chars='abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789'
   local str=''
-  local i
+  local i=0
   for (( i=0; i<length; i++ )); do
     str="$str${chars:RANDOM%${#chars}:1}"
   done

src/helpers.sh

@@ -27,7 +27,7 @@ function bashunit::regex_match() {
 #
 function bashunit::helper::find_test_function_name() {
   local fallback_depth="${1:-2}"
-  local i
+  local i=0
   for ((i = 0; i < ${#FUNCNAME[@]}; i++)); do
     local fn="${FUNCNAME[$i]}"
     # Check if function starts with "test_" or "test" followed by uppercase
@@ -112,7 +112,7 @@ function bashunit::helper::interpolate_function_name() {
   [[ $# -gt 0 ]] && args=("$@")
   local result="$function_name"
 
-  local i
+  local i=0
   for ((i=0; i<args_count; i++)); do
     local placeholder="::$((i+1))::"
     # shellcheck disable=SC2155

src/learn.sh

@@ -123,7 +123,7 @@ function bashunit::learn::show_progress() {
   local total_lessons=10
   local completed=0
 
-  local i
+  local i=0
   for i in $(seq 1 $total_lessons); do
     if bashunit::learn::is_completed "lesson_$i"; then
       echo "  ${_BASHUNIT_COLOR_PASSED}✓${_BASHUNIT_COLOR_DEFAULT} Lesson $i completed"

src/main.sh

@@ -179,7 +179,7 @@ function bashunit::main::cmd_test() {
       args_count="$raw_args_count"
     else
       # Test mode: process file paths and extract inline filters
-      local arg
+      local arg=""
       for arg in "${raw_args[@]}"; do
         local parsed_path parsed_filter
         {
@@ -193,7 +193,7 @@ function bashunit::main::cmd_test() {
           inline_filter_file="$parsed_path"
         fi
 
-        local file
+        local file=""
         while IFS= read -r file; do
           args[args_count]="$file"; args_count=$((args_count + 1))
         done < <(bashunit::helper::find_files_recursive "$parsed_path" '*[tT]est.sh')