fix(google): keep reminder useDefault, detect fullText outside quoted values, accept numeric task list ids

Author: waleedlatif1

apps/sim/tools/google_calendar/utils.ts

@@ -105,13 +105,14 @@ const MAX_REMINDER_OVERRIDES = 5
 const MAX_REMINDER_MINUTES = 40320
 
 /**
- * Parse reminder overrides from a JSON string or array of `{ method, minutes }` objects.
- * Returns `undefined` when nothing usable was provided. An empty array (`[]`) means
- * "no reminders" and is returned as-is so callers can turn the calendar default off.
+ * Parse the reminders input into the Calendar `reminders` setting. Accepts a JSON string or
+ * value that is either an array of `{ method, minutes }` overrides (custom reminders; `[]`
+ * turns reminders off) or the API's own `{ useDefault, overrides }` object, so a previous
+ * event's reminders can be copied as-is. Returns `undefined` when nothing usable was provided.
  */
-function parseReminderOverrides(
-  reminders: string | GoogleCalendarReminderOverride[] | undefined
-): GoogleCalendarReminderOverride[] | undefined {
+function parseReminders(
+  reminders: unknown
+): { useDefault: boolean; overrides?: GoogleCalendarReminderOverride[] } | undefined {
   if (reminders === undefined || reminders === null) return undefined
 
   let parsed: unknown = reminders
@@ -125,10 +126,16 @@ function parseReminderOverrides(
     }
   }
 
-  if (isRecordLike(parsed) && Array.isArray(parsed.overrides)) {
-    parsed = parsed.overrides
+  if (isRecordLike(parsed) && !Array.isArray(parsed)) {
+    if (parsed.useDefault === true) return { useDefault: true }
+    const overrides = parsed.overrides === undefined ? [] : parsed.overrides
+    return { useDefault: false, overrides: validateReminderOverrides(overrides) }
   }
 
+  return { useDefault: false, overrides: validateReminderOverrides(parsed) }
+}
+
+function validateReminderOverrides(parsed: unknown): GoogleCalendarReminderOverride[] {
   if (!Array.isArray(parsed)) {
     throw new Error('Reminders must be an array of { method, minutes } objects')
   }
@@ -173,8 +180,8 @@ export function applyEventOptions(
   if (params.transparency) body.transparency = params.transparency
   const colorId = params.colorId?.toString().trim()
   if (colorId) body.colorId = colorId
-  const overrides = parseReminderOverrides(params.reminders)
-  if (overrides) body.reminders = { useDefault: false, overrides }
+  const reminders = parseReminders(params.reminders)
+  if (reminders) body.reminders = reminders
 }
 
 /** Fields shared by every event-shaped output beyond the core id/summary/time/people fields. */

apps/sim/tools/google_drive/search.ts

@@ -84,7 +84,8 @@ export const searchTool: ToolConfig<GoogleDriveSearchParams, GoogleDriveSearchRe
       url.searchParams.append('supportsAllDrives', 'true')
       url.searchParams.append('includeItemsFromAllDrives', 'true')
       const orderBy = params.orderBy?.trim()
-      if (orderBy && !/\bfullText\b/.test(params.query ?? '')) {
+      const queryWithoutLiterals = (params.query ?? '').replace(/'(?:[^'\\]|\\.)*'/g, "''")
+      if (orderBy && !/\bfullText\b/.test(queryWithoutLiterals)) {
         url.searchParams.append('orderBy', orderBy)
       }
 

apps/sim/tools/google_tasks/types.ts

@@ -126,7 +126,7 @@ interface TaskLink {
 
 /** Builds the `lists/{tasklist}` path segment, defaulting to the primary list. */
 export function taskListPath(taskListId: string | undefined): string {
-  return `lists/${safeUrlPathSegment(taskListId?.trim() || '@default', 'taskListId', { allowSeparators: true })}`
+  return `lists/${safeUrlPathSegment(String(taskListId ?? '').trim() || '@default', 'taskListId', { allowSeparators: true })}`
 }
 
 /** Builds the `lists/{tasklist}/tasks/{task}` path. */