Vasty improve linting performance

.gitignore

@@ -33,4 +33,6 @@ bun.lock
 */build/*
 
 # IntelliJ
-.idea
+.idea
+.eslintcache
+tsconfig.tsbuildinfo

.husky/pre-push

@@ -1 +1,15 @@
-pnpm typecheck --minimumFailingSeverity warning
+if git diff --cached --name-only | grep -q '\.astro$'; then
+    echo "Running full astro check"
+    astro check --minimumFailingSeverity warning
+else
+    if [ -n "${DEBUG}" ]; then 
+        echo "Using TypeScript version: $(tsc --version) at $(which tsc)"
+        start=$(date +%s%N)
+        tsc --noEmit
+        end=$(date +%s%N)
+        echo "TypeScript time (approx): $(((end - start)/1000000))ms"
+    else 
+        echo "Running TypeScript check"
+        tsc --noEmit
+    fi
+fi

.remarkrc.mjs

@@ -12,19 +12,22 @@ export default {
         remarkPresetLintRecommended,
         remarkNoInlineCodeFences,
         remarkNoHtmlLinks,
-        [
-            remarkLintNoDeadUrls,
-            {
-                skipLocalhost: false,
-                skipOffline: true,
-                skipUrlPatterns: [
-                    'https://github.com/signup',
-                    'https://code.visualstudio.com/',
-                    'https://www.conventionalcommits.org/en/v1.0.0/',
-                    'https://vale.sh/',
-                    'https://squoosh.app/',
-                ], // Add known flaky URL patterns here
-            },
-        ],
+        // only run dead link checker in CI to save time in dev
+        process.env.CI
+            ? [
+                  remarkLintNoDeadUrls,
+                  {
+                      skipLocalhost: false,
+                      skipOffline: true,
+                      skipUrlPatterns: [
+                          'https://github.com/signup',
+                          'https://code.visualstudio.com/',
+                          'https://www.conventionalcommits.org/en/v1.0.0/',
+                          'https://vale.sh/',
+                          'https://squoosh.app/',
+                      ], // Add known flaky URL patterns here
+                  },
+              ]
+            : () => undefined,
     ],
 };

eslint.config.mjs

@@ -1,7 +1,6 @@
 import js from '@eslint/js';
 import astro from 'eslint-plugin-astro';
 import globals from 'globals';
-import process from 'node:process';
 import tseslint from 'typescript-eslint';
 
 import { defineConfig } from 'eslint/config';
@@ -21,8 +20,7 @@ export default defineConfig([
                 { argsIgnorePattern: '^_' },
             ],
             '@typescript-eslint/no-explicit-any': 'error',
-            // ci runs `astro check` which runs a full typescript checker
-            'no-undef': process.env.CI ? 'off' : 'error',
+            'no-undef': 'off',
         },
     },
     {
@@ -42,8 +40,5 @@ export default defineConfig([
                 ImageMetadata: 'readonly',
             },
         },
-        rules: {
-            'no-undef': 'off',
-        },
     },
 ]);

lint-staged.config.mjs

@@ -1,6 +1,25 @@
+const runPrettierOn =
+    '**/*.{' +
+    [
+        'js',
+        'mjs',
+        'ts',
+        'json',
+        'json5',
+        'jsonc',
+        'css',
+        'md',
+        'mdx',
+        'yaml',
+        'yml',
+        'astro',
+    ].join(',') +
+    '}';
+
 /** @type {import('lint-staged').Configuration} */
 export default {
-    '**/*': (files) => `prettier --write --ignore-unknown ${files.join(' ')}`,
+    [runPrettierOn]: (files) =>
+        `prettier --write --ignore-unknown ${files.join(' ')}`,
     '**/*.{astro,ts,mjs,js}': (files) => `eslint --fix ${files.join(' ')}`,
     'src/content/**/*.{md,mdx}': (files) => [
         `pnpm remark ${files.join(' ')} --ext mdx --frail --no-stdout --quiet`,

package.json

@@ -17,7 +17,7 @@
         "prebuild": "tsx scripts/generate-glossary.ts",
         "predev": "tsx scripts/generate-glossary.ts",
         "lint": "pnpm lint:eslint && pnpm lint:md && pnpm lint:regions && pnpm lint:glossary && pnpm typecheck && pnpm vale",
-        "lint:eslint": "eslint .",
+        "lint:eslint": "eslint \"src/**/*.{ts,astro,js,mjs,css}\" \"*.mjs\" --cache",
         "lint:md": "remark src/content/docs --ext mdx --frail",
         "lint:regions": "tsx scripts/validate-regions.ts",
         "vale": "pnpm vale:setup && vale sync && vale src/content",
@@ -40,12 +40,13 @@
         "@eslint/js": "^10.0.1",
         "@types/estree": "^1.0.9",
         "@types/mdast": "^4.0.4",
+        "@types/node": "^26.1.2",
         "astro-eslint-parser": "^1.4.0",
         "eslint": "^10.5.0",
         "eslint-plugin-astro": "^1.7.0",
-        "mdast-util-mdx": "^3.0.0",
         "husky": "^9.1.7",
         "lint-staged": "^17.0.8",
+        "mdast-util-mdx": "^3.0.0",
         "mdast-util-mdx-jsx": "^3.2.0",
         "mdx2vast": "^0.3.1",
         "prettier": "^3.8.4",
@@ -61,6 +62,7 @@
         "tsx": "^4.22.4",
         "typescript": "^6.0.3",
         "typescript-eslint": "^8.61.1",
-        "vfile": "^6.0.3"
+        "vfile": "^6.0.3",
+        "typescript-7": "npm:typescript@^7.0.2"
     }
 }