feat(provider): more obvious, helpful errors when unavailable

#68

Author: NickvanDyke

lua/opencode/api/command.lua

@@ -31,7 +31,8 @@ function M.command(command, callback)
     -- No need to register SSE here - commands don't trigger any.
     -- (except maybe the `input_*` commands? but no reason for user to use those).
 
-    require("opencode.provider").show()
+    -- Swallow errors - more of a preference than a requirement here
+    pcall(require("opencode.provider").show)
 
     require("opencode.cli.client").tui_execute_command(command, port, callback)
   end)

lua/opencode/api/prompt.lua

@@ -33,7 +33,8 @@ function M.prompt(prompt, opts, callback)
       return
     end
 
-    require("opencode.provider").show()
+    -- Swallow errors - more of a preference than a requirement here
+    pcall(require("opencode.provider").show)
 
     require("opencode.util").chain({
       function(next)

lua/opencode/cli/server.lua

@@ -174,7 +174,10 @@ function M.get_port(callback)
       end
     end,
     function(next)
-      require("opencode.provider").start()
+      local ok, result = pcall(require("opencode.provider").start)
+      if not ok then
+        vim.notify("Failed to start `opencode`: " .. result, vim.log.levels.ERROR)
+      end
       -- Always proceed - even if `opencode` wasn't started, failing to find it will give a more helpful error message.
       next()
     end,

lua/opencode/health.lua

@@ -1,8 +1,5 @@
 local M = {}
 
-local external_instructions =
-  "Or launch `opencode` with your own method and optionally override `vim.g.opencode_opts.provider` for convenience, then use `opencode.nvim` normally."
-
 function M.check()
   vim.health.start("opencode.nvim")
 
@@ -86,16 +83,18 @@ function M.check()
     if snacks.picker and snacks.config.get("terminal", {}).enabled then
       vim.health.ok("`snacks.terminal` is enabled: will default to `snacks` provider.")
     else
-      vim.health.warn("`snacks.terminal` is disabled: the `snacks` provider will not be available.", {
+      vim.health.warn("`snacks.terminal` is disabled: the default `snacks` provider will not be available.", {
         "Enable `snacks.terminal`",
-        external_instructions,
+        "Or launch and manage `opencode` yourself",
+        "Or configure `vim.g.opencode_opts.provider`",
       })
     end
   else
     vim.health.warn("`snacks.nvim` is not available: `ask()` and `select()` will not be enhanced.")
-    vim.health.warn("`snacks.nvim` is not available: the `snacks` provider will not be available.", {
+    vim.health.warn("`snacks.nvim` is not available: the default `snacks` provider will not be available.", {
       "Install `snacks.nvim`",
-      external_instructions,
+      "Or launch and manage `opencode` yourself",
+      "Or configure `vim.g.opencode_opts.provider`",
     })
   end
 end

lua/opencode/provider.lua

@@ -60,11 +60,7 @@ function M.toggle()
     provider:toggle()
     started = true
   else
-    vim.notify(
-      "No `provider.toggle` available — configure `vim.g.opencode_opts.provider`, or install `snacks.nvim` to use the default provider",
-      vim.log.levels.ERROR,
-      { title = "opencode" }
-    )
+    error("No `provider.toggle` available — run `:checkhealth opencode` for details", 0)
   end
 end
 
@@ -74,21 +70,19 @@ function M.start()
     provider:start()
     started = true
   else
-    vim.notify(
-      "No `provider.start` available — start `opencode` yourself, or configure `vim.g.opencode_opts.provider`, or install `snacks.nvim` to use the default provider",
-      vim.log.levels.ERROR,
-      { title = "opencode" }
-    )
+    error("No `provider.start` available — run `:checkhealth opencode` for details", 0)
   end
 end
 
 ---Show `opencode` via `opts.provider`.
---- Only called if `provider.toggle` or `provider.start` was previously called.
+---Only called if `provider.toggle` or `provider.start` was previously called.
 function M.show()
-  if provider and provider.show and started then
-    provider:show()
-  else
-    -- no-op - this is more of a preference than a requirement.
+  if started then
+    if provider and provider.show then
+      provider:show()
+    else
+      error("No `provider.show` available — run `:checkhealth opencode` for details", 0)
+    end
   end
 end