@mimik/eslint-plugin-document-env 1.0.6 → 2.0.1

package/.eslintrc

@@ -0,0 +1,34 @@
+{
+  "env": {
+    "node": true
+  },
+  "parserOptions": {
+    "ecmaVersion": 2020
+  },
+  "extends": "airbnb",
+  "rules": {
+    "import/no-extraneous-dependencies": ["error", {"devDependencies": true}],
+    "brace-style": [1, "stroustrup", {"allowSingleLine": true}],
+    "no-confusing-arrow": [0], // arrow isnt confusing
+    "max-len": [1, 180, { "ignoreComments": true }],
+    "linebreak-style": 0,
+    "quotes": [1, "single"],
+    "semi": [1, "always"],
+    "no-process-env": ["error"]
+  },
+  "settings":{
+    "react": {
+      "version": "detect"
+    }
+  },
+  "globals": {
+    "module": true,
+    "require": true,
+    "const": false,
+    "it": false,
+    "describe": false,
+    "before": true,
+    "after": true,
+    "JSON": true
+  }
+}

package/eslint.config.js

@@ -0,0 +1,52 @@
+import importPlugin from 'eslint-plugin-import';
+import js from '@eslint/js';
+import stylistic from '@stylistic/eslint-plugin';
+
+const MAX_LENGTH_LINE = 180;
+const MAX_FUNCTION_PARAMETERS = 6;
+const MAX_LINES_IN_FILES = 400;
+const MAX_LINES_IN_FUNCTION = 100;
+const MAX_STATEMENTS_IN_FUNCTION = 40;
+const MIN_KEYS_IN_OBJECT = 10;
+
+export default [
+  {
+    ignores: ['mochawesome-report/**', 'node_modules/**', 'dist/**'],
+  },
+  importPlugin.flatConfigs.recommended,
+  stylistic.configs['recommended-flat'],
+  js.configs.all,
+  {
+    languageOptions: {
+      ecmaVersion: 2022,
+      globals: {
+        console: 'readonly',
+        describe: 'readonly',
+        it: 'readonly',
+        require: 'readonly',
+      },
+      sourceType: 'module',
+    },
+    rules: {
+      '@stylistic/brace-style': ['warn', 'stroustrup', { allowSingleLine: true }],
+      '@stylistic/semi': ['error', 'always'],
+      'capitalized-comments': ['off'],
+      'curly': ['off'],
+      'import/no-extraneous-dependencies': ['error', { devDependencies: true }],
+      'import/no-unresolved': ['error', { amd: true, caseSensitiveStrict: true, commonjs: true }],
+      'init-declarations': ['off'],
+      'linebreak-style': ['off'],
+      'max-len': ['warn', MAX_LENGTH_LINE, { ignoreComments: true }],
+      'max-lines': ['warn', MAX_LINES_IN_FILES],
+      'max-lines-per-function': ['warn', MAX_LINES_IN_FUNCTION],
+      'max-params': ['error', MAX_FUNCTION_PARAMETERS],
+      'max-statements': ['warn', MAX_STATEMENTS_IN_FUNCTION],
+      'no-confusing-arrow': ['off'], // arrow isnt confusing
+      'no-inline-comments': ['off'],
+      'no-process-env': ['error'],
+      'one-var': ['error', 'never'],
+      'quotes': ['warn', 'single'],
+      'sort-keys': ['error', 'asc', { caseSensitive: true, minKeys: MIN_KEYS_IN_OBJECT, natural: false }],
+    },
+  },
+];

package/index.js

@@ -1,23 +1,11 @@
 const HEADER = ' * | Env variable name | Description | Default | Comments |\n';
 const LINE = ' * | ----------------- | ----------- | ------- | -------- |\n';
-module.exports = {
+const SINGLE = 1;
+const NO_DESCRIPTION = 2;
+
+const plugin = {
   rules: {
     'validate-document-env': {
-      meta: {
-        type: 'problem',
-        docs: {
-          description: 'validate document for environment variables',
-          category: 'Node.js and CommonJS',
-          recommended: false,
-        },
-        schema: [],
-        messages: {
-          unDesribedProcessEnv: 'Undescribed environement variable: {{ name }}',
-          unDocumentedProcessEnv: 'Undocumented environment variable: {{ name }}',
-          duplicatedProcessEnv: 'Duplicated definition for environment variable: {{ name }}',
-        },
-      },
-
       create(context) {
         return {
           MemberExpression(node) {
@@ -26,12 +14,12 @@ module.exports = {
 
             if (objectName === 'process' && !node.computed && propertyName && propertyName === 'env') {
               if (node.parent && node.parent.property) {
-                const allComments = context.getAllComments();
+                const allComments = context.getSourceCode().ast.comments;
                 let foundDocument = false;
                 let propName = node.parent.property.name;
 
                 if (!propName) propName = context.getSourceCode().text.substring(node.parent.property.start, node.parent.property.end);
-                const reg = new RegExp(`^ \\* \\| (${propName.replace(/[|\\{}()[\]^$+*?.]/g, '\\$&')}) \\| *([^\\|]*)`, 'mg');
+                const reg = new RegExp(`^ \\* \\| (${propName.replace(/[|\\{}()[\]^$+*?.]/gu, '\\$&')}) \\| *([^\\|]*)/u`, 'mgu');
 
                 allComments.forEach((comment) => {
                   if (!foundDocument && comment.type === 'Block' && comment.value.includes(HEADER) && comment.value.includes(LINE)) {
@@ -39,20 +27,36 @@ module.exports = {
 
                     if (match) {
                       foundDocument = true;
-                      if (match.length > 1) context.report({ node, messageId: 'duplicatedProcessEnv', data: { name: propName } });
+                      if (match.length > SINGLE) context.report({ data: { name: propName }, messageId: 'duplicatedProcessEnv', node });
                       else {
                         match = reg.exec(comment.value);
-                        if (!match[2]) context.report({ node, messageId: 'unDesribedProcessEnv', data: { name: propName } });
+                        if (!match[NO_DESCRIPTION]) context.report({ data: { name: propName }, messageId: 'unDesribedProcessEnv', node });
                       }
                     }
                   }
                 });
-                if (!foundDocument) context.report({ node, messageId: 'unDocumentedProcessEnv', data: { name: propName } });
+                if (!foundDocument) context.report({ data: { name: propName }, messageId: 'unDocumentedProcessEnv', node });
               }
             }
           },
         };
       },
+      meta: {
+        docs: {
+          category: 'Node.js',
+          description: 'Validate document for environment variables',
+          recommended: false,
+        },
+        messages: {
+          duplicatedProcessEnv: 'Duplicated definition for environment variable: {{ name }}',
+          unDesribedProcessEnv: 'Undescribed environment variable: {{ name }}',
+          unDocumentedProcessEnv: 'Undocumented environment variable: {{ name }}',
+        },
+        schema: [],
+        type: 'problem',
+      },
     },
   },
 };
+
+export default plugin;

package/package.json

@@ -1,35 +1,38 @@
 {
   "name": "@mimik/eslint-plugin-document-env",
-  "version": "1.0.6",
+  "version": "2.0.1",
   "description": "validation of environement variable documentation",
-  "main": "index.js",
+  "main": "./index.js",
+  "type": "module",
+  "scripts": {
+    "lint": "eslint . --no-error-on-unmatched-pattern",
+    "test": "echo \"Error: no test specified\" && exit 0",
+     "prepublishOnly": "npm run lint && npm run test"
+  },
+  "husky": {
+    "hooks": {
+      "pre-commit": "npm run commit-ready",
+      "pre-push": "npm run test"
+    }
+  },
   "keywords": [
+    "mimik",
     "eslint",
     "eslint-plugin",
     "eslintplugin",
     "process.env"
   ],
-  "license": "MIT",
-  "files": [
-    "README.md",
-    "*.js"
-  ],
   "author": "mimik technology inc <support@mimik.com> (https://developer.mimik.com/)",
+  "license": "MIT",
   "repository": {
     "type": "git",
     "url": "https://bitbucket.org/mimiktech/eslit-plugin-document-env"
   },
-  "scripts": {
-    "lint": "eslint --ignore-path .gitignore .",
-    "test": "echo \"Error: no test specified\" && exit 0",
-     "prepublishOnly": "npm run lint && npm run test"
-  },
   "devDependencies": {
-    "eslint": "8.55.0",
-    "eslint-config-airbnb": "19.0.4",
-    "eslint-plugin-import": "2.29.0",
-    "eslint-plugin-jsx-a11y": "6.8.0",
-    "eslint-plugin-react": "7.33.2",
-    "eslint-plugin-react-hooks": "4.6.0"
+    "@eslint/js": "9.20.0",
+    "@stylistic/eslint-plugin": "3.1.0",
+    "eslint": "9.20.1",
+    "eslint-plugin-import": "2.31.0",
+    "husky": "9.1.7"
   }
 }