Enable tseslint strict rules

Author: henrymercer

eslint.config.mjs

@@ -27,6 +27,7 @@ export default [
   js.configs.recommended,
   // Type-checked rules from typescript-eslint
   ...tseslint.configs.recommendedTypeChecked,
+  ...tseslint.configs.strict,
   // eslint-plugin-github recommended config
   githubFlatConfigs.recommended,
   // eslint-plugin-github typescript config
@@ -127,6 +128,8 @@ export default [
       "no-async-foreach/no-async-foreach": "error",
       "no-sequences": "error",
       "no-shadow": "off",
+      // This is overly restrictive with unsetting `EnvVar`s
+      "@typescript-eslint/no-dynamic-delete": "off",
       "@typescript-eslint/no-shadow": "error",
       "@typescript-eslint/prefer-optional-chain": "error",
       "one-var": ["error", "never"],

src/upload-lib.ts

@@ -1130,11 +1130,7 @@ function sanitize(str?: string) {
 /**
  * An error that occurred due to an invalid SARIF upload request.
  */
-export class InvalidSarifUploadError extends Error {
-  constructor(message: string) {
-    super(message);
-  }
-}
+export class InvalidSarifUploadError extends Error {}
 
 function filterAlertsByDiffRange(logger: Logger, sarif: SarifFile): SarifFile {
   const diffRanges = readDiffRangesJsonFile(logger);

src/util.ts

@@ -690,11 +690,7 @@ export class HTTPError extends Error {
  * An Error class that indicates an error that occurred due to
  * a misconfiguration of the action or the CodeQL CLI.
  */
-export class ConfigurationError extends Error {
-  constructor(message: string) {
-    super(message);
-  }
-}
+export class ConfigurationError extends Error {}
 
 export function asHTTPError(arg: any): HTTPError | undefined {
   if (