moadim 3.2.7

Loop engine for AI agents — routines over REST, MCP, and a built-in web UI
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86

use super::*;
use crate::routine_storage::{remove_routine_dir, write_routine};
use crate::routines::Routine;

fn scratch_home() -> std::path::PathBuf {
    std::env::temp_dir().join(format!("moadim-walk-{}", uuid::Uuid::new_v4()))
}

// `body` is a `&dyn Fn()` rather than a generic `impl FnOnce()` so every call site shares one
// non-generic function body (and one set of coverage counters) instead of each closure
// monomorphizing its own copy of `with_home` — with per-call-site copies, this function's
// `Some`/`None` restore branches (below) could each be satisfied by a *different* copy and
// still leave individual copies under 100% line coverage.
fn with_home(body: &dyn Fn()) {
    let home = scratch_home();
    std::fs::create_dir_all(&home).unwrap();
    let previous = std::env::var_os("MOADIM_HOME_OVERRIDE");
    // SAFETY: test harness is single-threaded.
    unsafe {
        std::env::set_var("MOADIM_HOME_OVERRIDE", &home);
    }
    body();
    // SAFETY: test harness is single-threaded.
    unsafe {
        match previous {
            Some(value) => std::env::set_var("MOADIM_HOME_OVERRIDE", value),
            None => std::env::remove_var("MOADIM_HOME_OVERRIDE"),
        }
    }
    let _ = std::fs::remove_dir_all(&home);
}

fn make_routine(id: &str, title: &str) -> Routine {
    crate::test_fixtures::routine_fixture(id, title)
        .no_repositories()
        .times(1, 2)
        .build()
}

#[test]
fn walk_routines_recurses_into_nested_dirs() {
    with_home(&|| {
        write_routine(&make_routine("walk-nested-id", "team/ops/nightly triage")).unwrap();
        let mut routines = HashMap::new();
        walk_routines(
            &crate::paths::routines_dir(),
            &crate::paths::routines_dir(),
            &mut routines,
            &|base, rel| super::super::load_routine_from_base(base, rel),
        );
        assert!(routines.contains_key("walk-nested-id"));
        assert!(crate::paths::routine_toml_path("team/ops/nightly-triage").exists());
        remove_routine_dir("team/ops/nightly-triage").unwrap();
    });
}

#[test]
fn with_home_restores_a_pre_existing_home_override() {
    // `with_home` is nested here so the inner call's `previous` capture is
    // `Some(outer_home)`, exercising the restore-a-prior-value branch that
    // no other test reaches (every other caller starts from an unset
    // MOADIM_HOME_OVERRIDE, so only the `None` branch ever ran).
    with_home(&|| {
        let outer_home = std::env::var_os("MOADIM_HOME_OVERRIDE").unwrap();
        with_home(&|| {});
        assert_eq!(std::env::var_os("MOADIM_HOME_OVERRIDE"), Some(outer_home));
    });
}

#[test]
fn walk_routines_skips_plain_dirs_without_routine_toml() {
    with_home(&|| {
        std::fs::create_dir_all(crate::paths::routines_dir().join("archive/old")).unwrap();
        write_routine(&make_routine("walk-flat-id", "flat routine")).unwrap();
        let mut routines = HashMap::new();
        walk_routines(
            &crate::paths::routines_dir(),
            &crate::paths::routines_dir(),
            &mut routines,
            &|base, rel| super::super::load_routine_from_base(base, rel),
        );
        assert!(routines.contains_key("walk-flat-id"));
        assert_eq!(routines.len(), 1);
        remove_routine_dir("flat-routine").unwrap();
    });
}