Remove check for generateSummarySymbolMap CLI feature

koesie10 · koesie10 · commit eb9d12b89932 · 2025-07-11T16:40:58.000+02:00
diff --git a/extensions/ql-vscode/src/codeql-cli/cli-version.ts b/extensions/ql-vscode/src/codeql-cli/cli-version.ts
@@ -11,7 +11,6 @@ interface VersionResult {
 
 export interface CliFeatures {
   featuresInVersionResult?: boolean;
-  generateSummarySymbolMap?: boolean;
   queryServerRunQueries?: boolean;
 }
 
diff --git a/extensions/ql-vscode/src/codeql-cli/cli.ts b/extensions/ql-vscode/src/codeql-cli/cli.ts
@@ -269,7 +269,7 @@ export class CodeQLCliServer implements Disposable {
   /** Path to current codeQL executable, or undefined if not running yet. */
   codeQlPath: string | undefined;
 
-  cliConstraints = new CliVersionConstraint(this);
+  cliConstraints = new CliVersionConstraint();
 
   /**
    * When set to true, ignore some modal popups and assume user has clicked "yes".
@@ -1191,15 +1191,12 @@ export class CodeQLCliServer implements Disposable {
     outputPath: string,
     endSummaryPath: string,
   ): Promise<string> {
-    const supportsGenerateSummarySymbolMap =
-      await this.cliConstraints.supportsGenerateSummarySymbolMap();
     const subcommandArgs = [
       "--format=text",
       `--end-summary=${endSummaryPath}`,
       "--sourcemap",
-      ...(supportsGenerateSummarySymbolMap
-        ? ["--summary-symbol-map", "--minify-output"]
-        : []),
+      "--summary-symbol-map",
+      "--minify-output",
       inputPath,
       outputPath,
     ];
@@ -1905,12 +1902,4 @@ export class CliVersionConstraint {
   // The oldest version of the CLI that we support. This is used to determine
   // whether to show a warning about the CLI being too old on startup.
   public static OLDEST_SUPPORTED_CLI_VERSION = new SemVer("2.18.4");
-
-  constructor(private readonly cli: CodeQLCliServer) {
-    /**/
-  }
-
-  async supportsGenerateSummarySymbolMap(): Promise<boolean> {
-    return (await this.cli.getFeatures()).generateSummarySymbolMap === true;
-  }
 }
diff --git a/extensions/ql-vscode/src/log-insights/summary-parser.ts b/extensions/ql-vscode/src/log-insights/summary-parser.ts
@@ -1,6 +1,3 @@
-import { createReadStream, writeFile } from "fs-extra";
-import { LINE_ENDINGS, splitStreamAtSeparators } from "../common/split-stream";
-
 /**
  * Location information for a single pipeline invocation in the RA.
  */
@@ -32,103 +29,3 @@ interface PredicateSymbol {
 export interface SummarySymbols {
   predicates: Record<string, PredicateSymbol>;
 }
-
-// Tuple counts for Expr::Expr::getParent#dispred#f0820431#ff@76d6745o:
-const NON_RECURSIVE_TUPLE_COUNT_REGEXP =
-  /^Evaluated relational algebra for predicate (?<predicateName>\S+) with tuple counts:$/;
-// Tuple counts for Expr::Expr::getEnclosingStmt#f0820431#bf@923ddwj9 on iteration 0 running pipeline base:
-const RECURSIVE_TUPLE_COUNT_REGEXP =
-  /^Evaluated relational algebra for predicate (?<predicateName>\S+) on iteration (?<iteration>\d+) running pipeline (?<pipeline>\S+) with tuple counts:$/;
-const RETURN_REGEXP = /^\s*return /;
-
-/**
- * Parse a human-readable evaluation log summary to find the location of the RA for each pipeline
- * run.
- *
- * TODO: Once we're more certain about the symbol format, we should have the CLI generate this as it
- * generates the human-readabe summary to avoid having to rely on regular expression matching of the
- * human-readable text.
- *
- * @param summaryPath The path to the summary file.
- * @param symbolsPath The path to the symbols file to generate.
- */
-export async function generateSummarySymbolsFile(
-  summaryPath: string,
-  symbolsPath: string,
-): Promise<void> {
-  const symbols = await generateSummarySymbols(summaryPath);
-  await writeFile(symbolsPath, JSON.stringify(symbols));
-}
-
-/**
- * Parse a human-readable evaluation log summary to find the location of the RA for each pipeline
- * run.
- *
- * @param fileLocation The path to the summary file.
- * @returns Symbol information for the summary file.
- */
-async function generateSummarySymbols(
-  summaryPath: string,
-): Promise<SummarySymbols> {
-  const stream = createReadStream(summaryPath, {
-    encoding: "utf-8",
-  });
-  try {
-    const lines = splitStreamAtSeparators(stream, LINE_ENDINGS);
-
-    const symbols: SummarySymbols = {
-      predicates: {},
-    };
-
-    let lineNumber = 0;
-    let raStartLine = 0;
-    let iteration = 0;
-    let predicateName: string | undefined = undefined;
-    let startLine = 0;
-    for await (const line of lines) {
-      if (predicateName === undefined) {
-        // Looking for the start of the predicate.
-        const nonRecursiveMatch = line.match(NON_RECURSIVE_TUPLE_COUNT_REGEXP);
-        if (nonRecursiveMatch) {
-          iteration = 0;
-          predicateName = nonRecursiveMatch.groups!.predicateName;
-        } else {
-          const recursiveMatch = line.match(RECURSIVE_TUPLE_COUNT_REGEXP);
-          if (recursiveMatch?.groups) {
-            predicateName = recursiveMatch.groups.predicateName;
-            iteration = parseInt(recursiveMatch.groups.iteration);
-          }
-        }
-        if (predicateName !== undefined) {
-          startLine = lineNumber;
-          raStartLine = lineNumber + 1;
-        }
-      } else {
-        const returnMatch = line.match(RETURN_REGEXP);
-        if (returnMatch) {
-          let symbol = symbols.predicates[predicateName];
-          if (symbol === undefined) {
-            symbol = {
-              iterations: {},
-              recursionSummaries: {},
-            };
-            symbols.predicates[predicateName] = symbol;
-          }
-          symbol.iterations[iteration] = {
-            startLine,
-            raStartLine,
-            raEndLine: lineNumber,
-          };
-
-          predicateName = undefined;
-        }
-      }
-
-      lineNumber++;
-    }
-
-    return symbols;
-  } finally {
-    stream.close();
-  }
-}
diff --git a/extensions/ql-vscode/src/run-queries-shared.ts b/extensions/ql-vscode/src/run-queries-shared.ts
@@ -27,7 +27,6 @@ import type {
 import type { BaseLogger } from "./common/logging";
 import { showAndLogWarningMessage } from "./common/logging";
 import { extLogger } from "./common/logging/vscode";
-import { generateSummarySymbolsFile } from "./log-insights/summary-parser";
 import { getErrorMessage } from "./common/helpers-pure";
 import { createHash } from "crypto";
 import { QueryOutputDir } from "./local-queries/query-output-dir";
@@ -570,15 +569,6 @@ export async function generateEvalLogSummaries(
 
     if (humanReadableSummary !== undefined) {
       summarySymbols = outputDir.evalLogSummarySymbolsPath;
-      if (
-        !(await cliServer.cliConstraints.supportsGenerateSummarySymbolMap())
-      ) {
-        // We're using an old CLI that cannot generate the summary symbols file while generating the
-        // human-readable log summary. As a fallback, create it by parsing the human-readable
-        // summary.
-        progress(progressUpdate(3, 3, "Generating summary symbols file"));
-        await generateSummarySymbolsFile(humanReadableSummary, summarySymbols);
-      }
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,6 @@ interface VersionResult {`
`11`	`11`
`12`	`12`	`export interface CliFeatures {`
`13`	`13`	`featuresInVersionResult?: boolean;`
`14`		`- generateSummarySymbolMap?: boolean;`
`15`	`14`	`queryServerRunQueries?: boolean;`
`16`	`15`	`}`
`17`	`16`