fix(framework): don't enable profiler per default

deepkit · Nov 9, 2023 · 52a7e52 · 52a7e52
1 parent 3b165d6
commit 52a7e52
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/packages/framework/src/debug/stopwatch/store.ts b/packages/framework/src/debug/stopwatch/store.ts
@@ -151,5 +151,4 @@ export class FileStopwatchStore extends StopwatchStore {
             this.syncMutex.unlock();
         }
     }
-
 }
diff --git a/packages/framework/src/module.config.ts b/packages/framework/src/module.config.ts
@@ -113,7 +113,11 @@ export class FrameworkConfig {
 
     debugUrl: string = '_debug';
 
-    debugProfiler: boolean = true;
+    /**
+     * Whether profiling is enabled. This is automatically enabled when debug is enabled,
+     * but can be enabled separately.
+     */
+    profile: boolean = false;
 
     /**
      * @description IP:Port or unix socket name or named pipes.

diff --git a/packages/framework/src/module.ts b/packages/framework/src/module.ts
@@ -215,9 +215,9 @@ export class FrameworkModule extends createModule({
         this.addProvider({ provide: StopwatchStore, useExisting: FileStopwatchStore });
         this.addProvider({
             provide: Stopwatch,
-            useFactory(store: StopwatchStore, profile: FrameworkConfig['debugProfiler']) {
+            useFactory(store: StopwatchStore, config: FrameworkConfig) {
                 const stopwatch = new Stopwatch(store);
-                if (profile) {
+                if (config.profile || config.debug) {
                     stopwatch.enable();
                 } else {
                     stopwatch.disable();
@@ -243,7 +243,7 @@ export class FrameworkModule extends createModule({
             db.module.setupProvider(0, db.classType).eventDispatcher = injectorReference(EventDispatcher);
         }
 
-        if (this.config.debug && this.config.debugProfiler) {
+        if (this.config.debug && this.config.profile) {
             for (const db of this.dbs) {
                 db.module.setupProvider(0, db.classType).stopwatch = injectorReference(Stopwatch);
             }