create-forgeon 0.1.30 → 0.1.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-forgeon",
-  "version": "0.1.30",
+  "version": "0.1.32",
   "description": "Forgeon project generator CLI",
   "license": "MIT",
   "author": "Forgeon",

package/src/modules/executor.test.mjs

@@ -299,6 +299,12 @@ describe('addModule', () => {
       assert.match(apiDockerfile, /COPY packages\/logger packages\/logger/);
       assert.match(apiDockerfile, /RUN pnpm --filter @forgeon\/logger build/);
 
+      const loggerTsconfig = fs.readFileSync(
+        path.join(projectRoot, 'packages', 'logger', 'tsconfig.json'),
+        'utf8',
+      );
+      assert.match(loggerTsconfig, /"extends": "\.\.\/\.\.\/tsconfig\.base\.node\.json"/);
+
       const apiEnv = fs.readFileSync(path.join(projectRoot, 'apps', 'api', '.env.example'), 'utf8');
       assert.match(apiEnv, /LOGGER_LEVEL=log/);
       assert.match(apiEnv, /LOGGER_HTTP_ENABLED=true/);
@@ -317,6 +323,11 @@ describe('addModule', () => {
       assert.match(compose, /LOGGER_HTTP_ENABLED: \$\{LOGGER_HTTP_ENABLED\}/);
       assert.match(compose, /LOGGER_REQUEST_ID_HEADER: \$\{LOGGER_REQUEST_ID_HEADER\}/);
 
+      const rootReadme = fs.readFileSync(path.join(projectRoot, 'README.md'), 'utf8');
+      assert.match(rootReadme, /## Logger Module/);
+      assert.match(rootReadme, /LOGGER_LEVEL=log/);
+      assert.match(rootReadme, /docker compose logs api/);
+
       const moduleDoc = fs.readFileSync(result.docsPath, 'utf8');
       assert.match(moduleDoc, /Logger/);
       assert.match(moduleDoc, /Status: implemented/);

package/src/modules/logger.mjs

@@ -218,6 +218,44 @@ function patchCompose(targetRoot) {
   fs.writeFileSync(composePath, `${content.trimEnd()}\n`, 'utf8');
 }
 
+function patchReadme(targetRoot) {
+  const readmePath = path.join(targetRoot, 'README.md');
+  if (!fs.existsSync(readmePath)) {
+    return;
+  }
+
+  const marker = '## Logger Module';
+  let content = fs.readFileSync(readmePath, 'utf8').replace(/\r\n/g, '\n');
+  if (content.includes(marker)) {
+    return;
+  }
+
+  const section = `## Logger Module
+
+The logger add-module provides:
+- request id middleware (default header: \`x-request-id\`)
+- HTTP access logs with method/path/status/duration/ip/requestId
+- Nest logger integration via \`app.useLogger(...)\`
+
+Configuration (env):
+- \`LOGGER_LEVEL=log\` (\`error|warn|log|debug|verbose\`)
+- \`LOGGER_HTTP_ENABLED=true\`
+- \`LOGGER_REQUEST_ID_HEADER=x-request-id\`
+
+Where to see logs:
+- local dev: API terminal output
+- Docker: \`docker compose logs api\`
+`;
+
+  if (content.includes('## Prisma In Docker Start')) {
+    content = content.replace('## Prisma In Docker Start', `${section}\n## Prisma In Docker Start`);
+  } else {
+    content = `${content.trimEnd()}\n\n${section}\n`;
+  }
+
+  fs.writeFileSync(readmePath, `${content.trimEnd()}\n`, 'utf8');
+}
+
 export function applyLoggerModule({ packageRoot, targetRoot }) {
   copyFromPreset(packageRoot, targetRoot, path.join('packages', 'logger'));
   patchApiPackage(targetRoot);
@@ -225,6 +263,7 @@ export function applyLoggerModule({ packageRoot, targetRoot }) {
   patchAppModule(targetRoot);
   patchApiDockerfile(targetRoot);
   patchCompose(targetRoot);
+  patchReadme(targetRoot);
 
   upsertEnvLines(path.join(targetRoot, 'apps', 'api', '.env.example'), [
     'LOGGER_LEVEL=log',
@@ -238,4 +277,3 @@ export function applyLoggerModule({ packageRoot, targetRoot }) {
     'LOGGER_REQUEST_ID_HEADER=x-request-id',
   ]);
 }
-

package/templates/module-presets/logger/packages/logger/src/http-logging.interceptor.ts

@@ -1,5 +1,5 @@
 import { CallHandler, ExecutionContext, Injectable, NestInterceptor } from '@nestjs/common';
-import { Observable, tap } from 'rxjs';
+import { Observable } from 'rxjs';
 import { ForgeonLoggerService } from './forgeon-logger.service';
 import { LoggerConfigService } from './logger-config.service';
 
@@ -17,6 +17,7 @@ interface RequestLike {
 
 interface ResponseLike {
   statusCode?: number;
+  once?: (event: string, listener: () => void) => void;
 }
 
 @Injectable()
@@ -46,30 +47,34 @@ export class ForgeonHttpLoggingInterceptor implements NestInterceptor {
       request.requestId ?? this.readHeader(request.headers, this.loggerConfig.requestIdHeader);
     const startedAt = Date.now();
 
-    return next.handle().pipe(
-      tap({
-        next: () => {
-          this.logger.logHttpRequest({
-            method,
-            path,
-            statusCode: response.statusCode ?? 200,
-            durationMs: Date.now() - startedAt,
-            requestId,
-            ip,
-          });
-        },
-        error: () => {
-          this.logger.logHttpRequest({
-            method,
-            path,
-            statusCode: response.statusCode ?? 500,
-            durationMs: Date.now() - startedAt,
-            requestId,
-            ip,
-          });
-        },
-      }),
-    );
+    if (typeof response.once === 'function') {
+      response.once('finish', () => {
+        this.logger.logHttpRequest({
+          method,
+          path,
+          statusCode: response.statusCode ?? 200,
+          durationMs: Date.now() - startedAt,
+          requestId,
+          ip,
+        });
+      });
+    } else {
+      // Fallback for non-standard response mocks.
+      try {
+        this.logger.logHttpRequest({
+          method,
+          path,
+          statusCode: response.statusCode ?? 200,
+          durationMs: Date.now() - startedAt,
+          requestId,
+          ip,
+        });
+      } catch {
+        // no-op
+      }
+    }
+
+    return next.handle();
   }
 
   private readHeader(headers: HeadersRecord | undefined, name: string): string | undefined {
@@ -87,4 +92,3 @@ export class ForgeonHttpLoggingInterceptor implements NestInterceptor {
     return undefined;
   }
 }
-

package/templates/module-presets/logger/packages/logger/tsconfig.json

@@ -1,5 +1,5 @@
 {
-  "extends": "../../../../../tsconfig.base.node.json",
+  "extends": "../../tsconfig.base.node.json",
   "compilerOptions": {
     "rootDir": "src",
     "outDir": "dist",
@@ -7,4 +7,3 @@
   },
   "include": ["src/**/*.ts"]
 }
-