Skip to content

[JS]Removes usage of non-prod regions in observability. - #65

Merged
PengF (fpfp100) merged 4 commits into
mainfrom
users/pefan/tokenscope
Nov 15, 2025
Merged

[JS]Removes usage of non-prod regions in observability.#65
PengF (fpfp100) merged 4 commits into
mainfrom
users/pefan/tokenscope

Conversation

@fpfp100

@fpfp100 PengF (fpfp100) commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Removes usage of non-prod regions in observability. The Basic export sample has passed with the changes.

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.

lets keep the cluster category. And default the envutils and the
PowerPlatformApiDiscovery to hoor the cluster category and return prod endpoint for dev,test and preprod urls

@fpfp100

Copy link
Copy Markdown
Contributor Author

lets keep the cluster category. And default the envutils and the PowerPlatformApiDiscovery to hoor the cluster category and return prod endpoint for dev,test and preprod urls

updated.

@nikhilNava
Nikhil Navakiran (nikhilNava) marked this pull request as ready for review November 15, 2025 01:05
Copilot AI review requested due to automatic review settings November 15, 2025 01:05

Copilot AI left a comment

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.

Pull Request Overview

This PR removes usage of non-production regions in observability by consolidating dev, test, and preprod environments to use the production domain (api.powerplatform.com) instead of their previously dedicated subdomains.

  • Removes non-production observability scopes (TEST and PREPROD)
  • Updates API host name suffixes for dev/test/preprod to use production domain
  • Simplifies getObservabilityAuthenticationScope() to always return production scope
  • Updates all related test expectations to reflect the new domain mapping

Reviewed Changes

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

File Description
packages/agents-a365-runtime/src/environment-utils.ts Removed TEST and PREPROD observability scopes and simplified authentication scope function to always return production
packages/agents-a365-runtime/src/power-platform-api-discovery.ts Updated dev, test, and preprod cluster categories to use production API domain instead of environment-specific subdomains
packages/agents-a365-observability/src/tracing/exporter/Agent365Exporter.ts Updated comment to clarify endpoint resolution defaults to production
tests/common/power-platform-api-discovery.test.ts Updated test expectations for dev, test, and preprod to expect production domain across all API discovery methods

Comment thread packages/agents-a365-runtime/src/power-platform-api-discovery.ts
Comment thread packages/agents-a365-runtime/src/power-platform-api-discovery.ts
Comment thread packages/agents-a365-runtime/src/power-platform-api-discovery.ts
Comment thread packages/agents-a365-runtime/src/environment-utils.ts
@fpfp100
PengF (fpfp100) merged commit e372683 into main Nov 15, 2025
7 checks passed
@pontemonti
Johan Broberg (pontemonti) deleted the users/pefan/tokenscope branch December 5, 2025 17:09
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.

4 participants