From 78db4ffa0742063618ef62c0235433c50dea3d99 Mon Sep 17 00:00:00 2001
From: Sara <sara@objectionable.solutions>
Date: Thu, 12 Mar 2026 21:43:52 +0100
Subject: [PATCH] feat: added configurationwarnings to AlwaysSuccess

---
 modules/behaviour_nodes/decorator_nodes.cpp | 9 +++++++++
 modules/behaviour_nodes/decorator_nodes.h   | 2 ++
 2 files changed, 11 insertions(+)

diff --git a/modules/behaviour_nodes/decorator_nodes.cpp b/modules/behaviour_nodes/decorator_nodes.cpp
index 83207893..00bb62d8 100644
--- a/modules/behaviour_nodes/decorator_nodes.cpp
+++ b/modules/behaviour_nodes/decorator_nodes.cpp
@@ -1,8 +1,17 @@
 #include "decorator_nodes.h"
 #include "behaviour_nodes/behaviour_node.h"
+#include "core/variant/variant.h"
 
 void BehaviourAlwaysSuccess::_bind_methods() {}
 
+PackedStringArray BehaviourAlwaysSuccess::get_configuration_warnings() const {
+	PackedStringArray warnings{ super_type::get_configuration_warnings() };
+	if (get_child_behaviours().size() != 1) {
+		warnings.push_back("BehaviourAlwaysSuccess should have exactly one child");
+	}
+	return warnings;
+}
+
 void BehaviourAlwaysSuccess::execute() {
 	if (get_child_behaviours().is_empty()) {
 		set_status(Fail);
diff --git a/modules/behaviour_nodes/decorator_nodes.h b/modules/behaviour_nodes/decorator_nodes.h
index d3fd9ad8..fc95249a 100644
--- a/modules/behaviour_nodes/decorator_nodes.h
+++ b/modules/behaviour_nodes/decorator_nodes.h
@@ -2,12 +2,14 @@
 
 #include "behaviour_nodes/behaviour_composite.h"
 #include "behaviour_nodes/behaviour_node.h"
+#include "core/variant/variant.h"
 
 class BehaviourAlwaysSuccess : public BehaviourComposite {
 	GDCLASS(BehaviourAlwaysSuccess, BehaviourComposite);
 	static void _bind_methods();
 
 public:
+	PackedStringArray get_configuration_warnings() const override;
 	void execute() override;
 	BehaviourNode *get_next() override;
 };