feat(logger): load package.json based on CWD to remove deprecated module code

Author: Steven Weingärtner

packages/logger/src/config.js

@@ -1,6 +1,6 @@
 /* eslint-disable import/no-dynamic-require, global-require, no-param-reassign */
 import fs from 'fs';
-import { join } from 'path';
+import { dirname, join, resolve } from 'path';
 import merge from 'deepmerge';
 
 const defaultConfig = {
@@ -25,9 +25,7 @@ const defaultConfig = {
     plugins: [],
 };
 
-function getPackageConfigInPath(path) {
-    const packageJsonPath = join(path, '..', 'package.json');
-
+function getConfigFromPackage(packageJsonPath) {
     if (!fs.existsSync(packageJsonPath)) {
         return null;
     }
@@ -37,19 +35,38 @@ function getPackageConfigInPath(path) {
     return { channel: pkg.name || defaultConfig.channel, ...pkg.nodeLogger };
 }
 
+function findPackageJson() {
+    let dir = resolve(process.cwd());
+
+    do {
+        const pkgFile = join(dir, 'package.json');
+
+        if (!fs.existsSync(pkgFile) || !fs.statSync(pkgFile).isFile()) {
+            dir = join(dir, '..');
+            continue;
+        }
+
+        return pkgFile;
+    } while (dir !== resolve(dir, '..'));
+
+    return null;
+}
+
 function loadConfigFromPackage() {
-    const { paths } = process.mainModule;
+    const pkgFile = findPackageJson();
+
+    if (!pkgFile) {
+        return { path: null, config: {} };
+    }
 
-    const pkgConfig = paths
-        .map(path => ({ path, config: getPackageConfigInPath(path) }))
-        .reverse()
-        .find(config => !!config.config);
+    const config = getConfigFromPackage(pkgFile);
 
-    if (!pkgConfig) {
+    if (!config) {
         return { path: null, config: {} };
     }
 
-    return pkgConfig;
+    const modulePath = join(dirname(pkgFile), 'node_modules');
+    return { path: modulePath, config };
 }
 
 function loadModule(moduleName, path) {

packages/logger/src/config.test.js

@@ -37,15 +37,11 @@ jest.mock('/some/root/testpackage/package.json', () => ({
     },
 }), { virtual: true });
 
-process.mainModule = {
-    paths: [
-        '/some/root/testpackage/node_modules',
-    ],
-};
-
 describe('loadConfiguration', () => {
     beforeEach(() => {
         fs.existsSync.mockReturnValue(false);
+
+        jest.spyOn(process, 'cwd').mockReturnValue('/some/root/testpackage');
     });
 
     describe('validation', () => {
@@ -147,6 +143,7 @@ describe('loadConfiguration', () => {
     describe('with package.json', () => {
         beforeEach(() => {
             fs.existsSync.mockReturnValue(true);
+            fs.statSync.mockReturnValue({ isFile: jest.fn().mockReturnValue(true) });
         });
 
         it('merges configuration with defaults', () => {