internaltool-mcp 1.5.0 → 1.5.1

package/index.js

@@ -448,74 +448,92 @@ Use this when the developer says "wrap up", "end of day", or "I'm done for today
 
   server.tool(
     'kickoff_task',
-    `Kick off a task — brief the developer and move it to in_progress.
-Fetches the full task details plus recent project commits for context.
-Returns everything needed to start: task plan, what recently changed in the repo, and the git branch command to run.
-Use this when a developer says "start task", "brief me on", or "what do I need to do for TASK-X".
+    `Kick off a task — read the implementation plan (README) and brief the developer, then move the task to in_progress.
 
-Set confirmed=false first to preview what will happen, then confirmed=true to actually move the task to in_progress.`,
+The README/implementation plan IS the brief. Always read it fully and present it to the developer
+before they write a single line of code. If no README exists, warn the developer that the plan is
+missing and suggest they write one before starting.
+
+Workflow:
+1. confirmed=false  → show the full README brief + subtask checklist so developer can read the plan
+2. confirmed=true   → move task to in_progress, return branch name and recent repo commits
+
+Use this when a developer says "start task", "brief me on", or "what do I need to do for TASK-X".`,
     {
       taskId:    z.string().describe("Task's MongoDB ObjectId"),
-      confirmed: z.boolean().optional().default(false).describe('Set true to move the task to in_progress'),
+      confirmed: z.boolean().optional().default(false).describe('Set true after reading the plan to move the task to in_progress'),
     },
     async ({ taskId, confirmed = false }) => {
-      // Get full task
       const taskRes = await api.get(`/api/tasks/${taskId}`)
       if (!taskRes?.success) return errorText('Task not found')
       const task = taskRes.data.task
 
-      // Get recent commits from the project
-      let recentCommits = []
-      try {
-        const commitsRes = await api.get(
-          `/api/projects/${task.project}/github/commits?per_page=10`
-        )
-        if (commitsRes?.success) recentCommits = commitsRes.data.commits || []
-      } catch { /* GitHub may not be linked */ }
-
       const suggestedBranch = `feature/${task.key?.toLowerCase()}-${task.title
         .toLowerCase()
         .replace(/[^a-z0-9]+/g, '-')
         .slice(0, 40)}`
 
-      // Preview mode — show what will happen without touching the task
+      const hasReadme   = !!(task.readmeMarkdown && task.readmeMarkdown.trim().length > 0)
+      const subtasks    = (task.subtasks || []).map(s => ({
+        title: s.title,
+        done:  s.done,
+        status: s.done ? '✅' : '⬜',
+      }))
+      const subtasksDone  = subtasks.filter(s => s.done).length
+      const subtasksTotal = subtasks.length
+
+      // ── Preview: show the full plan before touching anything ──
       if (!confirmed) {
         return text({
-          preview: {
-            action:          'kickoff_task',
-            task:            { key: task.key, title: task.title, currentColumn: task.column, priority: task.priority },
-            willMoveTo:      'in_progress',
+          brief: {
+            key:         task.key,
+            title:       task.title,
+            priority:    task.priority,
+            description: task.description || '(no description)',
+            // The implementation plan — Claude must read and present this to the developer
+            implementationPlan: hasReadme
+              ? task.readmeMarkdown
+              : '⚠️  No implementation plan (README) found for this task. The developer should write one before starting work.',
+            subtasks: subtasksTotal > 0
+              ? { items: subtasks, progress: `${subtasksDone}/${subtasksTotal} done` }
+              : null,
+            approvalState: task.approval?.state || 'none',
+          },
+          meta: {
+            currentColumn:  task.column,
+            willMoveTo:     'in_progress',
             suggestedBranch,
           },
           requiresConfirmation: true,
-          message: `This will move "${task.key}: ${task.title}" to in_progress. Call kickoff_task again with confirmed=true to proceed.`,
+          message: hasReadme
+            ? `Read the implementation plan above carefully, then call kickoff_task again with confirmed=true to start the task.`
+            : `⚠️ This task has no implementation plan. Consider writing one first (use update_task with readmeMarkdown), then confirm to start anyway.`,
         })
       }
 
-      // Move to in_progress
+      // ── Confirmed: move to in_progress and fetch recent commits ──
+      let recentCommits = []
+      try {
+        const commitsRes = await api.get(`/api/projects/${task.project}/github/commits?per_page=10`)
+        if (commitsRes?.success) recentCommits = commitsRes.data.commits || []
+      } catch { /* GitHub may not be linked */ }
+
       let moved = false
       try {
-        const moveRes = await api.post(`/api/tasks/${taskId}/move`, {
-          column: 'in_progress',
-          toIndex: 0,
-        })
+        const moveRes = await api.post(`/api/tasks/${taskId}/move`, { column: 'in_progress', toIndex: 0 })
         moved = moveRes?.success ?? false
       } catch { /* might already be in_progress */ }
 
       return text({
-        task: {
-          id:          task._id,
-          key:         task.key,
-          title:       task.title,
-          description: task.description,
-          priority:    task.priority,
-          column:      moved ? 'in_progress' : task.column,
-          readme:      task.readmeMarkdown,
-          subtasks:    task.subtasks,
-          parkNote:    task.parkNote,
-          approval:    task.approval,
+        started: {
+          key:      task.key,
+          title:    task.title,
+          priority: task.priority,
+          column:   moved ? 'in_progress' : task.column,
+          subtasks,
         },
-        recentCommits: recentCommits.slice(0, 5).map((c) => ({
+        implementationPlan: hasReadme ? task.readmeMarkdown : null,
+        recentCommits: recentCommits.slice(0, 5).map(c => ({
           sha:     c.sha?.slice(0, 7),
           message: c.commit?.message?.split('\n')[0],
           author:  c.commit?.author?.name,
@@ -523,7 +541,7 @@ Set confirmed=false first to preview what will happen, then confirmed=true to ac
         })),
         movedToInProgress: moved,
         suggestedBranch,
-        nextStep: `Branch created? Run: git checkout -b ${suggestedBranch}\nOr use the create_branch tool to create it on GitHub automatically.`,
+        nextStep: `Use create_branch to create "${suggestedBranch}" on GitHub, then run:\n  git fetch origin\n  git checkout ${suggestedBranch}`,
       })
     }
   )

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "internaltool-mcp",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "MCP server for InternalTool — connect AI assistants (Claude Code, Cursor) to your project and task management platform",
   "type": "module",
   "main": "index.js",