@smithers-orchestrator/devtools 0.20.3 → 0.20.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithers-orchestrator/devtools",
-  "version": "0.20.3",
+  "version": "0.20.4",
   "description": "Devtools and debugging utilities for Smithers workflow inspection",
   "type": "module",
   "sideEffects": false,

package/src/diffSnapshots.js

@@ -181,25 +181,18 @@ export function diffSnapshots(a, b) {
         .map((id) => ({ op: "removeNode", id }));
     const topLevelAddIds = [...addSet].filter((id) => {
         const parentId = to.get(id)?.parentId;
-        return parentId === null || !addSet.has(parentId);
+        return parentId !== null && !addSet.has(parentId);
     });
     /** @type {DevToolsDeltaOp[]} */
     const addOps = topLevelAddIds
         .sort((leftId, rightId) => (to.get(leftId)?.node.depth ?? 0) - (to.get(rightId)?.node.depth ?? 0))
         .map((id) => {
         const entry = to.get(id);
-        if (!entry || entry.parentId === null) {
-            return /** @type {DevToolsDeltaOp} */ ({
-                op: "updateProps",
-                id,
-                props: /** @type {Record<string, unknown>} */ (cloneValue(entry?.node.props ?? {})),
-            });
-        }
         return /** @type {DevToolsDeltaOp} */ ({
             op: "addNode",
-            parentId: entry.parentId,
-            index: entry.index,
-            node: /** @type {DevToolsNode} */ (cloneValue(entry.node)),
+            parentId: /** @type {number} */ (entry?.parentId),
+            index: entry?.index ?? 0,
+            node: /** @type {DevToolsNode} */ (cloneValue(entry?.node)),
         });
     });
     return {

package/src/snapshotSerializer.js

@@ -11,9 +11,6 @@ const SNAPSHOT_SERIALIZER_DEFAULT_MAX_ENTRIES = 100_000;
  * @returns {value is Record<string, unknown>}
  */
 function isPlainObject(value) {
-    if (!value || typeof value !== "object") {
-        return false;
-    }
     const proto = Object.getPrototypeOf(value);
     return proto === Object.prototype || proto === null;
 }
@@ -76,9 +73,6 @@ function serializeInternal(value, state, depth, path) {
             ? "[Date: Invalid]"
             : `[Date: ${value.toISOString()}]`;
     }
-    if (valueType !== "object") {
-        return String(value);
-    }
     if (state.traversed >= state.maxEntries) {
         warn("MaxEntriesExceeded", path, state.onWarning);
         return "[MaxEntries]";
@@ -96,10 +90,7 @@ function serializeInternal(value, state, depth, path) {
         }
         if (!isPlainObject(value)) {
             const ctorName = value.constructor?.name;
-            if (ctorName && ctorName !== "Object") {
-                warn("UnsupportedType", path, state.onWarning, ctorName);
-                return `[${ctorName}]`;
-            }
+            if (ctorName && ctorName !== "Object") return warn("UnsupportedType", path, state.onWarning, ctorName), `[${ctorName}]`;
         }
         /** @type {Record<string, unknown>} */
         const out = {};