opencode-sonarqube 2.0.1 → 2.0.3

package/dist/index.js

@@ -4787,7 +4787,10 @@ class SonarQubeClient {
     return this.request("/api/authentication/validate");
   }
   async getVersion() {
-    const response = await fetch(`${this.baseUrl}/api/server/version`);
+    const url2 = buildUrl(this.baseUrl, "/api/server/version");
+    const response = await fetch(url2, {
+      headers: { Authorization: buildAuthHeader(this.auth) }
+    });
     return response.text();
   }
   async healthCheck() {
@@ -4987,10 +4990,8 @@ async function loadProjectState(directory) {
 async function saveProjectState(directory, state) {
   const stateDir = getStateDir(directory);
   const statePath = getStatePath(directory);
-  const dirExists = await Bun.file(stateDir).exists();
-  if (!dirExists) {
-    await Bun.write(`${stateDir}/.gitkeep`, "");
-  }
+  const { mkdir } = await import("node:fs/promises");
+  await mkdir(stateDir, { recursive: true });
   const content = JSON.stringify(state, null, 2);
   await Bun.write(statePath, content);
   logger5.info("Saved project state", { projectKey: state.projectKey });
@@ -19600,32 +19601,37 @@ function formatAnalysisResult(result) {
 init_bootstrap();
 init_logger();
 var logger7 = new Logger("sonarqube-hooks");
-var editedFiles = new Set;
-var lastAnalysisTime = 0;
 var ANALYSIS_COOLDOWN_MS = 5 * 60 * 1000;
-var bootstrapInProgress = false;
+function createHookState() {
+  return {
+    editedFiles: new Set,
+    lastAnalysisTime: 0,
+    bootstrapInProgress: false
+  };
+}
+var activeState = createHookState();
 function isAnalysisEnabled(config3) {
   return config3 !== null && config3.level !== "off" && config3.autoAnalyze;
 }
-function isInCooldown() {
+function isInCooldown(state) {
   const now = Date.now();
-  return now - lastAnalysisTime < ANALYSIS_COOLDOWN_MS;
+  return now - state.lastAnalysisTime < ANALYSIS_COOLDOWN_MS;
 }
-async function handleBootstrap(config3, directory) {
-  if (bootstrapInProgress) {
+async function handleBootstrap(config3, directory, hookState) {
+  if (hookState.bootstrapInProgress) {
     logger7.debug("Bootstrap already in progress");
     return;
   }
   logger7.info("First run detected - initializing SonarQube integration");
-  bootstrapInProgress = true;
+  hookState.bootstrapInProgress = true;
   try {
     const result = await bootstrap({ config: config3, directory });
-    bootstrapInProgress = false;
+    hookState.bootstrapInProgress = false;
     return result.success ? formatBootstrapMessage(result) : `**SonarQube Setup Failed**
 
 ${result.message}`;
   } catch (error45) {
-    bootstrapInProgress = false;
+    hookState.bootstrapInProgress = false;
     logger7.error("Bootstrap failed", { error: String(error45) });
     const errorMsg = error45 instanceof Error ? error45.message : String(error45);
     return `**SonarQube Setup Error**
@@ -19633,7 +19639,7 @@ ${result.message}`;
 ${errorMsg}`;
   }
 }
-async function performAnalysis(config3, state, directory) {
+async function performAnalysis(config3, state, directory, hookState) {
   const projectKey = state.projectKey;
   const projectName = config3.projectName ?? projectKey;
   const result = await runAnalysis(config3, state, {
@@ -19642,7 +19648,9 @@ async function performAnalysis(config3, state, directory) {
     sources: config3.sources,
     tests: config3.tests
   }, directory);
-  editedFiles.clear();
+  if (result.success) {
+    hookState.editedFiles.clear();
+  }
   return formatAnalysisOutput(result, config3);
 }
 function formatAnalysisOutput(result, config3) {
@@ -19667,7 +19675,7 @@ function formatActionPrompt(result, _config) {
 
 **Action Required:** Found ${blockerCount} blocker(s) and ${criticalCount} critical issue(s). Please review and fix these issues before continuing.`;
 }
-function createIdleHook(getConfig, getDirectory) {
+function createIdleHook(getConfig, getDirectory, hookState = activeState) {
   return async function handleSessionIdle() {
     const rawConfig = getConfig()?.["sonarqube"];
     const config3 = loadConfig(rawConfig);
@@ -19676,25 +19684,25 @@ function createIdleHook(getConfig, getDirectory) {
     }
     const directory = getDirectory();
     if (await needsBootstrap(directory)) {
-      return handleBootstrap(config3, directory);
+      return handleBootstrap(config3, directory, hookState);
     }
-    if (isInCooldown()) {
+    if (isInCooldown(hookState)) {
       logger7.debug("Skipping auto-analysis (cooldown)");
       return;
     }
-    if (editedFiles.size === 0) {
+    if (hookState.editedFiles.size === 0) {
       logger7.debug("Skipping auto-analysis (no edited files)");
       return;
     }
     logger7.info("Session idle - triggering auto-analysis");
-    lastAnalysisTime = Date.now();
+    hookState.lastAnalysisTime = Date.now();
     try {
       const state = await getProjectState(directory);
       if (!state) {
         logger7.warn("No project state found, cannot run analysis");
         return;
       }
-      return await performAnalysis(config3, state, directory);
+      return await performAnalysis(config3, state, directory, hookState);
     } catch (error45) {
       logger7.error(`Auto-analysis failed: ${error45}`);
       return;
@@ -19732,22 +19740,26 @@ var IGNORED_FILE_PATTERNS = [
 function shouldIgnoreFile(filePath) {
   return IGNORED_FILE_PATTERNS.some((pattern) => pattern.test(filePath));
 }
-function createFileEditedHook() {
+function createFileEditedHook(hookState = activeState) {
   return function handleFileEdited(input) {
     const { filePath } = input;
     if (!shouldIgnoreFile(filePath)) {
-      editedFiles.add(filePath);
+      hookState.editedFiles.add(filePath);
+      activeState = hookState;
       logger7.debug(`Tracked edited file: ${filePath}`);
     }
   };
 }
 function getEditedFiles() {
-  return Array.from(editedFiles);
+  return Array.from(activeState.editedFiles);
 }
 function createHooks(getConfig, getDirectory) {
+  const hookState = createHookState();
+  activeState = hookState;
   return {
-    sessionIdle: createIdleHook(getConfig, getDirectory),
-    fileEdited: createFileEditedHook()
+    sessionIdle: createIdleHook(getConfig, getDirectory, hookState),
+    fileEdited: createFileEditedHook(hookState),
+    getEditedFiles: () => Array.from(hookState.editedFiles)
   };
 }
 
@@ -20495,7 +20507,8 @@ var SonarQubeToolArgsSchema = exports_external2.object({
 });
 async function executeSonarQubeTool(args, context) {
   const directory = context.directory ?? process.cwd();
-  const config3 = loadConfig(context.config);
+  const rawConfig = context.config?.["sonarqube"] ?? context.config;
+  const config3 = loadConfig(rawConfig);
   if (!config3) {
     return formatError2(ErrorMessages.configurationMissing("SonarQube configuration not found.", [
       "Set environment variables: SONAR_HOST_URL, SONAR_USER, and SONAR_PASSWORD",
@@ -20568,7 +20581,7 @@ ${setupResult.message}`);
 }
 
 // src/utils/shared-state.ts
-import { readFileSync, writeFileSync } from "node:fs";
+import { readFileSync, writeFileSync, renameSync } from "node:fs";
 var SHARED_STATE_FILE = "/tmp/sonarqube-plugin-shared-state.json";
 var readSharedState = () => {
   try {
@@ -20581,7 +20594,9 @@ var readSharedState = () => {
 var writeSharedState = (state) => {
   try {
     state.lastUpdated = new Date().toISOString();
-    writeFileSync(SHARED_STATE_FILE, JSON.stringify(state, null, 2));
+    const tmpFile = `${SHARED_STATE_FILE}.tmp.${process.pid}`;
+    writeFileSync(tmpFile, JSON.stringify(state, null, 2));
+    renameSync(tmpFile, SHARED_STATE_FILE);
   } catch {}
 };
 var mapSessionToDirectory = (sessionId, directory) => {
@@ -20843,6 +20858,10 @@ var SonarQubePlugin = async ({ client, directory, worktree }) => {
     safeLog("client.app.log failed (non-fatal)");
   }
   let pluginConfig;
+  let pluginConfigLoadedAt = 0;
+  const CONFIG_RELOAD_INTERVAL_MS = 60000;
+  const transformCacheMap = new Map;
+  const TRANSFORM_CACHE_TTL_MS = 60000;
   let lastAnalysisResult;
   const getConfig = () => pluginConfig;
   const getDirectory = () => {
@@ -20853,8 +20872,9 @@ var SonarQubePlugin = async ({ client, directory, worktree }) => {
     return effectiveDirectory;
   };
   const loadPluginConfig = async () => {
-    if (pluginConfig)
+    if (pluginConfig && Date.now() - pluginConfigLoadedAt < CONFIG_RELOAD_INTERVAL_MS)
       return;
+    pluginConfig = undefined;
     const dir = getDirectory();
     const sonarConfigPath = `${dir}/.sonarqube/config.json`;
     try {
@@ -20862,16 +20882,18 @@ var SonarQubePlugin = async ({ client, directory, worktree }) => {
       if (await configFile.exists()) {
         const config3 = await configFile.json();
         pluginConfig = { sonarqube: config3 };
+        pluginConfigLoadedAt = Date.now();
         safeLog(`Config loaded from ${sonarConfigPath}`);
         return;
       }
     } catch {}
     pluginConfig = {};
+    pluginConfigLoadedAt = Date.now();
     safeLog("Using environment variables for config");
   };
   const hooks = createHooks(getConfig, getDirectory);
   let currentSessionId;
-  let initialCheckDone = false;
+  const initialCheckSessions = new Set;
   const buildQualityNotification = (qgStatus, counts, qgFailed) => {
     const statusEmoji = qgFailed ? "[FAIL]" : "[WARN]";
     const blockerNote = counts.blocker > 0 ? `**Action Required:** There are BLOCKER issues that should be fixed.
@@ -20884,9 +20906,9 @@ var SonarQubePlugin = async ({ client, directory, worktree }) => {
 ${blockerNote}Use \`sonarqube({ action: "issues" })\` to see details or \`sonarqube({ action: "analyze" })\` to re-analyze.`;
   };
   const performInitialQualityCheck = async (sessionId) => {
-    if (initialCheckDone)
+    if (initialCheckSessions.has(sessionId))
       return;
-    initialCheckDone = true;
+    initialCheckSessions.add(sessionId);
     try {
       await loadPluginConfig();
       const sonarConfig = pluginConfig?.["sonarqube"];
@@ -20933,7 +20955,7 @@ ${blockerNote}Use \`sonarqube({ action: "issues" })\` to see details or \`sonarq
       });
     } catch {}
   };
-  const qualityGatePattern = /Quality Gate: \[(PASS|FAIL)\] (\w+)/;
+  const qualityGatePattern = /Quality Gate[:\s*]*\[(?:PASS|FAIL)\]\s*(\w+)|\*\*Quality Gate:\s*(\w+)\*\*/;
   const issueCountPattern = /Blockers: (\d+), Critical: (\d+), Major: (\d+), Minor: (\d+), Info: (\d+)/;
   const parseAnalysisResult = (message) => {
     const resultMatch = qualityGatePattern.exec(message);
@@ -20942,7 +20964,7 @@ ${blockerNote}Use \`sonarqube({ action: "issues" })\` to see details or \`sonarq
       return;
     }
     return {
-      qualityGate: resultMatch[2],
+      qualityGate: resultMatch[1] || resultMatch[2],
       issues: {
         blocker: Number.parseInt(issueMatch[1], 10),
         critical: Number.parseInt(issueMatch[2], 10),
@@ -20958,10 +20980,11 @@ ${blockerNote}Use \`sonarqube({ action: "issues" })\` to see details or \`sonarq
       return;
     }
     const payload = event.properties;
-    if (payload?.id) {
-      currentSessionId = payload.id;
+    const sessionId = payload?.info?.id;
+    if (sessionId) {
+      currentSessionId = sessionId;
       if (event.type === "session.created") {
-        mapSessionToDirectory(payload.id, effectiveDirectory);
+        mapSessionToDirectory(sessionId, effectiveDirectory);
       }
     }
   };
@@ -20970,8 +20993,8 @@ ${blockerNote}Use \`sonarqube({ action: "issues" })\` to see details or \`sonarq
       return;
     }
     const payload = event.properties;
-    if (payload?.path && !shouldIgnoreFile2(payload.path)) {
-      hooks.fileEdited({ filePath: payload.path });
+    if (payload?.file && !shouldIgnoreFile2(payload.file)) {
+      hooks.fileEdited({ filePath: payload.file });
     }
   };
   const injectAnalysisResults = async (message, _config, sessionId) => {
@@ -21001,8 +21024,8 @@ ${blockerNote}Use \`sonarqube({ action: "issues" })\` to see details or \`sonarq
     if (!config3 || config3.level === "off" || !config3.autoAnalyze) {
       return;
     }
-    const editedFiles2 = getEditedFiles();
-    if (editedFiles2.length === 0) {
+    const editedFiles = hooks.getEditedFiles?.() ?? getEditedFiles();
+    if (editedFiles.length === 0) {
       return;
     }
     const message = await hooks.sessionIdle();
@@ -21013,7 +21036,7 @@ ${blockerNote}Use \`sonarqube({ action: "issues" })\` to see details or \`sonarq
     if (parsedResult) {
       lastAnalysisResult = parsedResult;
     }
-    const passed = message.includes("[PASS]");
+    const passed = message.includes("[PASS]") || message.includes("Quality Gate") && message.includes("OK");
     await showToast(passed ? "SonarQube: Quality Gate Passed" : "SonarQube: Issues Found", passed ? "success" : "error");
     await injectAnalysisResults(message, config3, currentSessionId);
   };
@@ -21112,7 +21135,7 @@ Git operation completed with changes. Consider running:
   const handleGitPush = async (command, outputData) => {
     if (!/git\s+push\b/.test(command))
       return;
-    if (outputData?.includes("error") || outputData?.includes("rejected"))
+    if (outputData?.includes("error:") || outputData?.includes("fatal:") || outputData?.includes("rejected"))
       return;
     await showToast("Code pushed - SonarQube will analyze on server", "info");
   };
@@ -21171,16 +21194,27 @@ Git operation completed with changes. Consider running:
       safeLog(`  effectiveDirectory (fallback): "${effectiveDirectory}"`);
       const dir = sessionDir || effectiveDirectory;
       safeLog(`  FINAL dir used: "${dir}"`);
+      const now = Date.now();
+      const cachedEntry = transformCacheMap.get(dir);
+      if (cachedEntry && now - cachedEntry.timestamp < TRANSFORM_CACHE_TTL_MS) {
+        if (cachedEntry.data) {
+          output.system.push(cachedEntry.data);
+        }
+        return;
+      }
       await loadPluginConfig();
       const sonarConfig = pluginConfig?.["sonarqube"];
       const config3 = loadConfig(sonarConfig);
       if (!config3 || config3.level === "off") {
         safeLog(`  config level is off or null, returning early`);
+        transformCacheMap.set(dir, { data: undefined, timestamp: now });
         return;
       }
       const state = await getProjectState(dir);
-      if (!state?.projectKey)
+      if (!state?.projectKey) {
+        transformCacheMap.set(dir, { data: undefined, timestamp: now });
         return;
+      }
       const api2 = createSonarQubeAPI(config3, state);
       const [qgStatus, counts, newCodeResponse] = await Promise.all([
         api2.qualityGate.getStatus(state.projectKey),
@@ -21196,6 +21230,7 @@ Git operation completed with changes. Consider running:
       const qgFailed = qgStatus.projectStatus.status !== "OK";
       const newCodeIssues = newCodeResponse.paging.total;
       if (!hasIssues && !qgFailed && newCodeIssues === 0) {
+        transformCacheMap.set(dir, { data: undefined, timestamp: now });
         return;
       }
       const systemContext = `## SonarQube Code Quality Status
@@ -21212,6 +21247,7 @@ ${config3.level === "enterprise" ? `This project follows enterprise-level qualit
 - \`sonarqube({ action: "newissues" })\` - See issues in your recent changes (Clean as You Code)
 - \`sonarqube({ action: "worstfiles" })\` - Find files needing most attention
 - \`sonarqube({ action: "issues" })\` - See all issues`;
+      transformCacheMap.set(dir, { data: systemContext, timestamp: now });
       output.system.push(systemContext);
     }, "experimental.chat.system.transform"),
     tool: {
@@ -21270,6 +21306,7 @@ Example usage:
             if (parsedResult) {
               lastAnalysisResult = parsedResult;
             }
+            transformCacheMap.clear();
           }
           return result;
         }

package/package.json

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