Refactor AI-powered features and prompt management system

eranif · eranif · commit 8424b1738e7d · 2026-03-04T00:19:22.000+02:00
Remove the AI-powered toolbar dropdown and its associated release notes generation functionality from GitConsole. The `GenerateReleaseNotes`, `DoCodeReview`, and `FinaliseReleaseNotes` methods, along with the `OnGitAIDropDown` handler and related UI elements, have been removed to simplify the git console interface.

Restructure the prompt configuration system in Config to support dynamic prompt management. The `SetPrompt` method now delegates to a new `AddPrompt` helper, while three new public methods enable flexible prompt handling: `GetAllPrompts` retrieves all configured prompts as label-prompt pairs, `AddPrompt` replaces or adds prompts by label, and `DeletePrompt` removes prompts by label. All operations are thread-safe via mutex locking.

Remove the `kReleaseNotesMerge` prompt kind enum value and its corresponding default prompt configuration, simplifying the prompt system. Update the prompt templates in Prompts.cpp for consistency and clarity, and integrate the new `GetAllPrompts` method into ChatAIWindow to provide users with quick access to insert stored prompts via a menu.

* Plugin/ai/Config.cpp - Refactor SetPrompt, add AddPrompt, DeletePrompt, GetAllPrompts methods
* Plugin/ai/Config.hpp - Remove kReleaseNotesMerge enum, add new public prompt methods
* Plugin/ai/Prompts.cpp - Remove PROMPT_GIT_RELEASE_NOTES_MERGE, update prompt formatting
* Plugin/ai/ChatAIWindow.cpp - Integrate GetAllPrompts for prompt insertion menu
* git/GitConsole.h - Remove AI feature declarations and method signatures
* git/GitConsole.cpp - Remove AI toolbar button, dropdown handler, and release notes generation logic

**Generated by CodeLite**

Signed-off-by: Eran Ifrah &lt;eran@codelite.org&gt;
diff --git a/Plugin/ai/ChatAIWindow.cpp b/Plugin/ai/ChatAIWindow.cpp
@@ -226,6 +226,23 @@ void ChatAIWindow::OnOptions(wxAuiToolBarEvent& event)
         },
         XRCID("wxID_CACHING_POLICY_STATIC_CONTENT"));
     menu.Append(wxID_ANY, _("Prompt Caching"), caching_policy_menu);
+    auto prompts = llm_manager.GetConfig().GetAllPrompts();
+    if (!prompts.empty()) {
+        wxMenu* prompt_selection = new wxMenu;
+        for (const auto& [label, prompt] : prompts) {
+            auto item_id = wxXmlResource::GetXRCID(label);
+            prompt_selection->Append(item_id, label);
+            prompt_selection->Bind(
+                wxEVT_MENU,
+                [prompt, this](wxCommandEvent& event) {
+                    wxUnusedVar(event);
+                    m_stcInput->SetText(prompt);
+                    m_stcInput->CallAfter(&wxStyledTextCtrl::SetFocus);
+                },
+                item_id);
+        }
+        menu.Append(wxID_ANY, _("Insert Prompt"), prompt_selection);
+    }
     menu.AppendSeparator();
     menu.Append(XRCID("chat_history"), _("Chat History"))->Enable(CurrentEndpointHasHistory());
     menu.Bind(wxEVT_MENU, &ChatAIWindow::OnHistory, this, XRCID("chat_history"));
diff --git a/Plugin/ai/Config.cpp b/Plugin/ai/Config.cpp
@@ -25,7 +25,6 @@ T ReadValue(const nlohmann::json& j, const std::string& name, const T& d = {})
 const std::map<std::string, std::string> kDefaultPromptTable = {
     {PromptKindToString(PromptKind::kCommentGeneration), PROMPT_DOCSTRING_GEN},
     {PromptKindToString(PromptKind::kReleaseNotesGenerate), PROMPT_GIT_RELEASE_NOTES},
-    {PromptKindToString(PromptKind::kReleaseNotesMerge), PROMPT_GIT_RELEASE_NOTES_MERGE},
     {PromptKindToString(PromptKind::kGitChangesCodeReview), PROMPT_GIT_CODE_REVIEW},
     {PromptKindToString(PromptKind::kGitCommitMessage), PROMPT_GIT_COMMIT_MSG}};
 
@@ -127,17 +126,40 @@ wxString Config::GetPrompt(PromptKind kind) const
     return wxEmptyString;
 }
 
-void Config::SetPrompt(PromptKind kind, const wxString& prompt)
+void Config::DeletePrompt(const wxString& label)
+{
+    std::scoped_lock lk{m_mutex};
+    m_prompts.erase(label.ToStdString(wxConvUTF8));
+}
+
+void Config::AddPrompt(const wxString& label, const wxString& prompt)
 {
-    auto label = PromptKindToString(kind);
     std::scoped_lock lk{m_mutex};
+    std::string utf8 = label.ToStdString(wxConvUTF8);
+    m_prompts.erase(utf8);
+    m_prompts.insert({utf8, prompt.ToStdString(wxConvUTF8)});
+}
+
+std::vector<std::pair<wxString, wxString>> Config::GetAllPrompts() const
+{
+    std::scoped_lock lk{m_mutex};
+    std::vector<std::pair<wxString, wxString>> result;
+    result.reserve(m_prompts.size());
+    for (const auto& [label, prompt] : m_prompts) {
+        result.push_back({wxString::FromUTF8(label), wxString::FromUTF8(prompt)});
+    }
+    return result;
+}
+
+void Config::SetPrompt(PromptKind kind, const wxString& prompt)
+{
     switch (kind) {
     case PromptKind::kMax:
         break;
-    default:
-        m_prompts.erase(label);
-        m_prompts.insert({label, prompt.ToStdString(wxConvUTF8)});
-        break;
+    default: {
+        auto label = PromptKindToString(kind);
+        AddPrompt(wxString::FromUTF8(label), prompt);
+    } break;
     }
 }
 
diff --git a/Plugin/ai/Config.hpp b/Plugin/ai/Config.hpp
@@ -22,7 +22,6 @@ enum class PromptKind {
     kCommentGeneration,
     kGitCommitMessage,
     kReleaseNotesGenerate,
-    kReleaseNotesMerge,
     kGitChangesCodeReview,
     kMax, // Must be last
 };
@@ -61,8 +60,6 @@ inline CachePolicy CachePolicyFromString(const wxString& policy)
 inline std::string PromptKindToString(PromptKind kind)
 {
     switch (kind) {
-    case llm::PromptKind::kReleaseNotesMerge:
-        return "Git Release Notes: Merge";
     case llm::PromptKind::kReleaseNotesGenerate:
         return "Git Release Notes: Generate";
     case llm::PromptKind::kCommentGeneration:
@@ -411,6 +408,56 @@ class WXDLLIMPEXP_SDK Config
      */
     void SetPrompt(PromptKind kind, const wxString& prompt);
 
+    /**
+     * Retrieves all configured prompts as a vector of label-prompt pairs.
+     *
+     * This method provides thread-safe access to all prompts stored in the configuration.
+     * Each prompt is converted from UTF-8 encoding to wxString format. The method acquires
+     * a mutex lock to ensure consistent read access and prevents concurrent modifications.
+     *
+     * Parameters:
+     *   (none)
+     *
+     * Returns:
+     *   A vector of pairs where each pair contains a prompt label (first) and its
+     *   corresponding prompt text (second), both as wxString objects. Returns an empty
+     *   vector if no prompts are configured.
+     */
+    std::vector<std::pair<wxString, wxString>> GetAllPrompts() const;
+
+    /**
+     * Adds or replaces a prompt entry in the configuration.
+     *
+     * This method thread-safely adds a new prompt to the internal prompts map,
+     * using the provided label as the key. If an entry with the same label already
+     * exists, it is first removed and then replaced with the new prompt value.
+     * Both the label and prompt are converted from wxString to UTF-8 encoded
+     * standard strings before storage.
+     *
+     * Parameters:
+     *   label - A wxString containing the unique identifier for the prompt entry.
+     *   prompt - A wxString containing the prompt text or content to be stored.
+     *
+     * Returns:
+     *   Nothing (void).
+     */
+    void AddPrompt(const wxString& label, const wxString& prompt);
+    /**
+     * Deletes a prompt configuration entry by its label.
+     *
+     * This function removes the prompt associated with the given label from the
+     * internal prompts collection. Thread-safe operation is ensured via an internal
+     * mutex lock that is held for the duration of the erase operation.
+     *
+     * Parameters:
+     *   label - A wxString containing the label of the prompt to delete. The label
+     *           is converted to a UTF-8 encoded std::string for lookup and removal.
+     *
+     * Returns:
+     *   void - No value is returned.
+     */
+    void DeletePrompt(const wxString& label);
+
     /**
      * @brief Resets the prompt collection to its default state.
      *
diff --git a/Plugin/ai/Prompts.cpp b/Plugin/ai/Prompts.cpp
@@ -15,7 +15,7 @@ The documentation comment must contain **all** of the following sections (in thi
 4. **Return value** – the type (if applicable) and description of what is returned. If the function returns nothing, explicitly state that.
 5. **Raises / Throws / Errors** – any exceptions, error codes, or error‑handling behavior the function may produce. If none, you may omit this section.
 
-Follow the exact syntax for the target language’s doc‑comment style (e.g., `/** … */` for Java, `""" … """` for Python, `/// …` for Rust, `/** … */` for TypeScript/JSDoc, `/*** … */` for Doxygen, etc.).
+Follow the exact syntax for the target language's doc‑comment style (e.g., `/** … */` for Java, `""" … """` for Python, `/// …` for Rust, `/** … */` for TypeScript/JSDoc, `/*** … */` for Doxygen, etc.).
 
 Do **not** include any extra explanatory text outside the comment block.
 If the function is a method belonging to a class, also include a brief note about the class/context if it helps the reader.
@@ -42,17 +42,17 @@ const std::string PROMPT_GIT_COMMIT_MSG =
 
 ## Requirements
 * The first line (the **title**) must be ≤ 80 characters.
-* The title must be written in **title case**, start with a **imperative verb** (e.g. “Fix”, “Add”, “Refactor”, “Update”), and must not end with a period.
+* The title must be written in **title case**, start with a **imperative verb** (e.g. "Fix", "Add", "Refactor", "Update"), and must not end with a period.
 * After the title leave a **blank line**.
 * The body may contain one or more paragraphs that:
   * Summarize **what** was changed (high‑level overview, not a line‑by‑line description).
-* If the change touches multiple logical areas, add a short **“* <area>”** bullet list at the end of the body.
+* If the change touches multiple logical areas, add a short **"* <area>"** bullet list at the end of the body.
 * Do **not** repeat the diff verbatim in the commit message; use it only as source material for understanding the change.
-* Append a footer with the following text at the end of the generated commit message: "** Generated by CodeLite. **"
+* Append a footer with the following text at the end of the generated commit message: "**Generated by CodeLite**"
 
 ## Input
-Below is the `git diff` of the change you need to describe. Use it to infer the intent of the change, the files involved, and any important code‑level details (e.g. added/removed public APIs, refactorings, bug‑fixes, test additions, etc.).
-
+Below is the `git diff` of the change you need to describe. Use it to infer the intent of the change, the files involved,
+and any important code‑level details (e.g. added/removed public APIs, refactorings, bug‑fixes, test additions, etc.).
 
 ```diff
 {{context}}
@@ -66,14 +66,15 @@ const std::string PROMPT_GIT_RELEASE_NOTES =
     R"(You are an expert technical writer who creates concise, well‑structured release notes for public websites.
 
 Your job:
+
 1. Obtain all git commits from HEAD to the latest tag. Use `git describe --tags --abbrev=0` to find the latest tag, then `git log <tag>..HEAD --oneline` to list the commits.
 2. Group the commits into the following sections **only if there is at least one item**:
    - **✨ New Features** – new public‑facing functionality.
    - **🐛 Bug Fixes** – corrections of defects.
    - **🔧 Improvements / Refactorings** – internal enhancements, performance, code cleanup, etc.
    - **📝 Documentation** – docs, READMEs, comments, examples, API docs.
    - **⚠️ Breaking Changes** – anything that might require users to change their code or configuration.
-   - **🚀 Other** – anything that doesn’t fit above (e.g., CI/CD updates, test additions, build scripts).
+   - **🚀 Other** – anything that doesn't fit above (e.g., CI/CD updates, test additions, build scripts).
 
 3. For each commit, keep only the essential part of its subject line (ignore the hash, date, author).
    - If the commit message already contains a conventional prefix (e.g., `feat:`, `fix:`, `docs:`), **use that to decide the section**.
@@ -91,19 +92,7 @@ Your job:
 # Release {{VERSION}} – {{RELEASE_DATE}}
 ```
 
-6. End the notes with a short “_Thank you for using our software!_” line.)";
-
-const std::string PROMPT_GIT_RELEASE_NOTES_MERGE =
-    R"(Rephrase the following release notes text. Remove duplicate entries and merge the sections.
-
-Add a summary section that contains list of contributors and the number of commit they did.
-
-Here are the release notes:
-
-```markdown
-{{context}}
-```
-)";
+6. End the notes with a short "_Thank you for using our software!_" line.)";
 
 const std::string PROMPT_GIT_CODE_REVIEW =
     R"(You are an expert developer performing a code review. Your task is to analyze the provided git diff and offer constructive feedback.
@@ -118,11 +107,7 @@ Your review should cover the following points:
 Provide a concise summary at the beginning of your review. For each suggestion, use a GCC style format.
 If you provide a code example for an improvement, include the filename where the change should be applied.
 
-Here is the git diff to review:
-
-```diff
-{{context}}
-```
+To obtain the git diff, use the command: `git --no-pager diff HEAD`
 
 Here are some examples:
 
diff --git a/git/GitConsole.cpp b/git/GitConsole.cpp
diff --git a/git/GitConsole.h b/git/GitConsole.h
