Fix legacy SARIF categories for CQ in default setup

Author: mbg

lib/analyze.js

@@ -697,7 +697,10 @@ export async function runQueries(
       let qualityAnalysisSummary: string | undefined = undefined;
       if (config.augmentationProperties.qualityQueriesInput !== undefined) {
         logger.info(`Interpreting quality results for ${language}`);
-        const qualityCategory = automationDetailsId;
+        const qualityCategory = util.fixCodeQualityCategory(
+          logger,
+          automationDetailsId,
+        );
         const qualitySarifFile = path.join(
           sarifFolder,
           `${language}.quality.sarif`,

lib/analyze.js.map

@@ -497,3 +497,45 @@ test("getCgroupCpuCountFromCpus returns undefined if the CPU file exists but is
     );
   });
 });
+
+test("fixCodeQualityCategory", (t) => {
+  const logger = getRunnerLogger(true);
+  process.env["GITHUB_EVENT_NAME"] = "dynamic";
+
+  // Categories that should get adjusted.
+  t.is(util.fixCodeQualityCategory(logger, "/language:c#"), "/language:csharp");
+  t.is(util.fixCodeQualityCategory(logger, "/language:cpp"), "/language:c-cpp");
+  t.is(util.fixCodeQualityCategory(logger, "/language:c"), "/language:c-cpp");
+  t.is(
+    util.fixCodeQualityCategory(logger, "/language:java"),
+    "/language:java-kotlin",
+  );
+  t.is(
+    util.fixCodeQualityCategory(logger, "/language:javascript"),
+    "/language:javascript-typescript",
+  );
+  t.is(
+    util.fixCodeQualityCategory(logger, "/language:typescript"),
+    "/language:javascript-typescript",
+  );
+  t.is(
+    util.fixCodeQualityCategory(logger, "/language:kotlin"),
+    "/language:java-kotlin",
+  );
+
+  // Categories that should not get adjusted.
+  t.is(
+    util.fixCodeQualityCategory(logger, "/language:csharp"),
+    "/language:csharp",
+  );
+  t.is(util.fixCodeQualityCategory(logger, "/language:go"), "/language:go");
+  t.is(
+    util.fixCodeQualityCategory(logger, "/language:actions"),
+    "/language:actions",
+  );
+
+  // Other cases.
+  t.is(util.fixCodeQualityCategory(logger, undefined), undefined);
+  t.is(util.fixCodeQualityCategory(logger, "random string"), "random string");
+  t.is(util.fixCodeQualityCategory(logger, "kotlin"), "kotlin");
+});

lib/util.js

@@ -18,6 +18,7 @@ import type { Config, Pack } from "./config-utils";
 import { EnvVar } from "./environment";
 import { Language } from "./languages";
 import { Logger } from "./logging";
+import { isDefaultSetup } from "./actions-util";
 
 /**
  * Specifies bundle versions that are known to be broken
@@ -1250,3 +1251,44 @@ export async function isBinaryAccessible(
     return false;
   }
 }
+
+/** Adjusts the category string for a Code Quality SARIF file if an "old"
+ * category identifier is used by Default Setup.
+ */
+export function fixCodeQualityCategory(
+  logger: Logger,
+  category?: string,
+): string | undefined {
+  // A mapping from old categories to new ones.
+  const qualityCategoryMapping: Record<string, string> = {
+    "c#": "csharp",
+    cpp: "c-cpp",
+    c: "c-cpp",
+    "c++": "c-cpp",
+    java: "java-kotlin",
+    javascript: "javascript-typescript",
+    typescript: "javascript-typescript",
+    kotlin: "java-kotlin",
+  };
+
+  // The `category` should always be set by Default Setup. We perform this check
+  // to avoid potential issues if Code Quality supports Advanced Setup in the future
+  // and before this workaround is removed.
+  if (
+    category !== undefined &&
+    isDefaultSetup() &&
+    category.startsWith("/language:")
+  ) {
+    const language = category.substring("/language:".length);
+    const mappedLanguage = qualityCategoryMapping[language];
+    if (mappedLanguage) {
+      const newCategory = `/language:${mappedLanguage}`;
+      logger.info(
+        `Adjusted category for Code Quality from '${category}' to '${newCategory}'.`,
+      );
+      return newCategory;
+    }
+  }
+
+  return category;
+}