Walkthrough

InviteService.inviteUserToRoom now returns a payload including event_id, room_id, and a typed event created via PersistentEventFactory.createFromRawEvent based on the room’s version. Both early-return paths populate the new event field.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Caller
  participant InviteService
  participant PersistentEventFactory as EventFactory
  participant RoomStore as Room Version
  Caller->>InviteService: inviteUserToRoom(userId, roomId, sender, isDirectMessage?)
  InviteService->>RoomStore: getRoomVersion(roomId)
  RoomStore-->>InviteService: roomVersion
  Note over InviteService: Build raw m.room.member invite event
  InviteService->>EventFactory: createFromRawEvent(rawEvent, roomVersion)
  EventFactory-->>InviteService: PersistentEvent
  InviteService-->>Caller: { event_id, room_id, event }

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• refactor: standardize event creation in RoomService and PersistentEve… #188 — Adjusts invite/event construction in the same service, moving to unified createFromRawEvent and returning the typed event.
• Improve types of homeserver.matrix.message #207 — Related updates to PersistentEventFactory usage and persistent event types within invite.service.ts.

Poem

A rabbit taps code with a careful ear,
Invites now return events crystal clear.
Hop, hop—factory spins the thread,
Typed tales of members neatly spread.
In room and event, IDs align—
Carrots for tests; the build’s divine. 🥕

Pre-merge checks and finishing touches

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.86%. Comparing base (9f32b43) to head (9e25d90).
⚠️ Report is 144 commits behind head on main.

@@           Coverage Diff           @@
##             main     #241   +/-   ##
=======================================
  Coverage   81.86%   81.86%           
=======================================
  Files          63       63           
  Lines        4709     4709           
=======================================
  Hits         3855     3855           
  Misses        854      854           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Generating unit tests... This may take up to 20 minutes.