edsger 0.27.8 → 0.27.10

package/dist/services/checklist.js

@@ -1,119 +1,53 @@
 /**
  * Checklist service for pipeline integration
  */
+import { callMcpEndpoint } from '../api/mcp-client.js';
 /**
  * Fetch checklists for a specific phase
  */
 export async function getChecklistsForPhase(options, phase) {
     const { featureId } = options;
-    const mcpServerUrl = process.env.EDSGER_MCP_SERVER_URL;
-    const mcpToken = process.env.EDSGER_MCP_TOKEN;
     // Convert phase name from hyphen to underscore format for database compatibility
     // e.g., 'feature-analysis' -> 'feature_analysis'
     const dbPhase = phase.replace(/-/g, '_');
-    // Log the request details for debugging
-    const requestBody = {
-        jsonrpc: '2.0',
-        id: 1,
-        method: 'checklists/phase',
-        params: {
-            feature_id: featureId,
-            phase: dbPhase, // Use underscore format for database
-        },
-    };
     console.log(`🔍 Fetching checklists: phase="${phase}" (db: "${dbPhase}"), feature_id="${featureId}"`);
-    const response = await fetch(`${mcpServerUrl}/mcp`, {
-        method: 'POST',
-        headers: {
-            'Content-Type': 'application/json',
-            Authorization: `Bearer ${mcpToken}`,
-        },
-        body: JSON.stringify(requestBody),
+    const result = await callMcpEndpoint('checklists/phase', {
+        feature_id: featureId,
+        phase: dbPhase,
     });
-    if (!response.ok) {
-        const errorText = await response.text();
-        throw new Error(`Failed to fetch checklists for phase "${phase}": ${response.status} ${response.statusText}. Response: ${errorText}`);
-    }
-    const data = await response.json();
-    if (data.error) {
-        throw new Error(`MCP Error for phase "${phase}": ${data.error.message || JSON.stringify(data.error)}`);
-    }
     // Handle empty result gracefully
-    if (!data.result) {
+    if (!result) {
         return {
             phase,
             feature_id: featureId,
             checklists: [],
         };
     }
-    return data.result;
+    return result;
 }
 /**
  * Validate checklist completion for a phase
  */
 export async function validateChecklistsForPhase(options, phase) {
     const { featureId } = options;
-    const mcpServerUrl = process.env.EDSGER_MCP_SERVER_URL;
-    const mcpToken = process.env.EDSGER_MCP_TOKEN;
     // Convert phase name from hyphen to underscore format for database compatibility
     const dbPhase = phase.replace(/-/g, '_');
-    const response = await fetch(`${mcpServerUrl}/mcp`, {
-        method: 'POST',
-        headers: {
-            'Content-Type': 'application/json',
-            Authorization: `Bearer ${mcpToken}`,
-        },
-        body: JSON.stringify({
-            jsonrpc: '2.0',
-            id: 1,
-            method: 'checklists/validate',
-            params: {
-                feature_id: featureId,
-                phase: dbPhase, // Use underscore format for database
-            },
-        }),
+    const result = await callMcpEndpoint('checklists/validate', {
+        feature_id: featureId,
+        phase: dbPhase,
     });
-    if (!response.ok) {
-        throw new Error(`Failed to validate checklists for phase: ${response.statusText}`);
-    }
-    const data = await response.json();
-    if (data.error) {
-        throw new Error(`MCP Error: ${data.error.message}`);
-    }
-    return data.result;
+    return result;
 }
 /**
  * Create checklist results (mark as started/in-progress)
  */
 export async function createChecklistResult(options, checklistId, itemResults) {
     const { featureId } = options;
-    const mcpServerUrl = process.env.EDSGER_MCP_SERVER_URL;
-    const mcpToken = process.env.EDSGER_MCP_TOKEN;
-    const response = await fetch(`${mcpServerUrl}/mcp`, {
-        method: 'POST',
-        headers: {
-            'Content-Type': 'application/json',
-            Authorization: `Bearer ${mcpToken}`,
-        },
-        body: JSON.stringify({
-            jsonrpc: '2.0',
-            id: 1,
-            method: 'checklist_results/create',
-            params: {
-                feature_id: featureId,
-                checklist_id: checklistId,
-                item_results: itemResults,
-            },
-        }),
+    return await callMcpEndpoint('checklist_results/create', {
+        feature_id: featureId,
+        checklist_id: checklistId,
+        item_results: itemResults,
     });
-    if (!response.ok) {
-        throw new Error(`Failed to create checklist result: ${response.statusText}`);
-    }
-    const data = await response.json();
-    if (data.error) {
-        throw new Error(`MCP Error: ${data.error.message}`);
-    }
-    return data.result;
 }
 /**
  * Strictly validate that all required checklists have results
@@ -171,35 +105,13 @@ export async function validateRequiredChecklistResults(options, phase, verbose)
  */
 export async function createChecklistItemResult(options, checklistItemId, isPassed, value, notes) {
     const { featureId } = options;
-    const mcpServerUrl = process.env.EDSGER_MCP_SERVER_URL;
-    const mcpToken = process.env.EDSGER_MCP_TOKEN;
-    const response = await fetch(`${mcpServerUrl}/mcp`, {
-        method: 'POST',
-        headers: {
-            'Content-Type': 'application/json',
-            Authorization: `Bearer ${mcpToken}`,
-        },
-        body: JSON.stringify({
-            jsonrpc: '2.0',
-            id: 1,
-            method: 'checklist_item_results/create',
-            params: {
-                feature_id: featureId,
-                checklist_item_id: checklistItemId,
-                is_passed: isPassed,
-                value: value,
-                notes: notes,
-            },
-        }),
+    return await callMcpEndpoint('checklist_item_results/create', {
+        feature_id: featureId,
+        checklist_item_id: checklistItemId,
+        is_passed: isPassed,
+        value: value,
+        notes: notes,
     });
-    if (!response.ok) {
-        throw new Error(`Failed to create checklist item result: ${response.statusText}`);
-    }
-    const data = await response.json();
-    if (data.error) {
-        throw new Error(`MCP Error: ${data.error.message}`);
-    }
-    return data.result;
 }
 /**
  * Process checklist item results from phase JSON response and create MCP results

package/dist/utils/git-branch-manager.js

@@ -616,7 +616,7 @@ export async function syncFeatBranchWithMain(featBranch, githubToken, owner, rep
                 if (verbose) {
                     logInfo(`ℹ️  ${featBranch} branch does not exist, skipping sync`);
                 }
-                return true; // Not an error, just no feat branch yet
+                return false; // No feat branch yet, no sync needed (no force push required)
             }
             throw error;
         }

package/package.json

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