Skip to content

Commit d2cebca

Browse files
waleedlatif1claude
andcommitted
fix(polling): fix Drive pre-check never activating in Sheets poller
isDriveFileUnchanged short-circuited when lastModifiedTime was undefined, never calling the Drive API — so currentModifiedTime was never populated, creating a permanent chicken-and-egg loop. Now always calls the Drive API and returns the modifiedTime regardless of whether there's a previous value to compare against. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 8e8620c commit d2cebca

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

apps/sim/lib/webhooks/polling/google-sheets.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -226,13 +226,13 @@ async function isDriveFileUnchanged(
226226
requestId: string,
227227
logger: ReturnType<typeof import('@sim/logger').createLogger>
228228
): Promise<{ unchanged: boolean; currentModifiedTime?: string }> {
229-
if (!lastModifiedTime) return { unchanged: false }
230-
231229
try {
232230
const currentModifiedTime = await getDriveFileModifiedTime(accessToken, spreadsheetId, logger)
231+
if (!lastModifiedTime || !currentModifiedTime) {
232+
return { unchanged: false, currentModifiedTime }
233+
}
233234
return { unchanged: currentModifiedTime === lastModifiedTime, currentModifiedTime }
234235
} catch (error) {
235-
// If Drive check fails, proceed with Sheets API (don't skip)
236236
logger.warn(`[${requestId}] Drive modifiedTime check failed, proceeding with Sheets API`)
237237
return { unchanged: false }
238238
}

0 commit comments

Comments
 (0)