fix(kit): add null-safety check to detect vue reactive linked list node (#661)

webfansplz · web-flow · commit c3f19787c6f9 · 2024-10-31T05:26:18.000+08:00
diff --git a/packages/devtools-kit/src/shared/transfer.ts b/packages/devtools-kit/src/shared/transfer.ts
@@ -13,7 +13,7 @@ function isArray(_data: unknown, proto: string): _data is unknown[] {
 
 // See https://github1s.com/vuejs/core/blob/HEAD/packages/reactivity/src/dep.ts#L32-L33
 function isVueReactiveLinkNode(node) {
-  const constructorName = node.constructor.name
+  const constructorName = node?.constructor?.name
   return (constructorName === 'Dep' && 'activeLink' in node) || (constructorName === 'Link' && 'dep' in node)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ function isArray(_data: unknown, proto: string): _data is unknown[] {`
`13`	`13`
`14`	`14`	`// See https://github1s.com/vuejs/core/blob/HEAD/packages/reactivity/src/dep.ts#L32-L33`
`15`	`15`	`function isVueReactiveLinkNode(node) {`
`16`		`- const constructorName = node.constructor.name`
	`16`	`+ const constructorName = node?.constructor?.name`
`17`	`17`	`return (constructorName === 'Dep' && 'activeLink' in node) \|\| (constructorName === 'Link' && 'dep' in node)`
`18`	`18`	`}`
`19`	`19`