FIX: @W-20065451@: Fix fast-glob import and enforce non-synthetic interpretations (#1924)

Author: stephen-carter-at-sf

src/lib/utils/WorkspaceUtil.ts

@@ -1,4 +1,4 @@
-import * as fg from 'fast-glob';
+import fg from 'fast-glob';
 import {CodeAnalyzer, Workspace} from '@salesforce/code-analyzer-core';
 import {getMessage, BundleName} from '../messages.js';
 

test/lib/actions/RunAction.test.ts

@@ -1,4 +1,4 @@
-import path from 'node:path';
+import * as path from 'node:path';
 import * as fs from 'node:fs';
 import ansis from 'ansis';
 import {SfError} from '@salesforce/core';

tsconfig.json

@@ -6,6 +6,7 @@
 		"sourceMap": true,
 		"declaration": true,
 		"moduleResolution": "NodeNext",
+		"allowSyntheticDefaultImports": false,
 		"alwaysStrict": true,
 		"noUnusedLocals": true,
 		"outDir": "./lib",

vitest.config.ts

@@ -2,6 +2,11 @@ import { defineConfig } from "vitest/config";
 
 export default defineConfig({
     test: {
+        deps: {
+            // We turn this off so that vitest catches "default imports" vs "namespace imports" issues
+            // instead of treating both types of imports as the same.
+            interopDefault: false
+        },
         setupFiles: ["test/setup-tests.ts"],
         testTimeout: 60000,
         environment: "node",