Skip to content

[Nodejs] Update openai, claude and langchain sample Observability code#136

Merged
fpfp100 merged 5 commits intomainfrom
users/pefan/hosting2
Dec 29, 2025
Merged

[Nodejs] Update openai, claude and langchain sample Observability code#136
fpfp100 merged 5 commits intomainfrom
users/pefan/hosting2

Conversation

@fpfp100
Copy link
Contributor

@fpfp100 fpfp100 commented Dec 18, 2025

Update openai, claude and langchain sample Observability setup to use @microsoft/agents-a365-observability-hosting to configure Agent365ExporterOptions, provide custom token resolver example, preloadObservabilityToken in agent code for exporter and use BaggageBuilderUtils.fromTurnContext to populate baggageScope.

…e @microsoft/agents-a365-observability-hosting to configure Agent365ExporterOptions, provide custom token resolver example, preloadObservabilityToken in agent code for exporter and use BaggageBuilderUtils.fromTurnContext to populate baggageScope.
@fpfp100 fpfp100 changed the title Update openai, claude and langchain sample Observability setup Update openai, claude and langchain sample Observability code Dec 19, 2025
juliomenendez
juliomenendez previously approved these changes Dec 22, 2025
@Josina20 Josina20 force-pushed the users/pefan/hosting2 branch from 750b393 to f123564 Compare December 22, 2025 20:02
@fpfp100 fpfp100 marked this pull request as ready for review December 22, 2025 20:54
@fpfp100 fpfp100 requested a review from a team as a code owner December 22, 2025 20:54
Copilot AI review requested due to automatic review settings December 22, 2025 20:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the observability configuration across OpenAI, Claude (Anthropic), and LangChain samples to use the @microsoft/agents-a365-observability-hosting package. The changes introduce custom token resolver patterns, preload observability tokens in agent code for the exporter, and use BaggageBuilderUtils.fromTurnContext to populate baggage scope. Additionally, new Python scripts for contributor leaderboard management are introduced.

Key Changes:

  • Modernized observability setup using token resolvers and baggage builders
  • Added token caching utilities across Python samples
  • New leaderboard management scripts for contributor tracking
  • Updated dependencies and configuration across multiple samples

Reviewed changes

Copilot reviewed 179 out of 208 changed files in this pull request and generated 26 comments.

Show a summary per file
File Description
scripts/update_leaderboard.py New script for generating contributor leaderboard with proper copyright header
scripts/config_points.yml Configuration for contributor point system
scripts/assign_points.py New script for assigning contributor points with comprehensive error handling
python/openai/sample-agent/token_cache.py Token caching utility with correct copyright header
python/openai/sample-agent/agent.py Updated observability setup with token resolver pattern
python/openai/sample-agent/host_agent_server.py Generic host server with baggage builder integration
python/agent-framework/sample-agent/* Similar observability updates for Agent Framework sample
python/google-adk/sample-agent/* Google ADK sample with observability configuration
nodejs/vercel-sdk/sample-agent/* Vercel AI SDK sample with observability manager
nodejs/perplexity/sample-agent/* Perplexity agent with comprehensive telemetry setup

@fpfp100
Copy link
Contributor Author

fpfp100 commented Dec 29, 2025

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

@fpfp100 fpfp100 changed the title Update openai, claude and langchain sample Observability code [Nodejs] Update openai, claude and langchain sample Observability code Dec 29, 2025
@fpfp100 fpfp100 merged commit d1eefb9 into main Dec 29, 2025
12 checks passed
@fpfp100 fpfp100 deleted the users/pefan/hosting2 branch December 29, 2025 21:01
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