update extractInternal

Author: beyondkmp

packages/app-builder-lib/src/node-module-collector/nodeModulesCollector.ts

@@ -64,17 +64,18 @@ export abstract class NodeModulesCollector<T extends Dependency<T, OptionalsType
   }
 
   protected extractInternal(deps: T["dependencies"]): T["dependencies"] {
-    return deps && Object.keys(deps).length > 0
-      ? Object.entries(deps).reduce((accum, [packageName, depObjectOrVersionString]) => {
-          return {
-            ...accum,
-            [packageName]:
-              typeof depObjectOrVersionString === "object" && Object.keys(depObjectOrVersionString).length > 0
-                ? this.extractRelevantData(depObjectOrVersionString)
-                : depObjectOrVersionString,
-          }
-        }, {})
-      : undefined
+    if (!deps || Object.keys(deps).length === 0) {
+      return undefined
+    }
+
+    return Object.fromEntries(
+      Object.entries(deps).map(([packageName, depObjectOrVersionString]) => [
+        packageName,
+        typeof depObjectOrVersionString === "object" && Object.keys(depObjectOrVersionString).length > 0
+          ? this.extractRelevantData(depObjectOrVersionString)
+          : depObjectOrVersionString,
+      ])
+    ) as T["dependencies"]
   }
 
   protected resolvePath(filePath: string): string {

packages/app-builder-lib/src/node-module-collector/pnpmNodeModulesCollector.ts

@@ -46,6 +46,7 @@ export class PnpmNodeModulesCollector extends NodeModulesCollector<PnpmDependenc
     const prodDependencies = { ...packageJson.dependencies, ...packageJson.optionalDependencies }
 
     const deps = { ...(tree.dependencies || {}), ...(tree.optionalDependencies || {}) }
+    this.productionGraph[newKey] = { dependencies: [] }
     const dependencies = Object.entries(deps)
       .map(([packageName, dependency]) => {
         // check if the optional dependency's path is existing