MM-68540: Drop redundant user_id from refresh WS payload

[nang2049]

Summary

Followup to #1002. The WS event is purely a doorbell now and the recipient is already targeted by WebsocketBroadcast.UserId and the webapp handler doesn't read msg.data. Sending user_id was both unused
and unnecessary.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 42a8aef1-fdc3-49f0-97b4-164845c906eb

📥 Commits

Reviewing files that changed from the base of the PR and between fb313de and 4a2412a.

📒 Files selected for processing (1)

• server/plugin/plugin.go

---

📝 Walkthrough

Walkthrough

The PublishWebSocketEvent call within sendRefreshEvent is modified to publish a nil payload instead of a map containing user_id. This is a single-line payload alteration in the WebSocket event publishing logic.

Changes

WebSocket Event Payload Simplification

Layer / File(s)	Summary
Event Publishing server/plugin/plugin.go	sendRefreshEvent now publishes the refresh event with a nil payload instead of a map containing user_id.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A payload once carried a user's delight,
Now nil flows swift through the WebSocket's night,
One line refined, the event still shines bright,
Simpler it goes, and that feels so right!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main change: removing a redundant user_id field from a WebSocket event payload, which aligns perfectly with the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch MM-68540-followup-empty-refresh-payload

---

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