ci: improve tsconfig setup (#48)

package.json

@@ -35,8 +35,8 @@
   "scripts": {
     "check-types": "tsc",
     "check-types:watch": "tsc -w",
-    "transpile": "tsc -p tsconfig.task.transpile.json",
-    "build:watch": "tsc -w",
+    "build": "tsc -p tsconfig.build.json",
+    "build:watch": "tsc -p tsconfig.build.json -w",
     "test": "vitest run",
     "test:watch": "vitest",
     "test:ci": "vitest run --silent --coverage --coverage.reporter=json-summary",

tsconfig.base.json

@@ -3,6 +3,9 @@
   "compilerOptions": {
     /* Reasonable default base options: */
 
+    // No emitting by default.
+    "noEmit": true,
+
     // Of course, this is common practice in Node but also part of ES standards.
     // todo: is there a way to enforce "with type"?
     "resolveJsonModule": true,
@@ -29,8 +32,7 @@
     "noUncheckedIndexedAccess": true,
     "noImplicitOverride": true,
 
-    // restrictions are determined by what kind of transpiler and/or bundler we use.
-    // so by default, we preserve it.
-    "module": "Node16",
+    // Allow "Deno"-style imports
+    "allowImportingTsExtensions": true,
   }
 }

tsconfig.task.transpile.json → tsconfig.build.json

@@ -1,20 +1,13 @@
 {
   "extends": ["./tsconfig.base.json"],
   "compilerOptions": {
-    // transpiling here, not checking. Yes... double-negatives.
+    "noCheck": true,
     "noEmit": false,
-
     "module": "NodeNext",
     "moduleResolution": "NodeNext",
-
-    // please, yes.
-    "allowImportingTsExtensions": true,
     "rewriteRelativeImportExtensions": true,
-
-    // yes, we're publishing a library here
     "declaration": true,
     "sourceMap": true,
-
     "outDir": "dist"
   },
   "include": [

tsconfig.json

@@ -1,3 +1,6 @@
 {
-  "extends": "./tsconfig.task.check-types.json"
+  "extends": "./tsconfig.base.json",
+  "include": [
+    "./**/*.ts"
+  ]
 }