opencrabs 0.3.68

The autonomous, self-improving AI agent. Single Rust binary. Every channel. Install with: cargo install opencrabs
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
87
88
89
90
91
92
93
94
95

//! Tests for the pinned plan reminder (discussion #177): the active plan's
//! incomplete tasks are re-injected at the end of the prompt each turn so the
//! model doesn't forget it's mid-plan in a long conversation.
//!
//! Lifecycle-engine rule: only an Active checklist is reminded. Editing
//! plans (pre-init flag or design prose) get no execution nagging: telling
//! an Editing session to `plan start` would corrupt the design track.

use crate::brain::agent::service::format_plan_reminder;
use crate::tui::plan::{PlanDocument, PlanStatus, PlanTask, TaskStatus, TaskType};
use uuid::Uuid;

fn plan_with(status: PlanStatus, tasks: Vec<(&str, TaskStatus)>) -> PlanDocument {
    let mut p = PlanDocument::new(Uuid::new_v4(), "Ship login flow".to_string());
    p.status = status;
    for (i, (title, st)) in tasks.into_iter().enumerate() {
        let mut t = PlanTask::new(i, title.to_string(), String::new(), TaskType::Edit);
        t.status = st;
        p.add_task(t);
    }
    p
}

#[test]
fn active_plan_with_incomplete_tasks_is_reminded() {
    let plan = plan_with(
        PlanStatus::Active,
        vec![
            ("wire auth service", TaskStatus::Completed),
            ("build login form", TaskStatus::InProgress),
            ("add tests", TaskStatus::Pending),
        ],
    );
    let out = format_plan_reminder(&plan).expect("should remind for an in-flight plan");
    assert!(out.contains("ACTIVE PLAN REMINDER"));
    assert!(out.contains("1/3 done"));
    assert!(
        out.contains("→ Task 1: build login form"),
        "in-progress task must be flagged with its order + title, got: {out}"
    );
    assert!(
        out.contains("☐ Task 2: add tests"),
        "pending task must be listed with its order + title, got: {out}"
    );
    // Completed tasks are not listed.
    assert!(!out.contains("wire auth service"));
}

#[test]
fn editing_plan_is_not_reminded() {
    // An Editing plan (design prose, checklist not live) must not nag.
    let plan = plan_with(PlanStatus::Editing, vec![("x", TaskStatus::Pending)]);
    assert!(format_plan_reminder(&plan).is_none());
}

#[test]
fn pre_init_sidecar_is_not_reminded() {
    // The durable pre-init flag must never trigger checklist nagging, even
    // if a stray status/tasks combination sneaks onto the sidecar.
    let mut plan = plan_with(PlanStatus::Active, vec![("x", TaskStatus::Pending)]);
    plan.pre_init_editing = true;
    assert!(format_plan_reminder(&plan).is_none());
}

#[test]
fn legacy_editing_statuses_map_to_editing_and_are_not_reminded() {
    // Legacy "Draft" / "PendingApproval" / "Rejected" strings deserialize
    // to Editing, which never nags.
    for legacy in ["Draft", "PendingApproval", "Rejected"] {
        let status: PlanStatus = serde_json::from_str(&format!("\"{legacy}\"")).unwrap();
        assert_eq!(status, PlanStatus::Editing, "{legacy} must map to Editing");
        let plan = plan_with(status, vec![("x", TaskStatus::Pending)]);
        assert!(format_plan_reminder(&plan).is_none());
    }
}

#[test]
fn fully_resolved_plan_is_not_reminded() {
    // All tasks done or skipped → nothing left, no reminder.
    let plan = plan_with(
        PlanStatus::Active,
        vec![("a", TaskStatus::Completed), ("b", TaskStatus::Skipped)],
    );
    assert!(format_plan_reminder(&plan).is_none());
}

#[test]
fn legacy_active_statuses_map_to_active_and_are_reminded() {
    for legacy in ["Approved", "InProgress"] {
        let status: PlanStatus = serde_json::from_str(&format!("\"{legacy}\"")).unwrap();
        assert_eq!(status, PlanStatus::Active, "{legacy} must map to Active");
        let plan = plan_with(status, vec![("only task", TaskStatus::Pending)]);
        assert!(format_plan_reminder(&plan).is_some());
    }
}