always lower template literals containing "</script>"

CHANGELOG.md

@@ -32,11 +32,12 @@
 
 * Avoid generating the character sequence `</script>` ([#1322](https://github.com/evanw/esbuild/issues/1322))
 
-    If the output of esbuild is inlined into a `<script>...</script>` tag inside an HTML file, the character sequence `</script>` inside the JavaScript code will accidentally cause the script tag to be terminated early. There are at least three such cases where this can happen:
+    If the output of esbuild is inlined into a `<script>...</script>` tag inside an HTML file, the character sequence `</script>` inside the JavaScript code will accidentally cause the script tag to be terminated early. There are at least four such cases where this can happen:
 
     ```js
     console.log('</script>')
     console.log(1</script>/.exec(x).length)
+    console.log(String.raw`</script>`)
     // @license </script>
     ```
 
@@ -45,11 +46,10 @@
     ```js
     console.log('<\/script>');
     console.log(1< /script>/.exec(x).length);
-    // @license <∕script>
+    console.log(String.raw(__template(["<\/script>"], ["<\/script>"])));
+    // @license <\/script>
     ```
 
-    The `@license` comment uses a different slash character. I'm not sure how to handle that one since it's meant for humans but that seemed like the least intrusive method. I expect this case to not ever come up in practice anyway.
-
 ## 0.12.4
 
 * Reorder name preservation before TypeScript decorator evaluation ([#1316](https://github.com/evanw/esbuild/issues/1316))

internal/js_parser/js_parser.go

@@ -10724,8 +10724,25 @@ func (p *parser) visitExprInOut(expr js_ast.Expr, in exprIn) (js_ast.Expr, exprO
 			expr = p.mangleTemplate(expr.Loc, e)
 		}
 
+		shouldLowerTemplateLiteral := p.options.unsupportedJSFeatures.Has(compat.TemplateLiteral)
+
+		// Lower tagged template literals that include "</script>"
+		// since we won't be able to escape it without lowering it
+		if !shouldLowerTemplateLiteral && e.TagOrNil.Data != nil {
+			if strings.Contains(e.HeadRaw, "</script>") {
+				shouldLowerTemplateLiteral = true
+			} else {
+				for _, part := range e.Parts {
+					if strings.Contains(part.TailRaw, "</script>") {
+						shouldLowerTemplateLiteral = true
+						break
+					}
+				}
+			}
+		}
+
 		// Convert template literals to older syntax if this is still a template literal
-		if p.options.unsupportedJSFeatures.Has(compat.TemplateLiteral) {
+		if shouldLowerTemplateLiteral {
 			if e, ok := expr.Data.(*js_ast.ETemplate); ok {
 				return p.lowerTemplateLiteral(expr.Loc, e), exprOut{}
 			}

internal/js_printer/js_printer.go

@@ -2710,7 +2710,7 @@ func (p *printer) printIf(s *js_ast.SIf) {
 
 func (p *printer) printIndentedComment(text string) {
 	// Avoid generating a comment containing the character sequence "</script>"
-	text = strings.ReplaceAll(text, "</script>", "<\u2215script>")
+	text = strings.ReplaceAll(text, "</script>", "<\\/script>")
 
 	if strings.HasPrefix(text, "/*") {
 		// Re-indent multi-line comments

internal/js_printer/js_printer_test.go

@@ -914,7 +914,9 @@ func TestAvoidSlashScript(t *testing.T) {
 	expectPrinted(t, "x = `${y}</script>`", "x = `${y}<\\/script>`;\n")
 	expectPrintedMinify(t, "x = 1 < /script>/.exec(y).length", "x=1< /script>/.exec(y).length;")
 	expectPrintedMinify(t, "x = 1 << /script>/.exec(y).length", "x=1<< /script>/.exec(y).length;")
-	expectPrinted(t, "//! </script>", "//! <\u2215script>\n")
+	expectPrinted(t, "//! </script>", "//! <\\/script>\n")
+	expectPrinted(t, "String.raw`</script>`",
+		"String.raw(__template([\"<\\/script>\"], [\"<\\/script>\"]));\nimport {\n  __template\n} from \"<runtime>\";\n")
 
 	// Negative cases
 	expectPrinted(t, "x = '</'", "x = \"</\";\n")