edsger 0.2.7 → 0.2.8

package/dist/phases/technical-design/analyzer-helpers.d.ts

@@ -1,6 +1,7 @@
 import { EdsgerConfig } from '../../types/index.js';
 import { ChecklistPhaseContext } from '../../services/checklist.js';
 import { type ChecklistVerificationResult } from '../technical-design-verification/index.js';
+import { FeedbacksContext } from '../../services/feedbacks.js';
 export interface TechnicalDesignResult {
     featureId: string;
     technicalDesign: string | null;
@@ -21,7 +22,7 @@ interface VerificationCycleResult {
 /**
  * Perform verification and determine if iteration should continue
  */
-export declare function performVerificationCycle(technicalDesign: string, checklistContext: ChecklistPhaseContext | null, featureId: string, featureName: string, featureDescription: string | undefined, config: EdsgerConfig, currentIteration: number, maxIterations: number, mcpServerUrl: string, mcpToken: string, verbose?: boolean): Promise<VerificationCycleResult>;
+export declare function performVerificationCycle(technicalDesign: string, checklistContext: ChecklistPhaseContext | null, feedbacksContext: FeedbacksContext | null, featureId: string, featureName: string, featureDescription: string | undefined, config: EdsgerConfig, currentIteration: number, maxIterations: number, mcpServerUrl: string, mcpToken: string, verbose?: boolean): Promise<VerificationCycleResult>;
 /**
  * Build successful design result
  */

package/dist/phases/technical-design/analyzer-helpers.js

@@ -6,7 +6,7 @@ import { logFeatureVerificationEvent } from '../../services/audit-logs.js';
 /**
  * Perform verification and determine if iteration should continue
  */
-export async function performVerificationCycle(technicalDesign, checklistContext, featureId, featureName, featureDescription, config, currentIteration, maxIterations, mcpServerUrl, mcpToken, verbose) {
+export async function performVerificationCycle(technicalDesign, checklistContext, feedbacksContext, featureId, featureName, featureDescription, config, currentIteration, maxIterations, mcpServerUrl, mcpToken, verbose) {
     // No verification needed if no checklist context
     if (!checklistContext || checklistContext.checklists.length === 0) {
         return { passed: true, verificationResult: null };
@@ -16,16 +16,24 @@ export async function performVerificationCycle(technicalDesign, checklistContext
     }
     const verificationResult = await verifyTechnicalDesignCompliance({
         checklistContext,
+        feedbacksContext,
         featureId,
         featureName,
         featureDescription,
         technicalDesign,
         verbose,
     }, config);
-    // Verification passed
-    if (verificationResult.rejected_count === 0) {
+    // Verification passed (both checklists and feedbacks)
+    const feedbacksPassed = !verificationResult.feedbacks_rejected_count ||
+        verificationResult.feedbacks_rejected_count === 0;
+    const checklistsPassed = verificationResult.rejected_count === 0;
+    if (checklistsPassed && feedbacksPassed) {
         if (verbose) {
             logInfo(`✅ Checklist verification passed: ${verificationResult.confirmed_count} confirmed, ${verificationResult.uncertain_count} uncertain`);
+            if (verificationResult.total_feedbacks &&
+                verificationResult.total_feedbacks > 0) {
+                logInfo(`✅ Feedback verification passed: ${verificationResult.feedbacks_confirmed_count || 0} confirmed, ${verificationResult.feedbacks_uncertain_count || 0} uncertain`);
+            }
         }
         // Log verification success
         await logFeatureVerificationEvent(mcpServerUrl, mcpToken, {
@@ -51,7 +59,14 @@ export async function performVerificationCycle(technicalDesign, checklistContext
         return { passed: true, verificationResult };
     }
     // Verification failed
-    logError(`❌ Iteration ${currentIteration}: Checklist verification FAILED - ${verificationResult.rejected_count} items rejected, ${verificationResult.uncertain_count} uncertain`);
+    const errorMessages = [];
+    if (!checklistsPassed) {
+        errorMessages.push(`${verificationResult.rejected_count} checklist items rejected, ${verificationResult.uncertain_count} uncertain`);
+    }
+    if (!feedbacksPassed && verificationResult.feedbacks_rejected_count) {
+        errorMessages.push(`${verificationResult.feedbacks_rejected_count} feedbacks not addressed, ${verificationResult.feedbacks_uncertain_count || 0} uncertain`);
+    }
+    logError(`❌ Iteration ${currentIteration}: Verification FAILED - ${errorMessages.join('; ')}`);
     // Log verification failure with improvement suggestions
     await logFeatureVerificationEvent(mcpServerUrl, mcpToken, {
         featureId,
@@ -78,6 +93,28 @@ export async function performVerificationCycle(technicalDesign, checklistContext
                 concerns: item.concerns || [],
                 improvement_suggestions: item.improvement_suggestions || [],
             })),
+            // Include feedback verification results
+            feedbacks_confirmed_count: verificationResult.feedbacks_confirmed_count || 0,
+            feedbacks_rejected_count: verificationResult.feedbacks_rejected_count || 0,
+            feedbacks_uncertain_count: verificationResult.feedbacks_uncertain_count || 0,
+            rejected_feedbacks: verificationResult.feedback_verifications
+                ?.filter((fb) => fb.verification_status === 'rejected')
+                .map((fb) => ({
+                feedback_id: fb.feedback_id,
+                feedback_title: fb.feedback_title,
+                reason: fb.verification_reason || 'No reason provided',
+                concerns: fb.concerns || [],
+                improvement_suggestions: fb.improvement_suggestions || [],
+            })) || [],
+            uncertain_feedbacks: verificationResult.feedback_verifications
+                ?.filter((fb) => fb.verification_status === 'uncertain')
+                .map((fb) => ({
+                feedback_id: fb.feedback_id,
+                feedback_title: fb.feedback_title,
+                reason: fb.verification_reason || 'No reason provided',
+                concerns: fb.concerns || [],
+                improvement_suggestions: fb.improvement_suggestions || [],
+            })) || [],
             summary: verificationResult.summary,
             overall_suggestions: verificationResult.overall_suggestions || [],
         },

package/dist/phases/technical-design/analyzer.js

@@ -89,7 +89,7 @@ export const generateTechnicalDesign = async (options, config, checklistContext)
             }
             await saveTechnicalDesign(mcpServerUrl, mcpToken, featureId, structuredDesignResult.technical_design, verbose);
             // Perform verification cycle
-            const verificationCycle = await performVerificationCycle(structuredDesignResult.technical_design, checklistContext || null, featureId, context.featureName, context.featureDescription, config, currentIteration, maxIterations, mcpServerUrl, mcpToken, verbose);
+            const verificationCycle = await performVerificationCycle(structuredDesignResult.technical_design, checklistContext || null, context.feedbacksContext, featureId, context.featureName, context.featureDescription, config, currentIteration, maxIterations, mcpServerUrl, mcpToken, verbose);
             verificationResult = verificationCycle.verificationResult;
             // If verification passed, exit
             if (verificationCycle.passed) {
@@ -158,6 +158,7 @@ async function prepareDesignContext(mcpServerUrl, mcpToken, featureId, checklist
     }
     let finalContextInfo = contextInfo;
     let hasFeedbacks = false;
+    let feedbacksContext = null;
     // Check if there's existing technical design
     const existingTechnicalDesign = context.feature.technical_design;
     const hasExistingDesign = !!existingTechnicalDesign && existingTechnicalDesign.trim().length > 0;
@@ -167,7 +168,7 @@ async function prepareDesignContext(mcpServerUrl, mcpToken, featureId, checklist
     }
     // Add feedbacks context to the design prompt
     try {
-        const feedbacksContext = await getFeedbacksForPhase({ featureId, mcpServerUrl, mcpToken, verbose }, 'technical-design');
+        feedbacksContext = await getFeedbacksForPhase({ featureId, mcpServerUrl, mcpToken, verbose }, 'technical-design');
         if (feedbacksContext.feedbacks.length > 0) {
             hasFeedbacks = true;
             const feedbacksInfo = formatFeedbacksForContext(feedbacksContext);
@@ -193,14 +194,25 @@ async function prepareDesignContext(mcpServerUrl, mcpToken, featureId, checklist
     }
     // Decide on prompt mode based on existing design and feedbacks
     const isIncrementalUpdate = hasExistingDesign && hasFeedbacks;
-    if (isIncrementalUpdate && verbose) {
-        logInfo('🔄 Mode: Incremental Update (existing design + feedbacks)');
-    }
-    else if (hasExistingDesign && verbose) {
-        logInfo('🔄 Mode: Full Redesign (existing design but no feedbacks)');
-    }
-    else if (verbose) {
-        logInfo('✨ Mode: New Design (no existing design)');
+    if (verbose) {
+        logInfo('\n📋 Design Mode Decision:');
+        logInfo(`   - Has existing design: ${hasExistingDesign}`);
+        logInfo(`   - Has feedbacks: ${hasFeedbacks} (${feedbacksContext?.feedbacks.length || 0} feedbacks)`);
+        logInfo(`   - Mode: ${isIncrementalUpdate ? '🔄 Incremental Update' : hasExistingDesign ? '🔄 Full Redesign' : '✨ New Design'}`);
+        if (isIncrementalUpdate && feedbacksContext) {
+            logInfo('\n🎯 Feedbacks that will be addressed:');
+            feedbacksContext.feedbacks.forEach((fb, idx) => {
+                const priorityBadge = fb.priority >= 9
+                    ? '🔴'
+                    : fb.priority >= 7
+                        ? '🟠'
+                        : fb.priority >= 5
+                            ? '🟡'
+                            : '🟢';
+                logInfo(`   ${idx + 1}. [${fb.feedback_type}] ${fb.title} ${priorityBadge}`);
+                logInfo(`      ${fb.content.substring(0, 100)}${fb.content.length > 100 ? '...' : ''}`);
+            });
+        }
     }
     const designPrompt = createTechnicalDesignPromptWithContext(featureId, finalContextInfo, existingTechnicalDesign, isIncrementalUpdate);
     return {
@@ -209,6 +221,7 @@ async function prepareDesignContext(mcpServerUrl, mcpToken, featureId, checklist
         designPrompt,
         hasExistingDesign,
         hasFeedbacks,
+        feedbacksContext,
     };
 }
 /**

package/dist/phases/technical-design-verification/verifier.d.ts

@@ -4,6 +4,7 @@
  */
 import { EdsgerConfig } from '../../types/index.js';
 import { ChecklistPhaseContext } from '../../services/checklist.js';
+import { FeedbacksContext } from '../../services/feedbacks.js';
 export interface ChecklistItemVerificationResult {
     checklist_item_id: string;
     is_satisfied: boolean;
@@ -12,6 +13,15 @@ export interface ChecklistItemVerificationResult {
     concerns?: string[];
     improvement_suggestions?: string[];
 }
+export interface FeedbackVerificationResult {
+    feedback_id: string;
+    feedback_title: string;
+    is_addressed: boolean;
+    verification_status: 'confirmed' | 'rejected' | 'uncertain';
+    verification_reason: string;
+    concerns?: string[];
+    improvement_suggestions?: string[];
+}
 export interface ChecklistVerificationResult {
     all_verified: boolean;
     total_items: number;
@@ -19,11 +29,18 @@ export interface ChecklistVerificationResult {
     rejected_count: number;
     uncertain_count: number;
     item_verifications: ChecklistItemVerificationResult[];
+    feedbacks_verified?: boolean;
+    total_feedbacks?: number;
+    feedbacks_confirmed_count?: number;
+    feedbacks_rejected_count?: number;
+    feedbacks_uncertain_count?: number;
+    feedback_verifications?: FeedbackVerificationResult[];
     summary: string;
     overall_suggestions?: string[];
 }
 export interface VerifyTechnicalDesignOptions {
     checklistContext: ChecklistPhaseContext;
+    feedbacksContext?: FeedbacksContext | null;
     featureId: string;
     featureName: string;
     featureDescription?: string;

package/dist/phases/technical-design-verification/verifier.js

@@ -18,16 +18,22 @@ async function* prompt(verificationPrompt) {
  * Verify checklist compliance for technical design using an independent AI agent
  */
 export async function verifyTechnicalDesignCompliance(options, config) {
-    const { checklistContext, featureId, featureName, featureDescription, technicalDesign, verbose, } = options;
+    const { checklistContext, feedbacksContext, featureId, featureName, featureDescription, technicalDesign, verbose, } = options;
     const totalChecklistItems = checklistContext.checklists.reduce((sum, checklist) => sum + checklist.items.length, 0);
+    const totalFeedbacks = feedbacksContext?.feedbacks?.length || 0;
+    const hasFeedbacks = totalFeedbacks > 0;
     if (verbose) {
         logInfo('🔍 Starting technical design verification...');
         logInfo(`   Verifying ${totalChecklistItems} checklist items`);
+        if (hasFeedbacks) {
+            logInfo(`   Verifying ${totalFeedbacks} human feedbacks`);
+        }
     }
     try {
         const systemPrompt = createTechnicalDesignVerificationSystemPrompt(config);
         const verificationPrompt = createTechnicalDesignVerificationPrompt({
             checklistContext,
+            feedbacksContext: feedbacksContext || undefined,
             featureId,
             featureName,
             featureDescription,
@@ -100,13 +106,20 @@ export async function verifyTechnicalDesignCompliance(options, config) {
         }
         if (verbose) {
             logInfo('\n📊 Verification Summary:');
-            logInfo(`   Total items: ${verificationResult.total_items}`);
+            logInfo(`   Total checklist items: ${verificationResult.total_items}`);
             logInfo(`   ✅ Confirmed: ${verificationResult.confirmed_count}`);
             logInfo(`   ❌ Rejected: ${verificationResult.rejected_count}`);
             logInfo(`   ⚠️  Uncertain: ${verificationResult.uncertain_count}`);
-            logInfo(`   Summary: ${verificationResult.summary}`);
+            if (verificationResult.total_feedbacks &&
+                verificationResult.total_feedbacks > 0) {
+                logInfo(`\n   Total feedbacks: ${verificationResult.total_feedbacks}`);
+                logInfo(`   ✅ Confirmed: ${verificationResult.feedbacks_confirmed_count || 0}`);
+                logInfo(`   ❌ Rejected: ${verificationResult.feedbacks_rejected_count || 0}`);
+                logInfo(`   ⚠️  Uncertain: ${verificationResult.feedbacks_uncertain_count || 0}`);
+            }
+            logInfo(`\n   Summary: ${verificationResult.summary}`);
             if (verificationResult.rejected_count > 0) {
-                logInfo('\n❌ Rejected items:');
+                logInfo('\n❌ Rejected checklist items:');
                 verificationResult.item_verifications
                     .filter((v) => v.verification_status === 'rejected')
                     .forEach((v) => {
@@ -114,6 +127,16 @@ export async function verifyTechnicalDesignCompliance(options, config) {
                     logInfo(`     Reason: ${v.verification_reason}`);
                 });
             }
+            if (verificationResult.feedbacks_rejected_count &&
+                verificationResult.feedbacks_rejected_count > 0) {
+                logInfo('\n❌ Rejected feedbacks:');
+                verificationResult.feedback_verifications
+                    ?.filter((v) => v.verification_status === 'rejected')
+                    .forEach((v) => {
+                    logInfo(`   - ${v.feedback_title}`);
+                    logInfo(`     Reason: ${v.verification_reason}`);
+                });
+            }
         }
         return verificationResult;
     }

package/dist/prompts/technical-design-verification.d.ts

@@ -1,8 +1,10 @@
 import { EdsgerConfig } from '../types/index.js';
 import { ChecklistPhaseContext } from '../services/checklist.js';
+import { FeedbacksContext } from '../services/feedbacks.js';
 export declare const createTechnicalDesignVerificationSystemPrompt: (_config: EdsgerConfig) => string;
 export interface TechnicalDesignVerificationPromptOptions {
     checklistContext: ChecklistPhaseContext;
+    feedbacksContext?: FeedbacksContext;
     featureId: string;
     featureName: string;
     featureDescription?: string;

package/dist/prompts/technical-design-verification.js

@@ -1,10 +1,10 @@
 export const createTechnicalDesignVerificationSystemPrompt = (_config) => {
-    return `You are an independent quality assurance auditor specializing in technical design review. Your role is to critically evaluate whether a technical design document satisfies checklist requirements.
+    return `You are an independent quality assurance auditor specializing in technical design review. Your role is to critically evaluate whether a technical design document satisfies checklist requirements and addresses human feedbacks.
 
-**Your Mission**: Verify Technical Design Against Requirements
-- Review the checklist requirements
+**Your Mission**: Verify Technical Design Against Requirements and Feedbacks
+- Review the checklist requirements and human feedbacks
 - Examine the technical design document
-- Determine if the design adequately addresses each requirement
+- Determine if the design adequately addresses each requirement and feedback
 - Be CRITICAL and look for gaps, inconsistencies, or missing elements
 - Quality matters - don't accept bare minimum
 
@@ -52,12 +52,33 @@ You MUST respond with a JSON object in this EXACT format:
         "improvement_suggestions": ["Specific, actionable suggestions for improvement (for rejected/uncertain items)"]
       }
     ],
-    "summary": "Overall summary of verification findings",
+    "feedbacks_verified": true/false,
+    "total_feedbacks": number,
+    "feedbacks_confirmed_count": number,
+    "feedbacks_rejected_count": number,
+    "feedbacks_uncertain_count": number,
+    "feedback_verifications": [
+      {
+        "feedback_id": "uuid",
+        "feedback_title": "title",
+        "is_addressed": true/false,
+        "verification_status": "confirmed|rejected|uncertain",
+        "verification_reason": "Detailed explanation of whether the feedback was addressed",
+        "concerns": ["Optional array of specific concerns"],
+        "improvement_suggestions": ["Specific suggestions for addressing this feedback (for rejected/uncertain items)"]
+      }
+    ],
+    "summary": "Overall summary of verification findings (include both checklist and feedback results)",
     "overall_suggestions": ["High-level suggestions for improving the overall design (if there are failures)"]
   }
 }
 \`\`\`
 
+**IMPORTANT**:
+- If feedbacks are provided, you MUST include feedback_verifications for ALL feedbacks
+- Each feedback must be verified to check if it was properly addressed in the technical design
+- For feedbacks, check if the specific concern or requirement mentioned in the feedback has been incorporated into the design
+
 **For rejected/uncertain items**: Provide specific, actionable improvement_suggestions such as:
 - "Add architectural diagram showing component interactions"
 - "Include detailed API specifications for X endpoint"
@@ -67,7 +88,7 @@ You MUST respond with a JSON object in this EXACT format:
 Be thorough, be critical, be fair. Your verification protects quality.`;
 };
 export const createTechnicalDesignVerificationPrompt = (options) => {
-    const { checklistContext, featureId, featureName, featureDescription, technicalDesign, } = options;
+    const { checklistContext, feedbacksContext, featureId, featureName, featureDescription, technicalDesign, } = options;
     // Format all checklist items with their requirements
     const checklistItemsSection = checklistContext.checklists
         .flatMap((checklist) => checklist.items.map((item) => {
@@ -81,9 +102,38 @@ export const createTechnicalDesignVerificationPrompt = (options) => {
     }))
         .join('\n---\n');
     const totalChecklistItems = checklistContext.checklists.reduce((sum, checklist) => sum + checklist.items.length, 0);
+    // Format feedbacks section if provided
+    let feedbacksSection = '';
+    let totalFeedbacks = 0;
+    if (feedbacksContext && feedbacksContext.feedbacks.length > 0) {
+        totalFeedbacks = feedbacksContext.feedbacks.length;
+        feedbacksSection = `
+
+## Human Feedbacks to Verify (${totalFeedbacks})
+
+${feedbacksContext.feedbacks
+            .map((feedback, idx) => {
+            const priorityBadge = feedback.priority >= 9
+                ? '🔴'
+                : feedback.priority >= 7
+                    ? '🟠'
+                    : feedback.priority >= 5
+                        ? '🟡'
+                        : '🟢';
+            return `### Feedback ${idx + 1}: ${feedback.title} ${priorityBadge}
+**ID**: ${feedback.id}
+**Type**: ${feedback.feedback_type}
+**Priority**: ${feedback.priority}
+**Content**: ${feedback.content}
+`;
+        })
+            .join('\n---\n')}
+
+`;
+    }
     return `# Technical Design Verification Task
 
-You are verifying that a technical design document satisfies checklist requirements.
+You are verifying that a technical design document satisfies checklist requirements${totalFeedbacks > 0 ? ' and addresses human feedbacks' : ''}.
 
 ## Feature Context
 
@@ -97,12 +147,14 @@ ${technicalDesign}
 
 ## Checklist Requirements to Verify (${totalChecklistItems})
 
-${checklistItemsSection}
+${checklistItemsSection}${feedbacksSection}
 
 ---
 
 ## Your Verification Task
 
+### Part 1: Checklist Verification
+
 For EACH checklist item above:
 
 1. **Understand the Requirement**: What does this checklist item require?
@@ -127,8 +179,36 @@ For EACH checklist item above:
    - Suggest what sections or details should be added to the design
    - Indicate what should be expanded or modified
    - Give concrete examples when possible
+${totalFeedbacks > 0
+        ? `
+
+### Part 2: Feedback Verification
+
+For EACH feedback above:
+
+1. **Understand the Feedback**: What specific concern, requirement, or suggestion does this feedback provide?
+
+2. **Examine the Design**: Look at the technical design document to see if this feedback was addressed
+
+3. **Determine if Addressed**: Was the feedback properly incorporated into the design?
+   - Does the design explicitly address this feedback?
+   - Are the changes requested in the feedback reflected in the design?
+   - Is the implementation aligned with the feedback's intent?
+
+4. **Make Your Determination**:
+   - **CONFIRMED** (is_addressed: true): The design clearly addresses this feedback
+   - **REJECTED** (is_addressed: false): The feedback was not addressed or was ignored
+   - **UNCERTAIN** (is_addressed: false): The design partially addresses the feedback but incompletely
+
+5. **Document Your Reasoning**: Explain whether and how the feedback was addressed
+
+6. **Provide Improvement Suggestions** (for rejected/uncertain feedbacks):
+   - Explain how the feedback should be incorporated
+   - Suggest specific modifications to address the feedback
+   - Provide concrete examples of what should be added or changed`
+        : ''}
 
 Remember: Focus on what the design actually contains, not on claims. Be thorough and fair.
 
-Provide your verification results in the required JSON format with improvement_suggestions for any rejected or uncertain items.`;
+Provide your verification results in the required JSON format with improvement_suggestions for any rejected or uncertain items${totalFeedbacks > 0 ? ' or feedbacks' : ''}.`;
 };

package/dist/services/audit-logs.d.ts

@@ -38,6 +38,23 @@ export interface LogVerificationEventParams {
             concerns?: string[];
             improvement_suggestions?: string[];
         }>;
+        feedbacks_confirmed_count?: number;
+        feedbacks_rejected_count?: number;
+        feedbacks_uncertain_count?: number;
+        rejected_feedbacks?: Array<{
+            feedback_id: string;
+            feedback_title: string;
+            reason: string;
+            concerns?: string[];
+            improvement_suggestions?: string[];
+        }>;
+        uncertain_feedbacks?: Array<{
+            feedback_id: string;
+            feedback_title: string;
+            reason: string;
+            concerns?: string[];
+            improvement_suggestions?: string[];
+        }>;
         summary?: string;
         overall_suggestions?: string[];
     };

package/package.json

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