Fix "should not sleep" to catch world.Export and world.Import (#216)

Fixes #208.

Author: spookydonut

src/dreamchecker/lib.rs

@@ -1702,10 +1702,13 @@ impl<'o, 's> AnalyzeProc<'o, 's> {
             },
 
             Term::Call(unscoped_name, args) => {
-                if unscoped_name == "sleep" || unscoped_name == "alert" || unscoped_name == "shell" || unscoped_name == "winexists" || unscoped_name == "winget" {
-                    if self.inside_newcontext == 0 {
+                if self.inside_newcontext == 0 && matches!(unscoped_name.as_str(),
+                    "sleep"
+                    | "alert"
+                    | "shell"
+                    | "winexists"
+                    | "winget") {
                         self.env.sleeping_procs.insert_violator(self.proc_ref, unscoped_name, location);
-                    }
                 }
                 let src = self.ty;
                 if let Some(proc) = self.ty.get_proc(unscoped_name) {
@@ -1952,6 +1955,13 @@ impl<'o, 's> AnalyzeProc<'o, 's> {
                                     .register(self.context);
                             }
                         }
+                        if ty.get().path.as_str() == "/world" {
+                            if self.inside_newcontext == 0 && matches!(name.as_str(),
+                                "Import"
+                                | "Export") {
+                                self.env.sleeping_procs.insert_violator(self.proc_ref, format!("world.{}", name).as_str(), location);
+                            }
+                        }
                         self.visit_call(location, ty, proc, arguments, false, local_vars)
                     } else {
                         error(location, format!("undefined proc: {:?} on {}", name, ty))

src/dreamchecker/tests/directive_tests.rs

@@ -107,4 +107,4 @@ fn no_can_be_redefined() {
     return
 "##.trim();
     check_errors_match(code, NO_CAN_BE_REDEFINED_ERRORS);
-}
+}

src/dreamchecker/tests/sleep_pure_tests.rs

@@ -112,6 +112,21 @@ fn sleep3() {
     ]);
 }
 
+pub const SLEEP_ERROR4: &[(u32, u16, &str)] = &[
+    (1, 16, "/mob/proc/test1 sets SpacemanDMM_should_not_sleep but calls blocking built-in(s)"),
+];
+
+#[test]
+fn sleep4() {
+    let code = r##"
+/mob/proc/test1()
+    set SpacemanDMM_should_not_sleep = TRUE
+    world.Export()
+    world.Import()
+"##.trim();
+    check_errors_match(code, SLEEP_ERROR4);
+}
+
 pub const PURE_ERRORS: &[(u32, u16, &str)] = &[
     (12, 16, "/mob/proc/test2 sets SpacemanDMM_should_be_pure but calls a /proc/impure that does impure operations"),
 ];