fix: don't use WorkerEntrypoint env or ctx in email handlers

Author: LuisDuarte1

worker-build/src/js/shim.js

@@ -21,8 +21,10 @@ class Entrypoint extends WorkerEntrypoint {
         return await imports.scheduled(event, this.env, this.ctx)
     }
 
-    async email(message) {
-        return await imports.email(message, this.env, this.ctx)
+    // For some reason, email events doesn't seem to use WorkerEntrypoint so we get the env and ctx from
+    //  from the function itself.
+    async email(message, _env, _ctx) {
+        return await imports.email(message, _env, _ctx)
     }
 }
 

worker-macros/src/event.rs

@@ -190,7 +190,7 @@ pub fn expand_macro(attr: TokenStream, item: TokenStream) -> TokenStream {
                 &(input_fn.sig.ident.to_string() + "_email_glue"),
                 input_fn.sig.ident.span(),
             );
-            let wrapper_fn_ident = Ident::new("email", input_fn.sig.ident.span());
+            let wrapper_fn_ident = Ident::new("email_handler", input_fn.sig.ident.span());
             // rename the original attributed fn
             input_fn.sig.ident = input_fn_ident.clone();