Include :noreply return types in custom callbacks

Connects #71

CHANGELOG.md

@@ -6,6 +6,23 @@ The format is based on [Keep a
 Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## UNRELEASED
+
+### Fixed
+
+- Fixed callback return types to include `{:noreply, new_socket}` and
+  `{:noreply, new_socket, _rest}` (for returning timeouts or triggering
+  `handle_continue/2` for example). These return types were always
+  supported but not declared in the callback, so the dialyzer would
+  complain if these values were returned.
+    - `c:Slipstream.handle_connect/1`
+    - `c:Slipstream.handle_disconnect/2`
+    - `c:Slipstream.handle_join/3`
+    - `c:Slipstream.handle_message/4`
+    - `c:Slipstream.handle_reply/3`
+    - `c:Slipstream.handle_topic_close/3`
+    - `c:Slipstream.handle_leave/2`
+
 ## 1.1.2 - 2024-09-26
 
 ### Added

lib/slipstream.ex

@@ -526,12 +526,15 @@ defmodule Slipstream do
 
       @impl Slipstream
       def handle_connect(socket) do
-        {:noreply, socket}
+        {:ok, socket}
       end
   """
   @doc since: "0.1.0"
   @callback handle_connect(socket :: Socket.t()) ::
               {:ok, new_socket}
+              | {:noreply, new_socket}
+              | {:noreply, new_socket,
+                 timeout() | :hibernate | {:continue, term()}}
               | {:stop, reason :: term(), new_socket}
             when new_socket: Socket.t()
 
@@ -553,6 +556,9 @@ defmodule Slipstream do
   @doc since: "0.1.0"
   @callback handle_disconnect(reason :: term(), socket :: Socket.t()) ::
               {:ok, new_socket}
+              | {:noreply, new_socket}
+              | {:noreply, new_socket,
+                 timeout() | :hibernate | {:continue, term()}}
               | {:stop, stop_reason :: term(), new_socket}
             when new_socket: Socket.t()
 
@@ -575,6 +581,9 @@ defmodule Slipstream do
               socket :: Socket.t()
             ) ::
               {:ok, new_socket}
+              | {:noreply, new_socket}
+              | {:noreply, new_socket,
+                 timeout() | :hibernate | {:continue, term()}}
               | {:stop, reason :: term(), new_socket}
             when new_socket: Socket.t()
 
@@ -605,6 +614,9 @@ defmodule Slipstream do
               socket :: Socket.t()
             ) ::
               {:ok, new_socket}
+              | {:noreply, new_socket}
+              | {:noreply, new_socket,
+                 timeout() | :hibernate | {:continue, term()}}
               | {:stop, reason :: term(), new_socket}
             when new_socket: Socket.t()
 
@@ -638,6 +650,9 @@ defmodule Slipstream do
               socket :: Socket.t()
             ) ::
               {:ok, new_socket}
+              | {:noreply, new_socket}
+              | {:noreply, new_socket,
+                 timeout() | :hibernate | {:continue, term()}}
               | {:stop, reason :: term(), new_socket}
             when new_socket: Socket.t()
 
@@ -665,6 +680,9 @@ defmodule Slipstream do
               socket :: Socket.t()
             ) ::
               {:ok, new_socket}
+              | {:noreply, new_socket}
+              | {:noreply, new_socket,
+                 timeout() | :hibernate | {:continue, term()}}
               | {:stop, stop_reason :: term(), new_socket}
             when new_socket: Socket.t()
 
@@ -691,6 +709,9 @@ defmodule Slipstream do
               socket :: Socket.t()
             ) ::
               {:ok, new_socket}
+              | {:noreply, new_socket}
+              | {:noreply, new_socket,
+                 timeout() | :hibernate | {:continue, term()}}
               | {:stop, stop_reason :: term(), new_socket}
             when new_socket: Socket.t()