infra: reduce tsconfig complexity (#2102)

Author: Shinigami92

.eslintrc.js

@@ -21,7 +21,7 @@ module.exports = defineConfig({
   ],
   parser: '@typescript-eslint/parser',
   parserOptions: {
-    project: ['./tsconfig.lint.json'],
+    project: ['./tsconfig.json'],
     sourceType: 'module',
     warnOnUnsupportedTypeScriptVersion: false,
   },

.github/workflows/ci.yml

@@ -167,10 +167,10 @@ jobs:
       - name: Check formatting
         run: pnpm prettier --check .
 
-  ts-check-scripts:
+  ts-check:
     runs-on: ubuntu-latest
     timeout-minutes: 10
-    name: 'TS-Check Scripts: node-18, ubuntu-latest'
+    name: 'TS-Check: node-18, ubuntu-latest'
     steps:
       - name: Checkout
         uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9 # v3.5.3
@@ -192,34 +192,7 @@ jobs:
           CYPRESS_INSTALL_BINARY: 0
 
       - name: Check scripts
-        run: pnpm run ts-check:scripts
-
-  ts-check-tests:
-    runs-on: ubuntu-latest
-    timeout-minutes: 10
-    name: 'TS-Check Tests: node-18, ubuntu-latest'
-    steps:
-      - name: Checkout
-        uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9 # v3.5.3
-        with:
-          fetch-depth: 0
-
-      - name: Install pnpm
-        uses: pnpm/action-setup@c3b53f6a16e57305370b4ae5a540c2077a1d50dd # v2.2.4
-
-      - name: Set node version to 18
-        uses: actions/setup-node@64ed1c7eab4cce3362f8c340dee64e5eaeef8f7c # v3.6.0
-        with:
-          node-version: 18
-          cache: 'pnpm'
-
-      - name: Install deps
-        run: pnpm install
-        env:
-          CYPRESS_INSTALL_BINARY: 0
-
-      - name: Check tests
-        run: pnpm run ts-check:tests
+        run: pnpm run ts-check
 
   codecov:
     runs-on: ubuntu-latest

CONTRIBUTING.md

@@ -16,8 +16,7 @@ This is a shorthand for running the following scripts in order:
 - `pnpm run build:code` - builds the code, both CommonJS and ESM versions
 - `pnpm run build:types` - builds the TypeScript type definitions
 - `pnpm run test:update-snapshots ` - runs all tests, and updates any snapshots if needed
-- `pnpm run ts-check:scripts` - checks that there are no TypeScript errors in script files
-- `pnpm run ts-check:tests` - checks that there are no TypeScript errors in test or script files
+- `pnpm run ts-check` - checks that there are no TypeScript errors in any files
 
 ## Good to know
 

docs/.vitepress/tsconfig.json

@@ -53,16 +53,14 @@
   },
   "files": [
     "CHANGELOG.md",
-    "CHANGELOG_old.md",
     "dist",
-    "locale",
-    "tsconfig.json"
+    "locale"
   ],
   "scripts": {
     "clean": "rimraf coverage .eslintcache dist docs/.vitepress/dist pnpm-lock.yaml node_modules",
     "build:clean": "rimraf dist",
     "build:code": "tsx ./scripts/bundle.ts",
-    "build:types": "tsc --emitDeclarationOnly --outDir dist/types",
+    "build:types": "tsc --project tsconfig.build.json",
     "build": "run-s build:clean build:code build:types",
     "generate": "run-s generate:locales generate:api-docs",
     "generate:api-docs": "tsx ./scripts/apidoc.ts",
@@ -77,9 +75,7 @@
     "docs:diff": "tsx ./scripts/diff.ts",
     "format": "prettier --cache --write .",
     "lint": "eslint --cache --cache-strategy content --report-unused-disable-directives .",
-    "ts-check": "run-s ts-check:scripts ts-check:tests",
-    "ts-check:scripts": "tsc --project tsconfig.check-scripts.json",
-    "ts-check:tests": "tsc --project tsconfig.check-tests.json",
+    "ts-check": "tsc",
     "test": "vitest",
     "test:update-snapshots": "vitest run -u",
     "coverage": "vitest run --coverage",

package.json

@@ -34,7 +34,6 @@ export function loadExampleMethods(): Record<string, SignatureReflection> {
   return loadProjectModules(
     {
       entryPoints: ['test/scripts/apidoc/signature.example.ts'],
-      tsconfig: 'test/scripts/apidoc/tsconfig.json',
     },
     true
   )['SignatureTest'][1];
@@ -47,7 +46,6 @@ export function loadExampleModules(): Record<string, DeclarationReflection> {
   const modules = loadProjectModules(
     {
       entryPoints: ['test/scripts/apidoc/module.example.ts'],
-      tsconfig: 'test/scripts/apidoc/tsconfig.json',
     },
     true
   );

test/scripts/apidoc/tsconfig.json

@@ -0,0 +1,17 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "target": "ES2019",
+    "noEmit": false,
+    "emitDeclarationOnly": true,
+    "rootDir": "src",
+    "outDir": "dist/types",
+
+    // This negates what is set in the extended tsconfig.json
+    "noImplicitAny": true,
+    "skipLibCheck": false,
+    "allowSyntheticDefaultImports": false,
+    "resolveJsonModule": false
+  },
+  "include": ["src"]
+}

test/scripts/apidoc/utils.ts

@@ -1,20 +1,22 @@
 {
   "compilerOptions": {
-    "target": "ES2019",
-    "moduleResolution": "Node",
-    "rootDir": "src",
-    "outDir": "dist",
+    "target": "ESNext",
+    "moduleResolution": "node",
+    "module": "ESNext",
+    "strict": true,
+    "noEmit": true,
     "declaration": true,
-    "esModuleInterop": true,
-    "allowJs": true,
-    "alwaysStrict": true,
-    "strictFunctionTypes": true,
-    "noImplicitAny": true,
-    "noImplicitThis": true,
-    "useUnknownInCatchVariables": true,
     "stripInternal": true,
-    "baseUrl": "."
+
+    // We need to disable these for now, and need to tackle them in another PR
+    "strictNullChecks": false,
+    "strictBindCallApply": false,
+    "noImplicitAny": false,
+
+    // These are configs specifically for !build and have to be reverted in the tsconfig.build.json
+    "skipLibCheck": true,
+    "allowSyntheticDefaultImports": true,
+    "resolveJsonModule": true
   },
-  "include": ["src/**/*"],
-  "exclude": ["node_modules"]
+  "exclude": ["node_modules", "dist", "locale"]
 }