fix: now properly emits HTML files

__tests__/build.test.js

@@ -90,6 +90,14 @@ Build("Should rewrite to a hashed script for index.html", () => {
   });
 });
 
-Build("Should inject an 'application.css' stylesheet", () => {});
+Build("Should create HTML files", () => {
+  const HTMLFiles = ["index.html", "html/index.html"];
+
+  HTMLFiles.forEach((file) => {
+    file = path.resolve(buildDir, file);
+
+    assert.is(fs.existsSync(file), true);
+  });
+});
 
 Build.run();

__tests__/emitHtmlFiles.test.js

@@ -35,14 +35,12 @@ EmitHtmlFiles("Rewrite relative import to absolute", () => {
   const firstScript = domDoc.querySelector("script");
   firstScript.src = "entrypoints/application.js";
 
-  console.log("HI");
   const newDom = rewriteScripts({
     dom,
     manifest: mockManifest,
     baseUrl: "/",
   });
 
-  console.log("HI");
   const newScript = newDom.window.document.querySelector("script");
   assert.is(newScript.src, mockManifest.entrypoints.application.js);
 });

__tests__/examples/example_dir/yarn.lock

@@ -4694,7 +4694,7 @@ snapdragon@^0.8.1:
     use "^3.1.0"
 
 "snowpack-plugin-rollup-bundle@file:../../../pkg":
-  version "0.2.3"
+  version "0.3.0"
   dependencies:
     "@rollup/plugin-commonjs" "^15.1.0"
     "@rollup/plugin-node-resolve" "^9.0.0"

src/index.js

@@ -68,6 +68,7 @@ async function rollupBuild({
 
   // Add assets to manifest, use path.relative to fix minor issues
   glob.sync(`${TMP_BUILD_DIRECTORY}/**/*.*`).forEach((fileName) => {
+    console.log(fileName);
     fileName = path.relative(TMP_BUILD_DIRECTORY, fileName);
     const chunkOrAsset = { fileName, map: null };
     addToManifest({
@@ -84,10 +85,18 @@ async function rollupBuild({
     manifestJSON
   );
 
+  // HTML files will not be pulled in by rollup, its up to us
+  // to manually pull them in.
   if (pluginOptions.emitHtml === true) {
-    glob.sync(buildDirectory + "**/*.html").forEach((file) => {
+    glob.sync(buildDirectory + "/**/*.html").forEach((file) => {
       let destFile = path.relative(buildDirectory, file);
-      destFile = path.join(TMP_BUILD_DIRECTORY, destFile);
+      destFile = path.resolve(TMP_BUILD_DIRECTORY, destFile);
+      const destDir = path.parse(destFile).dir;
+
+      if (!fs.existsSync(destDir)) {
+        fs.mkdirSync(destDir, { recursive: true });
+      }
+
       emitHtmlFiles({ file, manifest, destFile, baseUrl });
     });
   }

src/options.js

@@ -2,7 +2,7 @@ import path from "path";
 
 // plugins
 import resolve from "@rollup/plugin-node-resolve";
-import commonjs from '@rollup/plugin-commonjs';
+import commonjs from "@rollup/plugin-commonjs";
 import styles from "rollup-plugin-styles";
 import { terser } from "rollup-plugin-terser";
 import url from "@rollup/plugin-url";
@@ -26,7 +26,7 @@ export function defaultInputOptions({ buildDirectory, tmpDir }) {
         limit: 0, // extract all files
         fileName: "[dirname]/[name]-[hash][extname]",
       }),
-      commonjs()
+      commonjs(),
     ],
   };
 }