fix: prod build doesnt inject live preview styles are replace comments stripped by build

Author: abose

gulpfile.js/index.js

@@ -99,6 +99,12 @@ function makeJSDist() {
             mangle: false,
             compress: {
                 unused: false
+            },
+            preserveComments: function (node, comment) {
+                const text = (comment.value || "").trim();
+
+                // Preserve ONLY comments starting with "DONT_STRIP_MINIFY:"
+                return text.includes("DONT_STRIP_MINIFY:");
             }
         }))
         .pipe(sourcemaps.write('./'))
@@ -555,9 +561,7 @@ function inlineTextRequire(file, content, srcDir) {
                 textContentMap[requirePath] = fileContent;
                 textContent = fileContent;
             }
-            if(textContent.includes("`")) {
-                console.log("Not inlining file as it contains a backquote(`) :", requirePath);
-            } else if(requirePath.endsWith(".js") || requirePath.endsWith(".json")) {
+            if(requirePath.endsWith(".js") || requirePath.endsWith(".json")) {
                 console.log("Not inlining JS/JSON file:", requirePath);
             } else {
                 console.log("Inlining", requireStatement);
@@ -568,7 +572,7 @@ function inlineTextRequire(file, content, srcDir) {
                     throw `Error inlining ${requireStatement} in ${file}: Regex: ${detectedRegEx}`+
                     "\nRegular expression of the form /*/ is not allowed for minification please use RegEx constructor";
                 }
-                content = content.replaceAll(requireStatement, "`"+textContent+"`");
+                content = content.replaceAll(requireStatement, `${JSON.stringify(textContent)}`);
             }
         }
 

src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

@@ -103,7 +103,7 @@ function RemoteFunctions(config = {}) {
 
     let strings = {}, icons = {}, cssStyles = {}; // these will be injected below
     // the below code comment is replaced by added scripts for extensibility
-    // REPLACE_WITH_ADDED_REMOTE_CONSTANT_SCRIPTS
+    // DONT_STRIP_MINIFY:REPLACE_WITH_ADDED_REMOTE_CONSTANT_SCRIPTS
 
     /**
      * this function is responsible to auto scroll the live preview when
@@ -5298,7 +5298,7 @@ function RemoteFunctions(config = {}) {
 
     let customReturns = {};
     // the below code comment is replaced by added scripts for extensibility
-    // REPLACE_WITH_ADDED_REMOTE_SCRIPTS
+    // DONT_STRIP_MINIFY:REPLACE_WITH_ADDED_REMOTE_SCRIPTS
 
     return {
         ...customReturns,

src/LiveDevelopment/MultiBrowserImpl/protocol/LiveDevProtocol.js

@@ -343,8 +343,8 @@ define(function (require, exports, module) {
         const remoteScriptText = Array.from(remoteFunctionsScripts.values()).join("\n");
         const remoteConstantScriptText = Array.from(remoteFunctionsConstantsScripts.values()).join("\n");
         effectiveRemoteFunctionsScripts = RemoteFunctions
-            .replace("// REPLACE_WITH_ADDED_REMOTE_CONSTANT_SCRIPTS", remoteConstantScriptText)
-            .replace("// REPLACE_WITH_ADDED_REMOTE_SCRIPTS", remoteScriptText);
+            .replace("// DONT_STRIP_MINIFY:REPLACE_WITH_ADDED_REMOTE_CONSTANT_SCRIPTS", remoteConstantScriptText)
+            .replace("// DONT_STRIP_MINIFY:REPLACE_WITH_ADDED_REMOTE_SCRIPTS", remoteScriptText);
     }
 
     function addRemoteFunctionScript(scriptName, scriptText) {
@@ -353,7 +353,7 @@ define(function (require, exports, module) {
             return false;
         }
         remoteFunctionsScripts.set(scriptName, scriptText);
-        if(!RemoteFunctions.includes("// REPLACE_WITH_ADDED_REMOTE_SCRIPTS")){
+        if(!RemoteFunctions.includes("// DONT_STRIP_MINIFY:REPLACE_WITH_ADDED_REMOTE_SCRIPTS")){
             throw new Error("RemoteFunctions script is missing the placeholder // REPLACE_WITH_ADDED_REMOTE_SCRIPTS");
         }
         _computeRemoteScript();
@@ -366,7 +366,7 @@ define(function (require, exports, module) {
             return false;
         }
         remoteFunctionsConstantsScripts.set(scriptName, scriptText);
-        if(!RemoteFunctions.includes("// REPLACE_WITH_ADDED_REMOTE_CONSTANT_SCRIPTS")){
+        if(!RemoteFunctions.includes("// DONT_STRIP_MINIFY:REPLACE_WITH_ADDED_REMOTE_CONSTANT_SCRIPTS")){
             throw new Error("RemoteFunctions script missing placeholder // REPLACE_WITH_ADDED_REMOTE_CONSTANT_SCRIPTS");
         }
         _computeRemoteScript();