piece-signal-cli-rest-api 0.2.14 → 0.2.16

package/package.json

@@ -1,6 +1,10 @@
 {
   "name": "piece-signal-cli-rest-api",
-  "version": "0.2.14",
+  "version": "0.2.16",
+  "scripts": {
+    "build": "echo \"piece-signal-cli-rest-api: kein zusätzlicher Build-Schritt erforderlich (bereits als JS ausgeliefert)\"",
+    "publish:npm": "npm publish --access public"
+  },
   "dependencies": {
     "@sinclair/typebox": "0.34.11",
     "ai": "5.0.104",
@@ -26,4 +30,4 @@
   "types": "./src/index.d.ts",
   "main": "./src/index.js",
   "type": "commonjs"
-}
+}

package/src/lib/actions/request-approval-message.js

@@ -83,7 +83,12 @@ exports.requestApprovalMessage = (0, pieces_framework_1.createAction)({
                 const targetAuthor = formattedNumber;
                 const createdAt = Math.floor(Date.now() / 1000); // Unix timestamp in seconds
                 // Create store key using milliseconds for precise matching
-                const storeKey = `approval:${messageTimestampMs}:${targetAuthor}`;
+                // Store-Key mit Timestamp + eindeutiger Flow-Run-ID
+                // flowRunId ist pro Flow-Ausführung eindeutig, verhindert Kollisionen
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'request-approval-message.js:88',message:'Store-Key creation - checking context.run.id',data:{flowRunId:context.run.id,flowRunIdType:typeof context.run.id,messageTimestampMs,timestampSeconds,recipients:Array.isArray(recipients)?recipients.length:'not-array',recipientsType:typeof recipients},timestamp:Date.now(),hypothesisId:'A'})}).catch(()=>{});
+                // #endregion
+                const storeKey = `approval:${messageTimestampMs}:${context.run.id}`;
                 // DEBUG: Log storing approval mapping
                 console.log('[RequestApprovalMessage] DEBUG - Storing approval mapping:', {
                     storeKey,
@@ -117,9 +122,15 @@ exports.requestApprovalMessage = (0, pieces_framework_1.createAction)({
                 // DEBUG: Verify it was stored
                 const verifyMapping = yield context.store.get(storeKey, pieces_framework_1.StoreScope.PROJECT);
                 console.log('[RequestApprovalMessage] DEBUG - Verified stored mapping:', verifyMapping);
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'request-approval-message.js:118',message:'Mapping stored - verifying recipients format',data:{storeKey,recipients:verifyMapping?.recipients,recipientsType:typeof verifyMapping?.recipients,recipientsIsArray:Array.isArray(verifyMapping?.recipients),messageTimestampMs:verifyMapping?.messageTimestampMs},timestamp:Date.now(),hypothesisId:'C'})}).catch(()=>{});
+                // #endregion
                 // Add key to approval keys list
                 const keysListKey = 'approval:keys';
                 const existingKeys = (yield context.store.get(keysListKey, pieces_framework_1.StoreScope.PROJECT)) || [];
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'request-approval-message.js:123',message:'Adding key to approval keys list',data:{storeKey,existingKeysCount:existingKeys.length,keyAlreadyExists:existingKeys.includes(storeKey)},timestamp:Date.now(),hypothesisId:'D'})}).catch(()=>{});
+                // #endregion
                 if (!existingKeys.includes(storeKey)) {
                     existingKeys.push(storeKey);
                     yield context.store.put(keysListKey, existingKeys, pieces_framework_1.StoreScope.PROJECT);

package/src/lib/common/message-utils.js

@@ -9,6 +9,8 @@ const pieces_framework_1 = require("@activepieces/pieces-framework");
 const pieces_common_1 = require("@activepieces/pieces-common");
 const ws_1 = tslib_1.__importDefault(require("ws"));
 // Function to cleanup expired approval mappings
+// Diese Funktion ist die EINZIGE Stelle, an der entschieden wird, ob ein Approval abgelaufen ist.
+// Alle abgelaufenen Approvals werden hier gelöscht, bevor das Matching stattfindet.
 function cleanupExpiredApprovals(store) {
     return tslib_1.__awaiter(this, void 0, void 0, function* () {
         const keysListKey = 'approval:keys';
@@ -16,11 +18,19 @@ function cleanupExpiredApprovals(store) {
         const currentTimestamp = Math.floor(Date.now() / 1000);
         const validKeys = [];
         const keysToDelete = [];
+        // #region agent log
+        fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:19',message:'Cleanup started',data:{existingKeysCount:existingKeys.length,currentTimestamp},timestamp:Date.now(),hypothesisId:'E'})}).catch(()=>{});
+        // #endregion
         for (const key of existingKeys) {
             const mapping = yield store.get(key, pieces_framework_1.StoreScope.PROJECT);
             if (mapping) {
                 // Check if expired
-                if (currentTimestamp - mapping.createdAt > mapping.timeoutSeconds) {
+                const ageInSeconds = currentTimestamp - mapping.createdAt;
+                const isExpired = ageInSeconds > mapping.timeoutSeconds;
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:26',message:'Checking expiration',data:{key,ageInSeconds,timeoutSeconds:mapping.timeoutSeconds,isExpired,createdAt:mapping.createdAt},timestamp:Date.now(),hypothesisId:'E'})}).catch(()=>{});
+                // #endregion
+                if (isExpired) {
                     // Expired - delete it and flowRunId mapping
                     yield store.delete(key, pieces_framework_1.StoreScope.PROJECT);
                     const flowRunMappingKey = `approval:flowRun:${mapping.flowRunId}`;
@@ -40,6 +50,9 @@ function cleanupExpiredApprovals(store) {
         if (keysToDelete.length > 0) {
             yield store.put(keysListKey, validKeys, pieces_framework_1.StoreScope.PROJECT);
         }
+        // #region agent log
+        fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:42',message:'Cleanup completed',data:{validKeysCount:validKeys.length,deletedKeysCount:keysToDelete.length},timestamp:Date.now(),hypothesisId:'E'})}).catch(()=>{});
+        // #endregion
     });
 }
 // Function to automatically resume approval flows
@@ -67,10 +80,15 @@ function tryResumeApprovalFlow(message, store, apiUrl) {
             isGroupMessage: isGroupMessage,
         });
         // Cleanup expired approvals
+        // WICHTIG: cleanupExpiredApprovals ist die EINZIGE Stelle für Timeout-Entscheidungen.
+        // Alle nachfolgenden Matching-Schritte arbeiten nur auf bereits gefilterten (nicht abgelaufenen) Approvals.
         yield cleanupExpiredApprovals(store);
         const keysListKey = 'approval:keys';
         const existingKeys = (yield store.get(keysListKey, pieces_framework_1.StoreScope.PROJECT)) || [];
         const currentTimestamp = Math.floor(Date.now() / 1000);
+        // #region agent log
+        fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:76',message:'After cleanup - checking available keys',data:{existingKeysCount:existingKeys.length,existingKeys:existingKeys.slice(0,5),isReaction,isTextReply,isGroupMessage},timestamp:Date.now(),hypothesisId:'D'})}).catch(()=>{});
+        // #endregion
         // Try to find matching approval
         let mapping = null;
         let storeKey = null;
@@ -83,19 +101,87 @@ function tryResumeApprovalFlow(message, store, apiUrl) {
             // signal-cli returns timestamps in SECONDS, convert to milliseconds for matching
             const targetTimestampSeconds = reaction.targetSentTimestamp || reaction.targetTimestamp;
             const targetTimestampMs = targetTimestampSeconds * 1000;
-            const targetAuthor = reaction.targetAuthor;
             const reactionEmoji = reaction.emoji;
-            storeKey = `approval:${targetTimestampMs}:${targetAuthor}`;
-            mapping = yield store.get(storeKey, pieces_framework_1.StoreScope.PROJECT);
-            if (mapping) {
-                const acceptModes = mapping.acceptReactionModes || [];
+            const messageSource = message.envelope.source;
+            
+            console.log('[ReceiveMessages] DEBUG - Processing emoji reaction:', {
+                targetTimestampMs,
+                targetTimestampSeconds,
+                reactionEmoji,
+                messageSource,
+                isGroupMessage,
+            });
+            // #region agent log
+            fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:93',message:'Emoji reaction - starting matching loop',data:{targetTimestampMs,targetTimestampSeconds,existingKeysCount:existingKeys.length,messageSource,isGroupMessage},timestamp:Date.now(),hypothesisId:'B'})}).catch(()=>{});
+            // #endregion
+            
+            // Alle Approvals durchsuchen: Exakter Timestamp-Match + Empfänger-Validierung
+            for (const key of existingKeys) {
+                const candidateMapping = yield store.get(key, pieces_framework_1.StoreScope.PROJECT);
+                if (!candidateMapping) continue;
+                
+                // Exakter Timestamp-Match (beide vom Signal-Netzwerk, müssen identisch sein)
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:102',message:'Checking timestamp match',data:{key,candidateTimestampMs:candidateMapping.messageTimestampMs,targetTimestampMs,timestampMatch:candidateMapping.messageTimestampMs===targetTimestampMs},timestamp:Date.now(),hypothesisId:'B'})}).catch(()=>{});
+                // #endregion
+                if (candidateMapping.messageTimestampMs !== targetTimestampMs) continue;
+                
+                console.log('[ReceiveMessages] DEBUG - Timestamp match found, checking recipient:', {
+                    key,
+                    candidateTimestampMs: candidateMapping.messageTimestampMs,
+                    recipients: candidateMapping.recipients,
+                });
+                
+                // Empfänger-Validierung (1:1 vs. Gruppe)
+                const recipients = candidateMapping.recipients || [];
+                let isValidRecipient = false;
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:110',message:'Checking recipient validation',data:{key,recipients,recipientsType:typeof recipients,recipientsIsArray:Array.isArray(recipients),isGroupMessage,messageSource},timestamp:Date.now(),hypothesisId:'C'})}).catch(()=>{});
+                // #endregion
+                
+                if (isGroupMessage) {
+                    // Bei Gruppen: Prüfe Gruppen-ID
+                    const groupId = dataMessage.groupInfo.groupId;
+                    isValidRecipient = Array.isArray(recipients) && recipients.includes(groupId);
+                    console.log('[ReceiveMessages] DEBUG - Group recipient check:', {
+                        groupId,
+                        isValidRecipient,
+                    });
+                    // #region agent log
+                    fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:117',message:'Group recipient validation result',data:{key,groupId,recipients,isValidRecipient},timestamp:Date.now(),hypothesisId:'C'})}).catch(()=>{});
+                    // #endregion
+                } else {
+                    // Bei 1:1: Prüfe, ob Absender in recipients-Liste steht
+                    isValidRecipient = Array.isArray(recipients) && recipients.includes(messageSource);
+                    console.log('[ReceiveMessages] DEBUG - 1:1 recipient check:', {
+                        messageSource,
+                        isValidRecipient,
+                    });
+                    // #region agent log
+                    fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:125',message:'1:1 recipient validation result',data:{key,messageSource,recipients,isValidRecipient},timestamp:Date.now(),hypothesisId:'C'})}).catch(()=>{});
+                    // #endregion
+                }
+                
+                if (!isValidRecipient) continue;
+                
+                // Prüfe, ob Emoji-Modus erlaubt ist
+                const acceptModes = candidateMapping.acceptReactionModes || [];
                 if (acceptModes.includes('emoji')) {
-                    responseContent = reactionEmoji; // Return the emoji directly
+                    mapping = candidateMapping;
+                    storeKey = key;
+                    responseContent = reactionEmoji;
                     reactionType = 'emoji';
-                }
-                else {
-                    // Mapping found but emoji mode not allowed
-                    mapping = null;
+                    console.log('[ReceiveMessages] DEBUG - Emoji reaction match found:', {
+                        key,
+                        flowRunId: mapping.flowRunId,
+                        reactionEmoji,
+                    });
+                    break; // Gefunden, Suche beenden
+                } else {
+                    console.log('[ReceiveMessages] DEBUG - Emoji mode not allowed for this approval:', {
+                        key,
+                        acceptModes,
+                    });
                 }
             }
         }
@@ -105,20 +191,87 @@ function tryResumeApprovalFlow(message, store, apiUrl) {
             // signal-cli returns timestamps in SECONDS, convert to milliseconds for matching
             const quoteTimestampSeconds = quote.id || quote.timestamp;
             const quoteTimestampMs = quoteTimestampSeconds * 1000;
-            const quoteAuthor = quote.author;
             const replyText = dataMessage.message || '';
-            // Try to find mapping by quote timestamp
-            storeKey = `approval:${quoteTimestampMs}:${quoteAuthor}`;
-            mapping = yield store.get(storeKey, pieces_framework_1.StoreScope.PROJECT);
-            if (mapping) {
-                const acceptModes = mapping.acceptReactionModes || [];
+            const messageSource = message.envelope.source;
+            
+            console.log('[ReceiveMessages] DEBUG - Processing text reply (quote):', {
+                quoteTimestampMs,
+                quoteTimestampSeconds,
+                replyText: replyText.substring(0, 50), // First 50 chars for logging
+                messageSource,
+                isGroupMessage,
+            });
+            // #region agent log
+            fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:145',message:'Text reply - starting matching loop',data:{quoteTimestampMs,quoteTimestampSeconds,existingKeysCount:existingKeys.length,messageSource,isGroupMessage},timestamp:Date.now(),hypothesisId:'B'})}).catch(()=>{});
+            // #endregion
+            
+            // Alle Approvals durchsuchen: Exakter Timestamp-Match + Empfänger-Validierung
+            for (const key of existingKeys) {
+                const candidateMapping = yield store.get(key, pieces_framework_1.StoreScope.PROJECT);
+                if (!candidateMapping) continue;
+                
+                // Exakter Timestamp-Match (beide vom Signal-Netzwerk, müssen identisch sein)
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:154',message:'Checking timestamp match (quote)',data:{key,candidateTimestampMs:candidateMapping.messageTimestampMs,quoteTimestampMs,timestampMatch:candidateMapping.messageTimestampMs===quoteTimestampMs},timestamp:Date.now(),hypothesisId:'B'})}).catch(()=>{});
+                // #endregion
+                if (candidateMapping.messageTimestampMs !== quoteTimestampMs) continue;
+                
+                console.log('[ReceiveMessages] DEBUG - Timestamp match found, checking recipient:', {
+                    key,
+                    candidateTimestampMs: candidateMapping.messageTimestampMs,
+                    recipients: candidateMapping.recipients,
+                });
+                
+                // Empfänger-Validierung (1:1 vs. Gruppe)
+                const recipients = candidateMapping.recipients || [];
+                let isValidRecipient = false;
+                // #region agent log
+                fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:162',message:'Checking recipient validation (quote)',data:{key,recipients,recipientsType:typeof recipients,recipientsIsArray:Array.isArray(recipients),isGroupMessage,messageSource},timestamp:Date.now(),hypothesisId:'C'})}).catch(()=>{});
+                // #endregion
+                
+                if (isGroupMessage) {
+                    // Bei Gruppen: Prüfe Gruppen-ID
+                    const groupId = dataMessage.groupInfo.groupId;
+                    isValidRecipient = Array.isArray(recipients) && recipients.includes(groupId);
+                    console.log('[ReceiveMessages] DEBUG - Group recipient check:', {
+                        groupId,
+                        isValidRecipient,
+                    });
+                    // #region agent log
+                    fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:169',message:'Group recipient validation result (quote)',data:{key,groupId,recipients,isValidRecipient},timestamp:Date.now(),hypothesisId:'C'})}).catch(()=>{});
+                    // #endregion
+                } else {
+                    // Bei 1:1: Prüfe, ob Absender in recipients-Liste steht
+                    isValidRecipient = Array.isArray(recipients) && recipients.includes(messageSource);
+                    console.log('[ReceiveMessages] DEBUG - 1:1 recipient check:', {
+                        messageSource,
+                        isValidRecipient,
+                    });
+                    // #region agent log
+                    fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:177',message:'1:1 recipient validation result (quote)',data:{key,messageSource,recipients,isValidRecipient},timestamp:Date.now(),hypothesisId:'C'})}).catch(()=>{});
+                    // #endregion
+                }
+                
+                if (!isValidRecipient) continue;
+                
+                // Prüfe, ob Text-Modus erlaubt ist
+                const acceptModes = candidateMapping.acceptReactionModes || [];
                 if (acceptModes.includes('text')) {
-                    responseContent = replyText; // Return the text directly
+                    mapping = candidateMapping;
+                    storeKey = key;
+                    responseContent = replyText;
                     reactionType = 'text';
-                }
-                else {
-                    // Mapping found but text mode not allowed
-                    mapping = null;
+                    console.log('[ReceiveMessages] DEBUG - Text reply match found:', {
+                        key,
+                        flowRunId: mapping.flowRunId,
+                        replyText: replyText.substring(0, 50),
+                    });
+                    break; // Gefunden, Suche beenden
+                } else {
+                    console.log('[ReceiveMessages] DEBUG - Text mode not allowed for this approval:', {
+                        key,
+                        acceptModes,
+                    });
                 }
             }
         }
@@ -127,17 +280,17 @@ function tryResumeApprovalFlow(message, store, apiUrl) {
             const messageText = dataMessage.message || '';
             const messageSource = message.envelope.source;
             
+            console.log('[ReceiveMessages] DEBUG - Processing direct message:', {
+                messageText: messageText.substring(0, 50), // First 50 chars for logging
+                messageSource,
+            });
+            
             // Iterate through all approval keys to find a match
+            // Keine Timeout-Prüfung mehr - cleanupExpiredApprovals hat bereits alle abgelaufenen entfernt
             for (const key of existingKeys) {
                 const candidateMapping = yield store.get(key, pieces_framework_1.StoreScope.PROJECT);
                 if (!candidateMapping) continue;
                 
-                // Check if expired
-                const ageInSeconds = currentTimestamp - candidateMapping.createdAt;
-                if (ageInSeconds > candidateMapping.timeoutSeconds) {
-                    continue; // Skip expired mappings
-                }
-                
                 // Check if direct mode is enabled
                 const acceptModes = candidateMapping.acceptReactionModes || [];
                 if (!acceptModes.includes('direct')) {
@@ -152,11 +305,19 @@ function tryResumeApprovalFlow(message, store, apiUrl) {
                     storeKey = key;
                     responseContent = messageText; // Use the message text as response
                     reactionType = 'direct';
+                    console.log('[ReceiveMessages] DEBUG - Direct message match found:', {
+                        key,
+                        flowRunId: mapping.flowRunId,
+                        messageText: messageText.substring(0, 50),
+                    });
                     break; // Found a match, stop searching
                 }
             }
         }
         // If no matching approval found, return
+        // #region agent log
+        fetch('http://10.3.0.249:7243/ingest/103f08fb-273f-440d-bbe5-4f1e30168ab7',{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify({location:'message-utils.js:200',message:'Matching result check',data:{mappingFound:!!mapping,responseContent:!!responseContent,reactionType,existingKeysCount:existingKeys.length},timestamp:Date.now(),hypothesisId:'D'})}).catch(()=>{});
+        // #endregion
         if (!mapping || !responseContent) {
             return { resumed: false };
         }
@@ -168,19 +329,7 @@ function tryResumeApprovalFlow(message, store, apiUrl) {
             reactionType,
             responseContent,
         });
-        // Check if expired
-        const ageInSeconds = currentTimestamp - mapping.createdAt;
-        const isExpired = ageInSeconds > mapping.timeoutSeconds;
-        if (isExpired) {
-            // Expired - delete and return
-            yield store.delete(storeKey, pieces_framework_1.StoreScope.PROJECT);
-            const flowRunMappingKey = `approval:flowRun:${mapping.flowRunId}`;
-            yield store.delete(flowRunMappingKey, pieces_framework_1.StoreScope.PROJECT);
-            const updatedKeys = existingKeys.filter(key => key !== storeKey);
-            yield store.put(keysListKey, updatedKeys, pieces_framework_1.StoreScope.PROJECT);
-            console.log('[ReceiveMessages] DEBUG - Approval expired');
-            return { resumed: false };
-        }
+        // Keine Timeout-Prüfung mehr - cleanupExpiredApprovals hat bereits alle abgelaufenen entfernt
         // Resume the flow with responseContent
         const resumeUrl = `${apiUrl}v1/flow-runs/${mapping.flowRunId}/requests/${mapping.requestId}?responseContent=${encodeURIComponent(responseContent)}&reactionType=${reactionType}&responder=${encodeURIComponent(responder || '')}`;
         console.log('[ReceiveMessages] DEBUG - Attempting resume:', {