ci: fix script bugs found during fork testing

- Fix empty associative array crash under set -u in checkManualItTests()
  by using a counter variable instead of ${#it_sources[@]}
- Fix duplicate modules in CI comment by stripping [packaging] suffix
  from Maven reactor output before deduplication

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: gnodet

.github/actions/incremental-build/incremental-build.sh

@@ -228,6 +228,7 @@ checkManualItTests() {
 
   declare -A it_commands
   declare -A it_sources
+  local it_found=0
 
   while IFS=: read -r source_id it_module command; do
     # Skip comments and empty lines
@@ -241,11 +242,12 @@ checkManualItTests() {
       if [[ "$(basename "$module_path")" == "$source_id" ]]; then
         it_commands["$it_module"]="$command"
         it_sources["$it_module"]="${it_sources[$it_module]:-}${it_sources[$it_module]:+, }\`${module_path}\`"
+        it_found=1
       fi
     done
   done < "$mapping_file"
 
-  if [[ ${#it_sources[@]} -gt 0 ]]; then
+  if [[ "$it_found" -eq 1 ]]; then
     echo "" >> "$comment_file"
     echo ":bulb: **Manual integration tests recommended:**" >> "$comment_file"
     for it_module in "${!it_sources[@]}"; do
@@ -561,7 +563,7 @@ main() {
   # Append reactor module list from build log
   if [[ -f "$log" ]]; then
     local reactor_modules
-    reactor_modules=$(grep '^\[INFO\] Camel ::' "$log" | sed 's/\[INFO\] //' | sed 's/ \..*$//' | sort -u || true)
+    reactor_modules=$(grep '^\[INFO\] Camel ::' "$log" | sed 's/\[INFO\] //' | sed 's/ \..*$//' | sed 's/  *\[.*\]$//' | sort -u || true)
     if [[ -n "$reactor_modules" ]]; then
       local count
       count=$(echo "$reactor_modules" | wc -l | tr -d ' ')