git_add reports 'Files staged successfully' when nothing was staged

Aperta Adatta ai principianti
#4,763 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
74/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
git, python
Ambito
devtools

Direzione di ricerca

Inizia da src/git/src/mcp_server_git/server.py:132-153 ed esamina i test git_add esistenti, che attualmente coprono lo staging di una modifica reale. Riproduci i casi di lista vuota e albero pulito, quindi aggiungi test che dimostrino che il risultato riportato corrisponde all’indice e che una lista di file vuota viene gestita come specificato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Describe the bug

git_add returns the constant string Files staged successfully regardless of what the index actually took.

src/git/src/mcp_server_git/server.py:132-153:

def git_add(repo: git.Repo, files: list[str]) -> str:
    if files == ["."]:
        repo.git.add(".")
    else:
        ...
        repo.git.add("--", *files)
    return "Files staged successfully"

git add exits 0 when it stages nothing, so two calls report success having done nothing:

  • files: [] runs git add -- with no pathspec, which is a no-op. The schema puts no minimum on the list.
  • files: ["."] on a tree with no changes stages nothing.

A nonexistent path is not affected — git add -- nope.txt already fails with "pathspec did not match any files".

The practical failure: an agent computes a file list that comes back empty — a filter, a glob, a diff parse that matched nothing — and passes it. It is told the files are staged. Paired with #4762, it can then stage nothing, commit nothing, and report the work as committed.

To Reproduce
import git
repo = git.Repo(".")             # a tracked file, edited but not staged

repo.git.add("--")               # files: []
repo.git.diff("--cached", "--name-only")   # '' — nothing staged
# git_add returns: "Files staged successfully"

Same on a clean tree with files: ["."]:

tree clean?      True
git_add(["."]) -> "Files staged successfully"
actually staged: ''
Expected behavior

An empty files list should be rejected, since it can never stage anything. Otherwise the reported outcome should reflect what the index actually holds, rather than the exit status.

Additional context

No test covers either case; the existing git_add tests all stage a real change first.

Lingua principale
TypeScript
Stelle
90.5k
Fork
11.7k
Merge medio
2g 2h
PR unite (30g)
5

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di modelcontextprotocol/servers

Tutte le issue di modelcontextprotocol/servers

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.