Skip to content

Conversation

@jtomaszewski
Copy link

Summary

  • Add optional immediateProcessing boolean to InboxOutboxModuleEventOptions (default: true for backward compatibility)
  • When false, events are only saved to DB and processed later by the poller, enabling a safer "fire and forget" pattern for crash recovery scenarios
  • Update README with documentation and trade-offs table

Test plan

  • Added unit tests for immediateProcessing: false with both emit and emitAsync
  • Added unit tests for immediateProcessing: true (explicit)
  • Added unit tests for immediateProcessing: undefined (default behavior)
  • All existing tests pass

🤖 Generated with Claude Code

@jtomaszewski jtomaszewski force-pushed the vk/cb52-make-immediate-p branch from 23ef214 to 601c725 Compare December 3, 2025 01:12
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 3, 2025

Open in StackBlitz

npm i https://pkg.pr.new/fullstackhouse/nestjs-inbox-outbox/@nestixis/nestjs-inbox-outbox@5
npm i https://pkg.pr.new/fullstackhouse/nestjs-inbox-outbox/@nestixis/nestjs-inbox-outbox-mikroorm-driver@5
npm i https://pkg.pr.new/fullstackhouse/nestjs-inbox-outbox/@nestixis/nestjs-inbox-outbox-typeorm-driver@5

commit: 660e05e

jtomaszewski and others added 3 commits December 3, 2025 02:16
Add optional `immediateProcessing` boolean to InboxOutboxModuleEventOptions
(default: true for backward compatibility). When set to false, events are
only saved to DB and processed later by the poller, enabling a safer
"fire and forget" pattern for crash recovery scenarios.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
verify that it is not processed immediately, but eventually once poller runs
@jtomaszewski jtomaszewski force-pushed the vk/cb52-make-immediate-p branch from 7c99c58 to 660e05e Compare December 3, 2025 01:18
@jtomaszewski jtomaszewski merged commit 11f0016 into main Dec 3, 2025
4 checks passed
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