fix: avoid to duplicate register ts-node in dev (#5015)

web-infra-dev · Nov 29, 2023 · a3f2269 · a3f2269
1 parent 8793e64
commit a3f2269
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.changeset/clever-weeks-wink.md b/.changeset/clever-weeks-wink.md
@@ -0,0 +1,6 @@
+---
+'@modern-js/bff-core': patch
+---
+
+fix: avoid to duplicate register ts-node in dev
+fix: 避免在开发环境重复注册 ts-node
diff --git a/packages/server/bff-core/src/router/index.ts b/packages/server/bff-core/src/router/index.ts
@@ -68,16 +68,12 @@ export class ApiRouter {
     this.lambdaDir = this.getExactLambdaDir(this.apiDir, lambdaDir);
     this.existLambdaDir = fs.existsSync(this.lambdaDir);
     debug(`apiDir:`, this.apiDir, `lambdaDir:`, this.lambdaDir);
-    this.enableRegister();
   }
 
   private enableRegister() {
     // eslint-disable-next-line node/no-deprecated-api
     const existTsLoader = Boolean(require.extensions['.ts']);
-    if (
-      !existTsLoader &&
-      (process.env.NODE_ENV !== 'production' || this.isBuild)
-    ) {
+    if (!existTsLoader) {
       try {
         const projectSearchDir = this.appDir || this.apiDir;
         const tsNode: typeof import('ts-node') = require('ts-node');
@@ -313,6 +309,7 @@ export class ApiRouter {
   }
 
   private getModuleInfo(filename: string) {
+    this.enableRegister();
     try {
       const module = requireHandlerModule(filename);
       return {