Conversation
Co-Authored-By: Claude Opus 4.6 <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Thank you for following the naming conventions! 🙏 |
commit: |
Replace Express-style middleware export with proper NestJS module: - EvlogModule.forRoot() / forRootAsync() dynamic module pattern - NestModule.configure() for middleware registration - Type-only @nestjs/common imports (no runtime dependency) - NestJS example with controllers, decorators, exception filters - Updated all documentation for new API Co-Authored-By: Claude Opus 4.6 <[email protected]>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds first-class NestJS support to the evlog package, aligning it with existing framework integrations (Express/Fastify/Elysia/etc.) and extending the docs/examples/CI metadata to include the new scope.
Changes:
- Added
evlog/nestjsintegration withEvlogModule+useLogger()powered byAsyncLocalStorage. - Added comprehensive NestJS tests, plus a runnable example app and docs pages.
- Updated package exports/build config and repo metadata (changeset, PR scopes, docs landing/framework tabs).
Reviewed changes
Copilot reviewed 22 out of 23 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/evlog/SKILL.md | Updates skill description to include NestJS among supported setups. |
| packages/evlog/tsdown.config.ts | Adds NestJS entrypoint and externalizes Nest deps for bundling. |
| packages/evlog/test/nestjs.test.ts | Adds test suite validating middleware pipeline + useLogger() behavior for NestJS integration. |
| packages/evlog/src/nestjs/index.ts | Implements EvlogModule, NestJS middleware, and useLogger() API. |
| packages/evlog/package.json | Adds ./nestjs export + typesVersions mapping and peer/dev dependency updates. |
| packages/evlog/README.md | Documents NestJS usage and links to the new example. |
| package.json | Adds example:nestjs script for running the example from the monorepo root. |
| examples/nestjs/tsconfig.json | Adds TS config for the NestJS example project. |
| examples/nestjs/src/ui.ts | Adds interactive UI HTML for exercising example routes. |
| examples/nestjs/src/main.ts | Adds Nest bootstrap entrypoint for the example app. |
| examples/nestjs/src/app.module.ts | Wires EvlogModule.forRoot() into the example app (with PostHog drain). |
| examples/nestjs/src/app.controller.ts | Adds example routes + exception filter demonstrating structured errors + logging. |
| examples/nestjs/package.json | Defines the NestJS example package and its dependencies. |
| examples/nestjs/README.md | Adds instructions for running the NestJS example. |
| bun.lock | Locks new NestJS-related dependencies for the repo and example package. |
| apps/docs/content/6.examples/7.nestjs.md | Adds a full NestJS example guide in the docs. |
| apps/docs/content/1.getting-started/2.installation.md | Adds NestJS section to the installation/getting-started docs. |
| apps/docs/content/0.landing.md | Adds NestJS tab snippet to the landing page framework examples. |
| apps/docs/app/components/features/FeatureFrameworks.vue | Adds NestJS to the frameworks tab UI and shifts tab indices accordingly. |
| AGENTS.md | Adds NestJS integration guidance for internal agent/docs patterns. |
| .github/workflows/semantic-pull-request.yml | Adds nestjs to allowed semantic PR scopes. |
| .github/pull_request_template.md | Adds nestjs to the scope list in the PR template. |
| .changeset/nestjs-integration.md | Declares a minor release for adding the NestJS integration. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Make `req.log` optional in module augmentation (undefined when skipped) - Add `express-serve-static-core` augmentation for NestJS Express adapter - Read options lazily at request time to fix forRootAsync timing Co-Authored-By: Claude Opus 4.6 <[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.
Summary
evlog/nestjsintegration with Express-compatible middleware,useLogger()via AsyncLocalStorage, and full pipeline support (drain, enrich, keep)