Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

🤖 fix: retry barrier keyboard gaps (no Retry shortcut; Esc on context_exceeded disables auto-retry)

Aperta
#5,111 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

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
68/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
typescript, vscode

Direzione di ricerca

Start with src/browser/components/ChatPane/retryBarrierDerivation.ts, ChatPane, RetryBarrier, and useAIViewKeybinds; compare the desktop behavior with the webview fix in #5108. Trace how manual Retry and Shift+R are handled, then verify that context_exceeded with no visible barrier produces no interruptStream call. Done means the visible Retry action has a keyboard shortcut in both surfaces and the regression test passes.

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

Descrizione

Found by Codex review on #5108 (VS Code webview retry barrier), which fixed the webview side. Both are pre-existing on desktop ChatPane.

Problems

  1. Retry has no keyboard shortcut. The RetryBarrier's Retry button (after a stream error) has no keybind. Shift+R (KEYBINDS.RESUME_STREAM) only resumes a user-aborted turn through the interrupted divider (useAIViewKeybinds gets canResumeInterruptedStream, which is false while the RetryBarrier is visible). AGENTS.md asks for a shortcut for every operation. The webview mirrors desktop here, so both lack it.
  2. Esc on a context_exceeded error disables auto-retry. ChatPane passes the raw showRetryBarrier to useAIViewKeybinds. For a context_exceeded stream error the shared derivation (src/browser/components/ChatPane/retryBarrierDerivation.ts) keeps showRetryBarrier true but suppresses the barrier (showRetryBarrierUI false), so Esc calls stopStream(..., { disableAutoRetry: true }) although nothing visible can be stopped, and the backend persists the opt-out. #5108 gates the webview's Esc on showRetryBarrierUI.

Proposed fix

  1. Route Shift+R (or a dedicated keybind) to the visible RetryBarrier's manual Retry on desktop and in the webview, sharing one handler (for example by lifting the manual-retry action out of RetryBarrierContent or exposing it through a callback ref).
  2. Pass showRetryBarrierUI instead of showRetryBarrier to useAIViewKeybinds, with a test that Esc on a context_exceeded error makes no interruptStream call.

Refs #4968


Generated with xum • Model: anthropic:claude-opus-5-5 • Thinking: high

Lingua principale
TypeScript
Stelle
2k
Fork
136
Merge medio
7h 27m
PR unite (30g)
710

Preparare l'ambiente

Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 coder/xum

Tutte le issue di coder/xum

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.