chore: add ESLint config file and deps

Author: Soare-Robert-Daniel

.eslintrc

@@ -0,0 +1,52 @@
+{
+    "root": true,
+    "env": {
+      "browser": true
+    },
+    "extends": "plugin:@wordpress/eslint-plugin/recommended",
+    "parserOptions": {
+      "ecmaFeatures": {
+        "jsx": true
+      },
+      "ecmaVersion": "latest"
+    },
+    "ignorePatterns": ["**/vendor/**", "**/build/**", "**/*.min.js", "**/node_modules/**"],
+    "rules": {
+      "@wordpress/i18n-text-domain": ["error", {
+        "allowedTextDomain": ["feedzy-rss-feeds"]
+      }],
+      "@wordpress/i18n-no-flanking-whitespace": "error",
+      "@wordpress/data-no-store-string-literals": "error",
+      "@wordpress/wp-global-usage": "error",
+      "prettier/prettier": "warn",
+      "import/no-extraneous-dependencies": "warn",
+      "prefer-const": "warn",
+      "dot-notation": "warn",
+      "react/jsx-key": "warn",
+      "jsx-a11y/no-static-element-interactions": "warn",
+      "no-undef": "warn",
+      "object-shorthand": "warn",
+      "no-else-return": "warn",
+      "jsx-a11y/click-events-have-key-events": "warn",
+      "jsx-a11y/anchor-is-valid": "warn",
+      "camelcase": "warn",
+      "jsdoc/check-line-alignment": "warn",
+      "jsdoc/check-alignment": "warn",
+      "jsx-a11y/label-has-associated-control": "warn",
+      "react/no-unknown-property": "warn",
+      "@wordpress/no-unused-vars-before-return": "warn",
+      "jsdoc/empty-tags": "warn",
+      "no-lonely-if": "warn",
+      "no-var": "warn",
+      "no-unused-vars": "warn",
+      "no-console": "warn",
+      "jsx-a11y/alt-text": "warn",
+      "react/no-deprecated": "warn",
+      "jsx-a11y/no-noninteractive-element-interactions": "warn",
+      "jsx-a11y/no-autofocus": "warn",
+      "no-nested-ternary": "warn",
+      "jsdoc/require-param": "warn",
+      "jsdoc/require-returns-description": "warn",
+      "jsdoc/require-returns-type": "warn"
+    }
+  }

.github/workflows/test-lint-js.yml

@@ -1,31 +1,27 @@
-name: NPM Tests
+name: Check JS files and build
 concurrency:
   group: ${{ github.workflow }}-${{ github.event_name == 'pull_request' && github.head_ref || github.ref }}
   cancel-in-progress: true
 on:
   push:
     branches-ignore:
-      - 'master'
-      - 'dependabot/**'
+      - "master"
+      - "dependabot/**"
 
 jobs:
   npm:
     runs-on: ubuntu-latest
-    env:
-      CYPRESS_INSTALL_BINARY: 0
-    strategy:
-      matrix:
-        node-version: [18.x]
     steps:
       - name: Checkout source code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
       - name: Setup node
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: "18"
+          cache: "npm"
       - name: Install Dependencies
         run: npm install
       - name: Run JS check
-        run: npm run lint:js || true # Remove `|| true` when working on https://github.com/Codeinwp/feedzy-rss-feeds/issues/948
-      - name: Build source
+        run: npm run lint:js
+      - name: Check if we can build the project
         run: npm run build

js/ActionPopup/action-popup.js

@@ -212,13 +212,13 @@ const ActionModal = () => {
 				}
 			}
 			let tag = event.target.getAttribute( 'data-action_popup' ) || '';
-			let fieldName = event.target.getAttribute( 'data-field-name' ) || '';
+			let dataFieldName = event.target.getAttribute( 'data-field-name' ) || '';
 			if ( '' === tag ) {
 				event.target.closest('.dropdown-item').click();
 				return;
 			}
 			setShortCode( tag );
-			setFieldName( fieldName );
+			setFieldName( dataFieldName );
 			openModal();
 		} );
 	} );

js/FeedBack/feedback-form.js

@@ -41,7 +41,7 @@ const helpTextByStatus = {
  * Displays a button that opens a modal for sending feedback
  *
  * @param {import('./type').FeedbackFormProps} props
- * @returns
+ * @return
  */
 const FeedbackForm = ({
 	source,

js/FeedBack/feedback.js

@@ -34,7 +34,7 @@ const FeedBack = () => {
 
 	document.querySelectorAll( '[role="button"]' ).forEach((button) => {
 		button.addEventListener('keydown', (evt) => { 
-			if(evt.keyCode == 13 || evt.keyCode == 32) {
+			if(evt.keyCode === 13 || evt.keyCode === 32) {
 				button.click();
 			}
 		});

js/FeedzyBlock/utils.js

@@ -62,7 +62,9 @@ export const filterData = ( arr, sortType, allowedKeywords, bannedKeywords, maxS
 };
 
 export const inArray = ( value, arr ) => {
-	if ( arr === undefined ) return false;
+	if ( arr === undefined ) {
+		return false;
+	};
 	let exists = false;
 	for( let i = 0; i < arr.length; i++ ) {
 		let name = arr[i];