Fixes #36

Improved logging to note missing bundles, properly process logged errors

Author: Silic0nS0ldier

src/bundle-orchestrator.ts

@@ -201,7 +201,20 @@ export class BundleOrchestrator extends Transform {
         try {
             // Produce error if there are bundles without all requirements
             if (this.scriptBundles.size > 0 || this.styleBundles.size > 0) {
-                throw new Error("Stream completed before all bundles received their dependencies");
+                const missingBundles: {
+                    type: BundleType,
+                    name: string,
+                    remainingFiles: string[],
+                }[] = [];
+                for (const bundle of this.scriptBundles) {
+                    missingBundles.push(bundle.report());
+                }
+                for (const bundle of this.styleBundles) {
+                    missingBundles.push(bundle.report());
+                }
+                const errMessage = "Stream completed before all bundles received their dependencies";
+                this.logger.error(errMessage, missingBundles);
+                throw new Error(errMessage);
             }
 
             // Invoke results callback
@@ -210,7 +223,7 @@ export class BundleOrchestrator extends Transform {
             callback();
         }
         catch (error) {
-            this.logger.error("_flush completed with error", { error });
+            this.logger.error("_flush completed with error", { error: error?.toString() ?? error });
             callback(new PluginError(PluginName, error));
         }
     }

src/bundle.ts

@@ -142,4 +142,16 @@ export class Bundle {
             return false;
         }
     }
+
+    /**
+     * Returns an object used for reporting bundle status when the stream completes before all required
+     * files have been received.
+     */
+    public report() {
+        return {
+            type: this.type,
+            name: this.name,
+            remainingFiles: this.remainingPaths.slice(),
+        };
+    }
 }