Remove unused properties of SessionStrategy (#4835)

.changeset/eleven-waves-hope.md

@@ -0,0 +1,6 @@
+---
+'@keystone-next/keystone': patch
+'@keystone-next/types': patch
+---
+
+Removed the unused `connect` and `disconnect` properties of `SessionStrategy`.

packages-next/keystone/src/session/index.ts

@@ -178,8 +178,6 @@ export function storedSessions({
     let store = typeof storeOption === 'function' ? storeOption({ maxAge }) : storeOption;
     let isConnected = false;
     return {
-      connect: store.connect,
-      disconnect: store.disconnect,
       async get({ req, createContext }) {
         let sessionId = await get({ req, createContext });
         if (typeof sessionId === 'string') {

packages-next/types/src/session.ts

@@ -3,9 +3,6 @@ import type { ServerResponse, IncomingMessage } from 'http';
 import { CreateContext } from '.';
 
 export type SessionStrategy<StoredSessionData, StartSessionData = never> = {
-  connect?: () => Promise<void>;
-  disconnect?: () => Promise<void>;
-  // -- these two are invoked from mutations
   // creates token from data, sets the cookie with token via res, returns token
   start: (args: {
     res: ServerResponse;
@@ -30,15 +27,10 @@ export type SessionStore = {
   connect?: () => Promise<void>;
   disconnect?: () => Promise<void>;
   get(key: string): undefined | JSONValue | Promise<JSONValue | undefined>;
-  set(
-    key: string,
-    value: JSONValue
-  ): // 😞 using any here rather than void to be compatible with Map. note that `| Promise<void>` doesn't actually do anything type wise because it just turns into any, it's just to show intent here
-  any | Promise<void>;
-  delete(
-    key: string
-  ): // 😞 | boolean is for compatibility with Map
-  void | boolean | Promise<void>;
+  // � using any here rather than void to be compatible with Map. note that `| Promise<void>` doesn't actually do anything type wise because it just turns into any, it's just to show intent here
+  set(key: string, value: JSONValue): any | Promise<void>;
+  // � | boolean is for compatibility with Map
+  delete(key: string): void | boolean | Promise<void>;
 };
 
 export type SessionStoreFunction = (args: {