Require tools feature flag

And now that we have this, drop the restriction to `github` org.

lib/analyze-action-post.js

@@ -162221,11 +162221,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/analyze-action.js

@@ -107970,11 +107970,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/autobuild-action.js

@@ -104270,11 +104270,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/init-action-post.js

@@ -165707,11 +165707,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/init-action.js

@@ -105530,11 +105530,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,
@@ -109094,14 +109091,13 @@ function cleanupDatabaseClusterDirectory(config, logger, options = {}, rmSync2 =
     }
   }
 }
-async function getFileCoverageInformationEnabled(debugMode, repositoryNwo, features) {
+async function getFileCoverageInformationEnabled(debugMode, codeql, features) {
   return (
     // Always enable file coverage information in debug mode
     debugMode || // We're most interested in speeding up PRs, and we want to keep
     // submitting file coverage information for the default branch since
     // it is used to populate the status page.
-    !isAnalyzingPullRequest() || // For now, restrict this feature to the GitHub org
-    repositoryNwo.owner !== "github" || !await features.getValue("skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */)
+    !isAnalyzingPullRequest() || !await features.getValue("skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */, codeql)
   );
 }
 
@@ -109743,7 +109739,7 @@ async function run(startedAt) {
       repositoryProperties: repositoryPropertiesResult.orElse({}),
       enableFileCoverageInformation: await getFileCoverageInformationEnabled(
         debugMode,
-        repositoryNwo,
+        codeql,
         features
       ),
       logger

lib/resolve-environment-action.js

@@ -104261,11 +104261,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/setup-codeql-action.js

@@ -104158,11 +104158,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/start-proxy-action-post.js

@@ -161627,11 +161627,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/start-proxy-action.js

@@ -120950,11 +120950,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/upload-lib.js

@@ -107417,11 +107417,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/upload-sarif-action-post.js

@@ -161789,11 +161789,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

lib/upload-sarif-action.js

@@ -107131,11 +107131,8 @@ var featureConfig = {
   ["skip_file_coverage_on_prs" /* SkipFileCoverageOnPrs */]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
-    minimumVersion: void 0
+    minimumVersion: void 0,
+    toolsFeature: "suppressesMissingFileBaselineWarning" /* SuppressesMissingFileBaselineWarning */
   },
   ["start_proxy_remove_unused_registries" /* StartProxyRemoveUnusedRegistries */]: {
     defaultValue: false,

src/feature-flags.ts

@@ -330,11 +330,8 @@ export const featureConfig = {
   [Feature.SkipFileCoverageOnPrs]: {
     defaultValue: false,
     envVar: "CODEQL_ACTION_SKIP_FILE_COVERAGE_ON_PRS",
-    // For testing, this is not behind a CLI version check yet. However
-    // before rolling this out externally, we should set a minimum version here
-    // since current versions of the CodeQL CLI will log if baseline information
-    // cannot be found when interpreting results.
     minimumVersion: undefined,
+    toolsFeature: ToolsFeature.SuppressesMissingFileBaselineWarning,
   },
   [Feature.StartProxyRemoveUnusedRegistries]: {
     defaultValue: false,

src/init-action.ts

@@ -378,7 +378,7 @@ async function run(startedAt: Date) {
       repositoryProperties: repositoryPropertiesResult.orElse({}),
       enableFileCoverageInformation: await getFileCoverageInformationEnabled(
         debugMode,
-        repositoryNwo,
+        codeql,
         features,
       ),
       logger,

src/init.test.ts

@@ -13,7 +13,6 @@ import {
   getFileCoverageInformationEnabled,
 } from "./init";
 import { KnownLanguage } from "./languages";
-import { parseRepositoryNwo } from "./repository";
 import {
   createFeatures,
   LoggedMessage,
@@ -456,7 +455,7 @@ test("file coverage information enabled when debugMode is true", async (t) => {
   t.true(
     await getFileCoverageInformationEnabled(
       true, // debugMode
-      parseRepositoryNwo("github/codeql-action"),
+      createStubCodeQL({}),
       createFeatures([Feature.SkipFileCoverageOnPrs]),
     ),
   );
@@ -470,22 +469,7 @@ test.serial(
     t.true(
       await getFileCoverageInformationEnabled(
         false, // debugMode
-        parseRepositoryNwo("github/codeql-action"),
-        createFeatures([Feature.SkipFileCoverageOnPrs]),
-      ),
-    );
-  },
-);
-
-test.serial(
-  "file coverage information enabled when owner is not 'github'",
-  async (t) => {
-    sinon.stub(actionsUtil, "isAnalyzingPullRequest").returns(true);
-
-    t.true(
-      await getFileCoverageInformationEnabled(
-        false, // debugMode
-        parseRepositoryNwo("other-org/some-repo"),
+        createStubCodeQL({}),
         createFeatures([Feature.SkipFileCoverageOnPrs]),
       ),
     );
@@ -500,7 +484,7 @@ test.serial(
     t.true(
       await getFileCoverageInformationEnabled(
         false, // debugMode
-        parseRepositoryNwo("github/codeql-action"),
+        createStubCodeQL({}),
         createFeatures([]),
       ),
     );
@@ -515,7 +499,7 @@ test.serial(
     t.false(
       await getFileCoverageInformationEnabled(
         false, // debugMode
-        parseRepositoryNwo("github/codeql-action"),
+        createStubCodeQL({}),
         createFeatures([Feature.SkipFileCoverageOnPrs]),
       ),
     );

src/init.ts

@@ -20,7 +20,6 @@ import {
 } from "./feature-flags";
 import { KnownLanguage, Language } from "./languages";
 import { Logger, withGroupAsync } from "./logging";
-import { RepositoryNwo } from "./repository";
 import { ToolsSource } from "./setup-codeql";
 import { ZstdAvailability } from "./tar";
 import { ToolsDownloadStatusReport } from "./tools-download";
@@ -300,7 +299,7 @@ export function cleanupDatabaseClusterDirectory(
 
 export async function getFileCoverageInformationEnabled(
   debugMode: boolean,
-  repositoryNwo: RepositoryNwo,
+  codeql: CodeQL,
   features: FeatureEnablement,
 ): Promise<boolean> {
   return (
@@ -310,8 +309,6 @@ export async function getFileCoverageInformationEnabled(
     // submitting file coverage information for the default branch since
     // it is used to populate the status page.
     !isAnalyzingPullRequest() ||
-    // For now, restrict this feature to the GitHub org
-    repositoryNwo.owner !== "github" ||
-    !(await features.getValue(Feature.SkipFileCoverageOnPrs))
+    !(await features.getValue(Feature.SkipFileCoverageOnPrs, codeql))
   );
 }

src/tools-features.ts

@@ -10,6 +10,7 @@ export enum ToolsFeature {
   ForceOverwrite = "forceOverwrite",
   IndirectTracingSupportsStaticBinaries = "indirectTracingSupportsStaticBinaries",
   PythonDefaultIsToNotExtractStdlib = "pythonDefaultIsToNotExtractStdlib",
+  SuppressesMissingFileBaselineWarning = "suppressesMissingFileBaselineWarning",
 }
 
 /**