@extk/expressive 0.4.4 → 0.5.2

package/README.md

@@ -40,17 +40,18 @@ npm install @extk/expressive express
 
 ```ts
 import express from 'express';
-import { bootstrap, getDefaultFileLogger, ApiResponse, NotFoundError, SWG } from '@extk/expressive';
+import { bootstrap, ApiResponse, NotFoundError, SWG } from '@extk/expressive';
 
-// 1. Bootstrap with a logger
+// 1. Bootstrap with a logger (bring your own)
 const {
   expressiveServer,
   expressiveRouter,
   swaggerBuilder,
   notFoundMiddleware,
   getErrorHandlerMiddleware,
+  silently,
 } = bootstrap({
-  logger: getDefaultFileLogger('my-api'),
+  logger: console, // any object with info/warn/error/debug
 });
 
 // 2. Configure swagger metadata
@@ -246,16 +247,56 @@ addRoute({
 
 This way your Zod schemas serve as the single source of truth for both runtime validation and API documentation.
 
+## silently
+
+`silently` runs a function — sync or async — and suppresses any errors it throws. Errors are forwarded to `alertHandler` (if configured) or logged via the container logger.
+
+```ts
+// fire-and-forget without crashing the process
+silently(() => sendAnalyticsEvent(req));
+silently(async () => await notifySlack('Server started'));
+```
+
 ## Logging
 
-Winston-based logging with daily rotating files in production and console output in development.
+Expressive does not bundle a logger. Instead, `bootstrap` accepts any object that satisfies the `Logger` interface:
 
 ```ts
-import { getDefaultFileLogger, getDefaultConsoleLogger } from '@extk/expressive';
+export type Logger = {
+  info(message: string, ...args: any[]): void;
+  error(message: string | Error | unknown, ...args: any[]): void;
+  warn(message: string, ...args: any[]): void;
+  debug(message: string, ...args: any[]): void;
+};
+```
 
-const logger = getDefaultFileLogger('my-service'); // logs to ./logs/my-service-YYYY-MM-DD.log
-logger.info('Server started on port %d', 3000);
-logger.error('Something went wrong: %s', err.message);
+This means you can pass `console` directly, or plug in any logging library (Winston, Pino, etc.):
+
+```ts
+bootstrap({ logger: console });
+```
+
+The `@extk/logger-cloudwatch` package from the same org is a drop-in fit:
+
+```ts
+import { getCloudwatchLogger, getConsoleLogger } from '@extk/logger-cloudwatch';
+
+// development
+bootstrap({ logger: getConsoleLogger() });
+
+// production — streams structured JSON logs to AWS CloudWatch
+bootstrap({
+  logger: getCloudwatchLogger({
+    aws: {
+      region: 'us-east-1',
+      logGroup: '/my-app/production',
+      credentials: {
+        accessKeyId: process.env.AWS_ACCESS_KEY_ID!,
+        secretAccessKey: process.env.AWS_SECRET_ACCESS_KEY!,
+      },
+    },
+  }),
+});
 ```
 
 ## Utilities

package/dist/index.d.mts

@@ -1,8 +1,7 @@
 import * as express from 'express';
-import express__default, { RequestHandler, Request } from 'express';
+import express__default, { RequestHandler } from 'express';
 import * as express_serve_static_core from 'express-serve-static-core';
 import { RouteParameters } from 'express-serve-static-core';
-import winston, { Logger as Logger$1 } from 'winston';
 import { HelmetOptions } from 'helmet';
 import morgan from 'morgan';
 
@@ -17,19 +16,18 @@ type Pagination = {
     limit: number;
     offset: number;
 };
-type ReqSnapshot = {
-    readonly query: qs.ParsedQs;
-    readonly path: string;
-    readonly method: string;
-    readonly userId: string | number | undefined;
-};
 
 type HttpMethod = 'get' | 'post' | 'put' | 'patch' | 'delete' | 'head' | 'options' | 'trace';
 type OtherString = string & {};
 type OtherUnknown = unknown & {};
 type ContentType = 'application/json' | 'application/xml' | 'text/plain' | 'text/html' | OtherString;
-type AlertHandler = (err: Error & Record<string, any>, reqSnapshot?: ReqSnapshot) => void | Promise<void>;
-type Logger = Logger$1;
+type AlertHandler = (err: Error & Record<string, any>, context?: string) => void | Promise<void>;
+type Logger = {
+    info(message: string, ...args: any[]): void;
+    error(message: string | Error | unknown, ...args: any[]): void;
+    warn(message: string, ...args: any[]): void;
+    debug(message: string, ...args: any[]): void;
+};
 type Container = {
     logger: Logger;
     alertHandler?: AlertHandler;
@@ -263,21 +261,12 @@ declare function slugify(text: string): string;
 declare function getTmpDir(): string;
 declare function getTmpPath(...steps: string[]): string;
 declare function parseDefaultPagination(query: PaginationQuery): Pagination;
-declare function createReqSnapshot(req: Request & {
-    user?: {
-        id?: string | number;
-    };
-}): ReqSnapshot;
 
 declare function isDev(): boolean;
 declare function isProd(): boolean;
 declare function getEnvVar(configName: string): string;
 type Env = 'dev' | 'prod' | OtherString;
 
-declare const createLogger: (options?: winston.LoggerOptions) => winston.Logger;
-declare const getDefaultFileLogger: (name?: string) => winston.Logger;
-declare const getDefaultConsoleLogger: (name?: string) => winston.Logger;
-
 declare class ApiErrorResponse<T = undefined> {
     readonly status = "error";
     readonly message: string;
@@ -294,7 +283,7 @@ declare class ApiResponse<T = undefined> {
 
 declare function bootstrap(container: Container): {
     swaggerBuilder: () => SwaggerBuilder;
-    silently: (fn: () => Promise<void>, reqSnapshot?: ReqSnapshot) => void;
+    silently: (fn: () => Promise<void> | void) => Promise<void>;
     notFoundMiddleware: (_req: express.Request, res: express.Response, _next: express.NextFunction) => void;
     getErrorHandlerMiddleware: (errorMapper?: (err: Error & Record<string, unknown>) => ApiError | null | undefined) => (err: Error & Record<string, unknown>, req: express.Request, res: express.Response, _next: express.NextFunction) => Promise<void>;
     expressiveServer: (configs?: {
@@ -321,4 +310,4 @@ declare function bootstrap(container: Container): {
     };
 };
 
-export { type AlertHandler, ApiError, ApiErrorResponse, ApiResponse, type AuthMethod, BadRequestError, type Container, type Content, type ContentType, DuplicateError, type Env, type ExpressHandler, type ExpressLocalsObj, type ExpressRoute, FileTooBigError, ForbiddenError, type HttpMethod, InternalError, InvalidCredentialsError, InvalidFileTypeError, type Logger, NotFoundError, type OtherString, type OtherUnknown, type Pagination, type PaginationQuery, type Param, type PathItem, type ReqSnapshot, SWG, type Schema, SchemaValidationError, type SecurityScheme, type Servers, type SwaggerConfig, TokenExpiredError, TooManyRequestsError, UserUnauthorizedError, bootstrap, createLogger, createReqSnapshot, getDefaultConsoleLogger, getDefaultFileLogger, getEnvVar, getTmpDir, getTmpPath, isDev, isProd, parseDefaultPagination, parseIdOrFail, parsePositiveInteger, slugify };
+export { type AlertHandler, ApiError, ApiErrorResponse, ApiResponse, type AuthMethod, BadRequestError, type Container, type Content, type ContentType, DuplicateError, type Env, type ExpressHandler, type ExpressLocalsObj, type ExpressRoute, FileTooBigError, ForbiddenError, type HttpMethod, InternalError, InvalidCredentialsError, InvalidFileTypeError, type Logger, NotFoundError, type OtherString, type OtherUnknown, type Pagination, type PaginationQuery, type Param, type PathItem, SWG, type Schema, SchemaValidationError, type SecurityScheme, type Servers, type SwaggerConfig, TokenExpiredError, TooManyRequestsError, UserUnauthorizedError, bootstrap, getEnvVar, getTmpDir, getTmpPath, isDev, isProd, parseDefaultPagination, parseIdOrFail, parsePositiveInteger, slugify };

package/dist/index.d.ts

@@ -1,8 +1,7 @@
 import * as express from 'express';
-import express__default, { RequestHandler, Request } from 'express';
+import express__default, { RequestHandler } from 'express';
 import * as express_serve_static_core from 'express-serve-static-core';
 import { RouteParameters } from 'express-serve-static-core';
-import winston, { Logger as Logger$1 } from 'winston';
 import { HelmetOptions } from 'helmet';
 import morgan from 'morgan';
 
@@ -17,19 +16,18 @@ type Pagination = {
     limit: number;
     offset: number;
 };
-type ReqSnapshot = {
-    readonly query: qs.ParsedQs;
-    readonly path: string;
-    readonly method: string;
-    readonly userId: string | number | undefined;
-};
 
 type HttpMethod = 'get' | 'post' | 'put' | 'patch' | 'delete' | 'head' | 'options' | 'trace';
 type OtherString = string & {};
 type OtherUnknown = unknown & {};
 type ContentType = 'application/json' | 'application/xml' | 'text/plain' | 'text/html' | OtherString;
-type AlertHandler = (err: Error & Record<string, any>, reqSnapshot?: ReqSnapshot) => void | Promise<void>;
-type Logger = Logger$1;
+type AlertHandler = (err: Error & Record<string, any>, context?: string) => void | Promise<void>;
+type Logger = {
+    info(message: string, ...args: any[]): void;
+    error(message: string | Error | unknown, ...args: any[]): void;
+    warn(message: string, ...args: any[]): void;
+    debug(message: string, ...args: any[]): void;
+};
 type Container = {
     logger: Logger;
     alertHandler?: AlertHandler;
@@ -263,21 +261,12 @@ declare function slugify(text: string): string;
 declare function getTmpDir(): string;
 declare function getTmpPath(...steps: string[]): string;
 declare function parseDefaultPagination(query: PaginationQuery): Pagination;
-declare function createReqSnapshot(req: Request & {
-    user?: {
-        id?: string | number;
-    };
-}): ReqSnapshot;
 
 declare function isDev(): boolean;
 declare function isProd(): boolean;
 declare function getEnvVar(configName: string): string;
 type Env = 'dev' | 'prod' | OtherString;
 
-declare const createLogger: (options?: winston.LoggerOptions) => winston.Logger;
-declare const getDefaultFileLogger: (name?: string) => winston.Logger;
-declare const getDefaultConsoleLogger: (name?: string) => winston.Logger;
-
 declare class ApiErrorResponse<T = undefined> {
     readonly status = "error";
     readonly message: string;
@@ -294,7 +283,7 @@ declare class ApiResponse<T = undefined> {
 
 declare function bootstrap(container: Container): {
     swaggerBuilder: () => SwaggerBuilder;
-    silently: (fn: () => Promise<void>, reqSnapshot?: ReqSnapshot) => void;
+    silently: (fn: () => Promise<void> | void) => Promise<void>;
     notFoundMiddleware: (_req: express.Request, res: express.Response, _next: express.NextFunction) => void;
     getErrorHandlerMiddleware: (errorMapper?: (err: Error & Record<string, unknown>) => ApiError | null | undefined) => (err: Error & Record<string, unknown>, req: express.Request, res: express.Response, _next: express.NextFunction) => Promise<void>;
     expressiveServer: (configs?: {
@@ -321,4 +310,4 @@ declare function bootstrap(container: Container): {
     };
 };
 
-export { type AlertHandler, ApiError, ApiErrorResponse, ApiResponse, type AuthMethod, BadRequestError, type Container, type Content, type ContentType, DuplicateError, type Env, type ExpressHandler, type ExpressLocalsObj, type ExpressRoute, FileTooBigError, ForbiddenError, type HttpMethod, InternalError, InvalidCredentialsError, InvalidFileTypeError, type Logger, NotFoundError, type OtherString, type OtherUnknown, type Pagination, type PaginationQuery, type Param, type PathItem, type ReqSnapshot, SWG, type Schema, SchemaValidationError, type SecurityScheme, type Servers, type SwaggerConfig, TokenExpiredError, TooManyRequestsError, UserUnauthorizedError, bootstrap, createLogger, createReqSnapshot, getDefaultConsoleLogger, getDefaultFileLogger, getEnvVar, getTmpDir, getTmpPath, isDev, isProd, parseDefaultPagination, parseIdOrFail, parsePositiveInteger, slugify };
+export { type AlertHandler, ApiError, ApiErrorResponse, ApiResponse, type AuthMethod, BadRequestError, type Container, type Content, type ContentType, DuplicateError, type Env, type ExpressHandler, type ExpressLocalsObj, type ExpressRoute, FileTooBigError, ForbiddenError, type HttpMethod, InternalError, InvalidCredentialsError, InvalidFileTypeError, type Logger, NotFoundError, type OtherString, type OtherUnknown, type Pagination, type PaginationQuery, type Param, type PathItem, SWG, type Schema, SchemaValidationError, type SecurityScheme, type Servers, type SwaggerConfig, TokenExpiredError, TooManyRequestsError, UserUnauthorizedError, bootstrap, getEnvVar, getTmpDir, getTmpPath, isDev, isProd, parseDefaultPagination, parseIdOrFail, parsePositiveInteger, slugify };

package/dist/index.js

@@ -47,10 +47,6 @@ __export(index_exports, {
   TooManyRequestsError: () => TooManyRequestsError,
   UserUnauthorizedError: () => UserUnauthorizedError,
   bootstrap: () => bootstrap,
-  createLogger: () => createLogger,
-  createReqSnapshot: () => createReqSnapshot,
-  getDefaultConsoleLogger: () => getDefaultConsoleLogger,
-  getDefaultFileLogger: () => getDefaultFileLogger,
   getEnvVar: () => getEnvVar,
   getTmpDir: () => getTmpDir,
   getTmpPath: () => getTmpPath,
@@ -248,8 +244,16 @@ function buildExpressive(container, swaggerDoc) {
           const route = pathOverride ?? convertExpressPath(context.path);
           const pathItem = {
             // -- defaults --
-            responses: {},
-            // has to be defined or else responses are not documented... ¯\_(ツ)_/¯
+            responses: {
+              // has to be defined or else responses are not documented... ¯\_(ツ)_/¯
+              "200": { description: "OK" },
+              "201": { description: "Created" },
+              "204": { description: "No Content" },
+              "400": { description: "Bad Request" },
+              "401": { description: "User Unauthorized" },
+              "403": { description: "Forbidden" },
+              "500": { description: "Internal Server Error" }
+            },
             // -- group defaults --
             ...configs?.oapi || {},
             // -- overrides --
@@ -276,8 +280,22 @@ function buildExpressive(container, swaggerDoc) {
   };
 }
 
-// src/common.ts
-var import_path = __toESM(require("path"));
+// src/env.ts
+var import_dotenv = __toESM(require("dotenv"));
+import_dotenv.default.config();
+function isDev() {
+  return getEnvVar("ENV") === "dev";
+}
+function isProd() {
+  return getEnvVar("ENV") === "prod";
+}
+function getEnvVar(configName) {
+  const config = process.env[configName];
+  if (!config) {
+    throw new Error(`Missing config '${configName}'`);
+  }
+  return config;
+}
 
 // src/errors.ts
 var ApiError = class extends Error {
@@ -356,59 +374,6 @@ var UserUnauthorizedError = class extends ApiError {
   }
 };
 
-// src/common.ts
-function parsePositiveInteger(v, defaultValue, max) {
-  const value = Number(v);
-  return Number.isInteger(value) && value > 0 && (!max || value <= max) ? value : defaultValue;
-}
-function parseIdOrFail(v) {
-  const value = Number(v);
-  if (!Number.isInteger(value) || value <= 0) {
-    throw new ApiError("Invalid Id", 400, "INVALID_ID");
-  }
-  return value;
-}
-function slugify(text) {
-  return text.toString().normalize("NFKD").toLowerCase().trim().replace(/[\s\n_+.]/g, "-").replace(/--+/g, "-");
-}
-function getTmpDir() {
-  return import_path.default.resolve("tmp");
-}
-function getTmpPath(...steps) {
-  return import_path.default.join(getTmpDir(), ...steps);
-}
-function parseDefaultPagination(query) {
-  const limit = parsePositiveInteger(query.limit, 50, 100);
-  const page = parsePositiveInteger(query.page, 1, 1e3);
-  const offset = (page - 1) * limit;
-  return { limit, offset };
-}
-function createReqSnapshot(req) {
-  return {
-    query: req.query,
-    path: req.path,
-    method: req.method,
-    userId: req?.user?.id
-  };
-}
-
-// src/env.ts
-var import_dotenv = __toESM(require("dotenv"));
-import_dotenv.default.config();
-function isDev() {
-  return getEnvVar("ENV") === "dev";
-}
-function isProd() {
-  return getEnvVar("ENV") === "prod";
-}
-function getEnvVar(configName) {
-  const config = process.env[configName];
-  if (!config) {
-    throw new Error(`Missing config '${configName}'`);
-  }
-  return config;
-}
-
 // src/response/ApiErrorResponse.ts
 var ApiErrorResponse = class {
   status = "error";
@@ -448,7 +413,7 @@ var buildMiddleware = (container) => {
             logger.error("Cause: %s", err.cause);
           }
           if (alertHandler) {
-            alertHandler(err, createReqSnapshot(req));
+            alertHandler(err);
           }
           finalError = new InternalError();
           if (!isProd()) {
@@ -461,63 +426,34 @@ var buildMiddleware = (container) => {
   };
 };
 
-// src/logger.ts
-var import_winston = __toESM(require("winston"));
-var import_winston_daily_rotate_file = __toESM(require("winston-daily-rotate-file"));
-var loggerRegistry = {};
-var defaultFormat = import_winston.default.format.combine(
-  import_winston.default.format.timestamp({ format: "DD/MM/YYYY HH:mm:ss" }),
-  import_winston.default.format.splat(),
-  // String interpolation splat for %d %s-style messages.
-  import_winston.default.format.errors({ stack: true }),
-  import_winston.default.format.printf(({ level, message, timestamp, stack }) => `${timestamp}[${level.toUpperCase()}]: ${stack || message}`)
-);
-var consoleTransport = new import_winston.default.transports.Console({
-  handleExceptions: true
-});
-var createFileLogger = (filename) => {
-  const logger = import_winston.default.createLogger({
-    format: defaultFormat,
-    transports: [
-      new import_winston_daily_rotate_file.default({
-        level: "debug",
-        filename: `./logs/${filename}-%DATE%.log`,
-        handleExceptions: true,
-        // exitOnError: false, // do not exit on handled exceptions
-        datePattern: "YYYY-MM-DD",
-        zippedArchive: true,
-        auditFile: "./logs/audit.json",
-        maxSize: "20m",
-        maxFiles: isProd() ? "30d" : "1d"
-        // format: defaultFormat,
-        // the nested 'format' field causes issues with logging errors;
-        // use the 'format' field on logger, instead of transport;
-      })
-    ]
-  });
-  if (isDev()) {
-    logger.add(consoleTransport);
-  }
-  return logger;
-};
-var createLogger = import_winston.default.createLogger;
-var getDefaultFileLogger = (name = "app") => {
-  if (!loggerRegistry[name]) {
-    loggerRegistry[name] = createFileLogger(name.toString());
-  }
-  return loggerRegistry[name];
-};
-var getDefaultConsoleLogger = (name = "console") => {
-  if (!loggerRegistry[name]) {
-    loggerRegistry[name] = import_winston.default.createLogger({
-      format: defaultFormat,
-      transports: [
-        consoleTransport
-      ]
-    });
+// src/common.ts
+var import_path = __toESM(require("path"));
+function parsePositiveInteger(v, defaultValue, max) {
+  const value = Number(v);
+  return Number.isInteger(value) && value > 0 && (!max || value <= max) ? value : defaultValue;
+}
+function parseIdOrFail(v) {
+  const value = Number(v);
+  if (!Number.isInteger(value) || value <= 0) {
+    throw new ApiError("Invalid Id", 400, "INVALID_ID");
   }
-  return loggerRegistry[name];
-};
+  return value;
+}
+function slugify(text) {
+  return text.toString().normalize("NFKD").toLowerCase().trim().replace(/[\s\n_+.]/g, "-").replace(/--+/g, "-");
+}
+function getTmpDir() {
+  return import_path.default.resolve("tmp");
+}
+function getTmpPath(...steps) {
+  return import_path.default.join(getTmpDir(), ...steps);
+}
+function parseDefaultPagination(query) {
+  const limit = parsePositiveInteger(query.limit, 50, 100);
+  const page = parsePositiveInteger(query.page, 1, 1e3);
+  const offset = (page - 1) * limit;
+  return { limit, offset };
+}
 
 // src/response/ApiResponse.ts
 var ApiResponse = class {
@@ -541,14 +477,16 @@ function bootstrap(container) {
     ...buildExpressive(container, swaggerDoc),
     ...buildMiddleware(container),
     swaggerBuilder: () => new SwaggerBuilder(swaggerDoc),
-    silently: (fn, reqSnapshot) => {
-      fn().catch((e) => {
+    silently: async (fn) => {
+      try {
+        await fn();
+      } catch (e) {
         if (container.alertHandler && e instanceof Error) {
-          container.alertHandler(e, reqSnapshot);
+          container.alertHandler(e);
         } else {
           container.logger.error(e);
         }
-      });
+      }
     }
   };
 }
@@ -571,10 +509,6 @@ function bootstrap(container) {
   TooManyRequestsError,
   UserUnauthorizedError,
   bootstrap,
-  createLogger,
-  createReqSnapshot,
-  getDefaultConsoleLogger,
-  getDefaultFileLogger,
   getEnvVar,
   getTmpDir,
   getTmpPath,

package/dist/index.mjs

@@ -183,8 +183,16 @@ function buildExpressive(container, swaggerDoc) {
           const route = pathOverride ?? convertExpressPath(context.path);
           const pathItem = {
             // -- defaults --
-            responses: {},
-            // has to be defined or else responses are not documented... ¯\_(ツ)_/¯
+            responses: {
+              // has to be defined or else responses are not documented... ¯\_(ツ)_/¯
+              "200": { description: "OK" },
+              "201": { description: "Created" },
+              "204": { description: "No Content" },
+              "400": { description: "Bad Request" },
+              "401": { description: "User Unauthorized" },
+              "403": { description: "Forbidden" },
+              "500": { description: "Internal Server Error" }
+            },
             // -- group defaults --
             ...configs?.oapi || {},
             // -- overrides --
@@ -211,8 +219,22 @@ function buildExpressive(container, swaggerDoc) {
   };
 }
 
-// src/common.ts
-import path from "path";
+// src/env.ts
+import dotenv from "dotenv";
+dotenv.config();
+function isDev() {
+  return getEnvVar("ENV") === "dev";
+}
+function isProd() {
+  return getEnvVar("ENV") === "prod";
+}
+function getEnvVar(configName) {
+  const config = process.env[configName];
+  if (!config) {
+    throw new Error(`Missing config '${configName}'`);
+  }
+  return config;
+}
 
 // src/errors.ts
 var ApiError = class extends Error {
@@ -291,59 +313,6 @@ var UserUnauthorizedError = class extends ApiError {
   }
 };
 
-// src/common.ts
-function parsePositiveInteger(v, defaultValue, max) {
-  const value = Number(v);
-  return Number.isInteger(value) && value > 0 && (!max || value <= max) ? value : defaultValue;
-}
-function parseIdOrFail(v) {
-  const value = Number(v);
-  if (!Number.isInteger(value) || value <= 0) {
-    throw new ApiError("Invalid Id", 400, "INVALID_ID");
-  }
-  return value;
-}
-function slugify(text) {
-  return text.toString().normalize("NFKD").toLowerCase().trim().replace(/[\s\n_+.]/g, "-").replace(/--+/g, "-");
-}
-function getTmpDir() {
-  return path.resolve("tmp");
-}
-function getTmpPath(...steps) {
-  return path.join(getTmpDir(), ...steps);
-}
-function parseDefaultPagination(query) {
-  const limit = parsePositiveInteger(query.limit, 50, 100);
-  const page = parsePositiveInteger(query.page, 1, 1e3);
-  const offset = (page - 1) * limit;
-  return { limit, offset };
-}
-function createReqSnapshot(req) {
-  return {
-    query: req.query,
-    path: req.path,
-    method: req.method,
-    userId: req?.user?.id
-  };
-}
-
-// src/env.ts
-import dotenv from "dotenv";
-dotenv.config();
-function isDev() {
-  return getEnvVar("ENV") === "dev";
-}
-function isProd() {
-  return getEnvVar("ENV") === "prod";
-}
-function getEnvVar(configName) {
-  const config = process.env[configName];
-  if (!config) {
-    throw new Error(`Missing config '${configName}'`);
-  }
-  return config;
-}
-
 // src/response/ApiErrorResponse.ts
 var ApiErrorResponse = class {
   status = "error";
@@ -383,7 +352,7 @@ var buildMiddleware = (container) => {
             logger.error("Cause: %s", err.cause);
           }
           if (alertHandler) {
-            alertHandler(err, createReqSnapshot(req));
+            alertHandler(err);
           }
           finalError = new InternalError();
           if (!isProd()) {
@@ -396,63 +365,34 @@ var buildMiddleware = (container) => {
   };
 };
 
-// src/logger.ts
-import winston from "winston";
-import DailyRotateFile from "winston-daily-rotate-file";
-var loggerRegistry = {};
-var defaultFormat = winston.format.combine(
-  winston.format.timestamp({ format: "DD/MM/YYYY HH:mm:ss" }),
-  winston.format.splat(),
-  // String interpolation splat for %d %s-style messages.
-  winston.format.errors({ stack: true }),
-  winston.format.printf(({ level, message, timestamp, stack }) => `${timestamp}[${level.toUpperCase()}]: ${stack || message}`)
-);
-var consoleTransport = new winston.transports.Console({
-  handleExceptions: true
-});
-var createFileLogger = (filename) => {
-  const logger = winston.createLogger({
-    format: defaultFormat,
-    transports: [
-      new DailyRotateFile({
-        level: "debug",
-        filename: `./logs/${filename}-%DATE%.log`,
-        handleExceptions: true,
-        // exitOnError: false, // do not exit on handled exceptions
-        datePattern: "YYYY-MM-DD",
-        zippedArchive: true,
-        auditFile: "./logs/audit.json",
-        maxSize: "20m",
-        maxFiles: isProd() ? "30d" : "1d"
-        // format: defaultFormat,
-        // the nested 'format' field causes issues with logging errors;
-        // use the 'format' field on logger, instead of transport;
-      })
-    ]
-  });
-  if (isDev()) {
-    logger.add(consoleTransport);
-  }
-  return logger;
-};
-var createLogger = winston.createLogger;
-var getDefaultFileLogger = (name = "app") => {
-  if (!loggerRegistry[name]) {
-    loggerRegistry[name] = createFileLogger(name.toString());
-  }
-  return loggerRegistry[name];
-};
-var getDefaultConsoleLogger = (name = "console") => {
-  if (!loggerRegistry[name]) {
-    loggerRegistry[name] = winston.createLogger({
-      format: defaultFormat,
-      transports: [
-        consoleTransport
-      ]
-    });
+// src/common.ts
+import path from "path";
+function parsePositiveInteger(v, defaultValue, max) {
+  const value = Number(v);
+  return Number.isInteger(value) && value > 0 && (!max || value <= max) ? value : defaultValue;
+}
+function parseIdOrFail(v) {
+  const value = Number(v);
+  if (!Number.isInteger(value) || value <= 0) {
+    throw new ApiError("Invalid Id", 400, "INVALID_ID");
   }
-  return loggerRegistry[name];
-};
+  return value;
+}
+function slugify(text) {
+  return text.toString().normalize("NFKD").toLowerCase().trim().replace(/[\s\n_+.]/g, "-").replace(/--+/g, "-");
+}
+function getTmpDir() {
+  return path.resolve("tmp");
+}
+function getTmpPath(...steps) {
+  return path.join(getTmpDir(), ...steps);
+}
+function parseDefaultPagination(query) {
+  const limit = parsePositiveInteger(query.limit, 50, 100);
+  const page = parsePositiveInteger(query.page, 1, 1e3);
+  const offset = (page - 1) * limit;
+  return { limit, offset };
+}
 
 // src/response/ApiResponse.ts
 var ApiResponse = class {
@@ -476,14 +416,16 @@ function bootstrap(container) {
     ...buildExpressive(container, swaggerDoc),
     ...buildMiddleware(container),
     swaggerBuilder: () => new SwaggerBuilder(swaggerDoc),
-    silently: (fn, reqSnapshot) => {
-      fn().catch((e) => {
+    silently: async (fn) => {
+      try {
+        await fn();
+      } catch (e) {
         if (container.alertHandler && e instanceof Error) {
-          container.alertHandler(e, reqSnapshot);
+          container.alertHandler(e);
         } else {
           container.logger.error(e);
         }
-      });
+      }
     }
   };
 }
@@ -505,10 +447,6 @@ export {
   TooManyRequestsError,
   UserUnauthorizedError,
   bootstrap,
-  createLogger,
-  createReqSnapshot,
-  getDefaultConsoleLogger,
-  getDefaultFileLogger,
   getEnvVar,
   getTmpDir,
   getTmpPath,

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@extk/expressive",
-    "version": "0.4.4",
+    "version": "0.5.2",
     "type": "commonjs",
     "publishConfig": {
         "access": "public"
@@ -46,9 +46,7 @@
         "helmet": "^8.0.0",
         "morgan": "^1.10.0",
         "qs": "^6.14.1",
-        "swagger-ui-express": "^5.0.1",
-        "winston": "^3.11.0",
-        "winston-daily-rotate-file": "^5.0.0"
+        "swagger-ui-express": "^5.0.1"
     },
     "author": "",
     "license": "ISC",