Refactor recent community PR

[FelonEkonom]

No description provided.

[Copilot]

Pull request overview

This PR refactors the Signaling module API by introducing a new Signaling.new/1 function to properly separate GenServer process spawning from struct wrapping. This addresses an issue where the previous implementation of directly instantiating %Signaling{pid: signaling_pid} could lead to orphaned processes when not properly supervised.

Changes:

• Added Signaling.new/1 function to wrap an already-spawned Signaling GenServer pid in a struct
• Updated all internal usages to call Signaling.new(signaling_pid) instead of direct struct instantiation
• Enhanced documentation for new/0, new/1, and start_link/1 functions
• Bumped version from 0.26.2 to 0.26.3

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
mix.exs	Version bump from 0.26.2 to 0.26.3
README.md	Updated dependency version reference to 0.26.3
lib/membrane_webrtc/signaling.ex	Added new/1 function with documentation, improved docs for existing functions
lib/membrane_webrtc/simple_websocket_server.ex	Replaced direct struct instantiation with Signaling.new/1 call
lib/membrane_webrtc/ex_webrtc/source.ex	Replaced direct struct instantiation with Signaling.new/1 call
lib/membrane_webrtc/ex_webrtc/sink.ex	Replaced direct struct instantiation with Signaling.new/1 call

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.