Renderer actions that fail silently or run twice: delete/clear while busy, secret Remove, Ctrl+Enter commit
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 74/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- desktop
Direzione di ricerca
Start with the named handlers in src/renderer/src/views/dialogs.ts and the commit flow in src/renderer/src/views/panels.ts; read how they report errors and track in-flight work. Check src/main/chat_manager.ts:450 for the busy-state failure. Done means failures are shown to the user and a second commit cannot run before the first finishes; verify the affected actions and Ctrl+Enter behavior.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Found in the 2026-10-07 code review (read).
Problem
- Silent failures:
history:deleteandhistory:clearthrow while the agent is busy (src/main/chat_manager.ts:450). Their asynconclickhandlers (src/renderer/src/views/dialogs.ts:652-656,697-701) and the secret "Remove" button (dialogs.ts:96-100) have no catch. The user sees nothing; the error only reaches the renderer error log. - Double commit: Ctrl+Enter in the commit message box calls
commit()with no in-flight guard (src/renderer/src/views/panels.ts:366-371,610-640), and arefresh()during a slow push re-enables the button. Pressing twice gives a "nothing to commit" error or two concurrent pushes.
Fix
- Wrap these handlers in try/catch and show the error.
- Add a
committingflag that blocks a second commit until the first finishes.
- 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 priority: low security severity: low
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
PierrunoYT/patch#260 ·
I maintainer di solito rispondono entro 1 giorno
-
bug priority: low severity: low
Difficoltà 3/5 1-2 giorni Idoneità per principianti 25/100
PierrunoYT/patch#259 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
platform: macos priority: medium security severity: low tests
Difficoltà 4/5 3-5 giorni Idoneità per principianti 15/100
PierrunoYT/patch#257 ·
I maintainer di solito rispondono entro 1 giorno
-
bug platform: windows priority: medium severity: medium
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
PierrunoYT/patch#242 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
bug priority: high severity: high
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
PierrunoYT/patch#240 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di PierrunoYT/patch
Issue simili
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
lingdojo/kana-dojo#32018 · 1 commento · 5 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
paperclipai/paperclip#15751 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
BuilderIO/agent-native#7275 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno