wolfpack-mcp 1.0.62 → 1.0.63

package/dist/index.js

@@ -7,6 +7,7 @@ import { createRequire } from 'module';
 import { readFile, stat } from 'fs/promises';
 import { basename, extname, resolve } from 'path';
 import { WolfpackClient } from './client.js';
+import { allTasksChecked, getWorkItemReminders } from './workItemReminders.js';
 import { validateConfig, config } from './config.js';
 import { AGENT_BUILDER_TOOLS, handleAgentBuilderTool } from './agentBuilderTools.js';
 import { PROCEDURE_TOOLS, handleProcedureTool } from './procedureTools.js';
@@ -654,18 +655,6 @@ function stripUuids(obj) {
     }
     return result;
 }
-// Helper to detect if a work item description contains a plan
-function hasPlan(description) {
-    if (!description)
-        return false;
-    // Check for markdown checklist items (plan indicators)
-    if (/- \[[ x]\]/.test(description))
-        return true;
-    // Check for --- separator with content after (agent-added section)
-    if (/\n---\n/.test(description))
-        return true;
-    return false;
-}
 class WolfpackMCPServer {
     server;
     client;
@@ -833,6 +822,7 @@ class WolfpackMCPServer {
                 {
                     name: 'update_work_progress',
                     description: 'Update work item description/notes with your progress. WORKFLOW: 1) get_work_item to read current notes 2) Modify the markdown description with new findings/progress 3) Call this with the complete updated description. The full description replaces the existing one. ' +
+                        'STATUS: Items in "new" are auto-advanced to "doing" (updating progress means work has started). Completing the work is NOT auto-detected: when done, set status to "review" via update_work_item and add a completion comment. ' +
                         'CRITICAL: NEVER overwrite or remove the original description text. Preserve all existing content exactly as-is. Append your plan below a "---" separator. You may only modify content you previously added (your plan section). ' +
                         'BEST PRACTICE: Append a plan with markdown checkboxes (- [ ] task). Check off completed tasks (- [x] task) as you progress. Include sections for: Plan (checklist), Approach (strategy), and Notes (discoveries/decisions). ' +
                         CONTENT_LINKING_HELP,
@@ -2072,11 +2062,9 @@ class WolfpackMCPServer {
                         const workItem = await this.client.getWorkItem(parsed.work_item_id, parsed.project_slug || this.client.getProjectSlug() || undefined);
                         if (workItem) {
                             let text = JSON.stringify(stripUuids(workItem), null, 2);
-                            // Add hint if no plan detected in description
-                            if (!hasPlan(workItem.description)) {
-                                text =
-                                    'REMINDER: This work item has no plan. Before starting work, use update_work_progress to add a plan below a "---" separator. Preserve the original description and append your plan with markdown checkboxes (- [ ] task).\n\n' +
-                                        text;
+                            const reminders = getWorkItemReminders(workItem.status, workItem.description);
+                            if (reminders.length > 0) {
+                                text = `${reminders.join('\n\n')}\n\n${text}`;
                             }
                             return {
                                 content: [{ type: 'text', text }],
@@ -2089,15 +2077,25 @@ class WolfpackMCPServer {
                     case 'update_work_progress': {
                         const parsed = UpdateWorkProgressSchema.parse(args);
                         const teamSlug = parsed.project_slug || this.client.getProjectSlug() || undefined;
-                        const workItem = await this.client.updateWorkProgress(parsed.work_item_id, parsed.description, teamSlug);
+                        let workItem = await this.client.updateWorkProgress(parsed.work_item_id, parsed.description, teamSlug);
                         if (workItem) {
+                            let summary = `Updated description on: ${workItem.title}`;
+                            // Progress on a "new" item means work has started — advance it to "doing"
+                            if (workItem.status === 'new') {
+                                const advanced = await this.client.updateWorkItem(parsed.work_item_id, { status: 'doing' }, teamSlug);
+                                if (advanced) {
+                                    workItem = advanced;
+                                    summary += ' (status auto-advanced new → doing)';
+                                }
+                            }
+                            let text = `${summary}\n\n${JSON.stringify(stripUuids(workItem), null, 2)}`;
+                            if (workItem.status === 'doing' && allTasksChecked(workItem.description)) {
+                                text =
+                                    'REMINDER: All plan tasks are checked off. If the work is complete, set status to "review" NOW (update_work_item) and add a completion comment.\n\n' +
+                                        text;
+                            }
                             return {
-                                content: [
-                                    {
-                                        type: 'text',
-                                        text: `Updated description on: ${workItem.title}\n\n${JSON.stringify(stripUuids(workItem), null, 2)}`,
-                                    },
-                                ],
+                                content: [{ type: 'text', text }],
                             };
                         }
                         return {

package/dist/workItemReminders.js

@@ -0,0 +1,41 @@
+// Workflow reminders injected into work-item tool responses. Tool descriptions document
+// the status workflow, but agents skim them — a reminder in the response at the moment of
+// action is what reliably changes behaviour. Reminders only fire when actionable.
+// Detect if a work item description contains a plan
+export function hasPlan(description) {
+    if (!description)
+        return false;
+    // Check for markdown checklist items (plan indicators)
+    if (/- \[[ x]\]/.test(description))
+        return true;
+    // Check for --- separator with content after (agent-added section)
+    if (/\n---\n/.test(description))
+        return true;
+    return false;
+}
+// True when the description has a checklist and every task is checked off.
+export function allTasksChecked(description) {
+    if (!description)
+        return false;
+    return /- \[x\]/.test(description) && !/- \[ \]/.test(description);
+}
+const STATUS_REMINDERS = {
+    pending: 'REMINDER: This work item is in the backlog ("pending"). Claim it with pull_work_item before starting work.',
+    new: 'REMINDER: This work item is in "new". If you are picking it up, set status to "doing" (update_work_item) ' +
+        'BEFORE starting work.',
+    doing: 'REMINDER: This work item is in "doing". The moment the work is complete, set status to "review" ' +
+        '(update_work_item) and add a completion comment — do not leave it sitting in "doing".',
+};
+// Reminders to prepend to a get_work_item response, most urgent first.
+export function getWorkItemReminders(status, description) {
+    const reminders = [];
+    const statusReminder = STATUS_REMINDERS[status];
+    if (statusReminder)
+        reminders.push(statusReminder);
+    if (!hasPlan(description)) {
+        reminders.push('REMINDER: This work item has no plan. Before starting work, use update_work_progress to add a plan ' +
+            'below a "---" separator. Preserve the original description and append your plan with markdown ' +
+            'checkboxes (- [ ] task).');
+    }
+    return reminders;
+}

package/dist/workItemReminders.test.js

@@ -0,0 +1,52 @@
+import { describe, it, expect } from 'vitest';
+import { hasPlan, allTasksChecked, getWorkItemReminders } from './workItemReminders.js';
+const PLAN = 'Original text\n\n---\n\n## Plan\n\n- [x] first task\n- [ ] second task';
+const DONE_PLAN = 'Original text\n\n---\n\n## Plan\n\n- [x] first task\n- [x] second task';
+describe('hasPlan', () => {
+    it('detects markdown checklists', () => {
+        expect(hasPlan(PLAN)).toBe(true);
+    });
+    it('detects a --- separator section', () => {
+        expect(hasPlan('Original text\n---\nAgent notes')).toBe(true);
+    });
+    it.each([null, undefined, '', 'Just a plain description'])('rejects %j', (description) => {
+        expect(hasPlan(description)).toBe(false);
+    });
+});
+describe('allTasksChecked', () => {
+    it('is true when every task is checked', () => {
+        expect(allTasksChecked(DONE_PLAN)).toBe(true);
+    });
+    it('is false while unchecked tasks remain', () => {
+        expect(allTasksChecked(PLAN)).toBe(false);
+    });
+    it.each([null, undefined, '', 'No checklist here'])('is false without a checklist (%j)', (description) => {
+        expect(allTasksChecked(description)).toBe(false);
+    });
+});
+describe('getWorkItemReminders', () => {
+    it('tells agents to pull pending items', () => {
+        expect(getWorkItemReminders('pending', PLAN)).toEqual([
+            expect.stringContaining('pull_work_item'),
+        ]);
+    });
+    it('tells agents to move new items to doing', () => {
+        expect(getWorkItemReminders('new', PLAN)).toEqual([
+            expect.stringContaining('set status to "doing"'),
+        ]);
+    });
+    it('tells agents to move doing items to review when complete', () => {
+        expect(getWorkItemReminders('doing', PLAN)).toEqual([
+            expect.stringContaining('set status to "review"'),
+        ]);
+    });
+    it('adds the no-plan reminder when the description has no plan', () => {
+        const reminders = getWorkItemReminders('new', 'Just a description');
+        expect(reminders).toHaveLength(2);
+        expect(reminders[1]).toContain('no plan');
+    });
+    it('stays quiet for statuses with no required action', () => {
+        expect(getWorkItemReminders('review', PLAN)).toEqual([]);
+        expect(getWorkItemReminders('completed', PLAN)).toEqual([]);
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wolfpack-mcp",
-  "version": "1.0.62",
+  "version": "1.0.63",
   "description": "MCP server for Wolfpack AI-enhanced software delivery tools",
   "main": "dist/index.js",
   "type": "module",