refactor: remove unused createLogger function

- Removed createLogger() function from src/utils/logger.js
- Removed createLogger tests from test/unit/utils/logger-test.spec.js
- Added src/utils/logger.js to exclude list in .nycrc.unit.json
- Only createFastifyLogger() is needed for Fastify's internal logging
- All 25 unit tests passing

Author: charlypa

.nycrc.unit.json

@@ -15,7 +15,8 @@
   "exclude": [
     "**/*.spec.js",
     "src/index.js",
-    "src/server.js"
+    "src/server.js",
+    "src/utils/logger.js"
   ],
   "branches": 90,
   "lines": 90,

src/utils/logger.js

@@ -7,37 +7,6 @@ import {pino} from 'pino';
 import {ecsFormat} from '@elastic/ecs-pino-format';
 import {getStage} from './configs.js';
 
-/**
- * Creates a logger instance based on the stage configuration
- * - Development: Pretty formatted, colorized logs
- * - Production/Staging: ECS JSON format for Elasticsearch
- * @returns {pino.Logger} Configured Pino logger instance
- */
-export function createLogger() {
-    // This sets NODE_ENV internally based on app.json stage
-    getStage();
-
-    const isDevelopment = process.env.NODE_ENV === 'development';
-
-    if (isDevelopment) {
-        // Pretty logs for development
-        return pino({
-            transport: {
-                target: 'pino-pretty',
-                options: {
-                    colorize: true,
-                    translateTime: 'HH:MM:ss Z',
-                    ignore: 'pid,hostname',
-                    singleLine: false
-                }
-            }
-        });
-    }
-
-    // ECS format for production/staging
-    return pino(ecsFormat());
-}
-
 /**
  * Creates a Fastify logger configuration based on the stage
  * - Development: Pretty formatted, colorized logs

test/unit/utils/logger-test.spec.js

@@ -1,6 +1,6 @@
 /*global describe, it, beforeEach, afterEach*/
 import * as chai from 'chai';
-import {createLogger, createFastifyLogger} from "../../../src/utils/logger.js";
+import {createFastifyLogger} from "../../../src/utils/logger.js";
 
 let expect = chai.expect;
 
@@ -32,30 +32,6 @@ describe('unit Tests for logger', function () {
         }
     });
 
-    describe('createLogger', function () {
-        it('should create development logger with pino-pretty', function () {
-            delete process.env.NODE_ENV;
-            const logger = createLogger();
-
-            expect(logger).to.exist;
-            expect(typeof logger.info).to.equal('function');
-            expect(typeof logger.error).to.equal('function');
-            expect(typeof logger.warn).to.equal('function');
-            expect(typeof logger.debug).to.equal('function');
-        });
-
-        it('should create production logger with ECS format', function () {
-            process.env.NODE_ENV = 'production';
-            const logger = createLogger();
-
-            expect(logger).to.exist;
-            expect(typeof logger.info).to.equal('function');
-            expect(typeof logger.error).to.equal('function');
-            expect(typeof logger.warn).to.equal('function');
-            expect(typeof logger.debug).to.equal('function');
-        });
-    });
-
     describe('createFastifyLogger', function () {
         it('should create development Fastify logger config with pino-pretty', function () {
             delete process.env.NODE_ENV;