fix(data-schemas): add local type declaration for DailyRotateFile transport

The `winston-daily-rotate-file` package ships a module augmentation for
`winston/lib/winston/transports`, but it fails when winston and
winston-daily-rotate-file resolve from different node_modules trees
(which happens in this monorepo due to npm hoisting).

Add a local `.d.ts` declaration that augments the same module path from
within data-schemas' compilation unit, so `tsc --noEmit` passes while
keeping the original runtime pattern (`new winston.transports.DailyRotateFile`).

Author: danny-avila

package-lock.json

@@ -61,8 +61,7 @@
     "rollup-plugin-peer-deps-external": "^2.2.4",
     "rollup-plugin-typescript2": "^0.35.0",
     "ts-node": "^10.9.2",
-    "typescript": "^5.0.4",
-    "winston-daily-rotate-file": "^5.0.0"
+    "typescript": "^5.0.4"
   },
   "peerDependencies": {
     "jsonwebtoken": "^9.0.2",

packages/data-schemas/package.json

@@ -1,5 +1,5 @@
 import winston from 'winston';
-import DailyRotateFile from 'winston-daily-rotate-file';
+import 'winston-daily-rotate-file';
 import { getLogDirectory } from './utils';
 
 const logDir = getLogDirectory();
@@ -42,7 +42,7 @@ const fileFormat = winston.format.combine(
 
 const logLevel = useDebugLogging ? 'debug' : 'error';
 const transports: winston.transport[] = [
-  new DailyRotateFile({
+  new winston.transports.DailyRotateFile({
     level: logLevel,
     filename: `${logDir}/meiliSync-%DATE%.log`,
     datePattern: 'YYYY-MM-DD',

packages/data-schemas/src/config/meiliLogger.ts

@@ -1,5 +1,5 @@
 import winston from 'winston';
-import DailyRotateFile from 'winston-daily-rotate-file';
+import 'winston-daily-rotate-file';
 import { redactFormat, redactMessage, debugTraverse, jsonTruncateFormat } from './parsers';
 import { getLogDirectory } from './utils';
 
@@ -44,7 +44,7 @@ const fileFormat = winston.format.combine(
 );
 
 const transports: winston.transport[] = [
-  new DailyRotateFile({
+  new winston.transports.DailyRotateFile({
     level: 'error',
     filename: `${logDir}/error-%DATE%.log`,
     datePattern: 'YYYY-MM-DD',
@@ -57,7 +57,7 @@ const transports: winston.transport[] = [
 
 if (useDebugLogging) {
   transports.push(
-    new DailyRotateFile({
+    new winston.transports.DailyRotateFile({
       level: 'debug',
       filename: `${logDir}/debug-%DATE%.log`,
       datePattern: 'YYYY-MM-DD',

packages/data-schemas/src/config/winston.ts

@@ -0,0 +1,34 @@
+import type TransportStream from 'winston-transport';
+
+/**
+ * Module augmentation for winston's transports namespace.
+ *
+ * `winston-daily-rotate-file` ships its own augmentation targeting
+ * `'winston/lib/winston/transports'`, but it fails when winston and
+ * winston-daily-rotate-file resolve from different node_modules trees
+ * (which happens in this monorepo due to npm hoisting). This local
+ * declaration bridges the gap so `tsc --noEmit` passes.
+ */
+declare module 'winston/lib/winston/transports' {
+  interface Transports {
+    DailyRotateFile: new (
+      opts?: {
+        level?: string;
+        filename?: string;
+        datePattern?: string;
+        zippedArchive?: boolean;
+        maxSize?: string | number;
+        maxFiles?: string | number;
+        dirname?: string;
+        stream?: NodeJS.WritableStream;
+        frequency?: string;
+        utc?: boolean;
+        extension?: string;
+        createSymlink?: boolean;
+        symlinkName?: string;
+        auditFile?: string;
+        format?: import('logform').Format;
+      } & TransportStream.TransportStreamOptions,
+    ) => TransportStream;
+  }
+}