Skip to content

feat(chat): add DeleteMessage gRPC RPC - #1621

Merged
kkopanidis merged 1 commit into
mainfrom
feat/chat-delete-message-rpc
Sep 25, 2026
Merged

kkopanidis merged 1 commit into
mainfrom
feat/chat-delete-message-rpc

Conversation

@JohnChantz

Copy link
Copy Markdown
Contributor

Summary

  • Adds Chat.DeleteMessage, matching EditMessage, so other modules can delete a message and still emit message-deleted on the chat namespace.
  • Moves the client delete path into deleteChatMessage: sender check, allowMessageDelete, auditMode hard-delete vs deleted: true, socket push, and bus publish.

Test plan

  • modules/chat unit tests for hard-delete, audit-mode soft-delete, missing message, non-sender, and allowMessageDelete: false
  • Publish @conduitplatform/grpc-sdk and @conduitplatform/module-tools 0.16.31 from the v0.16.x backport before Viz main-service depends on deleteMessage

Share delete with a helper so callers can remove a message and still broadcast message-deleted.

Copy link
Copy Markdown
Contributor

@JohnChantz CI check failed on this head (8e0b6f02): security/snyk (kkopanidis) — error, “1 test has failed”.

https://app.snyk.io/org/kkopanidis/pr-checks/1f46beda-fe28-4a24-881c-32f4bdf310f5

@kkopanidis
kkopanidis merged commit 368a96b into main Sep 25, 2026
19 of 20 checks passed
@kkopanidis
kkopanidis deleted the feat/chat-delete-message-rpc branch September 25, 2026 15:10
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.

3 participants