$E_FN crashes or uses the wrong file outside :ExecutorRun

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#45 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
76/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
lua, neovim
Bereich
devtools

Rechercherichtung

Lies zuerst run_one_off_cmd, set_task_command und get_relative_buffer_name, bei denen das Issue den veralteten oder nil Buffer-Pfad meldet. Reproduziere beide Fälle in Neovim: Führe :ExecutorOneOff vor :ExecutorRun aus und führe es dann nach :ExecutorRun aus einem anderen Buffer aus. Aktualisiere die Buffer-Auswahl für die betroffenen Pfade und überprüfe, dass $E_FN die aktuelle Datei ohne Absturz verwendet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

: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.

Vorherrschende Sprache
Lua
Sterne
161
Forks
12
Ø Merge
25 Min.
Gemergte PRs (30 T.)
1

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus google/executor.nvim

Alle Issues in google/executor.nvim

Ähnliche Issues

Weitere Issues zu Lua

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.