xo 0.53.1 → 0.54.0

package/node_modules/eslint-config-xo-typescript/index.js

@@ -79,8 +79,10 @@ module.exports = {
 	parserOptions: {
 		warnOnUnsupportedTypeScriptVersion: false,
 		sourceType: 'module',
-		jsx: true,
-		project: 'tsconfig.json'
+		project: 'tsconfig.json',
+		ecmaFeatures: {
+			jsx: true
+		}
 	},
 	plugins: [
 		'@typescript-eslint'
@@ -236,7 +238,8 @@ module.exports = {
 		// 		allowExpressions: true,
 		// 		allowTypedFunctionExpressions: true,
 		// 		allowHigherOrderFunctions: true,
-		// 		allowConciseArrowFunctionExpressionsStartingWithVoid: false
+		// 		allowConciseArrowFunctionExpressionsStartingWithVoid: false,
+		// 		allowIIFE: true
 		// 	}
 		// ],
 
@@ -321,7 +324,6 @@ module.exports = {
 
 					'private-decorated-field',
 					'private-instance-field',
-					'private-abstract-field',
 					'private-field',
 
 					'instance-field',
@@ -346,7 +348,6 @@ module.exports = {
 
 					'private-decorated-method',
 					'private-instance-method',
-					'private-abstract-method',
 					'private-method',
 
 					'instance-method',
@@ -375,6 +376,7 @@ module.exports = {
 		'@typescript-eslint/no-dupe-class-members': 'error',
 		'@typescript-eslint/no-confusing-void-expression': 'error',
 		'@typescript-eslint/no-duplicate-enum-values': 'error',
+		'@typescript-eslint/no-duplicate-type-constituents': 'error',
 		'@typescript-eslint/no-dynamic-delete': 'error',
 		'no-empty-function': 'off',
 		'@typescript-eslint/no-empty-function': 'error',
@@ -440,7 +442,6 @@ module.exports = {
 			}
 		],
 		'@typescript-eslint/no-for-in-array': 'error',
-		'@typescript-eslint/no-implicit-any-catch': 'error',
 		'@typescript-eslint/no-inferrable-types': 'error',
 
 		// Disabled for now as it has too many false-positives.
@@ -518,11 +519,7 @@ module.exports = {
 		'@typescript-eslint/no-unnecessary-type-arguments': 'error',
 		'@typescript-eslint/no-unnecessary-type-assertion': 'error',
 		'@typescript-eslint/no-unnecessary-type-constraint': 'error',
-
-		// Disabled for now. When using try/catch, the error is by default `any` and if you pass the error anywhere, it will trigger this rule. This is a very common occurence.
-		// TODO: Enable this rule when TypeScript has more strongly typed errors, probably 2023 at the earliest.
-		// '@typescript-eslint/no-unsafe-argument': 'error',
-
+		'@typescript-eslint/no-unsafe-argument': 'error',
 		'@typescript-eslint/no-unsafe-assignment': 'error',
 		'@typescript-eslint/no-unsafe-call': 'error',
 		'@typescript-eslint/no-unsafe-declaration-merging': 'error',

package/node_modules/eslint-config-xo-typescript/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "eslint-config-xo-typescript",
-	"version": "0.55.0",
+	"version": "0.57.0",
 	"description": "ESLint shareable config for TypeScript to be used with eslint-config-xo",
 	"license": "MIT",
 	"repository": "xojs/eslint-config-xo-typescript",
@@ -44,15 +44,15 @@
 		"simple"
 	],
 	"devDependencies": {
-		"@typescript-eslint/eslint-plugin": "^5.43.0",
-		"@typescript-eslint/parser": "^5.43.0",
+		"@typescript-eslint/eslint-plugin": "^5.57.0",
+		"@typescript-eslint/parser": "^5.57.0",
 		"ava": "^2.4.0",
 		"eslint": "^7.8.1",
 		"typescript": ">=4.4"
 	},
 	"peerDependencies": {
-		"@typescript-eslint/eslint-plugin": ">=5.43.0",
-		"@typescript-eslint/parser": ">=5.43.0",
+		"@typescript-eslint/eslint-plugin": ">=5.57.0",
+		"@typescript-eslint/parser": ">=5.57.0",
 		"eslint": ">=8.0.0",
 		"typescript": ">=4.4"
 	}

package/node_modules/eslint-visitor-keys/README.md

@@ -2,8 +2,7 @@
 
 [![npm version](https://img.shields.io/npm/v/eslint-visitor-keys.svg)](https://www.npmjs.com/package/eslint-visitor-keys)
 [![Downloads/month](https://img.shields.io/npm/dm/eslint-visitor-keys.svg)](http://www.npmtrends.com/eslint-visitor-keys)
-[![Build Status](https://travis-ci.org/eslint/eslint-visitor-keys.svg?branch=master)](https://travis-ci.org/eslint/eslint-visitor-keys)
-[![Dependency Status](https://david-dm.org/eslint/eslint-visitor-keys.svg)](https://david-dm.org/eslint/eslint-visitor-keys)
+[![Build Status](https://github.com/eslint/eslint-visitor-keys/workflows/CI/badge.svg)](https://github.com/eslint/eslint-visitor-keys/actions)
 
 Constants and utilities about visitor keys to traverse AST.
 

package/node_modules/eslint-visitor-keys/dist/eslint-visitor-keys.cjs

@@ -10,14 +10,6 @@ Object.defineProperty(exports, '__esModule', { value: true });
  * @type {VisitorKeys}
  */
 const KEYS = {
-    AssignmentExpression: [
-        "left",
-        "right"
-    ],
-    AssignmentPattern: [
-        "left",
-        "right"
-    ],
     ArrayExpression: [
         "elements"
     ],
@@ -28,16 +20,24 @@ const KEYS = {
         "params",
         "body"
     ],
+    AssignmentExpression: [
+        "left",
+        "right"
+    ],
+    AssignmentPattern: [
+        "left",
+        "right"
+    ],
     AwaitExpression: [
         "argument"
     ],
-    BlockStatement: [
-        "body"
-    ],
     BinaryExpression: [
         "left",
         "right"
     ],
+    BlockStatement: [
+        "body"
+    ],
     BreakStatement: [
         "label"
     ],
@@ -79,6 +79,12 @@ const KEYS = {
         "test"
     ],
     EmptyStatement: [],
+    ExperimentalRestProperty: [
+        "argument"
+    ],
+    ExperimentalSpreadProperty: [
+        "argument"
+    ],
     ExportAllDeclaration: [
         "exported",
         "source"
@@ -98,18 +104,6 @@ const KEYS = {
     ExpressionStatement: [
         "expression"
     ],
-    ExperimentalRestProperty: [
-        "argument"
-    ],
-    ExperimentalSpreadProperty: [
-        "argument"
-    ],
-    ForStatement: [
-        "init",
-        "test",
-        "update",
-        "body"
-    ],
     ForInStatement: [
         "left",
         "right",
@@ -120,6 +114,12 @@ const KEYS = {
         "right",
         "body"
     ],
+    ForStatement: [
+        "init",
+        "test",
+        "update",
+        "body"
+    ],
     FunctionDeclaration: [
         "id",
         "params",
@@ -160,6 +160,7 @@ const KEYS = {
     JSXClosingElement: [
         "name"
     ],
+    JSXClosingFragment: [],
     JSXElement: [
         "openingElement",
         "children",
@@ -169,6 +170,11 @@ const KEYS = {
     JSXExpressionContainer: [
         "expression"
     ],
+    JSXFragment: [
+        "openingFragment",
+        "children",
+        "closingFragment"
+    ],
     JSXIdentifier: [],
     JSXMemberExpression: [
         "object",
@@ -182,22 +188,19 @@ const KEYS = {
         "name",
         "attributes"
     ],
+    JSXOpeningFragment: [],
     JSXSpreadAttribute: [
         "argument"
     ],
-    JSXText: [],
-    JSXFragment: [
-        "openingFragment",
-        "children",
-        "closingFragment"
+    JSXSpreadChild: [
+        "expression"
     ],
-    JSXClosingFragment: [],
-    JSXOpeningFragment: [],
-    Literal: [],
+    JSXText: [],
     LabeledStatement: [
         "label",
         "body"
     ],
+    Literal: [],
     LogicalExpression: [
         "left",
         "right"
@@ -252,14 +255,14 @@ const KEYS = {
         "body"
     ],
     Super: [],
-    SwitchStatement: [
-        "discriminant",
-        "cases"
-    ],
     SwitchCase: [
         "test",
         "consequent"
     ],
+    SwitchStatement: [
+        "discriminant",
+        "cases"
+    ],
     TaggedTemplateExpression: [
         "tag",
         "quasi"
@@ -379,4 +382,3 @@ function unionWith(additionalKeys) {
 exports.KEYS = KEYS;
 exports.getKeys = getKeys;
 exports.unionWith = unionWith;
-//# sourceMappingURL=eslint-visitor-keys.cjs.map

package/node_modules/eslint-visitor-keys/lib/visitor-keys.js

@@ -6,14 +6,6 @@
  * @type {VisitorKeys}
  */
 const KEYS = {
-    AssignmentExpression: [
-        "left",
-        "right"
-    ],
-    AssignmentPattern: [
-        "left",
-        "right"
-    ],
     ArrayExpression: [
         "elements"
     ],
@@ -24,16 +16,24 @@ const KEYS = {
         "params",
         "body"
     ],
+    AssignmentExpression: [
+        "left",
+        "right"
+    ],
+    AssignmentPattern: [
+        "left",
+        "right"
+    ],
     AwaitExpression: [
         "argument"
     ],
-    BlockStatement: [
-        "body"
-    ],
     BinaryExpression: [
         "left",
         "right"
     ],
+    BlockStatement: [
+        "body"
+    ],
     BreakStatement: [
         "label"
     ],
@@ -75,6 +75,12 @@ const KEYS = {
         "test"
     ],
     EmptyStatement: [],
+    ExperimentalRestProperty: [
+        "argument"
+    ],
+    ExperimentalSpreadProperty: [
+        "argument"
+    ],
     ExportAllDeclaration: [
         "exported",
         "source"
@@ -94,18 +100,6 @@ const KEYS = {
     ExpressionStatement: [
         "expression"
     ],
-    ExperimentalRestProperty: [
-        "argument"
-    ],
-    ExperimentalSpreadProperty: [
-        "argument"
-    ],
-    ForStatement: [
-        "init",
-        "test",
-        "update",
-        "body"
-    ],
     ForInStatement: [
         "left",
         "right",
@@ -116,6 +110,12 @@ const KEYS = {
         "right",
         "body"
     ],
+    ForStatement: [
+        "init",
+        "test",
+        "update",
+        "body"
+    ],
     FunctionDeclaration: [
         "id",
         "params",
@@ -156,6 +156,7 @@ const KEYS = {
     JSXClosingElement: [
         "name"
     ],
+    JSXClosingFragment: [],
     JSXElement: [
         "openingElement",
         "children",
@@ -165,6 +166,11 @@ const KEYS = {
     JSXExpressionContainer: [
         "expression"
     ],
+    JSXFragment: [
+        "openingFragment",
+        "children",
+        "closingFragment"
+    ],
     JSXIdentifier: [],
     JSXMemberExpression: [
         "object",
@@ -178,22 +184,19 @@ const KEYS = {
         "name",
         "attributes"
     ],
+    JSXOpeningFragment: [],
     JSXSpreadAttribute: [
         "argument"
     ],
-    JSXText: [],
-    JSXFragment: [
-        "openingFragment",
-        "children",
-        "closingFragment"
+    JSXSpreadChild: [
+        "expression"
     ],
-    JSXClosingFragment: [],
-    JSXOpeningFragment: [],
-    Literal: [],
+    JSXText: [],
     LabeledStatement: [
         "label",
         "body"
     ],
+    Literal: [],
     LogicalExpression: [
         "left",
         "right"
@@ -248,14 +251,14 @@ const KEYS = {
         "body"
     ],
     Super: [],
-    SwitchStatement: [
-        "discriminant",
-        "cases"
-    ],
     SwitchCase: [
         "test",
         "consequent"
     ],
+    SwitchStatement: [
+        "discriminant",
+        "cases"
+    ],
     TaggedTemplateExpression: [
         "tag",
         "quasi"

package/node_modules/eslint-visitor-keys/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-visitor-keys",
-  "version": "3.3.0",
+  "version": "3.4.0",
   "description": "Constants and utilities about visitor keys to traverse AST.",
   "type": "module",
   "main": "dist/eslint-visitor-keys.cjs",
@@ -25,24 +25,33 @@
     "node": "^12.22.0 || ^14.17.0 || >=16.0.0"
   },
   "devDependencies": {
-    "c8": "^7.7.3",
+    "@types/estree": "^0.0.51",
+    "@types/estree-jsx": "^0.0.1",
+    "@typescript-eslint/parser": "^5.14.0",
+    "c8": "^7.11.0",
+    "chai": "^4.3.6",
     "eslint": "^7.29.0",
     "eslint-config-eslint": "^7.0.0",
     "eslint-plugin-jsdoc": "^35.4.0",
     "eslint-plugin-node": "^11.1.0",
     "eslint-release": "^3.2.0",
-    "mocha": "^9.0.1",
+    "esquery": "^1.4.0",
+    "json-diff": "^0.7.3",
+    "mocha": "^9.2.1",
     "opener": "^1.5.2",
-    "rollup": "^2.52.1",
+    "rollup": "^2.70.0",
     "tsd": "^0.19.1",
-    "typescript": "^4.5.5"
+    "typescript": "^4.6.2"
   },
   "scripts": {
     "prepare": "npm run build",
-    "build": "rollup -c && npm run tsc",
+    "bundle": "rollup -c",
+    "build": "npm run bundle && npm run tsc",
+    "build:debug": "npm run bundle -- -m && npm run tsc",
     "lint": "eslint .",
     "tsc": "tsc",
     "tsd": "tsd",
+    "build-keys": "node tools/build-keys-from-ts",
     "test": "mocha tests/lib/**/*.cjs && c8 mocha tests/lib/**/*.js && npm run tsd",
     "coverage": "c8 report --reporter lcov && opener coverage/lcov-report/index.html",
     "generate-release": "eslint-generate-release",
@@ -52,6 +61,7 @@
     "publish-release": "eslint-publish-release"
   },
   "repository": "eslint/eslint-visitor-keys",
+  "funding": "https://opencollective.com/eslint",
   "keywords": [],
   "author": "Toru Nagashima (https://github.com/mysticatea)",
   "license": "Apache-2.0",

package/node_modules/fastq/README.md

@@ -225,7 +225,7 @@ Same than `kill` but the `drain` function will be called before reset to empty.
 ### queue.error(handler)
 
 Set a global error handler. `handler(err, task)` will be called
-when any of the tasks return an error.
+each time a task is completed, `err` will be not null if the task has thrown an error.
 
 -------------------------------------------------------
 <a name="concurrency"></a>

package/node_modules/fastq/index.d.ts

@@ -17,7 +17,7 @@ declare namespace fastq {
     getQueue(): T[]
     kill(): any
     killAndDrain(): any
-    error(handler: errorHandler): void
+    error(handler: errorHandler<T>): void
     concurrency: number
     drain(): any
     empty: () => void

package/node_modules/fastq/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fastq",
-  "version": "1.13.0",
+  "version": "1.15.0",
   "description": "Fast, in memory work queue",
   "main": "queue.js",
   "scripts": {

package/node_modules/fastq/queue.js

@@ -266,6 +266,12 @@ function queueAsPromised (context, worker, concurrency) {
   }
 
   function drained () {
+    if (queue.idle()) {
+      return new Promise(function (resolve) {
+        resolve()
+      })
+    }
+
     var previousDrain = queue.drain
 
     var p = new Promise(function (resolve) {

package/node_modules/fastq/test/promise.js

@@ -129,6 +129,33 @@ test('drained with drain function', async function (t) {
   t.equal(drainCalled, true)
 })
 
+test('drained while idle should resolve', async function (t) {
+  const queue = buildQueue(worker, 2)
+
+  async function worker (arg) {
+    await sleep(arg)
+  }
+
+  await queue.drained()
+})
+
+test('drained while idle should not call the drain function', async function (t) {
+  let drainCalled = false
+  const queue = buildQueue(worker, 2)
+
+  queue.drain = function () {
+    drainCalled = true
+  }
+
+  async function worker (arg) {
+    await sleep(arg)
+  }
+
+  await queue.drained()
+
+  t.equal(drainCalled, false)
+})
+
 test('set this', async function (t) {
   t.plan(1)
   const that = {}

package/node_modules/grapheme-splitter/LICENSE

@@ -0,0 +1,22 @@
+The MIT License (MIT)
+
+Copyright (c) 2015 Orlin Georgiev
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+

package/node_modules/grapheme-splitter/README.md

@@ -0,0 +1,111 @@
+# Background
+
+In JavaScript there is not always a one-to-one relationship between string characters and what a user would call a separate visual "letter". Some symbols are represented by several characters. This can cause issues when splitting strings and inadvertently cutting a multi-char letter in half, or when you need the actual number of letters in a string.
+
+For example, emoji characters like "🌷","🎁","💩","😜" and "👍" are represented by two JavaScript characters each (high surrogate and low surrogate). That is, 
+
+```javascript
+"🌷".length == 2
+```
+The combined emoji are even longer:
+```javascript
+"🏳️‍🌈".length == 6
+```
+
+What's more, some languages often include combining marks - characters that are used to modify the letters before them. Common examples are the German letter ü and the Spanish letter ñ. Sometimes they can be represented alternatively both as a single character and as a letter + combining mark, with both forms equally valid:
+    
+```javascript
+var two = "ñ"; // unnormalized two-char n+◌̃  , i.e. "\u006E\u0303";
+var one = "ñ"; // normalized single-char, i.e. "\u00F1"
+console.log(one!=two); // prints 'true'
+```
+
+Unicode normalization, as performed by the popular punycode.js library or ECMAScript 6's String.normalize, can **sometimes** fix those differences and turn two-char sequences into single characters. But it is **not** enough in all cases. Some languages like Hindi make extensive use of combining marks on their letters, that have no dedicated single-codepoint Unicode sequences, due to the sheer number of possible combinations.
+For example, the Hindi word "अनुच्छेद" is comprised of 5 letters and 3 combining marks:
+
+अ + न + ु + च + ् + छ + े + द
+
+which is in fact just 5 user-perceived letters:
+
+अ + नु + च् + छे + द
+
+and which Unicode normalization would not combine properly.
+There are also the unusual letter+combining mark combinations which have no dedicated Unicode codepoint. The string Z͑ͫ̓ͪ̂ͫ̽͏̴̙̤̞͉͚̯̞̠͍A̴̵̜̰͔ͫ͗͢L̠ͨͧͩ͘G̴̻͈͍͔̹̑͗̎̅͛́Ǫ̵̹̻̝̳͂̌̌͘ obviously has 5 separate letters, but is in fact comprised of 58 JavaScript characters, most of which are combining marks.
+
+Enter the grapheme-splitter.js library. It can be used to properly split JavaScript strings into what a human user would call separate letters (or "extended grapheme clusters" in Unicode terminology), no matter what their internal representation is. It is an implementation on the [Default Grapheme Cluster Boundary](http://unicode.org/reports/tr29/#Default_Grapheme_Cluster_Table) of [UAX #29](http://www.unicode.org/reports/tr29/). 
+
+# Installation
+
+You can use the index.js file directly as-is. Or you you can install `grapheme-splitter` to your project using the NPM command below:
+
+```
+$ npm install --save grapheme-splitter
+```
+
+# Tests
+
+To run the tests on `grapheme-splitter`, use the command below:
+
+```
+$ npm test
+```
+
+# Usage
+
+Just initialize and use:
+
+```javascript
+var splitter = new GraphemeSplitter();
+
+// split the string to an array of grapheme clusters (one string each)
+var graphemes = splitter.splitGraphemes(string);
+
+// iterate the string to an iterable iterator of grapheme clusters (one string each)
+var graphemes = splitter.iterateGraphemes(string);
+
+// or do this if you just need their number
+var graphemeCount = splitter.countGraphemes(string);
+```
+
+# Examples
+
+```javascript
+var splitter = new GraphemeSplitter();
+
+// plain latin alphabet - nothing spectacular
+splitter.splitGraphemes("abcd"); // returns ["a", "b", "c", "d"]
+
+// two-char emojis and six-char combined emoji
+splitter.splitGraphemes("🌷🎁💩😜👍🏳️‍🌈"); // returns ["🌷","🎁","💩","😜","👍","🏳️‍🌈"]
+
+// diacritics as combining marks, 10 JavaScript chars
+splitter.splitGraphemes("Ĺo͂řȩm̅"); // returns ["Ĺ","o͂","ř","ȩ","m̅"]
+
+// individual Korean characters (Jamo), 4 JavaScript chars
+splitter.splitGraphemes("뎌쉐"); // returns ["뎌","쉐"]
+
+// Hindi text with combining marks, 8 JavaScript chars
+splitter.splitGraphemes("अनुच्छेद"); // returns ["अ","नु","च्","छे","द"]
+
+// demonic multiple combining marks, 75 JavaScript chars
+splitter.splitGraphemes("Z͑ͫ̓ͪ̂ͫ̽͏̴̙̤̞͉͚̯̞̠͍A̴̵̜̰͔ͫ͗͢L̠ͨͧͩ͘G̴̻͈͍͔̹̑͗̎̅͛́Ǫ̵̹̻̝̳͂̌̌͘!͖̬̰̙̗̿̋ͥͥ̂ͣ̐́́͜͞"); // returns ["Z͑ͫ̓ͪ̂ͫ̽͏̴̙̤̞͉͚̯̞̠͍","A̴̵̜̰͔ͫ͗͢","L̠ͨͧͩ͘","G̴̻͈͍͔̹̑͗̎̅͛́","Ǫ̵̹̻̝̳͂̌̌͘","!͖̬̰̙̗̿̋ͥͥ̂ͣ̐́́͜͞"]
+```
+
+# TypeScript
+
+Grapheme splitter includes TypeScript declarations.
+
+```typescript
+import GraphemeSplitter = require('grapheme-splitter')
+
+const splitter = new GraphemeSplitter()
+
+const split: string[] = splitter.splitGraphemes('Z͑ͫ̓ͪ̂ͫ̽͏̴̙̤̞͉͚̯̞̠͍A̴̵̜̰͔ͫ͗͢L̠ͨͧͩ͘G̴̻͈͍͔̹̑͗̎̅͛́Ǫ̵̹̻̝̳͂̌̌͘!͖̬̰̙̗̿̋ͥͥ̂ͣ̐́́͜͞')
+```
+
+# Acknowledgements
+
+This library is heavily influenced by Devon Govett's excellent grapheme-breaker CoffeeScript library at https://github.com/devongovett/grapheme-breaker with an emphasis on ease of integration and pure JavaScript implementation.
+
+
+