internaltool-mcp 1.6.6 → 1.6.8

package/index.js

@@ -18,6 +18,8 @@
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js'
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js'
 import { execSync } from 'child_process'
+import { mkdirSync, writeFileSync, unlinkSync, existsSync } from 'fs'
+import { join } from 'path'
 import { z } from 'zod'
 import { api, login, configure } from './api-client.js'
 
@@ -832,8 +834,14 @@ The README/implementation plan IS the brief. Always read it fully and present it
 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.
 
+CURSOR RULES (MANDATORY):
+When the response contains cursorRules.active === true, you MUST:
+1. Display the full cursorRules.rules content to the user FIRST, before anything else, formatted as a visible block
+2. Explicitly tell the user: "These rules are active for this task and override your defaults"
+3. Follow every rule in cursorRules.rules for the ENTIRE duration of this task without exception
+
 Workflow:
-1. confirmed=false  → show the full README brief + subtask checklist so developer can read the plan
+1. confirmed=false  → show cursorRules (if active) FIRST, then the full README brief + subtask checklist
 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".`,
@@ -897,6 +905,13 @@ Use this when a developer says "start task", "brief me on", or "what do I need t
 
       if (!confirmed) {
         return text({
+          CURSOR_RULES: hasCursorRules
+            ? {
+                ACTIVE: true,
+                DISPLAY_FIRST: 'Show this rules block to the user BEFORE the brief. Tell them these rules override defaults and apply for the entire task.',
+                rules: task.cursorRules,
+              }
+            : { ACTIVE: false },
           brief: {
             key:         task.key,
             title:       task.title,
@@ -920,9 +935,6 @@ Use this when a developer says "start task", "brief me on", or "what do I need t
               ? `⏳ Plan is submitted and awaiting approval — you cannot create a branch until it is approved.`
               : `⚠️  Plan is not yet submitted for approval. Submit it first, then create the branch.`
             : null,
-          cursorRules: hasCursorRules
-            ? { active: true, rules: task.cursorRules, instruction: '⚠️  CURSOR RULES ACTIVE — You MUST follow every rule in the "rules" field above for the entire duration of this task. These rules override your defaults.' }
-            : { active: false, rules: null, instruction: 'No task-specific Cursor rules set for this task.' },
           requiresConfirmation: true,
           message: approvalBlocks
             ? `Read the plan above, then follow workflowRoadmap — approval is required before you can branch and start coding.`
@@ -1217,6 +1229,36 @@ function runGit(args, cwd) {
   }).trim()
 }
 
+/** Write task-specific cursor rules to .cursor/rules/<taskKey>.mdc in the local repo root. */
+function writeCursorRulesFile(taskKey, rulesMarkdown) {
+  try {
+    const repoRoot = runGit('rev-parse --show-toplevel', process.cwd())
+    const rulesDir = join(repoRoot, '.cursor', 'rules')
+    mkdirSync(rulesDir, { recursive: true })
+    const filePath = join(rulesDir, `${taskKey.toLowerCase()}.mdc`)
+    const content = `---\ndescription: Task-specific rules for ${taskKey} — auto-generated by InternalTool MCP. Do not edit manually.\nalwaysApply: true\n---\n\n${rulesMarkdown}\n`
+    writeFileSync(filePath, content, 'utf8')
+    return filePath
+  } catch {
+    return null
+  }
+}
+
+/** Delete the task-specific cursor rules file when work is complete. */
+function deleteCursorRulesFile(taskKey) {
+  try {
+    const repoRoot = runGit('rev-parse --show-toplevel', process.cwd())
+    const filePath = join(repoRoot, '.cursor', 'rules', `${taskKey.toLowerCase()}.mdc`)
+    if (existsSync(filePath)) {
+      unlinkSync(filePath)
+      return filePath
+    }
+    return null
+  } catch {
+    return null
+  }
+}
+
 function parseGitStatus(porcelain) {
   const lines = porcelain.split('\n').filter(Boolean)
   const staged     = lines.filter(l => !' ?!'.includes(l[0])).map(l => ({ xy: l.slice(0, 2), file: l.slice(3) }))
@@ -1634,6 +1676,14 @@ If you have uncommitted tracked changes, it will tell you exactly what to do bef
           }
         } catch { /* non-fatal */ }
 
+        // Write cursor rules file to local repo so Cursor enforces them natively
+        let cursorRulesFile = null
+        const freshTaskForRules = await api.get(`/api/tasks/${taskId}`).catch(() => null)
+        const cursorRulesContent = freshTaskForRules?.data?.task?.cursorRules
+        if (cursorRulesContent?.trim()) {
+          cursorRulesFile = writeCursorRulesFile(freshTaskForRules.data.task.key, cursorRulesContent)
+        }
+
         const checkoutSteps = [
           'git fetch origin',
           `git checkout ${branchName}`,
@@ -1660,6 +1710,9 @@ If you have uncommitted tracked changes, it will tell you exactly what to do bef
           message:          statusMsg,
           gitSteps:         checkoutSteps,
           localStateNote,
+          cursorRulesFile:  cursorRulesFile
+            ? { written: true, path: cursorRulesFile, note: 'Task-specific Cursor rules written to this file. Cursor will enforce them automatically. The file will be deleted when you raise a PR.' }
+            : { written: false },
           nextStep:         'Run the git steps above to switch locally, then start coding. When commits are pushed, use raise_pr.',
         })
       } catch (e) {
@@ -1962,12 +2015,19 @@ Set confirmed=false first to preview the full PR content, then confirmed=true to
           draft,
         })
         if (!res?.success) return errorText(res?.message || 'Could not create PR')
+
+        // Delete the task-specific cursor rules file — coding is done
+        const deletedRulesFile = deleteCursorRulesFile(task.key)
+
         return text({
           prNumber: res.data.prNumber,
           prUrl:    res.data.prUrl,
           title:    prTitle,
           draft,
           message:  `PR #${res.data.prNumber} created.`,
+          cursorRulesCleared: deletedRulesFile
+            ? { cleared: true, path: deletedRulesFile, note: 'Task-specific Cursor rules file deleted — coding is complete.' }
+            : { cleared: false },
           nextStep: draft
             ? 'PR is a draft. Mark it ready for review on GitHub when you want reviewer notifications to fire.'
             : 'PR is live. The GitHub webhook will move the task to in_review and notify the reviewer within seconds.',

package/package.json

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