internaltool-mcp 1.6.34 → 1.6.35

package/index.js

@@ -7580,6 +7580,7 @@ async function runWorkspaceScan(taskId, task, project, overrideRepoPath) {
   // ── 1. Detect repo root ─────────────────────────────────────────────────────
   // Cursor starts the MCP with cwd = the folder the developer has open.
   // Walk up to find the .git root so we always scan the full repo, not a subfolder.
+  // Returns null if no .git found (avoids false positives from home-dir .git repos).
   function findGitRoot(startDir) {
     let dir = startDir
     for (let i = 0; i < 10; i++) {
@@ -7588,28 +7589,50 @@ async function runWorkspaceScan(taskId, task, project, overrideRepoPath) {
       if (parent === dir) break
       dir = parent
     }
-    return startDir
+    return null  // no .git found — caller must handle
   }
   const detectedRoot = findGitRoot(overrideRepoPath || process.cwd())
 
+  if (!detectedRoot) {
+    const startedFrom = overrideRepoPath || process.cwd()
+    return {
+      error: true,
+      message: `No .git directory found in or above "${startedFrom}". Make sure Cursor is open inside your project folder, or pass repoPath explicitly.`,
+      hint: `scan_workspace repoPath="/absolute/path/to/your/repo"`,
+    }
+  }
+
   // ── 2. Verify it's the right repo via git remote ────────────────────────────
   // Compares local "origin" remote to project.githubRepoFullName.
+  // Case-insensitive — GitHub repo names are not case-sensitive.
   // Prevents scanning the wrong repo when multiple projects are on the same machine.
+  const norm = url => url.replace(/^(https?:\/\/github\.com\/|git@github\.com:)/, '').replace(/\.git$/, '').trim().toLowerCase()
   let repoVerification = { checked: false, matched: null, localRemote: null, projectRepo: project.githubRepoFullName || null }
-  if (project.githubRepoFullName) {
-    try {
-      const remoteUrl = runGit('remote get-url origin', detectedRoot).trim()
-      const norm = url => url.replace(/^(https?:\/\/github\.com\/|git@github\.com:)/, '').replace(/\.git$/, '').trim()
-      const localRepo   = norm(remoteUrl)
-      const projectRepo = norm(project.githubRepoFullName)
-      repoVerification  = { checked: true, matched: localRepo === projectRepo, localRemote: localRepo, projectRepo }
+  try {
+    const remoteUrl = runGit('remote get-url origin', detectedRoot).trim()
+    const localRepo   = norm(remoteUrl)
+    const projectRepo = project.githubRepoFullName ? norm(project.githubRepoFullName) : null
+
+    if (projectRepo) {
+      // Project has a GitHub link — verify and block on mismatch
+      repoVerification = { checked: true, matched: localRepo === projectRepo, localRemote: localRepo, projectRepo }
       if (!repoVerification.matched) {
         return {
           error: true,
-          message: `Wrong repo. Cursor is open in "${localRepo}" but this task belongs to "${projectRepo}". Open Cursor in the correct repo or pass repoPath explicitly.`,
+          message: `Wrong repo — Cursor is open in "${localRepo}" but this task belongs to "${projectRepo}". Open Cursor in the correct repo or pass repoPath explicitly.`,
+          fix: [
+            `Option 1: Run scan_workspace repoPath="/path/to/${projectRepo.split('/')[1]}"`,
+            `Option 2: Make sure git remote origin is set to the correct repo`,
+          ],
         }
       }
-    } catch { /* git unavailable — skip verification */ }
+    } else {
+      // No GitHub link — record the remote as a hint so user can tell if it looks right
+      repoVerification = { checked: false, matched: null, localRemote: localRepo, projectRepo: null, warning: `Project has no GitHub repo linked — could not verify identity. Detected remote: "${localRepo}". If this is the wrong repo, pass repoPath explicitly.` }
+    }
+  } catch {
+    // git unavailable or no remote — surface the detected root as a hint
+    repoVerification = { checked: false, matched: null, localRemote: null, projectRepo: project.githubRepoFullName || null, warning: `Could not read git remote — make sure git is installed and origin is configured. Scanned: "${detectedRoot}".` }
   }
 
   // ── 3. Read .cursor/ subdirectories ─────────────────────────────────────────

package/package.json

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