-
Notifications
You must be signed in to change notification settings - Fork 6
Select model and behavior #47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 5 commits
37d60e9
4f223a6
50b5c90
dc066c4
f1011e3
71448b1
87e3231
db32ed9
a21fac3
9b3d6bd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -346,4 +346,34 @@ function M.setup() | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Logger.debug("ECA commands registered") | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| end | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| vim.api.nvim_create_user_command("EcaChatSelectModel", function() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| local state = require("eca.state"):new() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| local models = state.config.models.list | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| vim.ui.select(models, { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| prompt = "Select ECA Chat Model:", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }, function(choice) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| if choice then | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| state:update_selected_model(choice) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| end | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| end) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| end, { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| desc = "Select Current ECA Chat model", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| vim.api.nvim_create_user_command("EcaChatSelectBehavior", function() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| local state = require("eca.state"):new() | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| local behaviors = state.config.behaviors.list | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| vim.ui.select(behaviors, { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| prompt = "Select ECA Chat Behavior:", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }, function(choice) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| if choice then | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| state:update_selected_behavior(choice) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| end | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| end) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| end, { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| desc = "Select Current ECA Chat behavior", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| }) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Logger.debug("ECA commands registered") | |
| end | |
| vim.api.nvim_create_user_command("EcaChatSelectModel", function() | |
| local state = require("eca.state"):new() | |
| local models = state.config.models.list | |
| vim.ui.select(models, { | |
| prompt = "Select ECA Chat Model:", | |
| }, function(choice) | |
| if choice then | |
| state:update_selected_model(choice) | |
| end | |
| end) | |
| end, { | |
| desc = "Select Current ECA Chat model", | |
| }) | |
| vim.api.nvim_create_user_command("EcaChatSelectBehavior", function() | |
| local state = require("eca.state"):new() | |
| local behaviors = state.config.behaviors.list | |
| vim.ui.select(behaviors, { | |
| prompt = "Select ECA Chat Behavior:", | |
| }, function(choice) | |
| if choice then | |
| state:update_selected_behavior(choice) | |
| end | |
| end) | |
| end, { | |
| desc = "Select Current ECA Chat behavior", | |
| }) | |
| vim.api.nvim_create_user_command("EcaChatSelectModel", function() | |
| local state = require("eca.state"):new() | |
| local models = state.config.models.list | |
| vim.ui.select(models, { | |
| prompt = "Select ECA Chat Model:", | |
| }, function(choice) | |
| if choice then | |
| state:update_selected_model(choice) | |
| end | |
| end) | |
| end, { | |
| desc = "Select Current ECA Chat model", | |
| }) | |
| vim.api.nvim_create_user_command("EcaChatSelectBehavior", function() | |
| local state = require("eca.state"):new() | |
| local behaviors = state.config.behaviors.list | |
| vim.ui.select(behaviors, { | |
| prompt = "Select ECA Chat Behavior:", | |
| }, function(choice) | |
| if choice then | |
| state:update_selected_behavior(choice) | |
| end | |
| end) | |
| end, { | |
| desc = "Select Current ECA Chat behavior", | |
| }) | |
| Logger.debug("ECA commands registered") | |
| end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,168 @@ | ||
| local MiniTest = require("mini.test") | ||
| local eq = MiniTest.expect.equality | ||
| local child = MiniTest.new_child_neovim() | ||
|
|
||
| local T = MiniTest.new_set({ | ||
| hooks = { | ||
| pre_case = function() | ||
| child.restart({ "-u", "scripts/minimal_init.lua" }) | ||
| child.lua([[ | ||
| -- Setup commands | ||
| require('eca.commands').setup() | ||
|
|
||
| -- Instantiate state singleton | ||
| _G.State = require('eca.state').new() | ||
|
|
||
| -- Mock vim.ui.select for testing | ||
| _G.selected_choice = nil | ||
| _G.shown_items = nil | ||
| _G.shown_prompt = nil | ||
| _G.original_select = vim.ui.select | ||
|
|
||
| _G.mock_select = function(choice) | ||
| _G.selected_choice = choice | ||
| vim.ui.select = function(items, opts, on_choice) | ||
| _G.shown_items = items | ||
| _G.shown_prompt = opts.prompt | ||
| on_choice(choice) | ||
| end | ||
| end | ||
|
|
||
| _G.restore_select = function() | ||
| vim.ui.select = _G.original_select | ||
| end | ||
| ]]) | ||
| end, | ||
| post_case = function() | ||
| child.lua([[_G.restore_select()]]) | ||
| end, | ||
| post_once = child.stop, | ||
| }, | ||
| }) | ||
|
|
||
| -- Test EcaChatSelectModel command | ||
| T["EcaChatSelectModel"] = MiniTest.new_set() | ||
|
|
||
| T["EcaChatSelectModel"]["command is registered"] = function() | ||
| local commands = child.lua_get("vim.api.nvim_get_commands({})") | ||
| eq(type(commands.EcaChatSelectModel), "table") | ||
| eq(commands.EcaChatSelectModel.name, "EcaChatSelectModel") | ||
| end | ||
|
|
||
| T["EcaChatSelectModel"]["updates state when model selected"] = function() | ||
| -- Setup initial state with models | ||
| child.lua([[ | ||
| _G.State.config.models.list = { "model1", "model2", "model3" } | ||
| _G.State.config.models.selected = "model1" | ||
|
|
||
| -- Mock vim.ui.select to auto-select model2 | ||
| _G.mock_select("model2") | ||
| ]]) | ||
|
|
||
| -- Execute command | ||
| child.cmd("EcaChatSelectModel") | ||
|
|
||
| -- Check that state was updated | ||
| eq(child.lua_get("_G.State.config.models.selected"), "model2") | ||
| end | ||
|
|
||
| T["EcaChatSelectModel"]["handles nil selection"] = function() | ||
| -- Setup initial state | ||
| child.lua([[ | ||
| _G.State.config.models.list = { "model1", "model2" } | ||
| _G.State.config.models.selected = "model1" | ||
|
|
||
| -- Mock vim.ui.select to return nil (user cancelled) | ||
| _G.mock_select(nil) | ||
| ]]) | ||
|
|
||
| -- Execute command | ||
| child.cmd("EcaChatSelectModel") | ||
|
|
||
| -- Check that state was NOT updated (still model1) | ||
| eq(child.lua_get("_G.State.config.models.selected"), "model1") | ||
| end | ||
|
|
||
| T["EcaChatSelectModel"]["displays all available models"] = function() | ||
| -- Setup models list | ||
| child.lua([[ | ||
| _G.State.config.models.list = { "gpt-4", "gpt-3.5-turbo", "claude-3" } | ||
|
|
||
| -- Mock vim.ui.select to capture the items shown | ||
| _G.mock_select(nil) | ||
| ]]) | ||
|
|
||
| -- Execute command | ||
| child.cmd("EcaChatSelectModel") | ||
|
|
||
| -- Verify all models were shown | ||
| local shown_items = child.lua_get("_G.shown_items") | ||
| eq(shown_items[1], "gpt-4") | ||
| eq(shown_items[2], "gpt-3.5-turbo") | ||
| eq(shown_items[3], "claude-3") | ||
| end | ||
|
|
||
| -- Test EcaChatSelectBehavior command | ||
| T["EcaChatSelectBehavior"] = MiniTest.new_set() | ||
|
|
||
| T["EcaChatSelectBehavior"]["command is registered"] = function() | ||
| local commands = child.lua_get("vim.api.nvim_get_commands({})") | ||
| eq(type(commands.EcaChatSelectBehavior), "table") | ||
| eq(commands.EcaChatSelectBehavior.name, "EcaChatSelectBehavior") | ||
| end | ||
|
|
||
| T["EcaChatSelectBehavior"]["updates state when behavior selected"] = function() | ||
| -- Setup initial state with behaviors | ||
| child.lua([[ | ||
| _G.State.config.behaviors.list = { "helpful", "creative", "concise" } | ||
| _G.State.config.behaviors.selected = "helpful" | ||
|
|
||
| -- Mock vim.ui.select to auto-select creative | ||
| _G.mock_select("creative") | ||
| ]]) | ||
|
|
||
| -- Execute command | ||
| child.cmd("EcaChatSelectBehavior") | ||
|
|
||
| -- Check that state was updated | ||
| eq(child.lua_get("_G.State.config.behaviors.selected"), "creative") | ||
| end | ||
|
|
||
| T["EcaChatSelectBehavior"]["handles nil selection"] = function() | ||
| -- Setup initial state | ||
| child.lua([[ | ||
| _G.State.config.behaviors.list = { "helpful", "creative" } | ||
| _G.State.config.behaviors.selected = "helpful" | ||
|
|
||
| -- Mock vim.ui.select to return nil (user cancelled) | ||
| _G.mock_select(nil) | ||
| ]]) | ||
|
|
||
| -- Execute command | ||
| child.cmd("EcaChatSelectBehavior") | ||
|
|
||
| -- Check that state was NOT updated (still helpful) | ||
| eq(child.lua_get("_G.State.config.behaviors.selected"), "helpful") | ||
| end | ||
|
|
||
| T["EcaChatSelectBehavior"]["displays all available behaviors"] = function() | ||
| -- Setup behaviors list | ||
| child.lua([[ | ||
| _G.State.config.behaviors.list = { "helpful", "creative", "concise", "technical" } | ||
|
|
||
| -- Mock vim.ui.select to capture the items shown | ||
| _G.mock_select(nil) | ||
| ]]) | ||
|
|
||
| -- Execute command | ||
| child.cmd("EcaChatSelectBehavior") | ||
|
|
||
| -- Verify all behaviors were shown | ||
| local shown_items = child.lua_get("_G.shown_items") | ||
| eq(shown_items[1], "helpful") | ||
| eq(shown_items[2], "creative") | ||
| eq(shown_items[3], "concise") | ||
| eq(shown_items[4], "technical") | ||
| end | ||
|
|
||
| return T |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent capitalization in command description. Should be 'Select current ECA Chat model' to match the pattern used in the behavior command description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense