opencode-sonarqube 1.2.7 → 1.2.8

package/dist/index.js

@@ -20575,68 +20575,50 @@ Fix these issues before ${operationType === "commit" ? "committing" : "pushing"}
     await showToast(`SonarQube: ${operationType} BLOCKED`, "error");
     return { block: true, message };
   };
+  const isGitCommit = (cmd) => /git\s+commit\b/.test(cmd) && !/--amend/.test(cmd);
+  const isGitPush = (cmd) => /git\s+push\b/.test(cmd);
+  const processAnalysisResult = async (analysisResult, blockingSeverity, shouldBlock, fixBeforeCommit, autoFix) => {
+    const passedCheck = analysisResult.qualityGateStatus === "OK" || !checkBlockingIssues(analysisResult.issues, blockingSeverity);
+    if (passedCheck) {
+      await showToast("SonarQube: Quality check passed!", "success");
+      return { block: false };
+    }
+    if (fixBeforeCommit && autoFix) {
+      await sendAutoFixPrompt(analysisResult);
+      return { block: shouldBlock, message: "SonarQube is fixing issues. Please wait and try again." };
+    }
+    const warningMessage = await sendBlockingMessage(analysisResult.issues, shouldBlock, autoFix);
+    return { block: shouldBlock, message: warningMessage };
+  };
   const handleGitOperationCheck = async (output, operationType) => {
-    safeLog(`[handleGitOperationCheck] ENTERED operationType=${operationType}`);
     const args = output.args;
     const command = args?.command ?? "";
-    safeLog(`[handleGitOperationCheck] command=${command.slice(0, 80)}...`);
-    const isCommit = /git\s+commit\b/.test(command) && !/--amend/.test(command);
-    const isPush = /git\s+push\b/.test(command);
-    const isMatchingOperation = operationType === "commit" && isCommit || operationType === "push" && isPush;
-    if (!isMatchingOperation) {
-      safeLog(`[handleGitOperationCheck] NOT matching operation, returning block=false`);
+    const isMatch = operationType === "commit" ? isGitCommit(command) : isGitPush(command);
+    if (!isMatch)
       return { block: false };
-    }
     await loadPluginConfig();
     const sonarConfig = pluginConfig?.["sonarqube"];
     const config2 = loadConfig(sonarConfig);
-    safeLog(`[handleGitOperationCheck] config loaded: level=${config2?.level}, blockCommit=${config2?.blockCommit}`);
-    if (!config2 || config2.level === "off") {
-      safeLog(`[handleGitOperationCheck] config off or missing, returning block=false`);
+    safeLog(`[handleGitOperationCheck] ${operationType}: level=${config2?.level}`);
+    if (!config2 || config2.level === "off")
       return { block: false };
-    }
     const { analyzeBeforeCommit = true, blockCommit = false, blockPush = false } = config2;
     const { fixBeforeCommit = false, blockingSeverity = "CRITICAL", autoFix = false } = config2;
-    const shouldBlock = operationType === "commit" && blockCommit || operationType === "push" && blockPush;
-    safeLog(`[handleGitOperationCheck] shouldBlock=${shouldBlock}, blockCommit=${blockCommit}, blockingSeverity=${blockingSeverity}`);
+    const shouldBlock = operationType === "commit" ? blockCommit : blockPush;
     try {
       const dir = getDirectory();
-      safeLog(`[handleGitOperationCheck] directory=${dir}`);
       const state = await getProjectState(dir);
-      safeLog(`[handleGitOperationCheck] projectKey=${state?.projectKey ?? "NONE"}`);
-      if (!state?.projectKey) {
-        safeLog(`[handleGitOperationCheck] No projectKey, returning block=false`);
+      safeLog(`[handleGitOperationCheck] dir=${dir}, projectKey=${state?.projectKey ?? "NONE"}`);
+      if (!state?.projectKey)
         return { block: false };
-      }
       const api2 = createSonarQubeAPI(config2, state);
       if (operationType === "push" || !analyzeBeforeCommit) {
-        safeLog(`[handleGitOperationCheck] Checking existing issues (push or no analyzeBeforeCommit)`);
         return checkExistingIssuesAndBlock(api2, state.projectKey, operationType, blockingSeverity, shouldBlock);
       }
-      safeLog(`[handleGitOperationCheck] Running pre-commit analysis...`);
       await showToast("SonarQube: Running pre-commit analysis...", "info");
       const analysisResult = await runAnalysis(config2, state, { projectKey: state.projectKey }, dir);
-      safeLog(`[handleGitOperationCheck] analysisResult: qualityGate=${analysisResult.qualityGateStatus}, issues=${JSON.stringify(analysisResult.issues)}`);
-      if (analysisResult.qualityGateStatus === "OK") {
-        safeLog(`[handleGitOperationCheck] Quality gate OK, returning block=false`);
-        await showToast("SonarQube: Quality check passed!", "success");
-        return { block: false };
-      }
-      const hasBlockingIssues = checkBlockingIssues(analysisResult.issues, blockingSeverity);
-      safeLog(`[handleGitOperationCheck] hasBlockingIssues=${hasBlockingIssues}`);
-      if (!hasBlockingIssues) {
-        safeLog(`[handleGitOperationCheck] No blocking issues, returning block=false`);
-        await showToast("SonarQube: Quality check passed!", "success");
-        return { block: false };
-      }
-      if (fixBeforeCommit && autoFix) {
-        safeLog(`[handleGitOperationCheck] Auto-fixing issues...`);
-        await sendAutoFixPrompt(analysisResult);
-        return { block: shouldBlock, message: "SonarQube is fixing issues. Please wait and try again." };
-      }
-      safeLog(`[handleGitOperationCheck] BLOCKING! shouldBlock=${shouldBlock}`);
-      const warningMessage = await sendBlockingMessage(analysisResult.issues, shouldBlock, autoFix);
-      return { block: shouldBlock, message: warningMessage };
+      safeLog(`[handleGitOperationCheck] qualityGate=${analysisResult.qualityGateStatus}`);
+      return processAnalysisResult(analysisResult, blockingSeverity, shouldBlock, fixBeforeCommit, autoFix);
     } catch (err) {
       safeLog(`[handleGitOperationCheck] ERROR: ${err instanceof Error ? err.message : String(err)}`);
       return { block: false };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-sonarqube",
-  "version": "1.2.7",
+  "version": "1.2.8",
   "description": "OpenCode Plugin for SonarQube integration - Enterprise-level code quality from the start",
   "type": "module",
   "main": "dist/index.js",