From 9c0d26bc843cc241e26eae73151fae600d50e520 Mon Sep 17 00:00:00 2001
From: Karl Seguin <k@openmymind.io>
Date: Thu, 17 Jul 2025 20:51:05 +0800
Subject: [PATCH] add note about incomplete removal

---
 src/browser/dom/element.zig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/browser/dom/element.zig b/src/browser/dom/element.zig
index 02ad9e7f..41c882e8 100644
--- a/src/browser/dom/element.zig
+++ b/src/browser/dom/element.zig
@@ -510,6 +510,10 @@ pub const Element = struct {
     }
 
     pub fn _remove(self: *parser.Element) !void {
+        // TODO: This hasn't been tested to make sure all references to this
+        // node are properly updated. A lot of libdom is lazy and will look
+        // for related elements JIT by walking the tree, but there could be
+        // cases in libdom or the Zig WebAPI where this reference is kept
         const as_node: *parser.Node = @ptrCast(self);
         const parent = try parser.nodeParentNode(as_node) orelse return;
         _ = try Node._removeChild(parent, as_node);