n8n-nodes-onedrive-business 1.0.0 → 1.1.2

package/dist/nodes/OneDriveBusinessTrigger.node.js

@@ -119,6 +119,39 @@ class OneDriveBusinessTrigger {
                     placeholder: 'contoso.sharepoint.com,site-id,web-id',
                     description: 'SharePoint site ID',
                 },
+                // Watch Scope
+                {
+                    displayName: 'Watch Scope',
+                    name: 'watchScope',
+                    type: 'options',
+                    options: [
+                        {
+                            name: 'Entire Drive',
+                            value: 'drive',
+                            description: 'Monitor changes in the entire drive',
+                        },
+                        {
+                            name: 'Specific Folder',
+                            value: 'folder',
+                            description: 'Monitor changes in a specific folder and its subfolders',
+                        },
+                    ],
+                    default: 'drive',
+                    description: 'Scope of monitoring',
+                },
+                {
+                    displayName: 'Folder Path',
+                    name: 'watchPath',
+                    type: 'string',
+                    displayOptions: {
+                        show: {
+                            watchScope: ['folder'],
+                        },
+                    },
+                    default: '',
+                    placeholder: '/Documents/ProjectX',
+                    description: 'Path relative to the root (e.g., /MyFolder). Leave empty for root.',
+                },
                 // Event Selection
                 {
                     displayName: 'Trigger On',
@@ -206,18 +239,26 @@ class OneDriveBusinessTrigger {
                     await stateStore.initialize(tenantId, driveId, driveLocation.userId, driveLocation.siteId);
                     // Create webhook handler
                     const webhookHandler = new WebhookHandler_1.WebhookHandler(graphClient, stateStore, subscriptionResource);
-                    // Create subscription
-                    const subscription = await webhookHandler.createOrRenewSubscription(webhookUrl);
-                    // Store subscription ID in workflow static data
-                    const webhookData = this.getWorkflowStaticData('node');
-                    webhookData.subscriptionId = subscription.id;
-                    // Store instances for webhook handling
+                    // Store instances for webhook handling BEFORE creating subscription
+                    // This is critical because Microsoft verifies the webhook during creation
                     OneDriveBusinessTrigger.instances.set(nodeId, {
                         stateStore,
                         webhookHandler,
                     });
-                    console.log(`OneDrive Business Trigger activated for node ${nodeId}`);
-                    return true;
+                    try {
+                        // Create subscription
+                        const subscription = await webhookHandler.createOrRenewSubscription(webhookUrl);
+                        // Store subscription ID in workflow static data
+                        const webhookData = this.getWorkflowStaticData('node');
+                        webhookData.subscriptionId = subscription.id;
+                        console.log(`OneDrive Business Trigger activated for node ${nodeId}`);
+                        return true;
+                    }
+                    catch (error) {
+                        // Cleanup if subscription fails
+                        OneDriveBusinessTrigger.instances.delete(nodeId);
+                        throw error;
+                    }
                 },
                 async delete() {
                     const webhookData = this.getWorkflowStaticData('node');
@@ -287,6 +328,19 @@ class OneDriveBusinessTrigger {
             siteId: driveType === 'site' ? this.getNodeParameter('siteId') : undefined,
         };
         const drivePath = (0, helpers_1.resolveDrivePath)(driveLocation);
+        // Get watch scope settings
+        const watchScope = this.getNodeParameter('watchScope');
+        let watchPath;
+        if (watchScope === 'folder') {
+            watchPath = this.getNodeParameter('watchPath');
+            // Normalize path: ensure it doesn't end with / (unless it's just /)
+            if (watchPath && watchPath.length > 1 && watchPath.endsWith('/')) {
+                watchPath = watchPath.slice(0, -1);
+            }
+            // Ensure no leading slash for logic comparison later (Microsoft usually returns paths without leading slash in parentReference? Need to check)
+            // Microsoft parentReference.path is usually: /drives/{drive-id}/root:/path/to/folder
+            // So we will leave it as is for now and handle normalization in DeltaProcessor
+        }
         // Initialize state store
         const credentials = await this.getCredentials('oneDriveBusinessOAuth2Api');
         const tenantId = credentials.tenantId;
@@ -306,7 +360,7 @@ class OneDriveBusinessTrigger {
         // Create delta processor
         const deltaProcessor = new DeltaProcessor_1.DeltaProcessor(graphClient, instance.stateStore, drivePath);
         // Process delta query
-        const triggerEvents = await deltaProcessor.processDeltaQuery(eventFilter);
+        const triggerEvents = await deltaProcessor.processDeltaQuery(eventFilter, watchPath);
         // Clean up old versions periodically
         const state = instance.stateStore.getState();
         const processedCount = Object.keys(state.processedVersions).length;

package/dist/nodes/onedrive.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 64 64"><path d="M41.8 17H41c-7.3 0-13.4 5.3-14.7 12.3-.6-.1-1.2-.2-1.8-.2-6.5 0-11.9 4.6-13.2 10.8C5 40.8 0 46.1 0 52.5c0 6.4 5.2 11.5 11.5 11.5h31.9c11.4 0 20.6-9.2 20.6-20.6 0-10.4-7.7-19-17.7-20.3-1.1-9.9-9.5-17.6-19.5-17.6z" fill="#0078D4"/></svg>

package/dist/utils/DeltaProcessor.d.ts

@@ -44,7 +44,13 @@ export declare class DeltaProcessor {
      *
      * This is the main entry point called by the trigger node
      */
-    processDeltaQuery(eventFilter: Set<string>): Promise<TriggerEvent[]>;
+    /**
+     * Process delta query and return triggerable events
+     *
+     * This is the main entry point called by the trigger node
+     */
+    processDeltaQuery(eventFilter: Set<string>, // e.g., ['file.created', 'file.updated']
+    watchPath?: string): Promise<TriggerEvent[]>;
     /**
      * Fetch all pages from delta query
      * Handles @odata.nextLink pagination and stores @odata.deltaLink
@@ -55,6 +61,10 @@ export declare class DeltaProcessor {
      * Returns a TriggerEvent if the item should trigger a workflow
      */
     private processItem;
+    /**
+     * Check if item is within the watched path
+     */
+    private isItemInScope;
     /**
      * Classify event as created vs updated
      *

package/dist/utils/DeltaProcessor.js

@@ -48,8 +48,13 @@ class DeltaProcessor {
      *
      * This is the main entry point called by the trigger node
      */
-    async processDeltaQuery(eventFilter // e.g., ['file.created', 'file.updated']
-    ) {
+    /**
+     * Process delta query and return triggerable events
+     *
+     * This is the main entry point called by the trigger node
+     */
+    async processDeltaQuery(eventFilter, // e.g., ['file.created', 'file.updated']
+    watchPath) {
         const events = [];
         try {
             // Get current state
@@ -70,7 +75,7 @@ class DeltaProcessor {
             const items = await this.fetchAllDeltaPages(deltaEndpoint);
             // Process each item
             for (const item of items) {
-                const event = await this.processItem(item, eventFilter);
+                const event = await this.processItem(item, eventFilter, watchPath);
                 if (event) {
                     events.push(event);
                 }
@@ -114,7 +119,7 @@ class DeltaProcessor {
      * Process a single delta item
      * Returns a TriggerEvent if the item should trigger a workflow
      */
-    async processItem(item, eventFilter) {
+    async processItem(item, eventFilter, watchPath) {
         // Skip deleted items (unless specifically requested)
         if (item.deleted) {
             return null;
@@ -123,6 +128,10 @@ class DeltaProcessor {
         if (!item.parentReference) {
             return null;
         }
+        // Check scope (folder filtering)
+        if (watchPath && !this.isItemInScope(item, watchPath)) {
+            return null;
+        }
         // Determine event type
         const eventType = this.classifyEvent(item);
         // Check if this event type is enabled
@@ -154,6 +163,42 @@ class DeltaProcessor {
             timestamp: new Date().toISOString(),
         };
     }
+    /**
+     * Check if item is within the watched path
+     */
+    isItemInScope(item, watchPath) {
+        if (!item.parentReference || !item.parentReference.path) {
+            return false;
+        }
+        // Extract relative path from parentReference.path
+        // Format: /drives/{drive-id}/root:/path/to/folder
+        const parts = item.parentReference.path.split('root:');
+        if (parts.length < 2) {
+            // Item is in root
+            // If watchPath is /, then it's a match. Otherwise no.
+            return watchPath === '/' || watchPath === '';
+        }
+        let relativePath = parts[1]; // /path/to/folder
+        // Ensure paths start with / for consistent comparison
+        if (!relativePath.startsWith('/')) {
+            relativePath = '/' + relativePath;
+        }
+        // Ensure watchPath starts with /
+        let normalizedWatchPath = watchPath;
+        if (!normalizedWatchPath.startsWith('/')) {
+            normalizedWatchPath = '/' + normalizedWatchPath;
+        }
+        // Check if relative path matches or is a subfolder
+        // 1. Exact match: relativePath === normalizedWatchPath
+        // 2. Subfolder: relativePath startsWith normalizedWatchPath + '/'
+        if (relativePath === normalizedWatchPath) {
+            return true;
+        }
+        if (relativePath.startsWith(normalizedWatchPath + '/')) {
+            return true;
+        }
+        return false;
+    }
     /**
      * Classify event as created vs updated
      *

package/dist/utils/WebhookHandler.js

@@ -80,7 +80,7 @@ class WebhookHandler {
         const expirationDateTime = new Date();
         expirationDateTime.setDate(expirationDateTime.getDate() + this.SUBSCRIPTION_DURATION_DAYS);
         const subscriptionRequest = {
-            changeType: 'created,updated',
+            changeType: 'updated',
             notificationUrl,
             resource: this.subscriptionResource,
             expirationDateTime: expirationDateTime.toISOString(),

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "n8n-nodes-onedrive-business",
-    "version": "1.0.0",
+    "version": "1.1.2",
     "description": "n8n custom node for OneDrive Business with robust deduplication and webhook triggers",
     "keywords": [
         "n8n",
@@ -23,7 +23,7 @@
     "main": "index.js",
     "scripts": {
         "build": "tsc && npm run copy-icons",
-        "copy-icons": "copyfiles -u 1 \"nodes/**/*.{png,svg}\" dist/",
+        "copy-icons": "copyfiles \"nodes/**/*.{png,svg}\" dist/",
         "dev": "tsc --watch",
         "format": "prettier nodes --write",
         "lint": "tslint -p tsconfig.json -c tslint.json",