fix(microsoft-excel): clean up dead code found during integration audit

- Remove unreachable 'update' operation subblocks (Google Sheets
  copy-paste leftover — never a selectable dropdown option and not
  handled by the tool selector or any registered tool)
- Fix microsoft_excel_table_add to trim spreadsheetId and OData-escape
  tableName, matching every other tool in this file
- Drop phantom/dead type fields: Google-Sheets-only insertDataOption
  and responseValueRenderOption (never used), never-populated
  sheetId/sheetName/title/sheets metadata fields, unused rowIndex, and
  the unconfigurable majorDimension param (hardcoded to 'ROWS' now
  that it's inlined)

Author: waleedlatif1

apps/sim/blocks/blocks/microsoft_excel.ts

@@ -234,41 +234,6 @@ Return ONLY the JSON array - no explanations, no markdown, no extra text.`,
       ],
       condition: { field: 'operation', value: 'write' },
     },
-    {
-      id: 'values',
-      title: 'Values',
-      type: 'long-input',
-      placeholder:
-        'Enter values as JSON array of arrays (e.g., [["A1", "B1"], ["A2", "B2"]]) or an array of objects (e.g., [{"name":"John", "age":30}, {"name":"Jane", "age":25}])',
-      condition: { field: 'operation', value: 'update' },
-      required: true,
-      wandConfig: {
-        enabled: true,
-        prompt: `Generate Microsoft Excel data as a JSON array based on the user's description.
-
-Format options:
-1. Array of arrays: [["Header1", "Header2"], ["Value1", "Value2"]]
-2. Array of objects: [{"column1": "value1", "column2": "value2"}]
-
-Examples:
-- "update with new prices" -> [["Product", "Price"], ["Widget A", 29.99], ["Widget B", 49.99]]
-- "quarterly targets" -> [{"Q1": 10000, "Q2": 12000, "Q3": 15000, "Q4": 18000}]
-
-Return ONLY the JSON array - no explanations, no markdown, no extra text.`,
-        placeholder: 'Describe the data you want to update...',
-        generationType: 'json-object',
-      },
-    },
-    {
-      id: 'valueInputOption',
-      title: 'Value Input Option',
-      type: 'dropdown',
-      options: [
-        { label: 'User Entered (Parse formulas)', id: 'USER_ENTERED' },
-        { label: "Raw (Don't parse formulas)", id: 'RAW' },
-      ],
-      condition: { field: 'operation', value: 'update' },
-    },
     {
       id: 'values',
       title: 'Values',

apps/sim/tools/microsoft_excel/table_add.ts

@@ -3,7 +3,11 @@ import type {
   MicrosoftExcelTableAddResponse,
   MicrosoftExcelTableToolParams,
 } from '@/tools/microsoft_excel/types'
-import { getItemBasePath, getSpreadsheetWebUrl } from '@/tools/microsoft_excel/utils'
+import {
+  escapeODataString,
+  getItemBasePath,
+  getSpreadsheetWebUrl,
+} from '@/tools/microsoft_excel/utils'
 import type { ToolConfig } from '@/tools/types'
 
 export const tableAddTool: ToolConfig<
@@ -59,15 +63,27 @@ export const tableAddTool: ToolConfig<
 
   request: {
     url: (params) => {
-      const tableName = encodeURIComponent(params.tableName)
-      const basePath = getItemBasePath(params.spreadsheetId, params.driveId)
-      return `${basePath}/workbook/tables('${tableName}')/rows/add`
+      const spreadsheetId = params.spreadsheetId?.trim()
+      if (!spreadsheetId) {
+        throw new Error('Spreadsheet ID is required')
+      }
+      const tableName = params.tableName?.trim()
+      if (!tableName) {
+        throw new Error('Table name is required')
+      }
+      const basePath = getItemBasePath(spreadsheetId, params.driveId)
+      return `${basePath}/workbook/tables('${encodeURIComponent(escapeODataString(tableName))}')/rows/add`
     },
     method: 'POST',
-    headers: (params) => ({
-      Authorization: `Bearer ${params.accessToken}`,
-      'Content-Type': 'application/json',
-    }),
+    headers: (params) => {
+      if (!params.accessToken) {
+        throw new Error('Access token is required')
+      }
+      return {
+        Authorization: `Bearer ${params.accessToken}`,
+        'Content-Type': 'application/json',
+      }
+    },
     body: (params) => {
       let processedValues: any = params.values || []
 

apps/sim/tools/microsoft_excel/types.ts

@@ -4,23 +4,13 @@ import type { ToolResponse } from '@/tools/types'
 export type ExcelCellValue = string | number | boolean | null
 
 interface MicrosoftExcelRange {
-  sheetId?: number
-  sheetName?: string
   range: string
   values: ExcelCellValue[][]
 }
 
 interface MicrosoftExcelMetadata {
   spreadsheetId: string
   spreadsheetUrl?: string
-  title?: string
-  sheets?: {
-    sheetId: number
-    title: string
-    index: number
-    rowCount?: number
-    columnCount?: number
-  }[]
 }
 
 export interface MicrosoftExcelReadResponse extends ToolResponse {
@@ -67,10 +57,7 @@ export interface MicrosoftExcelToolParams {
   range?: string
   values?: ExcelCellValue[][]
   valueInputOption?: 'RAW' | 'USER_ENTERED'
-  insertDataOption?: 'OVERWRITE' | 'INSERT_ROWS'
   includeValuesInResponse?: boolean
-  responseValueRenderOption?: 'FORMATTED_VALUE' | 'UNFORMATTED_VALUE' | 'FORMULA'
-  majorDimension?: 'ROWS' | 'COLUMNS'
 }
 
 export interface MicrosoftExcelTableToolParams {
@@ -79,7 +66,6 @@ export interface MicrosoftExcelTableToolParams {
   driveId?: string
   tableName: string
   values: ExcelCellValue[][]
-  rowIndex?: number
 }
 
 export interface MicrosoftExcelWorksheetToolParams {
@@ -217,7 +203,6 @@ export interface MicrosoftExcelV2ToolParams {
   values?: ExcelCellValue[][]
   valueInputOption?: 'RAW' | 'USER_ENTERED'
   includeValuesInResponse?: boolean
-  majorDimension?: 'ROWS' | 'COLUMNS'
 }
 
 export interface MicrosoftExcelV2ReadResponse extends ToolResponse {

apps/sim/tools/microsoft_excel/write.ts

@@ -145,7 +145,7 @@ export const writeTool: ToolConfig<MicrosoftExcelToolParams, MicrosoftExcelWrite
       }
 
       const body: Record<string, any> = {
-        majorDimension: params.majorDimension || 'ROWS',
+        majorDimension: 'ROWS',
         values: processedValues,
       }
 
@@ -337,7 +337,7 @@ export const writeV2Tool: ToolConfig<MicrosoftExcelV2ToolParams, MicrosoftExcelV
       }
 
       const body: Record<string, any> = {
-        majorDimension: params.majorDimension || 'ROWS',
+        majorDimension: 'ROWS',
         values: processedValues,
       }