chore(eslint): Update ESLint configurations and fix linting scripts

Author: better-salmon

.changeset/changelog-github-custom.ts

@@ -12,13 +12,15 @@ const changelogFunctions: ChangelogFunctions = {
         'Please provide a repo to this changelog generator like this:\n"changelog": ["@changesets/changelog-github", { "repo": "org/repo" }]'
       );
     }
-    if (dependenciesUpdated.length === 0) return '';
+    if (dependenciesUpdated.length === 0) {
+      return '';
+    }
 
     const changesetLink = `- Updated dependencies [${(
       await Promise.all(
         changesets.map(async (cs) => {
           if (cs.commit) {
-            let { links } = await getInfo({
+            const { links } = await getInfo({
               repo: options.repo,
               commit: cs.commit,
             });
@@ -45,12 +47,14 @@ const changelogFunctions: ChangelogFunctions = {
 
     let prFromSummary: number | undefined;
     let commitFromSummary: string | undefined;
-    let usersFromSummary: string[] = [];
+    const usersFromSummary: string[] = [];
 
     const replacedChangelog = changeset.summary
       .replace(/^\s*(?:pr|pull|pull\s+request):\s*#?(\d+)/im, (_, pr) => {
-        let num = Number(pr);
-        if (!isNaN(num)) prFromSummary = num;
+        const num = Number(pr);
+        if (!isNaN(num)) {
+          prFromSummary = num;
+        }
         return '';
       })
       .replace(/^\s*commit:\s*([^\s]+)/im, (_, commit) => {
@@ -91,7 +95,7 @@ const changelogFunctions: ChangelogFunctions = {
       }
       const commitToFetchFrom = commitFromSummary || changeset.commit;
       if (commitToFetchFrom) {
-        let { links } = await getInfo({
+        const { links } = await getInfo({
           repo: options.repo,
           commit: commitToFetchFrom,
         });

.changeset/giant-carpets-brake.md

@@ -0,0 +1,7 @@
+---
+'create-qwik': patch
+'@builder.io/qwik-city': patch
+'@builder.io/qwik': patch
+---
+
+Fix linting errors which were previously being ignored across the monorepo.

eslint.config.js

@@ -22,23 +22,26 @@ const ignores = [
   '**/tsdoc-metadata.json',
   '**/.DS_Store',
   '**/*.mp4',
-  'scripts/**/*',
+  'scripts',
   '**/server/**/*.js',
   '**/*.tsbuildinfo',
-  'packages/docs/api/**/*',
+  'packages/docs/api',
   'packages/docs/public/repl/repl-sw.js*',
-  'packages/docs/src/routes/examples/apps/**/*',
-  'packages/docs/src/routes/playground/app/**/*',
-  'packages/docs/src/routes/tutorial/**/*',
-  'packages/qwik-labs/lib/**/*',
-  'packages/qwik-labs/lib-types/**/*',
-  'packages/qwik-labs/vite/**/*',
+  'packages/docs/src/routes/examples/apps',
+  'packages/docs/src/routes/playground/app',
+  'packages/docs/src/routes/tutorial',
+  'packages/qwik-labs/lib',
+  'packages/qwik-labs/lib-types',
+  'packages/qwik-labs/vite',
   'packages/insights/drizzle.config.ts',
   'packages/insights/panda.config.ts',
   'starters/apps/base',
   'starters/apps/library',
   'starters/templates',
   '**/vite.config.ts',
+  // packages with eslint.config.mjs
+  'packages/qwik-labs',
+  'packages/insights',
 ];
 
 export default tseslint.config(
@@ -89,6 +92,17 @@ export default tseslint.config(
       '@typescript-eslint/no-var-requires': 'off',
       curly: 'error',
       'no-new-func': 'error',
+      '@typescript-eslint/no-empty-object-type': 'off',
+      '@typescript-eslint/no-unused-expressions': 'off',
+      '@typescript-eslint/no-unsafe-function-type': 'off',
+      '@typescript-eslint/no-require-imports': 'off',
+      '@typescript-eslint/no-wrapper-object-types': 'off',
+    },
+  },
+  {
+    files: ['packages/docs/**/*.{ts,tsx}'],
+    rules: {
+      'no-console': 'off',
     },
   }
 );

package.json

@@ -226,8 +226,8 @@
     "link.dist.npm": "cd packages/qwik && npm link && cd ../qwik-city && npm link && cd ../eslint-plugin-qwik && npm link && cd ../qwik-react && npm link",
     "link.dist.yarn": "cd packages/qwik && yarn link && cd ../qwik-city && yarn link && cd ../eslint-plugin-qwik && yarn link && cd ../qwik-react && yarn link",
     "lint": "pnpm lint.eslint && pnpm lint.prettier && pnpm lint.rust",
-    "lint.eslint": "eslint --cache \"**/*.ts*\"",
-    "lint.fix": "eslint --fix \"**/*.ts*\" && pnpm prettier.fix",
+    "lint.eslint": "eslint --cache \"**/*.ts*\" && pnpm -r --parallel lint",
+    "lint.fix": "eslint --fix \"**/*.ts*\" && pnpm -r --parallel lint.fix && pnpm prettier.fix",
     "lint.prettier": "prettier --cache --check .",
     "lint.rust": "make lint",
     "lint.syncpack": "syncpack list-mismatches",

packages/create-qwik/index.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import { runCreateCli } from './src/run-create-cli';
 import { runCreateInteractiveCli } from './src/run-create-interactive-cli';
 import { panic, printHeader } from '../qwik/src/cli/utils/utils';

packages/create-qwik/src/run-create-interactive-cli.ts

@@ -7,7 +7,7 @@ import { join, relative } from 'node:path';
 import type { CreateAppResult } from '../../qwik/src/cli/types';
 import { clearDir } from './helpers/clearDir';
 import { createApp } from './create-app';
-/* eslint-disable no-console */
+
 import fs from 'node:fs';
 import { getRandomJoke } from './helpers/jokes';
 import { installDepsCli } from './helpers/installDepsCli';

packages/docs/codesandbox.sync.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import { readdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 

packages/docs/contributors.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import { fetch } from 'undici';
 import fs from 'node:fs';
 import path from 'node:path';

packages/docs/src/components/router-head/router-head.tsx

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import { component$ } from '@builder.io/qwik';
 import { useDocumentHead, useLocation } from '@builder.io/qwik-city';
 import { Social } from './social';

packages/docs/src/repl/repl-version.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import { QWIK_PKG_NAME, bundled } from './bundled';
 
 const bundledVersion = bundled[QWIK_PKG_NAME].version;