123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125 |
- /**
- * @fileoverview Rule to disallow use of the `RegExp` constructor in favor of regular expression literals
- * @author Milos Djermanovic
- */
- "use strict";
- //------------------------------------------------------------------------------
- // Requirements
- //------------------------------------------------------------------------------
- const astUtils = require("./utils/ast-utils");
- const { CALL, CONSTRUCT, ReferenceTracker, findVariable } = require("eslint-utils");
- //------------------------------------------------------------------------------
- // Helpers
- //------------------------------------------------------------------------------
- /**
- * Determines whether the given node is a string literal.
- * @param {ASTNode} node Node to check.
- * @returns {boolean} True if the node is a string literal.
- */
- function isStringLiteral(node) {
- return node.type === "Literal" && typeof node.value === "string";
- }
- /**
- * Determines whether the given node is a template literal without expressions.
- * @param {ASTNode} node Node to check.
- * @returns {boolean} True if the node is a template literal without expressions.
- */
- function isStaticTemplateLiteral(node) {
- return node.type === "TemplateLiteral" && node.expressions.length === 0;
- }
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
- module.exports = {
- meta: {
- type: "suggestion",
- docs: {
- description: "disallow use of the `RegExp` constructor in favor of regular expression literals",
- category: "Best Practices",
- recommended: false,
- url: "https://eslint.org/docs/rules/prefer-regex-literals"
- },
- schema: [],
- messages: {
- unexpectedRegExp: "Use a regular expression literal instead of the 'RegExp' constructor."
- }
- },
- create(context) {
- /**
- * Determines whether the given identifier node is a reference to a global variable.
- * @param {ASTNode} node `Identifier` node to check.
- * @returns {boolean} True if the identifier is a reference to a global variable.
- */
- function isGlobalReference(node) {
- const scope = context.getScope();
- const variable = findVariable(scope, node);
- return variable !== null && variable.scope.type === "global" && variable.defs.length === 0;
- }
- /**
- * Determines whether the given node is a String.raw`` tagged template expression
- * with a static template literal.
- * @param {ASTNode} node Node to check.
- * @returns {boolean} True if the node is String.raw`` with a static template.
- */
- function isStringRawTaggedStaticTemplateLiteral(node) {
- return node.type === "TaggedTemplateExpression" &&
- node.tag.type === "MemberExpression" &&
- node.tag.object.type === "Identifier" &&
- node.tag.object.name === "String" &&
- isGlobalReference(node.tag.object) &&
- astUtils.getStaticPropertyName(node.tag) === "raw" &&
- isStaticTemplateLiteral(node.quasi);
- }
- /**
- * Determines whether the given node is considered to be a static string by the logic of this rule.
- * @param {ASTNode} node Node to check.
- * @returns {boolean} True if the node is a static string.
- */
- function isStaticString(node) {
- return isStringLiteral(node) ||
- isStaticTemplateLiteral(node) ||
- isStringRawTaggedStaticTemplateLiteral(node);
- }
- return {
- Program() {
- const scope = context.getScope();
- const tracker = new ReferenceTracker(scope);
- const traceMap = {
- RegExp: {
- [CALL]: true,
- [CONSTRUCT]: true
- }
- };
- for (const { node } of tracker.iterateGlobalReferences(traceMap)) {
- const args = node.arguments;
- if (
- (args.length === 1 || args.length === 2) &&
- args.every(isStaticString)
- ) {
- context.report({ node, messageId: "unexpectedRegExp" });
- }
- }
- }
- };
- }
- };
|