tsconfig: Add support for comments

Author: IvanGoncharov

resources/build-npm.ts

@@ -18,8 +18,14 @@ fs.rmSync('./npmDist', { recursive: true, force: true });
 fs.mkdirSync('./npmDist');
 
 // Based on https://github.com/Microsoft/TypeScript/wiki/Using-the-Compiler-API#getting-the-dts-from-a-javascript-file
-const tsConfig = JSON.parse(
-  fs.readFileSync(localRepoPath('tsconfig.json'), 'utf-8'),
+const tsConfigPath = localRepoPath('tsconfig.json');
+const { config: tsConfig, error: tsConfigError } = ts.parseConfigFileTextToJson(
+  tsConfigPath,
+  fs.readFileSync(tsConfigPath, 'utf-8'),
+);
+assert(
+  tsConfigError === undefined,
+  'Fail to parse config: ' + tsConfigError,
 );
 assert(
   tsConfig.compilerOptions,

tsconfig.json

@@ -14,16 +14,20 @@
     "isolatedModules": true,
     "importsNotUsedAsValues": "error",
     "forceConsistentCasingInFileNames": true,
+
+    // Type Checking
+    // https://www.typescriptlang.org/tsconfig#Type_Checking_6248
     "strict": true,
-    "useUnknownInCatchVariables": false,
+    "useUnknownInCatchVariables": false, // FIXME part of 'strict' but is temporary disabled
+    // All checks that are not part of "strict"
     "allowUnreachableCode": false,
     "allowUnusedLabels": false,
-    "exactOptionalPropertyTypes": false,
-    "noFallthroughCasesInSwitch": false,
-    "noImplicitOverride": false,
-    "noImplicitReturns": false,
-    "noPropertyAccessFromIndexSignature": false,
-    "noUncheckedIndexedAccess": false,
+    "exactOptionalPropertyTypes": false, // FIXME
+    "noFallthroughCasesInSwitch": false, // TODO consider
+    "noImplicitOverride": false, // FIXME
+    "noImplicitReturns": false, // TODO consider
+    "noPropertyAccessFromIndexSignature": false, // TODO consider
+    "noUncheckedIndexedAccess": false, // FIXME
     "noUnusedLocals": true,
     "noUnusedParameters": true
   }