XS✔ ◾ Streamline unit test suite

.github/workflows/build.yml

@@ -187,9 +187,10 @@ jobs:
           MARKDOWN_CONFIG_FILE: ../../.markdownlint.json
           SAVE_SUPER_LINTER_SUMMARY: true
           SPELL_CODESPELL_CONFIG_FILE: .codespellrc
-          # Disable linters that conflict with other linters.
+          # Disable linters that conflict with other linters or are not enforced.
           VALIDATE_BIOME_FORMAT: false
           VALIDATE_BIOME_LINT: false
+          VALIDATE_GIT_COMMITLINT: false
           VALIDATE_JSON: false
           VALIDATE_PYTHON_BLACK: false
           VALIDATE_TYPESCRIPT_ES: false

eslint.config.mjs

@@ -30,6 +30,7 @@ export default tseslint.config(
     },
     rules: {
       "@typescript-eslint/consistent-type-exports": "error",
+      "@typescript-eslint/consistent-type-imports": "error",
       "@typescript-eslint/default-param-last": "error",
       "@typescript-eslint/explicit-function-return-type": "error",
       "@typescript-eslint/explicit-member-accessibility": "error",

package.json

@@ -7,8 +7,8 @@
   "main": "dist/index.mjs",
   "type": "module",
   "scripts": {
-    "build:initialization:debug": "npm ci && node scripts/fs-helpers.mjs cp src debug",
-    "build:initialization:release": "npm ci && node scripts/fs-helpers.mjs cp src release",
+    "build:initialization:debug": "npm ci && node scripts/fs-helpers.mjs rm debug && node scripts/fs-helpers.mjs cp src debug",
+    "build:initialization:release": "npm ci && node scripts/fs-helpers.mjs rm release && node scripts/fs-helpers.mjs cp src release",
     "build:debug": "npm run build:initialization:debug && cd debug/task && tsc --sourceMap",
     "build:release": "npm run build:initialization:release && npm run build:release:compile && npm run build:release:clean && npm run build:release:package",
     "build:release:compile": "cd release/task && tsc && ncc build index.js --out . --minify",
@@ -28,7 +28,7 @@
     "deploy:upload": "tfx build tasks upload --task-path release/task --no-prompt",
     "lint": "eslint --fix **/*.ts",
     "test": "npm run build:debug && cd debug/task && c8 --reporter=text --reporter=text-summary mocha tests/**/*.spec.js",
-    "test:fast": "node scripts/fs-helpers.mjs cp src debug && cd debug/task && tsc --sourceMap && c8 --reporter=text --reporter=text-summary mocha tests/**/*.spec.js",
+    "test:fast": "node scripts/fs-helpers.mjs rm debug && node scripts/fs-helpers.mjs cp src debug && cd debug/task && tsc --sourceMap && c8 --reporter=text --reporter=text-summary mocha tests/**/*.spec.js",
     "update:dependencies": "npm update",
     "update:versions": "ncu -u"
   },

src/task/src/repos/reposInvokerInterface.d.ts

@@ -3,9 +3,9 @@
  * Licensed under the MIT License.
  */
 
-import CommentData from "./interfaces/commentData.js";
-import { CommentThreadStatus } from "azure-devops-node-api/interfaces/GitInterfaces.js";
-import PullRequestDetailsInterface from "./interfaces/pullRequestDetailsInterface.js";
+import type CommentData from "./interfaces/commentData.js";
+import type { CommentThreadStatus } from "azure-devops-node-api/interfaces/GitInterfaces.js";
+import type PullRequestDetailsInterface from "./interfaces/pullRequestDetailsInterface.js";
 
 /**
  * An interface for invoking repository functionality with any underlying repository store.

src/task/src/runners/runnerInvokerInterface.d.ts

@@ -3,8 +3,8 @@
  * Licensed under the MIT License.
  */
 
-import { EndpointAuthorization } from "./endpointAuthorization.js";
-import ExecOutput from "./execOutput.js";
+import type { EndpointAuthorization } from "./endpointAuthorization.js";
+import type ExecOutput from "./execOutput.js";
 
 /**
  * An interface for invoking runner functionality with any underlying runner.