From fec964ae2b4ec9ca5a04e8bbf24cef1df353f0cb Mon Sep 17 00:00:00 2001
From: Theauditor <228822721+TheAuditorTool@users.noreply.github.com>
Date: Mon, 13 Apr 2026 14:36:10 +0700
Subject: [PATCH] fix(scripts): remove dead sonar-report dep and re-enable
 container cleanup

Closes #187. The jq ARG_MAX overflow was fixed in 32933c4e6 by replacing
shell-based result aggregation with SonarReport.java. This commit removes
leftover artifacts: the sonar-report tool check that blocks script execution
despite no longer being used, and uncomments docker stop so the SonarQube
container is properly cleaned up after a run.
---
 scripts/runSonarQube.sh | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/scripts/runSonarQube.sh b/scripts/runSonarQube.sh
index 761294d373..7666841029 100755
--- a/scripts/runSonarQube.sh
+++ b/scripts/runSonarQube.sh
@@ -13,11 +13,6 @@ requireCommand curl
 requireCommand docker
 requireCommand jq
 
-if ! command -v "sonar-report" &> /dev/null; then
-  echo "sonar-report is required. Please install it via https://github.com/soprasteria/sonar-report and then try again."
-  exit 1
-fi
-
 # Check for install/updates at https://github.com/SonarSource/sonarqube
 
 container_name="sonarqube-benchmark"
@@ -91,4 +86,4 @@ mvn exec:java -Dexec.mainClass="org.owasp.benchmark.report.sonarqube.SonarReport
 
 echo "Shutting down SonarQube..."
 
-#docker stop "$container_name"
+docker stop "$container_name"