greptile comments

Author: aadamgough

apps/sim/tools/google_drive/download.ts

@@ -150,7 +150,8 @@ export const downloadTool: ToolConfig<GoogleDriveToolParams, GoogleDriveDownload
       }
 
       const includeRevisions = params?.includeRevisions !== false
-      if (includeRevisions) {
+      const canReadRevisions = metadata.capabilities?.canReadRevisions === true
+      if (includeRevisions && canReadRevisions) {
         try {
           const revisionsResponse = await fetch(
             `https://www.googleapis.com/drive/v3/files/${fileId}/revisions?fields=revisions(${ALL_REVISION_FIELDS})&pageSize=100`,
@@ -179,6 +180,10 @@ export const downloadTool: ToolConfig<GoogleDriveToolParams, GoogleDriveDownload
             error: revisionError.message,
           })
         }
+      } else if (includeRevisions && !canReadRevisions) {
+        logger.info('Skipping revision fetch - user does not have canReadRevisions permission', {
+          fileId,
+        })
       }
 
       const resolvedName = params?.fileName || metadata.name || 'download'

apps/sim/tools/google_drive/get_content.ts

@@ -138,7 +138,8 @@ export const getContentTool: ToolConfig<GoogleDriveToolParams, GoogleDriveGetCon
       }
 
       const includeRevisions = params?.includeRevisions !== false
-      if (includeRevisions) {
+      const canReadRevisions = metadata.capabilities?.canReadRevisions === true
+      if (includeRevisions && canReadRevisions) {
         try {
           const revisionsResponse = await fetch(
             `https://www.googleapis.com/drive/v3/files/${fileId}/revisions?fields=revisions(${ALL_REVISION_FIELDS})&pageSize=100`,
@@ -167,6 +168,10 @@ export const getContentTool: ToolConfig<GoogleDriveToolParams, GoogleDriveGetCon
             error: revisionError.message,
           })
         }
+      } else if (includeRevisions && !canReadRevisions) {
+        logger.info('Skipping revision fetch - user does not have canReadRevisions permission', {
+          fileId,
+        })
       }
 
       logger.info('File content retrieved successfully', {

apps/sim/tools/google_drive/list.ts

@@ -62,19 +62,25 @@ export const listTool: ToolConfig<GoogleDriveToolParams, GoogleDriveListResponse
       url.searchParams.append('supportsAllDrives', 'true')
       url.searchParams.append('includeItemsFromAllDrives', 'true')
 
+      // Helper to escape single quotes for Google Drive query syntax
+      const escapeQueryValue = (value: string): string =>
+        value.replace(/\\/g, '\\\\').replace(/'/g, "\\'")
+
       // Build the query conditions
       const conditions = ['trashed = false'] // Always exclude trashed files
       const folderId = params.folderId || params.folderSelector
       if (folderId) {
-        conditions.push(`'${folderId}' in parents`)
+        const escapedFolderId = escapeQueryValue(folderId)
+        conditions.push(`'${escapedFolderId}' in parents`)
       }
 
       // Combine all conditions with AND
       url.searchParams.append('q', conditions.join(' and '))
 
       if (params.query) {
         const existingQ = url.searchParams.get('q')
-        const queryPart = `name contains '${params.query}'`
+        const escapedQuery = escapeQueryValue(params.query)
+        const queryPart = `name contains '${escapedQuery}'`
         url.searchParams.set('q', `${existingQ} and ${queryPart}`)
       }
       if (params.pageSize) {