fix tsconfig errors (#306)

* cleanup tsc settings * add base tsconfig to DRY up tsconfigs. build sourcemaps for all. * DRY up remaining repeated fields * strict false for web5spec * Minor fix to remove trailing commas from JSON --------- Co-authored-by: Frank Hinek <[email protected]>
decentralized-identity · Nov 30, 2023 · d4bf741 · d4bf741
1 parent dce370d
commit d4bf741
Show file tree

Hide file tree

Showing 21 changed files with 67 additions and 178 deletions.
diff --git a/.web5-spec/tsconfig.json b/.web5-spec/tsconfig.json
@@ -1,22 +1,9 @@
 {
+  "extends": "../tsconfig.json",
   "compilerOptions": {
-    // "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "NodeNext",
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
     "outDir": "dist",
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true,
-    "resolveJsonModule": true
+    "strict": false
   },
   "include": [
     "main.ts",

diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/agent/src/dwn-manager.ts b/packages/agent/src/dwn-manager.ts
@@ -1,3 +1,5 @@
+import type { Readable } from 'readable-stream';
+
 import {
   Signer,
   GenericMessage,
@@ -10,7 +12,6 @@ import {
 import { Jose } from '@web5/crypto';
 import { Convert } from '@web5/common';
 import { DidResolver } from '@web5/dids';
-import { Readable } from 'readable-stream';
 import { utils as didUtils } from '@web5/dids';
 
 import {

diff --git a/packages/agent/tsconfig.cjs.json b/packages/agent/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/agent/tsconfig.json b/packages/agent/tsconfig.json
@@ -1,25 +1,11 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    "sourceMap": true,
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
-    "src",
+    "src"
   ],
   "exclude": [
     "node_modules"

diff --git a/packages/api/tsconfig.cjs.json b/packages/api/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/api/tsconfig.json b/packages/api/tsconfig.json
@@ -1,25 +1,12 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    // "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "ESNext",
-    "declaration": true,
-    "declarationMap": true,
+    "strict": false,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true,
-    "resolveJsonModule": true
+    "outDir": "dist/esm"
   },
   "include": [
-    "src",
+    "src"
   ],
   "exclude": [
     "node_modules"

diff --git a/packages/common/tsconfig.json b/packages/common/tsconfig.json
@@ -1,25 +1,11 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    "sourceMap": true,
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
-    "src",
+    "src"
   ],
   "exclude": [
     "node_modules"

diff --git a/packages/credentials/tsconfig.cjs.json b/packages/credentials/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/credentials/tsconfig.json b/packages/credentials/tsconfig.json
@@ -1,25 +1,11 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    "sourceMap": true,
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
-    "src",
+    "src"
   ],
   "exclude": [
     "node_modules"

diff --git a/packages/crypto/tsconfig.cjs.json b/packages/crypto/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/crypto/tsconfig.json b/packages/crypto/tsconfig.json
@@ -1,21 +1,8 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
     "src"

diff --git a/packages/dids/tsconfig.cjs.json b/packages/dids/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/dids/tsconfig.json b/packages/dids/tsconfig.json
@@ -1,19 +1,9 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
+    "strict": false,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
     "src",
@@ -22,4 +12,4 @@
   "exclude": [
     "node_modules"
   ]
-}
+}
diff --git a/packages/identity-agent/tsconfig.cjs.json b/packages/identity-agent/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/identity-agent/tsconfig.json b/packages/identity-agent/tsconfig.json
@@ -1,22 +1,8 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    "sourceMap": true,
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
     "src",

diff --git a/packages/proxy-agent/tsconfig.cjs.json b/packages/proxy-agent/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/proxy-agent/tsconfig.json b/packages/proxy-agent/tsconfig.json
@@ -1,21 +1,8 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    "sourceMap": true,
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
     "src",

diff --git a/packages/user-agent/tsconfig.cjs.json b/packages/user-agent/tsconfig.cjs.json
@@ -3,7 +3,7 @@
   "compilerOptions": {
     "lib": [
       "DOM",
-      "ES5",
+      "ES5"
     ],
     "target": "ES5",
     "module": "CommonJS",

diff --git a/packages/user-agent/tsconfig.json b/packages/user-agent/tsconfig.json
@@ -1,25 +1,11 @@
 {
+  "extends": "../../tsconfig.json",
   "compilerOptions": {
-    "strict": true,
-    "lib": [
-      "DOM",
-      "ES6"
-    ],
-    "allowJs": true,
-    "target": "es6",
-    "module": "ESNext", // Required for enabling JavaScript import assertion support
-    "declaration": true,
-    "declarationMap": true,
     "declarationDir": "dist/types",
-    "outDir": "dist/esm",
-    "sourceMap": true,
-    // `NodeNext` will throw compilation errors if relative import paths are missing file extension
-    // reference: https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
-    "moduleResolution": "NodeNext",
-    "esModuleInterop": true
+    "outDir": "dist/esm"
   },
   "include": [
-    "src",
+    "src"
   ],
   "exclude": [
     "node_modules"