fix(office-excel): delegate to parseGraphErrorFromData and handle array embed param

Author: waleedlatif1

apps/sim/app/chat/[identifier]/page.tsx

@@ -15,7 +15,7 @@ export default async function ChatPage({
 }) {
   const { identifier } = await params
   const { embed } = await searchParams
-  const isOfficeEmbed = embed === 'office'
+  const isOfficeEmbed = embed === 'office' || (Array.isArray(embed) && embed.includes('office'))
 
   return (
     <>

apps/sim/tools/error-extractors.ts

@@ -19,6 +19,8 @@
  * 2. Add the ID to ErrorExtractorId constant at the bottom of this file
  */
 
+import { parseGraphErrorFromData } from '@/tools/microsoft_excel/utils'
+
 export interface ErrorInfo {
   status?: number
   statusText?: string
@@ -189,47 +191,7 @@ const ERROR_EXTRACTORS: ErrorExtractorConfig[] = [
     description:
       'Microsoft Graph error format with nested innerError chain and details[] (Excel, OneDrive, SharePoint, Outlook). See https://learn.microsoft.com/en-us/graph/errors',
     examples: ['Microsoft Excel', 'Microsoft OneDrive', 'Microsoft SharePoint'],
-    extract: (errorInfo) => {
-      const data = errorInfo?.data
-      if (!data || typeof data !== 'object') return undefined
-
-      const root = (data as { error?: any }).error
-      if (root && typeof root === 'object') {
-        const messages: string[] = []
-        if (typeof root.message === 'string' && root.message.trim()) {
-          messages.push(root.message.trim())
-        }
-
-        // Walk nested innerError chain. Spec uses `innererror` (lowercase),
-        // Graph commonly returns `innerError` — accept both. Cap depth.
-        let inner: any = root.innererror ?? root.innerError
-        let depth = 0
-        while (inner && depth < 5) {
-          if (typeof inner.message === 'string' && inner.message.trim()) {
-            const msg = inner.message.trim()
-            if (!messages.includes(msg)) messages.push(msg)
-          }
-          inner = inner.innererror ?? inner.innerError
-          depth++
-        }
-
-        if (Array.isArray(root.details)) {
-          for (const detail of root.details) {
-            if (detail && typeof detail.message === 'string' && detail.message.trim()) {
-              const msg = detail.message.trim()
-              if (!messages.includes(msg)) messages.push(msg)
-            }
-          }
-        }
-
-        if (messages.length > 0) return messages.join(' — ')
-        if (typeof root.code === 'string' && root.code.trim()) return root.code.trim()
-      }
-
-      const topMessage = (data as { message?: unknown }).message
-      if (typeof topMessage === 'string' && topMessage.trim()) return topMessage.trim()
-      return undefined
-    },
+    extract: (errorInfo) => parseGraphErrorFromData(errorInfo?.data),
   },
   {
     id: 'nested-error-object',