Skip to content

Conversation

@imdhemy
Copy link
Member

@imdhemy imdhemy commented Dec 27, 2025

Q A
License GPLv3
Issue Closes #<issue_number>

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 refactors the event bus middleware to HTTP kernel, consolidating event-related functionality under the Kernel module for better organization and naming clarity.

  • Moves event bus middleware functionality from src/Event/EventBus.ts to src/Kernel/HttpKernel.ts
  • Renames EventBusStorage interface to KernelStorage and eventBusMiddleware function to httpKernel
  • Updates all imports and references to use the new Kernel module location

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/Kernel/types.ts Renames EventBusStorage interface to KernelStorage
src/Kernel/index.ts Adds new barrel export file for the Kernel module
src/Kernel/HttpKernel.ts Implements HTTP kernel with migrated event bus functionality (storage creation, middleware, and utility hooks)
src/Kernel/HttpKernel.test.ts Updates test suite with new import paths and naming conventions
src/Event/index.ts Adds deprecation notice and re-exports httpKernel as eventBusMiddleware for backward compatibility
src/Event/EventBus.ts Removes old EventBus implementation (now in HttpKernel)
src/Config/types.ts Updates EventSubscriber import from @/Event to @/Kernel

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@imdhemy imdhemy merged commit a9b0713 into 2.x Dec 27, 2025
1 check passed
@imdhemy imdhemy deleted the add-central-middleware branch December 27, 2025 23:20
imdhemy added a commit that referenced this pull request Jan 2, 2026
* feat(kernel): replace event bus middleware with http kernel

* chore(docs): add migration guide for upgrading from 2.x to 3.0

* chore(bc): export additional types for event bus middleware

* docs: update upgrade guide to reflect event bus middleware removal and type replacements
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