immosquare-cleaner 0.1.39 → 0.1.41

data/node_modules/eslint/lib/{source-code → languages/js/source-code}/source-code.js

@@ -11,18 +11,16 @@
 const
     { isCommentToken } = require("@eslint-community/eslint-utils"),
     TokenStore = require("./token-store"),
-    astUtils = require("../shared/ast-utils"),
-    Traverser = require("../shared/traverser"),
-    globals = require("../../conf/globals"),
+    astUtils = require("../../../shared/ast-utils"),
+    Traverser = require("../../../shared/traverser"),
+    globals = require("../../../../conf/globals"),
     {
         directivesPattern
-    } = require("../shared/directives"),
+    } = require("../../../shared/directives"),
 
-    /* eslint-disable n/no-restricted-require -- Should eventually be moved into SourceCode. */
-    CodePathAnalyzer = require("../linter/code-path-analysis/code-path-analyzer"),
-    createEmitter = require("../linter/safe-emitter"),
-    ConfigCommentParser = require("../linter/config-comment-parser"),
-    /* eslint-enable n/no-restricted-require -- Should eventually be moved into SourceCode. */
+    CodePathAnalyzer = require("../../../linter/code-path-analysis/code-path-analyzer"),
+    createEmitter = require("../../../linter/safe-emitter"),
+    ConfigCommentParser = require("../../../linter/config-comment-parser"),
 
     eslintScope = require("eslint-scope");
 
@@ -31,6 +29,10 @@ const
 //------------------------------------------------------------------------------
 
 /** @typedef {import("eslint-scope").Variable} Variable */
+/** @typedef {import("eslint-scope").Scope} Scope */
+/** @typedef {import("@eslint/core").SourceCode} ISourceCode */
+/** @typedef {import("@eslint/core").Directive} IDirective */
+/** @typedef {import("@eslint/core").TraversalStep} ITraversalStep */
 
 //------------------------------------------------------------------------------
 // Private
@@ -375,6 +377,7 @@ class TraversalStep {
 
 /**
  * A class to represent a directive comment.
+ * @implements {IDirective}
  */
 class Directive {
 
@@ -431,34 +434,39 @@ const caches = Symbol("caches");
 
 /**
  * Represents parsed source code.
+ * @implements {ISourceCode}
  */
 class SourceCode extends TokenStore {
 
     /**
      * The cache of steps that were taken while traversing the source code.
-     * @type {Array<TraversalStep>}
+     * @type {Array<ITraversalStep>}
      */
     #steps;
 
     /**
+     * Creates a new instance.
      * @param {string|Object} textOrConfig The source code text or config object.
      * @param {string} textOrConfig.text The source code text.
      * @param {ASTNode} textOrConfig.ast The Program node of the AST representing the code. This AST should be created from the text that BOM was stripped.
+     * @param {boolean} textOrConfig.hasBOM Indicates if the text has a Unicode BOM.
      * @param {Object|null} textOrConfig.parserServices The parser services.
      * @param {ScopeManager|null} textOrConfig.scopeManager The scope of this source code.
      * @param {Object|null} textOrConfig.visitorKeys The visitor keys to traverse AST.
      * @param {ASTNode} [astIfNoConfig] The Program node of the AST representing the code. This AST should be created from the text that BOM was stripped.
      */
     constructor(textOrConfig, astIfNoConfig) {
-        let text, ast, parserServices, scopeManager, visitorKeys;
+        let text, hasBOM, ast, parserServices, scopeManager, visitorKeys;
 
-        // Process overloading.
+        // Process overloading of arguments
         if (typeof textOrConfig === "string") {
             text = textOrConfig;
             ast = astIfNoConfig;
+            hasBOM = false;
         } else if (typeof textOrConfig === "object" && textOrConfig !== null) {
             text = textOrConfig.text;
             ast = textOrConfig.ast;
+            hasBOM = textOrConfig.hasBOM;
             parserServices = textOrConfig.parserServices;
             scopeManager = textOrConfig.scopeManager;
             visitorKeys = textOrConfig.visitorKeys;
@@ -476,18 +484,39 @@ class SourceCode extends TokenStore {
             ["configNodes", void 0]
         ]);
 
+        /**
+         * Indicates if the AST is ESTree compatible.
+         * @type {boolean}
+         */
+        this.isESTree = ast.type === "Program";
+
+        /*
+         * Backwards compatibility for BOM handling.
+         *
+         * The `hasBOM` property has been available on the `SourceCode` object
+         * for a long time and is used to indicate if the source contains a BOM.
+         * The linter strips the BOM and just passes the `hasBOM` property to the
+         * `SourceCode` constructor to make it easier for languages to not deal with
+         * the BOM.
+         *
+         * However, the text passed in to the `SourceCode` constructor might still
+         * have a BOM if the constructor is called outside of the linter, so we still
+         * need to check for the BOM in the text.
+         */
+        const textHasBOM = text.charCodeAt(0) === 0xFEFF;
+
         /**
          * The flag to indicate that the source code has Unicode BOM.
          * @type {boolean}
          */
-        this.hasBOM = (text.charCodeAt(0) === 0xFEFF);
+        this.hasBOM = textHasBOM || !!hasBOM;
 
         /**
          * The original text source code.
          * BOM was stripped from this text.
          * @type {string}
          */
-        this.text = (this.hasBOM ? text.slice(1) : text);
+        this.text = (textHasBOM ? text.slice(1) : text);
 
         /**
          * The parsed AST for the source code.
@@ -815,7 +844,7 @@ class SourceCode extends TokenStore {
     /**
      * Gets the scope for the given node
      * @param {ASTNode} currentNode The node to get the scope of
-     * @returns {eslint-scope.Scope} The scope information for this node
+     * @returns {Scope} The scope information for this node
      * @throws {TypeError} If the `currentNode` argument is missing.
      */
     getScope(currentNode) {
@@ -1076,7 +1105,7 @@ class SourceCode extends TokenStore {
     /**
      * Applies configuration found inside of the source code. This method is only
      * called when ESLint is running with inline configuration allowed.
-     * @returns {{problems:Array<Problem>,configs:{config:FlatConfigArray,node:ASTNode}}} Information
+     * @returns {{problems:Array<Problem>,configs:{config:FlatConfigArray,loc:Location}}} Information
      *      that ESLint needs to further process the inline configuration.
      */
     applyInlineConfig() {
@@ -1124,17 +1153,21 @@ class SourceCode extends TokenStore {
                     break;
 
                 case "eslint": {
-                    const parseResult = commentParser.parseJsonConfig(directiveValue, comment.loc);
+                    const parseResult = commentParser.parseJsonConfig(directiveValue);
 
                     if (parseResult.success) {
                         configs.push({
                             config: {
                                 rules: parseResult.config
                             },
-                            node: comment
+                            loc: comment.loc
                         });
                     } else {
-                        problems.push(parseResult.error);
+                        problems.push({
+                            ruleId: null,
+                            loc: comment.loc,
+                            message: parseResult.error.message
+                        });
                     }
 
                     break;
@@ -1164,12 +1197,11 @@ class SourceCode extends TokenStore {
      */
     finalize() {
 
-        // Step 1: ensure that all of the necessary variables are up to date
         const varsCache = this[caches].get("vars");
-        const globalScope = this.scopeManager.scopes[0];
         const configGlobals = varsCache.get("configGlobals");
         const inlineGlobals = varsCache.get("inlineGlobals");
         const exportedVariables = varsCache.get("exportedVariables");
+        const globalScope = this.scopeManager.scopes[0];
 
         addDeclaredGlobals(globalScope, configGlobals, inlineGlobals);
 
@@ -1227,9 +1259,7 @@ class SourceCode extends TokenStore {
          * Program node at the top level. This is not a perfect heuristic, but it
          * is good enough for now.
          */
-        const isESTree = this.ast.type === "Program";
-
-        if (isESTree) {
+        if (this.isESTree) {
             analyzer = new CodePathAnalyzer(analyzer);
 
             CODE_PATH_EVENTS.forEach(eventName => {

data/node_modules/eslint/lib/languages/js/validate-language-options.js

@@ -0,0 +1,181 @@
+/**
+ * @fileoverview The schema to validate language options
+ * @author Nicholas C. Zakas
+ */
+
+"use strict";
+
+//-----------------------------------------------------------------------------
+// Data
+//-----------------------------------------------------------------------------
+
+const globalVariablesValues = new Set([
+    true, "true", "writable", "writeable",
+    false, "false", "readonly", "readable", null,
+    "off"
+]);
+
+//------------------------------------------------------------------------------
+// Helpers
+//------------------------------------------------------------------------------
+
+/**
+ * Check if a value is a non-null object.
+ * @param {any} value The value to check.
+ * @returns {boolean} `true` if the value is a non-null object.
+ */
+function isNonNullObject(value) {
+    return typeof value === "object" && value !== null;
+}
+
+/**
+ * Check if a value is a non-null non-array object.
+ * @param {any} value The value to check.
+ * @returns {boolean} `true` if the value is a non-null non-array object.
+ */
+function isNonArrayObject(value) {
+    return isNonNullObject(value) && !Array.isArray(value);
+}
+
+/**
+ * Check if a value is undefined.
+ * @param {any} value The value to check.
+ * @returns {boolean} `true` if the value is undefined.
+ */
+function isUndefined(value) {
+    return typeof value === "undefined";
+}
+
+//-----------------------------------------------------------------------------
+// Schemas
+//-----------------------------------------------------------------------------
+
+/**
+ * Validates the ecmaVersion property.
+ * @param {string|number} ecmaVersion The value to check.
+ * @returns {void}
+ * @throws {TypeError} If the value is invalid.
+ */
+function validateEcmaVersion(ecmaVersion) {
+
+    if (isUndefined(ecmaVersion)) {
+        throw new TypeError("Key \"ecmaVersion\": Expected an \"ecmaVersion\" property.");
+    }
+
+    if (typeof ecmaVersion !== "number" && ecmaVersion !== "latest") {
+        throw new TypeError("Key \"ecmaVersion\": Expected a number or \"latest\".");
+    }
+
+}
+
+/**
+ * Validates the sourceType property.
+ * @param {string} sourceType The value to check.
+ * @returns {void}
+ * @throws {TypeError} If the value is invalid.
+ */
+function validateSourceType(sourceType) {
+
+    if (typeof sourceType !== "string" || !/^(?:script|module|commonjs)$/u.test(sourceType)) {
+        throw new TypeError("Key \"sourceType\": Expected \"script\", \"module\", or \"commonjs\".");
+    }
+
+}
+
+/**
+ * Validates the globals property.
+ * @param {Object} globals The value to check.
+ * @returns {void}
+ * @throws {TypeError} If the value is invalid.
+ */
+function validateGlobals(globals) {
+
+    if (!isNonArrayObject(globals)) {
+        throw new TypeError("Key \"globals\": Expected an object.");
+    }
+
+    for (const key of Object.keys(globals)) {
+
+        // avoid hairy edge case
+        if (key === "__proto__") {
+            continue;
+        }
+
+        if (key !== key.trim()) {
+            throw new TypeError(`Key "globals": Global "${key}" has leading or trailing whitespace.`);
+        }
+
+        if (!globalVariablesValues.has(globals[key])) {
+            throw new TypeError(`Key "globals": Key "${key}": Expected "readonly", "writable", or "off".`);
+        }
+    }
+}
+
+/**
+ * Validates the parser property.
+ * @param {Object} parser The value to check.
+ * @returns {void}
+ * @throws {TypeError} If the value is invalid.
+ */
+function validateParser(parser) {
+
+    if (!parser || typeof parser !== "object" ||
+        (typeof parser.parse !== "function" && typeof parser.parseForESLint !== "function")
+    ) {
+        throw new TypeError("Key \"parser\": Expected object with parse() or parseForESLint() method.");
+    }
+
+}
+
+/**
+ * Validates the language options.
+ * @param {Object} languageOptions The language options to validate.
+ * @returns {void}
+ * @throws {TypeError} If the language options are invalid.
+ */
+function validateLanguageOptions(languageOptions) {
+
+    if (!isNonArrayObject(languageOptions)) {
+        throw new TypeError("Expected an object.");
+    }
+
+    const {
+        ecmaVersion,
+        sourceType,
+        globals,
+        parser,
+        parserOptions,
+        ...otherOptions
+    } = languageOptions;
+
+    if ("ecmaVersion" in languageOptions) {
+        validateEcmaVersion(ecmaVersion);
+    }
+
+    if ("sourceType" in languageOptions) {
+        validateSourceType(sourceType);
+    }
+
+    if ("globals" in languageOptions) {
+        validateGlobals(globals);
+    }
+
+    if ("parser" in languageOptions) {
+        validateParser(parser);
+    }
+
+    if ("parserOptions" in languageOptions) {
+        if (!isNonArrayObject(parserOptions)) {
+            throw new TypeError("Key \"parserOptions\": Expected an object.");
+        }
+    }
+
+    const otherOptionKeys = Object.keys(otherOptions);
+
+    if (otherOptionKeys.length > 0) {
+        throw new TypeError(`Unexpected key "${otherOptionKeys[0]}" found.`);
+    }
+
+}
+
+module.exports = { validateLanguageOptions };

data/node_modules/eslint/lib/linter/apply-disable-directives.js

@@ -10,6 +10,9 @@
 //------------------------------------------------------------------------------
 
 /** @typedef {import("../shared/types").LintMessage} LintMessage */
+/** @typedef {import("@eslint/core").Language} Language */
+/** @typedef {import("@eslint/core").Position} Position */
+/** @typedef {import("@eslint/core").RulesConfig} RulesConfig */
 
 //------------------------------------------------------------------------------
 // Module Definition
@@ -24,8 +27,8 @@ const {
 
 /**
  * Compares the locations of two objects in a source file
- * @param {{line: number, column: number}} itemA The first object
- * @param {{line: number, column: number}} itemB The second object
+ * @param {Position} itemA The first object
+ * @param {Position} itemB The second object
  * @returns {number} A value less than 1 if itemA appears before itemB in the source file, greater than 1 if
  * itemA appears after itemB in the source file, or 0 if itemA and itemB have the same location.
  */
@@ -369,6 +372,8 @@ function applyDirectives(options) {
 
     const processed = processUnusedDirectives(unusedDisableDirectivesToReport)
         .concat(processUnusedDirectives(unusedEnableDirectivesToReport));
+    const columnOffset = options.language.columnStart === 1 ? 0 : 1;
+    const lineOffset = options.language.lineStart === 1 ? 0 : 1;
 
     const unusedDirectives = processed
         .map(({ description, fix, unprocessedDirective }) => {
@@ -388,8 +393,8 @@ function applyDirectives(options) {
             return {
                 ruleId: null,
                 message,
-                line: type === "disable-next-line" ? parentDirective.node.loc.start.line : line,
-                column: type === "disable-next-line" ? parentDirective.node.loc.start.column + 1 : column,
+                line: type === "disable-next-line" ? parentDirective.node.loc.start.line + lineOffset : line,
+                column: type === "disable-next-line" ? parentDirective.node.loc.start.column + columnOffset : column,
                 severity: options.reportUnusedDisableDirectives === "warn" ? 1 : 2,
                 nodeType: null,
                 ...options.disableFixes ? {} : { fix }
@@ -403,6 +408,7 @@ function applyDirectives(options) {
  * Given a list of directive comments (i.e. metadata about eslint-disable and eslint-enable comments) and a list
  * of reported problems, adds the suppression information to the problems.
  * @param {Object} options Information about directives and problems
+ * @param {Language} options.language The language being linted.
  * @param {{
  *      type: ("disable"|"enable"|"disable-line"|"disable-next-line"),
  *      ruleId: (string|null),
@@ -415,13 +421,13 @@ function applyDirectives(options) {
  * @param {{ruleId: (string|null), line: number, column: number}[]} options.problems
  * A list of problems reported by rules, sorted by increasing location in the file, with one-based columns.
  * @param {"off" | "warn" | "error"} options.reportUnusedDisableDirectives If `"warn"` or `"error"`, adds additional problems for unused directives
- * @param {Object} options.configuredRules The rules configuration.
+ * @param {RulesConfig} options.configuredRules The rules configuration.
  * @param {Function} options.ruleFilter A predicate function to filter which rules should be executed.
  * @param {boolean} options.disableFixes If true, it doesn't make `fix` properties.
  * @returns {{ruleId: (string|null), line: number, column: number, suppressions?: {kind: string, justification: string}}[]}
  * An object with a list of reported problems, the suppressed of which contain the suppression information.
  */
-module.exports = ({ directives, disableFixes, problems, configuredRules, ruleFilter, reportUnusedDisableDirectives = "off" }) => {
+module.exports = ({ language, directives, disableFixes, problems, configuredRules, ruleFilter, reportUnusedDisableDirectives = "off" }) => {
     const blockDirectives = directives
         .filter(directive => directive.type === "disable" || directive.type === "enable")
         .map(directive => Object.assign({}, directive, { unprocessedDirective: directive }))
@@ -470,6 +476,7 @@ module.exports = ({ directives, disableFixes, problems, configuredRules, ruleFil
     }
 
     const blockDirectivesResult = applyDirectives({
+        language,
         problems,
         directives: blockDirectives,
         disableFixes,
@@ -477,6 +484,7 @@ module.exports = ({ directives, disableFixes, problems, configuredRules, ruleFil
         rulesToIgnore
     });
     const lineDirectivesResult = applyDirectives({
+        language,
         problems: blockDirectivesResult.problems,
         directives: lineDirectives,
         disableFixes,

data/node_modules/eslint/lib/linter/config-comment-parser.js

@@ -22,12 +22,6 @@ const levn = require("levn"),
 
 const debug = require("debug")("eslint:config-comment-parser");
 
-//------------------------------------------------------------------------------
-// Typedefs
-//------------------------------------------------------------------------------
-
-/** @typedef {import("../shared/types").LintMessage} LintMessage */
-
 //------------------------------------------------------------------------------
 // Public Interface
 //------------------------------------------------------------------------------
@@ -69,10 +63,9 @@ module.exports = class ConfigCommentParser {
     /**
      * Parses a JSON-like config.
      * @param {string} string The string to parse.
-     * @param {Object} location Start line and column of comments for potential error message.
-     * @returns {({success: true, config: Object}|{success: false, error: LintMessage})} Result map object
+     * @returns {({success: true, config: Object}|{success: false, error: {message: string}})} Result map object
      */
-    parseJsonConfig(string, location) {
+    parseJsonConfig(string) {
         debug("Parsing JSON config");
 
         // Parses a JSON-like comment by the same way as parsing CLI option.
@@ -115,13 +108,7 @@ module.exports = class ConfigCommentParser {
             return {
                 success: false,
                 error: {
-                    ruleId: null,
-                    fatal: true,
-                    severity: 2,
-                    message: `Failed to parse JSON from '${normalizedString}': ${ex.message}`,
-                    line: location.start.line,
-                    column: location.start.column + 1,
-                    nodeType: null
+                    message: `Failed to parse JSON from '${normalizedString}': ${ex.message}`
                 }
             };