fix: add nullability to alias in bindings

Author: qubka

include/plugify/binding.hpp

@@ -24,14 +24,14 @@ namespace plugify {
 		[[nodiscard]] const std::string& GetName() const noexcept;
 		[[nodiscard]] const std::string& GetMethod() const noexcept;
 		[[nodiscard]] bool IsBindSelf() const noexcept;
-		[[nodiscard]] const std::inplace_vector<Alias, Signature::kMaxFuncArgs>& GetParamAliases() const noexcept;
-		[[nodiscard]] const Alias& GetRetAlias() const noexcept;
+		[[nodiscard]] const std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs>& GetParamAliases() const noexcept;
+		[[nodiscard]] std::optional<Alias> GetRetAlias() const noexcept;
 
 		// Setters (pass by value and move)
 		void SetName(std::string name);
 		void SetMethod(std::string method);
 		void SetBindSelf(bool bindSelf);
-		void SetParamAliases(std::inplace_vector<Alias, Signature::kMaxFuncArgs> paramAliases);
+		void SetParamAliases(std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs> paramAliases);
 		void SetRetAlias(Alias retAlias);
 
 		[[nodiscard]] bool operator==(const Binding& other) const noexcept;

schemas/plugin.schema.json

@@ -411,7 +411,7 @@
                 "type": "array",
                 "description": "Optional array of aliases for the method's parameters. Each alias provides an alternative name for the corresponding parameter at that position. The array length must match the number of parameters in the referenced method (after bindSelf is applied). Use this to provide more meaningful or language-appropriate parameter names.",
                 "items": {
-                  "type": "object",
+                  "type": ["object", "null"],
                   "description": "An alias for a parameter with optional ownership semantics.",
                   "required": ["name"],
                   "properties": {
@@ -431,7 +431,7 @@
                 "maxItems": 32
               },
               "retAlias": {
-                "type": "object",
+                "type": ["object", "null"],
                 "description": "Optional alias for the return value. Provides an alternative name and ownership semantics for the value returned by this method. Useful for providing more meaningful or language-appropriate return value names in generated bindings.",
                 "required": ["name"],
                 "properties": {

src/core/binding.cpp

@@ -25,8 +25,7 @@ Binding& Binding::operator=(const Binding& other) {
 Binding& Binding::operator=(Binding&& other) noexcept = default;
 
 // Static empty defaults for returning const references to empty containers
-static const std::inplace_vector<Alias, Signature::kMaxFuncArgs> emptyAliases;
-static const Alias emptyAlias;
+static const std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs> emptyAliases;
 
 const std::string& Binding::GetName() const noexcept {
 	return _impl->name;
@@ -40,12 +39,12 @@ bool Binding::IsBindSelf() const noexcept {
 	return _impl->bindSelf.value_or(false);
 }
 
-const std::inplace_vector<Alias, Signature::kMaxFuncArgs>& Binding::GetParamAliases() const noexcept {
+const std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs>& Binding::GetParamAliases() const noexcept {
 	return _impl->paramAliases ? *_impl->paramAliases : emptyAliases;
 }
 
-const Alias& Binding::GetRetAlias() const noexcept {
-	return _impl->retAlias ? *_impl->retAlias : emptyAlias;
+std::optional<Alias> Binding::GetRetAlias() const noexcept {
+	return _impl->retAlias;
 }
 
 void Binding::SetName(std::string name) {
@@ -60,7 +59,7 @@ void Binding::SetBindSelf(bool bindSelf) {
 	_impl->bindSelf = bindSelf;
 }
 
-void Binding::SetParamAliases(std::inplace_vector<Alias, Signature::kMaxFuncArgs> paramAliases) {
+void Binding::SetParamAliases(std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs> paramAliases) {
 	_impl->paramAliases = std::move(paramAliases);
 }
 

src/core/binding_impl.hpp

@@ -8,7 +8,7 @@ namespace plugify {
 		std::string name;
 		std::string method;
 		std::optional<bool> bindSelf;
-		std::optional<std::inplace_vector<Alias, Signature::kMaxFuncArgs>> paramAliases;
+		std::optional<std::inplace_vector<std::optional<Alias>, Signature::kMaxFuncArgs>> paramAliases;
 		std::optional<Alias> retAlias;
 	};
 }

src/core/manifest.cpp

@@ -282,12 +282,14 @@ namespace {
 		// Validate parameter aliases if present
 		if (binding.paramAliases) {
 			for (size_t i = 0; i < binding.paramAliases->size(); ++i) {
-				if (auto result = ValidateAlias(
-						*(*binding.paramAliases)[i]._impl,
-						std::format("{}: Binding '{}' paramAlias[{}]", context, binding.name, i)
-					);
-					!result) {
-					return result;
+				if (auto alias = (*binding.paramAliases)[i]) {
+					if (auto result = ValidateAlias(
+							*alias->_impl,
+							std::format("{}: Binding '{}' paramAlias[{}]", context, binding.name, i)
+						);
+						!result) {
+						return result;
+					}
 				}
 			}
 		}