From ac2a442f1228f8ab8c53b9b25c58a36da8905b69 Mon Sep 17 00:00:00 2001 From: Mika Vilpas Date: Fri, 19 Jul 2024 12:27:06 +0300 Subject: [PATCH] fix: crash without newest yazi There was a bug in opting in to the feature from 8114817 (feat: add targeted communication with the yazi instance (opt-in) (#225)). Even if you had not set `use_yazi_client_id_flag = true` in your config, the newest yazi was still required. This commit fixes that bug by removing the passing of the `yazi_id` (roll forward style). Later changes can still utilize the feature and build on top of this. --- lua/yazi/process/ya_process.lua | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lua/yazi/process/ya_process.lua b/lua/yazi/process/ya_process.lua index d81edc49..958e240e 100644 --- a/lua/yazi/process/ya_process.lua +++ b/lua/yazi/process/ya_process.lua @@ -32,12 +32,13 @@ end ---@param path Path ---@param yazi_id string +---@diagnostic disable-next-line: unused-local +-- selene: allow(unused_variable) function YaProcess:get_yazi_command(path, yazi_id) return string.format( - 'yazi %s --chooser-file "%s" --client-id "%s"', + 'yazi %s --chooser-file "%s"', vim.fn.shellescape(path.filename), - self.config.chosen_file_path, - yazi_id + self.config.chosen_file_path ) end