Improvement notification export for GDExtensions

[snatvb]

Improvement notification export for GDExtensions

[Bromeon]

As I understood, there are two things:

1. A feature request/discussion whether @export_file("*.filter") could potentially be allowed on Array -- visualized as a list in UI.
2. If not: the question whether the error message "@export_file" requires a variable of type "String" but type "XY" was given instead is exclusive to the GDScript parser, or it can/could also be used for GDExtension properties.

On Rust side, we should in parallel make sure that the property is registered properly with Godot.

[lilizoey]

i dont think this is a godot issue? isn't this just an issue with the rust bindings? im not sure what godot could do here.

[dsnopek]

2. If not: the question whether the error message "@export_file" requires a variable of type "String" but type "XY" was given instead is exclusive to the GDScript parser, or it can/could also be used for GDExtension properties.

Currently, this message comes from gdscript_parser.cpp.

I suppose we could put similar validation in ClassDB::add_property(), however, the error message couldn't really be as good.

From Godot's perspective, it's looking at data passed on the PropertyInfo struct, and the error message could only refer to that (for example, saying the PropertyInfo::hint won't work with the given PropertyInfo::type). However, the binding probably doesn't directly expose PropertyInfo, but instead some binding-specific way of doing it, and so it'd be able to provide a better error message that refers to that.

So, I personally think giving a nice error message should be up to the GDExtension binding (in this case, the Rust binding).

[dsnopek]

Discussed at the GDExtension meeting:

• It would be nice to have an API that that could check if a property is valid, that could potentially be used by GDScript, ClassDB and GDExtension bindings, and still provide nice error messages.
• However, some bindings may be able to check for property validity at compile time (making this unnecessary)
• The property hints have decent documentation but we'd need to make sure it's complete and stays that way.
• We'd also need to ensure that the potential future validation API would remain in sync.
• For now, we recommend that GDExtension bindings attempt to handle this validation themselves, based on the documentation linked above.