edsger 0.10.1 → 0.10.3

package/.claude/settings.local.json

@@ -14,7 +14,13 @@
       "WebFetch(domain:supabase.com)",
       "Bash(npm install:*)",
       "Bash(grep:*)",
-      "Bash(npx supabase gen types typescript --help:*)"
+      "Bash(npx supabase gen types typescript --help:*)",
+      "Bash(git -C /Users/steven/development/edsger status)",
+      "Bash(git -C /Users/steven/development/edsger diff)",
+      "Bash(git -C /Users/steven/development/edsger log --oneline -5)",
+      "Bash(git -C /Users/steven/development/edsger add supabase/migrations/20251231000000_drop_unused_views.sql)",
+      "Bash(git -C /Users/steven/development/edsger commit -m \"$\\(cat <<''EOF''\nchore: drop unused database views\n\nRemove test_report_summary and user_stories_with_context views that are defined but never used in the application.\n\n🤖 Generated with [Claude Code]\\(https://claude.com/claude-code\\)\n\nCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>\nEOF\n\\)\")",
+      "Bash(git -C /Users/steven/development/edsger commit -m \"$\\(cat <<''EOF''\nchore: drop unused database views\n\nRemove test_report_summary and user_stories_with_context views\nthat are defined but never used in the application.\n\n🤖 Generated with [Claude Code]\\(https://claude.com/claude-code\\)\n\nCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>\nEOF\n\\)\")"
     ],
     "deny": [],
     "ask": []

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.3",
   "type": "module",
   "bin": {
     "edsger": "dist/index.js"