[api] implement verbatimModuleSyntax option in tsconfig, move ts-node…

… to devDependencies [api] add consistent-type-imports eslint rule as linting proxy for verbatimModuleSyntax rule
theopensystemslab · Sep 12, 2024 · 4d1eca8 · 4d1eca8
1 parent de64cf4
commit 4d1eca8
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 100 deletions.
diff --git a/api.planx.uk/.eslintrc b/api.planx.uk/.eslintrc
@@ -18,6 +18,7 @@
       }
     ],
     "@typescript-eslint/no-non-null-assertion": "off",
+    "@typescript-eslint/consistent-type-imports": "error",
     "no-nested-ternary": "error",
     "@vitest/expect-expect": [
       "error",

diff --git a/api.planx.uk/package.json b/api.planx.uk/package.json
@@ -53,7 +53,6 @@
     "string-to-stream": "^3.0.1",
     "swagger-jsdoc": "^6.2.8",
     "swagger-ui-express": "^5.0.0",
-    "ts-node": "^10.9.2",
     "type-fest": "^4.18.1",
     "zod": "^3.23.5"
   },

diff --git a/api.planx.uk/pnpm-lock.yaml b/api.planx.uk/pnpm-lock.yaml
diff --git a/api.planx.uk/tsconfig.json b/api.planx.uk/tsconfig.json
@@ -16,8 +16,7 @@
     "target": "esnext",
     "types": ["vitest/globals"],
     // ensure the code is ready for per-file transpilation by tsx (used in dev mode)
-    "isolatedModules": true,
-    // TODO: implement "verbatimModuleSyntax" option (laborious)
+    "verbatimModuleSyntax": true,
   },
   "exclude": ["node_modules", "dist"],
 }