fix(server_job): set job_count to #M.requests

I've still seen occasional issues where job_count is either too high or
too low. I think this is the most correct way to set it.

Also, have to wrap notifys since we're in a fast context at that point.

Author: cameronr

lua/opencode/server_job.lua

@@ -37,44 +37,46 @@ function M.call_api(url, method, body)
         if err then
           local ok, pcall_err = pcall(call_promise.reject, call_promise, err)
           if not ok then
-            vim.notify('Error while handling API error response: ' .. vim.inspect(pcall_err))
+            vim.schedule(function()
+              vim.notify('Error while handling API error response: ' .. vim.inspect(pcall_err))
+            end)
           end
-          state.job_count = state.job_count - 1
         else
           local ok, pcall_err = pcall(call_promise.resolve, call_promise, result)
           if not ok then
-            vim.notify('Error while handling API response: ' .. vim.inspect(pcall_err))
+            vim.schedule(function()
+              vim.notify('Error while handling API response: ' .. vim.inspect(pcall_err))
+            end)
           end
-          state.job_count = state.job_count - 1
         end
       end)
     end,
     on_error = function(err)
       local ok, pcall_err = pcall(call_promise.reject, call_promise, err)
       if not ok then
-        vim.notify('Error while handling API on_error: ' .. vim.inspect(pcall_err))
+        vim.schedule(function()
+          vim.notify('Error while handling API on_error: ' .. vim.inspect(pcall_err))
+        end)
       end
-      state.job_count = state.job_count - 1
     end,
   }
 
   if body ~= nil then
     opts.body = body and vim.json.encode(body) or '{}'
   end
 
-  -- For promise tracking, remove promises that complete from requests
-  -- NOTE: can remove the request tracking code when we're happy with
-  -- request reliability
   local request_entry = { opts, call_promise }
   table.insert(M.requests, request_entry)
 
+  -- Remove completed promises from list, update job_count
   local function remove_from_requests()
     for i, entry in ipairs(M.requests) do
       if entry == request_entry then
         table.remove(M.requests, i)
         break
       end
     end
+    state.job_count = #M.requests
   end
 
   call_promise:and_then(function(result)