Merge 224ceda390 into 4c9028fc66

.github/workflows/all_builds.yml

@@ -5,7 +5,7 @@ on:
       godot-ref:
         description: A tag, branch or commit hash in the Godot repository.
         type: string
-        default: 4.3-stable
+        default: 4.3
       limboai-ref:
         description: A tag, branch or commit hash in the LimboAI repository.
         type: string

doc/source/classes/class_limbostate.rst

@@ -234,7 +234,7 @@ Called when the state is entered.
 
 |void| **_exit**\ (\ ) |virtual| :ref:`🔗<class_LimboState_private_method__exit>`
 
-Called when the state is exited. This happens on a transition to another state, and when the state machine is removed from the scene tree (e.g., when the node is freed with :ref:`Node.queue_free<class_Node_method_queue_free>` or the scene changes). Due to implementation details, :ref:`_exit<class_LimboState_private_method__exit>` will not be called on :ref:`Object.free<class_Object_method_free>`!
+Called when the state is exited.
 
 .. rst-class:: classref-item-separator
 

doc_classes/LimboState.xml

@@ -20,7 +20,7 @@
 		<method name="_exit" qualifiers="virtual">
 			<return type="void" />
 			<description>
-				Called when the state is exited. This happens on a transition to another state, and when the state machine is removed from the scene tree (e.g., when the node is freed with [method Node.queue_free] or the scene changes). Due to implementation details, [method _exit] will not be called on [method Object.free]!
+				Called when the state is exited.
 			</description>
 		</method>
 		<method name="_setup" qualifiers="virtual">

editor/limbo_ai_editor_plugin.cpp

@@ -66,7 +66,6 @@
 #include <godot_cpp/classes/ref_counted.hpp>
 #include <godot_cpp/classes/resource_loader.hpp>
 #include <godot_cpp/classes/resource_saver.hpp>
-#include <godot_cpp/classes/scene_tree.hpp>
 #include <godot_cpp/classes/script.hpp>
 #include <godot_cpp/classes/script_editor.hpp>
 #include <godot_cpp/classes/script_editor_base.hpp>
@@ -74,21 +73,6 @@
 #include <godot_cpp/core/error_macros.hpp>
 #endif // LIMBOAI_GDEXTENSION
 
-namespace {
-
-// If built-in resource - switch to the owner scene (open it if not already).
-inline void _switch_to_owner_scene_if_builtin(const Ref<BehaviorTree> &p_behavior_tree) {
-	if (p_behavior_tree.is_valid() && p_behavior_tree->get_path().contains("::")) {
-		String current_scene = SCENE_TREE()->get_edited_scene_root()->get_scene_file_path();
-		String scene_path = p_behavior_tree->get_path().get_slice("::", 0);
-		if (current_scene != scene_path) {
-			EditorInterface::get_singleton()->open_scene_from_path(scene_path);
-		}
-	}
-}
-
-} // unnamed namespace
-
 //**** LimboAIEditor
 
 _FORCE_INLINE_ String _get_script_template_path() {
@@ -307,8 +291,6 @@ void LimboAIEditor::_disable_editing() {
 void LimboAIEditor::edit_bt(const Ref<BehaviorTree> &p_behavior_tree, bool p_force_refresh) {
 	ERR_FAIL_COND_MSG(p_behavior_tree.is_null(), "p_behavior_tree is null");
 
-	_switch_to_owner_scene_if_builtin(p_behavior_tree);
-
 	if (!p_force_refresh && task_tree->get_bt() == p_behavior_tree) {
 		return;
 	}
@@ -494,14 +476,6 @@ void LimboAIEditor::_process_shortcut_input(const Ref<InputEvent> &p_event) {
 		} else if (LW_IS_SHORTCUT("limbo_ai/close_tab", p_event)) {
 			_tab_menu_option_selected(TAB_CLOSE);
 			handled = true;
-		} else if (LW_IS_SHORTCUT("limbo_ai/editor_save_scene", p_event)) {
-			// This intercepts the editor save action, but does not set the event as handled because we don't know the user's intention.
-			// We just want to save the currently edited BT as well, which may cause a loop with built-in resource if done from "_save_external_data".
-			// Workaround for: https://github.com/limbonaut/limboai/issues/240#issuecomment-2453087424
-			if (task_tree->get_bt().is_valid() && RESOURCE_IS_BUILT_IN(task_tree->get_bt())) {
-				_on_save_pressed();
-			}
-			handled = false; // intentionally not set as handled
 		}
 	}
 
@@ -900,7 +874,6 @@ void LimboAIEditor::_on_tree_task_activated() {
 
 void LimboAIEditor::_on_visibility_changed() {
 	if (task_tree->is_visible_in_tree()) {
-		_switch_to_owner_scene_if_builtin(task_tree->get_bt());
 		Ref<BTTask> sel = task_tree->get_selected();
 		if (sel.is_valid()) {
 			EDIT_RESOURCE(sel);
@@ -918,6 +891,16 @@ void LimboAIEditor::_on_visibility_changed() {
 	}
 }
 
+void LimboAIEditor::_on_header_pressed() {
+	task_tree->clear_selection();
+#ifdef LIMBOAI_MODULE
+	if (task_tree->get_bt().is_valid()) {
+		task_tree->get_bt()->editor_set_section_unfold("blackboard_plan", true);
+	}
+#endif // LIMBOAI_MODULE
+	EDIT_RESOURCE(task_tree->get_bt());
+}
+
 void LimboAIEditor::_on_save_pressed() {
 	if (task_tree->get_bt().is_null()) {
 		return;
@@ -1597,14 +1580,11 @@ LimboAIEditor::LimboAIEditor() {
 	LW_SHORTCUT("limbo_ai/save_behavior_tree", TTR("Save Behavior Tree"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY_MASK(ALT) | LW_KEY(S)));
 	LW_SHORTCUT("limbo_ai/load_behavior_tree", TTR("Load Behavior Tree"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY_MASK(ALT) | LW_KEY(L)));
 	LW_SHORTCUT("limbo_ai/open_debugger", TTR("Open Debugger"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY_MASK(ALT) | LW_KEY(D)));
-	LW_SHORTCUT("limbo_ai/jump_to_owner", TTR("Jump to Owner"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY(G)));
+	LW_SHORTCUT("limbo_ai/jump_to_owner", TTR("Jump to Owner"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY(J)));
 	LW_SHORTCUT("limbo_ai/close_tab", TTR("Close Tab"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY(W)));
 	LW_SHORTCUT("limbo_ai/find_task", TTR("Find Task"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY(F)));
 	LW_SHORTCUT("limbo_ai/hide_tree_search", TTR("Close Search"), (Key)(LW_KEY(ESCAPE)));
 
-	// Intercept editor save scene action.
-	LW_SHORTCUT("limbo_ai/editor_save_scene", TTR("Save Scene"), (Key)(LW_KEY_MASK(CMD_OR_CTRL) | LW_KEY(S)));
-
 	set_process_shortcut_input(true);
 
 	save_dialog = memnew(FileDialog);
@@ -1968,9 +1948,8 @@ void LimboAIEditorPlugin::edit(Object *p_object) {
 #elif LIMBOAI_GDEXTENSION
 void LimboAIEditorPlugin::_edit(Object *p_object) {
 #endif
-	Ref<BehaviorTree> bt = Object::cast_to<BehaviorTree>(p_object);
+	if (Object::cast_to<BehaviorTree>(p_object)) {
-	if (bt.is_valid()) {
+		limbo_ai_editor->edit_bt(Object::cast_to<BehaviorTree>(p_object));
-		limbo_ai_editor->edit_bt(bt);
 	}
 }
 

editor/limbo_ai_editor_plugin.h

@@ -243,6 +243,7 @@ private:
 	void _on_tree_task_selected(const Ref<BTTask> &p_task);
 	void _on_tree_task_activated();
 	void _on_visibility_changed();
+	void _on_header_pressed();
 	void _on_save_pressed();
 	void _on_history_back();
 	void _on_history_forward();

editor/task_tree.h

@@ -95,7 +95,7 @@ protected:
 public:
 	void load_bt(const Ref<BehaviorTree> &p_behavior_tree);
 	void unload();
-	_FORCE_INLINE_ Ref<BehaviorTree> get_bt() const { return bt; }
+	Ref<BehaviorTree> get_bt() const { return bt; }
 	void update_tree() { _update_tree(); }
 	void update_task(const Ref<BTTask> &p_task);
 	void add_selection(const Ref<BTTask> &p_task);