opencrabs 0.3.57

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
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120

//! Regression tests for the `/goal` slash command wiring (#232).
//!
//! Three things broke in practice and are pinned here:
//!
//! 1. The `slash_command` dispatch matches the command verbatim, so a model
//!    passing `command='/goal debug the build'` (everything in one field)
//!    fell through to the user-command path and failed. `normalize_command`
//!    splits the first token off so the call routes correctly regardless of
//!    how the model fills the two fields.
//!
//! 2. The shared `goal_command_prompt` directive must instruct
//!    `command='/goal'` with the rest in `args`, never `command='/goal …'`.
//!
//! 3. A bare `/goal` (no text) must be DENIED with a usage warning, never
//!    forwarded to the agent — `is_bare` gates that, and the warning is a
//!    display-only string.

use crate::brain::goal::{goal_command_prompt, goal_usage_warning, is_bare};
use crate::brain::tools::slash_command::normalize_command;

// ── normalize_command: the dispatch-robustness fix ──────────────────

#[test]
fn multiword_command_field_is_split_into_command_and_args() {
    // The exact failure from the screenshot: whole string in `command`.
    let (cmd, args) = normalize_command("/goal debug opencrabs last commits", "");
    assert_eq!(cmd, "/goal");
    assert_eq!(args, "debug opencrabs last commits");
}

#[test]
fn already_split_command_passes_through_unchanged() {
    let (cmd, args) = normalize_command("/goal", "debug the build");
    assert_eq!(cmd, "/goal");
    assert_eq!(args, "debug the build");
}

#[test]
fn bare_command_keeps_empty_args() {
    let (cmd, args) = normalize_command("/goal", "");
    assert_eq!(cmd, "/goal");
    assert_eq!(args, "");
}

#[test]
fn command_trailing_words_prepend_to_existing_args() {
    // Both fields populated: trailing words from `command` come first.
    let (cmd, args) = normalize_command("/goal set", "fix the flaky test");
    assert_eq!(cmd, "/goal");
    assert_eq!(args, "set fix the flaky test");
}

#[test]
fn whitespace_is_trimmed_on_both_fields() {
    let (cmd, args) = normalize_command("  /goal   status  ", "   ");
    assert_eq!(cmd, "/goal");
    assert_eq!(args, "status");
}

#[test]
fn normalize_is_generic_across_commands() {
    // Not goal-specific: /models with the model name stuffed into command.
    let (cmd, args) = normalize_command("/models gpt-5", "");
    assert_eq!(cmd, "/models");
    assert_eq!(args, "gpt-5");
}

// ── is_bare: the deny gate ──────────────────────────────────────────

#[test]
fn is_bare_true_for_empty_and_whitespace() {
    assert!(is_bare(""));
    assert!(is_bare("   "));
    assert!(is_bare("\t \n"));
}

#[test]
fn is_bare_false_when_text_present() {
    assert!(!is_bare("fix the build"));
    assert!(!is_bare("  status  "));
}

// ── goal_command_prompt: the shared directive ───────────────────────

#[test]
fn prompt_never_folds_args_into_command_field() {
    // The bug: directive said command='/goal <args>'. It must say
    // command='/goal' and args='<args>' separately.
    let p = goal_command_prompt("debug opencrabs last commits");
    assert!(
        p.contains("command='/goal'"),
        "directive must keep command as exactly /goal: {p}"
    );
    assert!(
        p.contains("args='debug opencrabs last commits'"),
        "directive must pass the goal text via args: {p}"
    );
    assert!(
        !p.contains("command='/goal debug"),
        "directive must NOT stuff the goal text into the command field: {p}"
    );
}

#[test]
fn prompt_trims_surrounding_whitespace_in_args() {
    let p = goal_command_prompt("   fix the build   ");
    assert!(p.contains("args='fix the build'"), "{p}");
}

// ── goal_usage_warning: the deny message ────────────────────────────

#[test]
fn usage_warning_shows_correct_shape() {
    let w = goal_usage_warning();
    // Tells the user the command-followed-by-goal shape.
    assert!(w.contains("/goal <your goal>"), "{w}");
    // Mentions the management subcommands.
    assert!(w.contains("/goal status"), "{w}");
    assert!(w.contains("/goal clear"), "{w}");
}