fix(typescript): Use consistent typescript configs

make all packages inherit tsconfig from the root tsconfig

docker/images/n8n-custom/Dockerfile

@@ -7,7 +7,7 @@ RUN \
 	apk --no-cache add git && \
 	npm install -g run-script-os turbo
 
-COPY turbo.json package.json package-lock.json ./
+COPY turbo.json package.json package-lock.json tsconfig.json ./
 COPY packages ./packages
 
 RUN chown -R node:node .
@@ -21,7 +21,7 @@ RUN \
 	# TODO: removing dev dependecies is deleting `bn.js`, which breaks the Snowflake node
 	npm prune --omit=dev && \
 	npm i --omit=dev bn.js && \
-	find . -type f -name "*.ts" -o -name "*.js.map" -o -name "*.vue" | xargs rm &&\
+	find . -type f -name "*.ts" -o -name "*.js.map" -o -name "*.vue" -o -name "tsconfig.json" | xargs rm &&\
 	rm -rf node_modules/.cache packages/*/node_modules/.cache packages/*/.turbo .config .npm /tmp/*
 
 

packages/cli/jest.config.js

@@ -1,7 +1,7 @@
 module.exports = {
 	verbose: true,
 	transform: {
-		'^.+\\.ts?$': 'ts-jest',
+		'^.+\\.ts$': 'ts-jest',
 	},
 	testURL: 'http://localhost/',
 	testRegex: '(/__tests__/.*|(\\.|/)(test))\\.ts$',

packages/cli/tsconfig.json

@@ -1,27 +1,16 @@
 {
+	"extends": "../../tsconfig.json",
 	"compilerOptions": {
-		"lib": [
-			"es2017",
-			"ES2020.Promise"
-		],
+		"outDir": "dist",
 		"types": [
 			"node",
 			"jest"
 		],
-		"module": "commonjs",
-		"noImplicitAny": true,
-		"removeComments": true,
-		"strictNullChecks": true,
-		"forceConsistentCasingInFileNames": true,
-		"noImplicitReturns": true,
-		"preserveConstEnums": true,
-		"declaration": true,
-		"outDir": "./dist/",
-		"target": "es2017",
-		"sourceMap": true,
 		"emitDecoratorMetadata": true,
 		"experimentalDecorators": true,
-		"esModuleInterop": true
+		// TODO: remove all options below this line
+		"strict": false,
+		"noUnusedLocals": false,
 	},
 	"include": [
 		"**/*.d.ts",
@@ -30,10 +19,5 @@
 		"config/**/*",
 		"src/**/*",
 		"test/**/*"
-	],
-	"exclude": [
-		"dist/**/*",
-		"node_modules/**/*",
-		"**/*.spec.ts"
 	]
 }

packages/core/package.json

@@ -61,17 +61,16 @@
   },
   "jest": {
     "transform": {
-      "^.+\\.tsx?$": "ts-jest"
+      "^.+\\.ts$": "ts-jest"
     },
     "testURL": "http://localhost/",
-    "testRegex": "(/__tests__/.*|(\\.|/)(test|spec))\\.(jsx?|tsx?)$",
+    "testRegex": "(/__tests__/.*|(\\.|/)(test|spec))\\.(js|ts)$",
     "testPathIgnorePatterns": [
       "/dist/",
       "/node_modules/"
     ],
     "moduleFileExtensions": [
       "ts",
-      "tsx",
       "js",
       "json",
       "node"

packages/core/tsconfig.json

@@ -1,34 +1,15 @@
 {
+	"extends": "../../tsconfig.json",
 	"compilerOptions": {
-		"lib": [
-			"es2019"
-		],
+		"outDir": "dist",
 		"types": [
 			"node",
 			"jest"
 		],
-		"module": "commonjs",
-		"removeComments": true,
-		"forceConsistentCasingInFileNames": true,
-		"noImplicitReturns": true,
-		"strict": true,
-		"noUnusedLocals": true,
-		"preserveConstEnums": true,
-		"declaration": true,
-		"outDir": "./dist/",
-		"target": "es2019",
-		"sourceMap": true,
-		"esModuleInterop": true,
-		"useUnknownInCatchVariables": false,
 	},
 	"include": [
 		"**/*.d.ts",
 		"src/**/*",
 		"test/**/*"
-	],
-	"exclude": [
-		"dist/**/*",
-		"node_modules/**/*",
-		"**/*.spec.ts"
 	]
 }

packages/design-system/.storybook/main.js

@@ -4,7 +4,7 @@ const path = require('path');
  * @type {import('@storybook/core-common').StorybookConfig}
  */
 module.exports = {
-	stories: ['../src/**/*.stories.mdx', '../src/**/*.stories.@(js|jsx|ts|tsx)'],
+	stories: ['../src/**/*.stories.mdx', '../src/**/*.stories.@(js|ts)'],
 	addons: [
 		'@storybook/addon-links',
 		'@storybook/addon-essentials',

packages/design-system/tsconfig.json

@@ -1,37 +1,40 @@
 {
+	"extends": "../../tsconfig.json",
 	"compilerOptions": {
+		"outDir": "dist",
 		"target": "esnext",
 		"module": "esnext",
-		"strict": true,
 		"importHelpers": true,
-		"moduleResolution": "node",
-		"declaration": true,
-		"experimentalDecorators": true,
 		"skipLibCheck": true,
-		"esModuleInterop": true,
 		"allowJs": true,
 		"allowSyntheticDefaultImports": true,
-		"sourceMap": true,
 		"baseUrl": ".",
-	    "outDir": "dist",
 		"types": [
-		  "webpack-env",
-		  "jest",
-		  "vitest/globals"
+			"webpack-env",
+			"jest",
+			"vitest/globals"
 		],
-	  	"typeRoots": [
-		  "@testing-library",
-		  "@types"
+		"typeRoots": [
+			"@testing-library",
+			"@types"
 		],
 		"paths": {
-			"@/*": ["src/*"]
+			"@/*": [
+				"src/*"
+			]
 		},
-		"lib": ["esnext", "dom", "dom.iterable", "scripthost"]
+		"lib": [
+			"esnext",
+			"dom",
+			"dom.iterable",
+			"scripthost"
+		],
+		// TODO: remove all options below this line
+		"noUnusedLocals": false,
+		"noImplicitReturns": false,
 	},
 	"include": [
 		"src/**/*.ts",
-		"src/**/*.tsx",
 		"src/**/*.vue"
-	],
-	"exclude": ["node_modules"]
+	]
 }

packages/editor-ui/jest.config.js

@@ -1,16 +1,14 @@
 module.exports = {
 	moduleFileExtensions: [
 		'js',
-		'jsx',
 		'json',
 		'vue',
 		'ts',
-		'tsx',
 	],
 	transform: {
 		'^.+\\.vue$': 'vue-jest',
 		'.+\\.(css|styl|less|sass|scss|svg|png|jpg|ttf|woff|woff2)$': 'jest-transform-stub',
-		'^.+\\.tsx?$': 'ts-jest',
+		'^.+\\.ts$': 'ts-jest',
 	},
 	moduleNameMapper: {
 		'^@/(.*)$': '<rootDir>/src/$1',
@@ -19,7 +17,7 @@ module.exports = {
 		'jest-serializer-vue',
 	],
 	testMatch: [
-		'**/tests/unit/**/*.spec.(js|jsx|ts|tsx)|**/__tests__/*.(js|jsx|ts|tsx)',
+		'**/tests/unit/**/*.spec.(js|ts)|**/__tests__/*.(js|ts)',
 	],
 	testURL: 'http://localhost/',
 };

packages/editor-ui/tsconfig.json

@@ -1,20 +1,12 @@
 {
+	"extends": "../../tsconfig.json",
 	"compilerOptions": {
 		"target": "esnext",
 		"module": "esnext",
-		"removeComments": true,
-		"forceConsistentCasingInFileNames": true,
-		"noImplicitReturns": true,
-		"strict": true,
-		"jsx": "preserve",
 		"skipLibCheck": true,
+		"allowJs": true,
 		"importHelpers": true,
-		"moduleResolution": "node",
-		"esModuleInterop": true,
 		"allowSyntheticDefaultImports": true,
-		"useUnknownInCatchVariables": false,
-		"resolveJsonModule": true,
-		"sourceMap": true,
 		"baseUrl": ".",
 		"types": [
 			"webpack-env",
@@ -30,16 +22,13 @@
 			"dom",
 			"dom.iterable",
 			"scripthost"
-		]
+		],
+		// TODO: remove all options below this line
+		"noUnusedLocals": false,
 	},
 	"include": [
 		"src/**/*.ts",
-		"src/**/*.tsx",
 		"src/**/*.vue",
 		"tests/**/*.ts",
-		"tests/**/*.tsx",
-	],
-	"exclude": [
-		"node_modules"
 	]
 }

packages/node-dev/src/tsconfig-build.json

@@ -1,21 +1,10 @@
 {
+	"extends": "../../tsconfig.json",
 	"compilerOptions": {
-		"lib": [
-			"es2017"
-		],
-		"types": [],
-		"module": "commonjs",
 		"importHelpers": true,
-		"preserveConstEnums": true,
-		"declaration": true,
-		"target": "es2017",
-		"sourceMap": true
 	},
 	"include": [
 		"*.credentials.ts",
 		"*.node.ts"
-	],
-	"exclude": [
-		"node_modules"
 	]
 }

packages/node-dev/tsconfig.json

@@ -1,31 +1,17 @@
 {
+	"extends": "../../tsconfig.json",
 	"compilerOptions": {
-		"lib": [
-			"es2019"
-		],
+		"outDir": "dist",
 		"types": [
 			"node"
 		],
-		"module": "commonjs",
-		"esModuleInterop": true,
-		"removeComments": true,
-		"strict": true,
-		"preserveConstEnums": true,
-		"declaration": true,
-		"outDir": "./dist/",
-		"target": "es2019",
-		"useUnknownInCatchVariables": false,
-		"sourceMap": true
+		// TODO: remove all options below this line
+		"noUnusedLocals": false,
 	},
 	"include": [
 		"**/*.d.ts",
 		"commands/**/*",
 		"index.ts",
 		"src/**/*"
-	],
-	"exclude": [
-		"dist",
-		"node_modules/**/*",
-		"**/*.spec.ts"
 	]
 }

packages/nodes-base/package.json

@@ -795,17 +795,16 @@
   },
   "jest": {
     "transform": {
-      "^.+\\.tsx?$": "ts-jest"
+      "^.+\\.ts$": "ts-jest"
     },
     "testURL": "http://localhost/",
-    "testRegex": "(/__tests__/.*|(\\.|/)(test|spec))\\.(jsx?|tsx?)$",
+    "testRegex": "(/__tests__/.*|(\\.|/)(test|spec))\\.(js|ts)$",
     "testPathIgnorePatterns": [
       "/dist/",
       "/node_modules/"
     ],
     "moduleFileExtensions": [
       "ts",
-      "tsx",
       "js",
       "json"
     ]