Merge pull request #52 from UmamiAppearance/irregular-import-statements---quick-fix

Irregular import statements  - quick fix

Author: UmamiAppearance

cjs/import-manager.cjs

@@ -388,14 +388,15 @@ class ImportManagerUnitMethods {
  * It handles code analysis, creates units from import
  * statements, attaches methods to the units and more.
  * 
- * @version 0.4.2
+ * @version 0.4.3
  * @author UmamiAppearance [mail@umamiappearance.eu]
  * @license MIT
  * @see https://github.com/UmamiAppearance/rollup-plugin-import-manager
  */
 
 
 
+
 class ImportManager {
 
     /**
@@ -497,6 +498,11 @@ class ImportManager {
 
             if (node.type === "ImportDeclaration") {
                 const unit = this.#es6NodeToUnit(node);
+                if (!unit) {
+                    this.#unitCreationFailedWarning(node);
+                    return;
+                }
+                
                 unit.id = es6Id ++;
                 unit.index = es6Index ++;
                 unit.hash = this.#makeHash(unit);
@@ -511,6 +517,11 @@ class ImportManager {
 
                     if (part.type === "ImportExpression") {
                         const unit = this.#dynamicNodeToUnit(node, part);
+                        if (!unit) {
+                            this.#unitCreationFailedWarning(node);
+                            return;
+                        }
+                        
                         unit.id = dynamicId ++;
                         unit.index = dynamicIndex ++;
                         unit.hash = this.#makeHash(unit);
@@ -520,6 +531,11 @@ class ImportManager {
 
                     else if (part.type === "Identifier" && part.name === "require") {
                         const unit = this.#cjsNodeToUnit(node);
+                        if (!unit) {
+                            this.#unitCreationFailedWarning(node);
+                            return;
+                        }
+                        
                         unit.id = cjsId ++;
                         unit.index = cjsIndex ++;
                         unit.hash = this.#makeHash(unit);
@@ -600,6 +616,7 @@ class ImportManager {
      * @returns {object} - Import Manager Unit Object.
      */
     #es6NodeToUnit(node, oStart, oEnd) {
+        if (!node) return;
 
         let code;
         if (typeof node === "string") {
@@ -736,6 +753,7 @@ class ImportManager {
      * @returns {object} - Import Manager Unit Object.
      */
     #dynamicNodeToUnit(node, importObject) {
+        if (!node) return;
 
         const code = this.code.slice(node.start, node.end);
 
@@ -772,6 +790,7 @@ class ImportManager {
      * @returns {object} - Import Manager Unit Object.
      */
     #cjsNodeToUnit(node) {
+        if (!node || !node.declarations) return;
 
         const code = this.code.slice(node.start, node.end);
 
@@ -1259,6 +1278,13 @@ class ImportManager {
         this.warnSpamProtection.add(hash);
         this.warn(msg);
     }
+
+
+    #unitCreationFailedWarning(node) {
+        const codeSnippet = this.code.slice(node.start, node.end);
+        const message = `Could not create a unit from code snippet:${os.EOL}---${os.EOL}${codeSnippet}${os.EOL}---${os.EOL}If the related code is correct, this might be a bug. You can report this on:${os.EOL}https://github.com/UmamiAppearance/ImportManager/issues${os.EOL}`; 
+        this.warn(message);
+    }
 }