Address review

sass · Jan 25, 2024 · f05aa50 · f05aa50
1 parent abd2a71
commit f05aa50
Showing 1 changed file with 11 additions and 12 deletions.
diff --git a/lib/src/importer-registry.ts b/lib/src/importer-registry.ts
@@ -17,6 +17,16 @@ export class NodePackageImporter {
   [entryPointPathKey]?: string;
 
   constructor(entryPointPath?: string) {
+    entryPointPath = entryPointPath
+      ? p.resolve(process.cwd(), entryPointPath)
+      : require.main?.filename;
+    if (!entryPointPath) {
+      throw new Error(
+        'The Node package importer cannot determine an entry point ' +
+          'because `require.main.filename` is not defined. ' +
+          'Please provide an `entryPointPath` to the `NodePackageImporter`.'
+      );
+    }
     this[entryPointPathKey] = entryPointPath;
   }
 }
@@ -58,18 +68,7 @@ export class ImporterRegistry<sync extends 'sync' | 'async'> {
     const message = new proto.InboundMessage_CompileRequest_Importer();
     if (importer instanceof NodePackageImporter) {
       const importerMessage = new proto.NodePackageImporter();
-      const entryPointPath = importer[entryPointPathKey]
-        ? p.resolve(process.cwd(), importer[entryPointPathKey])
-        : require.main?.filename;
-
-      if (!entryPointPath) {
-        throw new Error(
-          'The Node package importer cannot determine an entry point ' +
-            'because `require.main.filename` is not defined. ' +
-            'Please provide an `entryPointPath` to the `NodePackageImporter`.'
-        );
-      }
-      importerMessage.entryPointPath = entryPointPath;
+      importerMessage.entryPointPath = importer[entryPointPathKey]!;
       message.importer = {
         case: 'nodePackageImporter',
         value: importerMessage,