Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Thank you for following the naming conventions! 🙏 |
commit: |
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request adds logging adapters for Axiom and OTLP (OpenTelemetry Protocol) to the evlog package, enabling integration with external observability platforms. The adapters support zero-configuration setup via environment variables and include comprehensive test coverage.
Changes:
- Added
axiomandotlpadapters with factory functions (createAxiomDrain,createOTLPDrain) for sending logs to external services - Updated Nuxt module configuration to include adapter options
- Added comprehensive documentation for both adapters and custom adapter development
- Updated build configuration and package exports to support the new adapters
Reviewed changes
Copilot reviewed 16 out of 18 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/evlog/src/adapters/axiom.ts | Implements Axiom adapter with configuration priority and batch sending support |
| packages/evlog/src/adapters/otlp.ts | Implements OTLP adapter with OpenTelemetry log format conversion and resource attributes |
| packages/evlog/test/adapters/axiom.test.ts | Comprehensive test suite for Axiom adapter functionality |
| packages/evlog/test/adapters/otlp.test.ts | Comprehensive test suite for OTLP adapter including format conversion |
| packages/evlog/src/nuxt/module.ts | Adds TypeScript interface definitions for axiom and otlp configuration options |
| packages/evlog/build.config.ts | Adds build entries for the new adapter modules |
| packages/evlog/package.json | Exports axiom and otlp adapters as separate entry points |
| turbo.json | Adds AXIOM_TOKEN to environment variables for build |
| apps/playground/server/plugins/evlog-drain.ts | Updates example to demonstrate new Axiom adapter usage |
| apps/playground/nuxt.config.ts | Removes outdated comment about head sampling |
| apps/docs/content/3.adapters/*.md | Comprehensive documentation for adapters overview, Axiom, OTLP, and custom adapters |
| apps/docs/app/components/LandingFeatures.vue | Reorders and updates landing page features to highlight new adapter capabilities |
| apps/docs/app/assets/icons/*.svg | Adds icon assets for adapter documentation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces new adapters for sending logs to external services (Axiom and OTLP) in the
evlogpackage, adds their configuration to the Nuxt module, and updates the build and package exports to support these features.The main goal is to make it easier to forward logs from your Nuxt/Nitro app to observability platforms like Axiom and OpenTelemetry-compatible backends.