eslint-linter-browserify 8.44.0 → 8.45.0

package/linter.cjs

@@ -17038,7 +17038,7 @@ function requireLodash_merge () {
 }
 
 var name$1 = "eslint";
-var version$1 = "8.44.0";
+var version$1 = "8.45.0";
 var author = "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>";
 var description$3 = "An AST-based pattern checker for JavaScript.";
 var bin = {
@@ -17110,7 +17110,6 @@ var dependencies$3 = {
 	globals: "^13.19.0",
 	graphemer: "^1.4.0",
 	ignore: "^5.2.0",
-	"import-fresh": "^3.0.0",
 	imurmurhash: "^0.1.4",
 	"is-glob": "^4.0.0",
 	"is-path-inside": "^3.0.3",
@@ -17122,7 +17121,6 @@ var dependencies$3 = {
 	"natural-compare": "^1.4.0",
 	optionator: "^0.9.3",
 	"strip-ansi": "^6.0.1",
-	"strip-json-comments": "^3.1.0",
 	"text-table": "^0.2.0"
 };
 var devDependencies$1 = {
@@ -17181,7 +17179,6 @@ var devDependencies$1 = {
 	semver: "^7.5.3",
 	shelljs: "^0.8.2",
 	sinon: "^11.0.0",
-	temp: "^0.9.0",
 	webpack: "^5.23.0",
 	"webpack-cli": "^4.5.0",
 	yorkie: "^2.0.0"
@@ -39998,6 +39995,22 @@ function requireReportTranslator () {
 	    return descriptor.node.loc;
 	}
 
+	/**
+	 * Clones the given fix object.
+	 * @param {Fix|null} fix The fix to clone.
+	 * @returns {Fix|null} Deep cloned fix object or `null` if `null` or `undefined` was passed in.
+	 */
+	function cloneFix(fix) {
+	    if (!fix) {
+	        return null;
+	    }
+
+	    return {
+	        range: [fix.range[0], fix.range[1]],
+	        text: fix.text
+	    };
+	}
+
 	/**
 	 * Check that a fix has a valid range.
 	 * @param {Fix|null} fix The fix to validate.
@@ -40035,7 +40048,7 @@ function requireReportTranslator () {
 	        return null;
 	    }
 	    if (fixes.length === 1) {
-	        return fixes[0];
+	        return cloneFix(fixes[0]);
 	    }
 
 	    fixes.sort(compareFixesByRange);
@@ -40081,7 +40094,7 @@ function requireReportTranslator () {
 	    }
 
 	    assertValidFix(fix);
-	    return fix;
+	    return cloneFix(fix);
 	}
 
 	/**
@@ -41907,6 +41920,15 @@ function requireAstUtils () {
 
 		}
 
+		/**
+		 * Check whether the given node is a part of a directive prologue or not.
+		 * @param {ASTNode} node The node to check.
+		 * @returns {boolean} `true` if the node is a part of directive prologue.
+		 */
+		function isDirective(node) {
+		    return node.type === "ExpressionStatement" && typeof node.directive === "string";
+		}
+
 		//------------------------------------------------------------------------------
 		// Public Interface
 		//------------------------------------------------------------------------------
@@ -43059,7 +43081,8 @@ function requireAstUtils () {
 		    getSwitchCaseColonToken,
 		    getModuleExportName,
 		    isConstant,
-		    isTopLevelExpressionStatement
+		    isTopLevelExpressionStatement,
+		    isDirective
 		}; 
 	} (astUtils));
 	return astUtils.exports;
@@ -67272,7 +67295,7 @@ function requireIndent () {
 
 	            IfStatement(node) {
 	                addBlocklessNodeIndent(node.consequent);
-	                if (node.alternate && node.alternate.type !== "IfStatement") {
+	                if (node.alternate) {
 	                    addBlocklessNodeIndent(node.alternate);
 	                }
 	            },
@@ -101023,6 +101046,7 @@ function requireNoUselessEscape () {
 	        messages: {
 	            unnecessaryEscape: "Unnecessary escape character: \\{{character}}.",
 	            removeEscape: "Remove the `\\`. This maintains the current functionality.",
+	            removeEscapeDoNotKeepSemantics: "Remove the `\\` if it was inserted by mistake.",
 	            escapeBackslash: "Replace the `\\` with `\\\\` to include the actual backslash character."
 	        },
 
@@ -101054,7 +101078,10 @@ function requireNoUselessEscape () {
 	                data: { character },
 	                suggest: [
 	                    {
-	                        messageId: "removeEscape",
+
+	                        // Removing unnecessary `\` characters in a directive is not guaranteed to maintain functionality.
+	                        messageId: astUtils.isDirective(node.parent)
+	                            ? "removeEscapeDoNotKeepSemantics" : "removeEscape",
 	                        fix(fixer) {
 	                            return fixer.removeRange(range);
 	                        }
@@ -105513,42 +105540,6 @@ function requirePaddingLineBetweenStatements () {
 	    );
 	}
 
-	/**
-	 * Check whether the given node is a directive or not.
-	 * @param {ASTNode} node The node to check.
-	 * @param {SourceCode} sourceCode The source code object to get tokens.
-	 * @returns {boolean} `true` if the node is a directive.
-	 */
-	function isDirective(node, sourceCode) {
-	    return (
-	        astUtils.isTopLevelExpressionStatement(node) &&
-	        node.expression.type === "Literal" &&
-	        typeof node.expression.value === "string" &&
-	        !astUtils.isParenthesised(sourceCode, node.expression)
-	    );
-	}
-
-	/**
-	 * Check whether the given node is a part of directive prologue or not.
-	 * @param {ASTNode} node The node to check.
-	 * @param {SourceCode} sourceCode The source code object to get tokens.
-	 * @returns {boolean} `true` if the node is a part of directive prologue.
-	 */
-	function isDirectivePrologue(node, sourceCode) {
-	    if (isDirective(node, sourceCode)) {
-	        for (const sibling of node.parent.body) {
-	            if (sibling === node) {
-	                break;
-	            }
-	            if (!isDirective(sibling, sourceCode)) {
-	                return false;
-	            }
-	        }
-	        return true;
-	    }
-	    return false;
-	}
-
 	/**
 	 * Gets the actual last token.
 	 *
@@ -105742,12 +105733,10 @@ function requirePaddingLineBetweenStatements () {
 	            CJS_IMPORT.test(sourceCode.getText(node.declarations[0].init))
 	    },
 	    directive: {
-	        test: isDirectivePrologue
+	        test: astUtils.isDirective
 	    },
 	    expression: {
-	        test: (node, sourceCode) =>
-	            node.type === "ExpressionStatement" &&
-	            !isDirectivePrologue(node, sourceCode)
+	        test: node => node.type === "ExpressionStatement" && !astUtils.isDirective(node)
 	    },
 	    iife: {
 	        test: isIIFEStatement
@@ -105758,10 +105747,10 @@ function requirePaddingLineBetweenStatements () {
 	            isBlockLikeStatement(sourceCode, node)
 	    },
 	    "multiline-expression": {
-	        test: (node, sourceCode) =>
+	        test: node =>
 	            node.loc.start.line !== node.loc.end.line &&
 	            node.type === "ExpressionStatement" &&
-	            !isDirectivePrologue(node, sourceCode)
+	            !astUtils.isDirective(node)
 	    },
 
 	    "multiline-const": newMultilineKeywordTester("const"),

package/linter.js

@@ -17040,7 +17040,7 @@
     }
 
     var name$1 = "eslint";
-    var version$1 = "8.44.0";
+    var version$1 = "8.45.0";
     var author = "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>";
     var description$3 = "An AST-based pattern checker for JavaScript.";
     var bin = {
@@ -17112,7 +17112,6 @@
     	globals: "^13.19.0",
     	graphemer: "^1.4.0",
     	ignore: "^5.2.0",
-    	"import-fresh": "^3.0.0",
     	imurmurhash: "^0.1.4",
     	"is-glob": "^4.0.0",
     	"is-path-inside": "^3.0.3",
@@ -17124,7 +17123,6 @@
     	"natural-compare": "^1.4.0",
     	optionator: "^0.9.3",
     	"strip-ansi": "^6.0.1",
-    	"strip-json-comments": "^3.1.0",
     	"text-table": "^0.2.0"
     };
     var devDependencies$1 = {
@@ -17183,7 +17181,6 @@
     	semver: "^7.5.3",
     	shelljs: "^0.8.2",
     	sinon: "^11.0.0",
-    	temp: "^0.9.0",
     	webpack: "^5.23.0",
     	"webpack-cli": "^4.5.0",
     	yorkie: "^2.0.0"
@@ -40000,6 +39997,22 @@
     	    return descriptor.node.loc;
     	}
 
+    	/**
+    	 * Clones the given fix object.
+    	 * @param {Fix|null} fix The fix to clone.
+    	 * @returns {Fix|null} Deep cloned fix object or `null` if `null` or `undefined` was passed in.
+    	 */
+    	function cloneFix(fix) {
+    	    if (!fix) {
+    	        return null;
+    	    }
+
+    	    return {
+    	        range: [fix.range[0], fix.range[1]],
+    	        text: fix.text
+    	    };
+    	}
+
     	/**
     	 * Check that a fix has a valid range.
     	 * @param {Fix|null} fix The fix to validate.
@@ -40037,7 +40050,7 @@
     	        return null;
     	    }
     	    if (fixes.length === 1) {
-    	        return fixes[0];
+    	        return cloneFix(fixes[0]);
     	    }
 
     	    fixes.sort(compareFixesByRange);
@@ -40083,7 +40096,7 @@
     	    }
 
     	    assertValidFix(fix);
-    	    return fix;
+    	    return cloneFix(fix);
     	}
 
     	/**
@@ -41909,6 +41922,15 @@
 
     		}
 
+    		/**
+    		 * Check whether the given node is a part of a directive prologue or not.
+    		 * @param {ASTNode} node The node to check.
+    		 * @returns {boolean} `true` if the node is a part of directive prologue.
+    		 */
+    		function isDirective(node) {
+    		    return node.type === "ExpressionStatement" && typeof node.directive === "string";
+    		}
+
     		//------------------------------------------------------------------------------
     		// Public Interface
     		//------------------------------------------------------------------------------
@@ -43061,7 +43083,8 @@
     		    getSwitchCaseColonToken,
     		    getModuleExportName,
     		    isConstant,
-    		    isTopLevelExpressionStatement
+    		    isTopLevelExpressionStatement,
+    		    isDirective
     		}; 
     	} (astUtils));
     	return astUtils.exports;
@@ -67274,7 +67297,7 @@
 
     	            IfStatement(node) {
     	                addBlocklessNodeIndent(node.consequent);
-    	                if (node.alternate && node.alternate.type !== "IfStatement") {
+    	                if (node.alternate) {
     	                    addBlocklessNodeIndent(node.alternate);
     	                }
     	            },
@@ -101025,6 +101048,7 @@
     	        messages: {
     	            unnecessaryEscape: "Unnecessary escape character: \\{{character}}.",
     	            removeEscape: "Remove the `\\`. This maintains the current functionality.",
+    	            removeEscapeDoNotKeepSemantics: "Remove the `\\` if it was inserted by mistake.",
     	            escapeBackslash: "Replace the `\\` with `\\\\` to include the actual backslash character."
     	        },
 
@@ -101056,7 +101080,10 @@
     	                data: { character },
     	                suggest: [
     	                    {
-    	                        messageId: "removeEscape",
+
+    	                        // Removing unnecessary `\` characters in a directive is not guaranteed to maintain functionality.
+    	                        messageId: astUtils.isDirective(node.parent)
+    	                            ? "removeEscapeDoNotKeepSemantics" : "removeEscape",
     	                        fix(fixer) {
     	                            return fixer.removeRange(range);
     	                        }
@@ -105515,42 +105542,6 @@
     	    );
     	}
 
-    	/**
-    	 * Check whether the given node is a directive or not.
-    	 * @param {ASTNode} node The node to check.
-    	 * @param {SourceCode} sourceCode The source code object to get tokens.
-    	 * @returns {boolean} `true` if the node is a directive.
-    	 */
-    	function isDirective(node, sourceCode) {
-    	    return (
-    	        astUtils.isTopLevelExpressionStatement(node) &&
-    	        node.expression.type === "Literal" &&
-    	        typeof node.expression.value === "string" &&
-    	        !astUtils.isParenthesised(sourceCode, node.expression)
-    	    );
-    	}
-
-    	/**
-    	 * Check whether the given node is a part of directive prologue or not.
-    	 * @param {ASTNode} node The node to check.
-    	 * @param {SourceCode} sourceCode The source code object to get tokens.
-    	 * @returns {boolean} `true` if the node is a part of directive prologue.
-    	 */
-    	function isDirectivePrologue(node, sourceCode) {
-    	    if (isDirective(node, sourceCode)) {
-    	        for (const sibling of node.parent.body) {
-    	            if (sibling === node) {
-    	                break;
-    	            }
-    	            if (!isDirective(sibling, sourceCode)) {
-    	                return false;
-    	            }
-    	        }
-    	        return true;
-    	    }
-    	    return false;
-    	}
-
     	/**
     	 * Gets the actual last token.
     	 *
@@ -105744,12 +105735,10 @@
     	            CJS_IMPORT.test(sourceCode.getText(node.declarations[0].init))
     	    },
     	    directive: {
-    	        test: isDirectivePrologue
+    	        test: astUtils.isDirective
     	    },
     	    expression: {
-    	        test: (node, sourceCode) =>
-    	            node.type === "ExpressionStatement" &&
-    	            !isDirectivePrologue(node, sourceCode)
+    	        test: node => node.type === "ExpressionStatement" && !astUtils.isDirective(node)
     	    },
     	    iife: {
     	        test: isIIFEStatement
@@ -105760,10 +105749,10 @@
     	            isBlockLikeStatement(sourceCode, node)
     	    },
     	    "multiline-expression": {
-    	        test: (node, sourceCode) =>
+    	        test: node =>
     	            node.loc.start.line !== node.loc.end.line &&
     	            node.type === "ExpressionStatement" &&
-    	            !isDirectivePrologue(node, sourceCode)
+    	            !astUtils.isDirective(node)
     	    },
 
     	    "multiline-const": newMultilineKeywordTester("const"),