@damper/mcp 0.10.3 → 0.10.5

package/dist/index.js

@@ -106,6 +106,7 @@ const FeedbackSummarySchema = z.object({
     id: z.string(),
     title: z.string(),
     type: z.string(),
+    status: z.string(),
     voterCount: z.number(),
     isPublic: z.boolean(),
     linkedTaskId: z.string().nullish(),
@@ -687,8 +688,8 @@ server.registerTool('complete_task', {
         reviewInstructions: z.string().optional().describe('What the reviewer should test or verify (markdown). Recommended when sending to review (default behavior). ' +
             'Write specific, actionable steps: what to test in the UI, what edge cases to check, ' +
             'what areas might break. Not a restatement of the summary.'),
-        selfAssessment: SelfAssessmentSchema.optional().describe('Self-review of your session (required if you haven\'t called `review_task` separately). ' +
-            'Reflect on what went well, what knowledge was missing, what caused friction. Pass `{}` if nothing to report.'),
+        selfAssessment: z.any().optional().describe('Self-review of your session. Object with optional keys: missingContext (string[]), toolIssues (string[]), processGaps (string[]). ' +
+            'Reflect on what went well, what knowledge was missing, what caused friction. Pass {} if nothing to report.'),
     }),
     outputSchema: z.object({
         id: z.string(),
@@ -702,8 +703,19 @@ server.registerTool('complete_task', {
         openWorldHint: false,
     },
 }, async ({ taskId, summary, commits, confirmations, skipReview, reviewInstructions, selfAssessment }) => {
+    // Coerce selfAssessment to a valid object — LLMs may pass strings, nulls, or malformed values
+    let assessment = {};
+    if (selfAssessment && typeof selfAssessment === 'object') {
+        assessment = selfAssessment;
+    }
+    else if (typeof selfAssessment === 'string') {
+        try {
+            assessment = JSON.parse(selfAssessment);
+        }
+        catch { /* use empty */ }
+    }
     try {
-        const result = await api('POST', `/api/agent/tasks/${taskId}/complete`, { summary, commits, confirmations, skipReview, reviewInstructions, selfAssessment: selfAssessment ?? {} });
+        const result = await api('POST', `/api/agent/tasks/${taskId}/complete`, { summary, commits, confirmations, skipReview, reviewInstructions, selfAssessment: assessment });
         return {
             content: [{ type: 'text', text: formatCompleteTaskResponse(result) }],
             structuredContent: result,
@@ -1343,10 +1355,15 @@ server.registerTool('list_feedback', {
     const lines = data.feedback.map((f) => {
         const link = f.linkedTaskId ? ` → ${f.linkedTaskId}` : '';
         const hidden = !f.isPublic ? ' [hidden]' : '';
-        return `• ${f.id}: ${f.title} [${f.type}] (${f.voterCount} votes)${hidden}${link}`;
+        return `• ${f.id}: ${f.title} [${f.type}] (${f.status}) (${f.voterCount} votes)${hidden}${link}`;
     });
+    const newCount = data.feedback.filter((f) => f.status === 'new').length;
+    const nudge = newCount > 0
+        ? `\n\n💡 ${newCount} feedback item${newCount > 1 ? 's are' : ' is'} still "new". ` +
+            'Use `update_feedback` to triage (e.g., set to "under_review" or "planned").'
+        : '';
     return {
-        content: [{ type: 'text', text: `Feedback:\n${lines.join('\n')}` }],
+        content: [{ type: 'text', text: `Feedback:\n${lines.join('\n')}${nudge}` }],
         structuredContent: { feedback: data.feedback },
     };
 });
@@ -1412,6 +1429,10 @@ server.registerTool('get_feedback', {
         if (f.comments.length > 3)
             parts.push(`... and ${f.comments.length - 3} more`);
     }
+    if (f.status === 'new') {
+        parts.push('\n---\n💡 This feedback is still "new". Use `update_feedback` to triage it ' +
+            '(e.g., "under_review", "planned", or "in_progress").');
+    }
     return {
         content: [{ type: 'text', text: parts.join('\n') }],
         structuredContent: f,
@@ -1823,7 +1844,14 @@ server.registerTool('link_feedback_to_task', {
         };
     }
     return {
-        content: [{ type: 'text', text: `🔗 Linked ${linkedCount} feedback to task: ${result.feedbackTitles.join(', ')}` }],
+        content: [
+            {
+                type: 'text',
+                text: `🔗 Linked ${linkedCount} feedback to task: ${result.feedbackTitles.join(', ')}\n\n` +
+                    '💡 Consider updating the linked feedback status with `update_feedback` ' +
+                    '(e.g., to "planned" or "in_progress").',
+            },
+        ],
         structuredContent: result,
     };
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@damper/mcp",
-  "version": "0.10.3",
+  "version": "0.10.5",
   "description": "MCP server for Damper task management",
   "author": "Damper <hello@usedamper.com>",
   "repository": {