@pipedream/sharepoint 0.8.0 → 0.8.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pipedream/sharepoint",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Pipedream Microsoft Sharepoint Online Components",
   "main": "sharepoint.app.mjs",
   "keywords": [

package/sources/updated-file-instant/updated-file-instant.mjs

@@ -4,9 +4,9 @@ import { WEBHOOK_SUBSCRIPTION_RENEWAL_SECONDS } from "../../common/constants.mjs
 
 export default {
   key: "sharepoint-updated-file-instant",
-  name: "New File Updated (Instant)",
-  description: "Emit new event when specific files are updated in a SharePoint document library",
-  version: "0.0.1",
+  name: "File Updated or Deleted (Instant)",
+  description: "Emit a new event when specific files are updated or deleted in a SharePoint document library",
+  version: "0.0.2",
   type: "source",
   dedupe: "unique",
   props: {
@@ -67,8 +67,8 @@ export default {
       ],
       label: "Files to Monitor",
       description:
-        "Select one or more files to monitor for updates. " +
-        "You'll receive a real-time event whenever any of these files are modified.\n\n" +
+        "Select one or more files to monitor for changes. " +
+        "You'll receive a real-time event whenever any of these files are modified or deleted.\n\n" +
         "**Important:** Only the selected files will trigger events. Changes to other files in the drive will be ignored. " +
         "This ensures you only receive notifications for the documents you care about.",
     },
@@ -101,8 +101,39 @@ export default {
         clientState,
       });
 
-      // Store the file IDs we're monitoring (unwrap labeled values)
-      const fileIds = this.sharepoint.resolveWrappedArrayValues(this.fileIds);
+      // Store the file IDs we're monitoring (unwrap labeled values and parse JSON)
+      const wrappedFileIds = this.sharepoint.resolveWrappedArrayValues(this.fileIds);
+      // Parse JSON strings to extract just the IDs, handle objects, and trim strings
+      const fileIds = wrappedFileIds.map((fileId) => {
+        // Handle object values directly
+        if (typeof fileId === "object" && fileId !== null) {
+          if (!fileId.id) {
+            console.log(`Warning: Object fileId missing 'id' field: ${JSON.stringify(fileId)}`);
+            return String(fileId);
+          }
+          return fileId.id;
+        }
+
+        // Handle string values - trim whitespace first
+        if (typeof fileId === "string") {
+          const trimmedFileId = fileId.trim();
+          if (trimmedFileId.startsWith("{")) {
+            try {
+              const parsed = JSON.parse(trimmedFileId);
+              if (!parsed || !parsed.id) {
+                throw new Error("Parsed object missing 'id' field");
+              }
+              return parsed.id;
+            } catch (e) {
+              console.log(`Warning: Failed to parse fileId: ${trimmedFileId}, error: ${e.message}`);
+              return trimmedFileId;
+            }
+          }
+          return trimmedFileId;
+        }
+
+        return fileId;
+      });
       this._setMonitoredFileIds(fileIds);
 
       // Initialize delta tracking - get current state so we only see future changes
@@ -220,10 +251,23 @@ export default {
       }
     },
     generateMeta(file) {
-      const ts = Date.parse(file.lastModifiedDateTime);
+      // Use lastModifiedDateTime for updated files, deletedDateTime for deleted files
+      // Fall back to current time only if neither is available
+      let ts;
+      if (file.lastModifiedDateTime) {
+        ts = Date.parse(file.lastModifiedDateTime);
+      } else if (file.deletedDateTime) {
+        ts = Date.parse(file.deletedDateTime);
+      } else {
+        ts = Date.now();
+      }
+
+      const action = file.deleted
+        ? "deleted"
+        : "updated";
       return {
         id: `${file.id}-${ts}`,
-        summary: `File updated: ${file.name}`,
+        summary: `File ${action}: ${file.name}`,
         ts,
       };
     },
@@ -272,8 +316,8 @@ export default {
 
     const validNotifications = body.value.filter((notification) => {
       if (notification.clientState !== clientState) {
-        console.warn(
-          `Ignoring notification with unexpected clientState: ${notification.clientState}`,
+        console.log(
+          `Warning: Ignoring notification with unexpected clientState: ${notification.clientState}`,
         );
         return false;
       }
@@ -313,8 +357,9 @@ export default {
       });
 
       // Find files that changed and are in our monitored list
+      // Include both updated files (item.file) and deleted files (item.deleted)
       for (const item of deltaResponse.value || []) {
-        if (item.file && monitoredFileIds.includes(item.id)) {
+        if ((item.file || item.deleted) && monitoredFileIds.includes(item.id)) {
           changedFiles.push(item);
         }
       }
@@ -336,8 +381,9 @@ export default {
     // Emit events for each changed file
     for (const file of changedFiles) {
       // Delta response may not include downloadUrl - fetch fresh if needed
+      // Skip fetching download URL for deleted files (they no longer exist)
       let downloadUrl = file["@microsoft.graph.downloadUrl"];
-      if (!downloadUrl) {
+      if (!downloadUrl && !file.deleted) {
         try {
           const freshFile = await this.sharepoint.getDriveItem({
             driveId,