Filter out invalid rust modules from being used as scripts (#25)

Author: TitanNano

rust-script/src/runtime/metadata.rs

@@ -24,7 +24,7 @@ use crate::{
     },
 };
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub struct ScriptMetaData {
     class_name: ClassName,
     base_type_name: StringName,

rust-script/src/runtime/resource_loader.rs

@@ -62,6 +62,10 @@ impl IResourceFormatLoader for RustScriptResourceLoader {
             return GString::new();
         }
 
+        if script_lang.validate_path(path).is_empty() {
+            return GString::new();
+        }
+
         script_lang.get_type()
     }
 

rust-script/src/runtime/rust_script_language.rs

@@ -18,7 +18,7 @@ use itertools::Itertools;
 
 use crate::apply::Apply;
 
-use super::rust_script::RustScript;
+use super::{metadata::ScriptMetaData, rust_script::RustScript, SCRIPT_REGISTRY};
 
 #[derive(GodotClass)]
 #[class(base = ScriptLanguageExtension, tool)]
@@ -57,6 +57,14 @@ impl RustScriptLanguage {
             .get_singleton(RustScriptLanguage::class_name().to_string_name())
             .map(|gd| gd.cast())
     }
+
+    pub fn script_meta_data(class_name: &str) -> Option<ScriptMetaData> {
+        let reg = SCRIPT_REGISTRY
+            .read()
+            .expect("unable to obtain read access");
+
+        reg.get(class_name).map(ToOwned::to_owned)
+    }
 }
 
 #[godot_api]
@@ -141,10 +149,14 @@ impl IScriptLanguageExtension for RustScriptLanguage {
             return GString::from("rust file is not part of the scripts crate!");
         }
 
-        if !FileAccess::file_exists(path) {
+        if !FileAccess::file_exists(path.clone()) {
             return GString::from("RustScripts can not be created via the Godot editor!");
         }
 
+        if !self.get_global_class_name(path).contains_key("name") {
+            return GString::from("Rust script has not been complied into shared library yet!");
+        }
+
         GString::new()
     }
 
@@ -168,7 +180,14 @@ impl IScriptLanguageExtension for RustScriptLanguage {
     fn get_global_class_name(&self, path: GString) -> Dictionary {
         let class_name = Self::path_to_class_name(&path);
 
-        Dictionary::new().apply(|dict| dict.set("name", class_name))
+        let Some(script) = Self::script_meta_data(&class_name) else {
+            return Dictionary::new();
+        };
+
+        Dictionary::new().apply(|dict| {
+            dict.set("name", class_name);
+            dict.set("base_type", script.base_type_name());
+        })
     }
 
     fn overrides_external_editor(&mut self) -> bool {

rust-script/src/script_registry.rs

@@ -182,13 +182,13 @@ impl RemoteScriptMetaData {
 }
 
 #[abi_stable::sabi_trait]
-pub trait CreateScriptInstanceData: Debug + Sync + Send {
+pub trait CreateScriptInstanceData: Debug + Sync + Send + Clone {
     fn create(&self, base: RemoteValue) -> RemoteGodotScript_TO<'static, RBox<()>>;
 }
 
 impl<F> CreateScriptInstanceData for F
 where
-    F: Fn(Gd<Object>) -> RemoteGodotScript_TO<'static, RBox<()>> + Debug + Sync + Send,
+    F: Fn(Gd<Object>) -> RemoteGodotScript_TO<'static, RBox<()>> + Debug + Sync + Send + Clone,
 {
     fn create(&self, base: RemoteValue) -> RemoteGodotScript_TO<'static, RBox<()>> {
         let variant: Variant = base.into();