opencrabs 0.3.58

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

//! Tests for `evolve::diagnose_releases_latest_status` — the
//! honest-error replacement for the prior hardcoded
//! "rate limited or unavailable" suffix that conflated 404
//! (no published release) with 403 (rate limit) with 5xx
//! (GitHub unavailable). One user got "GitHub API returned 404
//! Not Found: rate limited or unavailable" and rationally
//! assumed they'd hit a rate limit when the actual cause was a
//! transient real 404.
//!
//! These tests pin: each branch produces a message that names
//! the actual condition; rate-limit headers are surfaced when
//! present; the body excerpt is appended when the API itself
//! returned an explanation.

use crate::brain::tools::evolve::diagnose_releases_latest_status;
use reqwest::StatusCode;

#[test]
fn _404_says_no_release_not_rate_limit() {
    let msg = diagnose_releases_latest_status(StatusCode::NOT_FOUND, "", None, None);
    assert!(
        msg.contains("no published"),
        "404 must explain that no published release exists, not lie about rate limit: {msg}"
    );
    assert!(
        !msg.contains("rate limit"),
        "404 must NOT mention rate limit — that was the prior bug: {msg}"
    );
}

#[test]
fn _403_names_rate_limit_and_suggests_token() {
    let msg = diagnose_releases_latest_status(
        StatusCode::FORBIDDEN,
        "API rate limit exceeded",
        Some("0"),
        Some("1717179600"),
    );
    assert!(
        msg.contains("rate limit"),
        "403 must name rate limit: {msg}"
    );
    assert!(
        msg.contains("GITHUB_TOKEN"),
        "403 must suggest the token escape hatch: {msg}"
    );
    assert!(
        msg.contains("x-ratelimit-remaining=0"),
        "403 must surface the remaining quota: {msg}"
    );
    assert!(
        msg.contains("x-ratelimit-reset=1717179600"),
        "403 must surface the reset epoch so the user knows when to retry: {msg}"
    );
    assert!(
        msg.contains("API rate limit exceeded"),
        "403 must quote the API's own error message: {msg}"
    );
}

#[test]
fn _429_treated_same_as_403() {
    let msg = diagnose_releases_latest_status(StatusCode::TOO_MANY_REQUESTS, "", Some("0"), None);
    assert!(
        msg.contains("rate limit"),
        "429 must name rate limit: {msg}"
    );
    assert!(msg.contains("x-ratelimit-remaining=0"));
}

#[test]
fn _5xx_says_transient_server_issue() {
    let msg = diagnose_releases_latest_status(StatusCode::BAD_GATEWAY, "", None, None);
    assert!(
        msg.contains("server-side") || msg.contains("retry"),
        "5xx must signal a transient server-side issue: {msg}"
    );
    assert!(
        !msg.contains("rate limit"),
        "5xx must NOT mention rate limit: {msg}"
    );
}

#[test]
fn unknown_4xx_falls_through_to_generic_with_status() {
    let msg = diagnose_releases_latest_status(StatusCode::UNAUTHORIZED, "", None, None);
    assert!(
        msg.contains("401"),
        "fallthrough must carry the actual status so it's debuggable: {msg}"
    );
}

#[test]
fn body_excerpt_is_appended_when_present() {
    let msg = diagnose_releases_latest_status(
        StatusCode::NOT_FOUND,
        r#"{"message":"Not Found","documentation_url":"..."}"#,
        None,
        None,
    );
    assert!(
        msg.contains("API said"),
        "body excerpt must be appended with a clear lead-in: {msg}"
    );
    assert!(
        msg.contains("\"message\":\"Not Found\""),
        "the actual API message must be quoted verbatim: {msg}"
    );
}

#[test]
fn empty_body_excerpt_omitted_cleanly() {
    let msg = diagnose_releases_latest_status(StatusCode::NOT_FOUND, "   ", None, None);
    assert!(
        !msg.contains("API said"),
        "whitespace-only body must not produce an empty 'API said:' tail: {msg}"
    );
}