utils - fix: making Keyv a dependency (#1462)

* utils - fix: making Keyv a dependency

* Update wrap.test.ts

* Update wrap.test.ts

* Update wrap.test.ts

* Update wrap.test.ts

* removing error on warning

* Update wrap.test.ts

* Update package.json

* lint with node-cache

* Update wrap.test.ts

Author: jaredwray

packages/cache-manager/package.json

@@ -62,7 +62,7 @@
     "keyv": "^5.5.4"
   },
   "devDependencies": {
-    "@biomejs/biome": "^2.3.4",
+    "@biomejs/biome": "^2.3.6",
     "@faker-js/faker": "^10.1.0",
     "@keyv/redis": "^5.1.3",
     "@types/node": "^24.10.0",

packages/cache-manager/test/wrap.test.ts

@@ -95,53 +95,50 @@ describe("wrap", () => {
 	it.each([
 		[500, 250],
 		[{ ttl: 500, refreshThreshold: 250 }, undefined],
-	])(
-		"should allow dynamic refreshThreshold on wrap function with ttl/options param as %s",
-		async (ttlOrOptions, refreshThreshold) => {
-			// 1st call should be cached
-			expect(
-				await cache.wrap(
-					data.key,
-					async () => 0,
-					ttlOrOptions as never,
-					refreshThreshold,
-				),
-			).toEqual(0);
-			await sleep(260);
-			// Background refresh, but stale value returned
-			expect(
-				await cache.wrap(
-					data.key,
-					async () => 1,
-					ttlOrOptions as never,
-					refreshThreshold,
-				),
-			).toEqual(0);
-			// New value in cache
-			expect(
-				await cache.wrap(
-					data.key,
-					async () => 2,
-					ttlOrOptions as never,
-					refreshThreshold,
-				),
-			).toEqual(1);
-
-			await sleep(260);
-			// No background refresh with the new override params
-			expect(await cache.wrap(data.key, async () => 3, undefined, 125)).toEqual(
-				1,
-			);
-			await sleep(140);
-			// Background refresh, but stale value returned
-			expect(await cache.wrap(data.key, async () => 4, undefined, 125)).toEqual(
-				1,
-			);
-			expect(await cache.wrap(data.key, async () => 5, undefined, 125)).toEqual(
-				4,
-			);
-		},
-	);
+	])("should allow dynamic refreshThreshold on wrap function with ttl/options param as %s", async (ttlOrOptions, refreshThreshold) => {
+		// 1st call should be cached
+		expect(
+			await cache.wrap(
+				data.key,
+				async () => 0,
+				ttlOrOptions as never,
+				refreshThreshold,
+			),
+		).toEqual(0);
+		await sleep(260);
+		// Background refresh, but stale value returned
+		expect(
+			await cache.wrap(
+				data.key,
+				async () => 1,
+				ttlOrOptions as never,
+				refreshThreshold,
+			),
+		).toEqual(0);
+		// New value in cache
+		expect(
+			await cache.wrap(
+				data.key,
+				async () => 2,
+				ttlOrOptions as never,
+				refreshThreshold,
+			),
+		).toEqual(1);
+
+		await sleep(260);
+		// No background refresh with the new override params
+		expect(await cache.wrap(data.key, async () => 3, undefined, 125)).toEqual(
+			1,
+		);
+		await sleep(140);
+		// Background refresh, but stale value returned
+		expect(await cache.wrap(data.key, async () => 4, undefined, 125)).toEqual(
+			1,
+		);
+		expect(await cache.wrap(data.key, async () => 5, undefined, 125)).toEqual(
+			4,
+		);
+	});
 
 	it("should allow refreshThreshold function on wrap function", async () => {
 		const config = {

packages/node-cache/package.json

@@ -42,7 +42,7 @@
 		"clean": "rimraf ./dist ./coverage ./node_modules"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "^2.3.5",
+		"@biomejs/biome": "^2.3.6",
 		"@faker-js/faker": "^10.1.0",
 		"@types/node": "^24.10.1",
 		"@vitest/coverage-v8": "^4.0.9",

packages/node-cache/src/index.ts

@@ -430,6 +430,26 @@ export class NodeCache<T> extends Hookified {
 		return this.intervalId;
 	}
 
+	public startInterval(): void {
+		if (this.options.checkperiod && this.options.checkperiod > 0) {
+			const checkPeriodinSeconds = this.options.checkperiod * 1000;
+			this.intervalId = setInterval(() => {
+				this.checkData();
+			}, checkPeriodinSeconds).unref();
+
+			return;
+		}
+
+		this.intervalId = 0;
+	}
+
+	public stopInterval(): void {
+		if (this.intervalId !== 0) {
+			clearInterval(this.intervalId);
+			this.intervalId = 0;
+		}
+	}
+
 	private formatKey(key: string | number): string {
 		return key.toString();
 	}
@@ -445,19 +465,6 @@ export class NodeCache<T> extends Hookified {
 		return expirationTimestamp;
 	}
 
-	private startInterval(): void {
-		if (this.options.checkperiod && this.options.checkperiod > 0) {
-			const checkPeriodinSeconds = this.options.checkperiod * 1000;
-			this.intervalId = setInterval(() => {
-				this.checkData();
-			}, checkPeriodinSeconds).unref();
-
-			return;
-		}
-
-		this.intervalId = 0;
-	}
-
 	private checkData(): void {
 		for (const [key, value] of this.store.entries()) {
 			if (value.ttl > 0 && value.ttl < Date.now()) {
@@ -470,13 +477,6 @@ export class NodeCache<T> extends Hookified {
 		}
 	}
 
-	private stopInterval(): void {
-		if (this.intervalId !== 0) {
-			clearInterval(this.intervalId);
-			this.intervalId = 0;
-		}
-	}
-
 	private createError(errorCode: string, key?: string): Error {
 		let error = errorCode;
 		/* v8 ignore next -- @preserve */

packages/utils/package.json

@@ -52,9 +52,7 @@
 		"LICENSE"
 	],
 	"dependencies": {
-		"hashery": "^1.2.0"
-	},
-	"peerDependencies": {
+		"hashery": "^1.2.0",
 		"keyv": "^5.5.4"
 	}
 }