Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Thank you for following the naming conventions! 🙏 |
commit: |
…pter-sentry-logging
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new Sentry adapter to evlog so wide events can be sent as Sentry Structured Logs (including Nuxt/Nitro “zero-config” support), and updates docs/exports/tests accordingly.
Changes:
- Introduces
packages/evlog/src/adapters/sentry.tswithcreateSentryDrain, event→Sentry log transformation, and envelope sending (single + batch). - Adds Vitest coverage for transformation, envelope formatting, batching, and timeout behavior.
- Wires up package exports/build entries and updates docs (monorepo + package README + docs site) to document the Sentry adapter.
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/evlog/src/adapters/sentry.ts | Implements Sentry transformation + envelope transport + Nuxt/Nitro drain factory. |
| packages/evlog/test/adapters/sentry.test.ts | Adds tests for Sentry transformation and transport behavior. |
| packages/evlog/src/nuxt/module.ts | Extends Nuxt module option types to include posthog and sentry config. |
| packages/evlog/package.json | Exposes evlog/sentry subpath export and typesVersions entry. |
| packages/evlog/build.config.ts | Adds build entry to emit dist/adapters/sentry.*. |
| packages/evlog/README.md | Documents using createSentryDrain and NUXT_SENTRY_DSN. |
| README.md | Mirrors Sentry adapter usage in root README. |
| AGENTS.md | Updates adapter matrix and usage snippet to include Sentry. |
| apps/playground/server/plugins/evlog-drain.ts | Adds commented playground example for createSentryDrain. |
| apps/playground/nuxt.config.ts | Ensures drain test route logs are always kept for sampling. |
| apps/docs/content/3.adapters/1.overview.md | Adds Sentry to docs adapter overview page and env var examples. |
| apps/docs/content/3.adapters/5.sentry.md | New docs page describing Sentry adapter setup/config/querying. |
| apps/docs/content/3.adapters/6.custom.md | New docs page describing how to build custom adapters. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
HugoRCD
deleted the
codex/linear-mention-evl-49-feature]-add-adapter-sentry-logging
branch
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
1 participant
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.
Resolves #50
This pull request adds Sentry integration to the
evloglogging package, enabling logs to be sent to Sentry with flexible configuration and support for Nuxt/Nitro runtime environments. The changes include a new Sentry adapter, documentation updates, configuration options, package exports, and comprehensive tests.