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

pub mod block_brace_spacing;
pub mod closure_brace_pipe_spacing;
pub mod closure_pipe_body_spacing;
pub mod no_trailing_spaces;
pub mod omit_list_commas;
pub mod pipe_spacing;
pub mod record_brace_spacing;
pub mod reflow_wide_lists;
pub mod reflow_wide_pipelines;
pub mod wrap_wide_records;

use nu_protocol::Span;

use crate::context::LintContext;

/// Checks if the source text for a block/closure contains explicit pipe
/// delimiters.
///
/// This distinguishes between:
/// - `{ body }` - block without pipe delimiters (returns false)
/// - `{|| body}` - closure with empty parameter pipes (returns true)
/// - `{|x| body}` - closure with parameters (returns true)
///
/// This is needed because Nu's AST doesn't distinguish between a block and a
/// closure with empty pipes - both have no parameters in the signature.
pub fn has_explicit_pipe_delimiters(context: &LintContext, span: Span) -> bool {
    let text = context.span_text(span);
    let mut chars = text.chars();

    if chars.next() != Some('{') {
        return false;
    }

    // Look for opening pipe after optional whitespace
    // Pattern: `{` followed by optional whitespace then `|`
    chars.find(|c| !c.is_whitespace()) == Some('|')
}

/// Determines if a type is a record type.
pub const fn is_record_type(ty: &nu_protocol::Type) -> bool {
    matches!(ty, nu_protocol::Type::Record(_))
}