refactor: eslint various

apps/web/__tests__/support/commands.ts

@@ -2,6 +2,7 @@ import 'cypress-wait-until';
 import 'cypress-iframe';
 
 declare global {
+  // eslint-disable-next-line @typescript-eslint/no-namespace
   namespace Cypress {
     interface Chainable {
       getByTestId(id: string): Chainable;

apps/web/__tests__/test/unit/apiUrl.spec.ts

@@ -10,7 +10,7 @@ describe('validateApiUrl function', () => {
     });
 
     it('should remove multiple trailing slashes', () => {
-        const apiUrl = `${expectedApiUrl}/\//`;
+        const apiUrl = `${expectedApiUrl}///`;
         expect(validateApiUrl(apiUrl)).toEqual(expectedApiUrl);
     });
 

apps/web/__tests__/test/unit/i18n.spec.ts

@@ -1,7 +1,9 @@
 import { describe, expect, it } from 'vitest'
 import { getLocaleObject } from '../../../configuration/locale.config'
 
+// eslint-disable-next-line @typescript-eslint/no-require-imports
 const en = require('../../../lang/en.json')
+// eslint-disable-next-line @typescript-eslint/no-require-imports
 const de = require('../../../lang/de.json')
 
 describe('i18n', () => {

apps/web/build/logs/Logger.ts

@@ -31,6 +31,7 @@ class BuildLogger implements Logger {
 
   info(message: string): void {
     const formattedMessage = this.formatMessage('INFO', message);
+    // eslint-disable-next-line no-console
     console.log(formattedMessage);
     this.logStream.write(`${formattedMessage}\n`);
   }

apps/web/build/validators/FileTypeValidator.ts

@@ -1,3 +1,4 @@
+// eslint-disable-next-line @typescript-eslint/no-extraneous-class
 export class FileTypeValidator {
   public static isImage(fileName: string): boolean {
     const allowedImageTypes = ['png', 'jpg', 'jpeg', 'gif', 'svg', 'avif', 'webp'];

apps/web/components/CategoryFilters/types.ts

@@ -7,7 +7,3 @@ export type CategoryFiltersProps = {
 export type FilterProps = {
   facet?: FilterGroup;
 };
-
-export type FilterEmits = {
-  // (event: 'update:selected', parameter: FilterProps['selected']): void;
-};

apps/web/components/ui/Banner/__tests__/Banner.image.spec.ts

@@ -21,8 +21,6 @@ describe('Banner image', () => {
 
     const image = wrapper.find('[data-testid="banner-image-0"]');
 
-    console.log(image.html());
-
     expect(image.attributes('src')).toBe('');
     expect(image.attributes('alt')).toBe('');
     expect(image.attributes('style')).toContain('height: 576px');

apps/web/composables/usePayPal/usePayPal.ts

@@ -120,7 +120,7 @@ export const usePayPal = () => {
         return paypalScript;
       })
       .finally(() => {
-        delete state.value.loadingScripts[scriptKey];
+        delete state.value.loadingScripts.scriptKey;
       });
 
     return state.value.loadingScripts[scriptKey];

apps/web/composables/useProductAttributes/useProductAttributes.ts

@@ -129,13 +129,13 @@ export const useProductAttributes = (): UseProductAttributesReturn => {
     const value = item?.values.find((value) => value.attributeValueId === valueId) || undefined;
 
     if (!value || !valueId) {
-      delete state.value.attributeValues[attributeId];
+      delete state.value.attributeValues.attributeId;
       disableAttributes();
       return;
     }
 
     if (value.disabled) {
-      delete state.value.attributeValues[attributeId];
+      delete state.value.attributeValues.attributeId;
       const oldValues = { ...state.value.attributeValues };
       state.value.attributeValues = {};
       state.value.attributeValues[attributeId] = valueId;

apps/web/configuration/tailwind.config.ts

@@ -91,5 +91,6 @@ export default {
       },
     },
   },
+  // eslint-disable-next-line @typescript-eslint/no-require-imports
   plugins: [sfTypography, require('@savvywombat/tailwindcss-grid-areas')],
 } as Config;

apps/web/eslint.config.mjs

@@ -35,21 +35,11 @@ export default withNuxt(
      */
     rules: {
       'arrow-parens': 'off',
-      'no-console': 'off',
       'no-constant-binary-expression': 'off',
-      'no-useless-escape': 'off',
-      '@typescript-eslint/ban-ts-comment': 'off',
-      '@typescript-eslint/no-dynamic-delete': 'off',
-      '@typescript-eslint/no-empty-object-type': 'off',
       '@typescript-eslint/no-explicit-any': 'off',
-      '@typescript-eslint/no-extraneous-class': 'off',
-      '@typescript-eslint/no-invalid-void-type': 'off',
-      '@typescript-eslint/no-namespace': 'off',
-      '@typescript-eslint/no-require-imports': 'off',
       '@typescript-eslint/no-unused-expressions': ['error', { allowTernary: true }],
-      'vue/no-export-in-script-setup': 'off',
-      'vue/no-multiple-template-root': 'off',
-      'vue/no-v-html': 'off',
+      'vue/no-multiple-template-root': ['off'],
+      'vue/no-v-html': ['off'],
       'vue/html-self-closing': 'off',
     }
   },

apps/web/nuxt.config.ts

@@ -178,7 +178,6 @@ export default defineNuxtConfig({
       navigationPreload: true,
       runtimeCaching: [
         {
-          // @ts-ignore
           urlPattern: ({ request }) => request.mode === 'navigate',
           handler: 'NetworkOnly',
           options: {

apps/web/vitest.config.ts

@@ -15,7 +15,7 @@ export default defineVitestConfig({
     clearMocks: true,
     setupFiles: './vitest.config.setup.ts',
     include: ['**/*/?(*.)+(spec|test).[jt]s'],
-    onConsoleLog: (log: string): boolean | void => {
+    onConsoleLog: (log: string): boolean | undefined => {
       if (silenceLogsFromSuspenseComponent(log)) {
         return false;
       }