Skip to content

refactor(config): replaced agentConfig usage with updated config#2507

Merged
aryamohanan merged 3 commits intofix-config-precedencefrom
fix-config-remove-agent
Apr 21, 2026
Merged

refactor(config): replaced agentConfig usage with updated config#2507
aryamohanan merged 3 commits intofix-config-precedencefrom
fix-config-remove-agent

Conversation

@aryamohanan
Copy link
Copy Markdown
Contributor

@aryamohanan aryamohanan commented Apr 21, 2026

In our codebase, agentConfig is used in several places such as spanBuffer and tracingUtil. This change replaces those usages.

We will also need to update the instrumentations, which will be addressed in a separate PR

@aryamohanan aryamohanan requested a review from a team as a code owner April 21, 2026 06:29
@aryamohanan aryamohanan changed the title chore: replaced external config with current config in activate chore: replaced agent config with current config in activate Apr 21, 2026
@aryamohanan aryamohanan changed the title chore: replaced agent config with current config in activate Refactor(config): replaced agentConfig usage with centralized update logic Apr 21, 2026
@aryamohanan aryamohanan changed the title Refactor(config): replaced agentConfig usage with centralized update logic Refactor(config): replaced agentConfig usage with updated config Apr 21, 2026
@aryamohanan aryamohanan changed the title Refactor(config): replaced agentConfig usage with updated config refactor(config): replaced agentConfig usage with updated config Apr 21, 2026
stackTraceLength = agentTraceConfig.stackTraceLength;
}
exports.activate = function activate(_config) {
stackTraceLength = _config.tracing?.stackTraceLength;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm I see different approaches in active method.

  if (_config.tracing.spanBatchingEnabled) {
    batchingEnabled = true;
stackTraceLength = _config.tracing?.stackTraceLength;
```

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

@aryamohanan aryamohanan merged commit 9f52cec into fix-config-precedence Apr 21, 2026
1 check was pending
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants