@damper/mcp 0.10.1 → 0.10.3

package/dist/formatters.js

@@ -52,8 +52,7 @@ export function formatStartTaskResponse(result) {
     lines.push('1. `add_note`: "Session started: <goal>"');
     lines.push('2. Work: use `add_commit` for commits, `add_note` for decisions');
     lines.push('3. `add_note`: "Session end: <summary>"');
-    lines.push('4. `review_task` (with `selfAssessment`) — required self-review before completion');
-    lines.push('5. `complete_task` (with `confirmations`) or `abandon_task`');
+    lines.push('4. `complete_task` (with `selfAssessment` and `confirmations`) or `abandon_task`');
     return lines.join('\n');
 }
 /**

package/dist/index.js

@@ -619,10 +619,9 @@ const SelfAssessmentSchema = z.object({
 // Tool: Review task (self-assessment gate before completion)
 server.registerTool('review_task', {
     title: 'Review Task',
-    description: 'Self-review gate — MUST be called before `complete_task`.\n\n' +
+    description: 'Optional standalone self-review step. You can also pass `selfAssessment` directly to `complete_task` instead.\n\n' +
         'Reflect on your session: what project knowledge was missing, what tools were confusing, ' +
         'what process gaps caused friction. This drives project improvements.\n\n' +
-        '**Flow:** `start_task` → work → `review_task` → `complete_task`\n\n' +
         'If you have findings, propose improvements to the user after this call ' +
         '(e.g., `update_context_section` for missing docs, `report_issue` for tool problems).',
     inputSchema: z.object({
@@ -659,12 +658,13 @@ server.registerTool('complete_task', {
     title: 'Complete Task',
     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' +
-        '**Requires `review_task` first** — completion will be rejected if you haven\'t called `review_task`.\n\n' +
+        '**Self-assessment (required):** Pass `selfAssessment` reflecting on your session — what project knowledge was missing, ' +
+        'what tools were confusing, what process gaps caused friction. Pass empty object `{}` if nothing to report. ' +
+        'You can also call `review_task` separately before this, but passing `selfAssessment` here is simpler.\n\n' +
         '**Before calling:**\n' +
-        '1. Call `review_task` (self-assessment)\n' +
-        '2. Push all commits\n' +
-        '3. Check if project context docs need updating\n\n' +
-        '**Completion checklist:** If the project has a completion checklist (shown in `start_task` and `review_task` responses), ' +
+        '1. Push all commits\n' +
+        '2. Check if project context docs need updating\n\n' +
+        '**Completion checklist:** If the project has a completion checklist (shown in `start_task` response), ' +
         'you MUST pass `confirmations` — an array of `{item, evidence}` objects. Each confirmation needs the checklist item text ' +
         'and concrete evidence proving you verified it (e.g., test output, build log snippet). ' +
         'Empty evidence or evidence identical to the item text will be rejected.\n\n' +
@@ -687,6 +687,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.'),
     }),
     outputSchema: z.object({
         id: z.string(),
@@ -699,9 +701,9 @@ server.registerTool('complete_task', {
         idempotentHint: true,
         openWorldHint: false,
     },
-}, async ({ taskId, summary, commits, confirmations, skipReview, reviewInstructions }) => {
+}, async ({ taskId, summary, commits, confirmations, skipReview, reviewInstructions, selfAssessment }) => {
     try {
-        const result = await api('POST', `/api/agent/tasks/${taskId}/complete`, { summary, commits, confirmations, skipReview, reviewInstructions });
+        const result = await api('POST', `/api/agent/tasks/${taskId}/complete`, { summary, commits, confirmations, skipReview, reviewInstructions, selfAssessment: selfAssessment ?? {} });
         return {
             content: [{ type: 'text', text: formatCompleteTaskResponse(result) }],
             structuredContent: result,
@@ -711,7 +713,7 @@ server.registerTool('complete_task', {
         const error = err;
         if (error.reviewRequired) {
             return {
-                content: [{ type: 'text', text: '❌ Completion blocked — self-review required.\n\nYou must call the `review_task` tool before `complete_task`.\nIf the tool is not loaded, search for it first (e.g., ToolSearch "review_task").\n\nThis is a required step for reflecting on your session.' }],
+                content: [{ type: 'text', text: '❌ Completion blocked — self-review required.\n\nAdd `selfAssessment` to your `complete_task` call:\n```json\n"selfAssessment": { "missingContext": [], "toolIssues": [], "processGaps": [] }\n```\nReflect on your session: what project knowledge was missing, what tools were confusing, what process gaps caused friction. Pass `{}` if nothing to report.' }],
                 isError: true,
             };
         }
@@ -861,11 +863,11 @@ This project uses Damper MCP for task tracking. **You MUST follow this workflow.
   2. **Abandon and hand off** - Call \`abandon_task\` with a detailed handoff summary for the next agent.
 - NEVER leave a started task without completing or abandoning it
 - If the project has a **completion checklist** (shown in \`start_task\` response), you MUST pass all items as \`confirmations\` when calling \`complete_task\`
-- **Self-review (required):** Call \`review_task\` before \`complete_task\`. Pass \`selfAssessment\` reflecting on:
+- **Self-review (required):** Pass \`selfAssessment\` in your \`complete_task\` call reflecting on:
   - \`missingContext\` — What project knowledge was missing or wrong?
   - \`toolIssues\` — Which tools or workflow steps were confusing or broken?
   - \`processGaps\` — What specs or conventions were missing?
-  If you identify improvements, **propose them to the user** — explain briefly what you found and why fixing it helps future tasks. Only act after the user approves.
+  Pass \`{}\` if nothing to report. If you identify improvements, **propose them to the user** before acting.
 
 ### Content Style Guide
 

package/package.json

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