edsger 0.2.13 → 0.2.14

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

@@ -179,17 +179,20 @@ async function prepareDesignContext(mcpServerUrl, mcpToken, featureId, checklist
                 feedbacksContext.feedbacks.forEach((fb, idx) => {
                     logInfo(`   ${idx + 1}. [${fb.feedback_type}] ${fb.title} (priority: ${fb.priority}, resolved: ${fb.is_resolved || false})`);
                     logInfo(`      Content: ${fb.content.substring(0, 150)}...`);
-                    if (fb.reference_section) {
-                        logInfo(`      Section: ${fb.reference_section}`);
+                    if (fb.target_type) {
+                        logInfo(`      🎯 Target Type: ${fb.target_type}`);
                     }
-                    if (fb.reference_line_start) {
-                        const lineInfo = fb.reference_line_end
-                            ? `Lines ${fb.reference_line_start}-${fb.reference_line_end}`
-                            : `Line ${fb.reference_line_start}`;
-                        logInfo(`      Location: ${lineInfo}`);
+                    if (fb.document_type) {
+                        logInfo(`      📄 Document: ${fb.document_type}`);
                     }
-                    if (fb.reference_content) {
-                        logInfo(`      Referenced: ${fb.reference_content.substring(0, 100)}...`);
+                    if (fb.line_number) {
+                        logInfo(`      📏 Line: ${fb.line_number}`);
+                    }
+                    else if (fb.line_range_start && fb.line_range_end) {
+                        logInfo(`      📏 Lines: ${fb.line_range_start}-${fb.line_range_end}`);
+                    }
+                    if (fb.context_snippet) {
+                        logInfo(`      Referenced: ${fb.context_snippet.substring(0, 100)}...`);
                     }
                 });
             }

package/dist/services/feedbacks.d.ts

@@ -5,6 +5,8 @@
 import { PipelinePhaseOptions } from '../types/pipeline.js';
 export type { PipelinePhaseOptions };
 export type FeedbackType = 'requirement' | 'constraint' | 'preference' | 'context' | 'quality_criteria' | 'issue' | 'suggestion';
+export type FeedbackTargetType = 'phase' | 'line' | 'range';
+export type DocumentType = 'technical_design' | 'code' | 'test_case' | 'user_story';
 export interface Feedback {
     id: string;
     feature_id: string | null;
@@ -21,10 +23,12 @@ export interface Feedback {
     created_by: string;
     created_at: string;
     updated_at: string;
-    reference_line_start?: number | null;
-    reference_line_end?: number | null;
-    reference_section?: string | null;
-    reference_content?: string | null;
+    target_type?: FeedbackTargetType | null;
+    document_type?: DocumentType | null;
+    line_number?: number | null;
+    line_range_start?: number | null;
+    line_range_end?: number | null;
+    context_snippet?: string | null;
 }
 export interface FeedbacksContext {
     phase: string;

package/dist/services/feedbacks.js

@@ -138,21 +138,32 @@ function formatFeedbacksList(feedbacks) {
     return sorted
         .map((feedback, idx) => {
         const priorityBadge = getPriorityBadge(feedback.priority);
-        // Build reference info if available
-        let referenceInfo = '';
-        if (feedback.reference_section) {
-            referenceInfo += `\n**Reference Section**: ${feedback.reference_section}`;
+        // Build context metadata section
+        let contextInfo = '';
+        // Document type
+        if (feedback.document_type) {
+            const docTypeDisplay = feedback.document_type
+                .split('_')
+                .map(word => word.charAt(0).toUpperCase() + word.slice(1))
+                .join(' ');
+            contextInfo += `\n**Document**: ${docTypeDisplay}`;
         }
-        if (feedback.reference_line_start) {
-            const lineRange = feedback.reference_line_end
-                ? `Lines ${feedback.reference_line_start}-${feedback.reference_line_end}`
-                : `Line ${feedback.reference_line_start}`;
-            referenceInfo += `\n**Reference Location**: ${lineRange}`;
+        // Target type
+        if (feedback.target_type && feedback.target_type !== 'phase') {
+            contextInfo += `\n**Target Type**: ${feedback.target_type}`;
         }
-        if (feedback.reference_content) {
-            referenceInfo += `\n**Referenced Content**:\n\`\`\`\n${feedback.reference_content}\n\`\`\``;
+        // Build line reference info if available
+        if (feedback.line_number) {
+            contextInfo += `\n**Line**: ${feedback.line_number}`;
         }
-        return `### ${idx + 1}. ${feedback.title} ${priorityBadge}\n\n${feedback.content}${referenceInfo}\n`;
+        else if (feedback.line_range_start && feedback.line_range_end) {
+            contextInfo += `\n**Lines**: ${feedback.line_range_start}-${feedback.line_range_end}`;
+        }
+        // Context snippet
+        if (feedback.context_snippet) {
+            contextInfo += `\n**Referenced Content**:\n\`\`\`\n${feedback.context_snippet}\n\`\`\``;
+        }
+        return `### ${idx + 1}. ${feedback.title} ${priorityBadge}\n\n${feedback.content}${contextInfo}\n`;
     })
         .join('\n');
 }

package/package.json

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