fix(core): undoing my change to cancel requests on stop

I think I misunderstood how the code should work. We don't just want to
cancel our outstanding requests to the opencode server, we want the
opencode server to cancel it's upstream requests.

Author: cameronr

lua/opencode/core.lua

@@ -176,11 +176,14 @@ function M.stop()
   if state.windows and state.active_session then
     if state.is_running() then
       vim.notify('Aborting current request...', vim.log.levels.WARN)
-      state.api_client:abort_session(state.active_session.id):wait()
 
-      -- Forcibly reject any pending requests as it seems like they
-      -- can sometimes get stuck
-      server_job.cancel_all_requests()
+      -- FIXME: I think my understanding / logic was wrong here. We don't
+      -- just want to cancel our requests to the opencode server, we
+      -- want the opencode server to cance it's requests. Commenting out
+      -- this code for now and will do more testing
+      -- server_job.cancel_all_requests()
+
+      state.api_client:abort_session(state.active_session.id):wait()
     end
     require('opencode.ui.footer').clear()
     input_window.set_content('')

lua/opencode/server_job.lua

@@ -139,19 +139,11 @@ end
 ---Forcibly reject any pending requests (they sometimes get stuck
 ---after an api abort)
 function M.cancel_all_requests()
-  if vim.deep_equal(M.requests, {}) then
-    -- If we're canceling again and we've already cleared the requests, set the
-    -- job_count to 0
-    state.job_count = 0
-  else
-    for _, entry in ipairs(M.requests) do
-      local promise = entry[2]
-      if not promise:is_resolved() then
-        pcall(promise.reject, promise, 'Request cancelled')
-      end
+  for _, entry in ipairs(M.requests) do
+    local promise = entry[2]
+    if not promise:is_resolved() then
+      pcall(promise.reject, promise, 'Request cancelled')
     end
-
-    M.requests = {}
   end
 end