refactor(collector): refcatored config module

[aryamohanan]

#2448 this needs to merge first

Normalize the collector config. No logic changes only refactored the code.

[kirrg001]

/review

[github-actions]

🤖 I've received your review request, and I'm working on it now! You can track my progress in the logs for more details.

[github-actions]

Cost: $0.07

[github-actions]

🟠 High: The init function uses config.logger without validation. If config is null, undefined, or doesn't have a logger property, this will throw a TypeError. Add defensive checks: if (!config || !config.logger) before attempting to access config.logger.

Prompt for Bob

Your task is to check if the following issue is valid. If so, understand the root cause and fix it.

<issue>
🟠 High: The `init` function uses `config.logger` without validation. If `config` is null, undefined, or doesn't have a `logger` property, this will throw a TypeError. Add defensive checks: `if (!config || !config.logger)` before attempting to access `config.logger`.
</issue>

You can find the issue in this file:
<path>
packages/collector/src/util/normalizeConfig.js
</path>

Around this line:
<line>
17
</line>

[kirrg001]

🤔

This looks wrong to me.
The collector initializes the core already early (packages/collector/src/index.js).
This call belongs to

packages/core/src/config/index.js
module.exports.init = _logger => {
logger = _logger;
configNormalizers.init({ logger });
};

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
96.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud