check if brands is an array before using .some (#8458)

reidbarber · web-flow · commit dfdb167959d7 · 2025-06-27T01:20:39.000Z
diff --git a/packages/@react-aria/utils/src/platform.ts b/packages/@react-aria/utils/src/platform.ts
@@ -14,10 +14,9 @@ function testUserAgent(re: RegExp) {
   if (typeof window === 'undefined' || window.navigator == null) {
     return false;
   }
-  return (
-    window.navigator['userAgentData']?.brands.some((brand: {brand: string, version: string}) => re.test(brand.brand))
-  ) ||
-  re.test(window.navigator.userAgent);
+  let brands = window.navigator['userAgentData']?.brands;
+  return Array.isArray(brands) && brands.some((brand: {brand: string, version: string}) => re.test(brand.brand)) ||
+    re.test(window.navigator.userAgent);
 }
 
 function testPlatform(re: RegExp) {

Original file line number	Diff line number	Diff line change
`@@ -14,10 +14,9 @@ function testUserAgent(re: RegExp) {`
`14`	`14`	`if (typeof window === 'undefined' \|\| window.navigator == null) {`
`15`	`15`	`return false;`
`16`	`16`	`}`
`17`		`- return (`
`18`		`- window.navigator['userAgentData']?.brands.some((brand: {brand: string, version: string}) => re.test(brand.brand))`
`19`		`- ) \|\|`
`20`		`- re.test(window.navigator.userAgent);`
	`17`	`+ let brands = window.navigator['userAgentData']?.brands;`
	`18`	`+ return Array.isArray(brands) && brands.some((brand: {brand: string, version: string}) => re.test(brand.brand)) \|\|`
	`19`	`+ re.test(window.navigator.userAgent);`
`21`	`20`	`}`
`22`	`21`
`23`	`22`	`function testPlatform(re: RegExp) {`