$E_FN crashes or uses the wrong file outside :ExecutorRun

Aperta Adatta ai principianti
  • Qualcosa non va?
#45 0 commenti 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
76/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
lua, neovim
Ambito
devtools

Direzione di ricerca

Inizia leggendo run_one_off_cmd, set_task_command e get_relative_buffer_name, dove l’issue segnala un percorso del buffer obsoleto o nil. Riproduci entrambi i casi in Neovim: esegui :ExecutorOneOff prima di :ExecutorRun, poi eseguilo da un buffer diverso dopo :ExecutorRun. Aggiorna la selezione del buffer per i percorsi interessati e verifica che $E_FN usi il file corrente senza andare in crash.

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

Descrizione

:ExecutorRun records the current buffer, and $E_FN uses that recorded buffer. Other ways of running or setting a command do not record it first, so $E_FN either crashes or uses the wrong file.

Steps to reproduce

Crash

  1. Start Neovim and open any file.
  2. Before running :ExecutorRun, run :ExecutorOneOff echo $E_FN.
  3. The command fails with Invalid 'buf': Expected Lua number. The error comes from nvim_buf_get_name(nil) in get_relative_buffer_name.

Wrong file

  1. Open a.js and run :ExecutorRun once.
  2. Switch to b.js.
  3. Run :ExecutorOneOff echo $E_FN. The command runs on a.js instead of b.js.

Affected paths

last_active_buf_id is set only in Executor.run(). These paths read it before it is set, or after it has gone stale:

  • :ExecutorOneOff (run_one_off_cmd).
  • Choosing a preset. set_task_command runs before Executor.run() records the buffer.
  • Choosing an entry from the history picker. This path also calls set_task_command before Executor.run().
  • require("executor").api.set_task_command and api.run_task.

Possible fix

Record the current buffer in run_one_off_cmd and set_task_command. Alternatively, read the current buffer directly when last_active_buf_id is nil.

Lingua principale
Lua
Stelle
161
Fork
12
Merge medio
25m
PR unite (30g)
1

Preparare l'ambiente

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 google/executor.nvim

Tutte le issue di google/executor.nvim

Issue simili

Altre issue su Lua

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.