From a5ef23d8cb46c380b6758ac35f2c7bcd843daa21 Mon Sep 17 00:00:00 2001
From: Edoardo Cavazza <edoardo.cavazza@chialab.it>
Date: Fri, 14 Jun 2024 11:10:39 +0200
Subject: [PATCH] Correctly handle empty virtual modules

---
 .changeset/pretty-experts-grin.md                        | 5 +++++
 .changeset/twelve-dragons-cover.md                       | 5 +++++
 packages/esbuild-plugin-html/lib/collectScripts.js       | 4 ++++
 packages/esbuild-plugin-html/lib/collectStyles.js        | 4 ++++
 packages/esbuild-plugin-html/test/fixture/index.css.html | 2 ++
 packages/esbuild-plugin-html/test/test.spec.js           | 4 ++++
 packages/esbuild-rna/lib/Build.js                        | 8 ++++----
 7 files changed, 28 insertions(+), 4 deletions(-)
 create mode 100644 .changeset/pretty-experts-grin.md
 create mode 100644 .changeset/twelve-dragons-cover.md
diff --git a/.changeset/pretty-experts-grin.md b/.changeset/pretty-experts-grin.md
new file mode 100644
index 00000000..7fc17414
--- /dev/null
+++ b/.changeset/pretty-experts-grin.md
@@ -0,0 +1,5 @@
+---
+'@chialab/esbuild-plugin-html': patch
+---
+
+Ignore empty `<script>` and `<style>` elements.
diff --git a/.changeset/twelve-dragons-cover.md b/.changeset/twelve-dragons-cover.md
new file mode 100644
index 00000000..ba209418
--- /dev/null
+++ b/.changeset/twelve-dragons-cover.md
@@ -0,0 +1,5 @@
+---
+'@chialab/esbuild-rna': patch
+---
+
+Correctly handle empty (`contents = ''`) virtual modules.
diff --git a/packages/esbuild-plugin-html/lib/collectScripts.js b/packages/esbuild-plugin-html/lib/collectScripts.js
index f1eb3e57..69047e9d 100644
--- a/packages/esbuild-plugin-html/lib/collectScripts.js
+++ b/packages/esbuild-plugin-html/lib/collectScripts.js
@@ -42,6 +42,10 @@ async function innerCollect($, dom, elements, target, format, type, attrs = {},
                 builds.set(element, resolvedFile.path);
                 entrypoints.set(resolvedFile.path, element);
             } else {
+                const contents = $(element).html() || '';
+                if (!contents) {
+                    return;
+                }
                 const entryPoint = path.join(options.sourceDir, helpers.createEntry('js'));
                 builds.set(element, {
                     path: entryPoint,
diff --git a/packages/esbuild-plugin-html/lib/collectStyles.js b/packages/esbuild-plugin-html/lib/collectStyles.js
index 85a86f5b..196bd7fc 100644
--- a/packages/esbuild-plugin-html/lib/collectStyles.js
+++ b/packages/esbuild-plugin-html/lib/collectStyles.js
@@ -37,6 +37,10 @@ export async function collectStyles($, dom, options, helpers) {
                 builds.set(element, resolvedFile.path);
                 entrypoints.set(resolvedFile.path, element);
             } else {
+                const contents = $(element).html() || '';
+                if (!contents) {
+                    return;
+                }
                 const entryPoint = path.join(options.sourceDir, helpers.createEntry('css'));
                 builds.set(element, {
                     path: entryPoint,
diff --git a/packages/esbuild-plugin-html/test/fixture/index.css.html b/packages/esbuild-plugin-html/test/fixture/index.css.html
index dee74692..95f23c77 100644
--- a/packages/esbuild-plugin-html/test/fixture/index.css.html
+++ b/packages/esbuild-plugin-html/test/fixture/index.css.html
@@ -11,6 +11,8 @@
             color: red;
         }
     </style>
+    <!-- Empty style tag -->
+    <style></style>
 </head>
 <body>
 
diff --git a/packages/esbuild-plugin-html/test/test.spec.js b/packages/esbuild-plugin-html/test/test.spec.js
index 882a477b..f950911d 100644
--- a/packages/esbuild-plugin-html/test/test.spec.js
+++ b/packages/esbuild-plugin-html/test/test.spec.js
@@ -714,6 +714,8 @@ body {
     <style>
         @import '1-EKADEBHI.css'
     </style>
+    <!-- Empty style tag -->
+    <style></style>
 </head>
 
 <body>
@@ -770,6 +772,8 @@ body {
     <style>
         @import '/public/1-VXBR4AUQ.css'
     </style>
+    <!-- Empty style tag -->
+    <style></style>
 </head>
 
 <body>
diff --git a/packages/esbuild-rna/lib/Build.js b/packages/esbuild-rna/lib/Build.js
index 44e38306..690cac27 100644
--- a/packages/esbuild-rna/lib/Build.js
+++ b/packages/esbuild-rna/lib/Build.js
@@ -666,7 +666,7 @@ export class Build {
             resolveDir = args.resolveDir;
         } else {
             const loadResult = await this.load(args);
-            if (!loadResult || !loadResult.contents) {
+            if (!loadResult || loadResult.contents == null) {
                 return;
             }
 
@@ -1025,7 +1025,7 @@ export class Build {
                 with: {},
             });
 
-            if (result && result.contents) {
+            if (result && result.contents != null) {
                 buffer = Buffer.from(result.contents);
             } else {
                 buffer = await readFile(source);
@@ -1119,7 +1119,7 @@ export class Build {
             value: true,
         });
 
-        if (options.contents) {
+        if (options.contents != null) {
             delete config.entryPoints;
             config.stdin = {
                 sourcefile: options.path,
@@ -1207,7 +1207,7 @@ export class Build {
                     if (typeof entryPoint !== 'object') {
                         return;
                     }
-                    if (entryPoint.contents) {
+                    if (entryPoint.contents != null) {
                         build.addVirtualModule(entryPoint);
                     }
                 });
