chore: apply linters

wayofdev · Jul 17, 2023 · f1156f5 · f1156f5
1 parent 112d425
commit f1156f5
Show file tree

Hide file tree

Showing 10 changed files with 20 additions and 79 deletions.
diff --git a/apps/web/.eslintrc.cjs b/apps/web/.eslintrc.cjs
@@ -7,9 +7,7 @@
 // Workaround for https://github.com/eslint/eslint/issues/3458 (re-export of @rushstack/eslint-patch)
 require('@wayofdev/eslint-config-bases/patch/modern-module-resolution')
 
-const {
-  getDefaultIgnorePatterns,
-} = require('@wayofdev/eslint-config-bases/helpers')
+const { getDefaultIgnorePatterns } = require('@wayofdev/eslint-config-bases/helpers')
 
 module.exports = {
   root: true,

diff --git a/apps/web/jest.config.js b/apps/web/jest.config.js
@@ -52,11 +52,7 @@ const config = {
   // false by default, overrides in cli, ie: yarn test:unit --collect-coverage=true
   collectCoverage: false,
   coverageDirectory: '<rootDir>/../coverage',
-  collectCoverageFrom: [
-    '<rootDir>/**/*.{ts,tsx,js,jsx}',
-    '!**/*.test.{js,ts}',
-    '!**/__mock__/*',
-  ],
+  collectCoverageFrom: ['<rootDir>/**/*.{ts,tsx,js,jsx}', '!**/*.test.{js,ts}', '!**/__mock__/*'],
   transformIgnorePatterns: ['/node_modules/(?!@vercel/analytics)/'],
 }
 

diff --git a/apps/web/lint-staged.config.js → apps/web/lint-staged.config.cjs b/apps/web/lint-staged.config.js → apps/web/lint-staged.config.cjs
diff --git a/apps/web/next.config.mjs b/apps/web/next.config.mjs
@@ -21,11 +21,7 @@ import { getValidatedServerEnv } from './src/config/validated-server-env.mjs'
 // validate server env
 getValidatedServerEnv()
 
-const workspaceRoot = path.resolve(
-  path.dirname(url.fileURLToPath(import.meta.url)),
-  '..',
-  '..'
-)
+const workspaceRoot = path.resolve(path.dirname(url.fileURLToPath(import.meta.url)), '..', '..')
 
 /**
  * Once supported replace by node / eslint / ts and out of experimental, replace by
@@ -41,18 +37,10 @@ const isProd = process.env.NODE_ENV === 'production'
 const isCI = trueEnv.includes(process.env?.CI ?? 'false')
 const enableCSP = true
 
-const NEXT_STANDALONE_BUILD = trueEnv.includes(
-  process.env?.NEXT_STANDALONE_BUILD ?? 'false'
-)
-const NEXT_IGNORE_TYPE_CHECK = trueEnv.includes(
-  process.env?.NEXT_IGNORE_TYPE_CHECK ?? 'false'
-)
-const NEXT_IGNORE_ESLINT = trueEnv.includes(
-  process.env?.NEXT_IGNORE_ESLINT ?? 'false'
-)
-const SENTRY_UPLOAD_DRY_RUN = trueEnv.includes(
-  process.env?.SENTRY_UPLOAD_DRY_RUN ?? 'false'
-)
+const NEXT_STANDALONE_BUILD = trueEnv.includes(process.env?.NEXT_STANDALONE_BUILD ?? 'false')
+const NEXT_IGNORE_TYPE_CHECK = trueEnv.includes(process.env?.NEXT_IGNORE_TYPE_CHECK ?? 'false')
+const NEXT_IGNORE_ESLINT = trueEnv.includes(process.env?.NEXT_IGNORE_ESLINT ?? 'false')
+const SENTRY_UPLOAD_DRY_RUN = trueEnv.includes(process.env?.SENTRY_UPLOAD_DRY_RUN ?? 'false')
 const DISABLE_SENTRY = trueEnv.includes(process.env?.DISABLE_SENTRY ?? 'false')
 const SENTRY_DEBUG = trueEnv.includes(process.env?.SENTRY_DEBUG ?? 'false')
 const SENTRY_TRACING = trueEnv.includes(process.env?.SENTRY_TRACING ?? 'false')
@@ -62,9 +50,7 @@ const SENTRY_TRACING = trueEnv.includes(process.env?.SENTRY_TRACING ?? 'false')
  * to deliver an image or deploy the files.
  * @link https://nextjs.org/docs/advanced-features/source-maps
  */
-const disableSourceMaps = trueEnv.includes(
-  process.env?.NEXT_DISABLE_SOURCEMAPS ?? 'false'
-)
+const disableSourceMaps = trueEnv.includes(process.env?.NEXT_DISABLE_SOURCEMAPS ?? 'false')
 
 if (disableSourceMaps) {
   console.log(
@@ -116,10 +102,7 @@ const secureHeaders = createSecureHeaders({
   },
   ...(enableCSP && process.env.NODE_ENV === 'production'
     ? {
-        forceHTTPSRedirect: [
-          true,
-          { maxAge: 60 * 60 * 24 * 4, includeSubDomains: true },
-        ],
+        forceHTTPSRedirect: [true, { maxAge: 60 * 60 * 24 * 4, includeSubDomains: true }],
       }
     : {}),
   referrerPolicy: 'same-origin',
@@ -185,9 +168,7 @@ const nextConfig = {
 
   // Standalone build
   // @link https://nextjs.org/docs/advanced-features/output-file-tracing#automatically-copying-traced-files-experimental
-  ...(NEXT_STANDALONE_BUILD
-    ? { output: 'standalone', outputFileTracing: true }
-    : {}),
+  ...(NEXT_STANDALONE_BUILD ? { output: 'standalone', outputFileTracing: true } : {}),
 
   experimental: {
     // @link https://nextjs.org/docs/advanced-features/output-file-tracing#caveats

diff --git a/apps/web/sentry.client.config.ts b/apps/web/sentry.client.config.ts
@@ -13,9 +13,7 @@ sentryInit({
   // @see https://develop.sentry.dev/sdk/performance/
   // To turn it off, remove the line
   // @see https://github.com/getsentry/sentry-javascript/discussions/4503#discussioncomment-2143116
-  tracesSampleRate: ['false', '0'].includes(process.env.SENTRY_TRACING ?? '')
-    ? undefined
-    : 1,
+  tracesSampleRate: ['false', '0'].includes(process.env.SENTRY_TRACING ?? '') ? undefined : 1,
 
   // Note: The Replay integration only needs to be added to your sentry.client.config.js file.
   // It will not run if it is added into sentry.server.config.js.

diff --git a/apps/web/src/config/validated-server-env.mjs b/apps/web/src/config/validated-server-env.mjs
@@ -24,11 +24,7 @@ export const getValidatedServerEnv = () => {
       process.exit(1)
     } else {
       throw new Error(
-        `Invalid server env(s): ${JSON.stringify(
-          parsedEnv.error.format(),
-          null,
-          2
-        )}}`
+        `Invalid server env(s): ${JSON.stringify(parsedEnv.error.format(), null, 2)}}`
       )
     }
   }

diff --git a/apps/web/src/features/auth/components/LoginForm.tsx b/apps/web/src/features/auth/components/LoginForm.tsx
@@ -38,12 +38,7 @@ export const LoginForm: FC<Props> = _props => {
       // callbackUrl: '/',
       redirect: false,
     })
-    const {
-      ok = false,
-      url,
-      status = 500,
-      error = 'Server or network Error',
-    } = result ?? {}
+    const { ok = false, url, status = 500, error = 'Server or network Error' } = result ?? {}
 
     if (ok) {
       console.log('Will redirect to ' + url)
@@ -67,9 +62,7 @@ export const LoginForm: FC<Props> = _props => {
           placeholder="Username or email"
           className="mt-1 block w-full rounded-md border-gray-300 shadow-sm focus:border-indigo-500 focus:ring-indigo-500 sm:text-sm"
         />
-        {errors?.username && (
-          <p className={'bg-amber-600'}>{errors.username.message}</p>
-        )}
+        {errors?.username && <p className={'bg-amber-600'}>{errors.username.message}</p>}
       </label>
       <label>
         Password
@@ -83,9 +76,7 @@ export const LoginForm: FC<Props> = _props => {
           placeholder="Password"
           className="mt-1 block w-full rounded-md border-gray-300 shadow-sm focus:border-indigo-500 focus:ring-indigo-500 sm:text-sm"
         />
-        {errors?.password && (
-          <p className={'bg-amber-600'}>{errors.password.message}</p>
-        )}
+        {errors?.password && <p className={'bg-amber-600'}>{errors.password.message}</p>}
       </label>
       <button
         type="submit"

diff --git a/apps/web/src/pages/_document.tsx b/apps/web/src/pages/_document.tsx
@@ -6,9 +6,7 @@ import Document, { Html, Head, Main, NextScript } from 'next/document'
 import { defaultLocale } from '../../next-i18next.config.mjs'
 
 class MyDocument extends Document {
-  static async getInitialProps(
-    ctx: DocumentContext
-  ): Promise<DocumentInitialProps> {
+  static async getInitialProps(ctx: DocumentContext): Promise<DocumentInitialProps> {
     return await Document.getInitialProps(ctx)
   }
 
@@ -38,17 +36,10 @@ class MyDocument extends Document {
             href="/images/favicon/favicon-16x16.png"
           />
           <link rel="manifest" href="/images/favicon/site.webmanifest" />
-          <link
-            rel="mask-icon"
-            href="/images/favicon/safari-pinned-tab.svg"
-            color="#5bbad5"
-          />
+          <link rel="mask-icon" href="/images/favicon/safari-pinned-tab.svg" color="#5bbad5" />
           <link rel="shortcut icon" href="/images/favicon/favicon.ico" />
           <meta name="msapplication-TileColor" content="#ffffff" />
-          <meta
-            name="msapplication-config"
-            content="/images/favicon/browserconfig.xml"
-          />
+          <meta name="msapplication-config" content="/images/favicon/browserconfig.xml" />
           <meta name="theme-color" content="#ffffff" />
         </Head>
         <body>

diff --git a/apps/web/src/themes/shared/browser-fonts.js b/apps/web/src/themes/shared/browser-fonts.js
@@ -21,14 +21,7 @@ export const browserFonts = {
     'Segoe UI Symbol',
     'Noto Color Emoji',
   ],
-  serif: [
-    'ui-serif',
-    'Georgia',
-    'Cambria',
-    'Times New Roman',
-    'Times',
-    'serif',
-  ],
+  serif: ['ui-serif', 'Georgia', 'Cambria', 'Times New Roman', 'Times', 'serif'],
   mono: [
     'ui-monospace',
     'SFMono-Regular',

diff --git a/apps/web/tailwind.config.ts b/apps/web/tailwind.config.ts
@@ -4,10 +4,7 @@ import { tailwindV3Colors } from './src/themes/shared/colors'
 import { tailwindTheme } from './src/themes/tailwind/tailwind.theme'
 
 const tailwindConfig: Config = {
-  content: [
-    './src/**/*.(js|jsx|ts|tsx)',
-    './node_modules/@wayofdev/ui/src/**/*.(js|jsx|ts|tsx)',
-  ],
+  content: ['./src/**/*.(js|jsx|ts|tsx)', './node_modules/@wayofdev/ui/src/**/*.(js|jsx|ts|tsx)'],
   theme: {
     screens: {
       ...defaultTheme.screens,