opencode-sonarqube 1.3.0 → 1.4.0

package/dist/index.js

@@ -3999,47 +3999,32 @@ function numberToRating(num) {
   const ratings = ["A", "B", "C", "D", "E"];
   return ratings[num - 1] ?? "?";
 }
-function SonarQubeError(message, code, statusCode) {
+function createSonarQubeError(name, message, code, statusCode) {
   const error45 = new Error(message);
-  error45.name = "SonarQubeError";
+  error45.name = name;
   error45.code = code;
   error45.statusCode = statusCode;
   return error45;
 }
+function SonarQubeError(message, code, statusCode) {
+  return createSonarQubeError("SonarQubeError", message, code, statusCode);
+}
 function ConnectionError(message) {
-  const error45 = new Error(message);
-  error45.name = "ConnectionError";
-  error45.code = "CONNECTION_ERROR";
-  return error45;
+  return createSonarQubeError("ConnectionError", message, "CONNECTION_ERROR");
 }
 function AuthenticationError(message) {
-  const error45 = new Error(message);
-  error45.name = "AuthenticationError";
-  error45.code = "AUTH_ERROR";
-  error45.statusCode = 401;
-  return error45;
+  return createSonarQubeError("AuthenticationError", message, "AUTH_ERROR", 401);
 }
 function ProjectNotFoundError(projectKey) {
-  const error45 = new Error(`Project '${projectKey}' not found`);
-  error45.name = "ProjectNotFoundError";
-  error45.code = "PROJECT_NOT_FOUND";
-  error45.statusCode = 404;
-  return error45;
+  return createSonarQubeError("ProjectNotFoundError", `Project '${projectKey}' not found`, "PROJECT_NOT_FOUND", 404);
 }
 function RateLimitError(retryAfter) {
   const hasRetryAfter = retryAfter !== undefined && retryAfter > 0;
   const retryMessage = hasRetryAfter ? `. Retry after ${retryAfter}s` : "";
-  const error45 = new Error(`Rate limit exceeded${retryMessage}`);
-  error45.name = "RateLimitError";
-  error45.code = "RATE_LIMIT";
-  error45.statusCode = 429;
-  return error45;
+  return createSonarQubeError("RateLimitError", `Rate limit exceeded${retryMessage}`, "RATE_LIMIT", 429);
 }
 function SetupError(message) {
-  const error45 = new Error(message);
-  error45.name = "SetupError";
-  error45.code = "SETUP_ERROR";
-  return error45;
+  return createSonarQubeError("SetupError", message, "SETUP_ERROR");
 }
 var SonarQubeConfigSchema, ProjectStateSchema, ENTERPRISE_THRESHOLDS, STANDARD_THRESHOLDS, RELAXED_THRESHOLDS;
 var init_types2 = __esm(() => {
@@ -4409,31 +4394,14 @@ async function parsePackageJson(directory) {
 }
 function detectLanguages(checks3) {
   const languages = [];
-  if (checks3.hasPackageJson) {
-    if (checks3.hasTsConfig) {
-      languages.push("typescript", "javascript");
-    } else {
-      languages.push("javascript");
+  for (const detector of LANGUAGE_DETECTORS) {
+    if (detector.check(checks3)) {
+      languages.push(detector.name);
+      if (detector.includes) {
+        languages.push(...detector.includes);
+      }
     }
   }
-  if (checks3.hasRequirements || checks3.hasPyproject) {
-    languages.push("python");
-  }
-  if (checks3.hasPom || checks3.hasGradle) {
-    languages.push("java");
-  }
-  if (checks3.hasGoMod) {
-    languages.push("go");
-  }
-  if (checks3.hasCargo) {
-    languages.push("rust");
-  }
-  if (checks3.hasComposer) {
-    languages.push("php");
-  }
-  if (checks3.hasGemfile) {
-    languages.push("ruby");
-  }
   return languages.length > 0 ? languages : ["generic"];
 }
 function detectSourceDirectories(directory) {
@@ -4470,11 +4438,25 @@ async function detectProjectType(directory) {
   });
   return result;
 }
-var logger2;
+var logger2, LANGUAGE_DETECTORS;
 var init_detection = __esm(() => {
   init_logger();
   init_patterns();
   logger2 = new Logger("scanner-config");
+  LANGUAGE_DETECTORS = [
+    {
+      name: "typescript",
+      check: (c) => c.hasPackageJson && c.hasTsConfig,
+      includes: ["javascript"]
+    },
+    { name: "javascript", check: (c) => c.hasPackageJson && !c.hasTsConfig },
+    { name: "python", check: (c) => c.hasRequirements || c.hasPyproject },
+    { name: "java", check: (c) => c.hasPom || c.hasGradle },
+    { name: "go", check: (c) => c.hasGoMod },
+    { name: "rust", check: (c) => c.hasCargo },
+    { name: "php", check: (c) => c.hasComposer },
+    { name: "ruby", check: (c) => c.hasGemfile }
+  ];
 });
 
 // src/scanner/config/properties.ts
@@ -4573,6 +4555,21 @@ class BaseAPI {
     this.client = client;
     this.logger = logger4 ?? new Logger(serviceName);
   }
+  validateProjectKey(projectKey, methodName) {
+    if (!projectKey) {
+      this.logger.error(`${methodName}: projectKey is empty/undefined!`);
+      return false;
+    }
+    return true;
+  }
+  async safeRequest(request, fallback, context) {
+    try {
+      return await request();
+    } catch (error45) {
+      this.logger.warn(`${context}: ${error45 instanceof Error ? error45.message : String(error45)}`);
+      return fallback;
+    }
+  }
 }
 var init_base_api = __esm(() => {
   init_logger();
@@ -17553,9 +17550,39 @@ init_base_api();
 init_client();
 init_projects();
 
+// src/utils/paths.ts
+function extractFilePathFromComponentKey(componentKey) {
+  const colonIndex = componentKey.indexOf(":");
+  if (colonIndex >= 0) {
+    return componentKey.substring(colonIndex + 1);
+  }
+  return componentKey;
+}
+function shortenPath(path, maxLength = 50) {
+  if (path.length <= maxLength)
+    return path;
+  return "..." + path.slice(-(maxLength - 3));
+}
+
 // src/api/issues.ts
 init_base_api();
 
+// src/constants.ts
+var API = {
+  DEFAULT_PAGE_SIZE: 500,
+  DEFAULT_LIMIT: 20,
+  MAX_PAGES: 10,
+  POLL_INTERVAL_MS: 2000,
+  ANALYSIS_TIMEOUT_MS: 300000,
+  SOURCE_CONTEXT_LINES: 5
+};
+var RATINGS = ["A", "B", "C", "D", "E"];
+function numericToRating(value) {
+  const index = Math.min(Math.max(Math.round(value) - 1, 0), 4);
+  return RATINGS[index];
+}
+
+// src/api/issues.ts
 class IssuesAPI extends BaseAPI {
   constructor(client, logger4) {
     super(client, "sonarqube-issues", logger4);
@@ -17570,7 +17597,7 @@ class IssuesAPI extends BaseAPI {
       resolved: options.resolved === undefined ? undefined : String(options.resolved),
       branch: options.onBranch,
       inNewCodePeriod: options.inNewCode === undefined ? undefined : String(options.inNewCode),
-      ps: options.pageSize ?? 500,
+      ps: options.pageSize ?? API.DEFAULT_PAGE_SIZE,
       p: options.page ?? 1
     });
   }
@@ -17585,7 +17612,7 @@ class IssuesAPI extends BaseAPI {
         types: options.types,
         resolved: false,
         page,
-        pageSize: 500
+        pageSize: API.DEFAULT_PAGE_SIZE
       });
       allIssues.push(...response.issues);
       const totalPages = Math.ceil(response.paging.total / response.paging.pageSize);
@@ -17621,11 +17648,7 @@ class IssuesAPI extends BaseAPI {
     }
     return issues.map((issue2) => {
       const componentPath = componentPaths.get(issue2.component);
-      let filePath = componentPath ?? issue2.component;
-      const colonIndex = filePath.indexOf(":");
-      if (colonIndex !== -1) {
-        filePath = filePath.substring(colonIndex + 1);
-      }
+      const filePath = extractFilePathFromComponentKey(componentPath ?? issue2.component);
       return {
         key: issue2.key,
         severity: issue2.severity,
@@ -17685,7 +17708,7 @@ class IssuesAPI extends BaseAPI {
     try {
       const response = await this.client.get("/api/hotspots/search", {
         projectKey,
-        ps: 500,
+        ps: API.DEFAULT_PAGE_SIZE,
         p: 1
       });
       this.logger.info(`Found ${response.hotspots.length} security hotspots for project ${projectKey}`);
@@ -18045,7 +18068,6 @@ class QualityGateAPI extends BaseAPI {
 }
 // src/api/rules.ts
 init_base_api();
-
 class RulesAPI extends BaseAPI {
   cache = new Map;
   constructor(client, logger4) {
@@ -18107,7 +18129,7 @@ class RulesAPI extends BaseAPI {
       try {
         const response = await this.client.get("/api/rules/search", {
           rule_key: uncached.join(","),
-          ps: 500
+          ps: API.DEFAULT_PAGE_SIZE
         });
         for (const rule of response.rules) {
           const details = this.parseRuleResponse(rule);
@@ -18148,6 +18170,32 @@ class RulesAPI extends BaseAPI {
 // src/api/sources.ts
 init_base_api();
 
+// src/utils/group-by.ts
+function groupBy(items, keyFn) {
+  const groups = {};
+  for (const item of items) {
+    const key = keyFn(item);
+    if (!groups[key]) {
+      groups[key] = [];
+    }
+    groups[key].push(item);
+  }
+  return groups;
+}
+
+// src/api/sources.ts
+function createIssueWithContext(issue2, sourceContext) {
+  return {
+    key: issue2.key,
+    rule: issue2.rule,
+    severity: issue2.severity,
+    message: issue2.message,
+    component: issue2.component,
+    line: issue2.line,
+    sourceContext
+  };
+}
+
 class SourcesAPI extends BaseAPI {
   constructor(client, logger4) {
     super(client, "sonarqube-sources", logger4);
@@ -18181,20 +18229,12 @@ class SourcesAPI extends BaseAPI {
     };
   }
   groupIssuesByComponent(issues) {
-    const grouped = new Map;
-    for (const issue2 of issues) {
-      const existing = grouped.get(issue2.component) ?? [];
-      existing.push(issue2);
-      grouped.set(issue2.component, existing);
-    }
-    return grouped;
+    const grouped = groupBy(issues, (issue2) => issue2.component);
+    return new Map(Object.entries(grouped));
   }
-  createIssueWithContext(issue2, component, lineMap, contextLines) {
-    if (issue2.line === undefined) {
-      return { ...issue2, sourceContext: undefined };
-    }
-    const contextStart = Math.max(1, issue2.line - contextLines);
-    const contextEnd = issue2.line + contextLines;
+  buildSourceContext(component, issueLine, lineMap, contextLines) {
+    const contextStart = Math.max(1, issueLine - contextLines);
+    const contextEnd = issueLine + contextLines;
     const contextSourceLines = [];
     for (let l = contextStart;l <= contextEnd; l++) {
       const sourceLine = lineMap.get(l);
@@ -18203,20 +18243,21 @@ class SourcesAPI extends BaseAPI {
       }
     }
     return {
-      ...issue2,
-      sourceContext: {
-        component,
-        lines: contextSourceLines,
-        startLine: contextStart,
-        endLine: contextEnd
-      }
+      component,
+      lines: contextSourceLines,
+      startLine: contextStart,
+      endLine: contextEnd
     };
   }
+  createIssueWithSourceContext(issue2, lineMap, contextLines) {
+    if (issue2.line === undefined) {
+      return createIssueWithContext(issue2);
+    }
+    const sourceContext = this.buildSourceContext(issue2.component, issue2.line, lineMap, contextLines);
+    return createIssueWithContext(issue2, sourceContext);
+  }
   addIssuesWithoutContext(issues) {
-    return issues.map((issue2) => ({
-      ...issue2,
-      sourceContext: undefined
-    }));
+    return issues.map((issue2) => createIssueWithContext(issue2));
   }
   async getIssuesWithContext(issues, contextLines = 3) {
     const results = [];
@@ -18233,7 +18274,7 @@ class SourcesAPI extends BaseAPI {
         const sourceLines = await this.getSourceLines(component, minLine, maxLine);
         const lineMap = new Map(sourceLines.map((l) => [l.line, l]));
         for (const issue2 of componentIssues) {
-          results.push(this.createIssueWithContext(issue2, component, lineMap, contextLines));
+          results.push(this.createIssueWithSourceContext(issue2, lineMap, contextLines));
         }
       } catch {
         results.push(...this.addIssuesWithoutContext(componentIssues));
@@ -18254,7 +18295,7 @@ class SourcesAPI extends BaseAPI {
     return ["", "```", ...codeLines, "```"];
   }
   formatIssueWithContext(issue2) {
-    const filePath = issue2.component.includes(":") ? issue2.component.split(":").slice(1).join(":") : issue2.component;
+    const filePath = extractFilePathFromComponentKey(issue2.component);
     const lines = [
       `### ${issue2.severity}: ${issue2.message}`,
       this.formatFileLocation(filePath, issue2.line),
@@ -18339,12 +18380,12 @@ class DuplicationsAPI extends BaseAPI {
         if (!originalFile || !duplicateFile)
           continue;
         result.push({
-          originalFile: this.extractFilePath(originalFile.key),
+          originalFile: extractFilePathFromComponentKey(originalFile.key),
           originalLines: {
             from: originalBlock.from,
             to: originalBlock.from + originalBlock.size - 1
           },
-          duplicateFile: this.extractFilePath(duplicateFile.key),
+          duplicateFile: extractFilePathFromComponentKey(duplicateFile.key),
           duplicateLines: {
             from: dupBlock.from,
             to: dupBlock.from + dupBlock.size - 1
@@ -18385,9 +18426,6 @@ class DuplicationsAPI extends BaseAPI {
     return lines.join(`
 `);
   }
-  extractFilePath(componentKey) {
-    return componentKey.includes(":") ? componentKey.split(":").slice(1).join(":") : componentKey;
-  }
 }
 // src/api/ce.ts
 init_base_api();
@@ -18433,7 +18471,7 @@ class ComputeEngineAPI extends BaseAPI {
         onlyCurrents: options.onlyCurrents?.toString(),
         maxExecutedAt: options.maxExecutedAt,
         minSubmittedAt: options.minSubmittedAt,
-        ps: options.pageSize ?? 20
+        ps: options.pageSize ?? API.DEFAULT_LIMIT
       });
       return response.tasks;
     } catch (error45) {
@@ -18452,29 +18490,42 @@ class ComputeEngineAPI extends BaseAPI {
     const { current, queue } = await this.getComponentTasks(componentKey);
     return current !== undefined || queue.length > 0;
   }
-  async waitForTask(taskId, pollIntervalMs = 2000, maxWaitMs = 300000) {
+  shouldContinuePolling(startTime, maxWaitMs) {
+    return Date.now() - startTime < maxWaitMs;
+  }
+  isTaskComplete(status) {
+    return ["SUCCESS", "FAILED", "CANCELED"].includes(status);
+  }
+  handleTaskNotFound(taskId, retries, maxRetries) {
+    if (retries >= maxRetries) {
+      this.logger.warn(`Failed to get task ${taskId}: Not found after ${maxRetries} retries`);
+      return { shouldContinue: false, newRetries: retries };
+    }
+    return { shouldContinue: true, newRetries: retries + 1 };
+  }
+  async waitForTask(taskId, pollIntervalMs = API.POLL_INTERVAL_MS, maxWaitMs = API.ANALYSIS_TIMEOUT_MS) {
     this.logger.info(`Waiting for task to complete: ${taskId}`);
     const startTime = Date.now();
     let taskNotFoundRetries = 0;
     const maxTaskNotFoundRetries = 10;
     debugLog(`waitForTask: starting for ${taskId}`);
-    while (Date.now() - startTime < maxWaitMs) {
+    while (this.shouldContinuePolling(startTime, maxWaitMs)) {
       const task = await this.getTask(taskId);
       if (!task) {
-        taskNotFoundRetries++;
-        debugLog(`waitForTask: task not found, retry ${taskNotFoundRetries}/${maxTaskNotFoundRetries}`);
-        if (taskNotFoundRetries >= maxTaskNotFoundRetries) {
-          this.logger.warn(`Task ${taskId} not found after ${maxTaskNotFoundRetries} retries`);
+        const result = this.handleTaskNotFound(taskId, taskNotFoundRetries, maxTaskNotFoundRetries);
+        taskNotFoundRetries = result.newRetries;
+        if (!result.shouldContinue) {
           debugLog(`waitForTask: giving up after ${maxTaskNotFoundRetries} retries`);
           return;
         }
+        debugLog(`waitForTask: task not found, retry ${taskNotFoundRetries}/${maxTaskNotFoundRetries}`);
         this.logger.debug(`Task ${taskId} not found yet, retrying (${taskNotFoundRetries}/${maxTaskNotFoundRetries})...`);
         await new Promise((resolve) => setTimeout(resolve, pollIntervalMs));
         continue;
       }
       taskNotFoundRetries = 0;
       debugLog(`waitForTask: task found, status=${task.status}`);
-      if (task.status === "SUCCESS" || task.status === "FAILED" || task.status === "CANCELED") {
+      if (this.isTaskComplete(task.status)) {
         this.logger.info(`Task completed with status: ${task.status}`);
         debugLog(`waitForTask: task completed with ${task.status}`);
         return task;
@@ -18486,7 +18537,7 @@ class ComputeEngineAPI extends BaseAPI {
     debugLog(`waitForTask: timeout after ${maxWaitMs}ms`);
     return;
   }
-  async waitForAnalysis(componentKey, timeoutMs = 300000, pollIntervalMs = 2000) {
+  async waitForAnalysis(componentKey, timeoutMs = API.ANALYSIS_TIMEOUT_MS, pollIntervalMs = API.POLL_INTERVAL_MS) {
     this.logger.info(`Waiting for analysis to complete: ${componentKey}`);
     const startTime = Date.now();
     let lastTaskId;
@@ -18550,8 +18601,7 @@ class ProjectAnalysesAPI extends BaseAPI {
       projectKey: options.projectKey,
       projectKeyLength: options.projectKey?.length
     });
-    if (!options.projectKey) {
-      this.logger.error(`getAnalyses: projectKey is empty/undefined!`);
+    if (!this.validateProjectKey(options.projectKey, "getAnalyses")) {
       return [];
     }
     try {
@@ -18614,18 +18664,14 @@ class QualityProfilesAPI extends BaseAPI {
   }
   async getProjectProfiles(projectKey) {
     this.logger.info(`>>> getProjectProfiles called`, { projectKey, projectKeyLength: projectKey?.length });
-    if (!projectKey) {
-      this.logger.error(`getProjectProfiles: projectKey is empty/undefined!`);
+    if (!this.validateProjectKey(projectKey, "getProjectProfiles")) {
       return [];
     }
-    try {
+    return this.safeRequest(async () => {
       const response = await this.client.get("/api/qualityprofiles/search", { project: projectKey });
       this.logger.info(`<<< getProjectProfiles success`, { profileCount: response.profiles.length });
       return response.profiles;
-    } catch (error45) {
-      this.logger.error(`getProjectProfiles failed`, { error: String(error45), projectKey });
-      return [];
-    }
+    }, [], `getProjectProfiles failed for ${projectKey}`);
   }
   async getAllProfiles(language) {
     const languageInfo = language ? ` for ${language}` : "";
@@ -18640,12 +18686,9 @@ class QualityProfilesAPI extends BaseAPI {
   }
   async getInheritance(profileKey) {
     this.logger.debug(`Getting inheritance for profile ${profileKey}`);
-    try {
+    return this.safeRequest(async () => {
       return await this.client.get("/api/qualityprofiles/inheritance", { qualityProfile: profileKey });
-    } catch (error45) {
-      this.logger.warn(`Failed to get inheritance: ${error45}`);
-      return;
-    }
+    }, undefined, `Failed to get inheritance for ${profileKey}`);
   }
   formatProfilesForAgent(profiles, projectKey) {
     if (profiles.length === 0) {
@@ -18705,18 +18748,14 @@ class BranchesAPI extends BaseAPI {
   }
   async getBranches(projectKey) {
     this.logger.info(`>>> getBranches called`, { projectKey, projectKeyLength: projectKey?.length });
-    if (!projectKey) {
-      this.logger.error(`getBranches: projectKey is empty/undefined!`);
+    if (!this.validateProjectKey(projectKey, "getBranches")) {
       return [];
     }
-    try {
+    return this.safeRequest(async () => {
       const response = await this.client.get("/api/project_branches/list", { project: projectKey });
       this.logger.info(`<<< getBranches success`, { branchCount: response.branches.length });
       return response.branches;
-    } catch (error45) {
-      this.logger.error(`getBranches failed`, { error: String(error45), projectKey });
-      return [];
-    }
+    }, [], `getBranches failed for ${projectKey}`);
   }
   async getMainBranch(projectKey) {
     const branches = await this.getBranches(projectKey);
@@ -18728,16 +18767,13 @@ class BranchesAPI extends BaseAPI {
   }
   async deleteBranch(projectKey, branchName) {
     this.logger.debug(`Deleting branch ${branchName} from ${projectKey}`);
-    try {
+    return this.safeRequest(async () => {
       await this.client.post("/api/project_branches/delete", {
         project: projectKey,
         branch: branchName
       });
       return true;
-    } catch (error45) {
-      this.logger.warn(`Failed to delete branch: ${error45}`);
-      return false;
-    }
+    }, false, `Failed to delete branch ${branchName}`);
   }
   async renameMainBranch(projectKey, newName) {
     this.logger.debug(`Renaming main branch to ${newName} in ${projectKey}`);
@@ -18819,6 +18855,36 @@ class BranchesAPI extends BaseAPI {
 }
 // src/api/metrics.ts
 init_base_api();
+
+// src/utils/metric-formatter.ts
+var MetricFormatter = {
+  rating(value) {
+    const num = typeof value === "string" ? Number.parseFloat(value) : value;
+    return numericToRating(num);
+  },
+  percentage(value, decimals = 1) {
+    const num = typeof value === "string" ? Number.parseFloat(value) : value;
+    return `${num.toFixed(decimals)}%`;
+  },
+  duration(minutes) {
+    if (minutes < 60)
+      return `${minutes}min`;
+    const hours = Math.floor(minutes / 60);
+    const mins = minutes % 60;
+    return `${hours}h ${mins}min`;
+  },
+  formatByType(metricKey, value) {
+    if (metricKey.includes("rating")) {
+      return this.rating(value);
+    }
+    if (metricKey.includes("coverage") || metricKey.includes("duplicat")) {
+      return this.percentage(value);
+    }
+    return value;
+  }
+};
+
+// src/api/metrics.ts
 var COMMON_METRICS = [
   "ncloc",
   "lines",
@@ -18854,28 +18920,22 @@ class MetricsAPI extends BaseAPI {
   }
   async getMetricDefinitions() {
     this.logger.debug("Getting metric definitions");
-    try {
-      const response = await this.client.get("/api/metrics/search", { ps: 500 });
+    return this.safeRequest(async () => {
+      const response = await this.client.get("/api/metrics/search", { ps: API.DEFAULT_PAGE_SIZE });
       return response.metrics;
-    } catch (error45) {
-      this.logger.warn(`Failed to get metrics: ${error45}`);
-      return [];
-    }
+    }, [], "Failed to get metric definitions");
   }
   async getMeasures(options) {
     const metrics = options.metricKeys ?? [...COMMON_METRICS];
     this.logger.debug(`Getting measures for ${options.componentKey}`);
-    try {
+    return this.safeRequest(async () => {
       const response = await this.client.get("/api/measures/component", {
         component: options.componentKey,
         metricKeys: metrics.join(","),
         branch: options.branch
       });
       return response.component.measures;
-    } catch (error45) {
-      this.logger.warn(`Failed to get measures: ${error45}`);
-      return [];
-    }
+    }, [], `Failed to get measures for ${options.componentKey}`);
   }
   async getMeasuresWithPeriod(options) {
     const metrics = options.metricKeys ?? [...COMMON_METRICS];
@@ -18956,19 +19016,21 @@ class MetricsAPI extends BaseAPI {
   }
   formatMetricValue(metricKey, value) {
     if (metricKey.includes("rating")) {
-      const ratings = ["A", "B", "C", "D", "E"];
-      const index = Number.parseInt(value, 10) - 1;
-      return ratings[index] ?? value;
+      const numericValue = Number.parseInt(value, 10);
+      if (numericValue >= 1 && numericValue <= 5) {
+        return MetricFormatter.rating(numericValue);
+      }
+      return value;
     }
     if (metricKey.includes("coverage") || metricKey.includes("density") || metricKey.includes("ratio")) {
-      return `${value}%`;
+      return MetricFormatter.percentage(value, 1);
     }
     if (metricKey === "sqale_index") {
       const minutes = Number.parseInt(value, 10);
       if (minutes < 60)
         return `${minutes}min`;
       if (minutes < 1440)
-        return `${Math.floor(minutes / 60)}h ${minutes % 60}min`;
+        return MetricFormatter.duration(minutes);
       return `${Math.floor(minutes / 1440)}d`;
     }
     return value;
@@ -19058,8 +19120,7 @@ class ComponentsAPI extends BaseAPI {
       "|------|------|-------|--------|-------|"
     ];
     for (const file2 of files.slice(0, 20)) {
-      const shortPath = this.shortenPath(file2.path);
-      lines.push(`| ${shortPath} | ${file2.bugs} | ${file2.vulnerabilities} | ${file2.codeSmells} | **${file2.total}** |`);
+      lines.push(`| ${shortenPath(file2.path)} | ${file2.bugs} | ${file2.vulnerabilities} | ${file2.codeSmells} | **${file2.total}** |`);
     }
     if (files.length > 20) {
       lines.push(`
@@ -19077,14 +19138,6 @@ class ComponentsAPI extends BaseAPI {
     return lines.join(`
 `);
   }
-  shortenPath(path) {
-    if (path.length <= 50)
-      return path;
-    const parts = path.split("/");
-    if (parts.length <= 2)
-      return path;
-    return `.../${parts.slice(-2).join("/")}`;
-  }
 }
 
 // src/api/index.ts
@@ -19156,6 +19209,18 @@ function createSonarQubeAPI(config3, state, logger4) {
 init_config3();
 
 // src/utils/severity.ts
+var SEVERITIES = ["BLOCKER", "CRITICAL", "MAJOR", "MINOR", "INFO"];
+var SEVERITY_LEVELS = {
+  blocker: ["BLOCKER"],
+  critical: ["BLOCKER", "CRITICAL"],
+  major: ["BLOCKER", "CRITICAL", "MAJOR"],
+  minor: ["BLOCKER", "CRITICAL", "MAJOR", "MINOR"],
+  info: ["BLOCKER", "CRITICAL", "MAJOR", "MINOR", "INFO"],
+  all: ["BLOCKER", "CRITICAL", "MAJOR", "MINOR", "INFO"]
+};
+function getSeveritiesFromLevel(level) {
+  return SEVERITY_LEVELS[level.toLowerCase()] ?? [...SEVERITIES];
+}
 var SEVERITY_PRIORITY = {
   BLOCKER: 5,
   CRITICAL: 4,
@@ -19652,6 +19717,128 @@ function formatEmptyState(entity, projectKey, positiveMessage) {
   return lines.join(`
 `);
 }
+// src/tools/formatters/suggestions.ts
+function formatFixSuggestions(issues) {
+  const blockers = issues.filter((i) => i.severity === "BLOCKER");
+  const critical = issues.filter((i) => i.severity === "CRITICAL");
+  if (blockers.length === 0 && critical.length === 0) {
+    return "";
+  }
+  let output = `
+
+---
+
+## Fix Suggestions
+
+`;
+  output += `Based on the issues found, here are recommended actions:
+
+`;
+  if (blockers.length > 0) {
+    output += `### Immediate Fixes Required (Blockers)
+`;
+    for (const issue2 of blockers.slice(0, 5)) {
+      output += `1. **${issue2.file}:${issue2.line ?? "?"}** - ${issue2.message}
+`;
+      output += `   - Rule: \`${issue2.rule}\` - Review and fix this security/reliability issue
+
+`;
+    }
+  }
+  if (critical.length > 0) {
+    output += `### High Priority Fixes (Critical)
+`;
+    for (const issue2 of critical.slice(0, 5)) {
+      output += `1. **${issue2.file}:${issue2.line ?? "?"}** - ${issue2.message}
+`;
+      output += `   - Rule: \`${issue2.rule}\`
+
+`;
+    }
+  }
+  return output;
+}
+// src/utils/error-messages.ts
+var ErrorMessages = {
+  missingParameter(param, example) {
+    let msg = `Missing required parameter: ${param}`;
+    if (example) {
+      msg += `
+
+Example: ${example}`;
+    }
+    return msg;
+  },
+  notFound(entity, key, suggestions) {
+    let msg = `${entity} not found: ${key}`;
+    if (suggestions?.length) {
+      const suggestionList = suggestions.map((s) => "- " + s).join(`
+`);
+      msg += `
+
+Did you mean:
+` + suggestionList;
+    }
+    return msg;
+  },
+  configurationMissing(message, requiredSteps) {
+    let msg = message;
+    if (requiredSteps?.length) {
+      const stepList = requiredSteps.map((s, i) => i + 1 + ". " + s).join(`
+`);
+      msg += `
+
+Required steps:
+` + stepList;
+    }
+    return msg;
+  },
+  apiError(action, message, serverUrl) {
+    let msg = `Action \`${action}\` failed: ${message}`;
+    const hints = [
+      serverUrl ? `SonarQube server is reachable at ${serverUrl}` : "SonarQube server is reachable",
+      "Credentials are valid",
+      'Run with action: "setup" to initialize'
+    ];
+    const hintList = hints.map((h, i) => i + 1 + ". " + h).join(`
+`);
+    msg += `
+
+Please check:
+` + hintList;
+    return msg;
+  },
+  connectionError(serverUrl) {
+    return `Cannot connect to SonarQube server at ${serverUrl}`;
+  },
+  authenticationError(reason) {
+    return `Authentication failed: ${reason}`;
+  },
+  ruleNotFound(ruleKey) {
+    return `Rule \`${ruleKey}\` was not found.
+
+Please check:
+1. The rule key is correct (e.g., "typescript:S1234")
+2. The rule is available on your SonarQube server
+3. The rule's language plugin is installed`;
+  },
+  missingRuleKey() {
+    return `Missing \`ruleKey\` parameter. Please provide a rule key to explain.
+
+Example usage:
+\`\`\`
+sonarqube({ action: "rule", ruleKey: "typescript:S1234" })
+\`\`\`
+
+Common rule prefixes:
+- \`typescript:\` - TypeScript rules
+- \`javascript:\` - JavaScript rules
+- \`java:\` - Java rules
+- \`python:\` - Python rules
+- \`common-\` - Language-agnostic rules`;
+  }
+};
+
 // src/tools/base-handler.ts
 function createHandlerContext(config3, state, projectKey, directory) {
   return {
@@ -19757,63 +19944,11 @@ async function handleAnalyze(ctx, args) {
   }, directory);
   let output = formatAnalysisResult(result);
   if (args.fix && result.formattedIssues.length > 0) {
-    output += `
-
----
-
-## Fix Suggestions
-
-`;
-    output += `Based on the issues found, here are recommended actions:
-
-`;
-    const blockers = result.formattedIssues.filter((i) => i.severity === "BLOCKER");
-    const critical = result.formattedIssues.filter((i) => i.severity === "CRITICAL");
-    if (blockers.length > 0) {
-      output += `### Immediate Fixes Required (Blockers)
-`;
-      for (const issue2 of blockers.slice(0, 5)) {
-        output += `1. **${issue2.file}:${issue2.line ?? "?"}** - ${issue2.message}
-`;
-        output += `   - Rule: \`${issue2.rule}\` - Review and fix this security/reliability issue
-
-`;
-      }
-    }
-    if (critical.length > 0) {
-      output += `### High Priority Fixes (Critical)
-`;
-      for (const issue2 of critical.slice(0, 5)) {
-        output += `1. **${issue2.file}:${issue2.line ?? "?"}** - ${issue2.message}
-`;
-        output += `   - Rule: \`${issue2.rule}\`
-
-`;
-      }
-    }
+    output += formatFixSuggestions(result.formattedIssues);
   }
   return output;
 }
 // src/tools/handlers/issues.ts
-init_logger();
-var logger9 = new Logger("sonarqube-handler-issues");
-function getSeveritiesFromLevel(level) {
-  switch (level.toLowerCase()) {
-    case "blocker":
-      return ["BLOCKER"];
-    case "critical":
-      return ["BLOCKER", "CRITICAL"];
-    case "major":
-      return ["BLOCKER", "CRITICAL", "MAJOR"];
-    case "minor":
-      return ["BLOCKER", "CRITICAL", "MAJOR", "MINOR"];
-    case "info":
-    case "all":
-      return ["BLOCKER", "CRITICAL", "MAJOR", "MINOR", "INFO"];
-    default:
-      return ["BLOCKER", "CRITICAL", "MAJOR", "MINOR", "INFO"];
-  }
-}
 async function handleIssues(ctx, args) {
   const { api: api2, projectKey } = ctx;
   const severities = args.severity === "all" ? undefined : getSeveritiesFromLevel(args.severity ?? "all");
@@ -19836,7 +19971,7 @@ async function handleIssues(ctx, args) {
         message: i.message,
         component: `${projectKey}:${i.file}`,
         line: i.line
-      })), 5);
+      })), API.SOURCE_CONTEXT_LINES);
       output += `
 
 ---
@@ -19853,9 +19988,7 @@ async function handleIssues(ctx, args) {
 
 `;
       }
-    } catch {
-      logger9.debug("Could not fetch source context for issues");
-    }
+    } catch {}
   }
   return output;
 }
@@ -19867,7 +20000,7 @@ async function handleNewIssues(ctx, args) {
     severities,
     inNewCode: true,
     resolved: false,
-    pageSize: 500
+    pageSize: API.DEFAULT_PAGE_SIZE
   });
   const issues = api2.issues.formatIssues(response.issues, response.components);
   if (issues.length === 0) {
@@ -19889,8 +20022,7 @@ These issues were introduced in your recent changes and should be fixed before m
       bySeverity[issue2.severity] = [];
     bySeverity[issue2.severity].push(issue2);
   }
-  const severityOrder = ["BLOCKER", "CRITICAL", "MAJOR", "MINOR", "INFO"];
-  for (const sev of severityOrder) {
+  for (const sev of SEVERITIES) {
     const sevIssues = bySeverity[sev];
     if (!sevIssues || sevIssues.length === 0)
       continue;
@@ -20066,31 +20198,14 @@ async function handleRule(ctx, ruleKey) {
   if (!ruleKey) {
     return `## SonarQube Error
 
-Missing \`ruleKey\` parameter. Please provide a rule key to explain.
-
-Example usage:
-\`\`\`
-sonarqube({ action: "rule", ruleKey: "typescript:S1234" })
-\`\`\`
-
-Common rule prefixes:
-- \`typescript:\` - TypeScript rules
-- \`javascript:\` - JavaScript rules
-- \`java:\` - Java rules
-- \`python:\` - Python rules
-- \`common-\` - Language-agnostic rules`;
+${ErrorMessages.missingRuleKey()}`;
   }
   const { api: api2 } = ctx;
   const rule = await api2.rules.getRule(ruleKey);
   if (!rule) {
     return `## SonarQube Rule Not Found
 
-Rule \`${ruleKey}\` was not found. 
-
-Please check:
-1. The rule key is correct (e.g., "typescript:S1234")
-2. The rule is available on your SonarQube server
-3. The rule's language plugin is installed`;
+${ErrorMessages.ruleNotFound(ruleKey)}`;
   }
   return `## SonarQube Rule: ${rule.name}
 
@@ -20186,7 +20301,7 @@ Run \`sonarqube({ action: "analyze" })\` to generate metrics.`));
   return output;
 }
 // src/tools/sonarqube.ts
-var logger10 = new Logger("sonarqube-tool");
+var logger9 = new Logger("sonarqube-tool");
 var SonarQubeToolArgsSchema = exports_external2.object({
   action: exports_external2.enum(["analyze", "issues", "newissues", "status", "init", "setup", "validate", "hotspots", "duplications", "rule", "history", "profile", "branches", "metrics", "worstfiles"]).describe("Action to perform: analyze (run scanner), issues (all issues), newissues (only new code issues), status (quality gate), init/setup (initialize), validate (enterprise check), hotspots (security review), duplications (code duplicates), rule (explain rule), history (past analyses), profile (quality profile), branches (branch status), metrics (detailed metrics), worstfiles (files with most issues)"),
   scope: exports_external2.enum(["all", "new", "changed"]).optional().default("all").describe("Scope of analysis: all files, only new code, or changed files"),
@@ -20201,12 +20316,11 @@ async function executeSonarQubeTool(args, context) {
   const directory = context.directory ?? process.cwd();
   const config3 = loadConfig(context.config);
   if (!config3) {
-    return formatError2(`SonarQube configuration not found.
-
-Please ensure you have one of:
-1. Environment variables: SONAR_HOST_URL and SONAR_TOKEN
-2. A .sonarqube/config.json file in your project
-3. Plugin configuration in opencode.json
+    return formatError2(ErrorMessages.configurationMissing("SonarQube configuration not found.", [
+      "Set environment variables: SONAR_HOST_URL and SONAR_TOKEN",
+      "Or create a .sonarqube/config.json file in your project",
+      "Or add plugin configuration in opencode.json"
+    ]) + `
 
 Required environment variables:
 - SONAR_HOST_URL (e.g., https://sonarqube.company.com)
@@ -20216,13 +20330,13 @@ Optional:
 - SONAR_USER
 - SONAR_PASSWORD`);
   }
-  logger10.info(`Executing SonarQube tool: ${args.action}`, { directory });
+  logger9.info(`Executing SonarQube tool: ${args.action}`, { directory });
   try {
     if (args.action === "init" || args.action === "setup") {
       return await handleSetup(config3, directory, args.force);
     }
     if (await needsBootstrap(directory)) {
-      logger10.info("First run detected, running bootstrap");
+      logger9.info("First run detected, running bootstrap");
       const { bootstrap: bootstrap2 } = await Promise.resolve().then(() => (init_bootstrap(), exports_bootstrap));
       const setupResult = await bootstrap2({ config: config3, directory });
       if (!setupResult.success) {
@@ -20230,7 +20344,7 @@ Optional:
 
 ${setupResult.message}`);
       }
-      logger10.info("Bootstrap completed", { projectKey: setupResult.projectKey });
+      logger9.info("Bootstrap completed", { projectKey: setupResult.projectKey });
     }
     const state = await getProjectState(directory);
     if (!state) {
@@ -20270,13 +20384,8 @@ ${setupResult.message}`);
     }
   } catch (error45) {
     const errorMessage = error45 instanceof Error ? error45.message : String(error45);
-    logger10.error(`Tool execution failed: ${errorMessage}`);
-    return formatError2(`Action \`${args.action}\` failed: ${errorMessage}
-
-Please check:
-1. SonarQube server is reachable at ${config3.url}
-2. Credentials are valid
-3. Run with action: "setup" to initialize`);
+    logger9.error(`Tool execution failed: ${errorMessage}`);
+    return formatError2(ErrorMessages.apiError(args.action, errorMessage, config3.url));
   }
 }
 

package/package.json

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