Add strict null checks warning

.eslintrc.shared.json

@@ -4,8 +4,9 @@
     "ecmaVersion": 6,
     "sourceType": "module"
   },
-  "plugins": ["@typescript-eslint", "import"],
+  "plugins": ["@typescript-eslint", "import", "strict-null-checks"],
   "rules": {
+    "strict-null-checks/all": "warn",
     "@typescript-eslint/semi": "warn",
     "curly": "warn",
     "eqeqeq": "warn",

binary/tsconfig.json

@@ -13,8 +13,8 @@
     // "noFallthroughCasesInSwitch": true, /* Report errors for fallthrough cases in switch statement. */
     // "noUnusedParameters": true,  /* Report errors on unused parameters. */
     "esModuleInterop": true /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */,
-    "resolveJsonModule": true /* Enable importing .json files */,
-    "types": ["jest"]
+    "types": ["jest"],
+    "resolveJsonModule": true /* Enable importing .json files */
   },
   "include": ["src/**/*", "test/**/*"]
 }

core/package.json

@@ -5,7 +5,6 @@
   "scripts": {
     "test": "cross-env NODE_OPTIONS=--experimental-vm-modules jest --runInBand",
     "tsc:check": "tsc -p ./ --noEmit",
-    "build:npm": "tsc -p ./tsconfig.npm.json",
     "lint": "eslint . --ext ts",
     "lint:fix": "eslint . --ext ts --fix"
   },

core/tsconfig.json

@@ -11,12 +11,13 @@
     "forceConsistentCasingInFileNames": true,
     "module": "ESNext",
     "moduleResolution": "Bundler",
-    "resolveJsonModule": true,
     "isolatedModules": true,
     "noEmit": true,
     "jsx": "react-jsx",
     "noEmitOnError": false,
-    "types": ["jest", "node"]
+    "types": ["jest", "node"],
+    "strictNullChecks": true,
+    "resolveJsonModule": true
   },
   "include": ["./**/*.ts", "./**/*.js", "./**/*.d.ts"],
   "exclude": ["dist"]

extensions/vscode/tsconfig.json

@@ -8,8 +8,9 @@
     "rootDir": "../../",
     "strict": true,
     "esModuleInterop": true,
-    "resolveJsonModule": true,
-    "types": ["mocha"] // This solves conflict between jest and mocha: https://github.com/cypress-io/cypress/issues/7435#issuecomment-631695007
+    "types": ["mocha"], // This solves conflict between jest and mocha: https://github.com/cypress-io/cypress/issues/7435#issuecomment-631695007
+    "strictNullChecks": true,
+    "resolveJsonModule": true
   },
   "include": ["src/**/*", "../../core/**/*.ts", "../../core/**/*.d.ts"],
   // core/test uses @types/jest, which conflicts with @types/mocha

gui/tsconfig.json

@@ -11,14 +11,13 @@
     "forceConsistentCasingInFileNames": true,
     "module": "ESNext",
     "moduleResolution": "Node",
-    "resolveJsonModule": true,
     "isolatedModules": true,
     "noEmit": true,
     "jsx": "react-jsx",
     "noEmitOnError": false,
-    "types": ["vitest/globals"]
-    // Disabled for now since it's causing too many errors
-    // "strictNullChecks": true
+    "types": ["vitest/globals"],
+    "strictNullChecks": true,
+    "resolveJsonModule": true
   },
   "include": ["src", "../src/util/messenger.ts"],
   "references": [{ "path": "./tsconfig.node.json" }]