fix: Fix ts-import strategy.

darkobits · Jun 25, 2023 · 435d525 · 435d525
1 parent 2916ee2
commit 435d525
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 6 deletions.
diff --git a/src/lib/configuration/loader.ts b/src/lib/configuration/loader.ts
@@ -104,8 +104,8 @@ export default async function configurationLoader(filePath: string, content: str
     errors.push(err);
   }
 
-
-  if (errors.length > 0) {
-    throw new AggregateError(errors, 'All parsing strategies failed.');
-  }
+  if (errors.length > 0) throw new AggregateError(
+    errors,
+    'All parsing strategies failed.'
+  );
 }
diff --git a/src/lib/configuration/strategies/esbuild.ts b/src/lib/configuration/strategies/esbuild.ts
@@ -64,6 +64,11 @@ export async function esbuildStrategy(filePath: string, pkgInfo: PackageInfo) {
     const result = await import(tempFilePath);
 
     return result?.default ?? result;
+  } catch (cause: any) {
+    throw new Error(
+      `${log.prefix('esbuildStrategy')} Failed to load configuration file: ${cause}`,
+      { cause }
+    );
   } finally {
     if (await fs.exists(tempFilePath)) {
       await fs.remove(tempFilePath);

diff --git a/src/lib/configuration/strategies/ts-import.ts b/src/lib/configuration/strategies/ts-import.ts
@@ -24,8 +24,11 @@ export async function tsImportStrategy(filePath: string) {
     });
 
     return result?.default || result;
-  } catch (err: any) {
-    log.error(log.prefix('ts-import'), 'Error removing cache directory:', err);
+  } catch (cause: any) {
+    throw new Error(
+      `${log.prefix('tsImportStrategy')} Failed to load configuration file: ${cause}`,
+      { cause }
+    );
   } finally {
     if (await fs.exists(cacheDir)) await fs.remove(cacheDir);
   }