@poveste/shared 0.15.0 → 0.16.0

package/dist/state.d.ts

@@ -1,5 +1,32 @@
 export declare function clone(data: any): any;
 export declare function omit(data: any, keys: string[]): Record<string, any>;
+/**
+ * Whether Vue has marked this value raw, and the walkers must stop here.
+ *
+ * Read as a property rather than through Vue's `isReactive`, because this
+ * package does not depend on Vue — and because the flag is what has to be
+ * asked anyway: `markRaw` defines `__v_skip` **non-enumerable**, so it crosses
+ * neither `structuredClone` nor JSON, and by the time a value has been through
+ * the sandbox bridge the mark is gone (#957).
+ *
+ * A marked value looks plain to every predicate here: a component's exposed
+ * object and `markRaw({ … })` both have `Object.prototype` and own enumerable
+ * keys, so without this the compare and the copy descend into whatever is
+ * behind them.
+ *
+ * The read is guarded because state is user data and may be a proxy with a
+ * throwing trap; an unanswerable value is treated as unmarked, which is the
+ * behaviour this had before.
+ */
+export declare function isMarkedRaw(value: any): boolean;
+/**
+ * Exported because `poveste-app`'s walker has to draw the same line. The two
+ * disagreeing about what a non-plain value is was #977: the walker flattened a
+ * `Date` to `{}` before `isEquivalent` ever saw it, so the defence below —
+ * written for exactly that case — was handed two empty plain objects and
+ * reported them equivalent.
+ */
+export declare function isPlainObject(value: any): boolean;
 /**
  * Structural comparison, deliberately narrow: it recurses into plain objects and
  * arrays and compares everything else by `Object.is`.

package/dist/state.js

@@ -24,7 +24,43 @@ export function omit(data, keys) {
     }
     return copy;
 }
-function isPlainObject(value) {
+/**
+ * Whether Vue has marked this value raw, and the walkers must stop here.
+ *
+ * Read as a property rather than through Vue's `isReactive`, because this
+ * package does not depend on Vue — and because the flag is what has to be
+ * asked anyway: `markRaw` defines `__v_skip` **non-enumerable**, so it crosses
+ * neither `structuredClone` nor JSON, and by the time a value has been through
+ * the sandbox bridge the mark is gone (#957).
+ *
+ * A marked value looks plain to every predicate here: a component's exposed
+ * object and `markRaw({ … })` both have `Object.prototype` and own enumerable
+ * keys, so without this the compare and the copy descend into whatever is
+ * behind them.
+ *
+ * The read is guarded because state is user data and may be a proxy with a
+ * throwing trap; an unanswerable value is treated as unmarked, which is the
+ * behaviour this had before.
+ */
+export function isMarkedRaw(value) {
+    if (value === null || typeof value !== 'object') {
+        return false;
+    }
+    try {
+        return value.__v_skip === true;
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Exported because `poveste-app`'s walker has to draw the same line. The two
+ * disagreeing about what a non-plain value is was #977: the walker flattened a
+ * `Date` to `{}` before `isEquivalent` ever saw it, so the defence below —
+ * written for exactly that case — was handed two empty plain objects and
+ * reported them equivalent.
+ */
+export function isPlainObject(value) {
     if (value === null || typeof value !== 'object') {
         return false;
     }
@@ -49,6 +85,12 @@ export function isEquivalent(a, b, seen) {
     if (Object.is(a, b)) {
         return true;
     }
+    // Same reason as the prototype test below, reached differently: a marked
+    // value is one Vue itself refuses to look inside, and two distinct ones are
+    // not the same value whatever their keys say.
+    if (isMarkedRaw(a) || isMarkedRaw(b)) {
+        return false;
+    }
     const bothArrays = Array.isArray(a) && Array.isArray(b);
     if (!bothArrays && !(isPlainObject(a) && isPlainObject(b))) {
         return false;
@@ -236,7 +278,7 @@ export function diffState(baseline, next) {
  * `Object.is`, so a reference is the identity the comparison is about.
  */
 function copyState(value, seen = new WeakMap()) {
-    if (!Array.isArray(value) && !isPlainObject(value)) {
+    if (isMarkedRaw(value) || (!Array.isArray(value) && !isPlainObject(value))) {
         return value;
     }
     if (seen.has(value)) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@poveste/shared",
   "type": "module",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "description": "Shared utilities for Poveste",
   "author": {
     "name": "Sorin Gitlan",
@@ -33,24 +33,26 @@
     "dist"
   ],
   "engines": {
-    "node": "^22.22.2 || ^24.15.0 || >=26.0.0"
+    "node": ">=22.22.2"
   },
   "peerDependencies": {
     "vite": "^8.0.0"
   },
   "dependencies": {
-    "@poveste/vendors": "^0.15.0",
+    "@poveste/vendors": "^0.16.0",
     "@types/fs-extra": "^11.0.4",
     "chokidar": "^5.0.0",
     "markdown-it": "^15.0.2",
     "pathe": "^2.0.3",
     "picocolors": "^1.1.1",
-    "unhead": "^3.1.0"
+    "unhead": "^3.4.1"
   },
   "devDependencies": {
+    "@types/node": "^26.6.3",
     "typescript": "6.0.3",
-    "vite": "^8.2.0",
-    "vitest": "^5.0.0"
+    "vite": "^8.3.1",
+    "vitest": "^5.0.2",
+    "vue-router": "^5.3.1"
   },
   "scripts": {
     "build": "rimraf dist && tsc",