@andrebuzeli/git-mcp 15.9.0 → 15.9.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andrebuzeli/git-mcp",
-  "version": "15.9.0",
+  "version": "15.9.1",
   "private": false,
   "description": "MCP server para Git com operações locais e sincronização paralela GitHub/Gitea",
   "license": "MIT",

package/src/utils/errors.js

@@ -16,14 +16,14 @@ export const ERROR_CODES = {
     suggestion: "Verifique se GITHUB_TOKEN está configurado e é válido. Gere um novo token em github.com/settings/tokens"
   },
   AUTH_GITEA_INVALID: {
-    code: "AUTH_GITEA_INVALID", 
+    code: "AUTH_GITEA_INVALID",
     suggestion: "Verifique se GITEA_TOKEN e GITEA_URL estão configurados corretamente"
   },
   AUTH_NO_PERMISSION: {
     code: "AUTH_NO_PERMISSION",
     suggestion: "Token não tem permissão para esta operação. Verifique os scopes do token"
   },
-  
+
   // Repo
   REPO_NOT_FOUND: {
     code: "REPO_NOT_FOUND",
@@ -37,7 +37,7 @@ export const ERROR_CODES = {
     code: "REPO_ALREADY_EXISTS",
     suggestion: "Repositório já existe. Use action='ensure' para verificar ou escolha outro nome"
   },
-  
+
   // Git Local
   NOT_A_GIT_REPO: {
     code: "NOT_A_GIT_REPO",
@@ -55,7 +55,7 @@ export const ERROR_CODES = {
     code: "NOTHING_TO_PUSH",
     suggestion: "Nenhuma mudança para push. Faça commits primeiro"
   },
-  
+
   // Branches
   BRANCH_NOT_FOUND: {
     code: "BRANCH_NOT_FOUND",
@@ -69,7 +69,7 @@ export const ERROR_CODES = {
     code: "CANNOT_DELETE_CURRENT",
     suggestion: "Não pode deletar branch atual. Faça checkout para outra branch primeiro"
   },
-  
+
   // Tags
   TAG_NOT_FOUND: {
     code: "TAG_NOT_FOUND",
@@ -79,7 +79,7 @@ export const ERROR_CODES = {
     code: "TAG_ALREADY_EXISTS",
     suggestion: "Tag já existe. Use outro nome ou delete a existente primeiro"
   },
-  
+
   // Refs
   REF_NOT_FOUND: {
     code: "REF_NOT_FOUND",
@@ -89,7 +89,7 @@ export const ERROR_CODES = {
     code: "INSUFFICIENT_HISTORY",
     suggestion: "Histórico insuficiente para HEAD~N. Use action='log' para verificar quantos commits existem"
   },
-  
+
   // Stash
   NOTHING_TO_STASH: {
     code: "NOTHING_TO_STASH",
@@ -99,7 +99,7 @@ export const ERROR_CODES = {
     code: "STASH_NOT_FOUND",
     suggestion: "Stash não encontrado. Use action='list' para ver stashes disponíveis"
   },
-  
+
   // Remote
   REMOTE_NOT_FOUND: {
     code: "REMOTE_NOT_FOUND",
@@ -113,7 +113,7 @@ export const ERROR_CODES = {
     code: "MERGE_CONFLICT",
     suggestion: "Conflito de merge. Resolva conflitos manualmente e faça novo commit"
   },
-  
+
   // Network
   NETWORK_TIMEOUT: {
     code: "NETWORK_TIMEOUT",
@@ -123,7 +123,7 @@ export const ERROR_CODES = {
     code: "RATE_LIMIT",
     suggestion: "Rate limit excedido. Aguarde alguns minutos antes de tentar novamente"
   },
-  
+
   // Validation
   VALIDATION_ERROR: {
     code: "VALIDATION_ERROR",
@@ -133,13 +133,13 @@ export const ERROR_CODES = {
     code: "MISSING_PARAMETER",
     suggestion: "Parâmetro obrigatório faltando"
   },
-  
+
   // Files
   FILE_NOT_FOUND: {
     code: "FILE_NOT_FOUND",
     suggestion: "Arquivo não encontrado no repositório. Use action='list' para ver arquivos disponíveis"
   },
-  
+
   // Issues/PRs
   ISSUE_NOT_FOUND: {
     code: "ISSUE_NOT_FOUND",
@@ -149,7 +149,7 @@ export const ERROR_CODES = {
     code: "PR_NOT_FOUND",
     suggestion: "Pull Request não encontrado. Use action='list' para ver PRs disponíveis"
   },
-  
+
   // Filesystem
   DISK_FULL: {
     code: "DISK_FULL",
@@ -163,7 +163,7 @@ export const ERROR_CODES = {
     code: "PERMISSION_DENIED_FS",
     suggestion: "Sem permissão para acessar arquivo/diretório. Verifique permissões do sistema de arquivos"
   },
-  
+
   // Git Corruption
   GIT_CORRUPTED: {
     code: "GIT_CORRUPTED",
@@ -171,9 +171,9 @@ export const ERROR_CODES = {
   },
   GIT_LOCK_FILE: {
     code: "GIT_LOCK_FILE",
-    suggestion: "Arquivo de lock presente (.git/index.lock). Outro processo git pode estar rodando. Se não, delete o arquivo de lock manualmente"
+    suggestion: "Arquivo de lock presente (.git/index.lock). Outro processo git pode estar rodando (comum em drives de rede). O sistema tentou auto-corrigir, mas se persistir, delete o arquivo de lock manualmente."
   },
-  
+
   // Remote/URL
   INVALID_REMOTE_URL: {
     code: "INVALID_REMOTE_URL",
@@ -187,7 +187,7 @@ export const ERROR_CODES = {
     code: "SSL_ERROR",
     suggestion: "Erro de SSL/TLS. Verifique certificados ou use http.sslVerify=false (não recomendado)"
   },
-  
+
   // Generic
   UNKNOWN_ERROR: {
     code: "UNKNOWN_ERROR",
@@ -207,14 +207,14 @@ export function createError(errorCode, details = {}) {
 // Função para mapear erros externos para códigos internos
 export function mapExternalError(error, context = {}) {
   const msg = error?.message?.toLowerCase() || String(error).toLowerCase();
-  
+
   // GitHub/Gitea API errors
   if (msg.includes("bad credentials") || msg.includes("401")) {
-    return context.provider === "gitea" 
+    return context.provider === "gitea"
       ? createError("AUTH_GITEA_INVALID", { originalError: msg })
       : createError("AUTH_GITHUB_INVALID", { originalError: msg });
   }
-  
+
   if (msg.includes("not found") || msg.includes("404")) {
     if (context.type === "repo") return createError("REPO_NOT_FOUND", { repo: context.repo, originalError: msg });
     if (context.type === "branch") return createError("BRANCH_NOT_FOUND", { branch: context.branch, originalError: msg });
@@ -224,116 +224,116 @@ export function mapExternalError(error, context = {}) {
     if (context.type === "pr") return createError("PR_NOT_FOUND", { number: context.number, originalError: msg });
     return createError("REF_NOT_FOUND", { ref: context.ref, originalError: msg });
   }
-  
+
   if (msg.includes("permission") || msg.includes("403") || msg.includes("forbidden")) {
     return createError("AUTH_NO_PERMISSION", { originalError: msg });
   }
-  
+
   if (msg.includes("rate limit") || msg.includes("429")) {
     return createError("RATE_LIMIT", { originalError: msg });
   }
-  
+
   if (msg.includes("timeout") || msg.includes("etimedout") || msg.includes("econnrefused")) {
     return createError("NETWORK_TIMEOUT", { originalError: msg });
   }
-  
+
   if (msg.includes("push rejected") || msg.includes("non-fast-forward")) {
     return createError("PUSH_REJECTED", { originalError: msg });
   }
-  
+
   if (msg.includes("conflict")) {
     return createError("MERGE_CONFLICT", { originalError: msg });
   }
-  
+
   if (msg.includes("already exists")) {
     if (context.type === "repo") return createError("REPO_ALREADY_EXISTS", { repo: context.repo, originalError: msg });
     if (context.type === "branch") return createError("BRANCH_ALREADY_EXISTS", { branch: context.branch, originalError: msg });
     if (context.type === "tag") return createError("TAG_ALREADY_EXISTS", { tag: context.tag, originalError: msg });
     return createError("BRANCH_ALREADY_EXISTS", { originalError: msg });
   }
-  
+
   // isomorphic-git specific errors
   if (msg.includes("could not find") && msg.includes("head~")) {
     return createError("INSUFFICIENT_HISTORY", { originalError: msg });
   }
-  
+
   if (msg.includes("is not a git repository") || msg.includes("could not find .git")) {
     return createError("NOT_A_GIT_REPO", { path: context.path, originalError: msg });
   }
-  
+
   if (msg.includes("nothing to commit") || msg.includes("working tree clean")) {
     return createError("NOTHING_TO_COMMIT", { originalError: msg });
   }
-  
+
   if (msg.includes("could not resolve ref") || msg.includes("invalid reference")) {
     return createError("REF_NOT_FOUND", { originalError: msg });
   }
-  
+
   if (msg.includes("branch") && (msg.includes("does not exist") || msg.includes("not found"))) {
     return createError("BRANCH_NOT_FOUND", { originalError: msg });
   }
-  
+
   if (msg.includes("tag") && (msg.includes("does not exist") || msg.includes("not found"))) {
     return createError("TAG_NOT_FOUND", { originalError: msg });
   }
-  
+
   if (msg.includes("no commits") || msg.includes("unknown revision")) {
     return createError("NO_COMMITS", { originalError: msg });
   }
-  
+
   if (msg.includes("nada para stash") || msg.includes("nothing to stash") || msg.includes("no local changes")) {
     return createError("NOTHING_TO_STASH", { originalError: msg });
   }
-  
+
   if (msg.includes("stash") && (msg.includes("not found") || msg.includes("não encontrado"))) {
     return createError("STASH_NOT_FOUND", { originalError: msg });
   }
-  
+
   if (msg.includes("author") || msg.includes("committer")) {
-    return createError("VALIDATION_ERROR", { 
-      message: "Configure user.name e user.email", 
+    return createError("VALIDATION_ERROR", {
+      message: "Configure user.name e user.email",
       suggestion: "Use git-config para configurar ou forneça tokens de provider",
-      originalError: msg 
+      originalError: msg
     });
   }
-  
+
   // Filesystem errors
   if (msg.includes("enospc") || msg.includes("no space left") || msg.includes("disk full")) {
     return createError("DISK_FULL", { originalError: msg });
   }
-  
+
   if (msg.includes("file too large") || msg.includes("exceeds") || msg.includes("size limit")) {
     return createError("FILE_TOO_LARGE", { originalError: msg });
   }
-  
+
   if (msg.includes("eacces") || msg.includes("eperm") || (msg.includes("permission") && msg.includes("denied"))) {
     return createError("PERMISSION_DENIED_FS", { originalError: msg });
   }
-  
+
   // Git corruption
-  if (msg.includes("corrupt") || msg.includes("bad object") || msg.includes("broken") || 
-      msg.includes("missing object") || msg.includes("invalid sha") || msg.includes("fatal: packed")) {
+  if (msg.includes("corrupt") || msg.includes("bad object") || msg.includes("broken") ||
+    msg.includes("missing object") || msg.includes("invalid sha") || msg.includes("fatal: packed")) {
     return createError("GIT_CORRUPTED", { originalError: msg });
   }
-  
+
   if (msg.includes(".git/index.lock") || msg.includes("unable to create") && msg.includes("lock")) {
     return createError("GIT_LOCK_FILE", { originalError: msg });
   }
-  
+
   // Remote/URL errors
   if (msg.includes("invalid url") || msg.includes("malformed") || msg.includes("not a valid")) {
     return createError("INVALID_REMOTE_URL", { originalError: msg });
   }
-  
-  if (msg.includes("could not resolve host") || msg.includes("name resolution") || 
-      msg.includes("no address") || msg.includes("unreachable")) {
+
+  if (msg.includes("could not resolve host") || msg.includes("name resolution") ||
+    msg.includes("no address") || msg.includes("unreachable")) {
     return createError("REMOTE_UNREACHABLE", { originalError: msg });
   }
-  
+
   if (msg.includes("ssl") || msg.includes("certificate") || msg.includes("tls")) {
     return createError("SSL_ERROR", { originalError: msg });
   }
-  
+
   // Generic fallback
   return createError("UNKNOWN_ERROR", { originalError: msg, context });
 }
@@ -347,27 +347,27 @@ export const NEXT_ACTIONS = {
   "workflow:add": { tool: "git-workflow", action: "commit", hint: "Crie um commit com uma mensagem descritiva" },
   "workflow:commit": { tool: "git-workflow", action: "push", hint: "Envie as mudanças para GitHub/Gitea" },
   "workflow:push": null, // Fluxo completo
-  
+
   // git-branches
   "branches:create": { tool: "git-branches", action: "checkout", hint: "Mude para a nova branch criada" },
   "branches:checkout": { tool: "git-workflow", action: "status", hint: "Verifique o estado da branch" },
   "branches:delete": null,
-  
+
   // git-stash
   "stash:save": { tool: "git-branches", action: "checkout", hint: "Agora pode trocar de branch com segurança" },
   "stash:pop": { tool: "git-workflow", action: "status", hint: "Verifique as mudanças restauradas" },
-  
+
   // git-merge
   "merge:merge": { tool: "git-workflow", action: "push", hint: "Envie o merge para os remotes" },
-  
+
   // git-tags
   "tags:create": { tool: "git-tags", action: "push", hint: "Envie a tag para os remotes" },
   "tags:push": null,
-  
+
   // git-remote
   "remote:ensure": { tool: "git-workflow", action: "push", hint: "Agora pode fazer push" },
   "remote:release-create": null,
-  
+
   // git-sync
   "sync:pull": { tool: "git-workflow", action: "status", hint: "Verifique mudanças após pull" },
   "sync:fetch": { tool: "git-sync", action: "pull", hint: "Aplique as mudanças baixadas" },
@@ -388,7 +388,7 @@ export function getNextAction(tool, action) {
 export function asToolResult(result, options = {}) {
   const { tool, action } = options;
   const response = { ...result };
-  
+
   // Adicionar próxima ação sugerida se disponível
   if (tool && action) {
     const next = getNextAction(tool, action);
@@ -398,7 +398,7 @@ export function asToolResult(result, options = {}) {
       response._flowComplete = true;
     }
   }
-  
+
   return {
     content: [{ type: "text", text: JSON.stringify(response, null, 2) }],
     isError: false,

package/src/utils/gitAdapter.js

@@ -28,7 +28,11 @@ export class GitAdapter {
     constructor(providerManager) {
         this.pm = providerManager;
         this.gitPath = "git"; // Initial default
-        this.gitEnv = { ...process.env, LANG: "en_US.UTF-8" }; // Base env
+        this.gitEnv = {
+            ...process.env,
+            LANG: "en_US.UTF-8",
+            GIT_OPTIONAL_LOCKS: "0", // Evita operações de background que seguram lock (importante para drives de rede)
+        };
         this.timeout = GIT_TIMEOUT;
         this.resolvePromise = this._resolveGit();
     }
@@ -145,13 +149,22 @@ export class GitAdapter {
 
                         if (shouldDelete) {
                             console.error("[GitAdapter] Auto-fix: removing stale .git/index.lock...");
-                            fs.unlinkSync(lockPath);
-                            // Espera um pouco para o sistema de arquivos (especialmente SMB) registrar
-                            await new Promise(r => setTimeout(r, 1000));
-                            return await this._exec(dir, args, { ...options, _lockRetried: true });
+                            try {
+                                fs.unlinkSync(lockPath);
+                            } catch (unlinkErr) {
+                                console.error("[GitAdapter] Failed to delete lock file:", unlinkErr.message);
+                                // Set shouldDelete to false so we don't try to retry if deletion failed
+                                shouldDelete = false;
+                            }
+
+                            if (shouldDelete) {
+                                // Espera um pouco para o sistema de arquivos (especialmente SMB) registrar
+                                await new Promise(r => setTimeout(r, 2000));
+                                return await this._exec(dir, args, { ...options, _lockRetried: true });
+                            }
                         }
-                    } catch (unlinkError) {
-                        console.error("[GitAdapter] Failed to check/remove lock file:", unlinkError.message);
+                    } catch (fsError) {
+                        console.error("[GitAdapter] Error handling lock file:", fsError.message);
                         // Continua para lançar o erro original
                     }
                 }