apply_patch changes have no Undo
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
Direzione di ricerca
Start with src/main/tools/apply_patch.ts and trace how write_file and edit_file return undo records, then follow the existing edit:undo handling. The proposal calls for backups and post-change hashes per changed file, with a single undo refused if any file has changed since; deleted and moved files also need records. Done means multi-file patches can be undone together while preserving that safety check.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Suggestion from the 2026-10-07 code review (not a bug).
write_file and edit_file return an undo record, so their cards offer Undo. apply_patch (src/main/tools/apply_patch.ts) returns none, so a multi-file patch can't be undone from the transcript, even though it is the tool the model is told to use for larger changes.
Proposal
Return one backup per changed file (before bytes plus the hash after the change) and let edit:undo restore them together. Refuse the undo when any file no longer matches, as single-file undo already does. Deleted and moved files need their own records.
- Lingua principale
- TypeScript
- Stelle
- 2
- Fork
- 2
- Merge medio
- 5h 28m
- PR unite (30g)
- 24
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PierrunoYT/patch
-
enhancement platform: windows priority: low severity: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
PierrunoYT/patch#198 ·
I maintainer di solito rispondono entro 1 giorno
-
priority: medium security severity: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
PierrunoYT/patch#66 ·
I maintainer di solito rispondono entro 1 giorno
-
bug platform: macos priority: low severity: low tests
Difficoltà 3/5 1-2 giorni Idoneità per principianti 56/100
PierrunoYT/patch#218 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement priority: low security severity: low
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
PierrunoYT/patch#211 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement platform: windows priority: low security severity: low
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
PierrunoYT/patch#207 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di PierrunoYT/patch
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 84/100
answerLoops/answerLoops#345 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
siyuan-note/siyuan#20313 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
LanternOps/breeze#8254 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
gofish-graphics/gofish-graphics#1084 ·
I maintainer di solito rispondono entro 1 giorno