Allow setting PerRequestSpanProcessor without directly exporting the …

[fpfp100]

Allow setting PerRequestSpanProcessor without directly exporting the configuration methods at index level.

[Copilot]

Pull request overview

Adds an internal override mechanism to enable per-request span exporting and to override PerRequestSpanProcessor guardrail settings without exposing additional configuration APIs at the package index level.

Changes:

• Introduces PerRequestProcessorOverrides internal module with get/set APIs for per-request export and processor guardrail overrides.
• Updates isPerRequestExportEnabled() to honor internal overrides before consulting configuration.
• Updates PerRequestSpanProcessor to apply internal override guardrails and adds tests covering override behavior and builder selection.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
tests/observability/internal/PerRequestProcessorOverrides.test.ts	Adds Jest coverage for override round-tripping, enablement precedence, and guardrail override application.
packages/agents-a365-observability/src/tracing/exporter/utils.ts	Checks internal overrides in isPerRequestExportEnabled() before config provider/env-driven configuration.
packages/agents-a365-observability/src/tracing/PerRequestSpanProcessor.ts	Applies internal override guardrail values when constructing the processor.
packages/agents-a365-observability/src/internal/PerRequestProcessorOverrides.ts	New internal module holding override state and exported override types.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

packages/agents-a365-observability/src/configuration/PerRequestSpanProcessorConfiguration.ts:33

• The class-level documentation and the "Inherited" comment still state that PerRequestSpanProcessorConfiguration inherits observability settings (e.g., isObservabilityExporterEnabled, observabilityLogLevel). After changing the base class to RuntimeConfiguration, those properties are no longer part of this configuration, so the comments are now misleading and should be updated accordingly.

 * Configuration for PerRequestSpanProcessor.
 * Inherits all observability and runtime settings, and adds per-request processor guardrails.
 *
 * This is separated from ObservabilityConfiguration because PerRequestSpanProcessor
 * is used only in specific scenarios and these settings should not be exposed
 * in the common ObservabilityConfiguration.
 */
export class PerRequestSpanProcessorConfiguration extends RuntimeConfiguration {
  protected get perRequestOverrides(): PerRequestSpanProcessorConfigurationOptions {
    return getPerRequestProcessorInternalOverrides() ?? this.overrides as PerRequestSpanProcessorConfigurationOptions;
  }

  constructor(overrides?: PerRequestSpanProcessorConfigurationOptions) {
    super(overrides);
  }

  // Inherited: clusterCategory, isDevelopmentEnvironment, isNodeEnvDevelopment,
  // observabilityAuthenticationScopes, isObservabilityExporterEnabled,
  // useCustomDomainForObservability, observabilityDomainOverride, observabilityLogLevel

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

[fpfp100]

@copilot open a new pull request to apply changes based on the comments in this thread

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.