bug(provider) fix system prompting

- bump version
frankroeder · Jul 16, 2024 · c45bbc3 · c45bbc3
1 parent 440159c
commit c45bbc3
Show file tree

Hide file tree

Showing 8 changed files with 5 additions and 38 deletions.
diff --git a/README.md b/README.md
@@ -68,7 +68,7 @@ Let the parrot fix your bugs.
 ```lua
 {
   "frankroeder/parrot.nvim",
-  tag = "v0.3.4",
+  tag = "v0.3.5",
   dependencies = { 'ibhagwan/fzf-lua', 'nvim-lua/plenary.nvim' },
   config = function()
     require("parrot").setup {

diff --git a/lua/parrot/init.lua b/lua/parrot/init.lua
@@ -1578,7 +1578,10 @@ M.Prompt = function(params, target, prompt, model, template, system_template, ag
       M.logger.error("Mismatch of agent and current provider " .. prov.name .. " and " .. agent_provider)
       return
     end
-    messages = prov:add_system_prompt(messages, sys_prompt)
+
+    if sys_prompt ~= "" then
+      table.insert(messages, { role = "system", content = sys_prompt })
+    end
 
     local repo_instructions = futils.find_repo_instructions()
     if repo_instructions ~= "" and sys_prompt ~= "" then

diff --git a/lua/parrot/provider/anthropic.lua b/lua/parrot/provider/anthropic.lua
@@ -75,10 +75,6 @@ function Anthropic:verify()
   end
 end
 
-function Anthropic:add_system_prompt(messages, _)
-  return messages
-end
-
 function Anthropic:process(line)
   if line:match("content_block_delta") and line:match("text_delta") then
     local decoded_line = vim.json.decode(line)

diff --git a/lua/parrot/provider/gemini.lua b/lua/parrot/provider/gemini.lua
@@ -85,10 +85,6 @@ function Gemini:verify()
   end
 end
 
-function Gemini:add_system_prompt(messages, _)
-  return messages
-end
-
 function Gemini:process(line)
   local pattern = '"text":'
   if line:match(pattern) then

diff --git a/lua/parrot/provider/mistral.lua b/lua/parrot/provider/mistral.lua
@@ -66,13 +66,6 @@ function Mistral:verify()
   end
 end
 
-function Mistral:add_system_prompt(messages, sys_prompt)
-  if sys_prompt ~= "" then
-    table.insert(messages, { role = "system", content = sys_prompt })
-  end
-  return messages
-end
-
 function Mistral:process(line)
   if line:match("chat%.completion%.chunk") or line:match("chat%.completion") then
     line = vim.json.decode(line)

diff --git a/lua/parrot/provider/ollama.lua b/lua/parrot/provider/ollama.lua
@@ -58,13 +58,6 @@ function Ollama:verify()
   return true
 end
 
-function Ollama:add_system_prompt(messages, sys_prompt)
-  if sys_prompt ~= "" then
-    table.insert(messages, { role = "system", content = sys_prompt })
-  end
-  return messages
-end
-
 function Ollama:process(line)
   if line:match("message") and line:match("content") then
     line = vim.json.decode(line)

diff --git a/lua/parrot/provider/openai.lua b/lua/parrot/provider/openai.lua
@@ -85,13 +85,6 @@ function OpenAI:verify()
   end
 end
 
-function OpenAI:add_system_prompt(messages, sys_prompt)
-  if sys_prompt ~= "" then
-    table.insert(messages, { role = "system", content = sys_prompt })
-  end
-  return messages
-end
-
 function OpenAI:process(line)
   if line:match("chat%.completion%.chunk") or line:match("chat%.completion") then
     line = vim.json.decode(line)

diff --git a/lua/parrot/provider/perplexity.lua b/lua/parrot/provider/perplexity.lua
@@ -69,13 +69,6 @@ function Perplexity:verify()
   end
 end
 
-function Perplexity:add_system_prompt(messages, sys_prompt)
-  if sys_prompt ~= "" then
-    table.insert(messages, { role = "system", content = sys_prompt })
-  end
-  return messages
-end
-
 function Perplexity:process(line)
   if line:match("chat%.completion%.chunk") or line:match("chat%.completion") then
     line = vim.json.decode(line)