fix(inspector): callFunctionOn and clear maps when document update.

b10902118 · b10902118 · commit 10c670f7fa83 · 2025-10-28T17:22:57.000+08:00
Otherwise new event's nodeId might refer to old node.
diff --git a/packages/@devtoolcss-inspector/src/DOMWrappers.ts b/packages/@devtoolcss-inspector/src/DOMWrappers.ts
@@ -21,15 +21,15 @@ export class NodeWrapper {
   }
   protected async callFunctionOn(args: any[], functionDeclaration: string) {
     // always do live check first
-    const nodeId = this.inspector.getCdpNode(this);
-    if (!nodeId) {
+    const cdpNode = this.inspector.getCdpNode(this);
+    if (!cdpNode) {
       throw new Error("Node is not tracked in the inspector");
     }
 
     if (!this.objectId) {
       // Get the remote object ID for this node
       const { object } = await this.inspector.sendCommand("DOM.resolveNode", {
-        nodeId,
+        nodeId: cdpNode.nodeId,
       });
       this.objectId = object.objectId;
     }
diff --git a/packages/@devtoolcss-inspector/src/Inspector.ts b/packages/@devtoolcss-inspector/src/Inspector.ts
@@ -405,6 +405,8 @@ export class Inspector extends EventEmitter {
   }
 
   private async initDOM(): Promise<void> {
+    this.idToNodes.clear();
+    this.nodeToId.clear();
     const { root } = await this.sendCommand("DOM.getDocument", {
       depth: 0,
     });

Original file line number	Diff line number	Diff line change
`@@ -21,15 +21,15 @@ export class NodeWrapper {`
`21`	`21`	`}`
`22`	`22`	`protected async callFunctionOn(args: any[], functionDeclaration: string) {`
`23`	`23`	`// always do live check first`
`24`		`- const nodeId = this.inspector.getCdpNode(this);`
`25`		`- if (!nodeId) {`
	`24`	`+ const cdpNode = this.inspector.getCdpNode(this);`
	`25`	`+ if (!cdpNode) {`
`26`	`26`	`throw new Error("Node is not tracked in the inspector");`
`27`	`27`	`}`
`28`	`28`
`29`	`29`	`if (!this.objectId) {`
`30`	`30`	`// Get the remote object ID for this node`
`31`	`31`	`const { object } = await this.inspector.sendCommand("DOM.resolveNode", {`
`32`		`- nodeId,`
	`32`	`+ nodeId: cdpNode.nodeId,`
`33`	`33`	`});`
`34`	`34`	`this.objectId = object.objectId;`
`35`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -405,6 +405,8 @@ export class Inspector extends EventEmitter {`
`405`	`405`	`}`
`406`	`406`
`407`	`407`	`private async initDOM(): Promise<void> {`
	`408`	`+ this.idToNodes.clear();`
	`409`	`+ this.nodeToId.clear();`
`408`	`410`	`const { root } = await this.sendCommand("DOM.getDocument", {`
`409`	`411`	`depth: 0,`
`410`	`412`	`});`