edsger 0.10.1 → 0.10.2

package/dist/api/features/approval-checker.js

@@ -110,7 +110,7 @@ export async function checkApprovalBeforePhaseExecution(featureId, verbose = fal
     }
 }
 /**
- * Create an approval request and send notification emails
+ * Create an approval request (emails are sent automatically by the MCP handler)
  * This is for the CURRENT status, not the next status
  */
 async function createApprovalRequestWithEmail(featureId, productId, currentStatus, verbose = false) {
@@ -121,6 +121,7 @@ async function createApprovalRequestWithEmail(featureId, productId, currentStatu
         // Create approval request via MCP endpoint
         // Note: We're requesting approval for the current status
         // The approval is asking: "Can we proceed from this status?"
+        // Email notifications are sent automatically by the MCP handler
         const result = (await callMcpEndpoint('approvals/create', {
             feature_id: featureId,
             requested_status: currentStatus,
@@ -133,30 +134,16 @@ async function createApprovalRequestWithEmail(featureId, productId, currentStatu
         }
         if (verbose) {
             logInfo(`✅ Approval request created: ${approvalId}`);
-        }
-        // Send approval emails via edge function
-        try {
-            if (verbose) {
-                logInfo('Sending approval notification emails...');
-            }
-            const emailResult = (await callMcpEndpoint('approvals/send_email', {
-                approval_id: approvalId,
-            }));
-            if (emailResult?.emails_sent) {
-                const sentCount = emailResult.emails_sent.filter((e) => e.status === 'sent').length;
-                if (verbose) {
-                    logInfo(`✅ Sent approval emails to ${sentCount} assignee(s)`);
+            // Log email results if available
+            if (result?.emails_sent) {
+                const sentCount = result.emails_sent.filter((e) => e.status === 'sent').length;
+                const failedCount = result.emails_sent.filter((e) => e.status !== 'sent').length;
+                logInfo(`📧 Sent approval emails to ${sentCount} assignee(s)`);
+                if (failedCount > 0) {
+                    logWarning(`⚠️ Failed to send ${failedCount} email(s)`);
                 }
             }
         }
-        catch (emailError) {
-            // Don't fail the whole process if email fails
-            const errorMessage = emailError instanceof Error ? emailError.message : String(emailError);
-            logWarning(`Failed to send approval emails: ${errorMessage}`);
-            if (verbose) {
-                logInfo('⚠️ Approval created but email notification failed');
-            }
-        }
         return approvalId;
     }
     catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edsger",
-  "version": "0.10.1",
+  "version": "0.10.2",
   "type": "module",
   "bin": {
     "edsger": "dist/index.js"