fix(kit): sync instanceMap when appRecord matched (#327)

webfansplz · web-flow · commit 5f901d181b8c · 2024-04-10T12:39:29.000+08:00
diff --git a/packages/devtools-kit/src/plugins/component.ts b/packages/devtools-kit/src/plugins/component.ts
@@ -119,7 +119,8 @@ export function registerComponentDevToolsPlugin(app: VueAppInstance) {
         if (!appRecord?.instanceMap.has(id)) {
           appRecord?.instanceMap.set(id, component)
           // force sync appRecord instanceMap
-          devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap
+          if (devtoolsAppRecords.active.id === appRecord?.id)
+            devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap
         }
       }
 
@@ -153,7 +154,8 @@ export function registerComponentDevToolsPlugin(app: VueAppInstance) {
         if (!appRecord?.instanceMap.has(id)) {
           // force sync appRecord instanceMap
           appRecord?.instanceMap.set(id, component)
-          devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap
+          if (devtoolsAppRecords.active.id === appRecord?.id)
+            devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap
         }
       }
 
@@ -186,7 +188,8 @@ export function registerComponentDevToolsPlugin(app: VueAppInstance) {
 
       appRecord?.instanceMap.delete(id)
       // force sync appRecord instanceMap
-      devtoolsAppRecords.active.instanceMap = appRecord.instanceMap
+      if (devtoolsAppRecords.active.id === appRecord?.id)
+        devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap
 
       debounceSendInspectorTree()
     })

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,8 @@ export function registerComponentDevToolsPlugin(app: VueAppInstance) {`
`119`	`119`	`if (!appRecord?.instanceMap.has(id)) {`
`120`	`120`	`appRecord?.instanceMap.set(id, component)`
`121`	`121`	`// force sync appRecord instanceMap`
`122`		`- devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap`
	`122`	`+ if (devtoolsAppRecords.active.id === appRecord?.id)`
	`123`	`+ devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap`
`123`	`124`	`}`
`124`	`125`	`}`
`125`	`126`
`@@ -153,7 +154,8 @@ export function registerComponentDevToolsPlugin(app: VueAppInstance) {`
`153`	`154`	`if (!appRecord?.instanceMap.has(id)) {`
`154`	`155`	`// force sync appRecord instanceMap`
`155`	`156`	`appRecord?.instanceMap.set(id, component)`
`156`		`- devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap`
	`157`	`+ if (devtoolsAppRecords.active.id === appRecord?.id)`
	`158`	`+ devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap`
`157`	`159`	`}`
`158`	`160`	`}`
`159`	`161`
`@@ -186,7 +188,8 @@ export function registerComponentDevToolsPlugin(app: VueAppInstance) {`
`186`	`188`
`187`	`189`	`appRecord?.instanceMap.delete(id)`
`188`	`190`	`// force sync appRecord instanceMap`
`189`		`- devtoolsAppRecords.active.instanceMap = appRecord.instanceMap`
	`191`	`+ if (devtoolsAppRecords.active.id === appRecord?.id)`
	`192`	`+ devtoolsAppRecords.active.instanceMap = appRecord!.instanceMap`
`190`	`193`
`191`	`194`	`debounceSendInspectorTree()`
`192`	`195`	`})`