opencrabs 0.5.3

The autonomous, self-improving AI agent. Single Rust binary. Every channel. Recommended: the 40MB prebuilt binary for macOS, Linux and Windows: https://github.com/adolfousier/opencrabs/releases
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

//! Regression for #682: in a group the agent addressed the owner by another
//! member's name — it called the current sender "Adi" because a different user
//! named Adi was in the injected recent-group-history.
//!
//! The current-speaker label must name WHO to reply to and flag history names as
//! other people, and the history block must read as prior context from various
//! senders, so history names can't bleed into who the agent addresses.

use crate::channels::group_history::{current_sender_label, frame_history};

/// Telegram's own arguments, so the shared helpers are exercised exactly as the
/// Telegram call site uses them.
fn group_current_sender_label(chat_title: &str, name: &str, handle: &str, role: &str) -> String {
    current_sender_label("Telegram group", chat_title, name, handle, role)
}

fn frame_group_history(history_lines: &str, count: usize) -> String {
    frame_history(history_lines, count, "group")
}

#[test]
fn current_sender_label_names_the_speaker_and_fences_off_history_names() {
    let label = group_current_sender_label("team room", "Adolfo", " (@adolfodev)", "owner");
    // Names the current sender and their handle + role.
    assert!(label.contains("Adolfo") && label.contains("(@adolfodev)"));
    assert!(label.contains("owner"));
    // Instructs the agent to reply to THIS sender.
    assert!(
        label.to_lowercase().contains("reply to adolfo"),
        "label must tell the agent who to reply to: {label}"
    );
    // Explicitly fences off history names — the crux of #682.
    assert!(
        label.contains("history") && label.to_lowercase().contains("other people"),
        "label must warn that history names are other people: {label}"
    );
}

#[test]
fn label_role_reflects_owner_vs_user() {
    assert!(group_current_sender_label("g", "Carlos", "", "user").contains("(user)"));
    assert!(group_current_sender_label("g", "Adolfo", "", "owner").contains("(owner)"));
}

#[test]
fn history_frame_marks_lines_as_other_senders_not_the_current_person() {
    let lines = "[14:02] Adi: hello everyone\n[14:03] Carlos: hi";
    let framed = frame_group_history(lines, 2);
    assert!(framed.contains(lines), "history content must be preserved");
    assert!(framed.contains("2 messages"));
    // Framing must make clear these are OTHER senders / context, not the
    // person being replied to.
    let lower = framed.to_lowercase();
    assert!(
        lower.contains("various senders") && lower.contains("not the"),
        "history must be framed as prior context from others: {framed}"
    );
    assert!(framed.contains("--- end history ---"));
}