fix(microsoft-excel): fix write output field mapping + OData escaping gaps

Found in an independent second-pass audit against the live Graph API
docs:

- microsoft_excel_write (v1) transformResponse read
  updatedRange/updatedRows/updatedColumns/updatedCells from the Graph
  response — none of these fields exist on the workbookRange resource
  (the real fields are address/rowCount/columnCount), so these four
  output fields were always undefined. Fixed to read the actual
  fields, matching what the v2 write tool already does correctly.
- read.ts and write.ts (v1 and v2) built worksheets('name') OData
  URLs with only encodeURIComponent, skipping escapeODataString — a
  worksheet name containing an apostrophe would produce a malformed
  request. Every other tool in this integration already escapes
  correctly; read/write were the outliers.
- write.ts (v1) also wasn't trimming spreadsheetId before use,
  inconsistent with every other tool here.

Author: waleedlatif1

apps/sim/tools/microsoft_excel/read.ts

@@ -7,6 +7,7 @@ import type {
   MicrosoftExcelV2ToolParams,
 } from '@/tools/microsoft_excel/types'
 import {
+  escapeODataString,
   getItemBasePath,
   getSpreadsheetWebUrl,
   parseGraphErrorMessage,
@@ -79,7 +80,7 @@ export const readTool: ToolConfig<MicrosoftExcelToolParams, MicrosoftExcelReadRe
       const rangeInput = params.range.trim()
 
       if (!rangeInput.includes('!')) {
-        const sheetOnly = encodeURIComponent(rangeInput)
+        const sheetOnly = encodeURIComponent(escapeODataString(rangeInput))
         return `${basePath}/workbook/worksheets('${sheetOnly}')/usedRange(valuesOnly=true)`
       }
 
@@ -91,7 +92,7 @@ export const readTool: ToolConfig<MicrosoftExcelToolParams, MicrosoftExcelReadRe
         )
       }
 
-      const sheetName = encodeURIComponent(match[1])
+      const sheetName = encodeURIComponent(escapeODataString(match[1]))
       const address = encodeURIComponent(match[2])
 
       return `${basePath}/workbook/worksheets('${sheetName}')/range(address='${address}')`
@@ -128,7 +129,7 @@ export const readTool: ToolConfig<MicrosoftExcelToolParams, MicrosoftExcelReadRe
       }
 
       const basePath = getItemBasePath(spreadsheetId, driveId)
-      const rangeUrl = `${basePath}/workbook/worksheets('${encodeURIComponent(firstSheetName)}')/usedRange(valuesOnly=true)`
+      const rangeUrl = `${basePath}/workbook/worksheets('${encodeURIComponent(escapeODataString(firstSheetName))}')/usedRange(valuesOnly=true)`
 
       const rangeResp = await fetch(rangeUrl, {
         headers: { Authorization: `Bearer ${accessToken}` },
@@ -278,7 +279,7 @@ export const readV2Tool: ToolConfig<MicrosoftExcelV2ToolParams, MicrosoftExcelV2
       }
 
       const basePath = getItemBasePath(spreadsheetId, params.driveId)
-      const encodedSheetName = encodeURIComponent(sheetName)
+      const encodedSheetName = encodeURIComponent(escapeODataString(sheetName))
 
       if (!params.cellRange) {
         return `${basePath}/workbook/worksheets('${encodedSheetName}')/usedRange(valuesOnly=true)`

apps/sim/tools/microsoft_excel/write.ts

@@ -5,7 +5,11 @@ import type {
   MicrosoftExcelV2WriteResponse,
   MicrosoftExcelWriteResponse,
 } 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'
 
 /**
@@ -82,17 +86,22 @@ export const writeTool: ToolConfig<MicrosoftExcelToolParams, MicrosoftExcelWrite
 
   request: {
     url: (params) => {
+      const spreadsheetId = params.spreadsheetId?.trim()
+      if (!spreadsheetId) {
+        throw new Error('Spreadsheet ID is required')
+      }
+
       const rangeInput = params.range?.trim()
       const match = rangeInput?.match(/^([^!]+)!(.+)$/)
 
       if (!match) {
         throw new Error(`Invalid range format: "${params.range}". Use the format "Sheet1!A1:B2"`)
       }
 
-      const sheetName = encodeURIComponent(match[1])
+      const sheetName = encodeURIComponent(escapeODataString(match[1]))
       const address = encodeURIComponent(match[2])
 
-      const basePath = getItemBasePath(params.spreadsheetId!, params.driveId)
+      const basePath = getItemBasePath(spreadsheetId, params.driveId)
       const url = new URL(
         `${basePath}/workbook/worksheets('${sheetName}')/range(address='${address}')`
       )
@@ -173,10 +182,10 @@ export const writeTool: ToolConfig<MicrosoftExcelToolParams, MicrosoftExcelWrite
     return {
       success: true,
       output: {
-        updatedRange: data.updatedRange,
-        updatedRows: data.updatedRows,
-        updatedColumns: data.updatedColumns,
-        updatedCells: data.updatedCells,
+        updatedRange: data.address ?? '',
+        updatedRows: data.rowCount ?? 0,
+        updatedColumns: data.columnCount ?? 0,
+        updatedCells: (data.rowCount ?? 0) * (data.columnCount ?? 0),
         metadata: {
           spreadsheetId,
           spreadsheetUrl: webUrl,
@@ -280,7 +289,7 @@ export const writeV2Tool: ToolConfig<MicrosoftExcelV2ToolParams, MicrosoftExcelV
       }
 
       const cellRange = params.cellRange?.trim() || 'A1'
-      const encodedSheetName = encodeURIComponent(sheetName)
+      const encodedSheetName = encodeURIComponent(escapeODataString(sheetName))
       const encodedAddress = encodeURIComponent(cellRange)
 
       const basePath = getItemBasePath(spreadsheetId, params.driveId)