More accurate fix for #69

build.js

@@ -167,7 +167,7 @@ const runBuildTest_e2e = async (template = "treemap") => {
   await bundle.write(outputOptions);
 };
 
-const runBuildTest_gh59 = async () => {
+const runBuildTest_gh59 = async (template) => {
   const input = {
     index: "test/gh59/src/index",
     "components/index": "test/gh59/src/components/index",
@@ -181,7 +181,7 @@ const runBuildTest_gh59 = async () => {
       require("./")({
         title: "test gh59",
         filename: `stats.gh59${fileExt}`,
-        template: "treemap",
+        template,
         ...simpleOptions,
       }),
     ],
@@ -198,7 +198,7 @@ const runBuildTest_gh59 = async () => {
   await bundle.write(outputOptions);
 };
 
-const runBuildTest_gh69 = async () => {
+const runBuildTest_gh69 = async (template) => {
   const input = "test/gh69/main.js";
 
   const inputOptions = {
@@ -207,7 +207,7 @@ const runBuildTest_gh69 = async () => {
       require("./")({
         title: "test gh69",
         filename: `stats.gh69${fileExt}`,
-        template: "treemap",
+        template,
         ...simpleOptions,
       }),
     ],
@@ -224,17 +224,20 @@ const runBuildTest_gh69 = async () => {
   await bundle.write(outputOptions);
 };
 
+const buildAll = (action) =>
+  Promise.all(templatesToBuild.map((t) => action(t)));
+
 const run = async () => {
   await Promise.all(TEMPLATE.map((t) => runBuild(t)));
   if (argv.dev) {
-    await Promise.all(templatesToBuild.map((t) => runBuildDev(t)));
+    await buildAll(runBuildDev);
   }
   if (argv.e2e) {
-    await Promise.all(templatesToBuild.map((t) => runBuildTest_e2e(t)));
+    await buildAll(runBuildTest_e2e);
   }
   if (argv.test) {
-    await runBuildTest_gh59();
-    await runBuildTest_gh69();
+    await buildAll(runBuildTest_gh59);
+    await buildAll(runBuildTest_gh69);
   }
 };
 

plugin/index.js

@@ -103,7 +103,6 @@ module.exports = function (opts) {
           tree = buildTree(id, Object.entries(modules), mapper);
         } else {
           const modules = Object.entries(bundle.modules);
-          if (modules.length === 0) continue; //TODO this is not exactly right
 
           tree = buildTree(id, modules, mapper);
         }

src/script-sunburst.js

@@ -263,9 +263,9 @@ const Main = ({ width, height, data: { tree, nodes, options = {} } }) => {
       for (const prop of availableSizeProperties) {
         value[prop] = 0;
       }
-      const children = node.children;
-      if (children != null) {
-        let i = children.length;
+      if (node.data.children != null) {
+        const children = node.children;
+        let i = node.data.children.length;
         while (--i >= 0) {
           for (const prop of availableSizeProperties) {
             value[prop] += children[i].originalValue[prop];

src/script-treemap.js

@@ -398,9 +398,13 @@ const Main = ({
       for (const prop of availableSizeProperties) {
         value[prop] = 0;
       }
-      const children = node.children;
-      if (children != null) {
-        let i = children.length;
+
+      // use node.data.children because if it is empty d3 will skip this node
+      // and it will look like it is actually a leaf - which technically it is but not exactly
+      // it is just a chunk without deps - usually just with imports
+      if (node.data.children != null) {
+        const children = node.children;
+        let i = node.data.children.length;
         while (--i >= 0) {
           for (const prop of availableSizeProperties) {
             value[prop] += children[i].originalValue[prop];