@damper/mcp 0.8.4 → 0.8.6

package/dist/formatters.js

@@ -95,7 +95,10 @@ export function formatSelfAssessmentActions(assessment) {
  * Format complete_task response
  */
 export function formatCompleteTaskResponse(result, assessment) {
-    const lines = [`✅ Completed ${result.id}`];
+    const statusMessage = result.status === 'done'
+        ? `✅ Task ${result.id} marked as done`
+        : `✅ Task ${result.id} moved to review`;
+    const lines = [statusMessage];
     // Self-assessment follow-up actions (shown first if provided)
     if (assessment) {
         const actions = formatSelfAssessmentActions(assessment);
@@ -131,7 +134,7 @@ export function formatAbandonTaskResponse(result, hasSummary, assessment) {
     if (!hasSummary) {
         lines.push('\n⚠️ Tip: Include a summary next time for better handoff.');
     }
-    lines.push('\n💡 Task returned to "planned". Notes preserved for next agent.');
+    lines.push('\n💡 Task returned to "planned" (from in_progress or in_review). Notes preserved for next agent.');
     // Self-assessment follow-up actions
     if (assessment) {
         const actions = formatSelfAssessmentActions(assessment);

package/dist/index.js

@@ -123,7 +123,7 @@ server.registerTool('list_tasks', {
         'If total > returned tasks, paginate with offset (e.g. offset=50 for page 2).\n\n' +
         '**Recommended:** Call `get_project_context` before starting work to understand codebase patterns.',
     inputSchema: z.object({
-        status: z.enum(['planned', 'in_progress', 'done', 'all']).optional(),
+        status: z.enum(['planned', 'in_progress', 'in_review', 'done', 'all']).optional(),
         type: z.enum(['bug', 'feature', 'improvement', 'task']).optional().describe('Filter by task type'),
         quarter: z.string().optional().describe('Filter by quarter (e.g., "Q1 2025") or "none" for unscheduled'),
         sort: z.enum(['importance', 'newest', 'votes']).optional().describe('Sort order: importance (priority+votes, default), newest, or votes'),
@@ -359,7 +359,7 @@ server.registerTool('create_task', {
         description: z.string().optional().describe('Task description (shown on public roadmap). Write for end users: 1-3 sentences on what changes and why it matters. ' +
             'No code references, file paths, or jargon.'),
         type: z.enum(['bug', 'feature', 'improvement', 'task']).optional().describe('Task type (default: feature)'),
-        status: z.enum(['planned', 'in_progress', 'done']).optional(),
+        status: z.enum(['planned', 'in_progress', 'in_review', 'done']).optional(),
         priority: z.enum(['high', 'medium', 'low']).optional().describe('Task priority'),
         implementationPlan: z.string().optional().describe('Implementation plan (markdown, internal only - never shown publicly)'),
         isPublic: z.boolean().optional().describe('Whether task is visible on public roadmap (default: true)'),
@@ -608,7 +608,8 @@ const DocumentationSchema = z.object({
 // Tool: Complete task
 server.registerTool('complete_task', {
     title: 'Complete Task',
-    description: 'Mark task done with a brief one-line summary. Optionally log final commits at completion.\n\n' +
+    description: 'Mark task as ready for review (in_review). A human will verify and move it to done. ' +
+        'Pass `skipReview: true` to mark as done immediately (use when the human explicitly asks).\n\n' +
         '**Before calling:**\n' +
         '1. Push all commits\n' +
         '2. Check if project context docs need updating\n\n' +
@@ -633,6 +634,7 @@ server.registerTool('complete_task', {
             item: z.string().describe('Checklist item text from start_task'),
             evidence: z.string().describe('Concrete proof of verification (e.g., "bun test: 47 passed, 0 failed")'),
         })).optional().describe('Completion checklist confirmations — echo back each item from the checklist shown in start_task'),
+        skipReview: z.boolean().optional().describe('Skip human review and mark task as done immediately (default: false, task goes to in_review)'),
     }),
     outputSchema: z.object({
         id: z.string(),
@@ -645,9 +647,9 @@ server.registerTool('complete_task', {
         idempotentHint: true,
         openWorldHint: false,
     },
-}, async ({ taskId, summary, selfAssessment, commits, confirmations }) => {
+}, async ({ taskId, summary, selfAssessment, commits, confirmations, skipReview }) => {
     try {
-        const result = await api('POST', `/api/agent/tasks/${taskId}/complete`, { summary, commits, confirmations });
+        const result = await api('POST', `/api/agent/tasks/${taskId}/complete`, { summary, commits, confirmations, skipReview });
         // Log self-assessment as a structured note if provided
         if (selfAssessment) {
             const assessmentLines = ['Self-Assessment:'];
@@ -713,7 +715,7 @@ server.registerTool('complete_task', {
 // Tool: Abandon task
 server.registerTool('abandon_task', {
     title: 'Abandon Task',
-    description: 'Release lock and return task to planned status. Use when stopping work.\n\n' +
+    description: 'Release lock and return task to planned status. Works for in_progress and in_review tasks.\n\n' +
         '**Before calling:**\n' +
         '1. Push any WIP commits\n\n' +
         '**Summary parameter:** What was done, what remains, blockers. Helps the next agent.\n\n' +

package/package.json

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