-
Notifications
You must be signed in to change notification settings - Fork 5
Description
SDK Parity Update Required
This issue was automatically created by the AI-First Polling workflow to maintain SDK parity.
Source PR Information
Original PR: microsoft/Agent365-dotnet#133 - [Observability] Scope and domain overrides
PR URL: microsoft/Agent365-dotnet#133
PR Author: @threddy
PR Description
Why? To enable 1P testing in pre-prodWhat? Enable overrides for scope and domain via environment variables "A365_OBSERVABILITY_DOMAIN_OVERRIDE" and "A365_OBSERVABILITY_SCOPE_OVERRIDE".------This PR introduces configuration-based overrides for the observability domain and authentication scope to enable testing against pre-production environments. The changes add two new configuration keys (A365_OBSERVABILITY_DOMAIN_OVERRIDE and A365_OBSERVABILITY_SCOPES_OVERRIDE) that allow runtime customization of Power Platform API endpoints and authentication scopes without code changes.Key ChangesAdded EnvironmentUtils.Initialize() method to cache scope override from configurationExtended PowerPlatformApiDiscovery and Agent365ExporterCore to accept optional domain overridesCreated comprehensive test coverage for both override mechanisms
Parity Task
Changes detected in: .NET/C# SDK
Target SDK: Node.js/TypeScript
Action Required
Please apply the changes described in the original PR (microsoft/Agent365-dotnet#133) to the Node.js/TypeScript SDK to maintain feature parity across all SDKs.
Before implementing:
- First, check if this feature already exists in the Node.js/TypeScript SDK
- If the feature is already implemented, close this issue with the
wontfixlabel and add a comment explaining that parity already exists
For bug fixes (IMPORTANT):
- Bug fixes require extra caution - only act if the same bug exists in the Node.js/TypeScript SDK
- Carefully analyze whether the bug described in the original PR replicates in the target platform
- If the bug does NOT exist in the Node.js/TypeScript SDK, close this issue with the
wontfixlabel and explain that the bug is not present - Stay strictly in context - only fix the specific bug described, do not make additional changes or "improvements"
- Do not port bug fixes that are specific to the source SDK's language or platform constraints
When creating the PR for this implementation:
- Assign the PR to @threddy (the author of the original PR)
- Reference this issue in the PR description
- Ensure the PR description explains what parity changes were made
Implementation Guidelines for Node.js/TypeScript
When implementing this feature in the TypeScript SDK:
- Review existing code in
nodejs/packages/to understand the codebase structure and patterns - Follow the coding style, naming conventions, and architectural patterns used in the TypeScript SDK
- Check for configuration files (e.g.,
.eslintrc.json,tsconfig.json,.prettierrc) that define coding standards - Ensure your implementation is consistent with existing SDK components
- Include appropriate tests following the testing patterns in the TypeScript SDK
Related Information
- Workflow Run: https://github.com/microsoft/AI-First-Development/actions/runs/20828582737
- Trigger: Polling workflow
- Detected: 2026-01-08 19:10:01 UTC
This is an automated issue. Please close with wontfix label if parity is not needed.