@mhmdhammoud/meritt-utils 1.5.2 → 1.5.4

package/.github/workflows/npm-publish.yml

@@ -3,41 +3,82 @@ name: NPM Publish on Release
 on:
   push:
     branches: [master]
+
 jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run type check
+        run: npm run test:types
+
+      - name: Run linting
+        run: npm run lint
+
+      - name: Run tests
+        run: npm test
+
   build:
+    needs: test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
         with:
-          node-version: 16
-      - run: npm ci
+          node-version: '20'
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build project
+        run: npm run build
+
+      - name: Upload build artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: dist
+          path: dist/
 
   publish-npm:
-    needs: build
+    needs: [test, build]
     runs-on: ubuntu-latest
+    if: success()
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
         with:
-          node-version: 16
+          node-version: '20'
           registry-url: https://registry.npmjs.org/
-      - run: npm ci
-      - run: npm publish
-        env:
-          NODE_AUTH_TOKEN: ${{secrets.npm_token}}
-  notify:
-    needs: [publish-npm]
-    runs-on: ubuntu-latest
-    steps:
-      - name: Notify by Email
-        uses: dawidd6/action-send-mail@v2
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Download build artifacts
+        uses: actions/download-artifact@v4
         with:
-          server_address: ${{ secrets.EMAIL_HOST }}
-          server_port: 465
-          username: ${{ secrets.EMAIL_USERNAME }}
-          password: ${{ secrets.EMAIL_PASSWORD }}
-          subject: ${{ github.event.head_commit.message }} ${{ github.job }} job of ${{ github.repository }} has ${{ job.status }}
-          body: ${{ github.job }} job in worflow ${{ github.workflow }} of ${{ github.repository }} has ${{ job.status }}
-          to: mohammad.hammoud.lb@hotmail.com,steef12009@gmail.com
-          from: Github Action
+          name: dist
+          path: dist/
+
+      - name: Publish to NPM
+        run: npm publish
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.npm_token }}

package/.github/workflows/push.yml

@@ -1,29 +1,83 @@
-name: Changes
+name: CI Pipeline
+
 on:
   push:
     branches: [dev]
+  pull_request:
+    branches: [dev, master]
+
 jobs:
-  type-check:
+  lint-and-test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - name: Checkout code
+        uses: actions/checkout@v4
+      
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
         with:
-          node-version: 16
-      - run: npm ci
-      - run: npm run test:types
-  notify:
-    needs: [type-check]
+          node-version: '20'
+          cache: 'npm'
+      
+      - name: Install dependencies
+        run: npm ci
+      
+      - name: Run type check
+        run: npm run test:types
+      
+      - name: Run linting
+        run: npm run lint
+      
+      - name: Run tests with coverage
+        run: npm run test:coverage
+      
+      - name: Upload coverage reports
+        uses: actions/upload-artifact@v4
+        if: always()
+        with:
+          name: coverage-report
+          path: coverage/
+
+  build:
+    needs: lint-and-test
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+      
+      - name: Install dependencies
+        run: npm ci
+      
+      - name: Build project
+        run: npm run build
+      
+      - name: Upload build artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: dist
+          path: dist/
+
+  security-audit:
     runs-on: ubuntu-latest
     steps:
-      - name: Notify by Email
-        uses: dawidd6/action-send-mail@v2
+      - name: Checkout code
+        uses: actions/checkout@v4
+      
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
         with:
-          server_address: ${{ secrets.EMAIL_HOST }}
-          server_port: 465
-          username: ${{ secrets.EMAIL_USERNAME }}
-          password: ${{ secrets.EMAIL_PASSWORD }}
-          subject: ${{ github.event.head_commit.message }} ${{ github.job }} job of ${{ github.repository }} has ${{ job.status }}
-          body: ${{ github.job }} job in worflow ${{ github.workflow }} of ${{ github.repository }} has ${{ job.status }}
-          to: mohammad.hammoud.lb@hotmail.com,steef12009@gmail.com
-          from: Github Action
+          node-version: '20'
+          cache: 'npm'
+      
+      - name: Install dependencies
+        run: npm ci
+      
+      - name: Run security audit
+        run: npm audit --audit-level=moderate
+

package/.prettierrc

@@ -0,0 +1,8 @@
+{
+	"semi": false,
+	"singleQuote": true,
+	"trailingComma": "es5",
+	"tabWidth": 2,
+	"useTabs": true,
+	"printWidth": 80
+}

package/dist/__tests__/logger.test.js

@@ -15,13 +15,23 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 const logger_1 = __importStar(require("../lib/logger"));
 const pino_1 = require("pino");

package/dist/lib/cypto.d.ts

@@ -63,7 +63,7 @@ declare class Crypto {
      * crypto.generateKeys()
      * ```
      * */
-    generateKeys: () => Record<'publicKey' | 'privateKey', number>;
+    generateKeys: () => Record<"publicKey" | "privateKey", number>;
     /**
      *
      * @param publicKey - The public key number

package/dist/lib/elastic-transport.d.ts

@@ -0,0 +1,27 @@
+/**
+ * Elasticsearch transport for Pino with connection lifecycle resilience.
+ *
+ * Based on pino-elasticsearch with a fix for GitHub issue #140:
+ * When maxRetries are exceeded and Elasticsearch nodes are DEAD, the bulk helper
+ * destroys the splitter stream, causing logs to stop permanently until restart.
+ *
+ * This implementation overrides splitter.destroy to BOTH resurrect the connection
+ * pool AND reinitialize the bulk handler, so logging continues after ES recovers.
+ *
+ * @see https://github.com/pinojs/pino-elasticsearch/issues/140
+ * @see https://github.com/pinojs/pino-elasticsearch/issues/72
+ */
+import type { ClientOptions } from '@elastic/elasticsearch';
+export interface ElasticTransportOptions extends Pick<ClientOptions, 'node' | 'auth' | 'cloud' | 'caFingerprint' | 'Connection' | 'ConnectionPool' | 'maxRetries' | 'requestTimeout'> {
+    sniffOnConnectionFault?: boolean;
+    index?: string | ((logTime: string) => string);
+    flushBytes?: number;
+    'flush-bytes'?: number;
+    flushInterval?: number;
+    'flush-interval'?: number;
+    esVersion?: number;
+    'es-version'?: number;
+    rejectUnauthorized?: boolean;
+    tls?: ClientOptions['tls'];
+}
+export declare const createElasticTransport: (opts?: ElasticTransportOptions) => NodeJS.ReadWriteStream;

package/dist/lib/elastic-transport.js

@@ -0,0 +1,137 @@
+"use strict";
+/**
+ * Elasticsearch transport for Pino with connection lifecycle resilience.
+ *
+ * Based on pino-elasticsearch with a fix for GitHub issue #140:
+ * When maxRetries are exceeded and Elasticsearch nodes are DEAD, the bulk helper
+ * destroys the splitter stream, causing logs to stop permanently until restart.
+ *
+ * This implementation overrides splitter.destroy to BOTH resurrect the connection
+ * pool AND reinitialize the bulk handler, so logging continues after ES recovers.
+ *
+ * @see https://github.com/pinojs/pino-elasticsearch/issues/140
+ * @see https://github.com/pinojs/pino-elasticsearch/issues/72
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.createElasticTransport = void 0;
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const split = require('split2');
+const elasticsearch_1 = require("@elastic/elasticsearch");
+function setDateTimeString(value) {
+    if (value !== null && typeof value === 'object' && 'time' in value) {
+        const t = value.time;
+        if ((typeof t === 'string' && t.length > 0) ||
+            (typeof t === 'number' && t >= 0)) {
+            return new Date(t).toISOString();
+        }
+    }
+    return new Date().toISOString();
+}
+function getIndexName(index, time) {
+    if (typeof index === 'function') {
+        return index(time);
+    }
+    return index.replace('%{DATE}', time.substring(0, 10));
+}
+function initializeBulkHandler(opts, client, splitter) {
+    var _a, _b, _c, _d, _e, _f, _g;
+    const esVersion = Number((_b = (_a = opts.esVersion) !== null && _a !== void 0 ? _a : opts['es-version']) !== null && _b !== void 0 ? _b : 7);
+    const index = (_c = opts.index) !== null && _c !== void 0 ? _c : 'pino';
+    const buildIndexName = typeof index === 'function' ? index : null;
+    const opType = esVersion >= 7 ? undefined : undefined;
+    // CRITICAL FIX (issue #140): When bulk helper destroys stream after retries exhausted,
+    // we must BOTH resurrect the pool AND reinitialize the bulk handler so logging continues.
+    // connectionPool.resurrect exists at runtime (elastic-transport) but may not be in types
+    const pool = client.connectionPool;
+    const splitterWithDestroy = splitter;
+    splitterWithDestroy.destroy = function () {
+        if (typeof pool.resurrect === 'function') {
+            pool.resurrect({ name: 'elasticsearch-js' });
+        }
+        // Reinitialize bulk handler - without this, logging stops permanently until restart
+        initializeBulkHandler(opts, client, splitter);
+    };
+    const indexName = (time = new Date().toISOString()) => buildIndexName ? buildIndexName(time) : getIndexName(index, time);
+    const bulkInsert = client.helpers.bulk({
+        datasource: splitter,
+        flushBytes: (_e = (_d = opts.flushBytes) !== null && _d !== void 0 ? _d : opts['flush-bytes']) !== null && _e !== void 0 ? _e : 1000,
+        flushInterval: (_g = (_f = opts.flushInterval) !== null && _f !== void 0 ? _f : opts['flush-interval']) !== null && _g !== void 0 ? _g : 3000,
+        refreshOnCompletion: indexName(),
+        onDocument(doc) {
+            var _a, _b;
+            const d = doc;
+            const date = (_b = (_a = d.time) !== null && _a !== void 0 ? _a : d['@timestamp']) !== null && _b !== void 0 ? _b : new Date().toISOString();
+            if (opType === 'create') {
+                d['@timestamp'] = date;
+            }
+            return {
+                index: {
+                    _index: indexName(date),
+                    op_type: opType,
+                },
+            };
+        },
+        onDrop(doc) {
+            const error = new Error('Dropped document');
+            error.document = doc;
+            splitter.emit('insertError', error);
+        },
+    });
+    bulkInsert.then((stats) => splitter.emit('insert', stats), (err) => splitter.emit('error', err));
+}
+const createElasticTransport = (opts = {}) => {
+    const splitter = split(function (line) {
+        let value;
+        try {
+            value = JSON.parse(line);
+        }
+        catch (error) {
+            this.emit('unknown', line, error);
+            return;
+        }
+        if (typeof value === 'boolean') {
+            this.emit('unknown', line, 'Boolean value ignored');
+            return;
+        }
+        if (value === null) {
+            this.emit('unknown', line, 'Null value ignored');
+            return;
+        }
+        if (typeof value !== 'object') {
+            value = { data: value, time: setDateTimeString(value) };
+        }
+        else {
+            const obj = value;
+            if (obj['@timestamp'] === undefined) {
+                ;
+                obj.time = setDateTimeString(obj);
+            }
+        }
+        return value;
+    }, { autoDestroy: true });
+    const clientOpts = {
+        node: opts.node,
+        auth: opts.auth,
+        cloud: opts.cloud,
+        tls: { rejectUnauthorized: opts.rejectUnauthorized, ...opts.tls },
+        maxRetries: opts.maxRetries,
+        requestTimeout: opts.requestTimeout,
+        sniffOnConnectionFault: opts.sniffOnConnectionFault,
+    };
+    if (opts.caFingerprint) {
+        clientOpts.caFingerprint = opts.caFingerprint;
+    }
+    if (opts.Connection) {
+        clientOpts.Connection = opts.Connection;
+    }
+    if (opts.ConnectionPool) {
+        clientOpts.ConnectionPool = opts.ConnectionPool;
+    }
+    const client = new elasticsearch_1.Client(clientOpts);
+    client.diagnostic.on('resurrect', () => {
+        initializeBulkHandler(opts, client, splitter);
+    });
+    initializeBulkHandler(opts, client, splitter);
+    return splitter;
+};
+exports.createElasticTransport = createElasticTransport;

package/dist/lib/logger.js

@@ -15,21 +15,28 @@ var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (
 }) : function(o, v) {
     o["default"] = v;
 });
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.isValidLogLevel = void 0;
+exports.isValidLogLevel = isValidLogLevel;
 const pino_1 = require("pino");
 const dotenv = __importStar(require("dotenv"));
-const pino_elasticsearch_1 = __importDefault(require("pino-elasticsearch"));
+const elastic_transport_1 = require("./elastic-transport");
 dotenv.config();
 /**
  * Pino logger backend - singleton
@@ -48,8 +55,13 @@ let shutdownHandlersRegistered = false;
  * @throws Error if required environment variables are missing.
  */
 function validateElasticsearchEnv() {
-    const required = ['ELASTICSEARCH_NODE', 'ELASTICSEARCH_USERNAME', 'ELASTICSEARCH_PASSWORD', 'SERVER_NICKNAME'];
-    const missing = required.filter(key => !process.env[key]);
+    const required = [
+        'ELASTICSEARCH_NODE',
+        'ELASTICSEARCH_USERNAME',
+        'ELASTICSEARCH_PASSWORD',
+        'SERVER_NICKNAME',
+    ];
+    const missing = required.filter((key) => !process.env[key]);
     if (missing.length > 0) {
         throw new Error(`Missing required Elasticsearch environment variables: ${missing.join(', ')}`);
     }
@@ -175,9 +187,8 @@ function getLogger(elasticConfig) {
             if (elasticConfig) {
                 Object.assign(esConfig, elasticConfig);
             }
-            // Create transport and store reference for cleanup
-            // Cast to PinoElasticOptions since our ElasticConfig includes ClientOptions properties
-            esTransport = (0, pino_elasticsearch_1.default)(esConfig);
+            // Create transport with connection lifecycle fix (pino-elasticsearch #140)
+            esTransport = (0, elastic_transport_1.createElasticTransport)(esConfig);
             // Handle Elasticsearch connection errors
             esTransport.on('error', (err) => {
                 console.error('[Logger] Elasticsearch transport error:', err.message);
@@ -220,7 +231,6 @@ function isValidLogLevel(level) {
     }
     return true;
 }
-exports.isValidLogLevel = isValidLogLevel;
 /**
  * Logger Wrapper.
  * Wraps a Pino logger instance and provides logging methods.

package/dist/lib/pdf.d.ts

@@ -9,7 +9,7 @@ declare class Pdf {
      * // => 'Hello-World'
      * ```
      * */
-    getBase64Images: (url: string) => string;
+    getBase64Images: (_url: string) => string;
 }
 declare const pdf: Pdf;
 export default pdf;

package/dist/lib/pdf.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+// import { AxiosInstance } from '../utilities'
 /*
      Author : Mustafa Halabi https://github.com/mustafahalabi
      Date : 2023-06-24
@@ -18,7 +19,7 @@ class Pdf {
          * // => 'Hello-World'
          * ```
          * */
-        this.getBase64Images = (url) => {
+        this.getBase64Images = (_url) => {
             return '';
         };
     }

package/eslint.config.mjs

@@ -0,0 +1,64 @@
+// @ts-check
+import eslint from '@eslint/js'
+import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended'
+import globals from 'globals'
+import tseslint from 'typescript-eslint'
+
+export default tseslint.config(
+	{
+		ignores: [
+			'eslint.config.mjs',
+			'dist/**',
+			'node_modules/**',
+			'coverage/**',
+			'*.d.ts',
+		],
+	},
+	eslint.configs.recommended,
+	...tseslint.configs.recommendedTypeChecked,
+	eslintPluginPrettierRecommended,
+	{
+		languageOptions: {
+			globals: {
+				...globals.node,
+			},
+			sourceType: 'module',
+			parserOptions: {
+				projectService: true,
+				tsconfigRootDir: import.meta.dirname,
+			},
+		},
+	},
+	{
+		rules: {
+			'@typescript-eslint/no-explicit-any': 'off',
+			'@typescript-eslint/no-floating-promises': 'warn',
+			'@typescript-eslint/no-unsafe-argument': 'off',
+			'@typescript-eslint/no-unsafe-assignment': 'off',
+			'@typescript-eslint/no-unsafe-return': 'off',
+			'@typescript-eslint/no-unsafe-member-access': 'off',
+			'@typescript-eslint/no-unsafe-call': 'off',
+			'@typescript-eslint/ban-ts-comment': 'off',
+			'@typescript-eslint/require-await': 'off',
+			'@typescript-eslint/await-thenable': 'off',
+			'@typescript-eslint/no-unsafe-enum-comparison': 'off',
+			'@typescript-eslint/no-base-to-string': 'off',
+			'@typescript-eslint/no-redundant-type-constituents': 'off',
+			'@typescript-eslint/no-unused-vars': [
+				'error',
+				{
+					argsIgnorePattern: '^_',
+					varsIgnorePattern: '^_',
+				},
+			],
+			'no-console': 'error',
+			'prettier/prettier': ['error', { semi: false }],
+		},
+	},
+	{
+		files: ['src/lib/logger.ts'],
+		rules: {
+			'no-console': 'off',
+		},
+	}
+)

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@mhmdhammoud/meritt-utils",
-	"version": "1.5.2",
+	"version": "1.5.4",
 	"description": "",
 	"main": "./dist/index.js",
 	"private": false,
@@ -25,25 +25,31 @@
 		"test:coverage": "jest --coverage"
 	},
 	"devDependencies": {
-		"@types/jest": "^27.5.2",
-		"@typescript-eslint/eslint-plugin": "^5.17.0",
-		"@typescript-eslint/parser": "^5.17.0",
-		"eslint": "^8.12.0",
-		"eslint-plugin-tsdoc": "^0.2.14",
-		"husky": "^8.0.0",
+		"@eslint/js": "^9.31.0",
+		"@types/jest": "^29.5.12",
+		"eslint": "^9.31.0",
+		"eslint-config-prettier": "^10.1.8",
+		"eslint-plugin-prettier": "^5.5.3",
+		"eslint-plugin-tsdoc": "^0.5.2",
+		"globals": "^16.0.0",
+		"husky": "^9.1.6",
 		"jest": "^29.7.0",
 		"pino-pretty": "^10.3.1",
+		"prettier": "^3.3.3",
 		"ts-jest": "^29.1.1",
-		"ts-node-dev": "^1.1.8",
-		"typescript": "^4.6.3"
+		"ts-node-dev": "^2.0.0",
+		"typescript": "^5.3.3",
+		"typescript-eslint": "^8.20.0"
 	},
 	"author": "Mhmdhammoud",
 	"license": "ISC",
 	"dependencies": {
+		"@elastic/elasticsearch": "^8.17.0",
 		"axios": "^1.4.0",
 		"dotenv": "^16.4.1",
 		"imagesloaded": "^5.0.0",
 		"pino": "^8.19.0",
-		"pino-elasticsearch": "^8.0.0"
+		"pino-elasticsearch": "^8.0.0",
+		"split2": "^4.2.0"
 	}
 }

package/src/__tests__/colorful.test.ts

@@ -1,4 +1,4 @@
-import {Colorful} from '../lib'
+import { Colorful } from '../lib'
 
 describe('Colorful Class', () => {
 	describe('rgbToHex method', () => {

package/src/__tests__/formatter.test.ts

@@ -1,4 +1,4 @@
-import {Formatter} from '../lib'
+import { Formatter } from '../lib'
 
 describe('Formatter', () => {
 	describe('toUpperFirst method', () => {

package/src/__tests__/logger.test.ts

@@ -1,5 +1,5 @@
-import Logger, {isValidLogLevel} from '../lib/logger'
-import {pino} from 'pino'
+import Logger, { isValidLogLevel } from '../lib/logger'
+import { pino } from 'pino'
 
 jest.mock('pino')
 

package/src/lib/elastic-transport.ts

@@ -0,0 +1,203 @@
+/**
+ * Elasticsearch transport for Pino with connection lifecycle resilience.
+ *
+ * Based on pino-elasticsearch with a fix for GitHub issue #140:
+ * When maxRetries are exceeded and Elasticsearch nodes are DEAD, the bulk helper
+ * destroys the splitter stream, causing logs to stop permanently until restart.
+ *
+ * This implementation overrides splitter.destroy to BOTH resurrect the connection
+ * pool AND reinitialize the bulk handler, so logging continues after ES recovers.
+ *
+ * @see https://github.com/pinojs/pino-elasticsearch/issues/140
+ * @see https://github.com/pinojs/pino-elasticsearch/issues/72
+ */
+
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const split = require('split2') as (
+	fn: (line: string) => unknown,
+	opts?: { autoDestroy?: boolean }
+) => NodeJS.ReadWriteStream
+import { Readable } from 'stream'
+import { Client } from '@elastic/elasticsearch'
+import type { ClientOptions } from '@elastic/elasticsearch'
+
+export interface ElasticTransportOptions extends Pick<
+	ClientOptions,
+	| 'node'
+	| 'auth'
+	| 'cloud'
+	| 'caFingerprint'
+	| 'Connection'
+	| 'ConnectionPool'
+	| 'maxRetries'
+	| 'requestTimeout'
+> {
+	sniffOnConnectionFault?: boolean
+	index?: string | ((logTime: string) => string)
+	flushBytes?: number
+	'flush-bytes'?: number
+	flushInterval?: number
+	'flush-interval'?: number
+	esVersion?: number
+	'es-version'?: number
+	rejectUnauthorized?: boolean
+	tls?: ClientOptions['tls']
+}
+
+interface LogDocument {
+	time?: string
+	'@timestamp'?: string
+	[k: string]: unknown
+}
+
+function setDateTimeString(value: unknown): string {
+	if (value !== null && typeof value === 'object' && 'time' in value) {
+		const t = (value as { time: unknown }).time
+		if (
+			(typeof t === 'string' && t.length > 0) ||
+			(typeof t === 'number' && t >= 0)
+		) {
+			return new Date(t).toISOString()
+		}
+	}
+	return new Date().toISOString()
+}
+
+function getIndexName(
+	index: string | ((logTime: string) => string),
+	time: string
+): string {
+	if (typeof index === 'function') {
+		return index(time)
+	}
+	return index.replace('%{DATE}', time.substring(0, 10))
+}
+
+function initializeBulkHandler(
+	opts: ElasticTransportOptions,
+	client: Client,
+	splitter: NodeJS.ReadWriteStream
+): void {
+	const esVersion = Number(opts.esVersion ?? opts['es-version'] ?? 7)
+	const index = opts.index ?? 'pino'
+	const buildIndexName = typeof index === 'function' ? index : null
+	const opType = esVersion >= 7 ? undefined : undefined
+
+	// CRITICAL FIX (issue #140): When bulk helper destroys stream after retries exhausted,
+	// we must BOTH resurrect the pool AND reinitialize the bulk handler so logging continues.
+	// connectionPool.resurrect exists at runtime (elastic-transport) but may not be in types
+	const pool = client.connectionPool as {
+		resurrect?: (opts: { name: string }) => void
+	}
+	const splitterWithDestroy = splitter as NodeJS.ReadWriteStream & {
+		destroy: (err?: Error) => void
+	}
+	splitterWithDestroy.destroy = function () {
+		if (typeof pool.resurrect === 'function') {
+			pool.resurrect({ name: 'elasticsearch-js' })
+		}
+		// Reinitialize bulk handler - without this, logging stops permanently until restart
+		initializeBulkHandler(opts, client, splitter)
+	}
+
+	const indexName = (time = new Date().toISOString()) =>
+		buildIndexName ? buildIndexName(time) : getIndexName(index as string, time)
+
+	const bulkInsert = client.helpers.bulk({
+		datasource: splitter as unknown as Readable,
+		flushBytes: opts.flushBytes ?? opts['flush-bytes'] ?? 1000,
+		flushInterval: opts.flushInterval ?? opts['flush-interval'] ?? 3000,
+		refreshOnCompletion: indexName(),
+		onDocument(doc: unknown) {
+			const d = doc as LogDocument
+			const date = d.time ?? d['@timestamp'] ?? new Date().toISOString()
+			if (opType === 'create') {
+				d['@timestamp'] = date
+			}
+			return {
+				index: {
+					_index: indexName(date),
+					op_type: opType,
+				},
+			}
+		},
+		onDrop(doc: unknown) {
+			const error = new Error('Dropped document') as Error & {
+				document: unknown
+			}
+			error.document = doc
+			splitter.emit('insertError', error)
+		},
+	})
+
+	bulkInsert.then(
+		(stats) => splitter.emit('insert', stats),
+		(err) => splitter.emit('error', err)
+	)
+}
+
+export const createElasticTransport = (
+	opts: ElasticTransportOptions = {}
+): NodeJS.ReadWriteStream => {
+	const splitter = split(
+		function (this: NodeJS.ReadWriteStream, line: string) {
+			let value: unknown
+
+			try {
+				value = JSON.parse(line) as unknown
+			} catch (error) {
+				this.emit('unknown', line, error)
+				return
+			}
+
+			if (typeof value === 'boolean') {
+				this.emit('unknown', line, 'Boolean value ignored')
+				return
+			}
+			if (value === null) {
+				this.emit('unknown', line, 'Null value ignored')
+				return
+			}
+			if (typeof value !== 'object') {
+				value = { data: value, time: setDateTimeString(value) }
+			} else {
+				const obj = value as Record<string, unknown>
+				if (obj['@timestamp'] === undefined) {
+					;(obj as LogDocument).time = setDateTimeString(obj)
+				}
+			}
+			return value
+		},
+		{ autoDestroy: true }
+	)
+
+	const clientOpts: ClientOptions = {
+		node: opts.node,
+		auth: opts.auth,
+		cloud: opts.cloud,
+		tls: { rejectUnauthorized: opts.rejectUnauthorized, ...opts.tls },
+		maxRetries: opts.maxRetries,
+		requestTimeout: opts.requestTimeout,
+		sniffOnConnectionFault: opts.sniffOnConnectionFault,
+	}
+
+	if (opts.caFingerprint) {
+		clientOpts.caFingerprint = opts.caFingerprint
+	}
+	if (opts.Connection) {
+		clientOpts.Connection = opts.Connection
+	}
+	if (opts.ConnectionPool) {
+		clientOpts.ConnectionPool = opts.ConnectionPool
+	}
+
+	const client = new Client(clientOpts)
+
+	client.diagnostic.on('resurrect', () => {
+		initializeBulkHandler(opts, client, splitter)
+	})
+
+	initializeBulkHandler(opts, client, splitter)
+
+	return splitter
+}

package/src/lib/imagefull.ts

@@ -28,7 +28,7 @@ class ImageFull {
 		return new Promise((resolve) => {
 			imagesLoaded(
 				document.querySelectorAll(selector),
-				{background: true},
+				{ background: true },
 				(event) => {
 					resolve(event)
 				}

package/src/lib/index.ts

@@ -1,6 +1,6 @@
-export {default as Crypto} from './cypto'
-export {default as Formatter} from './formatter'
-export {default as Pdf} from './formatter'
-export {default as Colorful} from './colorful'
-export {default as ImageFull} from './imagefull'
-export {default as Logger} from './logger'
+export { default as Crypto } from './cypto'
+export { default as Formatter } from './formatter'
+export { default as Pdf } from './formatter'
+export { default as Colorful } from './colorful'
+export { default as ImageFull } from './imagefull'
+export { default as Logger } from './logger'

package/src/lib/logger.ts

@@ -1,7 +1,7 @@
-import {Logger as PinoLogger, pino, stdTimeFunctions} from 'pino'
+import { Logger as PinoLogger, pino, stdTimeFunctions } from 'pino'
 import * as dotenv from 'dotenv'
-import pinoElastic, {Options as PinoElasticOptions} from 'pino-elasticsearch'
-import {LOG_LEVEL, LogEvent, ElasticConfig} from '../types'
+import { createElasticTransport } from './elastic-transport'
+import { LOG_LEVEL, LogEvent, ElasticConfig } from '../types'
 
 dotenv.config()
 
@@ -13,7 +13,7 @@ let pinoLogger: PinoLogger
 /**
  * Elasticsearch transport instance - kept for cleanup
  */
-let esTransport: ReturnType<typeof pinoElastic> | null = null
+let esTransport: NodeJS.ReadWriteStream | null = null
 
 /**
  * Flag to track if shutdown handlers are registered
@@ -25,8 +25,13 @@ let shutdownHandlersRegistered = false
  * @throws Error if required environment variables are missing.
  */
 function validateElasticsearchEnv(): void {
-	const required = ['ELASTICSEARCH_NODE', 'ELASTICSEARCH_USERNAME', 'ELASTICSEARCH_PASSWORD', 'SERVER_NICKNAME']
-	const missing = required.filter(key => !process.env[key])
+	const required = [
+		'ELASTICSEARCH_NODE',
+		'ELASTICSEARCH_USERNAME',
+		'ELASTICSEARCH_PASSWORD',
+		'SERVER_NICKNAME',
+	]
+	const missing = required.filter((key) => !process.env[key])
 
 	if (missing.length > 0) {
 		throw new Error(
@@ -43,7 +48,11 @@ function validateElasticsearchEnv(): void {
  * @returns The parsed integer or the default value.
  * @throws Error if the value is not a valid number.
  */
-function parseIntEnv(envValue: string | undefined, defaultValue: number, varName: string): number {
+function parseIntEnv(
+	envValue: string | undefined,
+	defaultValue: number,
+	varName: string
+): number {
 	if (!envValue) {
 		return defaultValue
 	}
@@ -177,8 +186,8 @@ function getLogger(elasticConfig?: ElasticConfig): PinoLogger {
 				index: process.env.SERVER_NICKNAME,
 				node: process.env.ELASTICSEARCH_NODE,
 				auth: {
-					username: process.env.ELASTICSEARCH_USERNAME as string,
-					password: process.env.ELASTICSEARCH_PASSWORD as string,
+					username: process.env.ELASTICSEARCH_USERNAME,
+					password: process.env.ELASTICSEARCH_PASSWORD,
 				},
 				// Configurable flush settings
 				flushInterval: flushIntervalMs,
@@ -193,18 +202,19 @@ function getLogger(elasticConfig?: ElasticConfig): PinoLogger {
 				Object.assign(esConfig, elasticConfig)
 			}
 
-			// Create transport and store reference for cleanup
-			// Cast to PinoElasticOptions since our ElasticConfig includes ClientOptions properties
-			esTransport = pinoElastic(esConfig as PinoElasticOptions)
+			// Create transport with connection lifecycle fix (pino-elasticsearch #140)
+			esTransport = createElasticTransport(esConfig)
 
 			// Handle Elasticsearch connection errors
-			esTransport.on('error', (err) => {
+			esTransport.on('error', (err: Error) => {
 				console.error('[Logger] Elasticsearch transport error:', err.message)
-				console.error('[Logger] Logs may not be reaching Kibana. Check Elasticsearch connection.')
+				console.error(
+					'[Logger] Logs may not be reaching Kibana. Check Elasticsearch connection.'
+				)
 			})
 
 			// Handle insert errors (document indexing failures)
-			esTransport.on('insertError', (err) => {
+			esTransport.on('insertError', (err: Error) => {
 				console.error('[Logger] Elasticsearch insert error:', err.message)
 				console.error('[Logger] Some logs failed to index to Elasticsearch.')
 			})

package/src/lib/pdf.ts

@@ -1,4 +1,4 @@
-import {AxiosInstance} from '../utilities'
+// import { AxiosInstance } from '../utilities'
 /*
 	 Author : Mustafa Halabi https://github.com/mustafahalabi
 	 Date : 2023-06-24
@@ -16,7 +16,7 @@ class Pdf {
 	 * // => 'Hello-World'
 	 * ```
 	 * */
-	getBase64Images = (url: string): string => {
+	getBase64Images = (_url: string): string => {
 		return ''
 	}
 }

package/src/utilities/index.ts

@@ -1 +1 @@
-export {default as AxiosInstance} from './axios'
+export { default as AxiosInstance } from './axios'

package/.eslintignore

@@ -1,2 +0,0 @@
-*.d.ts
-dist

package/.eslintrc.js

@@ -1,28 +0,0 @@
-module.exports = {
-	plugins: ['@typescript-eslint/eslint-plugin', 'eslint-plugin-tsdoc'],
-	extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended'],
-	parser: '@typescript-eslint/parser',
-	parserOptions: {
-		project: './tsconfig.json',
-		tsconfigRootDir: __dirname,
-		ecmaVersion: 2018,
-		sourceType: 'module',
-	},
-	rules: {
-		'tsdoc/syntax': 'warn',
-		'@typescript-eslint/ban-ts-comment': 'off',
-		'@typescript-eslint/ban-types': 'off',
-		'@typescript-eslint/no-namespace': 'off',
-		'@typescript-eslint/no-unused-vars': [
-			'warn',
-			{
-				vars: 'all',
-				args: 'after-used',
-				ignoreRestSiblings: true,
-			},
-		],
-		'@typescript-eslint/no-explicit-any': 'off',
-		'no-async-promise-executor': 'off',
-	},
-	root: true,
-}