fix(google-sheets): correct filter metadata for missing column and header-only sheets

- matchedRows is now 0 (not totalRows) when the filter column is not found,
  so it no longer contradicts applied=false / columnFound=false
- columnFound now reflects an actual header lookup for empty/header-only
  sheets instead of being hardcoded true
- add tests covering header-only and empty sheets with present/absent columns

Author: waleedlatif1

apps/sim/tools/google_sheets/filter.test.ts

@@ -172,18 +172,39 @@ describe('filterSheetRows', () => {
     })
     expect(result.applied).toBe(false)
     expect(result.columnFound).toBe(false)
+    expect(result.matchedRows).toBe(0)
     expect(result.values).toBe(VALUES)
     expect(result.totalRows).toBe(3)
   })
 
-  it('handles a header-only sheet without error', () => {
+  it('handles a header-only sheet and reports the column as found when it exists', () => {
     const headerOnly: unknown[][] = [['Name', 'Status']]
     const result = filterSheetRows(headerOnly, { filterColumn: 'Status', filterValue: 'Active' })
     expect(result.applied).toBe(false)
+    expect(result.columnFound).toBe(true)
+    expect(result.matchedRows).toBe(0)
     expect(result.totalRows).toBe(0)
     expect(result.values).toBe(headerOnly)
   })
 
+  it('reports columnFound=false for a header-only sheet when the column is absent', () => {
+    const headerOnly: unknown[][] = [['Name', 'Status']]
+    const result = filterSheetRows(headerOnly, { filterColumn: 'Nonexistent', filterValue: 'x' })
+    expect(result.applied).toBe(false)
+    expect(result.columnFound).toBe(false)
+    expect(result.matchedRows).toBe(0)
+    expect(result.values).toBe(headerOnly)
+  })
+
+  it('reports columnFound=false for an empty values array', () => {
+    const empty: unknown[][] = []
+    const result = filterSheetRows(empty, { filterColumn: 'Status', filterValue: 'Active' })
+    expect(result.applied).toBe(false)
+    expect(result.columnFound).toBe(false)
+    expect(result.matchedRows).toBe(0)
+    expect(result.totalRows).toBe(0)
+  })
+
   it('treats missing cells as empty strings', () => {
     const sparse: unknown[][] = [['Name', 'Status'], ['Alice'], ['Bob', 'Active']]
     const result = filterSheetRows(sparse, {

apps/sim/tools/google_sheets/filter.ts

@@ -119,18 +119,22 @@ export function filterSheetRows(
     return { values, applied: false, columnFound: true, matchedRows: totalRows, totalRows }
   }
 
-  if (values.length <= 1) {
-    return { values, applied: false, columnFound: true, matchedRows: 0, totalRows: 0 }
-  }
-
-  const headers = values[0]
+  const headers = values[0] ?? []
   const normalizedColumn = filterColumn.trim().toLowerCase()
   const columnIndex = headers.findIndex(
     (header) => String(header).trim().toLowerCase() === normalizedColumn
   )
+  const columnFound = columnIndex !== -1
+
+  // No data rows to evaluate (empty or header-only sheet): nothing matched, but
+  // still report whether the requested column actually exists in the header.
+  if (values.length <= 1) {
+    return { values, applied: false, columnFound, matchedRows: 0, totalRows: 0 }
+  }
 
-  if (columnIndex === -1) {
-    return { values, applied: false, columnFound: false, matchedRows: totalRows, totalRows }
+  // Column not found: leave rows untouched and report zero matches, not totalRows.
+  if (!columnFound) {
+    return { values, applied: false, columnFound: false, matchedRows: 0, totalRows }
   }
 
   const matchType = filterMatchType ?? DEFAULT_MATCH_TYPE

apps/sim/tools/google_sheets/read.test.ts

@@ -81,6 +81,7 @@ describe('readV2Tool.transformResponse', () => {
     expect(result.output.values).toEqual(SHEET_DATA.values)
     expect(result.output.filter?.columnFound).toBe(false)
     expect(result.output.filter?.applied).toBe(false)
+    expect(result.output.filter?.matchedRows).toBe(0)
   })
 
   it('handles a response with no values array', async () => {