edsger 0.10.0 → 0.10.2

package/.claude/settings.local.json

@@ -5,7 +5,16 @@
       "Bash(npm run build)",
       "Bash(node:*)",
       "Bash(git add:*)",
-      "Bash(git commit:*)"
+      "Bash(git commit:*)",
+      "Bash(ls:*)",
+      "Bash(cat:*)",
+      "Bash(npm run typecheck:*)",
+      "Bash(git diff:*)",
+      "WebSearch",
+      "WebFetch(domain:supabase.com)",
+      "Bash(npm install:*)",
+      "Bash(grep:*)",
+      "Bash(npx supabase gen types typescript --help:*)"
     ],
     "deny": [],
     "ask": []

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

@@ -21,12 +21,16 @@ export async function checkApprovalBeforePhaseExecution(featureId, verbose = fal
         const productId = feature.product_id;
         if (verbose) {
             logInfo(`🔍 Checking approval for current status: ${currentStatus}`);
+            logInfo(`📦 Product ID: ${productId}`);
         }
         // 2. Check if current status requires approval
         const requiresApprovalResult = (await callMcpEndpoint('approvals/requires_approval', {
             product_id: productId,
             feature_status: currentStatus,
         }));
+        if (verbose) {
+            logInfo(`📋 MCP requires_approval response: ${JSON.stringify(requiresApprovalResult)}`);
+        }
         const requiresApproval = requiresApprovalResult?.requires_approval === true;
         if (!requiresApproval) {
             if (verbose) {
@@ -106,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) {
@@ -117,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,
@@ -129,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/dist/api/github.js

@@ -89,7 +89,8 @@ export async function getGitHubConfig(featureId, verbose) {
         }
         return {
             configured: false,
-            message: result.message || 'GitHub not configured for this product. Please configure a developer with GitHub App installation in the product settings.',
+            message: result.message ||
+                'GitHub not configured for this product. Please configure a developer with GitHub App installation in the product settings.',
         };
     }
     catch (error) {

package/dist/phases/pull-request/handler.js

@@ -74,7 +74,10 @@ export async function handlePullRequestCreation({ featureId, results, testingRes
     }
     // Get GitHub configuration from product developer settings
     const githubConfig = await getGitHubConfig(featureId, verbose);
-    if (!githubConfig.configured || !githubConfig.token || !githubConfig.owner || !githubConfig.repo) {
+    if (!githubConfig.configured ||
+        !githubConfig.token ||
+        !githubConfig.owner ||
+        !githubConfig.repo) {
         if (verbose) {
             console.log('⚠️ GitHub configuration not found. Skipping pull request creation.');
             console.log(`   ${githubConfig.message || 'Please configure a developer with GitHub App installation in the product settings.'}`);
@@ -92,7 +95,7 @@ export async function handlePullRequestCreation({ featureId, results, testingRes
         }, verbose);
         return false;
     }
-    const { token: githubToken, owner: githubOwner, repo: githubRepo } = githubConfig;
+    const { token: githubToken, owner: githubOwner, repo: githubRepo, } = githubConfig;
     if (verbose) {
         console.log(`📌 Using GitHub config from product developer: ${githubOwner}/${githubRepo}`);
     }

package/package.json

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