From 03d5de1482fae8a2a20a51b3732d68376c1762f8 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Wed, 16 Aug 2023 16:07:04 +0200
Subject: [PATCH 01/24] Add keyboard shortcuts for most editor actions

---
 editor/limbo_ai_editor_plugin.cpp | 123 +++++++++++++++++++++---------
 editor/limbo_ai_editor_plugin.h   |   8 +-
 2 files changed, 95 insertions(+), 36 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index aa67d4c..3e01565 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -764,40 +764,66 @@ void LimboAIEditor::_mark_as_dirty(bool p_dirty) {
 	}
 }
 
-void LimboAIEditor::_on_tree_rmb(const Vector2 &p_menu_pos) {
+void LimboAIEditor::shortcut_input(const Ref<InputEvent> &p_event) {
+	if (!p_event->is_pressed() || !(has_focus() || is_ancestor_of(get_viewport()->gui_get_focus_owner()))) {
+		return;
+	}
+
+	if (ED_IS_SHORTCUT("limbo_ai/rename_task", p_event)) {
+		_action_selected(ACTION_RENAME);
+	} else if (ED_IS_SHORTCUT("limbo_ai/move_task_up", p_event)) {
+		_action_selected(ACTION_MOVE_UP);
+	} else if (ED_IS_SHORTCUT("limbo_ai/move_task_down", p_event)) {
+		_action_selected(ACTION_MOVE_DOWN);
+	} else if (ED_IS_SHORTCUT("limbo_ai/duplicate_task", p_event)) {
+		_action_selected(ACTION_DUPLICATE);
+	} else if (ED_IS_SHORTCUT("limbo_ai/remove_task", p_event)) {
+		_action_selected(ACTION_REMOVE);
+	} else if (ED_IS_SHORTCUT("limbo_ai/new_behavior_tree", p_event)) {
+		_new_bt();
+	} else if (ED_IS_SHORTCUT("limbo_ai/save_behavior_tree", p_event)) {
+		_on_save_pressed();
+	} else if (ED_IS_SHORTCUT("limbo_ai/load_behavior_tree", p_event)) {
+		load_dialog->popup_file_dialog();
+	} else {
+		return;
+	}
+
+	accept_event();
+}
+
+void LimboAIEditor::_create_popup_menu() {
 	menu->clear();
-	menu->add_icon_item(get_theme_icon(SNAME("Remove"), SNAME("EditorIcons")), TTR("Remove"), ACTION_REMOVE);
+	menu->add_icon_shortcut(get_theme_icon(SNAME("Rename"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/rename_task"), ACTION_RENAME);
 	menu->add_separator();
-	menu->add_icon_item(get_theme_icon(SNAME("MoveUp"), SNAME("EditorIcons")), TTR("Move Up"), ACTION_MOVE_UP);
-	menu->add_icon_item(get_theme_icon(SNAME("MoveDown"), SNAME("EditorIcons")), TTR("Move Down"), ACTION_MOVE_DOWN);
-	menu->add_icon_item(get_theme_icon(SNAME("Duplicate"), SNAME("EditorIcons")), TTR("Duplicate"), ACTION_DUPLICATE);
+	menu->add_icon_shortcut(get_theme_icon(SNAME("MoveUp"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/move_task_up"), ACTION_MOVE_UP);
+	menu->add_icon_shortcut(get_theme_icon(SNAME("MoveDown"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/move_task_down"), ACTION_MOVE_DOWN);
+	menu->add_icon_shortcut(get_theme_icon(SNAME("Duplicate"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/duplicate_task"), ACTION_DUPLICATE);
 	menu->add_icon_item(get_theme_icon(SNAME("NewRoot"), SNAME("EditorIcons")), TTR("Make Root"), ACTION_MAKE_ROOT);
+	menu->add_separator();
+	menu->add_icon_shortcut(get_theme_icon(SNAME("Remove"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/remove_task"), ACTION_REMOVE);
 
 	menu->reset_size();
+}
+
+void LimboAIEditor::_on_tree_rmb(const Vector2 &p_menu_pos) {
+	_create_popup_menu();
 	menu->set_position(p_menu_pos);
 	menu->popup();
 }
 
-void LimboAIEditor::_on_action_selected(int p_id) {
+void LimboAIEditor::_action_selected(int p_id) {
 	EditorUndoRedoManager *undo_redo = EditorUndoRedoManager::get_singleton();
 	switch (p_id) {
-		case ACTION_REMOVE: {
-			Ref<BTTask> sel = task_tree->get_selected();
-			if (sel.is_valid()) {
-				undo_redo->create_action(TTR("Remove BT Task"));
-				if (sel->get_parent().is_null()) {
-					undo_redo->add_do_method(task_tree->get_bt().ptr(), SNAME("set_root_task"), Variant());
-					undo_redo->add_undo_method(task_tree->get_bt().ptr(), SNAME("set_root_task"), task_tree->get_bt()->get_root_task());
-				} else {
-					undo_redo->add_do_method(sel->get_parent().ptr(), SNAME("remove_child"), sel);
-					undo_redo->add_undo_method(sel->get_parent().ptr(), SNAME("add_child_at_index"), sel, sel->get_parent()->get_child_index(sel));
-				}
-				undo_redo->add_do_method(task_tree, SNAME("update_tree"));
-				undo_redo->add_undo_method(task_tree, SNAME("update_tree"));
-				undo_redo->commit_action();
-				EditorNode::get_singleton()->edit_resource(task_tree->get_selected());
-				_mark_as_dirty(true);
+		case ACTION_RENAME: {
+			if (!task_tree->get_selected().is_valid()) {
+				return;
 			}
+
+			rename_dialog->popup_centered();
+			rename_edit->set_text(task_tree->get_selected()->get_custom_name());
+			rename_edit->select_all();
+			rename_edit->grab_focus();
 		} break;
 		case ACTION_MOVE_UP: {
 			Ref<BTTask> sel = task_tree->get_selected();
@@ -871,6 +897,24 @@ void LimboAIEditor::_on_action_selected(int p_id) {
 				_mark_as_dirty(true);
 			}
 		} break;
+		case ACTION_REMOVE: {
+			Ref<BTTask> sel = task_tree->get_selected();
+			if (sel.is_valid()) {
+				undo_redo->create_action(TTR("Remove BT Task"));
+				if (sel->get_parent().is_null()) {
+					undo_redo->add_do_method(task_tree->get_bt().ptr(), SNAME("set_root_task"), Variant());
+					undo_redo->add_undo_method(task_tree->get_bt().ptr(), SNAME("set_root_task"), task_tree->get_bt()->get_root_task());
+				} else {
+					undo_redo->add_do_method(sel->get_parent().ptr(), SNAME("remove_child"), sel);
+					undo_redo->add_undo_method(sel->get_parent().ptr(), SNAME("add_child_at_index"), sel, sel->get_parent()->get_child_index(sel));
+				}
+				undo_redo->add_do_method(task_tree, SNAME("update_tree"));
+				undo_redo->add_undo_method(task_tree, SNAME("update_tree"));
+				undo_redo->commit_action();
+				EditorNode::get_singleton()->edit_resource(task_tree->get_selected());
+				_mark_as_dirty(true);
+			}
+		} break;
 	}
 }
 
@@ -879,14 +923,7 @@ void LimboAIEditor::_on_tree_task_selected(const Ref<BTTask> &p_task) {
 }
 
 void LimboAIEditor::_on_tree_task_double_clicked() {
-	if (!task_tree->get_selected().is_valid()) {
-		return;
-	}
-
-	rename_dialog->popup_centered();
-	rename_edit->set_text(task_tree->get_selected()->get_custom_name());
-	rename_edit->select_all();
-	rename_edit->grab_focus();
+	_action_selected(ACTION_RENAME);
 }
 
 void LimboAIEditor::_on_panel_task_selected(String p_task) {
@@ -1100,6 +1137,7 @@ void LimboAIEditor::_notification(int p_what) {
 			history_back->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Back"), SNAME("EditorIcons")));
 			history_forward->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Forward"), SNAME("EditorIcons")));
 
+			_create_popup_menu();
 			_update_header();
 		}
 	}
@@ -1120,6 +1158,19 @@ void LimboAIEditor::_bind_methods() {
 LimboAIEditor::LimboAIEditor() {
 	idx_history = 0;
 
+	ED_SHORTCUT("limbo_ai/rename_task", TTR("Rename"), Key::F2);
+	ED_SHORTCUT_OVERRIDE("limbo_ai/rename_task", "macos", Key::ENTER);
+	ED_SHORTCUT("limbo_ai/move_task_up", TTR("Move Up"), KeyModifierMask::CMD_OR_CTRL | Key::UP);
+	ED_SHORTCUT("limbo_ai/move_task_down", TTR("Move Down"), KeyModifierMask::CMD_OR_CTRL | Key::DOWN);
+	ED_SHORTCUT("limbo_ai/duplicate_task", TTR("Duplicate"), KeyModifierMask::CMD_OR_CTRL | Key::D);
+	ED_SHORTCUT("limbo_ai/remove_task", TTR("Remove"), Key::KEY_DELETE);
+
+	ED_SHORTCUT("limbo_ai/new_behavior_tree", TTR("New Behavior Tree"), KeyModifierMask::CMD_OR_CTRL | KeyModifierMask::ALT | Key::N);
+	ED_SHORTCUT("limbo_ai/save_behavior_tree", TTR("Save Behavior Tree"), KeyModifierMask::CMD_OR_CTRL | KeyModifierMask::ALT | Key::S);
+	ED_SHORTCUT("limbo_ai/load_behavior_tree", TTR("Load Behavior Tree"), KeyModifierMask::CMD_OR_CTRL | KeyModifierMask::ALT | Key::L);
+
+	set_process_shortcut_input(true);
+
 	save_dialog = memnew(FileDialog);
 	save_dialog->set_file_mode(FileDialog::FILE_MODE_SAVE_FILE);
 	save_dialog->set_title("Save Behavior Tree");
@@ -1172,7 +1223,8 @@ LimboAIEditor::LimboAIEditor() {
 
 	new_btn = memnew(Button);
 	new_btn->set_text(TTR("New"));
-	new_btn->set_tooltip_text(TTR("Create new behavior tree."));
+	new_btn->set_tooltip_text(TTR("Create a new behavior tree."));
+	new_btn->set_shortcut(ED_GET_SHORTCUT("limbo_ai/new_behavior_tree"));
 	new_btn->set_flat(true);
 	new_btn->set_focus_mode(Control::FOCUS_NONE);
 	new_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_new_bt));
@@ -1180,7 +1232,8 @@ LimboAIEditor::LimboAIEditor() {
 
 	load_btn = memnew(Button);
 	load_btn->set_text(TTR("Load"));
-	load_btn->set_tooltip_text(TTR("Load behavior tree."));
+	load_btn->set_tooltip_text(TTR("Load behavior tree from a resource file."));
+	load_btn->set_shortcut(ED_GET_SHORTCUT("limbo_ai/load_behavior_tree"));
 	load_btn->set_flat(true);
 	load_btn->set_focus_mode(Control::FOCUS_NONE);
 	load_btn->connect("pressed", callable_mp(load_dialog, &FileDialog::popup_file_dialog));
@@ -1188,7 +1241,8 @@ LimboAIEditor::LimboAIEditor() {
 
 	save_btn = memnew(Button);
 	save_btn->set_text(TTR("Save"));
-	save_btn->set_tooltip_text(TTR("Save current behavior tree."));
+	save_btn->set_tooltip_text(TTR("Save edited behavior tree to a resource file."));
+	save_btn->set_shortcut(ED_GET_SHORTCUT("limbo_ai/save_behavior_tree"));
 	save_btn->set_flat(true);
 	save_btn->set_focus_mode(Control::FOCUS_NONE);
 	save_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_on_save_pressed));
@@ -1228,6 +1282,7 @@ LimboAIEditor::LimboAIEditor() {
 	hsc = memnew(HSplitContainer);
 	hsc->set_h_size_flags(SIZE_EXPAND_FILL);
 	hsc->set_v_size_flags(SIZE_EXPAND_FILL);
+	hsc->set_focus_mode(FOCUS_NONE);
 	vb->add_child(hsc);
 
 	task_tree = memnew(TaskTree);
@@ -1262,7 +1317,7 @@ LimboAIEditor::LimboAIEditor() {
 
 	menu = memnew(PopupMenu);
 	add_child(menu);
-	menu->connect("id_pressed", callable_mp(this, &LimboAIEditor::_on_action_selected));
+	menu->connect("id_pressed", callable_mp(this, &LimboAIEditor::_action_selected));
 
 	rename_dialog = memnew(ConfirmationDialog);
 	{
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index ed8d327..3b5c27b 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -133,11 +133,12 @@ class LimboAIEditor : public Control {
 
 private:
 	enum Action {
-		ACTION_REMOVE,
+		ACTION_RENAME,
 		ACTION_MOVE_UP,
 		ACTION_MOVE_DOWN,
 		ACTION_DUPLICATE,
 		ACTION_MAKE_ROOT,
+		ACTION_REMOVE,
 	};
 
 	Vector<Ref<BehaviorTree>> history;
@@ -179,6 +180,7 @@ private:
 	void _save_bt(String p_path);
 	void _load_bt(String p_path);
 	void _mark_as_dirty(bool p_dirty);
+	void _create_popup_menu();
 
 	void _reload_modified();
 	void _resave_modified(String _str = "");
@@ -186,7 +188,7 @@ private:
 	void _rename_task_confirmed();
 
 	void _on_tree_rmb(const Vector2 &p_menu_pos);
-	void _on_action_selected(int p_id);
+	void _action_selected(int p_id);
 	void _on_tree_task_selected(const Ref<BTTask> &p_task);
 	void _on_tree_task_double_clicked();
 	void _on_visibility_changed();
@@ -198,6 +200,8 @@ private:
 	void _on_task_dragged(Ref<BTTask> p_task, Ref<BTTask> p_to_task, int p_type);
 	void _on_resources_reload(const Vector<String> &p_resources);
 
+	virtual void shortcut_input(const Ref<InputEvent> &p_event) override;
+
 protected:
 	static void _bind_methods();
 

From a2b123a9be19fba0aefbfb7378475700055b154f Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Thu, 17 Aug 2023 11:41:40 +0200
Subject: [PATCH 02/24] Editor: Add "Edit Script" popup menu option

---
 editor/limbo_ai_editor_plugin.cpp | 19 +++++++++++++------
 editor/limbo_ai_editor_plugin.h   |  2 +-
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 3e01565..bac8586 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -792,22 +792,26 @@ void LimboAIEditor::shortcut_input(const Ref<InputEvent> &p_event) {
 	accept_event();
 }
 
-void LimboAIEditor::_create_popup_menu() {
+void LimboAIEditor::_on_tree_rmb(const Vector2 &p_menu_pos) {
 	menu->clear();
+
+	Ref<BTTask> task = task_tree->get_selected();
+	ERR_FAIL_COND_MSG(task.is_null(), "LimboAIEditor: get_selected() returned null");
+
 	menu->add_icon_shortcut(get_theme_icon(SNAME("Rename"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/rename_task"), ACTION_RENAME);
+	menu->add_icon_item(get_theme_icon(SNAME("Script"), SNAME("EditorIcons")), TTR("Edit Script"), ACTION_EDIT_SCRIPT);
+	menu->set_item_disabled(ACTION_EDIT_SCRIPT, task->get_script().is_null());
+
 	menu->add_separator();
 	menu->add_icon_shortcut(get_theme_icon(SNAME("MoveUp"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/move_task_up"), ACTION_MOVE_UP);
 	menu->add_icon_shortcut(get_theme_icon(SNAME("MoveDown"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/move_task_down"), ACTION_MOVE_DOWN);
 	menu->add_icon_shortcut(get_theme_icon(SNAME("Duplicate"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/duplicate_task"), ACTION_DUPLICATE);
 	menu->add_icon_item(get_theme_icon(SNAME("NewRoot"), SNAME("EditorIcons")), TTR("Make Root"), ACTION_MAKE_ROOT);
+
 	menu->add_separator();
 	menu->add_icon_shortcut(get_theme_icon(SNAME("Remove"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/remove_task"), ACTION_REMOVE);
 
 	menu->reset_size();
-}
-
-void LimboAIEditor::_on_tree_rmb(const Vector2 &p_menu_pos) {
-	_create_popup_menu();
 	menu->set_position(p_menu_pos);
 	menu->popup();
 }
@@ -825,6 +829,10 @@ void LimboAIEditor::_action_selected(int p_id) {
 			rename_edit->select_all();
 			rename_edit->grab_focus();
 		} break;
+		case ACTION_EDIT_SCRIPT: {
+			ERR_FAIL_COND(task_tree->get_selected().is_null());
+			EditorNode::get_singleton()->edit_resource(task_tree->get_selected()->get_script());
+		} break;
 		case ACTION_MOVE_UP: {
 			Ref<BTTask> sel = task_tree->get_selected();
 			if (sel.is_valid() && sel->get_parent().is_valid()) {
@@ -1137,7 +1145,6 @@ void LimboAIEditor::_notification(int p_what) {
 			history_back->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Back"), SNAME("EditorIcons")));
 			history_forward->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Forward"), SNAME("EditorIcons")));
 
-			_create_popup_menu();
 			_update_header();
 		}
 	}
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 3b5c27b..db2c741 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -134,6 +134,7 @@ class LimboAIEditor : public Control {
 private:
 	enum Action {
 		ACTION_RENAME,
+		ACTION_EDIT_SCRIPT,
 		ACTION_MOVE_UP,
 		ACTION_MOVE_DOWN,
 		ACTION_DUPLICATE,
@@ -180,7 +181,6 @@ private:
 	void _save_bt(String p_path);
 	void _load_bt(String p_path);
 	void _mark_as_dirty(bool p_dirty);
-	void _create_popup_menu();
 
 	void _reload_modified();
 	void _resave_modified(String _str = "");

From 985c4812f3089eb5801db1dca257a556e8662d3c Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Thu, 17 Aug 2023 15:29:12 +0200
Subject: [PATCH 03/24] Editor: Add "Open Documentation" popup menu option

---
 editor/limbo_ai_editor_plugin.cpp | 15 +++++++++++++++
 editor/limbo_ai_editor_plugin.h   |  1 +
 2 files changed, 16 insertions(+)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index bac8586..1d3471a 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -800,6 +800,7 @@ void LimboAIEditor::_on_tree_rmb(const Vector2 &p_menu_pos) {
 
 	menu->add_icon_shortcut(get_theme_icon(SNAME("Rename"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/rename_task"), ACTION_RENAME);
 	menu->add_icon_item(get_theme_icon(SNAME("Script"), SNAME("EditorIcons")), TTR("Edit Script"), ACTION_EDIT_SCRIPT);
+	menu->add_icon_item(get_theme_icon(SNAME("Help"), SNAME("EditorIcons")), TTR("Open Documentation"), ACTION_OPEN_DOC);
 	menu->set_item_disabled(ACTION_EDIT_SCRIPT, task->get_script().is_null());
 
 	menu->add_separator();
@@ -833,6 +834,20 @@ void LimboAIEditor::_action_selected(int p_id) {
 			ERR_FAIL_COND(task_tree->get_selected().is_null());
 			EditorNode::get_singleton()->edit_resource(task_tree->get_selected()->get_script());
 		} break;
+		case ACTION_OPEN_DOC: {
+			Ref<BTTask> task = task_tree->get_selected();
+			ERR_FAIL_COND(task.is_null());
+			String help_class;
+			if (!task->get_script().is_null()) {
+				Ref<Script> s = task->get_script();
+				help_class = s->get_language()->get_global_class_name(s->get_path());
+			}
+			if (help_class.is_empty()) {
+				help_class = task->get_class();
+			}
+			ScriptEditor::get_singleton()->goto_help("class_name:" + help_class);
+			EditorNode::get_singleton()->set_visible_editor(EditorNode::EDITOR_SCRIPT);
+		} break;
 		case ACTION_MOVE_UP: {
 			Ref<BTTask> sel = task_tree->get_selected();
 			if (sel.is_valid() && sel->get_parent().is_valid()) {
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index db2c741..7bd5be0 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -135,6 +135,7 @@ private:
 	enum Action {
 		ACTION_RENAME,
 		ACTION_EDIT_SCRIPT,
+		ACTION_OPEN_DOC,
 		ACTION_MOVE_UP,
 		ACTION_MOVE_DOWN,
 		ACTION_DUPLICATE,

From b3dedc737e232a011e7fc12ea9aef6fefadce5b9 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Thu, 17 Aug 2023 18:14:24 +0200
Subject: [PATCH 04/24] Add a popup menu for tasks in the TaskPanel

Adds such menu items (similar to task popup menu):
- Edit Script
- Open Documentation
---
 editor/limbo_ai_editor_plugin.cpp | 64 +++++++++++++++++++++++++++++--
 editor/limbo_ai_editor_plugin.h   | 15 +++++++-
 2 files changed, 73 insertions(+), 6 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 1d3471a..1089e75 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -343,10 +343,21 @@ TaskTree::~TaskTree() {
 
 //**** TaskSection
 
-void TaskSection::_on_task_button_pressed(const StringName &p_task) {
+void TaskSection::_on_task_button_pressed(const String &p_task) {
 	emit_signal(SNAME("task_button_pressed"), p_task);
 }
 
+void TaskSection::_on_task_button_gui_input(const Ref<InputEvent> &p_event, const String &p_task) {
+	if (!p_event->is_pressed()) {
+		return;
+	}
+
+	Ref<InputEventMouseButton> mb = p_event;
+	if (mb.is_valid() && mb->get_button_index() == MouseButton::RIGHT) {
+		emit_signal(SNAME("task_button_rmb"), p_task);
+	}
+}
+
 void TaskSection::_on_header_pressed() {
 	set_collapsed(!is_collapsed());
 }
@@ -373,7 +384,8 @@ void TaskSection::add_task_button(String p_name, const Ref<Texture> &icon, Varia
 	btn->set_text(p_name);
 	btn->set_icon(icon);
 	btn->add_theme_constant_override(SNAME("icon_max_width"), 16 * EDSCALE); // Force user icons to  be of the proper size.
-	btn->connect("pressed", callable_mp(this, &TaskSection::_on_task_button_pressed).bind(p_meta));
+	btn->connect(SNAME("pressed"), callable_mp(this, &TaskSection::_on_task_button_pressed).bind(p_meta));
+	btn->connect(SNAME("gui_input"), callable_mp(this, &TaskSection::_on_task_button_gui_input).bind(p_meta));
 	tasks_container->add_child(btn);
 }
 
@@ -395,6 +407,7 @@ void TaskSection::_notification(int p_what) {
 
 void TaskSection::_bind_methods() {
 	ADD_SIGNAL(MethodInfo("task_button_pressed"));
+	ADD_SIGNAL(MethodInfo("task_button_rmb"));
 }
 
 TaskSection::TaskSection(String p_category_name) {
@@ -415,10 +428,48 @@ TaskSection::~TaskSection() {
 
 //**** TaskPanel
 
-void TaskPanel::_on_task_button_pressed(const StringName &p_task) {
+void TaskPanel::_menu_action_selected(int p_id) {
+	ERR_FAIL_COND(context_task.is_empty());
+	switch (p_id) {
+		case MENU_OPEN_DOC: {
+			String help_class;
+			if (context_task.begins_with("res://")) {
+				Ref<Script> s = ResourceLoader::load(context_task, "Script");
+				help_class = s->get_language()->get_global_class_name(context_task);
+			}
+			if (help_class.is_empty()) {
+				// Assuming context task is core class.
+				help_class = context_task;
+			}
+			ScriptEditor::get_singleton()->goto_help("class_name:" + help_class);
+			EditorNode::get_singleton()->set_visible_editor(EditorNode::EDITOR_SCRIPT);
+		} break;
+		case MENU_EDIT_SCRIPT: {
+			ERR_FAIL_COND(!context_task.begins_with("res://"));
+			ScriptEditor::get_singleton()->open_file(context_task);
+		} break;
+	}
+}
+
+void TaskPanel::_on_task_button_pressed(const String &p_task) {
 	emit_signal(SNAME("task_selected"), p_task);
 }
 
+void TaskPanel::_on_task_button_rmb(const String &p_task) {
+	ERR_FAIL_COND(p_task.is_empty());
+
+	context_task = p_task;
+	menu->clear();
+
+	menu->add_icon_item(get_theme_icon(SNAME("Script"), SNAME("EditorIcons")), TTR("Edit Script"), MENU_EDIT_SCRIPT);
+	menu->set_item_disabled(MENU_EDIT_SCRIPT, !context_task.begins_with("res://"));
+	menu->add_icon_item(get_theme_icon(SNAME("Help"), SNAME("EditorIcons")), TTR("Open Documentation"), MENU_OPEN_DOC);
+
+	menu->reset_size();
+	menu->set_position(get_screen_position() + get_local_mouse_position());
+	menu->popup();
+}
+
 void TaskPanel::_on_filter_text_changed(String p_text) {
 	for (int i = 0; i < sections->get_child_count(); i++) {
 		TaskSection *sec = Object::cast_to<TaskSection>(sections->get_child(i));
@@ -506,7 +557,8 @@ void TaskPanel::refresh() {
 			sec->add_task_button(tname, icon, task_meta);
 		}
 		sec->set_filter("");
-		sec->connect("task_button_pressed", callable_mp(this, &TaskPanel::_on_task_button_pressed));
+		sec->connect(SNAME("task_button_pressed"), callable_mp(this, &TaskPanel::_on_task_button_pressed));
+		sec->connect(SNAME("task_button_rmb"), callable_mp(this, &TaskPanel::_on_task_button_rmb));
 		sections->add_child(sec);
 		sec->set_collapsed(collapsed_sections.has(cat));
 	}
@@ -628,6 +680,10 @@ TaskPanel::TaskPanel() {
 	sections->set_h_size_flags(SIZE_EXPAND_FILL);
 	sections->set_v_size_flags(SIZE_EXPAND_FILL);
 	sc->add_child(sections);
+
+	menu = memnew(PopupMenu);
+	add_child(menu);
+	menu->connect("id_pressed", callable_mp(this, &TaskPanel::_menu_action_selected));
 }
 
 TaskPanel::~TaskPanel() {
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 7bd5be0..80d8945 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -82,7 +82,8 @@ private:
 	FlowContainer *tasks_container;
 	Button *section_header;
 
-	void _on_task_button_pressed(const StringName &p_task);
+	void _on_task_button_pressed(const String &p_task);
+	void _on_task_button_gui_input(const Ref<InputEvent> &p_event, const String &p_task);
 	void _on_header_pressed();
 
 protected:
@@ -107,13 +108,23 @@ class TaskPanel : public PanelContainer {
 	GDCLASS(TaskPanel, PanelContainer)
 
 private:
+	enum MenuAction {
+		MENU_EDIT_SCRIPT,
+		MENU_OPEN_DOC,
+	};
+
 	LineEdit *filter_edit;
 	VBoxContainer *sections;
+	PopupMenu *menu;
+
+	String context_task;
 
 	void _populate_core_tasks_from_class(const StringName &p_base_class, List<String> *p_task_classes);
 	void _populate_from_user_dir(String p_path, HashMap<String, List<String>> *p_categories);
 	void _populate_scripted_tasks_from_dir(String p_path, List<String> *p_task_classes);
-	void _on_task_button_pressed(const StringName &p_task);
+	void _menu_action_selected(int p_id);
+	void _on_task_button_pressed(const String &p_task);
+	void _on_task_button_rmb(const String &p_task);
 	void _on_filter_text_changed(String p_text);
 
 protected:

From 5c3445236f9550d153e832f3273a31ec3fc24594 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Fri, 18 Aug 2023 12:12:55 +0200
Subject: [PATCH 05/24] Implement: Add/Remove task favorites

Favorite tasks are tasks selected to be shown on the toolbar.
Order can be edited in the project settings.
---
 editor/limbo_ai_editor_plugin.cpp | 156 ++++++++++++++++++------------
 editor/limbo_ai_editor_plugin.h   |   8 +-
 2 files changed, 97 insertions(+), 67 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 1089e75..e7f5807 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -448,6 +448,17 @@ void TaskPanel::_menu_action_selected(int p_id) {
 			ERR_FAIL_COND(!context_task.begins_with("res://"));
 			ScriptEditor::get_singleton()->open_file(context_task);
 		} break;
+		case MENU_FAVORITE: {
+			Array favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
+			if (favorite_tasks.has(context_task)) {
+				favorite_tasks.erase(context_task);
+			} else {
+				favorite_tasks.append(context_task);
+			}
+			ProjectSettings::get_singleton()->set_setting("limbo_ai/behavior_tree/favorite_tasks", favorite_tasks);
+			ProjectSettings::get_singleton()->save();
+			emit_signal(SNAME("favorite_tasks_changed"));
+		} break;
 	}
 }
 
@@ -465,6 +476,14 @@ void TaskPanel::_on_task_button_rmb(const String &p_task) {
 	menu->set_item_disabled(MENU_EDIT_SCRIPT, !context_task.begins_with("res://"));
 	menu->add_icon_item(get_theme_icon(SNAME("Help"), SNAME("EditorIcons")), TTR("Open Documentation"), MENU_OPEN_DOC);
 
+	menu->add_separator();
+	Array favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
+	if (favorite_tasks.has(context_task)) {
+		menu->add_icon_item(get_theme_icon(SNAME("NonFavorite"), SNAME("EditorIcons")), TTR("Remove from Favorites"), MENU_FAVORITE);
+	} else {
+		menu->add_icon_item(get_theme_icon(SNAME("Favorites"), SNAME("EditorIcons")), TTR("Add to Favorites"), MENU_FAVORITE);
+	}
+
 	menu->reset_size();
 	menu->set_position(get_screen_position() + get_local_mouse_position());
 	menu->popup();
@@ -659,6 +678,7 @@ void TaskPanel::_bind_methods() {
 	ClassDB::bind_method(D_METHOD("refresh"), &TaskPanel::refresh);
 
 	ADD_SIGNAL(MethodInfo("task_selected"));
+	ADD_SIGNAL(MethodInfo("favorite_tasks_changed"));
 }
 
 TaskPanel::TaskPanel() {
@@ -715,6 +735,33 @@ void LimboAIEditor::_add_task(const Ref<BTTask> &p_task) {
 	_mark_as_dirty(true);
 }
 
+void LimboAIEditor::_add_task_by_class_or_path(String p_class_or_path) {
+	Ref<BTTask> task;
+
+	if (p_class_or_path.begins_with("res:")) {
+		Ref<Script> s = ResourceLoader::load(p_class_or_path, "Script");
+		ERR_FAIL_COND_MSG(s.is_null() || !s->is_valid(), vformat("LimboAI: Failed to instantiate task. Bad script: %s", p_class_or_path));
+		Variant inst = ClassDB::instantiate(s->get_instance_base_type());
+		ERR_FAIL_COND_MSG(inst.is_zero(), vformat("LimboAI: Failed to instantiate base type \"%s\".", s->get_instance_base_type()));
+
+		if (unlikely(!((Object *)inst)->is_class("BTTask"))) {
+			if (!inst.is_ref_counted()) {
+				memdelete((Object *)inst);
+			}
+			ERR_PRINT(vformat("LimboAI: Failed to instantiate task. Script is not a BTTask: %s", p_class_or_path));
+			return;
+		}
+
+		if (inst && s.is_valid()) {
+			((Object *)inst)->set_script(s);
+			task = inst;
+		}
+	} else {
+		task = ClassDB::instantiate(p_class_or_path);
+	}
+	_add_task(task);
+}
+
 void LimboAIEditor::_remove_task(const Ref<BTTask> &p_task) {
 	ERR_FAIL_COND(p_task.is_null());
 	ERR_FAIL_COND(task_tree->get_bt().is_null());
@@ -1005,33 +1052,6 @@ void LimboAIEditor::_on_tree_task_double_clicked() {
 	_action_selected(ACTION_RENAME);
 }
 
-void LimboAIEditor::_on_panel_task_selected(String p_task) {
-	Ref<BTTask> task;
-
-	if (p_task.begins_with("res:")) {
-		Ref<Script> s = ResourceLoader::load(p_task, "Script");
-		ERR_FAIL_COND_MSG(s.is_null() || !s->is_valid(), vformat("LimboAI: Failed to instance task. Bad script: %s", p_task));
-		Variant inst = ClassDB::instantiate(s->get_instance_base_type());
-		ERR_FAIL_COND_MSG(inst.is_zero(), vformat("LimboAI: Failed to instance base type \"%s\".", s->get_instance_base_type()));
-
-		if (unlikely(!((Object *)inst)->is_class("BTTask"))) {
-			if (!inst.is_ref_counted()) {
-				memdelete((Object *)inst);
-			}
-			ERR_PRINT(vformat("LimboAI: Failed to instance task. Script is not a BTTask: %s", p_task));
-			return;
-		}
-
-		if (inst && s.is_valid()) {
-			((Object *)inst)->set_script(s);
-			task = inst;
-		}
-	} else {
-		task = ClassDB::instantiate(p_task);
-	}
-	_add_task(task);
-}
-
 void LimboAIEditor::_on_visibility_changed() {
 	if (task_tree->is_visible()) {
 		Ref<BTTask> sel = task_tree->get_selected();
@@ -1043,6 +1063,7 @@ void LimboAIEditor::_on_visibility_changed() {
 
 		task_panel->refresh();
 	}
+	_update_favorite_tasks();
 }
 
 void LimboAIEditor::_on_header_pressed() {
@@ -1189,6 +1210,32 @@ void LimboAIEditor::apply_changes() {
 	}
 }
 
+void LimboAIEditor::_update_favorite_tasks() {
+	for (int i = 0; i < fav_tasks_hbox->get_child_count(); i++) {
+		fav_tasks_hbox->get_child(i)->queue_free();
+	}
+	Array favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
+	for (int i = 0; i < favorite_tasks.size(); i++) {
+		String task_meta = favorite_tasks[i];
+		Button *btn = memnew(Button);
+		String task_name;
+		if (task_meta.begins_with("res:")) {
+			task_name = task_meta.get_file().get_basename().trim_prefix("BT").to_pascal_case();
+		} else {
+			task_name = task_meta.trim_prefix("BT");
+		}
+		btn->set_text(task_name);
+		btn->set_meta(SNAME("task_meta"), task_meta);
+		btn->set_icon(LimboUtility::get_singleton()->get_task_icon(task_meta));
+		btn->set_tooltip_text(vformat(TTR("Add %s task."), task_name));
+		btn->set_flat(true);
+		btn->add_theme_constant_override(SNAME("icon_max_width"), 16 * EDSCALE); // Force user icons to be of the proper size.
+		btn->set_focus_mode(Control::FOCUS_NONE);
+		btn->connect(SNAME("pressed"), callable_mp(this, &LimboAIEditor::_add_task_by_class_or_path).bind(task_meta));
+		fav_tasks_hbox->add_child(btn);
+	}
+}
+
 void LimboAIEditor::_notification(int p_what) {
 	switch (p_what) {
 		case NOTIFICATION_ENTER_TREE: {
@@ -1206,16 +1253,13 @@ void LimboAIEditor::_notification(int p_what) {
 			conf.save(conf_path);
 		} break;
 		case NOTIFICATION_THEME_CHANGED: {
-			selector_btn->set_icon(EditorNode::get_singleton()->get_class_icon("BTSelector"));
-			sequence_btn->set_icon(EditorNode::get_singleton()->get_class_icon("BTSequence"));
-			parallel_btn->set_icon(EditorNode::get_singleton()->get_class_icon("BTParallel"));
 			new_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("New"), SNAME("EditorIcons")));
 			load_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Load"), SNAME("EditorIcons")));
 			save_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Save"), SNAME("EditorIcons")));
 			new_script_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("ScriptCreate"), SNAME("EditorIcons")));
 			history_back->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Back"), SNAME("EditorIcons")));
 			history_forward->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Forward"), SNAME("EditorIcons")));
-
+			_update_favorite_tasks();
 			_update_header();
 		}
 	}
@@ -1270,34 +1314,19 @@ LimboAIEditor::LimboAIEditor() {
 	vb->set_anchor(SIDE_BOTTOM, ANCHOR_END);
 	add_child(vb);
 
-	HBoxContainer *panel = memnew(HBoxContainer);
-	vb->add_child(panel);
+	HBoxContainer *toolbar = memnew(HBoxContainer);
+	vb->add_child(toolbar);
 
-	selector_btn = memnew(Button);
-	selector_btn->set_text(TTR("Selector"));
-	selector_btn->set_tooltip_text(TTR("Add Selector task."));
-	selector_btn->set_flat(true);
-	selector_btn->set_focus_mode(Control::FOCUS_NONE);
-	selector_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_add_task_with_prototype).bind(Ref<BTTask>(memnew(BTSelector))));
-	panel->add_child(selector_btn);
+	Array favorite_tasks_default;
+	favorite_tasks_default.append("BTSelector");
+	favorite_tasks_default.append("BTSequence");
+	favorite_tasks_default.append("BTParallel");
+	GLOBAL_DEF(PropertyInfo(Variant::ARRAY, "limbo_ai/behavior_tree/favorite_tasks", PROPERTY_HINT_ARRAY_TYPE, "String"), favorite_tasks_default);
 
-	sequence_btn = memnew(Button);
-	sequence_btn->set_text(TTR("Sequence"));
-	sequence_btn->set_tooltip_text(TTR("Add Sequence task."));
-	sequence_btn->set_flat(true);
-	sequence_btn->set_focus_mode(Control::FOCUS_NONE);
-	sequence_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_add_task_with_prototype).bind(Ref<BTTask>(memnew(BTSequence))));
-	panel->add_child(sequence_btn);
+	fav_tasks_hbox = memnew(HBoxContainer);
+	toolbar->add_child(fav_tasks_hbox);
 
-	parallel_btn = memnew(Button);
-	parallel_btn->set_text(TTR("Parallel"));
-	parallel_btn->set_tooltip_text(TTR("Add Parallel task."));
-	parallel_btn->set_flat(true);
-	parallel_btn->set_focus_mode(Control::FOCUS_NONE);
-	parallel_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_add_task_with_prototype).bind(Ref<BTTask>(memnew(BTParallel))));
-	panel->add_child(parallel_btn);
-
-	panel->add_child(memnew(VSeparator));
+	toolbar->add_child(memnew(VSeparator));
 
 	new_btn = memnew(Button);
 	new_btn->set_text(TTR("New"));
@@ -1306,7 +1335,7 @@ LimboAIEditor::LimboAIEditor() {
 	new_btn->set_flat(true);
 	new_btn->set_focus_mode(Control::FOCUS_NONE);
 	new_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_new_bt));
-	panel->add_child(new_btn);
+	toolbar->add_child(new_btn);
 
 	load_btn = memnew(Button);
 	load_btn->set_text(TTR("Load"));
@@ -1315,7 +1344,7 @@ LimboAIEditor::LimboAIEditor() {
 	load_btn->set_flat(true);
 	load_btn->set_focus_mode(Control::FOCUS_NONE);
 	load_btn->connect("pressed", callable_mp(load_dialog, &FileDialog::popup_file_dialog));
-	panel->add_child(load_btn);
+	toolbar->add_child(load_btn);
 
 	save_btn = memnew(Button);
 	save_btn->set_text(TTR("Save"));
@@ -1324,20 +1353,20 @@ LimboAIEditor::LimboAIEditor() {
 	save_btn->set_flat(true);
 	save_btn->set_focus_mode(Control::FOCUS_NONE);
 	save_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_on_save_pressed));
-	panel->add_child(save_btn);
+	toolbar->add_child(save_btn);
 
-	panel->add_child(memnew(VSeparator));
+	toolbar->add_child(memnew(VSeparator));
 
 	new_script_btn = memnew(Button);
 	new_script_btn->set_text(TTR("New Task"));
 	new_script_btn->set_tooltip_text(TTR("Create new task script and edit it."));
 	new_script_btn->set_flat(true);
 	new_script_btn->set_focus_mode(Control::FOCUS_NONE);
-	panel->add_child(new_script_btn);
+	toolbar->add_child(new_script_btn);
 
 	HBoxContainer *nav = memnew(HBoxContainer);
 	nav->set_h_size_flags(SIZE_EXPAND | SIZE_SHRINK_END);
-	panel->add_child(nav);
+	toolbar->add_child(nav);
 
 	history_back = memnew(Button);
 	history_back->set_flat(true);
@@ -1389,7 +1418,8 @@ LimboAIEditor::LimboAIEditor() {
 
 	task_panel = memnew(TaskPanel());
 	hsc->set_split_offset(-300);
-	task_panel->connect("task_selected", callable_mp(this, &LimboAIEditor::_on_panel_task_selected));
+	task_panel->connect("task_selected", callable_mp(this, &LimboAIEditor::_add_task_by_class_or_path));
+	task_panel->connect("favorite_tasks_changed", callable_mp(this, &LimboAIEditor::_update_favorite_tasks));
 	task_panel->hide();
 	hsc->add_child(task_panel);
 
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 80d8945..0076a8b 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -111,6 +111,7 @@ private:
 	enum MenuAction {
 		MENU_EDIT_SCRIPT,
 		MENU_OPEN_DOC,
+		MENU_FAVORITE,
 	};
 
 	LineEdit *filter_edit;
@@ -168,10 +169,8 @@ private:
 	Button *history_back;
 	Button *history_forward;
 	TaskPanel *task_panel;
+	HBoxContainer *fav_tasks_hbox;
 
-	Button *selector_btn;
-	Button *sequence_btn;
-	Button *parallel_btn;
 	Button *new_btn;
 	Button *load_btn;
 	Button *save_btn;
@@ -185,10 +184,12 @@ private:
 	HashSet<String> disk_changed_files;
 
 	void _add_task(const Ref<BTTask> &p_task);
+	void _add_task_by_class_or_path(String p_class_or_path);
 	void _remove_task(const Ref<BTTask> &p_task);
 	_FORCE_INLINE_ void _add_task_with_prototype(const Ref<BTTask> &p_prototype) { _add_task(p_prototype->clone()); }
 	void _update_header() const;
 	void _update_history_buttons();
+	void _update_favorite_tasks();
 	void _new_bt();
 	void _save_bt(String p_path);
 	void _load_bt(String p_path);
@@ -206,7 +207,6 @@ private:
 	void _on_visibility_changed();
 	void _on_header_pressed();
 	void _on_save_pressed();
-	void _on_panel_task_selected(String p_task);
 	void _on_history_back();
 	void _on_history_forward();
 	void _on_task_dragged(Ref<BTTask> p_task, Ref<BTTask> p_to_task, int p_type);

From 107b94fad69f04ef4951ab2ef070af1fb961b146 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Fri, 18 Aug 2023 18:53:55 +0200
Subject: [PATCH 06/24] Emphasize tasks with custom name set

---
 editor/limbo_ai_editor_plugin.cpp | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index e7f5807..935672a 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -94,6 +94,13 @@ void TaskTree::_update_item(TreeItem *p_item) {
 	Ref<BTTask> task = p_item->get_metadata(0);
 	ERR_FAIL_COND_MSG(!task.is_valid(), "Invalid task reference in metadata.");
 	p_item->set_text(0, task->get_task_name());
+	if (task->get_custom_name().is_empty()) {
+		p_item->set_custom_font(0, nullptr);
+		// p_item->clear_custom_color(0);
+	} else {
+		p_item->set_custom_font(0, (get_theme_font(SNAME("bold"), SNAME("EditorFonts"))));
+		// p_item->set_custom_color(0, get_theme_color(SNAME("warning_color"), SNAME("Editor")));
+	}
 	String type_arg;
 	if (task->get_script_instance() && !task->get_script_instance()->get_script()->get_path().is_empty()) {
 		type_arg = task->get_script_instance()->get_script()->get_path();

From a717a3ed8f2fb2c3bd058ff050909780212fe7ff Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Fri, 18 Aug 2023 21:38:28 +0200
Subject: [PATCH 07/24] Add BTComment task

---
 bt/tasks/bt_comment.cpp           | 19 +++++++++++++++++++
 bt/tasks/bt_comment.h             | 26 ++++++++++++++++++++++++++
 bt/tasks/bt_task.cpp              | 13 +++++++++++--
 config.py                         |  1 +
 editor/limbo_ai_editor_plugin.cpp | 16 ++++++++++++++--
 editor/limbo_ai_editor_plugin.h   |  1 +
 register_types.cpp                |  3 +++
 7 files changed, 75 insertions(+), 4 deletions(-)
 create mode 100644 bt/tasks/bt_comment.cpp
 create mode 100644 bt/tasks/bt_comment.h

diff --git a/bt/tasks/bt_comment.cpp b/bt/tasks/bt_comment.cpp
new file mode 100644
index 0000000..aab4f4d
--- /dev/null
+++ b/bt/tasks/bt_comment.cpp
@@ -0,0 +1,19 @@
+/**
+ * bt_comment.cpp
+ * =============================================================================
+ * Copyright 2021-2023 Serhii Snitsaruk
+ *
+ * Use of this source code is governed by an MIT-style
+ * license that can be found in the LICENSE file or at
+ * https://opensource.org/licenses/MIT.
+ * =============================================================================
+ */
+
+#include "bt_comment.h"
+
+Ref<BTTask> BTComment::clone() const {
+	if (Engine::get_singleton()->is_editor_hint()) {
+		return BTTask::clone();
+	}
+	return nullptr;
+}
diff --git a/bt/tasks/bt_comment.h b/bt/tasks/bt_comment.h
new file mode 100644
index 0000000..7299415
--- /dev/null
+++ b/bt/tasks/bt_comment.h
@@ -0,0 +1,26 @@
+/**
+ * bt_comment.h
+ * =============================================================================
+ * Copyright 2021-2023 Serhii Snitsaruk
+ *
+ * Use of this source code is governed by an MIT-style
+ * license that can be found in the LICENSE file or at
+ * https://opensource.org/licenses/MIT.
+ * =============================================================================
+ */
+/* bt_comment.h */
+
+#ifndef BT_COMMENT_H
+#define BT_COMMENT_H
+
+#include "bt_task.h"
+
+class BTComment : public BTTask {
+	GDCLASS(BTComment, BTTask);
+
+private:
+public:
+	virtual Ref<BTTask> clone() const override;
+};
+
+#endif // BT_COMMENT
\ No newline at end of file
diff --git a/bt/tasks/bt_task.cpp b/bt/tasks/bt_task.cpp
index f3cc2fe..ec24ce5 100644
--- a/bt/tasks/bt_task.cpp
+++ b/bt/tasks/bt_task.cpp
@@ -104,10 +104,19 @@ Ref<BTTask> BTTask::clone() const {
 	inst->data.parent = nullptr;
 	inst->data.agent = nullptr;
 	inst->data.blackboard.unref();
+	int num_null = 0;
 	for (int i = 0; i < data.children.size(); i++) {
 		Ref<BTTask> c = get_child(i)->clone();
-		c->data.parent = inst.ptr();
-		inst->data.children.set(i, c);
+		if (c.is_valid()) {
+			c->data.parent = inst.ptr();
+			inst->data.children.set(i - num_null, c);
+		} else {
+			num_null += 1;
+		}
+	}
+	if (num_null > 0) {
+		// * BTComment tasks return nullptr at runtime - we remove those.
+		inst->data.children.resize(data.children.size() - num_null);
 	}
 
 	// Make BBParam properties unique.
diff --git a/config.py b/config.py
index ef9e5dd..3c1b166 100644
--- a/config.py
+++ b/config.py
@@ -65,6 +65,7 @@ def get_doc_classes():
         "BTCheckAgentProperty",
         "BTCheckTrigger",
         "BTCheckVar",
+        "BTComment",
         "BTComposite",
         "BTCondition",
         "BTConsolePrint",
diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 935672a..e2178d2 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -94,9 +94,12 @@ void TaskTree::_update_item(TreeItem *p_item) {
 	Ref<BTTask> task = p_item->get_metadata(0);
 	ERR_FAIL_COND_MSG(!task.is_valid(), "Invalid task reference in metadata.");
 	p_item->set_text(0, task->get_task_name());
-	if (task->get_custom_name().is_empty()) {
+	if (task->is_class("BTComment")) {
+		p_item->set_custom_font(0, (get_theme_font(SNAME("doc_italic"), SNAME("EditorFonts"))));
+		p_item->set_custom_color(0, get_theme_color(SNAME("disabled_font_color"), SNAME("Editor")));
+	} else if (task->get_custom_name().is_empty()) {
 		p_item->set_custom_font(0, nullptr);
-		// p_item->clear_custom_color(0);
+		p_item->clear_custom_color(0);
 	} else {
 		p_item->set_custom_font(0, (get_theme_font(SNAME("bold"), SNAME("EditorFonts"))));
 		// p_item->set_custom_color(0, get_theme_color(SNAME("warning_color"), SNAME("Editor")));
@@ -1333,6 +1336,15 @@ LimboAIEditor::LimboAIEditor() {
 	fav_tasks_hbox = memnew(HBoxContainer);
 	toolbar->add_child(fav_tasks_hbox);
 
+	comment_btn = memnew(Button);
+	comment_btn->set_text(TTR("Comment"));
+	comment_btn->set_icon(LimboUtility::get_singleton()->get_task_icon("BTComment"));
+	comment_btn->set_tooltip_text(TTR("Add a BTComment task."));
+	comment_btn->set_flat(true);
+	comment_btn->set_focus_mode(Control::FOCUS_NONE);
+	comment_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_add_task_by_class_or_path).bind("BTComment"));
+	toolbar->add_child(comment_btn);
+
 	toolbar->add_child(memnew(VSeparator));
 
 	new_btn = memnew(Button);
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 0076a8b..2321ebb 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -171,6 +171,7 @@ private:
 	TaskPanel *task_panel;
 	HBoxContainer *fav_tasks_hbox;
 
+	Button *comment_btn;
 	Button *new_btn;
 	Button *load_btn;
 	Button *save_btn;
diff --git a/register_types.cpp b/register_types.cpp
index 84743a2..6502698 100644
--- a/register_types.cpp
+++ b/register_types.cpp
@@ -60,6 +60,7 @@
 #include "bt/tasks/actions/bt_wait.h"
 #include "bt/tasks/actions/bt_wait_ticks.h"
 #include "bt/tasks/bt_action.h"
+#include "bt/tasks/bt_comment.h"
 #include "bt/tasks/bt_composite.h"
 #include "bt/tasks/bt_condition.h"
 #include "bt/tasks/bt_decorator.h"
@@ -120,6 +121,8 @@ void initialize_limboai_module(ModuleInitializationLevel p_level) {
 		GDREGISTER_CLASS(BTPlayer);
 		GDREGISTER_CLASS(BTState);
 
+		GDREGISTER_CLASS(BTComment);
+
 		GDREGISTER_CLASS(BTComposite);
 		GDREGISTER_CLASS(BTSequence);
 		GDREGISTER_CLASS(BTSelector);

From 17c72b7378980c15b75e8ba3d9a1ad566cbd4c07 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Fri, 18 Aug 2023 21:39:30 +0200
Subject: [PATCH 08/24] Icon for BTComment class

---
 icons/BTComment.svg | 1 +
 1 file changed, 1 insertion(+)
 create mode 100644 icons/BTComment.svg

diff --git a/icons/BTComment.svg b/icons/BTComment.svg
new file mode 100644
index 0000000..66699d4
--- /dev/null
+++ b/icons/BTComment.svg
@@ -0,0 +1 @@
+<svg enable-background="new 0 0 16 16" viewBox="0 0 16 16" xmlns="http://www.w3.org/2000/svg"><path d="m11.77 6.91-.75 2.22h2.8l-.21 1.97h-3.22l-1.26 3.91h-2.38l1.27-3.91h-2.1l-1.25 3.91h-2.33l1.22-3.91h-2.57l.2-1.97h3l.76-2.22h-2.74l.2-2h3.13l1.29-3.9h2.35l-1.27 3.9h2.16l1.26-3.9h2.33l-1.26 3.9h2.59l-.2 2zm-5.23 2.22h2.13l.75-2.22h-2.12z" fill="#808080"/></svg>
\ No newline at end of file

From 0ae2b6869a1f019f770b1a79ebe91444ba112be9 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Sat, 19 Aug 2023 12:19:58 +0200
Subject: [PATCH 09/24] Don't count BTComment tasks for configuration warnings

---
 bt/tasks/bt_action.cpp            |  2 +-
 bt/tasks/bt_composite.cpp         |  2 +-
 bt/tasks/bt_condition.cpp         |  2 +-
 bt/tasks/bt_decorator.cpp         |  2 +-
 bt/tasks/bt_task.cpp              | 12 ++++++++++++
 bt/tasks/bt_task.h                |  1 +
 editor/limbo_ai_editor_plugin.cpp |  3 ++-
 7 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/bt/tasks/bt_action.cpp b/bt/tasks/bt_action.cpp
index af5e6a2..9719a4e 100644
--- a/bt/tasks/bt_action.cpp
+++ b/bt/tasks/bt_action.cpp
@@ -13,7 +13,7 @@
 
 PackedStringArray BTAction::get_configuration_warnings() const {
 	PackedStringArray warnings = BTTask::get_configuration_warnings();
-	if (get_child_count() != 0) {
+	if (get_child_count_excluding_comments() != 0) {
 		warnings.append("Action can't have child tasks.");
 	}
 	return warnings;
diff --git a/bt/tasks/bt_composite.cpp b/bt/tasks/bt_composite.cpp
index 2fb35c9..364e483 100644
--- a/bt/tasks/bt_composite.cpp
+++ b/bt/tasks/bt_composite.cpp
@@ -13,7 +13,7 @@
 
 PackedStringArray BTComposite::get_configuration_warnings() const {
 	PackedStringArray warnings = BTTask::get_configuration_warnings();
-	if (get_child_count() < 1) {
+	if (get_child_count_excluding_comments() < 1) {
 		warnings.append("Composite should have at least one child task.");
 	}
 	return warnings;
diff --git a/bt/tasks/bt_condition.cpp b/bt/tasks/bt_condition.cpp
index 904f8ee..73bf254 100644
--- a/bt/tasks/bt_condition.cpp
+++ b/bt/tasks/bt_condition.cpp
@@ -13,7 +13,7 @@
 
 PackedStringArray BTCondition::get_configuration_warnings() const {
 	PackedStringArray warnings = BTTask::get_configuration_warnings();
-	if (get_child_count() != 0) {
+	if (get_child_count_excluding_comments() != 0) {
 		warnings.append("Condition task can't have child tasks.");
 	}
 	return warnings;
diff --git a/bt/tasks/bt_decorator.cpp b/bt/tasks/bt_decorator.cpp
index b887b6d..8ac2de1 100644
--- a/bt/tasks/bt_decorator.cpp
+++ b/bt/tasks/bt_decorator.cpp
@@ -13,7 +13,7 @@
 
 PackedStringArray BTDecorator::get_configuration_warnings() const {
 	PackedStringArray warnings = BTTask::get_configuration_warnings();
-	if (get_child_count() != 1) {
+	if (get_child_count_excluding_comments() != 1) {
 		warnings.append("Decorator should have a single child task.");
 	}
 	return warnings;
diff --git a/bt/tasks/bt_task.cpp b/bt/tasks/bt_task.cpp
index ec24ce5..31f6ecc 100644
--- a/bt/tasks/bt_task.cpp
+++ b/bt/tasks/bt_task.cpp
@@ -11,6 +11,7 @@
 
 #include "bt_task.h"
 
+#include "bt_comment.h"
 #include "modules/limboai/blackboard/blackboard.h"
 #include "modules/limboai/util/limbo_string_names.h"
 #include "modules/limboai/util/limbo_utility.h"
@@ -198,6 +199,16 @@ int BTTask::get_child_count() const {
 	return data.children.size();
 }
 
+int BTTask::get_child_count_excluding_comments() const {
+	int count = 0;
+	for (int i = 0; i < data.children.size(); i++) {
+		if (!data.children[i]->is_class_ptr(BTComment::get_class_ptr_static())) {
+			count += 1;
+		}
+	}
+	return count;
+}
+
 void BTTask::add_child(Ref<BTTask> p_child) {
 	ERR_FAIL_COND_MSG(p_child->get_parent().is_valid(), "p_child already has a parent!");
 	p_child->data.parent = this;
@@ -291,6 +302,7 @@ void BTTask::_bind_methods() {
 	ClassDB::bind_method(D_METHOD("execute", "p_delta"), &BTTask::execute);
 	ClassDB::bind_method(D_METHOD("get_child", "p_idx"), &BTTask::get_child);
 	ClassDB::bind_method(D_METHOD("get_child_count"), &BTTask::get_child_count);
+	ClassDB::bind_method(D_METHOD("get_child_count_excluding_comments"), &BTTask::get_child_count_excluding_comments);
 	ClassDB::bind_method(D_METHOD("add_child", "p_child"), &BTTask::add_child);
 	ClassDB::bind_method(D_METHOD("add_child_at_index", "p_child", "p_idx"), &BTTask::add_child_at_index);
 	ClassDB::bind_method(D_METHOD("remove_child", "p_child"), &BTTask::remove_child);
diff --git a/bt/tasks/bt_task.h b/bt/tasks/bt_task.h
index d21ae79..80e28c1 100644
--- a/bt/tasks/bt_task.h
+++ b/bt/tasks/bt_task.h
@@ -93,6 +93,7 @@ public:
 
 	Ref<BTTask> get_child(int p_idx) const;
 	int get_child_count() const;
+	int get_child_count_excluding_comments() const;
 	void add_child(Ref<BTTask> p_child);
 	void add_child_at_index(Ref<BTTask> p_child, int p_idx);
 	void remove_child(Ref<BTTask> p_child);
diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index e2178d2..aa0bad0 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -15,6 +15,7 @@
 
 #include "modules/limboai/bt/behavior_tree.h"
 #include "modules/limboai/bt/tasks/bt_action.h"
+#include "modules/limboai/bt/tasks/bt_comment.h"
 #include "modules/limboai/bt/tasks/bt_task.h"
 #include "modules/limboai/bt/tasks/composites/bt_parallel.h"
 #include "modules/limboai/bt/tasks/composites/bt_selector.h"
@@ -94,7 +95,7 @@ void TaskTree::_update_item(TreeItem *p_item) {
 	Ref<BTTask> task = p_item->get_metadata(0);
 	ERR_FAIL_COND_MSG(!task.is_valid(), "Invalid task reference in metadata.");
 	p_item->set_text(0, task->get_task_name());
-	if (task->is_class("BTComment")) {
+	if (task->is_class_ptr(BTComment::get_class_ptr_static())) {
 		p_item->set_custom_font(0, (get_theme_font(SNAME("doc_italic"), SNAME("EditorFonts"))));
 		p_item->set_custom_color(0, get_theme_color(SNAME("disabled_font_color"), SNAME("Editor")));
 	} else if (task->get_custom_name().is_empty()) {

From 3f673c68653ae3159b09db1f74167573f4aec89d Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Sat, 19 Aug 2023 12:39:51 +0200
Subject: [PATCH 10/24] Improve BTComment

Warn user about having comment as root task or having other tasks as children
---
 bt/tasks/bt_comment.cpp | 13 +++++++++++++
 bt/tasks/bt_comment.h   |  1 +
 2 files changed, 14 insertions(+)

diff --git a/bt/tasks/bt_comment.cpp b/bt/tasks/bt_comment.cpp
index aab4f4d..c57a8d3 100644
--- a/bt/tasks/bt_comment.cpp
+++ b/bt/tasks/bt_comment.cpp
@@ -11,9 +11,22 @@
 
 #include "bt_comment.h"
 
+#include "bt_task.h"
+
 Ref<BTTask> BTComment::clone() const {
 	if (Engine::get_singleton()->is_editor_hint()) {
 		return BTTask::clone();
 	}
 	return nullptr;
 }
+
+PackedStringArray BTComment::get_configuration_warnings() const {
+	PackedStringArray warnings = BTTask::get_configuration_warnings();
+	if (get_child_count_excluding_comments() > 0) {
+		warnings.append("Can only have other comment tasks as children.");
+	}
+	if (get_parent() == nullptr) {
+		warnings.append("Can't be the root task.");
+	}
+	return warnings;
+}
diff --git a/bt/tasks/bt_comment.h b/bt/tasks/bt_comment.h
index 7299415..442db08 100644
--- a/bt/tasks/bt_comment.h
+++ b/bt/tasks/bt_comment.h
@@ -21,6 +21,7 @@ class BTComment : public BTTask {
 private:
 public:
 	virtual Ref<BTTask> clone() const override;
+	virtual PackedStringArray get_configuration_warnings() const override;
 };
 
 #endif // BT_COMMENT
\ No newline at end of file

From e851011678f8f0f01bac2151510bbd8f0d019eb9 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Sat, 19 Aug 2023 13:12:24 +0200
Subject: [PATCH 11/24] Rename dialog now says "Edit comment" for comments

---
 editor/limbo_ai_editor_plugin.cpp | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index aa0bad0..de8a6cb 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -938,9 +938,18 @@ void LimboAIEditor::_action_selected(int p_id) {
 			if (!task_tree->get_selected().is_valid()) {
 				return;
 			}
-
+			Ref<BTTask> task = task_tree->get_selected();
+			if (task->is_class_ptr(BTComment::get_class_ptr_static())) {
+				rename_dialog->set_title(TTR("Edit Comment"));
+				rename_dialog->get_ok_button()->set_text(TTR("OK"));
+				rename_edit->set_placeholder(TTR("Comment"));
+			} else {
+				rename_dialog->set_title(TTR("Rename Task"));
+				rename_dialog->get_ok_button()->set_text(TTR("Rename"));
+				rename_edit->set_placeholder(TTR("Custom Name"));
+			}
+			rename_edit->set_text(task->get_custom_name());
 			rename_dialog->popup_centered();
-			rename_edit->set_text(task_tree->get_selected()->get_custom_name());
 			rename_edit->select_all();
 			rename_edit->grab_focus();
 		} break;
@@ -1449,19 +1458,15 @@ LimboAIEditor::LimboAIEditor() {
 
 	rename_dialog = memnew(ConfirmationDialog);
 	{
-		rename_dialog->set_title("Rename Task");
-
 		VBoxContainer *vbc = memnew(VBoxContainer);
 		rename_dialog->add_child(vbc);
 
 		rename_edit = memnew(LineEdit);
 		vbc->add_child(rename_edit);
-		rename_edit->set_placeholder("Custom Name");
 		rename_edit->set_h_size_flags(Control::SIZE_EXPAND_FILL);
 		rename_edit->set_custom_minimum_size(Size2(350.0, 0.0));
 
 		rename_dialog->register_text_enter(rename_edit);
-		rename_dialog->get_ok_button()->set_text(TTR("Rename"));
 		rename_dialog->connect("confirmed", callable_mp(this, &LimboAIEditor::_rename_task_confirmed));
 	}
 	add_child(rename_dialog);

From 2afdb4bc9502178f541ce513cc476f9253dcb86f Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Sun, 20 Aug 2023 11:34:13 +0200
Subject: [PATCH 12/24] Add "Open Debugger" menu option and "Misc" menu

---
 editor/debugger/limbo_debugger_plugin.cpp | 22 +++++++++++++--
 editor/debugger/limbo_debugger_plugin.h   |  8 ++++++
 editor/limbo_ai_editor_plugin.cpp         | 33 +++++++++++++++++++++++
 editor/limbo_ai_editor_plugin.h           |  6 +++++
 4 files changed, 67 insertions(+), 2 deletions(-)

diff --git a/editor/debugger/limbo_debugger_plugin.cpp b/editor/debugger/limbo_debugger_plugin.cpp
index 9e33e08..efb7fa5 100644
--- a/editor/debugger/limbo_debugger_plugin.cpp
+++ b/editor/debugger/limbo_debugger_plugin.cpp
@@ -32,9 +32,10 @@
 #include "scene/gui/label.h"
 #include "scene/gui/line_edit.h"
 #include "scene/gui/split_container.h"
+#include "scene/gui/tab_container.h"
 #include "scene/gui/texture_rect.h"
 
-/////////////////////// LimboDebuggerTab
+//**** LimboDebuggerTab
 
 void LimboDebuggerTab::start_session() {
 	bt_player_list->clear();
@@ -209,7 +210,9 @@ LimboDebuggerTab::LimboDebuggerTab(Ref<EditorDebuggerSession> p_session, WindowW
 	stop_session();
 }
 
-//////////////////////// LimboDebuggerPlugin
+//**** LimboDebuggerPlugin
+
+LimboDebuggerPlugin *LimboDebuggerPlugin::singleton = nullptr;
 
 void LimboDebuggerPlugin::_window_visibility_changed(bool p_visible) {
 }
@@ -259,8 +262,23 @@ bool LimboDebuggerPlugin::has_capture(const String &p_capture) const {
 	return p_capture == "limboai";
 }
 
+WindowWrapper *LimboDebuggerPlugin::get_session_tab() const {
+	return window_wrapper;
+}
+
+int LimboDebuggerPlugin::get_session_tab_index() const {
+	TabContainer *c = Object::cast_to<TabContainer>(window_wrapper->get_parent());
+	ERR_FAIL_COND_V(c == nullptr, -1);
+	return c->get_tab_idx_from_control(window_wrapper);
+}
+
 LimboDebuggerPlugin::LimboDebuggerPlugin() {
 	tab = nullptr;
+	singleton = this;
+}
+
+LimboDebuggerPlugin::~LimboDebuggerPlugin() {
+	singleton = nullptr;
 }
 
 #endif // TOOLS_ENABLED
diff --git a/editor/debugger/limbo_debugger_plugin.h b/editor/debugger/limbo_debugger_plugin.h
index ea5895e..fcfdfa6 100644
--- a/editor/debugger/limbo_debugger_plugin.h
+++ b/editor/debugger/limbo_debugger_plugin.h
@@ -70,17 +70,25 @@ class LimboDebuggerPlugin : public EditorDebuggerPlugin {
 	GDCLASS(LimboDebuggerPlugin, EditorDebuggerPlugin);
 
 private:
+	static LimboDebuggerPlugin *singleton;
+
 	LimboDebuggerTab *tab = nullptr;
 	WindowWrapper *window_wrapper = nullptr;
 
 	void _window_visibility_changed(bool p_visible);
 
 public:
+	static _FORCE_INLINE_ LimboDebuggerPlugin *get_singleton() { return singleton; }
+
 	void setup_session(int p_idx) override;
 	bool has_capture(const String &p_capture) const override;
 	bool capture(const String &p_message, const Array &p_data, int p_session) override;
 
+	WindowWrapper *get_session_tab() const;
+	int get_session_tab_index() const;
+
 	LimboDebuggerPlugin();
+	~LimboDebuggerPlugin();
 };
 
 #endif // LIMBO_DEBUGGER_PLUGIN
diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index de8a6cb..2f21acc 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -50,6 +50,8 @@
 #include "core/variant/callable.h"
 #include "core/variant/dictionary.h"
 #include "core/variant/variant.h"
+#include "editor/debugger/editor_debugger_node.h"
+#include "editor/debugger/script_editor_debugger.h"
 #include "editor/editor_file_system.h"
 #include "editor/editor_inspector.h"
 #include "editor/editor_node.h"
@@ -1064,6 +1066,21 @@ void LimboAIEditor::_action_selected(int p_id) {
 	}
 }
 
+void LimboAIEditor::_misc_option_selected(int p_id) {
+	switch (p_id) {
+		case MISC_OPEN_DEBUGGER: {
+			ERR_FAIL_COND(LimboDebuggerPlugin::get_singleton() == nullptr);
+			if (LimboDebuggerPlugin::get_singleton()->get_session_tab()->get_window_enabled()) {
+				LimboDebuggerPlugin::get_singleton()->get_session_tab()->set_window_enabled(true);
+			} else {
+				EditorNode::get_singleton()->make_bottom_panel_item_visible(EditorDebuggerNode::get_singleton());
+				EditorDebuggerNode::get_singleton()->get_default_debugger()->switch_to_debugger(
+						LimboDebuggerPlugin::get_singleton()->get_session_tab_index());
+			}
+		} break;
+	}
+}
+
 void LimboAIEditor::_on_tree_task_selected(const Ref<BTTask> &p_task) {
 	EditorNode::get_singleton()->edit_resource(p_task);
 }
@@ -1279,6 +1296,12 @@ void LimboAIEditor::_notification(int p_what) {
 			new_script_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("ScriptCreate"), SNAME("EditorIcons")));
 			history_back->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Back"), SNAME("EditorIcons")));
 			history_forward->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Forward"), SNAME("EditorIcons")));
+
+			misc_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Tools"), SNAME("EditorIcons")));
+			PopupMenu *misc_menu = misc_btn->get_popup();
+			misc_menu->clear();
+			misc_menu->add_icon_item(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Debug"), SNAME("EditorIcons")), TTR("Open Debugger"), MISC_OPEN_DEBUGGER);
+
 			_update_favorite_tasks();
 			_update_header();
 		}
@@ -1393,6 +1416,16 @@ LimboAIEditor::LimboAIEditor() {
 	new_script_btn->set_focus_mode(Control::FOCUS_NONE);
 	toolbar->add_child(new_script_btn);
 
+	// toolbar->add_child(memnew(VSeparator));
+
+	misc_btn = memnew(MenuButton);
+	misc_btn->set_text(TTR("Misc"));
+	misc_btn->set_flat(true);
+	misc_btn->get_popup()->connect("index_pressed", callable_mp(this, &LimboAIEditor::_misc_option_selected));
+	toolbar->add_child(misc_btn);
+
+	// toolbar->add_child(memnew(VSeparator));
+
 	HBoxContainer *nav = memnew(HBoxContainer);
 	nav->set_h_size_flags(SIZE_EXPAND | SIZE_SHRINK_END);
 	toolbar->add_child(nav);
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 2321ebb..cb843cc 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -155,6 +155,10 @@ private:
 		ACTION_REMOVE,
 	};
 
+	enum MiscMenu {
+		MISC_OPEN_DEBUGGER,
+	};
+
 	Vector<Ref<BehaviorTree>> history;
 	int idx_history;
 	HashSet<Ref<BehaviorTree>> dirty;
@@ -176,6 +180,7 @@ private:
 	Button *load_btn;
 	Button *save_btn;
 	Button *new_script_btn;
+	MenuButton *misc_btn;
 
 	ConfirmationDialog *rename_dialog;
 	LineEdit *rename_edit;
@@ -203,6 +208,7 @@ private:
 
 	void _on_tree_rmb(const Vector2 &p_menu_pos);
 	void _action_selected(int p_id);
+	void _misc_option_selected(int p_id);
 	void _on_tree_task_selected(const Ref<BTTask> &p_task);
 	void _on_tree_task_double_clicked();
 	void _on_visibility_changed();

From 79c71d06f51072d0aebf602877bc7fa29f1a8d19 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Sun, 20 Aug 2023 11:49:33 +0200
Subject: [PATCH 13/24] Add global shortcut for "Open Debugger" menu option

---
 editor/limbo_ai_editor_plugin.cpp | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 2f21acc..587474e 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -881,7 +881,20 @@ void LimboAIEditor::_mark_as_dirty(bool p_dirty) {
 }
 
 void LimboAIEditor::shortcut_input(const Ref<InputEvent> &p_event) {
-	if (!p_event->is_pressed() || !(has_focus() || is_ancestor_of(get_viewport()->gui_get_focus_owner()))) {
+	if (!p_event->is_pressed()) {
+		return;
+	}
+
+	// * Global shortcuts.
+
+	if (ED_IS_SHORTCUT("limbo_ai/open_debugger", p_event)) {
+		_misc_option_selected(MISC_OPEN_DEBUGGER);
+		accept_event();
+	}
+
+	// * Local shortcuts.
+
+	if (!(has_focus() || is_ancestor_of(get_viewport()->gui_get_focus_owner()))) {
 		return;
 	}
 
@@ -1300,7 +1313,7 @@ void LimboAIEditor::_notification(int p_what) {
 			misc_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Tools"), SNAME("EditorIcons")));
 			PopupMenu *misc_menu = misc_btn->get_popup();
 			misc_menu->clear();
-			misc_menu->add_icon_item(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Debug"), SNAME("EditorIcons")), TTR("Open Debugger"), MISC_OPEN_DEBUGGER);
+			misc_menu->add_icon_shortcut(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Debug"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/open_debugger"), MISC_OPEN_DEBUGGER);
 
 			_update_favorite_tasks();
 			_update_header();
@@ -1333,6 +1346,7 @@ LimboAIEditor::LimboAIEditor() {
 	ED_SHORTCUT("limbo_ai/new_behavior_tree", TTR("New Behavior Tree"), KeyModifierMask::CMD_OR_CTRL | KeyModifierMask::ALT | Key::N);
 	ED_SHORTCUT("limbo_ai/save_behavior_tree", TTR("Save Behavior Tree"), KeyModifierMask::CMD_OR_CTRL | KeyModifierMask::ALT | Key::S);
 	ED_SHORTCUT("limbo_ai/load_behavior_tree", TTR("Load Behavior Tree"), KeyModifierMask::CMD_OR_CTRL | KeyModifierMask::ALT | Key::L);
+	ED_SHORTCUT("limbo_ai/open_debugger", TTR("Open Debugger"), KeyModifierMask::CMD_OR_CTRL | KeyModifierMask::ALT | Key::D);
 
 	set_process_shortcut_input(true);
 

From 1cb3b64a29a0d8b5c97efc66686eaa14e1bb48db Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Mon, 21 Aug 2023 11:51:19 +0200
Subject: [PATCH 14/24] Add "Misc>Project  Settings..." menu option

---
 editor/limbo_ai_editor_plugin.cpp | 6 ++++++
 editor/limbo_ai_editor_plugin.h   | 1 +
 2 files changed, 7 insertions(+)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 587474e..840cfcb 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -62,6 +62,7 @@
 #include "editor/editor_undo_redo_manager.h"
 #include "editor/inspector_dock.h"
 #include "editor/plugins/script_editor_plugin.h"
+#include "editor/project_settings_editor.h"
 #include "scene/gui/box_container.h"
 #include "scene/gui/button.h"
 #include "scene/gui/control.h"
@@ -1091,6 +1092,10 @@ void LimboAIEditor::_misc_option_selected(int p_id) {
 						LimboDebuggerPlugin::get_singleton()->get_session_tab_index());
 			}
 		} break;
+		case MISC_PROJECT_SETTINGS: {
+			ProjectSettingsEditor::get_singleton()->set_general_page("limbo_ai/behavior_tree");
+			ProjectSettingsEditor::get_singleton()->popup_project_settings();
+		} break;
 	}
 }
 
@@ -1314,6 +1319,7 @@ void LimboAIEditor::_notification(int p_what) {
 			PopupMenu *misc_menu = misc_btn->get_popup();
 			misc_menu->clear();
 			misc_menu->add_icon_shortcut(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Debug"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/open_debugger"), MISC_OPEN_DEBUGGER);
+			misc_menu->add_item(TTR("Project Settings..."), MISC_PROJECT_SETTINGS);
 
 			_update_favorite_tasks();
 			_update_header();
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index cb843cc..8ead5e9 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -157,6 +157,7 @@ private:
 
 	enum MiscMenu {
 		MISC_OPEN_DEBUGGER,
+		MISC_PROJECT_SETTINGS,
 	};
 
 	Vector<Ref<BehaviorTree>> history;

From 4cabbc6082f0a54392b8dee73b8fed4a3b75d4e1 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Mon, 21 Aug 2023 13:46:35 +0200
Subject: [PATCH 15/24] Implement "Misc->Create Script Template"

---
 editor/limbo_ai_editor_plugin.cpp | 69 ++++++++++++++++++++++++++++---
 editor/limbo_ai_editor_plugin.h   |  2 +
 2 files changed, 66 insertions(+), 5 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 840cfcb..63a8ea1 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -1096,6 +1096,54 @@ void LimboAIEditor::_misc_option_selected(int p_id) {
 			ProjectSettingsEditor::get_singleton()->set_general_page("limbo_ai/behavior_tree");
 			ProjectSettingsEditor::get_singleton()->popup_project_settings();
 		} break;
+		case MISC_CREATE_SCRIPT_TEMPLATE: {
+			if (!FileAccess::exists("res://script_templates/BTTask/custom_task.gd")) {
+				Ref<DirAccess> dir = DirAccess::create(DirAccess::ACCESS_RESOURCES);
+				if (!dir->exists("res://script_templates/")) {
+					dir->make_dir("res://script_templates/");
+				}
+				if (!dir->exists("res://script_templates/BTTask")) {
+					dir->make_dir("res://script_templates/BTTask");
+				}
+				Error err;
+				Ref<FileAccess> f = FileAccess::open("res://script_templates/BTTask/custom_task.gd", FileAccess::WRITE, &err);
+				ERR_FAIL_COND(err != OK);
+
+				String script_template =
+						"# meta-name: Custom Task\n"
+						"# meta-description: Custom task to be used in a BehaviorTree\n"
+						"# meta-default: true\n"
+						"@tool\n"
+						"extends _BASE_\n"
+						"## _CLASS_\n"
+						"\n\n"
+						"# Display a customized name (requires @tool).\n"
+						"func _generate_name() -> String:\n"
+						"_TS_return \"_CLASS_\"\n"
+						"\n\n"
+						"# Called once during initialization.\n"
+						"func _setup() -> void:\n"
+						"_TS_pass\n"
+						"\n\n"
+						"# Called each time this task is entered.\n"
+						"func _enter() -> void:\n"
+						"_TS_pass\n"
+						"\n\n"
+						"# Called each time this task is exited.\n"
+						"func _exit() -> void:\n"
+						"_TS_pass\n"
+						"\n\n"
+						"# Called each time this task is ticked (aka executed).\n"
+						"func _tick(delta: float) -> int:\n"
+						"_TS_return SUCCESS\n";
+
+				f->store_string(script_template);
+				f->close();
+			}
+
+			ScriptEditor::get_singleton()->open_file("res://script_templates/BTTask/custom_task.gd");
+
+		} break;
 	}
 }
 
@@ -1291,6 +1339,20 @@ void LimboAIEditor::_update_favorite_tasks() {
 	}
 }
 
+void LimboAIEditor::_update_misc_menu() {
+	PopupMenu *misc_menu = misc_btn->get_popup();
+
+	misc_menu->clear();
+
+	misc_menu->add_icon_shortcut(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Debug"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/open_debugger"), MISC_OPEN_DEBUGGER);
+	misc_menu->add_item(TTR("Project Settings..."), MISC_PROJECT_SETTINGS);
+
+	misc_menu->add_separator();
+	misc_menu->add_item(
+			FileAccess::exists("res://script_templates/BTAction/custom_action.gd") ? TTR("Edit Script Template") : TTR("Create Script Template"),
+			MISC_CREATE_SCRIPT_TEMPLATE);
+}
+
 void LimboAIEditor::_notification(int p_what) {
 	switch (p_what) {
 		case NOTIFICATION_ENTER_TREE: {
@@ -1316,10 +1378,6 @@ void LimboAIEditor::_notification(int p_what) {
 			history_forward->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Forward"), SNAME("EditorIcons")));
 
 			misc_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Tools"), SNAME("EditorIcons")));
-			PopupMenu *misc_menu = misc_btn->get_popup();
-			misc_menu->clear();
-			misc_menu->add_icon_shortcut(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Debug"), SNAME("EditorIcons")), ED_GET_SHORTCUT("limbo_ai/open_debugger"), MISC_OPEN_DEBUGGER);
-			misc_menu->add_item(TTR("Project Settings..."), MISC_PROJECT_SETTINGS);
 
 			_update_favorite_tasks();
 			_update_header();
@@ -1441,7 +1499,8 @@ LimboAIEditor::LimboAIEditor() {
 	misc_btn = memnew(MenuButton);
 	misc_btn->set_text(TTR("Misc"));
 	misc_btn->set_flat(true);
-	misc_btn->get_popup()->connect("index_pressed", callable_mp(this, &LimboAIEditor::_misc_option_selected));
+	misc_btn->connect("pressed", callable_mp(this, &LimboAIEditor::_update_misc_menu));
+	misc_btn->get_popup()->connect("id_pressed", callable_mp(this, &LimboAIEditor::_misc_option_selected));
 	toolbar->add_child(misc_btn);
 
 	// toolbar->add_child(memnew(VSeparator));
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 8ead5e9..d8fabaf 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -158,6 +158,7 @@ private:
 	enum MiscMenu {
 		MISC_OPEN_DEBUGGER,
 		MISC_PROJECT_SETTINGS,
+		MISC_CREATE_SCRIPT_TEMPLATE,
 	};
 
 	Vector<Ref<BehaviorTree>> history;
@@ -197,6 +198,7 @@ private:
 	void _update_header() const;
 	void _update_history_buttons();
 	void _update_favorite_tasks();
+	void _update_misc_menu();
 	void _new_bt();
 	void _save_bt(String p_path);
 	void _load_bt(String p_path);

From 00ed65f0fb2e0568125873882a4e34f593e6470f Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Mon, 21 Aug 2023 15:27:30 +0200
Subject: [PATCH 16/24] Use ProjectSettings path for script templates

---
 editor/limbo_ai_editor_plugin.cpp | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 63a8ea1..11544cb 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -727,6 +727,11 @@ TaskPanel::~TaskPanel() {
 
 //**** LimboAIEditor
 
+_FORCE_INLINE_ String _get_script_template_path() {
+	String templates_search_path = GLOBAL_GET("editor/script/templates_search_path");
+	return templates_search_path.path_join("BTTask").path_join("custom_task.gd");
+}
+
 void LimboAIEditor::_add_task(const Ref<BTTask> &p_task) {
 	ERR_FAIL_COND(p_task.is_null());
 	ERR_FAIL_COND(task_tree->get_bt().is_null());
@@ -1097,16 +1102,18 @@ void LimboAIEditor::_misc_option_selected(int p_id) {
 			ProjectSettingsEditor::get_singleton()->popup_project_settings();
 		} break;
 		case MISC_CREATE_SCRIPT_TEMPLATE: {
-			if (!FileAccess::exists("res://script_templates/BTTask/custom_task.gd")) {
+			String template_path = _get_script_template_path();
+			String template_dir = template_path.get_base_dir();
+
+			if (!FileAccess::exists(template_path)) {
 				Ref<DirAccess> dir = DirAccess::create(DirAccess::ACCESS_RESOURCES);
-				if (!dir->exists("res://script_templates/")) {
-					dir->make_dir("res://script_templates/");
-				}
-				if (!dir->exists("res://script_templates/BTTask")) {
-					dir->make_dir("res://script_templates/BTTask");
-				}
 				Error err;
-				Ref<FileAccess> f = FileAccess::open("res://script_templates/BTTask/custom_task.gd", FileAccess::WRITE, &err);
+				if (!dir->exists(template_dir)) {
+					err = dir->make_dir_recursive(template_dir);
+					ERR_FAIL_COND(err != OK);
+				}
+
+				Ref<FileAccess> f = FileAccess::open(template_path, FileAccess::WRITE, &err);
 				ERR_FAIL_COND(err != OK);
 
 				String script_template =
@@ -1141,7 +1148,7 @@ void LimboAIEditor::_misc_option_selected(int p_id) {
 				f->close();
 			}
 
-			ScriptEditor::get_singleton()->open_file("res://script_templates/BTTask/custom_task.gd");
+			ScriptEditor::get_singleton()->open_file(template_path);
 
 		} break;
 	}
@@ -1349,7 +1356,7 @@ void LimboAIEditor::_update_misc_menu() {
 
 	misc_menu->add_separator();
 	misc_menu->add_item(
-			FileAccess::exists("res://script_templates/BTAction/custom_action.gd") ? TTR("Edit Script Template") : TTR("Create Script Template"),
+			FileAccess::exists(_get_script_template_path()) ? TTR("Edit Script Template") : TTR("Create Script Template"),
 			MISC_CREATE_SCRIPT_TEMPLATE);
 }
 

From b33b0c699395cb468248e695e9ed48ee91ca05c2 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Mon, 21 Aug 2023 15:51:18 +0200
Subject: [PATCH 17/24] Add "Refresh" button to TaskPanel

---
 editor/limbo_ai_editor_plugin.cpp | 22 +++++++++++++++++++---
 editor/limbo_ai_editor_plugin.h   |  3 ++-
 2 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 11544cb..a04a789 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -503,7 +503,7 @@ void TaskPanel::_on_task_button_rmb(const String &p_task) {
 	menu->popup();
 }
 
-void TaskPanel::_on_filter_text_changed(String p_text) {
+void TaskPanel::_apply_filter(const String &p_text) {
 	for (int i = 0; i < sections->get_child_count(); i++) {
 		TaskSection *sec = Object::cast_to<TaskSection>(sections->get_child(i));
 		ERR_FAIL_COND(sec == nullptr);
@@ -595,6 +595,10 @@ void TaskPanel::refresh() {
 		sections->add_child(sec);
 		sec->set_collapsed(collapsed_sections.has(cat));
 	}
+
+	if (!filter_edit->get_text().is_empty()) {
+		_apply_filter(filter_edit->get_text());
+	}
 }
 
 void TaskPanel::_populate_core_tasks_from_class(const StringName &p_base_class, List<String> *p_task_classes) {
@@ -681,6 +685,7 @@ void TaskPanel::_notification(int p_what) {
 			conf.save(conf_path);
 		} break;
 		case NOTIFICATION_THEME_CHANGED: {
+			refresh_btn->set_icon(get_theme_icon(SNAME("Reload"), SNAME("EditorIcons")));
 			if (is_visible_in_tree()) {
 				refresh();
 			}
@@ -699,11 +704,22 @@ TaskPanel::TaskPanel() {
 	VBoxContainer *vb = memnew(VBoxContainer);
 	add_child(vb);
 
+	HBoxContainer *hb = memnew(HBoxContainer);
+	vb->add_child(hb);
+
 	filter_edit = memnew(LineEdit);
 	filter_edit->set_clear_button_enabled(true);
 	filter_edit->set_placeholder(TTR("Filter tasks"));
-	filter_edit->connect("text_changed", callable_mp(this, &TaskPanel::_on_filter_text_changed));
-	vb->add_child(filter_edit);
+	filter_edit->connect("text_changed", callable_mp(this, &TaskPanel::_apply_filter));
+	filter_edit->set_h_size_flags(SIZE_EXPAND_FILL);
+	hb->add_child(filter_edit);
+
+	refresh_btn = memnew(Button);
+	refresh_btn->set_tooltip_text(TTR("Refresh tasks"));
+	refresh_btn->set_flat(true);
+	refresh_btn->set_focus_mode(FocusMode::FOCUS_NONE);
+	refresh_btn->connect("pressed", callable_mp(this, &TaskPanel::refresh));
+	hb->add_child(refresh_btn);
 
 	ScrollContainer *sc = memnew(ScrollContainer);
 	sc->set_h_size_flags(SIZE_EXPAND_FILL);
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index d8fabaf..2d339da 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -117,6 +117,7 @@ private:
 	LineEdit *filter_edit;
 	VBoxContainer *sections;
 	PopupMenu *menu;
+	Button *refresh_btn;
 
 	String context_task;
 
@@ -126,7 +127,7 @@ private:
 	void _menu_action_selected(int p_id);
 	void _on_task_button_pressed(const String &p_task);
 	void _on_task_button_rmb(const String &p_task);
-	void _on_filter_text_changed(String p_text);
+	void _apply_filter(const String &p_text);
 
 protected:
 	static void _bind_methods();

From 4942173e5b2d6880cbedfc8ebd88bd07004da29b Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Mon, 21 Aug 2023 18:17:34 +0200
Subject: [PATCH 18/24] Implement "User task folder doesn't exist" warning

---
 editor/limbo_ai_editor_plugin.cpp | 81 ++++++++++++++++++++++++++++---
 editor/limbo_ai_editor_plugin.h   |  6 +++
 2 files changed, 79 insertions(+), 8 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index a04a789..c32a806 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -902,6 +902,24 @@ void LimboAIEditor::_mark_as_dirty(bool p_dirty) {
 	}
 }
 
+void LimboAIEditor::_create_user_task_dir() {
+	String task_dir = GLOBAL_GET("limbo_ai/behavior_tree/user_task_dir_1");
+	ERR_FAIL_COND_MSG(DirAccess::exists(task_dir), "LimboAIEditor: Directory already exists: " + task_dir);
+
+	Ref<DirAccess> dir = DirAccess::create(DirAccess::ACCESS_RESOURCES);
+	Error err;
+	err = dir->make_dir_recursive(task_dir);
+	ERR_FAIL_COND_MSG(err != OK, "LimboAIEditor: Failed to create directory: " + task_dir);
+
+	_update_warnings();
+	EditorFileSystem::get_singleton()->scan_changes();
+}
+
+void LimboAIEditor::_edit_project_settings() {
+	ProjectSettingsEditor::get_singleton()->set_general_page("limbo_ai/behavior_tree");
+	ProjectSettingsEditor::get_singleton()->popup_project_settings();
+}
+
 void LimboAIEditor::shortcut_input(const Ref<InputEvent> &p_event) {
 	if (!p_event->is_pressed()) {
 		return;
@@ -1114,8 +1132,7 @@ void LimboAIEditor::_misc_option_selected(int p_id) {
 			}
 		} break;
 		case MISC_PROJECT_SETTINGS: {
-			ProjectSettingsEditor::get_singleton()->set_general_page("limbo_ai/behavior_tree");
-			ProjectSettingsEditor::get_singleton()->popup_project_settings();
+			_edit_project_settings();
 		} break;
 		case MISC_CREATE_SCRIPT_TEMPLATE: {
 			String template_path = _get_script_template_path();
@@ -1376,6 +1393,10 @@ void LimboAIEditor::_update_misc_menu() {
 			MISC_CREATE_SCRIPT_TEMPLATE);
 }
 
+void LimboAIEditor::_update_warnings() {
+	task_dir_warning->set_visible(!DirAccess::exists(GLOBAL_GET("limbo_ai/behavior_tree/user_task_dir_1")));
+}
+
 void LimboAIEditor::_notification(int p_what) {
 	switch (p_what) {
 		case NOTIFICATION_ENTER_TREE: {
@@ -1402,6 +1423,8 @@ void LimboAIEditor::_notification(int p_what) {
 
 			misc_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Tools"), SNAME("EditorIcons")));
 
+			warning_icon->set_texture(get_theme_icon(SNAME("StatusWarning"), SNAME("EditorIcons")));
+
 			_update_favorite_tasks();
 			_update_header();
 		}
@@ -1517,8 +1540,6 @@ LimboAIEditor::LimboAIEditor() {
 	new_script_btn->set_focus_mode(Control::FOCUS_NONE);
 	toolbar->add_child(new_script_btn);
 
-	// toolbar->add_child(memnew(VSeparator));
-
 	misc_btn = memnew(MenuButton);
 	misc_btn->set_text(TTR("Misc"));
 	misc_btn->set_flat(true);
@@ -1526,8 +1547,6 @@ LimboAIEditor::LimboAIEditor() {
 	misc_btn->get_popup()->connect("id_pressed", callable_mp(this, &LimboAIEditor::_misc_option_selected));
 	toolbar->add_child(misc_btn);
 
-	// toolbar->add_child(memnew(VSeparator));
-
 	HBoxContainer *nav = memnew(HBoxContainer);
 	nav->set_h_size_flags(SIZE_EXPAND | SIZE_SHRINK_END);
 	toolbar->add_child(nav);
@@ -1559,12 +1578,13 @@ LimboAIEditor::LimboAIEditor() {
 	task_tree = memnew(TaskTree);
 	task_tree->set_v_size_flags(SIZE_EXPAND_FILL);
 	task_tree->set_h_size_flags(SIZE_EXPAND_FILL);
+	task_tree->hide();
 	task_tree->connect("rmb_pressed", callable_mp(this, &LimboAIEditor::_on_tree_rmb));
 	task_tree->connect("task_selected", callable_mp(this, &LimboAIEditor::_on_tree_task_selected));
-	task_tree->connect("visibility_changed", callable_mp(this, &LimboAIEditor::_on_visibility_changed));
 	task_tree->connect("task_dragged", callable_mp(this, &LimboAIEditor::_on_task_dragged));
 	task_tree->connect("task_double_clicked", callable_mp(this, &LimboAIEditor::_on_tree_task_double_clicked));
-	task_tree->hide();
+	task_tree->connect("visibility_changed", callable_mp(this, &LimboAIEditor::_on_visibility_changed));
+	task_tree->connect("visibility_changed", callable_mp(this, &LimboAIEditor::_update_warnings));
 	hsc->add_child(task_tree);
 
 	usage_hint = memnew(Panel);
@@ -1587,6 +1607,51 @@ LimboAIEditor::LimboAIEditor() {
 	task_panel->hide();
 	hsc->add_child(task_panel);
 
+	task_dir_warning = memnew(MarginContainer);
+	{
+		task_dir_warning->add_theme_constant_override("margin_bottom", 4);
+		task_dir_warning->add_theme_constant_override("margin_top", 4);
+		task_dir_warning->add_theme_constant_override("margin_left", 4);
+		task_dir_warning->add_theme_constant_override("margin_right", 4);
+
+		HBoxContainer *hb = memnew(HBoxContainer);
+		hb->add_theme_constant_override("hseparation", 8);
+		task_dir_warning->add_child(hb);
+
+		warning_icon = memnew(TextureRect);
+		warning_icon->set_expand_mode(TextureRect::ExpandMode::EXPAND_KEEP_SIZE);
+		warning_icon->set_stretch_mode(TextureRect::StretchMode::STRETCH_KEEP_CENTERED);
+		hb->add_child(warning_icon);
+
+		Label *warning_label = memnew(Label);
+		warning_label->set_text(vformat(TTR("User task folder doesn't exist")));
+		warning_label->set_vertical_alignment(VERTICAL_ALIGNMENT_CENTER);
+		hb->add_child(warning_label);
+
+		Control *spacer = memnew(Control);
+		spacer->set_custom_minimum_size(Size2(0, 16));
+		hb->add_child(spacer);
+
+		Button *create_action = memnew(Button);
+		create_action->set_text(TTR("Create"));
+		create_action->connect("pressed", callable_mp(this, &LimboAIEditor::_create_user_task_dir));
+		hb->add_child(create_action);
+
+		Button *edit_action = memnew(Button);
+		edit_action->set_text(TTR("Edit Path..."));
+		edit_action->connect("pressed", callable_mp(this, &LimboAIEditor::_edit_project_settings));
+		hb->add_child(edit_action);
+
+		Button *refresh_action = memnew(Button);
+		refresh_action->set_text(TTR("Refresh"));
+		refresh_action->connect("pressed", callable_mp(task_panel, &TaskPanel::refresh));
+		refresh_action->connect("pressed", callable_mp(this, &LimboAIEditor::_update_warnings));
+		hb->add_child(refresh_action);
+
+		task_dir_warning->hide();
+	}
+	vb->add_child(task_dir_warning);
+
 	menu = memnew(PopupMenu);
 	add_child(menu);
 	menu->connect("id_pressed", callable_mp(this, &LimboAIEditor::_action_selected));
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 2d339da..e5cfbb5 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -26,6 +26,7 @@
 #include "scene/gui/file_dialog.h"
 #include "scene/gui/flow_container.h"
 #include "scene/gui/line_edit.h"
+#include "scene/gui/margin_container.h"
 #include "scene/gui/panel_container.h"
 #include "scene/gui/popup_menu.h"
 #include "scene/gui/split_container.h"
@@ -177,6 +178,8 @@ private:
 	Button *history_forward;
 	TaskPanel *task_panel;
 	HBoxContainer *fav_tasks_hbox;
+	MarginContainer *task_dir_warning;
+	TextureRect *warning_icon;
 
 	Button *comment_btn;
 	Button *new_btn;
@@ -200,10 +203,13 @@ private:
 	void _update_history_buttons();
 	void _update_favorite_tasks();
 	void _update_misc_menu();
+	void _update_warnings();
 	void _new_bt();
 	void _save_bt(String p_path);
 	void _load_bt(String p_path);
 	void _mark_as_dirty(bool p_dirty);
+	void _create_user_task_dir();
+	void _edit_project_settings();
 
 	void _reload_modified();
 	void _resave_modified(String _str = "");

From 24678e4332cc0f6c70509e56ffe4693c6b475ffd Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Tue, 22 Aug 2023 12:20:09 +0200
Subject: [PATCH 19/24] Implement ActionBanner class for actionable editor
 alerts

---
 editor/action_banner.cpp          | 79 ++++++++++++++++++++++++++++
 editor/action_banner.h            | 48 +++++++++++++++++
 editor/limbo_ai_editor_plugin.cpp | 85 ++++++++++---------------------
 editor/limbo_ai_editor_plugin.h   |  6 +--
 4 files changed, 158 insertions(+), 60 deletions(-)
 create mode 100644 editor/action_banner.cpp
 create mode 100644 editor/action_banner.h

diff --git a/editor/action_banner.cpp b/editor/action_banner.cpp
new file mode 100644
index 0000000..312139b
--- /dev/null
+++ b/editor/action_banner.cpp
@@ -0,0 +1,79 @@
+/**
+ * action_banner.cpp
+ * =============================================================================
+ * Copyright 2021-2023 Serhii Snitsaruk
+ *
+ * Use of this source code is governed by an MIT-style
+ * license that can be found in the LICENSE file or at
+ * https://opensource.org/licenses/MIT.
+ * =============================================================================
+ */
+
+#include "action_banner.h"
+
+#include "scene/gui/button.h"
+
+void ActionBanner::set_text(const String &p_text) {
+	message->set_text(p_text);
+}
+
+String ActionBanner::get_text() const {
+	return message->get_text();
+}
+
+void ActionBanner::close() {
+	queue_free();
+}
+
+void ActionBanner::add_action(const String &p_name, const Callable &p_action, bool p_auto_close) {
+	Button *action_btn = memnew(Button);
+	action_btn->set_text(p_name);
+	action_btn->connect(SNAME("pressed"), callable_mp(this, &ActionBanner::_execute_action).bind(p_action, p_auto_close));
+	hbox->add_child(action_btn);
+}
+
+void ActionBanner::_execute_action(const Callable &p_action, bool p_auto_close) {
+	Callable::CallError ce;
+	Variant ret;
+	p_action.callp(nullptr, 0, ret, ce);
+
+	if (p_auto_close) {
+		queue_free();
+	}
+}
+
+void ActionBanner::_notification(int p_what) {
+	switch (p_what) {
+		case NOTIFICATION_THEME_CHANGED: {
+			icon->set_texture(get_theme_icon(SNAME("StatusWarning"), SNAME("EditorIcons")));
+		} break;
+	}
+}
+
+void ActionBanner::_bind_methods() {
+}
+
+ActionBanner::ActionBanner() {
+	add_theme_constant_override("margin_bottom", 4);
+	add_theme_constant_override("margin_top", 4);
+	add_theme_constant_override("margin_left", 10);
+	add_theme_constant_override("margin_right", 10);
+
+	hbox = memnew(HBoxContainer);
+	hbox->add_theme_constant_override("hseparation", 8);
+	add_child(hbox);
+
+	icon = memnew(TextureRect);
+	icon->set_expand_mode(TextureRect::ExpandMode::EXPAND_KEEP_SIZE);
+	icon->set_stretch_mode(TextureRect::StretchMode::STRETCH_KEEP_CENTERED);
+	hbox->add_child(icon);
+
+	message = memnew(Label);
+	message->set_text(vformat(TTR("User task folder doesn't exist")));
+	message->set_vertical_alignment(VERTICAL_ALIGNMENT_CENTER);
+	hbox->add_child(message);
+
+	Control *spacer = memnew(Control);
+	spacer->set_custom_minimum_size(Size2(0, 16));
+	hbox->add_child(spacer);
+}
diff --git a/editor/action_banner.h b/editor/action_banner.h
new file mode 100644
index 0000000..a4a7ad0
--- /dev/null
+++ b/editor/action_banner.h
@@ -0,0 +1,48 @@
+/**
+ * action_banner.h
+ * =============================================================================
+ * Copyright 2021-2023 Serhii Snitsaruk
+ *
+ * Use of this source code is governed by an MIT-style
+ * license that can be found in the LICENSE file or at
+ * https://opensource.org/licenses/MIT.
+ * =============================================================================
+ */
+/* action_banner.h */
+
+#ifndef ACTION_BANNER_H
+#define ACTION_BANNER_H
+
+#include "scene/gui/margin_container.h"
+
+#include "scene/gui/box_container.h"
+#include "scene/gui/label.h"
+#include "scene/gui/texture_rect.h"
+
+class ActionBanner : public MarginContainer {
+	GDCLASS(ActionBanner, MarginContainer);
+
+private:
+	TextureRect *icon;
+	Label *message;
+	HBoxContainer *hbox;
+
+	void _execute_action(const Callable &p_action, bool p_auto_close);
+
+protected:
+	static void _bind_methods();
+
+	void _notification(int p_what);
+
+public:
+	void set_text(const String &p_text);
+	String get_text() const;
+
+	void add_action(const String &p_name, const Callable &p_action, bool p_auto_close = false);
+
+	void close();
+
+	ActionBanner();
+};
+
+#endif // ACTION_BANNER
\ No newline at end of file
diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index c32a806..3a27193 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -13,6 +13,7 @@
 
 #include "limbo_ai_editor_plugin.h"
 
+#include "action_banner.h"
 #include "modules/limboai/bt/behavior_tree.h"
 #include "modules/limboai/bt/tasks/bt_action.h"
 #include "modules/limboai/bt/tasks/bt_comment.h"
@@ -911,13 +912,14 @@ void LimboAIEditor::_create_user_task_dir() {
 	err = dir->make_dir_recursive(task_dir);
 	ERR_FAIL_COND_MSG(err != OK, "LimboAIEditor: Failed to create directory: " + task_dir);
 
-	_update_warnings();
 	EditorFileSystem::get_singleton()->scan_changes();
+	_update_banners();
 }
 
 void LimboAIEditor::_edit_project_settings() {
 	ProjectSettingsEditor::get_singleton()->set_general_page("limbo_ai/behavior_tree");
 	ProjectSettingsEditor::get_singleton()->popup_project_settings();
+	ProjectSettingsEditor::get_singleton()->connect(SNAME("visibility_changed"), callable_mp(this, &LimboAIEditor::_update_banners), CONNECT_ONE_SHOT);
 }
 
 void LimboAIEditor::shortcut_input(const Ref<InputEvent> &p_event) {
@@ -1393,8 +1395,21 @@ void LimboAIEditor::_update_misc_menu() {
 			MISC_CREATE_SCRIPT_TEMPLATE);
 }
 
-void LimboAIEditor::_update_warnings() {
-	task_dir_warning->set_visible(!DirAccess::exists(GLOBAL_GET("limbo_ai/behavior_tree/user_task_dir_1")));
+void LimboAIEditor::_update_banners() {
+	for (int i = 0; i < banners->get_child_count(); i++) {
+		banners->get_child(i)->queue_free();
+	}
+
+	for (String dir_setting : { "limbo_ai/behavior_tree/user_task_dir_1", "limbo_ai/behavior_tree/user_task_dir_2", "limbo_ai/behavior_tree/user_task_dir_3" }) {
+		String task_dir = GLOBAL_GET(dir_setting);
+		if (!task_dir.is_empty() && !DirAccess::exists(task_dir)) {
+			ActionBanner *banner = memnew(ActionBanner);
+			banner->set_text(vformat(TTR("Task folder not found: %s"), task_dir));
+			banner->add_action(TTR("Create"), callable_mp(this, &LimboAIEditor::_create_user_task_dir), true);
+			banner->add_action(TTR("Edit Path..."), callable_mp(this, &LimboAIEditor::_edit_project_settings));
+			banners->call_deferred(SNAME("add_child"), banner);
+		}
+	}
 }
 
 void LimboAIEditor::_notification(int p_what) {
@@ -1423,8 +1438,6 @@ void LimboAIEditor::_notification(int p_what) {
 
 			misc_btn->set_icon(EditorNode::get_singleton()->get_gui_base()->get_theme_icon(SNAME("Tools"), SNAME("EditorIcons")));
 
-			warning_icon->set_texture(get_theme_icon(SNAME("StatusWarning"), SNAME("EditorIcons")));
-
 			_update_favorite_tasks();
 			_update_header();
 		}
@@ -1476,13 +1489,13 @@ LimboAIEditor::LimboAIEditor() {
 	load_dialog->hide();
 	add_child(load_dialog);
 
-	VBoxContainer *vb = memnew(VBoxContainer);
-	vb->set_anchor(SIDE_RIGHT, ANCHOR_END);
-	vb->set_anchor(SIDE_BOTTOM, ANCHOR_END);
-	add_child(vb);
+	vbox = memnew(VBoxContainer);
+	vbox->set_anchor(SIDE_RIGHT, ANCHOR_END);
+	vbox->set_anchor(SIDE_BOTTOM, ANCHOR_END);
+	add_child(vbox);
 
 	HBoxContainer *toolbar = memnew(HBoxContainer);
-	vb->add_child(toolbar);
+	vbox->add_child(toolbar);
 
 	Array favorite_tasks_default;
 	favorite_tasks_default.append("BTSelector");
@@ -1567,13 +1580,13 @@ LimboAIEditor::LimboAIEditor() {
 	header->set_text_alignment(HORIZONTAL_ALIGNMENT_LEFT);
 	header->add_theme_constant_override("hseparation", 8);
 	header->connect("pressed", callable_mp(this, &LimboAIEditor::_on_header_pressed));
-	vb->add_child(header);
+	vbox->add_child(header);
 
 	hsc = memnew(HSplitContainer);
 	hsc->set_h_size_flags(SIZE_EXPAND_FILL);
 	hsc->set_v_size_flags(SIZE_EXPAND_FILL);
 	hsc->set_focus_mode(FOCUS_NONE);
-	vb->add_child(hsc);
+	vbox->add_child(hsc);
 
 	task_tree = memnew(TaskTree);
 	task_tree->set_v_size_flags(SIZE_EXPAND_FILL);
@@ -1584,7 +1597,7 @@ LimboAIEditor::LimboAIEditor() {
 	task_tree->connect("task_dragged", callable_mp(this, &LimboAIEditor::_on_task_dragged));
 	task_tree->connect("task_double_clicked", callable_mp(this, &LimboAIEditor::_on_tree_task_double_clicked));
 	task_tree->connect("visibility_changed", callable_mp(this, &LimboAIEditor::_on_visibility_changed));
-	task_tree->connect("visibility_changed", callable_mp(this, &LimboAIEditor::_update_warnings));
+	task_tree->connect("visibility_changed", callable_mp(this, &LimboAIEditor::_update_banners));
 	hsc->add_child(task_tree);
 
 	usage_hint = memnew(Panel);
@@ -1607,50 +1620,8 @@ LimboAIEditor::LimboAIEditor() {
 	task_panel->hide();
 	hsc->add_child(task_panel);
 
-	task_dir_warning = memnew(MarginContainer);
-	{
-		task_dir_warning->add_theme_constant_override("margin_bottom", 4);
-		task_dir_warning->add_theme_constant_override("margin_top", 4);
-		task_dir_warning->add_theme_constant_override("margin_left", 4);
-		task_dir_warning->add_theme_constant_override("margin_right", 4);
-
-		HBoxContainer *hb = memnew(HBoxContainer);
-		hb->add_theme_constant_override("hseparation", 8);
-		task_dir_warning->add_child(hb);
-
-		warning_icon = memnew(TextureRect);
-		warning_icon->set_expand_mode(TextureRect::ExpandMode::EXPAND_KEEP_SIZE);
-		warning_icon->set_stretch_mode(TextureRect::StretchMode::STRETCH_KEEP_CENTERED);
-		hb->add_child(warning_icon);
-
-		Label *warning_label = memnew(Label);
-		warning_label->set_text(vformat(TTR("User task folder doesn't exist")));
-		warning_label->set_vertical_alignment(VERTICAL_ALIGNMENT_CENTER);
-		hb->add_child(warning_label);
-
-		Control *spacer = memnew(Control);
-		spacer->set_custom_minimum_size(Size2(0, 16));
-		hb->add_child(spacer);
-
-		Button *create_action = memnew(Button);
-		create_action->set_text(TTR("Create"));
-		create_action->connect("pressed", callable_mp(this, &LimboAIEditor::_create_user_task_dir));
-		hb->add_child(create_action);
-
-		Button *edit_action = memnew(Button);
-		edit_action->set_text(TTR("Edit Path..."));
-		edit_action->connect("pressed", callable_mp(this, &LimboAIEditor::_edit_project_settings));
-		hb->add_child(edit_action);
-
-		Button *refresh_action = memnew(Button);
-		refresh_action->set_text(TTR("Refresh"));
-		refresh_action->connect("pressed", callable_mp(task_panel, &TaskPanel::refresh));
-		refresh_action->connect("pressed", callable_mp(this, &LimboAIEditor::_update_warnings));
-		hb->add_child(refresh_action);
-
-		task_dir_warning->hide();
-	}
-	vb->add_child(task_dir_warning);
+	banners = memnew(VBoxContainer);
+	vbox->add_child(banners);
 
 	menu = memnew(PopupMenu);
 	add_child(menu);
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index e5cfbb5..f59f418 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -167,9 +167,11 @@ private:
 	int idx_history;
 	HashSet<Ref<BehaviorTree>> dirty;
 
+	VBoxContainer *vbox;
 	Button *header;
 	HSplitContainer *hsc;
 	TaskTree *task_tree;
+	VBoxContainer *banners;
 	Panel *usage_hint;
 	PopupMenu *menu;
 	FileDialog *save_dialog;
@@ -178,8 +180,6 @@ private:
 	Button *history_forward;
 	TaskPanel *task_panel;
 	HBoxContainer *fav_tasks_hbox;
-	MarginContainer *task_dir_warning;
-	TextureRect *warning_icon;
 
 	Button *comment_btn;
 	Button *new_btn;
@@ -203,7 +203,7 @@ private:
 	void _update_history_buttons();
 	void _update_favorite_tasks();
 	void _update_misc_menu();
-	void _update_warnings();
+	void _update_banners();
 	void _new_bt();
 	void _save_bt(String p_path);
 	void _load_bt(String p_path);

From e2d596d44a17b44266c126cf00ea929593f41024 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Tue, 22 Aug 2023 14:43:45 +0200
Subject: [PATCH 20/24] Show warning banner when favorite task is not found

---
 editor/limbo_ai_editor_plugin.cpp | 38 +++++++++++++++++++++++++++----
 editor/limbo_ai_editor_plugin.h   |  1 +
 2 files changed, 35 insertions(+), 4 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 3a27193..53e38bc 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -464,7 +464,7 @@ void TaskPanel::_menu_action_selected(int p_id) {
 			ScriptEditor::get_singleton()->open_file(context_task);
 		} break;
 		case MENU_FAVORITE: {
-			Array favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
+			PackedStringArray favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
 			if (favorite_tasks.has(context_task)) {
 				favorite_tasks.erase(context_task);
 			} else {
@@ -922,6 +922,13 @@ void LimboAIEditor::_edit_project_settings() {
 	ProjectSettingsEditor::get_singleton()->connect(SNAME("visibility_changed"), callable_mp(this, &LimboAIEditor::_update_banners), CONNECT_ONE_SHOT);
 }
 
+void LimboAIEditor::_remove_task_from_favorite(const String &p_task) {
+	PackedStringArray favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
+	favorite_tasks.erase(p_task);
+	ProjectSettings::get_singleton()->set_setting("limbo_ai/behavior_tree/favorite_tasks", favorite_tasks);
+	ProjectSettings::get_singleton()->save();
+}
+
 void LimboAIEditor::shortcut_input(const Ref<InputEvent> &p_event) {
 	if (!p_event->is_pressed()) {
 		return;
@@ -1362,6 +1369,12 @@ void LimboAIEditor::_update_favorite_tasks() {
 	Array favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
 	for (int i = 0; i < favorite_tasks.size(); i++) {
 		String task_meta = favorite_tasks[i];
+
+		if (task_meta.is_empty() || (!FileAccess::exists(task_meta) && !ClassDB::class_exists(task_meta))) {
+			call_deferred(SNAME("_update_banners"));
+			continue;
+		}
+
 		Button *btn = memnew(Button);
 		String task_name;
 		if (task_meta.begins_with("res:")) {
@@ -1397,7 +1410,9 @@ void LimboAIEditor::_update_misc_menu() {
 
 void LimboAIEditor::_update_banners() {
 	for (int i = 0; i < banners->get_child_count(); i++) {
-		banners->get_child(i)->queue_free();
+		if (banners->get_child(i)->has_meta(SNAME("managed"))) {
+			banners->get_child(i)->queue_free();
+		}
 	}
 
 	for (String dir_setting : { "limbo_ai/behavior_tree/user_task_dir_1", "limbo_ai/behavior_tree/user_task_dir_2", "limbo_ai/behavior_tree/user_task_dir_3" }) {
@@ -1407,6 +1422,21 @@ void LimboAIEditor::_update_banners() {
 			banner->set_text(vformat(TTR("Task folder not found: %s"), task_dir));
 			banner->add_action(TTR("Create"), callable_mp(this, &LimboAIEditor::_create_user_task_dir), true);
 			banner->add_action(TTR("Edit Path..."), callable_mp(this, &LimboAIEditor::_edit_project_settings));
+			banner->set_meta(SNAME("managed"), Variant(true));
+			banners->call_deferred(SNAME("add_child"), banner);
+		}
+	}
+
+	Array favorite_tasks = GLOBAL_GET("limbo_ai/behavior_tree/favorite_tasks");
+	for (int i = 0; i < favorite_tasks.size(); i++) {
+		String task_meta = favorite_tasks[i];
+
+		if (task_meta.is_empty() || (!FileAccess::exists(task_meta) && !ClassDB::class_exists(task_meta))) {
+			ActionBanner *banner = memnew(ActionBanner);
+			banner->set_text(vformat(TTR("Favorite task not found: %s"), task_meta));
+			banner->add_action(TTR("Remove"), callable_mp(this, &LimboAIEditor::_remove_task_from_favorite).bind(task_meta), true);
+			banner->add_action(TTR("Edit Favorite Tasks..."), callable_mp(this, &LimboAIEditor::_edit_project_settings));
+			banner->set_meta(SNAME("managed"), Variant(true));
 			banners->call_deferred(SNAME("add_child"), banner);
 		}
 	}
@@ -1497,11 +1527,11 @@ LimboAIEditor::LimboAIEditor() {
 	HBoxContainer *toolbar = memnew(HBoxContainer);
 	vbox->add_child(toolbar);
 
-	Array favorite_tasks_default;
+	PackedStringArray favorite_tasks_default;
 	favorite_tasks_default.append("BTSelector");
 	favorite_tasks_default.append("BTSequence");
 	favorite_tasks_default.append("BTParallel");
-	GLOBAL_DEF(PropertyInfo(Variant::ARRAY, "limbo_ai/behavior_tree/favorite_tasks", PROPERTY_HINT_ARRAY_TYPE, "String"), favorite_tasks_default);
+	GLOBAL_DEF(PropertyInfo(Variant::PACKED_STRING_ARRAY, "limbo_ai/behavior_tree/favorite_tasks", PROPERTY_HINT_ARRAY_TYPE, "String"), favorite_tasks_default);
 
 	fav_tasks_hbox = memnew(HBoxContainer);
 	toolbar->add_child(fav_tasks_hbox);
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index f59f418..326b123 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -210,6 +210,7 @@ private:
 	void _mark_as_dirty(bool p_dirty);
 	void _create_user_task_dir();
 	void _edit_project_settings();
+	void _remove_task_from_favorite(const String &p_task);
 
 	void _reload_modified();
 	void _resave_modified(String _str = "");

From 2fdcff4886dfd3eeabcfc6fcbb9944f4b92ac315 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Tue, 22 Aug 2023 15:20:24 +0200
Subject: [PATCH 21/24] Show tasks without category in "Uncategorized" section

---
 editor/limbo_ai_editor_plugin.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 53e38bc..374dfef 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -555,7 +555,7 @@ void TaskPanel::refresh() {
 	categorized_tasks["Conditions"] = List<String>();
 	_populate_core_tasks_from_class("BTCondition", &categorized_tasks["Conditions"]);
 
-	categorized_tasks["User"] = List<String>();
+	categorized_tasks["Uncategorized"] = List<String>();
 
 	String dir1 = GLOBAL_GET("limbo_ai/behavior_tree/user_task_dir_1");
 	_populate_from_user_dir(dir1, &categorized_tasks);
@@ -625,7 +625,7 @@ void TaskPanel::_populate_from_user_dir(String p_path, HashMap<String, List<Stri
 				String category;
 				if (fn == ".") {
 					full_path = p_path;
-					category = "User";
+					category = "Uncategorized";
 				} else {
 					full_path = p_path.path_join(fn);
 					category = fn.capitalize();

From 067f73e11e24681e39e38b95b3a807255ff8708a Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Wed, 23 Aug 2023 13:06:18 +0200
Subject: [PATCH 22/24] Add documentation tooltips for tasks in the panel

---
 editor/limbo_ai_editor_plugin.cpp | 46 +++++++++++++++++++++++++++++--
 editor/limbo_ai_editor_plugin.h   |  9 +++++-
 2 files changed, 51 insertions(+), 4 deletions(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 374dfef..5b27ec0 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -54,6 +54,7 @@
 #include "editor/debugger/editor_debugger_node.h"
 #include "editor/debugger/script_editor_debugger.h"
 #include "editor/editor_file_system.h"
+#include "editor/editor_help.h"
 #include "editor/editor_inspector.h"
 #include "editor/editor_node.h"
 #include "editor/editor_paths.h"
@@ -356,6 +357,26 @@ TaskTree::~TaskTree() {
 
 //**** TaskTree ^
 
+//**** TaskButton
+
+Control *TaskButton::make_custom_tooltip(const String &p_text) const {
+	EditorHelpBit *help_bit = memnew(EditorHelpBit);
+	help_bit->get_rich_text()->set_custom_minimum_size(Size2(360 * EDSCALE, 1));
+
+	String help_text;
+	if (!p_text.is_empty()) {
+		help_text = p_text;
+	} else {
+		help_text = "[i]" + TTR("No description.") + "[/i]";
+	}
+
+	help_bit->set_text(help_text);
+
+	return help_bit;
+}
+
+//**** TaskButton ^
+
 //**** TaskSection
 
 void TaskSection::_on_task_button_pressed(const String &p_task) {
@@ -394,10 +415,11 @@ void TaskSection::set_filter(String p_filter_text) {
 	}
 }
 
-void TaskSection::add_task_button(String p_name, const Ref<Texture> &icon, Variant p_meta) {
-	Button *btn = memnew(Button);
+void TaskSection::add_task_button(const String &p_name, const Ref<Texture> &icon, const String &p_tooltip, Variant p_meta) {
+	TaskButton *btn = memnew(TaskButton);
 	btn->set_text(p_name);
 	btn->set_icon(icon);
+	btn->set_tooltip_text(p_tooltip);
 	btn->add_theme_constant_override(SNAME("icon_max_width"), 16 * EDSCALE); // Force user icons to  be of the proper size.
 	btn->connect(SNAME("pressed"), callable_mp(this, &TaskSection::_on_task_button_pressed).bind(p_meta));
 	btn->connect(SNAME("gui_input"), callable_mp(this, &TaskSection::_on_task_button_gui_input).bind(p_meta));
@@ -582,13 +604,31 @@ void TaskPanel::refresh() {
 		TaskSection *sec = memnew(TaskSection(cat));
 		for (String task_meta : tasks) {
 			Ref<Texture2D> icon = LimboUtility::get_singleton()->get_task_icon(task_meta);
+
 			String tname;
+			DocTools *dd = EditorHelp::get_doc_data();
+			HashMap<String, DocData::ClassDoc>::Iterator E;
 			if (task_meta.begins_with("res:")) {
 				tname = task_meta.get_file().get_basename().trim_prefix("BT").to_pascal_case();
+				E = dd->class_list.find(vformat("\"%s\"", task_meta.trim_prefix("res://")));
+				if (!E) {
+					E = dd->class_list.find(tname);
+				}
 			} else {
 				tname = task_meta.trim_prefix("BT");
+				E = dd->class_list.find(task_meta);
 			}
-			sec->add_task_button(tname, icon, task_meta);
+
+			String descr;
+			if (E) {
+				if (E->value.description.is_empty() || E->value.description.length() > 1000) {
+					descr = DTR(E->value.brief_description);
+				} else {
+					descr = DTR(E->value.description);
+				}
+			}
+
+			sec->add_task_button(tname, icon, descr, task_meta);
 		}
 		sec->set_filter("");
 		sec->connect(SNAME("task_button_pressed"), callable_mp(this, &TaskPanel::_on_task_button_pressed));
diff --git a/editor/limbo_ai_editor_plugin.h b/editor/limbo_ai_editor_plugin.h
index 326b123..c5ed8b2 100644
--- a/editor/limbo_ai_editor_plugin.h
+++ b/editor/limbo_ai_editor_plugin.h
@@ -76,6 +76,13 @@ public:
 	~TaskTree();
 };
 
+class TaskButton : public Button {
+	GDCLASS(TaskButton, Button);
+
+public:
+	virtual Control *make_custom_tooltip(const String &p_text) const override;
+};
+
 class TaskSection : public VBoxContainer {
 	GDCLASS(TaskSection, VBoxContainer);
 
@@ -94,7 +101,7 @@ protected:
 
 public:
 	void set_filter(String p_filter);
-	void add_task_button(String p_name, const Ref<Texture> &icon, Variant p_meta);
+	void add_task_button(const String &p_name, const Ref<Texture> &icon, const String &p_tooltip, Variant p_meta);
 
 	void set_collapsed(bool p_collapsed);
 	bool is_collapsed() const;

From a7d86c535647aaba5ad6e41855462d91b2147889 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Wed, 23 Aug 2023 13:24:22 +0200
Subject: [PATCH 23/24] Refine class docs

---
 doc_classes/BTParallel.xml | 20 ++++++++++----------
 doc_classes/BTSelector.xml | 10 +++++-----
 doc_classes/BTSequence.xml | 10 +++++-----
 3 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/doc_classes/BTParallel.xml b/doc_classes/BTParallel.xml
index 6253f9a..3b74ff6 100644
--- a/doc_classes/BTParallel.xml
+++ b/doc_classes/BTParallel.xml
@@ -1,27 +1,27 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <class name="BTParallel" inherits="BTComposite" version="4.1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="../../../doc/class.xsd">
 	<brief_description>
-		BT composite that executes tasks simultaneously.
+		BT composite that executes child tasks until one of the criteria is met.
 	</brief_description>
 	<description>
-		BT composite that executes tasks simultaneously until one of the criterea is met. BTParallel will execute each task from first to last at least once before returning a result.
-		If set to [member repeat], the tasks will be executed again, even if they returned [code]SUCCESS[/code] or [code]FAILURE[/code] on the previous tick.
-		Returns [code]FAILURE[/code] when a required number of tasks return [code]FAILURE[/code]. When [member repeat] is set to [code]false[/code], if none of the criteria were met, and all child tasks returned [code]SUCCESS[/code] or [code]FAILURE[/code], [BTParallel] will return [code]FAILURE[/code].
-		Returns [code]SUCCESS[/code] when a required number of tasks return [code]SUCCESS[/code].
-		Returns [code]RUNNING[/code] after executing all tasks from first to last, and for as long as the above criterea are not met.
+		BTParallel executes its child tasks until one of the criterea is met. It will execute each task at least once, from the first to the last, before returning a result.
+		If set to [member repeat], the child tasks will be re-executed, regardless of whether they previously resulted in a [code]SUCCESS[/code] or [code]FAILURE[/code].
+		Returns [code]FAILURE[/code] when the required number of tasks return [code]FAILURE[/code]. When [member repeat] is set to [code]false[/code], if none of the criteria were met, and all child tasks resulted in a [code]SUCCESS[/code] or [code]FAILURE[/code], BTParallel will return [code]FAILURE[/code].
+		Returns [code]SUCCESS[/code] when the required number of tasks return [code]SUCCESS[/code].
+		Returns [code]RUNNING[/code] if none of the criterea were fulfilled, and either [member repeat] is set to [code]true[/code] or a child task resulted in [code]RUNNING[/code].
 	</description>
 	<tutorials>
 	</tutorials>
 	<members>
 		<member name="num_failures_required" type="int" setter="set_num_failures_required" getter="get_num_failures_required" default="1">
-			When the specified number of child tasks return [code]SUCCESS[/code], [BTParallel] will also return [code]SUCCESS[/code].
+			When the specified number of child tasks return [code]SUCCESS[/code], BTParallel will also return [code]SUCCESS[/code].
 		</member>
 		<member name="num_successes_required" type="int" setter="set_num_successes_required" getter="get_num_successes_required" default="1">
-			When the specified number of child tasks return [code]FAILURE[/code], [BTParallel] will also return [code]FAILURE[/code].
+			When the specified number of child tasks return [code]FAILURE[/code], BTParallel will also return [code]FAILURE[/code].
 		</member>
 		<member name="repeat" type="bool" setter="set_repeat" getter="get_repeat" default="false">
-			When [code]true[/code], the tasks will be executed again, even if they returned [code]SUCCESS[/code] or [code]FAILURE[/code] on the previous tick.
-			When [code]false[/code], if none of the criteria were met, and all child tasks returned [code]SUCCESS[/code] or [code]FAILURE[/code], [BTParallel] will return [code]FAILURE[/code].
+			When [code]true[/code], the child tasks will be executed again, regardless of whether they previously resulted in a [code]SUCCESS[/code] or [code]FAILURE[/code].
+			When [code]false[/code], if none of the criteria were met, and all child tasks resulted in a [code]SUCCESS[/code] or [code]FAILURE[/code], BTParallel will return [code]FAILURE[/code].
 		</member>
 	</members>
 </class>
diff --git a/doc_classes/BTSelector.xml b/doc_classes/BTSelector.xml
index be57ecc..d18601a 100644
--- a/doc_classes/BTSelector.xml
+++ b/doc_classes/BTSelector.xml
@@ -1,13 +1,13 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <class name="BTSelector" inherits="BTComposite" version="4.1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="../../../doc/class.xsd">
 	<brief_description>
-		BT composite that executes tasks in turn until first [code]SUCCESS[/code].
+		BT composite that sequentially executes tasks until first [code]SUCCESS[/code].
 	</brief_description>
 	<description>
-		BT composite that executes child tasks from first to last until any child returns [code]SUCCESS[/code].
-		Returns [code]RUNNING[/code] if a task returns [code]RUNNING[/code], and remembers last [code]RUNNING[/code] child. BTSelector will continue where it left off on the next execution tick. 
-		Returns [code]FAILURE[/code] if all tasks return [code]FAILURE[/code]. 
-		Returns [code]SUCCESS[/code] if any task returns [code]SUCCESS[/code].
+		BTSelector executes its child tasks sequentially, from first to last, until any child returns [code]SUCCESS[/code].
+		Returns [code]RUNNING[/code] if any child task returns [code]RUNNING[/code]. The composite will also remember the last child task that returned [code]RUNNING[/code], ensuring it resumes from that point in the next execution tick.
+		Returns [code]FAILURE[/code] if all child tasks return [code]FAILURE[/code].
+		Returns [code]SUCCESS[/code] if a child task returns [code]SUCCESS[/code].
 	</description>
 	<tutorials>
 	</tutorials>
diff --git a/doc_classes/BTSequence.xml b/doc_classes/BTSequence.xml
index d92e33b..e10a786 100644
--- a/doc_classes/BTSequence.xml
+++ b/doc_classes/BTSequence.xml
@@ -1,13 +1,13 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <class name="BTSequence" inherits="BTComposite" version="4.1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="../../../doc/class.xsd">
 	<brief_description>
-		BT composite that executes tasks in turn for as long as they return [code]SUCCESS[/code].
+		BT composite that sequentially executes tasks as long as they return [code]SUCCESS[/code].
 	</brief_description>
 	<description>
-		BT composite that executes child tasks from first to last for as long as they return [code]SUCCESS[/code].
-		Returns [code]RUNNING[/code] if a task returns [code]RUNNING[/code], and remembers last [code]RUNNING[/code] child. BTSequence will continue where it left off on the next execution tick. 
-		Returns [code]FAILURE[/code] if any task returns [code]FAILURE[/code]. 
-		Returns [code]SUCCESS[/code] if all tasks return [code]SUCCESS[/code].
+		BTSequence executes its child tasks sequentially, from first to last, as long as they return [code]SUCCESS[/code].
+		Returns [code]RUNNING[/code] if any child task returns [code]RUNNING[/code]. The composite will also remember the last child task that returned [code]RUNNING[/code], ensuring it resumes from that point in the next execution tick.
+		Returns [code]FAILURE[/code] if a child task returns [code]FAILURE[/code].
+		Returns [code]SUCCESS[/code] if all child tasks return [code]SUCCESS[/code].
 	</description>
 	<tutorials>
 	</tutorials>

From f4897447556a280a8b8f85a99892e83bbf6d6163 Mon Sep 17 00:00:00 2001
From: Serhii Snitsaruk <limbonaut@pm.me>
Date: Wed, 23 Aug 2023 16:32:02 +0200
Subject: [PATCH 24/24] Fix: Pressing keys after editor starts result in errors
 printed.

"scene/main/node.cpp:1748 - Parameter "p_node" is null."
---
 editor/limbo_ai_editor_plugin.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/editor/limbo_ai_editor_plugin.cpp b/editor/limbo_ai_editor_plugin.cpp
index 5b27ec0..39f9275 100644
--- a/editor/limbo_ai_editor_plugin.cpp
+++ b/editor/limbo_ai_editor_plugin.cpp
@@ -983,7 +983,7 @@ void LimboAIEditor::shortcut_input(const Ref<InputEvent> &p_event) {
 
 	// * Local shortcuts.
 
-	if (!(has_focus() || is_ancestor_of(get_viewport()->gui_get_focus_owner()))) {
+	if (!(has_focus() || get_viewport()->gui_get_focus_owner() == nullptr || is_ancestor_of(get_viewport()->gui_get_focus_owner()))) {
 		return;
 	}