From 6a9f7e7fc3ed89bafa53a68079075ba34a2043f8 Mon Sep 17 00:00:00 2001 From: "Michael B. Gale" Date: Mon, 12 Oct 2026 22:50:11 +0100 Subject: [PATCH] Log if returning early from `recordOverlayStatus` due to no languages --- lib/entry-points.js | 7 ++++++- src/init-action-post-helper.ts | 8 ++++++-- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/lib/entry-points.js b/lib/entry-points.js index 600323420..8424aef54 100644 --- a/lib/entry-points.js +++ b/lib/entry-points.js @@ -187034,7 +187034,12 @@ function isConclusiveJobStatus(jobStatus) { } } async function recordOverlayStatus(codeql, config, features, jobStatus, env, logger2) { - if (config.overlayDatabaseMode !== "overlay-base" /* OverlayBase */ || env.getOptional("CODEQL_ACTION_ANALYZE_DID_COMPLETE_SUCCESSFULLY" /* ANALYZE_DID_COMPLETE_SUCCESSFULLY */) === "true" || !await features.getValue("overlay_analysis_status_save" /* OverlayAnalysisStatusSave */) || config.languages === void 0) { + if (config.overlayDatabaseMode !== "overlay-base" /* OverlayBase */ || env.getOptional("CODEQL_ACTION_ANALYZE_DID_COMPLETE_SUCCESSFULLY" /* ANALYZE_DID_COMPLETE_SUCCESSFULLY */) === "true" || !await features.getValue("overlay_analysis_status_save" /* OverlayAnalysisStatusSave */)) { + return; + } else if (config.languages === void 0) { + logger2.debug( + "Not recording overlay status, because the configuration contains no 'languages'." + ); return; } if (!isConclusiveJobStatus(jobStatus) && !didCodeQlReportError(env)) { diff --git a/src/init-action-post-helper.ts b/src/init-action-post-helper.ts index e9ebf93a3..b6aeb5bd3 100644 --- a/src/init-action-post-helper.ts +++ b/src/init-action-post-helper.ts @@ -471,10 +471,14 @@ async function recordOverlayStatus( if ( config.overlayDatabaseMode !== OverlayDatabaseMode.OverlayBase || env.getOptional(EnvVar.ANALYZE_DID_COMPLETE_SUCCESSFULLY) === "true" || - !(await features.getValue(Feature.OverlayAnalysisStatusSave)) || - config.languages === undefined + !(await features.getValue(Feature.OverlayAnalysisStatusSave)) ) { return; + } else if (config.languages === undefined) { + logger.debug( + "Not recording overlay status, because the configuration contains no 'languages'.", + ); + return; } // Only record a failure when the job outcome tells us something about the analysis. A cancelled