refactor(remix-server-runtime): expose SessionStorage's methods as …

…arrow functions (#6330)

.changeset/polite-garlics-invite.md

@@ -0,0 +1,5 @@
+---
+"@remix-run/server-runtime": patch
+---
+
+Expose methods in the `SessionStorage` interface as arrow functions so destructuring is correctly part of the contract.

contributors.yml

@@ -371,6 +371,7 @@
 - nareshbhatia
 - navid-kalaei
 - nexxeln
+- ngbrown
 - ni554n
 - nicholaschiang
 - nicksrandall

packages/remix-server-runtime/sessions.ts

@@ -174,28 +174,28 @@ export interface SessionStorage<Data = SessionData, FlashData = Data> {
    * Session. If there is no session associated with the cookie, this will
    * return a new Session with no data.
    */
-  getSession(
+  getSession: (
     cookieHeader?: string | null,
     options?: CookieParseOptions
-  ): Promise<Session<Data, FlashData>>;
+  ) => Promise<Session<Data, FlashData>>;
 
   /**
    * Stores all data in the Session and returns the Set-Cookie header to be
    * used in the HTTP response.
    */
-  commitSession(
+  commitSession: (
     session: Session<Data, FlashData>,
     options?: CookieSerializeOptions
-  ): Promise<string>;
+  ) => Promise<string>;
 
   /**
    * Deletes all data associated with the Session and returns the Set-Cookie
    * header to be used in the HTTP response.
    */
-  destroySession(
+  destroySession: (
     session: Session<Data, FlashData>,
     options?: CookieSerializeOptions
-  ): Promise<string>;
+  ) => Promise<string>;
 }
 
 /**