Unify getKeyForCacheEntry functions (#25403)

Splitting up changes from #25337. Have only one implementation of
getKeyForCacheEntry across odsp-driver and driver-web-cache.

Author: jzaffiro

packages/drivers/driver-web-cache/src/FluidCacheIndexedDb.ts

@@ -4,7 +4,10 @@
  */
 
 import { ITelemetryBaseLogger } from "@fluidframework/core-interfaces";
-import { ICacheEntry } from "@fluidframework/odsp-driver-definitions/internal";
+import {
+	ICacheEntry,
+	getKeyForCacheEntry as odspGetKeyForCacheEntry,
+} from "@fluidframework/odsp-driver-definitions/internal";
 import { createChildLogger } from "@fluidframework/telemetry-utils/internal";
 import { DBSchema, DeleteDBCallbacks, IDBPDatabase, deleteDB, openDB } from "idb";
 
@@ -25,7 +28,7 @@ export const oldVersionNameMapping: Partial<{ [key: number]: string }> = {
 };
 
 export function getKeyForCacheEntry(entry: ICacheEntry) {
-	return `${entry.file.docId}_${entry.type}_${entry.key}`;
+	return odspGetKeyForCacheEntry(entry);
 }
 
 export function getFluidCacheIndexedDbInstance(

packages/drivers/odsp-driver-definitions/api-report/odsp-driver-definitions.legacy.beta.api.md

@@ -56,6 +56,7 @@ export interface IEntry {
 // @beta @legacy (undocumented)
 export interface IFileEntry {
     docId: string;
+    fileVersion?: string;
     resolvedUrl: IResolvedUrl;
 }
 

packages/drivers/odsp-driver-definitions/src/odspCache.ts

@@ -53,6 +53,10 @@ export interface IFileEntry {
 	 * This is IOdspResolvedUrl in case of ODSP driver.
 	 */
 	resolvedUrl: IResolvedUrl;
+	/**
+	 * Optional version of the file.
+	 */
+	fileVersion?: string;
 }
 
 /**