refactor(state): rename opencode_server_job

It's started by server_job but opencode_server seems like the better
name as it's an OpencodeServer

Author: cameronr

lua/opencode/api_client.lua

@@ -23,12 +23,12 @@ end
 function OpencodeApiClient:_call(endpoint, method, body, query)
   if not self.base_url then
     local state = require('opencode.state')
-    state.opencode_server_job = server_job.ensure_server() --[[@as OpencodeServer]]
+    state.opencode_server = server_job.ensure_server() --[[@as OpencodeServer]]
     -- shouldn't normally happen but prevents error in replay tester
-    if not state.opencode_server_job then
+    if not state.opencode_server then
       return nil
     end
-    self.base_url = state.opencode_server_job.url:gsub('/$', '')
+    self.base_url = state.opencode_server.url:gsub('/$', '')
   end
   local url = self.base_url .. endpoint
 
@@ -363,7 +363,7 @@ end
 function OpencodeApiClient:subscribe_to_events(directory, on_event)
   if not self.base_url then
     local state = require('opencode.state')
-    self.base_url = state.opencode_server_job.url:gsub('/$', '')
+    self.base_url = state.opencode_server.url:gsub('/$', '')
   end
   local url = self.base_url .. '/event'
   if directory then

lua/opencode/core.lua

@@ -40,8 +40,8 @@ end
 function M.open(opts)
   opts = opts or { focus = 'input', new_session = false }
 
-  if not state.opencode_server_job or not state.opencode_server_job:is_running() then
-    state.opencode_server_job = server_job.ensure_server() --[[@as OpencodeServer]]
+  if not state.opencode_server or not state.opencode_server:is_running() then
+    state.opencode_server = server_job.ensure_server() --[[@as OpencodeServer]]
   end
 
   local are_windows_closed = state.windows == nil

lua/opencode/event_manager.lua

@@ -232,7 +232,7 @@ function EventManager:start()
   self.is_started = true
 
   state.subscribe(
-    'opencode_server_job',
+    'opencode_server',
     --- @param key string
     --- @param current OpencodeServer|nil
     --- @param prev OpencodeServer|nil

lua/opencode/opencode_server.lua

@@ -19,8 +19,8 @@ function OpencodeServer.new()
     group = vim.api.nvim_create_augroup('OpencodeVimLeavePre', { clear = true }),
     callback = function()
       local state = require('opencode.state')
-      if state.opencode_server_job then
-        state.opencode_server_job:shutdown()
+      if state.opencode_server then
+        state.opencode_server:shutdown()
       end
     end,
   })

lua/opencode/server_job.lua

@@ -148,17 +148,17 @@ function M.cancel_all_requests()
 end
 
 function M.ensure_server()
-  if state.opencode_server_job and state.opencode_server_job:is_running() then
-    return state.opencode_server_job
+  if state.opencode_server and state.opencode_server:is_running() then
+    return state.opencode_server
   end
 
   local promise = Promise.new()
-  state.opencode_server_job = opencode_server.new()
+  state.opencode_server = opencode_server.new()
 
   ---@diagnostic disable-next-line: missing-fields
-  state.opencode_server_job:spawn({
+  state.opencode_server:spawn({
     on_ready = function(_, base_url)
-      promise:resolve(state.opencode_server_job)
+      promise:resolve(state.opencode_server)
     end,
     on_error = function(err)
       promise:reject(err)

lua/opencode/state.lua

@@ -30,7 +30,7 @@ local config = require('opencode.config')
 ---@field cost number
 ---@field tokens_count number
 ---@field job_count number
----@field opencode_server_job OpencodeServer|nil
+---@field opencode_server OpencodeServer|nil
 ---@field api_client OpencodeApiClient
 ---@field event_manager EventManager|nil
 ---@field required_version string
@@ -69,7 +69,7 @@ local _state = {
   tokens_count = 0,
   -- job
   job_count = 0,
-  opencode_server_job = nil,
+  opencode_server = nil,
   api_client = nil,
   event_manager = nil,
 

tests/unit/core_spec.lua

@@ -83,7 +83,7 @@ describe('opencode.core', function()
     mock_api_client()
 
     -- Mock server job to avoid trying to start real server
-    state.opencode_server_job = {
+    state.opencode_server = {
       is_running = function()
         return true
       end,