fix(studio): use older supported syntax rather than Map.groupBy (supabase#41284)

charislam · web-flow · commit 85dde8f7db24 · 2025-12-11T22:02:43.000Z
Some users' browsers do not support Map.groupBy (used in
Storage.utils.ts)
diff --git a/apps/studio/components/interfaces/Storage/Storage.utils.ts b/apps/studio/components/interfaces/Storage/Storage.utils.ts
@@ -90,7 +90,13 @@ export const extractBucketNameFromDefinition = (definition: string | null) => {
 }
 
 const groupPoliciesByBucket = (policies: (PostgresPolicy & { bucket: string | Symbol })[]) => {
-  const policiesByBucket = Map.groupBy(policies, (policy) => policy.bucket)
+  const policiesByBucket = new Map<string | Symbol, PostgresPolicy[]>()
+  policies.forEach((policy) => {
+    if (!policiesByBucket.has(policy.bucket)) {
+      policiesByBucket.set(policy.bucket, [])
+    }
+    policiesByBucket.get(policy.bucket)?.push(policy)
+  })
   return [
     ...policiesByBucket.entries().map(([bucketName, policies]) => ({ name: bucketName, policies })),
   ]

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,13 @@ export const extractBucketNameFromDefinition = (definition: string \| null) => {`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`const groupPoliciesByBucket = (policies: (PostgresPolicy & { bucket: string \| Symbol })[]) => {`
`93`		`- const policiesByBucket = Map.groupBy(policies, (policy) => policy.bucket)`
	`93`	`+ const policiesByBucket = new Map<string \| Symbol, PostgresPolicy[]>()`
	`94`	`+ policies.forEach((policy) => {`
	`95`	`+ if (!policiesByBucket.has(policy.bucket)) {`
	`96`	`+ policiesByBucket.set(policy.bucket, [])`
	`97`	`+ }`
	`98`	`+ policiesByBucket.get(policy.bucket)?.push(policy)`
	`99`	`+ })`
`94`	`100`	`return [`
`95`	`101`	`...policiesByBucket.entries().map(([bucketName, policies]) => ({ name: bucketName, policies })),`
`96`	`102`	`]`