Remove MaybeSend requirement from Require trait and add to Lua::create_require_function instead

Author: khvzak

src/luau/mod.rs

@@ -7,6 +7,7 @@ use crate::error::Result;
 use crate::function::Function;
 use crate::state::{callback_error_ext, ExtraData, Lua};
 use crate::traits::{FromLuaMulti, IntoLua};
+use crate::types::MaybeSend;
 
 pub use require::{NavigateError, Require, TextRequirer};
 
@@ -17,7 +18,7 @@ impl Lua {
     /// and load modules.
     #[cfg(any(feature = "luau", doc))]
     #[cfg_attr(docsrs, doc(cfg(feature = "luau")))]
-    pub fn create_require_function<R: Require + 'static>(&self, require: R) -> Result<Function> {
+    pub fn create_require_function<R: Require + MaybeSend + 'static>(&self, require: R) -> Result<Function> {
         require::create_require_function(self, require)
     }
 

src/luau/require.rs

@@ -53,7 +53,7 @@ type WriteResult = ffi::luarequire_WriteResult;
 /// A trait for handling modules loading and navigation in the Luau `require-by-string` system.
 #[cfg(any(feature = "luau", doc))]
 #[cfg_attr(docsrs, doc(cfg(feature = "luau")))]
-pub trait Require: MaybeSend {
+pub trait Require {
     /// Returns `true` if "require" is permitted for the given chunk name.
     fn is_require_allowed(&self, chunk_name: &str) -> bool;
 
@@ -517,7 +517,10 @@ unsafe fn write_to_buffer(
 }
 
 #[cfg(feature = "luau")]
-pub(super) fn create_require_function<R: Require + 'static>(lua: &Lua, require: R) -> Result<Function> {
+pub(super) fn create_require_function<R: Require + MaybeSend + 'static>(
+    lua: &Lua,
+    require: R,
+) -> Result<Function> {
     unsafe extern "C-unwind" fn find_current_file(state: *mut ffi::lua_State) -> c_int {
         let mut ar: ffi::lua_Debug = mem::zeroed();
         for level in 2.. {

tests/table.rs

@@ -480,7 +480,7 @@ fn test_too_large_table() -> Result<()> {
     let lua = Lua::new();
 
     // Test creating a too large table
-    let result = lua.create_table_with_capacity(1 << 31, 1 << 27);
+    let result = lua.create_table_with_capacity(1 << 32, 1 << 27);
     assert!(
         matches!(&result, Err(Error::RuntimeError(err)) if err.contains("table overflow") || err.contains("memory allocation error")),
         "expected a runtime error (table overflow), got: {result:?}",