Follow-up to #6655 (#6664)

- Rename BaseCacheKey and ContextualCacheKey with "Data" on end, since
  the cache key is a string.
- Reintroduce CacheKey(Data) removed in #6655; make the
  GenerateCacheKeyFunction type be defined in terms of it. Now calls to
  the `const generateCacheKey` function are type-safe (ie, the second
  arg is a CacheKeyData rather than unknown).
- Describe generateCacheKey hook as a method rather than a field, like
  the other hooks in Options.

Author: glasser

packages/plugin-response-cache/src/ApolloServerPluginResponseCache.ts

@@ -106,14 +106,12 @@ export interface ApolloServerPluginResponseCacheOptions<
   // and that all relevant data will be found by the kind of iteration performed by
   // `JSON.stringify`, but you should not assume anything about the particular fields on
   // `keyData`.
-  generateCacheKey?: GenerateCacheKeyFunction;
+  generateCacheKey?(
+    requestContext: GraphQLRequestContext<Record<string, any>>,
+    keyData: unknown,
+  ): string;
 }
 
-type GenerateCacheKeyFunction = (
-  requestContext: GraphQLRequestContext<Record<string, any>>,
-  keyData: unknown,
-) => string;
-
 enum SessionMode {
   NoSession,
   Private,
@@ -124,18 +122,28 @@ function sha(s: string) {
   return createHash('sha256').update(s).digest('hex');
 }
 
-interface BaseCacheKey {
+interface BaseCacheKeyData {
   source: string;
   operationName: string | null;
   variables: { [name: string]: any };
   extra: any;
 }
 
-interface ContextualCacheKey {
+interface ContextualCacheKeyData {
   sessionMode: SessionMode;
   sessionId?: string | null;
 }
 
+// We split the CacheKey type into two pieces just for convenience in the code
+// below. Note that we don't actually export this type publicly (the
+// generateCacheKey hook gets an `unknown` argument).
+type CacheKeyData = BaseCacheKeyData & ContextualCacheKeyData;
+
+type GenerateCacheKeyFunction = (
+  requestContext: GraphQLRequestContext<Record<string, any>>,
+  keyData: CacheKeyData,
+) => string;
+
 interface CacheValue {
   // Note: we only store data responses in the cache, not errors.
   //
@@ -174,7 +182,7 @@ export default function plugin<TContext extends BaseContext>(
         options.generateCacheKey ?? ((_, key) => sha(JSON.stringify(key)));
 
       let sessionId: string | null = null;
-      let baseCacheKey: BaseCacheKey | null = null;
+      let baseCacheKey: BaseCacheKeyData | null = null;
       let age: number | null = null;
 
       return {
@@ -188,7 +196,7 @@ export default function plugin<TContext extends BaseContext>(
           }
 
           async function cacheGet(
-            contextualCacheKeyFields: ContextualCacheKey,
+            contextualCacheKeyFields: ContextualCacheKeyData,
           ): Promise<GraphQLResponse | null> {
             const cacheKeyData = {
               ...baseCacheKey!,
@@ -309,7 +317,7 @@ export default function plugin<TContext extends BaseContext>(
           }
 
           const cacheSetInBackground = (
-            contextualCacheKeyFields: ContextualCacheKey,
+            contextualCacheKeyFields: ContextualCacheKeyData,
           ): void => {
             const cacheKeyData = {
               ...baseCacheKey!,