framework performance improvements

Author: manusant

.changeset/empty-groups-take.md

@@ -0,0 +1,5 @@
+---
+"@nodeboot/starter-persistence": minor
+---
+
+Hnadle async init correctly

.changeset/little-beds-share.md

@@ -0,0 +1,7 @@
+---
+"@nodeboot/http-server": minor
+"@nodeboot/context": minor
+"@nodeboot/engine": minor
+---
+
+Performance improvements

.changeset/odd-planes-heal.md

@@ -0,0 +1,5 @@
+---
+"@nodeboot/express-server": minor
+---
+
+Improve express server performance

.changeset/young-pants-sink.md

@@ -0,0 +1,5 @@
+---
+"@nodeboot/encore-server": patch
+---
+
+Improve encore server

.eslintignore

@@ -13,3 +13,4 @@ jest.setup.js
 memory-leak-detector.js
 **.d.ts
 /starters/persistence/tests/docker/mongodb/init-replica-set.js
+/benchmarking

eslint.config.mjs

@@ -26,6 +26,7 @@ export default [
             "**/*.js",
             "**/.encore/**",
             "**/encore.gen/**",
+            "benchmarking/**",
         ],
     },
 

package.json

@@ -49,6 +49,7 @@
         "changeset:format": "pnpm prettier --write '{packages,samples,serverless,servers,starters}/*/{package.json,CHANGELOG.md}'",
         "release:status": "pnpm changeset status",
         "release:publish": "pnpm changeset publish",
+        "bench:all": "pnpm --dir benchmarking run bench:all",
         "nodeboot:update": "turbo run nodeboot:update",
         "nodeboot:check:type-deps": "node ./node_modules/@nodeboot/tools/dist/check-type-dependencies.js",
         "nodeboot:check:local-deps": "node ./node_modules/@nodeboot/tools/dist/verify-local-dependencies.js",

packages/context/src/metadata/ActionMetadata.ts

@@ -172,6 +172,10 @@ export class ActionMetadata {
      * Action metadata can be used only after its build.
      */
     build(responseHandlers: ResponseHandlerMetadata[]) {
+        // Sort params once at boot (this array is static after registration) instead of
+        // re-sorting on every single request in the hot path (see NodeBootEngine.executeAction).
+        this.params = this.params.sort((param1, param2) => param1.index - param2.index);
+
         const classTransformerResponseHandler = responseHandlers.find(
             handler => handler.type === "response-class-transform-options",
         );

packages/context/src/services/LoggerService.ts

@@ -14,4 +14,17 @@ export interface LoggerService {
     debug(message: string, meta?: Error | JsonObject): void;
 
     child(meta: JsonObject): LoggerService;
+
+    /**
+     * Cheaply checks whether a given log level would actually be emitted, without paying the
+     * cost of building the log message. Callers on hot paths (e.g. per-request logging in server
+     * drivers) should guard expensive message construction (string interpolation, property
+     * access on request/response objects, timing calls) behind this check instead of always
+     * building the message and letting the logger silently discard it.
+     *
+     * Optional for backwards compatibility with custom `LoggerService` implementations that
+     * don't support level introspection — callers must treat a missing method as "unknown,
+     * assume enabled" to avoid silently dropping logs.
+     */
+    isLevelEnabled?(level: string): boolean;
 }

packages/engine/src/core/NodeBootEngine.ts

@@ -93,10 +93,8 @@ export class NodeBootEngine<TServer, TDriver extends NodeBootDriver<TServer>> {
      * Executes given controller action.
      */
     protected async executeAction(actionMetadata: ActionMetadata, action: Action, interceptorFns: Function[]) {
-        // compute all parameters
-        const paramsPromises = actionMetadata.params
-            .sort((param1, param2) => param1.index - param2.index)
-            .map(param => this.parameterHandler.handle(action, param));
+        // compute all parameters (actionMetadata.params is pre-sorted once at boot in ActionMetadata.build())
+        const paramsPromises = actionMetadata.params.map(param => this.parameterHandler.handle(action, param));
 
         // after all parameters are computed
         try {