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

use super::RULE;

#[test]
fn test_fix_simple_split_get() {
    let bad_code = r#"
let split = ("a:b:c" | split row ":")
$split | get 0
"#;
    RULE.assert_fixed_contains(bad_code, r#"parse "{field0}:{field1}:{field2}""#);
    RULE.assert_fixed_contains(bad_code, "get 0.field0");
}

#[test]
fn test_fix_split_get_different_index() {
    let bad_code = r#"
let parts = ("hello world" | split row " ")
$parts | get 1
"#;
    RULE.assert_fixed_contains(bad_code, r#"parse "{field0} {field1}""#);
    RULE.assert_fixed_contains(bad_code, "get 0.field1");
}

#[test]
fn test_fix_with_filter() {
    let bad_code = r#"
let split = ("a,b,c" | split row "," | filter {|x| $x != "b"})
$split | get 0
"#;
    // Note: This fix won't preserve filter semantics perfectly, but suggests parse
    RULE.assert_fixed_contains(bad_code, r#"parse "{field0},{field1},{field2}""#);
    RULE.assert_fixed_contains(bad_code, "get 0.field0");
}

#[test]
fn test_fix_with_skip() {
    let bad_code = r#"
let data = ("one:two:three" | split row ":")
$data | skip 1
"#;
    RULE.assert_fixed_contains(bad_code, r#"parse "{field0}:{field1}:{field2}""#);
    RULE.assert_fixed_contains(bad_code, "get 0.field1");
}

#[test]
fn test_fix_multi_character_delimiter() {
    let bad_code = r#"
let parts = ("foo::bar::baz" | split row "::")
$parts | get 2
"#;
    RULE.assert_fixed_contains(bad_code, r#"parse "{field0}::{field1}::{field2}""#);
    RULE.assert_fixed_contains(bad_code, "get 0.field2");
}

#[test]
fn test_fix_many_fields() {
    let bad_code = r#"
let items = ("one|two|three|four|five" | split row "|")
$items | get 3
"#;
    // | is a regex special char, so should use --regex
    RULE.assert_fixed_contains(bad_code, "parse --regex");
    RULE.assert_fixed_contains(bad_code, "get 0.field3");
}

#[test]
fn test_fix_datetime_pattern() {
    let bad_code = r#"
let parts = ("2025-11-20 08:21:51" | split row " ")
$parts | get 1
"#;
    RULE.assert_fixed_contains(bad_code, r#"parse "{field0} {field1}""#);
    RULE.assert_fixed_contains(bad_code, "get 0.field1");
}

#[test]
fn test_fix_with_where_filter() {
    let bad_code = r#"
let split = ("a  b  c" | split row " " | where $it != "")
$split | get 1
"#;
    // The fix suggests parse but won't perfectly replicate the where filter
    // behavior
    RULE.assert_fixed_contains(
        bad_code,
        r#"parse "{field0} {field1} {field2} {field3} {field4}""#,
    );
    RULE.assert_fixed_contains(bad_code, "get 0.field1");
}

#[test]
fn test_fix_statements_in_between() {
    let bad_code = r#"
let split = ("a:b:c" | split row ":")
let x = 42
$split | get 2
"#;
    RULE.assert_fixed_contains(bad_code, r#"parse "{field0}:{field1}:{field2}""#);
    RULE.assert_fixed_contains(bad_code, "get 0.field2");
}

#[test]
fn test_fix_regex_special_char_delimiter() {
    let bad_code = r#"
let parts = ("a.b.c.d" | split row ".")
$parts | get 0
"#;
    // Should use --regex with escaped delimiter
    RULE.assert_fixed_contains(bad_code, "parse --regex");
    RULE.assert_fixed_contains(bad_code, "get 0.field0");
}