@bymax-one/nest-cache 1.2.6 → 1.2.7

package/CHANGELOG.md

@@ -6,6 +6,29 @@ project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+### Tests
+
+- `src/readme-quickstart.spec.ts` runs the README Quick Start as written: the standalone module and the
+  `ProfileService` cache-aside flow, over an in-memory Redis, asserting that the value is stored under
+  `app:user-profile:<userId>` with a 3600-second TTL and served from the cache on the second call. The
+  snippet gate compiles the Quick Start but cannot run it.
+
+## [1.2.7] - 2026-11-01
+
+### Fixed
+
+- A thrown or rejected value that cannot be printed no longer makes the error path throw. Caught values
+  were turned into text with `err instanceof Error ? err.message : String(err)`, and
+  `String()` throws for an object with no prototype (`Object.create(null)`) or a `toString` that throws.
+  A `toJSON` hook that threw such a value escaped `JsonSerializer.serialize` as a `TypeError` instead of
+  `cache.serialization_failed`; a Redis script rejection of that kind escaped `ScriptManagerService.eval`
+  as a `TypeError` instead of `cache.script_execution_failed`; and a Pub/Sub handler that threw one was
+  never reported to `onEvent`, because the reporter threw inside its own guard.
+- Those sites now share a `describeError` helper that never throws. It reads a string `message`
+  from the value itself instead of testing `instanceof Error`, which is false for an `Error` created in
+  another realm (`vm`, a worker) and for an error-shaped object; otherwise it uses `String()`, and
+  falls back to the fixed text `unprintable error`.
+
 ## [1.2.6] - 2026-10-31
 
 ### Added
@@ -496,7 +519,8 @@ type or export moved.
 - Published with npm OIDC provenance — no long-lived tokens
 - Zero direct runtime dependencies (`dependencies: {}`) — `ioredis` and NestJS via peer deps
 
-[Unreleased]: https://github.com/bymaxone/nest-cache/compare/v1.2.6...HEAD
+[Unreleased]: https://github.com/bymaxone/nest-cache/compare/v1.2.7...HEAD
+[1.2.7]: https://github.com/bymaxone/nest-cache/compare/v1.2.6...v1.2.7
 [1.2.6]: https://github.com/bymaxone/nest-cache/compare/v1.2.5...v1.2.6
 [1.2.5]: https://github.com/bymaxone/nest-cache/compare/v1.2.4...v1.2.5
 [1.2.4]: https://github.com/bymaxone/nest-cache/compare/v1.2.3...v1.2.4

package/dist/server/index.cjs

@@ -527,9 +527,21 @@ exports.ConnectionManager = __decorateClass([
   __decorateParam(1, common.Optional()),
   __decorateParam(1, common.Inject(BYMAX_CACHE_EVENTS))
 ], exports.ConnectionManager);
+
+// src/server/utils/describe-error.ts
+var UNPRINTABLE = "unprintable error";
+function describeError(reason) {
+  try {
+    const message = reason?.message;
+    return typeof message === "string" ? message : String(reason);
+  } catch {
+    return UNPRINTABLE;
+  }
+}
+
+// src/server/services/script-manager.service.ts
 var SCRIPT_LOAD = "LOAD";
 var NOSCRIPT_MARKER = "NOSCRIPT";
-var toErrorMessage = (err) => err instanceof Error ? err.message : String(err);
 exports.ScriptManagerService = class ScriptManagerService {
   /**
    * @param options - Resolved module options; `options.scripts` seeds the registry.
@@ -637,7 +649,7 @@ exports.ScriptManagerService = class ScriptManagerService {
       } catch (err) {
         throw new CacheException(CACHE_ERROR_CODES.SCRIPT_EXECUTION_FAILED, {
           name,
-          originalError: toErrorMessage(err)
+          originalError: describeError(err)
         });
       }
     }
@@ -645,10 +657,10 @@ exports.ScriptManagerService = class ScriptManagerService {
       const sha = entry.sha ?? await this.load(name);
       return await client.evalsha(sha, keys.length, ...keys, ...args);
     } catch (err) {
-      if (!toErrorMessage(err).includes(NOSCRIPT_MARKER)) {
+      if (!describeError(err).includes(NOSCRIPT_MARKER)) {
         throw new CacheException(CACHE_ERROR_CODES.SCRIPT_EXECUTION_FAILED, {
           name,
-          originalError: toErrorMessage(err)
+          originalError: describeError(err)
         });
       }
       try {
@@ -658,7 +670,7 @@ exports.ScriptManagerService = class ScriptManagerService {
       } catch (retryErr) {
         throw new CacheException(CACHE_ERROR_CODES.SCRIPT_EXECUTION_FAILED, {
           name,
-          originalError: toErrorMessage(retryErr)
+          originalError: describeError(retryErr)
         });
       }
     }
@@ -723,7 +735,6 @@ exports.KeyBuilder = __decorateClass([
   __decorateParam(0, common.Inject(BYMAX_CACHE_OPTIONS))
 ], exports.KeyBuilder);
 var DESERIALIZATION_ERROR = "stored value is not valid JSON";
-var extractErrorMessage = (err) => err instanceof Error ? err.message : String(err);
 exports.JsonSerializer = class JsonSerializer {
   /**
    * Encodes a value as a JSON string.
@@ -749,7 +760,7 @@ exports.JsonSerializer = class JsonSerializer {
       return JSON.stringify(value);
     } catch (err) {
       throw new CacheException(CACHE_ERROR_CODES.SERIALIZATION_FAILED, {
-        error: extractErrorMessage(err)
+        error: describeError(err)
       });
     }
   }
@@ -1622,7 +1633,7 @@ exports.PubSubService = class PubSubService {
         role: "subscriber",
         reason: "handler_error",
         channel,
-        error: error instanceof Error ? error.message : String(error)
+        error: describeError(error)
       });
     } catch {
     }

package/dist/server/index.mjs

@@ -525,9 +525,21 @@ ConnectionManager = __decorateClass([
   __decorateParam(1, Optional()),
   __decorateParam(1, Inject(BYMAX_CACHE_EVENTS))
 ], ConnectionManager);
+
+// src/server/utils/describe-error.ts
+var UNPRINTABLE = "unprintable error";
+function describeError(reason) {
+  try {
+    const message = reason?.message;
+    return typeof message === "string" ? message : String(reason);
+  } catch {
+    return UNPRINTABLE;
+  }
+}
+
+// src/server/services/script-manager.service.ts
 var SCRIPT_LOAD = "LOAD";
 var NOSCRIPT_MARKER = "NOSCRIPT";
-var toErrorMessage = (err) => err instanceof Error ? err.message : String(err);
 var ScriptManagerService = class {
   /**
    * @param options - Resolved module options; `options.scripts` seeds the registry.
@@ -635,7 +647,7 @@ var ScriptManagerService = class {
       } catch (err) {
         throw new CacheException(CACHE_ERROR_CODES.SCRIPT_EXECUTION_FAILED, {
           name,
-          originalError: toErrorMessage(err)
+          originalError: describeError(err)
         });
       }
     }
@@ -643,10 +655,10 @@ var ScriptManagerService = class {
       const sha = entry.sha ?? await this.load(name);
       return await client.evalsha(sha, keys.length, ...keys, ...args);
     } catch (err) {
-      if (!toErrorMessage(err).includes(NOSCRIPT_MARKER)) {
+      if (!describeError(err).includes(NOSCRIPT_MARKER)) {
         throw new CacheException(CACHE_ERROR_CODES.SCRIPT_EXECUTION_FAILED, {
           name,
-          originalError: toErrorMessage(err)
+          originalError: describeError(err)
         });
       }
       try {
@@ -656,7 +668,7 @@ var ScriptManagerService = class {
       } catch (retryErr) {
         throw new CacheException(CACHE_ERROR_CODES.SCRIPT_EXECUTION_FAILED, {
           name,
-          originalError: toErrorMessage(retryErr)
+          originalError: describeError(retryErr)
         });
       }
     }
@@ -721,7 +733,6 @@ KeyBuilder = __decorateClass([
   __decorateParam(0, Inject(BYMAX_CACHE_OPTIONS))
 ], KeyBuilder);
 var DESERIALIZATION_ERROR = "stored value is not valid JSON";
-var extractErrorMessage = (err) => err instanceof Error ? err.message : String(err);
 var JsonSerializer = class {
   /**
    * Encodes a value as a JSON string.
@@ -747,7 +758,7 @@ var JsonSerializer = class {
       return JSON.stringify(value);
     } catch (err) {
       throw new CacheException(CACHE_ERROR_CODES.SERIALIZATION_FAILED, {
-        error: extractErrorMessage(err)
+        error: describeError(err)
       });
     }
   }
@@ -1620,7 +1631,7 @@ var PubSubService = class {
         role: "subscriber",
         reason: "handler_error",
         channel,
-        error: error instanceof Error ? error.message : String(error)
+        error: describeError(error)
       });
     } catch {
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bymax-one/nest-cache",
-  "version": "1.2.6",
+  "version": "1.2.7",
   "description": "Typed Redis cache for NestJS based on ioredis 6, with namespace strategy, Pub/Sub and Lua script management.",
   "author": "Bymax One <support@bymax.one>",
   "license": "MIT",