Replace get<Raw>(...) with an overload (#805)

One overload with `options?: { raw?: false }` and one with `options: { raw: true }` allows being specific about the return type for each.

So, the behavior is unchanged but it becomes easier for external libraries to write an interface with a correct *partial* implementation.

My use-case: I'm working on a library that expects a keyv-like object passed in. The user is allowed to pass in a different implementation, as long as the basic `get`, `set` and `delete` exist. But they don't need to provide an implementation for the `{ raw: true }` variant. So, I'm trying to write an interface that corresponds to that, which `new Keyv<string>(...)` conforms to. This is hard/impossible to do without some `as any` hacks right now, but the change in this commit makes it possible.

Signed-off-by: Misha Kaletsky <[email protected]>

Author: mmkal

packages/keyv/src/index.d.ts

@@ -27,18 +27,14 @@ declare class Keyv<Value = any, Options extends Record<string, any> = Record<str
 	constructor(uri?: string, options?: Keyv.Options<Value> & Options);
 
 	/** Returns the value. */
-	get<Raw extends boolean = false>(key: string, options?: {raw?: Raw}):
-	Promise<(Raw extends false
-		? Value
-		: Keyv.DeserializedData<Value>) | undefined>;
+	get(key: string, options?: {raw?: false}): Promise<Value | undefined>;
+	/** Returns the raw value. */
+	get(key: string, options: {raw: true}): Promise<Keyv.DeserializedData<Value> | undefined>;
 
 	/** Returns an array of values. Uses `store.getMany` if it exists, otherwise uses parallel calls to `store.get`. */
-	get<Raw extends boolean = false>(
-		key: string[],
-		options?: {raw?: Raw}
-	): Promise<
-	Array<(Raw extends false ? Value : Keyv.DeserializedData<Value>) | undefined>
-	>;
+	get(key: string[], options?: {raw?: false}): Promise<Array<Value | undefined>>;
+	/** Returns an array of raw values. Uses `store.getMany` if it exists, otherwise uses parallel calls to `store.get`. */
+	get(key: string[], options: {raw: true}): Promise<Array<Keyv.DeserializedData<Value> | undefined>>;
 
 	/**
      * Set a value.