fix: preserve comments and whitespaces inside expressions

Author: som-sm

rules/prefer-string-raw.js

@@ -1,5 +1,5 @@
 import {isStringLiteral, isDirective} from './ast/index.js';
-import {fixSpaceAroundKeyword} from './fix/index.js';
+import {fixSpaceAroundKeyword, replaceTemplateElement} from './fix/index.js';
 
 const MESSAGE_ID = 'prefer-string-raw';
 const messages = {
@@ -67,11 +67,9 @@ const create = context => {
 			return;
 		}
 
-		let unescaped = '';
 		let hasBackslash = false;
 
-		for (let index = 0; index < node.quasis.length; index++) {
-			const quasi = node.quasis[index];
+		for (const quasi of node.quasis) {
 			const {raw, cooked} = quasi.value;
 
 			if (cooked.at(-1) === BACKSLASH) {
@@ -86,12 +84,6 @@ const create = context => {
 			if (cooked.includes(BACKSLASH)) {
 				hasBackslash = true;
 			}
-
-			if (index > 0) {
-				unescaped += '${' + context.sourceCode.getText(node.expressions[index - 1]) + '}';
-			}
-
-			unescaped += unescapedQuasi;
 		}
 
 		if (!hasBackslash) {
@@ -102,8 +94,9 @@ const create = context => {
 			node,
 			messageId: MESSAGE_ID,
 			* fix(fixer) {
-				yield fixer.replaceText(node, `String.raw\`${unescaped}\``);
 				yield * fixSpaceAroundKeyword(fixer, node, context.sourceCode);
+				yield * node.quasis.map(quasi => replaceTemplateElement(fixer, quasi, quasi.value.cooked));
+				yield fixer.insertTextBefore(node, 'String.raw');
 			},
 		};
 	});

test/prefer-string-raw.js

@@ -1,3 +1,4 @@
+
 /* eslint-disable no-template-curly-in-string */
 import outdent from 'outdent';
 import {getTester} from './utils/test.js';
@@ -88,6 +89,9 @@ test.snapshot({
 			b\${bar}c
 			d\\\\\\\\e\`
 		`,
+		'a = `a\\\\b${ foo /* bar */}c\\\\d`',
+		'a = `a\\\\b${ foo + bar }`',
+		'a = `${ foo .bar }a\\\\b`',
 	],
 });
 

test/snapshots/prefer-string-raw.js.md

@@ -295,3 +295,66 @@ Generated by [AVA](https://avajs.dev).
     > 3 | d\\\\\\\\e\`␊
         | ^^^^^^^^ \`String.raw\` should be used to avoid escaping \`\\\`.␊
     `
+
+## invalid(8): a = `a\\b${ foo /* bar */}c\\d`
+
+> Input
+
+    `␊
+      1 | a = \`a\\\\b${ foo /* bar */}c\\\\d\`␊
+    `
+
+> Output
+
+    `␊
+      1 | a = String.raw\`a\\b${ foo /* bar */}c\\d\`␊
+    `
+
+> Error 1/1
+
+    `␊
+    > 1 | a = \`a\\\\b${ foo /* bar */}c\\\\d\`␊
+        |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^ \`String.raw\` should be used to avoid escaping \`\\\`.␊
+    `
+
+## invalid(9): a = `a\\b${ foo + bar }`
+
+> Input
+
+    `␊
+      1 | a = \`a\\\\b${ foo + bar }\`␊
+    `
+
+> Output
+
+    `␊
+      1 | a = String.raw\`a\\b${ foo + bar }\`␊
+    `
+
+> Error 1/1
+
+    `␊
+    > 1 | a = \`a\\\\b${ foo + bar }\`␊
+        |     ^^^^^^^^^^^^^^^^^^^^ \`String.raw\` should be used to avoid escaping \`\\\`.␊
+    `
+
+## invalid(10): a = `${ foo .bar }a\\b`
+
+> Input
+
+    `␊
+      1 | a = \`${ foo .bar }a\\\\b\`␊
+    `
+
+> Output
+
+    `␊
+      1 | a = String.raw\`${ foo .bar }a\\b\`␊
+    `
+
+> Error 1/1
+
+    `␊
+    > 1 | a = \`${ foo .bar }a\\\\b\`␊
+        |     ^^^^^^^^^^^^^^^^^^^ \`String.raw\` should be used to avoid escaping \`\\\`.␊
+    `