nu-lint 1.2.0

Linter for Nu shell scripts that helpfully suggests improvements
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

use super::RULE;

#[test]
fn test_good_match_statement() {
    let good = "match $status { 'ok' => 'success', 'error' => 'failed', _ => 'unknown' }";

    RULE.assert_ignores(good);
}

#[test]
fn test_good_simple_if() {
    let good = "if $condition { 'yes' } else { 'no' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_different_variables() {
    let good = "if $x == 1 { 'one' } else if $y == 2 { 'two' } else { 'other' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_complex_conditions() {
    let good = "if $x > 5 and $y < 10 { 'range' } else { 'outside' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_match_with_guard() {
    let good = "match $value { x if $x > 10 => 'big', x if $x > 5 => 'medium', _ => 'small' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_single_if_no_else() {
    let good = "if $condition { print 'yes' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_two_branch_if_else() {
    // Only 2 branches should not trigger (we require 3+ for a chain)
    let good = r#"
if $x == 1 {
    "one"
} else if $x == 2 {
    "two"
} else {
    "other"
}
"#;
    RULE.assert_ignores(good);
}

#[test]
fn test_good_match_with_alternatives() {
    let good = "match $val { 1 | 2 | 3 => 'low', 4 | 5 | 6 => 'mid', _ => 'high' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_match_with_range() {
    let good = "match $num { 1..10 => 'small', 11..100 => 'medium', _ => 'large' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_match_with_record_destructuring() {
    let good = "match $rec { {a: $val} => $val, _ => 0 }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_match_with_list_destructuring() {
    let good = "match $list { [$first, ..$rest] => $first, _ => null }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_if_with_boolean_operators() {
    let good = r#"
if $x > 0 and $x < 10 {
    "range"
} else if $x >= 10 or $x <= -10 {
    "extreme"
} else {
    "normal"
}
"#;
    RULE.assert_ignores(good);
}

#[test]
fn test_good_if_with_method_calls() {
    let good = r#"
if ($str | str contains "error") {
    "error"
} else if ($str | str contains "warn") {
    "warning"
} else {
    "info"
}
"#;
    RULE.assert_ignores(good);
}

#[test]
fn test_good_if_with_not_operator() {
    let good = "if not $flag { 'disabled' } else { 'enabled' }";
    RULE.assert_ignores(good);
}

#[test]
fn test_good_match_in_pipeline() {
    let good = r#"
$data | match $in {
    {type: "user"} => "User data",
    {type: "admin"} => "Admin data",
    _ => "Unknown data"
}
"#;
    RULE.assert_ignores(good);
}