ac-logger 3.0.5 → 4.0.0

package/.github/workflows/node.js.yml

@@ -1,29 +1,24 @@
-# This workflow will do a clean installation of node dependencies, cache/restore them, build the source code and run tests across different versions of node
-# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
-
 name: Node.js CI
 
 on:
   push:
-    branches: [ master, develop ]
+    branches: [develop, master]
   pull_request:
-    branches: [ master, develop ]
+    branches: [develop, master]
 
 jobs:
   build:
-
+    permissions:
+      contents: read
     runs-on: ubuntu-latest
-
     strategy:
       matrix:
-        node-version: [16.x, 18.x]
-
+        node-version: [22.x, 24.x]
     steps:
-    - uses: actions/checkout@v3
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v3
-      with:
-        node-version: ${{ matrix.node-version }}
-    
-    - run: yarn install
-    - run: yarn run test
+      - uses: actions/checkout@v4
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: yarn install
+      - run: yarn run test

package/CHANGELOG.md

@@ -1,3 +1,29 @@
+# [4.0.0](https://github.com/admiralcloud/ac-logger/compare/v3.0.5..v4.0.0) (2026-04-04 13:14:46)
+
+
+### Bug Fix
+
+
+* **App:** Package updates | MP | [5361eb16cb8b8320b9416785f1e68f9e832ec391](https://github.com/admiralcloud/ac-logger/commit/5361eb16cb8b8320b9416785f1e68f9e832ec391)    
+Package updates  
+Related issues:
+* **Misc:** Requires Node 22+ | MP | [70a30278e37136a924ed67254071b66b55738772](https://github.com/admiralcloud/ac-logger/commit/70a30278e37136a924ed67254071b66b55738772)    
+Requires Node 22+  
+Related issues:
+### Style
+
+
+* **App:** Lint-fix | MP | [b93791fc3dbce7d7f0ed5f27c78711644b8bd88b](https://github.com/admiralcloud/ac-logger/commit/b93791fc3dbce7d7f0ed5f27c78711644b8bd88b)    
+Lint-fix  
+Related issues:
+### Chores
+
+
+* **App:** Updated workflows | MP | [9a71ac0964bfb761cec4d770cd510eda46067e4a](https://github.com/admiralcloud/ac-logger/commit/9a71ac0964bfb761cec4d770cd510eda46067e4a)    
+Updated workflows  
+Related issues:
+## BREAKING CHANGES
+* **Misc:** Requires Node 22+
 
 ## [3.0.5](https://github.com/admiralcloud/ac-logger/compare/v3.0.4..v3.0.5) (2026-04-01 08:28:30)
 

package/README.md

@@ -3,7 +3,7 @@ Unified logging function for all backend applications of AdmiralCloud. Use it in
 
 The purpose of the little app is to make sure that all applications use the same log format/notations.
 
-[![Node.js CI](https://github.com/AdmiralCloud/ac-logger/actions/workflows/node.js.yml/badge.svg)](https://github.com/AdmiralCloud/ac-logger/actions/workflows/node.js.yml)
+[![Node.js CI](https://github.com/AdmiralCloud/ac-logger/actions/workflows/node.js.yml/badge.svg)](https://github.com/AdmiralCloud/ac-logger/actions/workflows/node.js.yml) [![CodeQL](https://github.com/AdmiralCloud/ac-logger/actions/workflows/github-code-scanning/codeql/badge.svg)](https://github.com/AdmiralCloud/ac-logger/actions/workflows/github-code-scanning/codeql)
 
 ## Usage
 Use it like Winston!

package/eslint.config.js

@@ -1,32 +1,34 @@
 const globals = require('globals')
 
-module.exports = {
-  ignores: [
-    'config/env/**'
-  ],
-  languageOptions: {
-    ecmaVersion: 2022,
-    sourceType: 'module',
-    globals: {
-      ...globals.commonjs,
-      ...globals.es2015,
-      ...globals.node,
-      expect: 'readonly',
-      describe: 'readonly',
-      it: 'readonly'
+module.exports = [
+  {
+    files: ['index.js', 'test/**.js'],
+    languageOptions: {
+      ecmaVersion: 2022,
+      sourceType: 'module',
+      globals: {
+        ...globals.commonjs,
+        ...globals.es2015,
+        ...globals.node,
+        expect: 'readonly',
+        describe: 'readonly',
+        it: 'readonly'
+      }
+    },
+    rules: {
+      'no-const-assign': 'error',
+      'space-before-function-paren': 'off',
+      'no-extra-semi': 'off',
+      'object-curly-spacing': ['error', 'always'],
+      'brace-style': ['error', 'stroustrup', { allowSingleLine: true }],
+      'block-spacing': 'error',
+      'no-useless-escape': 'off',
+      'no-console': ['warn', { allow: ['warn', 'error'] }],
+      'no-unused-vars': 'error',
+      'eqeqeq': 'error',
+      'no-var': 'error',
+      'curly': 'error',
+      'prefer-const': ['error', { ignoreReadBeforeAssign: true }]
     }
-  },
-  rules: {
-    'no-const-assign': 'error', // Ensure this rule is enabled
-    'space-before-function-paren': 'off',
-    'no-extra-semi': 'off',
-    'object-curly-spacing': ['error', 'always'],
-    'brace-style': ['error', 'stroustrup', { allowSingleLine: true }],
-    'no-useless-escape': 'off',
-    'standard/no-callback-literal': 'off',
-    'new-cap': 'off',
-    'no-console': ['warn', { allow: ['warn', 'error'] }],
-    "no-unused-vars": "error", // we shouldn't clutter code with unused variables
-    "prefer-const": ["warn", { "ignoreReadBeforeAssign": true }],
   }
-}
+]

package/index.js

@@ -28,7 +28,7 @@ module.exports = ({ prefixFields = [], timestampFormat = 'YYYY-MM-DD HH:mm:ss',
       // display accessKey, link (if applicable)
       const message = data?.message ? ` ${data.message}` : ''
       let cuid = data?.customerId ? data?.customerId : ''
-      if (data?.userId) cuid += `/${data.userId} `
+      if (data?.userId) { cuid += `/${data.userId} ` }
       // colorize status codes
 
       const statusCode = data?.statusCode
@@ -56,9 +56,9 @@ module.exports = ({ prefixFields = [], timestampFormat = 'YYYY-MM-DD HH:mm:ss',
       
       // modern approach (fileName, functionName, sub) // TBD with team
       const functionIdentifier =  _.get(data, 'functionIdentifier') ? _.get(data, 'functionIdentifier') + ' | ' : ''
-      if (!fileName && _.get(data, 'fileName')) fileName =  _.get(data, 'fileName')
-      if (!functionName && _.get(data, 'functionName')) functionName =  _.get(data, 'functionName')
-      if (!subName && _.get(data, 'sub')) subName =  _.get(data, 'sub')
+      if (!fileName && _.get(data, 'fileName')) { fileName =  _.get(data, 'fileName') }
+      if (!functionName && _.get(data, 'functionName')) { functionName =  _.get(data, 'functionName') }
+      if (!subName && _.get(data, 'sub')) { subName =  _.get(data, 'sub') }
 
       
       let message = data?.message
@@ -76,7 +76,7 @@ module.exports = ({ prefixFields = [], timestampFormat = 'YYYY-MM-DD HH:mm:ss',
       if (data?.e instanceof Error) {
         // log the stack
         console.error(data?.e)
-        if (data?.e?.message) message += ' | ' + ( data?.e?.message  || '')
+        if (data?.e?.message) { message += ' | ' + ( data?.e?.message  || '') }
       }
       // TODO: for better readability, we shold use padding for fileName, functionname, etc
       return `${data?.timestamp} ${data?.level} ${fileName}${functionName}${functionIdentifier}${subName}${prefixData}${message}`
@@ -149,7 +149,7 @@ module.exports = ({ prefixFields = [], timestampFormat = 'YYYY-MM-DD HH:mm:ss',
   else {
     _.forEach(transporters, item => {
       const logTransport = new transports[item.type](item.options)
-      if (_.has(item, 'onRotate')) logTransport.on('rotate', item.onRotate)
+      if (_.has(item, 'onRotate')) { logTransport.on('rotate', item.onRotate) }
       logTransports.push(logTransport)
     })
   }
@@ -158,10 +158,10 @@ module.exports = ({ prefixFields = [], timestampFormat = 'YYYY-MM-DD HH:mm:ss',
     level,
     transports: logTransports
   }
-  if (_.get(customLevels, 'levels')) _.set(logConfig, 'levels', _.get(customLevels, 'levels'))
+  if (_.get(customLevels, 'levels')) { _.set(logConfig, 'levels', _.get(customLevels, 'levels')) }
 
   const acLogger = createLogger(logConfig)
-  if (_.get(customLevels, 'colors')) addColors(_.get(customLevels, 'colors'))
+  if (_.get(customLevels, 'colors')) { addColors(_.get(customLevels, 'colors')) }
   
     const changeLogLevel = (newLevel) => {
       acLogger.transports.forEach((transport) => {

package/package.json

@@ -3,17 +3,18 @@
   "author": "Mark Poepping (https://www.admiralcloud.com)",
   "license": "MIT",
   "repository": "admiralcloud/ac-logger",
-  "version": "3.0.5",
+  "version": "4.0.0",
   "dependencies": {
-    "lodash": "^4.17.23",
+    "lodash": "^4.18.1",
     "moment": "^2.30.1",
     "winston": "^3.19.0",
     "winston-daily-rotate-file": "^5.0.0"
   },
   "devDependencies": {
-    "ac-semantic-release": "^0.4.10",
+    "ac-semantic-release": "^1.0.1",
     "chai": "^4.5.0",
-    "eslint": "^9.39.2",
+    "eslint": "^10.2.0",
+    "globals": "^17.4.0",
     "intercept-stdout": "^0.1.2",
     "mocha": "^11.7.5"
   },
@@ -21,7 +22,7 @@
     "test": "mocha --reporter spec"
   },
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=22.0.0"
   },
   "resolutions": {
     "mocha/chokidar/braces": "^3.0.3",

package/test/test.js

@@ -1,11 +1,11 @@
 const { expect } = require('chai')
 const aclog = require('../index')
 
-var intercept = require("intercept-stdout");
+const intercept = require("intercept-stdout");
 
-var captured_text = "";
+let captured_text = "";
 
-var unhook_intercept = intercept(function(text) {
+const unhook_intercept = intercept(function(text) {
   captured_text += text;
 });