I can't recreate a scenario for this case in testing, so add a warning and a speculative mitigation

eoghanmurray · eoghanmurray · commit 1149d3a2638e · 2025-02-12T15:26:30.000Z
diff --git a/packages/rrweb/src/record/mutation.ts b/packages/rrweb/src/record/mutation.ts
@@ -295,6 +295,13 @@ export default class MutationBuffer {
         nextId = nextSibling && this.mirror.getId(nextSibling);
       }
       if (nextId === -1) {
+        // sibling not included in addedNodes, but
+        // no id as not yet serialized?
+        console.warn(
+          "Couldn't record new node. Couldn't find mirror id for nextSibling:",
+          nextSibling,
+        );
+        n = null; // don't iterate to previousSibling as we haven't successful serialized this node
         continue;
       }
       const sn = serializeNodeWithId(n, {

Original file line number	Diff line number	Diff line change
`@@ -295,6 +295,13 @@ export default class MutationBuffer {`
`295`	`295`	`nextId = nextSibling && this.mirror.getId(nextSibling);`
`296`	`296`	`}`
`297`	`297`	`if (nextId === -1) {`
	`298`	`+ // sibling not included in addedNodes, but`
	`299`	`+ // no id as not yet serialized?`
	`300`	`+ console.warn(`
	`301`	`+ "Couldn't record new node. Couldn't find mirror id for nextSibling:",`
	`302`	`+ nextSibling,`
	`303`	`+ );`
	`304`	`+ n = null; // don't iterate to previousSibling as we haven't successful serialized this node`
`298`	`305`	`continue;`
`299`	`306`	`}`
`300`	`307`	`const sn = serializeNodeWithId(n, {`