perl-lsp-code-actions 0.12.2

LSP code actions provider for Perl
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

//! Error checking code actions

use crate::types::{CodeAction, CodeActionEdit, CodeActionKind};
use perl_lsp_rename::TextEdit;
use perl_parser_core::ast::{Node, NodeKind};

/// Add error checking to file operations
pub fn add_error_checking(node: &Node, source: &str) -> Option<CodeAction> {
    if let NodeKind::FunctionCall { name, args: _ } = &node.kind {
        let func_name = name.as_str();

        // Check for file operations without error checking
        if matches!(
            func_name,
            "open" | "close" | "print" | "printf" | "write" | "read" | "seek" | "truncate"
        ) {
            // Check if already has error checking
            if !has_error_checking_nearby(source, node.location.end) {
                let expr_text = &source[node.location.start..node.location.end];

                return Some(CodeAction {
                    title: format!("Add error checking to '{}'", func_name),
                    kind: CodeActionKind::RefactorRewrite,
                    diagnostics: Vec::new(),
                    edit: CodeActionEdit {
                        changes: vec![TextEdit {
                            location: node.location,
                            new_text: format!(
                                "{} or die \"Failed to {}: $!\"",
                                expr_text, func_name
                            ),
                        }],
                    },
                    is_preferred: false,
                });
            }
        }
    }

    None
}

/// Check if there's error checking nearby
pub fn has_error_checking_nearby(source: &str, pos: usize) -> bool {
    // Check next 50 characters for "or", "||", "die", "warn"
    let check_text = &source[pos..std::cmp::min(pos + 50, source.len())];
    check_text.contains(" or ")
        || check_text.contains(" || ")
        || check_text.contains("die")
        || check_text.contains("warn")
}