Merge pull request #2698 from github/cklin/diff-informed-status-report

Add analysis_is_diff_informed to status report
github · Jan 16, 2025 · 94f08f3 · 94f08f3
2 parents 7ae5fc3 + 5889cfd
commit 94f08f3
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 2 deletions.
diff --git a/lib/analyze.js b/lib/analyze.js
diff --git a/lib/analyze.js.map b/lib/analyze.js.map
diff --git a/lib/analyze.test.js b/lib/analyze.test.js
diff --git a/lib/analyze.test.js.map b/lib/analyze.test.js.map
diff --git a/src/analyze.test.ts b/src/analyze.test.ts
@@ -108,6 +108,7 @@ test("status report fields", async (t) => {
         createFeatures([Feature.QaTelemetryEnabled]),
       );
       t.deepEqual(Object.keys(statusReport).sort(), [
+        "analysis_is_diff_informed",
         `analyze_builtin_queries_${language}_duration_ms`,
         "event_reports",
         `interpret_results_${language}_duration_ms`,

diff --git a/src/analyze.ts b/src/analyze.ts
@@ -108,6 +108,12 @@ export interface QueriesStatusReport {
   /** Time taken in ms to interpret results for swift (or undefined if this language was not analyzed). */
   interpret_results_swift_duration_ms?: number;
 
+  /**
+   * Whether the analysis is diff-informed (in the sense that the action generates a diff-range data
+   * extension for the analysis, regardless of whether the data extension is actually used by queries).
+   */
+  analysis_is_diff_informed?: boolean;
+
   /** Name of language that errored during analysis (or undefined if no language failed). */
   analyze_failure_language?: string;
   /** Reports on discrete events associated with this status report. */
@@ -545,6 +551,7 @@ export async function runQueries(
 ): Promise<QueriesStatusReport> {
   const statusReport: QueriesStatusReport = {};
 
+  statusReport.analysis_is_diff_informed = diffRangePackDir !== undefined;
   const dataExtensionFlags = diffRangePackDir
     ? [
         `--additional-packs=${diffRangePackDir}`,