@so1ve/eslint-plugin 0.107.2 → 0.108.0

package/dist/index.cjs

@@ -21,12 +21,12 @@ function getPreviousNode(node) {
   }
 }
 
-const RULE_NAME$6 = "function-style";
+const RULE_NAME$7 = "function-style";
 const START_RETURN = /^return /;
 const RETURN_ONLY = /^return\s*(?:;\s*)?$/;
 const END_SEMICOLON = /;$/;
 const functionStyle = createEslintRule({
-  name: RULE_NAME$6,
+  name: RULE_NAME$7,
   meta: {
     type: "problem",
     docs: {
@@ -76,9 +76,7 @@ const functionStyle = createEslintRule({
       haveThisAccess = false;
     }
     return {
-      FunctionExpression() {
-        setupScope();
-      },
+      "FunctionExpression": setupScope,
       "FunctionExpression:exit"(node) {
         if (node.parent?.id?.typeAnnotation || node.parent?.type !== types.AST_NODE_TYPES.VariableDeclarator || haveThisAccess) {
           clearThisAccess();
@@ -95,9 +93,7 @@ const functionStyle = createEslintRule({
         });
         clearThisAccess();
       },
-      "FunctionDeclaration:not(TSDeclareFunction + FunctionDeclaration)"() {
-        setupScope();
-      },
+      "FunctionDeclaration:not(TSDeclareFunction + FunctionDeclaration)": setupScope,
       "FunctionDeclaration:not(TSDeclareFunction + FunctionDeclaration):exit"(node) {
         if (haveThisAccess) {
           return;
@@ -134,9 +130,7 @@ const functionStyle = createEslintRule({
         });
         clearThisAccess();
       },
-      ArrowFunctionExpression() {
-        setupScope();
-      },
+      "ArrowFunctionExpression": setupScope,
       "ArrowFunctionExpression:exit"(node) {
         if (haveThisAccess) {
           return;
@@ -189,9 +183,9 @@ const functionStyle = createEslintRule({
   }
 });
 
-const RULE_NAME$5 = "import-dedupe";
+const RULE_NAME$6 = "import-dedupe";
 const importDedupe = createEslintRule({
-  name: RULE_NAME$5,
+  name: RULE_NAME$6,
   meta: {
     type: "problem",
     docs: {
@@ -237,14 +231,14 @@ const importDedupe = createEslintRule({
   })
 });
 
-const RULE_NAME$4 = "no-import-promises-as";
+const RULE_NAME$5 = "no-import-promises-as";
 const _POSSIBLE_IMPORT_SOURCES = ["dns", "fs", "readline", "stream"];
 const POSSIBLE_IMPORT_SOURCES = [
   ..._POSSIBLE_IMPORT_SOURCES,
   ..._POSSIBLE_IMPORT_SOURCES.map((s) => `node:${s}`)
 ];
 const noImportPromisesAs = createEslintRule({
-  name: RULE_NAME$4,
+  name: RULE_NAME$5,
   meta: {
     type: "problem",
     docs: {
@@ -295,9 +289,9 @@ import ${as} from "${node.source.value}/promises";`
   }
 });
 
-const RULE_NAME$3 = "no-inline-type-import";
+const RULE_NAME$4 = "no-inline-type-import";
 const noInlineTypeImport = createEslintRule({
-  name: RULE_NAME$3,
+  name: RULE_NAME$4,
   meta: {
     type: "layout",
     docs: {
@@ -361,7 +355,7 @@ const noInlineTypeImport = createEslintRule({
   }
 });
 
-const RULE_NAME$2 = "no-negated-comparison";
+const RULE_NAME$3 = "no-negated-comparison";
 const negatedToPositive = {
   "==": "!=",
   "===": "!==",
@@ -374,7 +368,7 @@ const negatedToPositive = {
 };
 const negatives = Object.keys(negatedToPositive);
 const noNegatedComparison = createEslintRule({
-  name: RULE_NAME$2,
+  name: RULE_NAME$3,
   meta: {
     type: "problem",
     docs: {
@@ -411,9 +405,9 @@ const noNegatedComparison = createEslintRule({
   })
 });
 
-const RULE_NAME$1 = "no-useless-template-string";
+const RULE_NAME$2 = "no-useless-template-string";
 const noUselessTemplateString = createEslintRule({
-  name: RULE_NAME$1,
+  name: RULE_NAME$2,
   meta: {
     type: "problem",
     docs: {
@@ -449,9 +443,9 @@ const noUselessTemplateString = createEslintRule({
   })
 });
 
-const RULE_NAME = "pad-after-last-import";
+const RULE_NAME$1 = "pad-after-last-import";
 const padAfterLastImport = createEslintRule({
-  name: RULE_NAME,
+  name: RULE_NAME$1,
   meta: {
     type: "problem",
     docs: {
@@ -492,6 +486,76 @@ const padAfterLastImport = createEslintRule({
   }
 });
 
+const RULE_NAME = "use-async-with-await";
+const useAsyncWithAwait = createEslintRule({
+  name: RULE_NAME,
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Enforce using async keyword with await.",
+      recommended: "error"
+    },
+    fixable: "code",
+    schema: [],
+    messages: {
+      useAsyncWithAwait: "Expect using async keyword with await."
+    }
+  },
+  defaultOptions: [],
+  create: (context) => {
+    const scopeStack = [];
+    let haveAwaitExpression = false;
+    function setupScope() {
+      scopeStack.push(context.getScope());
+    }
+    function clearAwaitExpression() {
+      scopeStack.pop();
+      haveAwaitExpression = false;
+    }
+    return {
+      "FunctionExpression[async=false]": setupScope,
+      "FunctionExpression[async=false]:exit"(node) {
+        if (!haveAwaitExpression) {
+          return;
+        }
+        context.report({
+          node,
+          messageId: "useAsyncWithAwait",
+          fix: (fixer) => fixer.insertTextBefore(node, "async ")
+        });
+        clearAwaitExpression();
+      },
+      "FunctionDeclaration[async=false]": setupScope,
+      "FunctionDeclaration[async=false]:exit"(node) {
+        if (!haveAwaitExpression) {
+          return;
+        }
+        context.report({
+          node,
+          messageId: "useAsyncWithAwait",
+          fix: (fixer) => fixer.insertTextBefore(node, "async ")
+        });
+        clearAwaitExpression();
+      },
+      "ArrowFunctionExpression[async=false]": setupScope,
+      "ArrowFunctionExpression[async=false]:exit"(node) {
+        if (!haveAwaitExpression) {
+          return;
+        }
+        context.report({
+          node,
+          messageId: "useAsyncWithAwait",
+          fix: (fixer) => fixer.insertTextBefore(node, "async ")
+        });
+        clearAwaitExpression();
+      },
+      AwaitExpression() {
+        haveAwaitExpression = scopeStack.includes(context.getScope());
+      }
+    };
+  }
+});
+
 const index = {
   rules: {
     "function-style": functionStyle,
@@ -500,7 +564,8 @@ const index = {
     "no-negated-comparison": noNegatedComparison,
     "no-useless-template-string": noUselessTemplateString,
     "no-import-promises-as": noImportPromisesAs,
-    "pad-after-last-import": padAfterLastImport
+    "pad-after-last-import": padAfterLastImport,
+    "use-async-with-await": useAsyncWithAwait
   }
 };
 

package/dist/index.d.ts

@@ -11,6 +11,7 @@ declare const _default: {
         "no-useless-template-string": _typescript_eslint_utils_dist_ts_eslint_Rule.RuleModule<"noUselessTemplateString", [], _typescript_eslint_utils_dist_ts_eslint_Rule.RuleListener>;
         "no-import-promises-as": _typescript_eslint_utils_dist_ts_eslint_Rule.RuleModule<"noImportPromisesAs", [], _typescript_eslint_utils_dist_ts_eslint_Rule.RuleListener>;
         "pad-after-last-import": _typescript_eslint_utils_dist_ts_eslint_Rule.RuleModule<"padAfterLastImport", [], _typescript_eslint_utils_dist_ts_eslint_Rule.RuleListener>;
+        "use-async-with-await": _typescript_eslint_utils_dist_ts_eslint_Rule.RuleModule<"useAsyncWithAwait", [], _typescript_eslint_utils_dist_ts_eslint_Rule.RuleListener>;
     };
 };
 

package/dist/index.mjs

@@ -19,12 +19,12 @@ function getPreviousNode(node) {
   }
 }
 
-const RULE_NAME$6 = "function-style";
+const RULE_NAME$7 = "function-style";
 const START_RETURN = /^return /;
 const RETURN_ONLY = /^return\s*(?:;\s*)?$/;
 const END_SEMICOLON = /;$/;
 const functionStyle = createEslintRule({
-  name: RULE_NAME$6,
+  name: RULE_NAME$7,
   meta: {
     type: "problem",
     docs: {
@@ -74,9 +74,7 @@ const functionStyle = createEslintRule({
       haveThisAccess = false;
     }
     return {
-      FunctionExpression() {
-        setupScope();
-      },
+      "FunctionExpression": setupScope,
       "FunctionExpression:exit"(node) {
         if (node.parent?.id?.typeAnnotation || node.parent?.type !== AST_NODE_TYPES.VariableDeclarator || haveThisAccess) {
           clearThisAccess();
@@ -93,9 +91,7 @@ const functionStyle = createEslintRule({
         });
         clearThisAccess();
       },
-      "FunctionDeclaration:not(TSDeclareFunction + FunctionDeclaration)"() {
-        setupScope();
-      },
+      "FunctionDeclaration:not(TSDeclareFunction + FunctionDeclaration)": setupScope,
       "FunctionDeclaration:not(TSDeclareFunction + FunctionDeclaration):exit"(node) {
         if (haveThisAccess) {
           return;
@@ -132,9 +128,7 @@ const functionStyle = createEslintRule({
         });
         clearThisAccess();
       },
-      ArrowFunctionExpression() {
-        setupScope();
-      },
+      "ArrowFunctionExpression": setupScope,
       "ArrowFunctionExpression:exit"(node) {
         if (haveThisAccess) {
           return;
@@ -187,9 +181,9 @@ const functionStyle = createEslintRule({
   }
 });
 
-const RULE_NAME$5 = "import-dedupe";
+const RULE_NAME$6 = "import-dedupe";
 const importDedupe = createEslintRule({
-  name: RULE_NAME$5,
+  name: RULE_NAME$6,
   meta: {
     type: "problem",
     docs: {
@@ -235,14 +229,14 @@ const importDedupe = createEslintRule({
   })
 });
 
-const RULE_NAME$4 = "no-import-promises-as";
+const RULE_NAME$5 = "no-import-promises-as";
 const _POSSIBLE_IMPORT_SOURCES = ["dns", "fs", "readline", "stream"];
 const POSSIBLE_IMPORT_SOURCES = [
   ..._POSSIBLE_IMPORT_SOURCES,
   ..._POSSIBLE_IMPORT_SOURCES.map((s) => `node:${s}`)
 ];
 const noImportPromisesAs = createEslintRule({
-  name: RULE_NAME$4,
+  name: RULE_NAME$5,
   meta: {
     type: "problem",
     docs: {
@@ -293,9 +287,9 @@ import ${as} from "${node.source.value}/promises";`
   }
 });
 
-const RULE_NAME$3 = "no-inline-type-import";
+const RULE_NAME$4 = "no-inline-type-import";
 const noInlineTypeImport = createEslintRule({
-  name: RULE_NAME$3,
+  name: RULE_NAME$4,
   meta: {
     type: "layout",
     docs: {
@@ -359,7 +353,7 @@ const noInlineTypeImport = createEslintRule({
   }
 });
 
-const RULE_NAME$2 = "no-negated-comparison";
+const RULE_NAME$3 = "no-negated-comparison";
 const negatedToPositive = {
   "==": "!=",
   "===": "!==",
@@ -372,7 +366,7 @@ const negatedToPositive = {
 };
 const negatives = Object.keys(negatedToPositive);
 const noNegatedComparison = createEslintRule({
-  name: RULE_NAME$2,
+  name: RULE_NAME$3,
   meta: {
     type: "problem",
     docs: {
@@ -409,9 +403,9 @@ const noNegatedComparison = createEslintRule({
   })
 });
 
-const RULE_NAME$1 = "no-useless-template-string";
+const RULE_NAME$2 = "no-useless-template-string";
 const noUselessTemplateString = createEslintRule({
-  name: RULE_NAME$1,
+  name: RULE_NAME$2,
   meta: {
     type: "problem",
     docs: {
@@ -447,9 +441,9 @@ const noUselessTemplateString = createEslintRule({
   })
 });
 
-const RULE_NAME = "pad-after-last-import";
+const RULE_NAME$1 = "pad-after-last-import";
 const padAfterLastImport = createEslintRule({
-  name: RULE_NAME,
+  name: RULE_NAME$1,
   meta: {
     type: "problem",
     docs: {
@@ -490,6 +484,76 @@ const padAfterLastImport = createEslintRule({
   }
 });
 
+const RULE_NAME = "use-async-with-await";
+const useAsyncWithAwait = createEslintRule({
+  name: RULE_NAME,
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Enforce using async keyword with await.",
+      recommended: "error"
+    },
+    fixable: "code",
+    schema: [],
+    messages: {
+      useAsyncWithAwait: "Expect using async keyword with await."
+    }
+  },
+  defaultOptions: [],
+  create: (context) => {
+    const scopeStack = [];
+    let haveAwaitExpression = false;
+    function setupScope() {
+      scopeStack.push(context.getScope());
+    }
+    function clearAwaitExpression() {
+      scopeStack.pop();
+      haveAwaitExpression = false;
+    }
+    return {
+      "FunctionExpression[async=false]": setupScope,
+      "FunctionExpression[async=false]:exit"(node) {
+        if (!haveAwaitExpression) {
+          return;
+        }
+        context.report({
+          node,
+          messageId: "useAsyncWithAwait",
+          fix: (fixer) => fixer.insertTextBefore(node, "async ")
+        });
+        clearAwaitExpression();
+      },
+      "FunctionDeclaration[async=false]": setupScope,
+      "FunctionDeclaration[async=false]:exit"(node) {
+        if (!haveAwaitExpression) {
+          return;
+        }
+        context.report({
+          node,
+          messageId: "useAsyncWithAwait",
+          fix: (fixer) => fixer.insertTextBefore(node, "async ")
+        });
+        clearAwaitExpression();
+      },
+      "ArrowFunctionExpression[async=false]": setupScope,
+      "ArrowFunctionExpression[async=false]:exit"(node) {
+        if (!haveAwaitExpression) {
+          return;
+        }
+        context.report({
+          node,
+          messageId: "useAsyncWithAwait",
+          fix: (fixer) => fixer.insertTextBefore(node, "async ")
+        });
+        clearAwaitExpression();
+      },
+      AwaitExpression() {
+        haveAwaitExpression = scopeStack.includes(context.getScope());
+      }
+    };
+  }
+});
+
 const index = {
   rules: {
     "function-style": functionStyle,
@@ -498,7 +562,8 @@ const index = {
     "no-negated-comparison": noNegatedComparison,
     "no-useless-template-string": noUselessTemplateString,
     "no-import-promises-as": noImportPromisesAs,
-    "pad-after-last-import": padAfterLastImport
+    "pad-after-last-import": padAfterLastImport,
+    "use-async-with-await": useAsyncWithAwait
   }
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@so1ve/eslint-plugin",
-  "version": "0.107.2",
+  "version": "0.108.0",
   "author": "Ray <i@mk1.io> (https://github.com/so1ve/)",
   "keywords": [
     "eslint",