2020-06-10 17:29:48 -04:00
|
|
|
// Copyright 2018-2020 the Deno authors. All rights reserved. MIT license.
|
|
|
|
|
|
|
|
//! This module provides file formating utilities using
|
|
|
|
//! [`deno_lint`](https://github.com/denoland/deno_lint).
|
|
|
|
//!
|
|
|
|
//! At the moment it is only consumed using CLI but in
|
|
|
|
//! the future it can be easily extended to provide
|
|
|
|
//! the same functions as ops available in JS runtime.
|
|
|
|
|
|
|
|
use crate::colors;
|
|
|
|
use crate::file_fetcher::map_file_extension;
|
|
|
|
use crate::fmt::collect_files;
|
2020-06-11 19:44:17 -04:00
|
|
|
use crate::fmt::run_parallelized;
|
2020-06-10 17:29:48 -04:00
|
|
|
use crate::fmt_errors;
|
|
|
|
use crate::swc_util;
|
|
|
|
use deno_core::ErrBox;
|
|
|
|
use deno_lint::diagnostic::LintDiagnostic;
|
|
|
|
use deno_lint::linter::Linter;
|
|
|
|
use deno_lint::rules;
|
|
|
|
use std::fs;
|
|
|
|
use std::path::PathBuf;
|
2020-06-11 19:44:17 -04:00
|
|
|
use std::sync::atomic::{AtomicUsize, Ordering};
|
|
|
|
use std::sync::{Arc, Mutex};
|
2020-06-10 17:29:48 -04:00
|
|
|
|
2020-06-11 19:44:17 -04:00
|
|
|
pub async fn lint_files(args: Vec<String>) -> Result<(), ErrBox> {
|
2020-06-10 17:29:48 -04:00
|
|
|
let target_files = collect_files(args)?;
|
2020-06-11 19:44:17 -04:00
|
|
|
debug!("Found {} files", target_files.len());
|
2020-06-10 17:29:48 -04:00
|
|
|
|
2020-06-11 19:44:17 -04:00
|
|
|
let error_count = Arc::new(AtomicUsize::new(0));
|
2020-06-10 17:29:48 -04:00
|
|
|
|
2020-06-11 19:44:17 -04:00
|
|
|
// prevent threads outputting at the same time
|
|
|
|
let output_lock = Arc::new(Mutex::new(0));
|
|
|
|
|
|
|
|
run_parallelized(target_files, {
|
|
|
|
let error_count = error_count.clone();
|
|
|
|
move |file_path| {
|
|
|
|
let r = lint_file(file_path.clone());
|
|
|
|
|
|
|
|
match r {
|
|
|
|
Ok(file_diagnostics) => {
|
|
|
|
error_count.fetch_add(file_diagnostics.len(), Ordering::SeqCst);
|
|
|
|
let _g = output_lock.lock().unwrap();
|
|
|
|
for d in file_diagnostics.iter() {
|
|
|
|
let fmt_diagnostic = format_diagnostic(d);
|
|
|
|
eprintln!("{}\n", fmt_diagnostic);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
Err(err) => {
|
|
|
|
eprintln!("Error linting: {}", file_path.to_string_lossy());
|
|
|
|
eprintln!(" {}", err);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
Ok(())
|
2020-06-10 17:29:48 -04:00
|
|
|
}
|
2020-06-11 19:44:17 -04:00
|
|
|
})
|
|
|
|
.await?;
|
2020-06-10 17:29:48 -04:00
|
|
|
|
2020-06-11 19:44:17 -04:00
|
|
|
let error_count = error_count.load(Ordering::SeqCst);
|
|
|
|
if error_count > 0 {
|
|
|
|
eprintln!("Found {} problems", error_count);
|
2020-06-10 17:29:48 -04:00
|
|
|
std::process::exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
Ok(())
|
|
|
|
}
|
|
|
|
|
2020-06-12 10:42:12 -04:00
|
|
|
pub fn print_rules_list() {
|
|
|
|
let lint_rules = rules::get_recommended_rules();
|
|
|
|
|
|
|
|
println!("Available rules:");
|
|
|
|
for rule in lint_rules {
|
|
|
|
println!(" - {}", rule.code());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-06-11 19:44:17 -04:00
|
|
|
fn create_linter() -> Linter {
|
|
|
|
Linter::new(
|
|
|
|
"deno-lint-ignore-file".to_string(),
|
|
|
|
vec![
|
|
|
|
"deno-lint-ignore".to_string(),
|
|
|
|
"eslint-disable-next-line".to_string(),
|
|
|
|
],
|
|
|
|
// TODO(bartlomieju): switch to true, once
|
|
|
|
// https://github.com/denoland/deno_lint/issues/156 is fixed
|
|
|
|
false,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
2020-06-10 17:29:48 -04:00
|
|
|
fn lint_file(file_path: PathBuf) -> Result<Vec<LintDiagnostic>, ErrBox> {
|
|
|
|
let file_name = file_path.to_string_lossy().to_string();
|
|
|
|
let source_code = fs::read_to_string(&file_path)?;
|
|
|
|
let media_type = map_file_extension(&file_path);
|
|
|
|
let syntax = swc_util::get_syntax_for_media_type(media_type);
|
|
|
|
|
2020-06-11 19:44:17 -04:00
|
|
|
let mut linter = create_linter();
|
2020-06-10 17:29:48 -04:00
|
|
|
let lint_rules = rules::get_recommended_rules();
|
|
|
|
|
|
|
|
let file_diagnostics =
|
|
|
|
linter.lint(file_name, source_code, syntax, lint_rules)?;
|
|
|
|
|
|
|
|
Ok(file_diagnostics)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn format_diagnostic(d: &LintDiagnostic) -> String {
|
|
|
|
let pretty_message = format!(
|
|
|
|
"({}) {}",
|
|
|
|
colors::gray(d.code.to_string()),
|
|
|
|
d.message.clone()
|
|
|
|
);
|
|
|
|
|
|
|
|
fmt_errors::format_stack(
|
|
|
|
true,
|
|
|
|
pretty_message,
|
|
|
|
Some(d.line_src.clone()),
|
|
|
|
Some(d.location.col as i64),
|
|
|
|
Some((d.location.col + d.snippet_length) as i64),
|
|
|
|
&[fmt_errors::format_location(
|
|
|
|
d.location.filename.clone(),
|
|
|
|
d.location.line as i64,
|
|
|
|
d.location.col as i64,
|
|
|
|
)],
|
|
|
|
0,
|
|
|
|
)
|
|
|
|
}
|