upskill 0.6.3

Author and distribute AI-assistance content across coding agents
Documentation 
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
121
122
123
124
125
126
127
128
129
130
131
132

//! End-to-end tests for the color disable chain (clig.dev §Output).
//!
//! Five disable signals + one re-enable signal are wired in
//! `src/style.rs`. The unit tests there cover the env-var precedence
//! exhaustively. These integration tests pin the contract through the
//! actual binary so a regression in `style::init` can't sneak past.

use assert_cmd::Command;
use tempfile::tempdir;

const ANSI_ESC: char = '\x1b';

fn assert_no_ansi(output: &str) {
    assert!(
        !output.contains(ANSI_ESC),
        "expected plain output, got ANSI: {output:?}"
    );
}

fn assert_has_ansi(output: &str) {
    assert!(
        output.contains(ANSI_ESC),
        "expected ANSI codes, got plain: {output:?}"
    );
}

#[test]
fn no_color_env_strips_ansi() {
    let cwd = tempdir().unwrap();
    let assert = Command::cargo_bin("upskill")
        .unwrap()
        .current_dir(cwd.path())
        .env("NO_COLOR", "1")
        .env_remove("FORCE_COLOR")
        .env_remove("CLICOLOR_FORCE")
        .args(["add", "not-a-valid-source"])
        .assert()
        .code(2);
    let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
    assert_eq!(stderr, "error: source must be in owner/repo format\n");
}

#[test]
fn no_color_flag_strips_ansi() {
    let cwd = tempdir().unwrap();
    let assert = Command::cargo_bin("upskill")
        .unwrap()
        .current_dir(cwd.path())
        .env_remove("NO_COLOR")
        .env("FORCE_COLOR", "1") // would force color, but --no-color takes precedence
        .args(["--no-color", "add", "not-a-valid-source"])
        .assert()
        .code(2);
    let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
    assert_no_ansi(&stderr);
}

#[test]
fn upskill_no_color_strips_ansi() {
    let cwd = tempdir().unwrap();
    let assert = Command::cargo_bin("upskill")
        .unwrap()
        .current_dir(cwd.path())
        .env_remove("NO_COLOR")
        .env("UPSKILL_NO_COLOR", "1")
        .env_remove("FORCE_COLOR")
        .args(["add", "not-a-valid-source"])
        .assert()
        .code(2);
    let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
    assert_no_ansi(&stderr);
}

#[test]
fn term_dumb_strips_ansi() {
    let cwd = tempdir().unwrap();
    let assert = Command::cargo_bin("upskill")
        .unwrap()
        .current_dir(cwd.path())
        .env("TERM", "dumb")
        .env_remove("NO_COLOR")
        .env_remove("FORCE_COLOR")
        .args(["add", "not-a-valid-source"])
        .assert()
        .code(2);
    let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
    assert_no_ansi(&stderr);
}

#[test]
fn piped_output_strips_ansi_by_default() {
    // assert_cmd runs the binary with stderr/stdout NOT a TTY. With no
    // FORCE_COLOR override, `colored` should auto-disable.
    let cwd = tempdir().unwrap();
    let assert = Command::cargo_bin("upskill")
        .unwrap()
        .current_dir(cwd.path())
        .env_remove("NO_COLOR")
        .env_remove("UPSKILL_NO_COLOR")
        .env_remove("FORCE_COLOR")
        .env_remove("CLICOLOR_FORCE")
        .env("TERM", "xterm")
        .args(["add", "not-a-valid-source"])
        .assert()
        .code(2);
    let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
    assert_no_ansi(&stderr);
}

#[test]
fn force_color_re_enables_when_piped() {
    // Even though stderr is piped (assert_cmd subprocess), FORCE_COLOR
    // tells `colored` to emit ANSI anyway. clig.dev §Environment Variables.
    let cwd = tempdir().unwrap();
    let assert = Command::cargo_bin("upskill")
        .unwrap()
        .current_dir(cwd.path())
        .env_remove("NO_COLOR")
        .env_remove("UPSKILL_NO_COLOR")
        .env("FORCE_COLOR", "1")
        .env("TERM", "xterm")
        .args(["add", "not-a-valid-source"])
        .assert()
        .code(2);
    let stderr = String::from_utf8(assert.get_output().stderr.clone()).unwrap();
    assert_has_ansi(&stderr);
    // The "error:" label should be styled (red bold = sequence containing 31).
    assert!(
        stderr.contains("\x1b[1;31m") || stderr.contains("\x1b[31;1m"),
        "expected red+bold on label: {stderr:?}"
    );
}