From 9ebf36b5719285fe4033256b2d78c729e48d1be3 Mon Sep 17 00:00:00 2001
From: PozzettiAndrea <andrea@pozzetti.it>
Date: Thu, 13 Aug 2026 11:41:01 +0200
Subject: [PATCH] Rename viewer bundles .js -> .mjs so they stop leaking into
 the main realm

ComfyUI auto-imports every web/**/*.js into the shared main page; the four
big bundles (three, vtk-gltf, viewer-bundle-vtk, gsplat) only ever run
inside the viewer iframes but were getting pulled into the main document
too, where they wrote window.THREE / window.vtk / globalThis.* and could
collide with any other 3D pack. The **/*.js glob only matches .js, so
renaming to .mjs drops them from the auto-import while the iframes still
load them via classic <script src> (mime resolves to text/javascript).
window.* pollution from these bundles is gone from the main realm.
---
 comfy-test.toml                               |  2 +-
 .../{gsplat-bundle.js => gsplat-bundle.mjs}   |  0
 ...undle-three.js => viewer-bundle-three.mjs} |  0
 ...er-bundle-vtk.js => viewer-bundle-vtk.mjs} |  0
 javascript/js/{vtk-gltf.js => vtk-gltf.mjs}   |  0
 javascript/viewer_dual.html                   |  2 +-
 javascript/viewer_dual_slider.html            |  4 +-
 javascript/viewer_dual_textured.html          |  2 +-
 javascript/viewer_fbx.html                    |  6 +--
 javascript/viewer_fbx_compare.html            |  2 +-
 javascript/viewer_fbx_debug.html              |  2 +-
 javascript/viewer_gaussian.html               |  2 +-
 javascript/viewer_multi.html                  |  2 +-
 javascript/viewer_multi_slider.html           |  4 +-
 javascript/viewer_slicer.html                 |  2 +-
 javascript/viewer_vtk.html                    |  4 +-
 javascript/viewer_vtk_textured.html           |  4 +-
 totest.md                                     | 48 -------------------
 18 files changed, 19 insertions(+), 67 deletions(-)
 rename javascript/js/{gsplat-bundle.js => gsplat-bundle.mjs} (100%)
 rename javascript/js/{viewer-bundle-three.js => viewer-bundle-three.mjs} (100%)
 rename javascript/js/{viewer-bundle-vtk.js => viewer-bundle-vtk.mjs} (100%)
 rename javascript/js/{vtk-gltf.js => vtk-gltf.mjs} (100%)
 delete mode 100644 totest.md

diff --git a/comfy-test.toml b/comfy-test.toml
index 3ba235fb..b508690e 100644
--- a/comfy-test.toml
+++ b/comfy-test.toml
@@ -1,5 +1,5 @@
 [test]
-levels = ["syntax", "coverage", "install", "registration", "instantiation", "execution"]
+levels = ["syntax", "coverage", "install", "javascript", "registration", "instantiation", "execution"]
 publish = true
 
 [test.workflows]
diff --git a/javascript/js/gsplat-bundle.js b/javascript/js/gsplat-bundle.mjs
similarity index 100%
rename from javascript/js/gsplat-bundle.js
rename to javascript/js/gsplat-bundle.mjs
diff --git a/javascript/js/viewer-bundle-three.js b/javascript/js/viewer-bundle-three.mjs
similarity index 100%
rename from javascript/js/viewer-bundle-three.js
rename to javascript/js/viewer-bundle-three.mjs
diff --git a/javascript/js/viewer-bundle-vtk.js b/javascript/js/viewer-bundle-vtk.mjs
similarity index 100%
rename from javascript/js/viewer-bundle-vtk.js
rename to javascript/js/viewer-bundle-vtk.mjs
diff --git a/javascript/js/vtk-gltf.js b/javascript/js/vtk-gltf.mjs
similarity index 100%
rename from javascript/js/vtk-gltf.js
rename to javascript/js/vtk-gltf.mjs
diff --git a/javascript/viewer_dual.html b/javascript/viewer_dual.html
index 4ad71c32..c9e6f371 100644
--- a/javascript/viewer_dual.html
+++ b/javascript/viewer_dual.html
@@ -350,7 +350,7 @@
         </div>
     </div>
 
-    <script src="./js/vtk-gltf.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
 
     <script>
         // Get VTK.js classes
diff --git a/javascript/viewer_dual_slider.html b/javascript/viewer_dual_slider.html
index d92fb1f3..ac10604c 100644
--- a/javascript/viewer_dual_slider.html
+++ b/javascript/viewer_dual_slider.html
@@ -341,8 +341,8 @@
         </label>
     </div>
 
-    <script src="./js/vtk-gltf.js"></script>
-    <script src="./js/viewer-bundle-vtk.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
+    <script src="./js/viewer-bundle-vtk.mjs"></script>
 
     <script>
         // Get VTK.js classes
diff --git a/javascript/viewer_dual_textured.html b/javascript/viewer_dual_textured.html
index 0d3bfaa7..814842a4 100644
--- a/javascript/viewer_dual_textured.html
+++ b/javascript/viewer_dual_textured.html
@@ -330,7 +330,7 @@
         </div>
     </div>
 
-    <script src="./js/vtk-gltf.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
 
     <script>
         // Get VTK.js classes
diff --git a/javascript/viewer_fbx.html b/javascript/viewer_fbx.html
index 5a69dac9..363b71b6 100644
--- a/javascript/viewer_fbx.html
+++ b/javascript/viewer_fbx.html
@@ -146,10 +146,10 @@
 
     <!--
     Load bundled Three.js and all dependencies.
-    The bundle is expected at ../js/viewer-bundle-three.js relative to this HTML file.
+    The bundle is expected at ../js/viewer-bundle-three.mjs relative to this HTML file.
     When copied by prestartup_script.py, the directory structure should be:
       web/viewer_fbx.html
-      web/js/viewer-bundle-three.js
+      web/js/viewer-bundle-three.mjs
     -->
     <script>
         // Dynamically determine the bundle path based on how this HTML is loaded
@@ -171,7 +171,7 @@
 
             // Create script element to load the bundle
             var bundleScript = document.createElement('script');
-            bundleScript.src = baseUrl + 'js/viewer-bundle-three.js';
+            bundleScript.src = baseUrl + 'js/viewer-bundle-three.mjs';
             bundleScript.onload = function() {
                 console.log('[FBX Viewer] Three.js bundle loaded from:', bundleScript.src);
                 initViewer();
diff --git a/javascript/viewer_fbx_compare.html b/javascript/viewer_fbx_compare.html
index a6466dde..2687fc99 100644
--- a/javascript/viewer_fbx_compare.html
+++ b/javascript/viewer_fbx_compare.html
@@ -293,7 +293,7 @@
             }
 
             var bundleScript = document.createElement('script');
-            bundleScript.src = baseUrl + 'js/viewer-bundle-three.js';
+            bundleScript.src = baseUrl + 'js/viewer-bundle-three.mjs';
             bundleScript.onload = function() {
                 console.log('[UniRig Compare] Three.js bundle loaded');
                 initViewer();
diff --git a/javascript/viewer_fbx_debug.html b/javascript/viewer_fbx_debug.html
index 9a6a994d..24e96179 100644
--- a/javascript/viewer_fbx_debug.html
+++ b/javascript/viewer_fbx_debug.html
@@ -439,7 +439,7 @@
             }
 
             var bundleScript = document.createElement('script');
-            bundleScript.src = baseUrl + 'js/viewer-bundle-three.js';
+            bundleScript.src = baseUrl + 'js/viewer-bundle-three.mjs';
             bundleScript.onload = function() {
                 console.log('[FBX Debug Viewer] Three.js bundle loaded from:', bundleScript.src);
                 initViewer();
diff --git a/javascript/viewer_gaussian.html b/javascript/viewer_gaussian.html
index 2c2b7460..6e829b67 100644
--- a/javascript/viewer_gaussian.html
+++ b/javascript/viewer_gaussian.html
@@ -133,7 +133,7 @@
     </div>
 
     <!-- Load bundled gsplat.js -->
-    <script src="./js/gsplat-bundle.js"></script>
+    <script src="./js/gsplat-bundle.mjs"></script>
 
     <script type="module">
         // Use the globally exported GSPLAT
diff --git a/javascript/viewer_multi.html b/javascript/viewer_multi.html
index e7089db0..8a205fe6 100644
--- a/javascript/viewer_multi.html
+++ b/javascript/viewer_multi.html
@@ -297,7 +297,7 @@
         </div>
     </div>
 
-    <script src="./js/vtk-gltf.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
 
     <script>
         // Get VTK.js classes
diff --git a/javascript/viewer_multi_slider.html b/javascript/viewer_multi_slider.html
index 32f610ff..953d54b4 100644
--- a/javascript/viewer_multi_slider.html
+++ b/javascript/viewer_multi_slider.html
@@ -81,8 +81,8 @@
         <button id="resetSlider">Reset Sliders</button>
     </div>
 
-    <script src="./js/vtk-gltf.js"></script>
-    <script src="./js/viewer-bundle-vtk.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
+    <script src="./js/viewer-bundle-vtk.mjs"></script>
 
     <script>
         const vtkFullScreenRenderWindow = vtk.Rendering.Misc.vtkFullScreenRenderWindow;
diff --git a/javascript/viewer_slicer.html b/javascript/viewer_slicer.html
index 09905d41..ec4d3983 100644
--- a/javascript/viewer_slicer.html
+++ b/javascript/viewer_slicer.html
@@ -16,7 +16,7 @@
         button.active { background: #2c5d8a; border-color: #4a90c0; }
         button:hover { background: #444; }
     </style>
-    <script src="./js/vtk-gltf.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
 </head>
 <body>
 <div id="wrap">
diff --git a/javascript/viewer_vtk.html b/javascript/viewer_vtk.html
index d649d72b..13f0a817 100644
--- a/javascript/viewer_vtk.html
+++ b/javascript/viewer_vtk.html
@@ -151,9 +151,9 @@
     </div>
 
     <!-- Load VTK.js bundle -->
-    <script src="./js/vtk-gltf.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
     <!-- Load unified viewer bundle -->
-    <script src="./js/viewer-bundle-vtk.js"></script>
+    <script src="./js/viewer-bundle-vtk.mjs"></script>
 
     <script>
         // Use unified viewer modules
diff --git a/javascript/viewer_vtk_textured.html b/javascript/viewer_vtk_textured.html
index 5973156f..c2246250 100644
--- a/javascript/viewer_vtk_textured.html
+++ b/javascript/viewer_vtk_textured.html
@@ -96,9 +96,9 @@
     </div>
 
     <!-- Load VTK.js bundle with GLTFImporter support -->
-    <script src="./js/vtk-gltf.js"></script>
+    <script src="./js/vtk-gltf.mjs"></script>
     <!-- Load unified viewer bundle -->
-    <script src="./js/viewer-bundle-vtk.js"></script>
+    <script src="./js/viewer-bundle-vtk.mjs"></script>
 
     <script>
         // Use the unified viewer module
diff --git a/totest.md b/totest.md
deleted file mode 100644
index 9c6fc710..00000000
--- a/totest.md
+++ /dev/null
@@ -1,48 +0,0 @@
-# Nodes Without Workflow Coverage
-
-35 of 74 nodes (~47%) have no workflow JSON. Checked against `workflows/*.json`.
-
-## IO (5 missing)
-- [ ] `GeomPackLoadMeshPath` — Load Mesh (Path)
-- [ ] `GeomPackLoadMeshGlob` — Load Mesh Batch (Glob)
-- [ ] `GeomPackSaveMesh` — Save Mesh
-- [ ] `GeomPackSaveMeshBatch` — Save Meshes to Folder
-- [ ] `GeomPackGetMeshFilename` — Get Mesh Filename
-
-## Analysis (4 missing)
-- [ ] `GeomPackConnectedComponents` — Connected Components
-- [ ] `GeomPackOpenEdges` — Open Edges
-- [ ] `GeomPackDegenerateFaces` — Degenerate Faces
-- [ ] `ScrambleIntField` — Scramble Int Field
-
-## Conversion (1 missing)
-- [ ] `GeomPackSubsamplePointCloud` — Subsample Point Cloud
-
-## Transforms (1 missing)
-- [ ] `GeomPackNormalizeMeshToBBox` — Normalize to BBox
-
-## Visualization (3 missing)
-- [ ] `GeomPackBackdraftView` — Backdraft View
-- [ ] `GeomPackPreviewGaussian` — Preview Gaussian
-- [ ] `GeomPackPreviewMeshAnalysis` — Preview Mesh (Analysis)
-
-## Repair (6 missing)
-- [ ] `GeomPackAddNormalsToPointCloud` — Add Normals to PointCloud
-- [ ] `GeomPackVisualizeNormalField` — Visualize Normal Field
-- [ ] `GeomPackMergeVertices` — Merge Vertices
-- [ ] `GeomPackFixSelfIntersectionsByRemoval` — Fix Self Intersections (Removal)
-- [ ] `GeomPackRemoveDegenerateFaces` — Remove Degenerate Faces
-- [ ] `GeomPackMeshFix` — MeshFix
-
-## Texture Remeshing (1 missing)
-- [ ] `GeomPackDepthNormalsToMesh` — Depth + Normals to Mesh
-
-## Combine (2 missing)
-- [ ] `GeomPackCombineMeshesBatch` — Combine Meshes (Batch)
-- [ ] `GeomPackSplitByField` — Split By Field
-
-## Boolean (1 missing)
-- [ ] `GeomPackBooleanBlender` — Boolean Blender
-
-## ParaView (1 missing)
-- [ ] `GeomPackParaViewFilter` — ParaView Filter