Merge pull request #1 from you-n-g/patch-2

Porting pending PR: jackMort#300

lua/chatgpt/api.lua

@@ -193,24 +193,24 @@ end
 
 local function loadConfigFromCommand(command, optionName, callback, defaultValue)
   local cmd = splitCommandIntoTable(command)
-  job
-    :new({
-      command = cmd[1],
-      args = vim.list_slice(cmd, 2, #cmd),
-      on_exit = function(j, exit_code)
-        if exit_code ~= 0 then
-          logger.warn("Config '" .. optionName .. "' did not return a value when executed")
-          return
-        end
-        local value = j:result()[1]:gsub("%s+$", "")
-        if value ~= nil and value ~= "" then
-          callback(value)
-        elseif defaultValue ~= nil and defaultValue ~= "" then
-          callback(defaultValue)
-        end
-      end,
-    })
-    :start()
+  local j = job:new({
+    command = cmd[1],
+    args = vim.list_slice(cmd, 2, #cmd),
+    on_exit = function(j, exit_code)
+      if exit_code ~= 0 then
+        logger.warn("Config '" .. optionName .. "' did not return a value when executed")
+        return
+      end
+      local value = j:result()[1]:gsub("%s+$", "")
+      if value ~= nil and value ~= "" then
+        callback(value)
+      elseif defaultValue ~= nil and defaultValue ~= "" then
+        callback(defaultValue)
+      end
+    end,
+  })
+  j:start()
+  j:wait() -- we should wait the job to finish. Otherwise we can't ensure correctly reading config before using it.
 end
 
 local function loadConfigFromEnv(envName, configName, callback)