Skip to content

Fix broadcast wrapper in communication controller#76

Merged
csecrestjr merged 1 commit into
mainfrom
codex/refactor-communication_controller.js-logic
Sep 1, 2025
Merged

Fix broadcast wrapper in communication controller#76
csecrestjr merged 1 commit into
mainfrom
codex/refactor-communication_controller.js-logic

Conversation

@csecrestjr

Copy link
Copy Markdown
Collaborator

Summary

  • Properly close global.broadcast block in sendMessage
  • Emit final message and HTTP response regardless of broadcast availability

Testing

  • node --test __tests__/messages.test.js __tests__/conversation.test.js __tests__/authMiddleware.test.js

https://chatgpt.com/codex/tasks/task_e_68b4ea36f2448321a19624ba81a5f8e1

@csecrestjr
csecrestjr merged commit 71414ac into main Sep 1, 2025
1 check passed
@csecrestjr
csecrestjr deleted the codex/refactor-communication_controller.js-logic branch November 11, 2025 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant