Skip to content

Comments

docs: Add additional context to Copilot instructions for the OTel-Web Sdk#2704

Merged
MSNev merged 1 commit intootel-sdkfrom
MSNev/OTelSdkInstructions
Feb 20, 2026
Merged

docs: Add additional context to Copilot instructions for the OTel-Web Sdk#2704
MSNev merged 1 commit intootel-sdkfrom
MSNev/OTelSdkInstructions

Conversation

@MSNev
Copy link
Collaborator

@MSNev MSNev commented Feb 19, 2026

No description provided.

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 PR adds comprehensive implementation guidance for the OpenTelemetry Web SDK by introducing a new CONTEXT.md document. This documentation provides complete architectural requirements, interface definitions, implementation patterns, usage examples, and validation criteria for building the OTel Web SDK within the Application Insights JavaScript SDK monorepo.

Changes:

  • Added shared/otel-core/CONTEXT.md (1359 lines) - Complete implementation context for the OTel Web SDK including architecture requirements (no global state, interface-first design, factory patterns), implementation patterns (closure-based, configuration handling), performance requirements, testing requirements, and extensive usage examples
  • Updated .github/copilot-instructions.md to reference the new OTel Web SDK context document

Reviewed changes

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

File Description
shared/otel-core/CONTEXT.md Comprehensive 1359-line implementation guide for the OTel Web SDK covering architecture, patterns, interfaces, examples, and validation criteria
.github/copilot-instructions.md Added reference to the OTel Web SDK context document in the "Related Context Documents" section

@MSNev MSNev merged commit dc7735a into otel-sdk Feb 20, 2026
14 checks passed
@MSNev MSNev deleted the MSNev/OTelSdkInstructions branch February 20, 2026 18:29
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