n8n-nodes-postwire 0.2.1 → 0.2.3

package/dist/nodes/PostWire/PostWire.node.js

@@ -51,7 +51,7 @@ class PostWire {
                             name: 'Get Account',
                             value: 'me',
                             description: 'Plan, usage this month and connected accounts',
-                            action: 'Get the PostWire account',
+                            action: 'Get account',
                         },
                         {
                             name: 'Publish',
@@ -142,6 +142,18 @@ class PostWire {
                         show: { operation: ['generate', 'publish', 'generateAndPublish', 'schedule'] },
                     },
                     options: [
+                        {
+                            displayName: 'Media Type',
+                            name: 'mediaType',
+                            type: 'options',
+                            default: 'auto',
+                            options: [
+                                { name: 'Detect From the URL', value: 'auto' },
+                                { name: 'Video', value: 'video' },
+                                { name: 'Image', value: 'image' },
+                            ],
+                            description: 'Only needed when the link has no file extension — a signed CDN URL or a share link. Detection reads the extension and, when there is none, follows what the chosen networks require.',
+                        },
                         {
                             displayName: 'Brand',
                             name: 'brandId',
@@ -204,21 +216,40 @@ class PostWire {
                     });
                 }
                 // Media rules, checked before the request so the message names the fix rather than the symptom.
-                const isVideo = /\.(mp4|mov|webm|m3u8|avi|mkv|m4v)(\?|$)/i.test(mediaUrl);
+                //
+                // The classifier used to answer only video-or-photo, and called everything without a
+                // known video extension a photo. A signed CDN link, one ending in #t=30, or a share URL
+                // with no extension at all was therefore "a photo", and the check below then threw and
+                // stopped the whole workflow — for a video that would have published perfectly well.
+                // Three answers now, and an unknown one is never grounds for refusing to try: the
+                // network is the thing that gets to decide, not a regular expression over a filename.
+                const declared = opts.mediaType || 'auto';
+                const kind = !mediaUrl
+                    ? 'none'
+                    : declared === 'video' || declared === 'image'
+                        ? declared
+                        : /\.(mp4|mov|webm|m3u8|avi|mkv|m4v)(\?|#|$)/i.test(mediaUrl)
+                            ? 'video'
+                            : /\.(jpe?g|png|gif|webp|heic|heif|avif|bmp|tiff?)(\?|#|$)/i.test(mediaUrl)
+                                ? 'image'
+                                : 'unknown';
                 const blocked = platforms.filter((p) => {
                     const need = needOf(p);
                     if (!need)
                         return false;
                     if (need === 'video')
-                        return !isVideo;
-                    return !mediaUrl;
+                        return kind === 'image' || kind === 'none';
+                    return kind === 'none';
                 });
                 if (blocked.length) {
                     const wantsVideo = blocked.some((p) => needOf(p) === 'video');
                     throw new n8n_workflow_1.NodeOperationError(this.getNode(), `${blocked.join(' and ')} will not accept a post without ${wantsVideo ? 'a video' : 'an image or video'}. Set Media URL to a direct file link, or remove ${blocked.length > 1 ? 'those networks' : 'that network'} from the list.`, { itemIndex: i });
                 }
-                const photoUrl = mediaUrl && !isVideo ? mediaUrl : undefined;
-                const videoUrl = isVideo ? mediaUrl : undefined;
+                // An unrecognised link still has to go in one of the two fields the API has. Send it as the
+                // one the chosen networks need — that is the only reading of it that can succeed.
+                const treatAsVideo = kind === 'video' || (kind === 'unknown' && platforms.some((p) => needOf(p) === 'video'));
+                const videoUrl = mediaUrl && treatAsVideo ? mediaUrl : undefined;
+                const photoUrl = mediaUrl && !treatAsVideo ? mediaUrl : undefined;
                 let perPlatform;
                 if (operation === 'generate' || operation === 'generateAndPublish') {
                     const prompt = this.getNodeParameter('prompt', i);
@@ -293,8 +324,9 @@ class PostWire {
                 // Our own checks already throw NodeOperationError with a message that names the fix;
                 // re-wrapping those would bury it. Anything else is an HTTP failure, and n8n renders
                 // a NodeApiError with the node's context instead of a bare fetch stack.
-                if (error instanceof n8n_workflow_1.NodeOperationError)
-                    throw error;
+                if (error instanceof n8n_workflow_1.NodeOperationError) {
+                    throw new n8n_workflow_1.NodeOperationError(this.getNode(), error.message, { itemIndex: i });
+                }
                 throw new n8n_workflow_1.NodeApiError(this.getNode(), error, { itemIndex: i });
             }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "n8n-nodes-postwire",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "Publish one idea natively to TikTok, Instagram, YouTube, LinkedIn, X, Bluesky, Mastodon, Telegram, Discord and Reddit from n8n — a different post written for each network, via PostWire.",
   "keywords": [
     "n8n-community-node-package",