fix(user_commands): respect active model for user_commands

User commands will now use either the configured model for the command or the
active_model.

This should fix #129

Author: sudo-tee

lua/opencode/api.lua

@@ -566,6 +566,15 @@ end
 --- @param args? string[] Additional arguments to pass to the command.
 function M.run_user_command(name, args)
   M.open_input()
+  local user_commands = config_file.get_user_commands()
+  local command_cfg = user_commands and user_commands[name]
+  if not command_cfg then
+    vim.notify('Unknown user command: ' .. name, vim.log.levels.WARN)
+    return
+  end
+
+  local model = command_cfg.model or state.current_model
+  local agent = command_cfg.agent or state.current_mode
 
   if not state.active_session then
     vim.notify('No active session', vim.log.levels.WARN)
@@ -575,6 +584,8 @@ function M.run_user_command(name, args)
     :send_command(state.active_session.id, {
       command = name,
       arguments = table.concat(args or {}, ' '),
+      model = model,
+      agent = agent,
     })
     :and_then(function()
       vim.schedule(function()

tests/unit/api_spec.lua

@@ -433,6 +433,81 @@ describe('opencode.api', function()
       config_file.get_user_commands = original_get_user_commands
     end)
 
+    describe('user command model/agent selection', function()
+      it('invokes run with correct model and agent', function()
+        local config_file = require('opencode.config_file')
+        local original_get_user_commands = config_file.get_user_commands
+
+        config_file.get_user_commands = function()
+          return {
+            ['test-no-model'] = { description = 'Run tests', template = 'Run tests with $ARGUMENTS' },
+            ['test-with-model'] = {
+              description = 'Run tests',
+              template = 'Run tests with $ARGUMENTS',
+              model = 'openai/gpt-4',
+              agent = 'tester',
+            },
+          }
+        end
+
+        local original_active_session = state.active_session
+        state.active_session = { id = 'test-session' }
+
+        local original_api_client = state.api_client
+        local send_command_calls = {}
+        state.api_client = {
+          send_command = function(self, session_id, command_data)
+            table.insert(send_command_calls, { session_id = session_id, command_data = command_data })
+            return {
+              and_then = function()
+                return {}
+              end,
+            }
+          end,
+        }
+
+        stub(api, 'open_input')
+
+        local slash_commands = api.get_slash_commands()
+
+        local test_no_model_cmd = nil
+        local test_with_model_cmd = nil
+
+        for _, cmd in ipairs(slash_commands) do
+          if cmd.slash_cmd == '/test-no-model' then
+            test_no_model_cmd = cmd
+          elseif cmd.slash_cmd == '/test-with-model' then
+            test_with_model_cmd = cmd
+          end
+        end
+
+        assert.truthy(test_no_model_cmd, 'Should find /test-no-model command')
+        assert.truthy(test_with_model_cmd, 'Should find /test-with-model command')
+
+        test_no_model_cmd.fn()
+        assert.equal(1, #send_command_calls)
+        assert.equal('test-session', send_command_calls[1].session_id)
+        assert.equal('test-no-model', send_command_calls[1].command_data.command)
+        assert.equal('', send_command_calls[1].command_data.arguments)
+        assert.equal(nil, send_command_calls[1].command_data.model)
+        assert.equal(nil, send_command_calls[1].command_data.agent)
+
+        send_command_calls = {}
+
+        test_with_model_cmd.fn()
+        assert.equal(1, #send_command_calls)
+        assert.equal('test-session', send_command_calls[1].session_id)
+        assert.equal('test-with-model', send_command_calls[1].command_data.command)
+        assert.equal('', send_command_calls[1].command_data.arguments)
+        assert.equal('openai/gpt-4', send_command_calls[1].command_data.model)
+        assert.equal('tester', send_command_calls[1].command_data.agent)
+
+        config_file.get_user_commands = original_get_user_commands
+        state.active_session = original_active_session
+        state.api_client = original_api_client
+      end)
+    end)
+
     it('uses default description when none provided', function()
       local config_file = require('opencode.config_file')
       local original_get_user_commands = config_file.get_user_commands