2021-01-11 12:13:41 -05:00
|
|
|
// Copyright 2018-2021 the Deno authors. All rights reserved. MIT license.
|
2020-12-07 05:46:39 -05:00
|
|
|
|
2021-02-04 13:53:02 -05:00
|
|
|
use super::text::LineIndex;
|
|
|
|
use super::tsc;
|
|
|
|
|
2020-12-07 05:46:39 -05:00
|
|
|
use crate::ast;
|
|
|
|
use crate::import_map::ImportMap;
|
|
|
|
use crate::media_type::MediaType;
|
|
|
|
use crate::module_graph::parse_deno_types;
|
|
|
|
use crate::module_graph::parse_ts_reference;
|
|
|
|
use crate::module_graph::TypeScriptReference;
|
|
|
|
use crate::tools::lint::create_linter;
|
|
|
|
|
2021-02-04 13:53:02 -05:00
|
|
|
use deno_core::error::custom_error;
|
2020-12-07 05:46:39 -05:00
|
|
|
use deno_core::error::AnyError;
|
2021-02-04 13:53:02 -05:00
|
|
|
use deno_core::futures::Future;
|
2021-01-31 22:30:41 -05:00
|
|
|
use deno_core::serde::Deserialize;
|
|
|
|
use deno_core::serde::Serialize;
|
2020-12-07 05:46:39 -05:00
|
|
|
use deno_core::ModuleSpecifier;
|
|
|
|
use deno_lint::rules;
|
2021-01-29 14:34:33 -05:00
|
|
|
use lspower::lsp;
|
|
|
|
use lspower::lsp::Position;
|
|
|
|
use lspower::lsp::Range;
|
2021-02-04 13:53:02 -05:00
|
|
|
use std::cmp::Ordering;
|
2020-12-07 05:46:39 -05:00
|
|
|
use std::collections::HashMap;
|
|
|
|
use std::rc::Rc;
|
|
|
|
|
2021-02-04 13:53:02 -05:00
|
|
|
lazy_static! {
|
|
|
|
/// Diagnostic error codes which actually are the same, and so when grouping
|
|
|
|
/// fixes we treat them the same.
|
|
|
|
static ref FIX_ALL_ERROR_CODES: HashMap<&'static str, &'static str> =
|
|
|
|
[("2339", "2339"), ("2345", "2339"),]
|
|
|
|
.iter()
|
|
|
|
.copied()
|
|
|
|
.collect();
|
|
|
|
|
|
|
|
/// Fixes which help determine if there is a preferred fix when there are
|
|
|
|
/// multiple fixes available.
|
|
|
|
static ref PREFERRED_FIXES: HashMap<&'static str, (u32, bool)> = [
|
|
|
|
("annotateWithTypeFromJSDoc", (1, false)),
|
|
|
|
("constructorForDerivedNeedSuperCall", (1, false)),
|
|
|
|
("extendsInterfaceBecomesImplements", (1, false)),
|
|
|
|
("awaitInSyncFunction", (1, false)),
|
|
|
|
("classIncorrectlyImplementsInterface", (3, false)),
|
|
|
|
("classDoesntImplementInheritedAbstractMember", (3, false)),
|
|
|
|
("unreachableCode", (1, false)),
|
|
|
|
("unusedIdentifier", (1, false)),
|
|
|
|
("forgottenThisPropertyAccess", (1, false)),
|
|
|
|
("spelling", (2, false)),
|
|
|
|
("addMissingAwait", (1, false)),
|
|
|
|
("fixImport", (0, true)),
|
|
|
|
]
|
|
|
|
.iter()
|
|
|
|
.copied()
|
|
|
|
.collect();
|
|
|
|
}
|
|
|
|
|
2020-12-07 05:46:39 -05:00
|
|
|
/// Category of self-generated diagnostic messages (those not coming from)
|
|
|
|
/// TypeScript.
|
|
|
|
pub enum Category {
|
|
|
|
/// A lint diagnostic, where the first element is the message.
|
|
|
|
Lint {
|
|
|
|
message: String,
|
|
|
|
code: String,
|
|
|
|
hint: Option<String>,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
|
|
|
|
/// A structure to hold a reference to a diagnostic message.
|
|
|
|
pub struct Reference {
|
|
|
|
category: Category,
|
|
|
|
range: Range,
|
|
|
|
}
|
|
|
|
|
|
|
|
fn as_lsp_range(range: &deno_lint::diagnostic::Range) -> Range {
|
|
|
|
Range {
|
|
|
|
start: Position {
|
|
|
|
line: (range.start.line - 1) as u32,
|
|
|
|
character: range.start.col as u32,
|
|
|
|
},
|
|
|
|
end: Position {
|
|
|
|
line: (range.end.line - 1) as u32,
|
|
|
|
character: range.end.col as u32,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
pub fn get_lint_references(
|
|
|
|
specifier: &ModuleSpecifier,
|
|
|
|
media_type: &MediaType,
|
|
|
|
source_code: &str,
|
|
|
|
) -> Result<Vec<Reference>, AnyError> {
|
|
|
|
let syntax = ast::get_syntax(media_type);
|
|
|
|
let lint_rules = rules::get_recommended_rules();
|
|
|
|
let mut linter = create_linter(syntax, lint_rules);
|
|
|
|
// TODO(@kitsonk) we should consider caching the swc source file versions for
|
|
|
|
// reuse by other processes
|
|
|
|
let (_, lint_diagnostics) =
|
|
|
|
linter.lint(specifier.to_string(), source_code.to_string())?;
|
|
|
|
|
|
|
|
Ok(
|
|
|
|
lint_diagnostics
|
|
|
|
.into_iter()
|
|
|
|
.map(|d| Reference {
|
|
|
|
category: Category::Lint {
|
|
|
|
message: d.message,
|
|
|
|
code: d.code,
|
|
|
|
hint: d.hint,
|
|
|
|
},
|
|
|
|
range: as_lsp_range(&d.range),
|
|
|
|
})
|
|
|
|
.collect(),
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
pub fn references_to_diagnostics(
|
|
|
|
references: Vec<Reference>,
|
2021-01-29 14:34:33 -05:00
|
|
|
) -> Vec<lsp::Diagnostic> {
|
2020-12-07 05:46:39 -05:00
|
|
|
references
|
|
|
|
.into_iter()
|
|
|
|
.map(|r| match r.category {
|
2021-01-29 14:34:33 -05:00
|
|
|
Category::Lint { message, code, .. } => lsp::Diagnostic {
|
2020-12-07 05:46:39 -05:00
|
|
|
range: r.range,
|
2021-01-29 14:34:33 -05:00
|
|
|
severity: Some(lsp::DiagnosticSeverity::Warning),
|
|
|
|
code: Some(lsp::NumberOrString::String(code)),
|
2020-12-07 05:46:39 -05:00
|
|
|
code_description: None,
|
|
|
|
source: Some("deno-lint".to_string()),
|
|
|
|
message,
|
|
|
|
related_information: None,
|
|
|
|
tags: None, // we should tag unused code
|
|
|
|
data: None,
|
|
|
|
},
|
|
|
|
})
|
|
|
|
.collect()
|
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Debug, Default, Clone, PartialEq, Eq)]
|
|
|
|
pub struct Dependency {
|
|
|
|
pub is_dynamic: bool,
|
2020-12-24 05:53:03 -05:00
|
|
|
pub maybe_code: Option<ResolvedDependency>,
|
|
|
|
pub maybe_code_specifier_range: Option<Range>,
|
|
|
|
pub maybe_type: Option<ResolvedDependency>,
|
2020-12-07 05:46:39 -05:00
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Debug, Clone, PartialEq, Eq)]
|
2020-12-24 05:53:03 -05:00
|
|
|
pub enum ResolvedDependency {
|
2020-12-07 05:46:39 -05:00
|
|
|
Resolved(ModuleSpecifier),
|
|
|
|
Err(String),
|
|
|
|
}
|
|
|
|
|
|
|
|
pub fn resolve_import(
|
|
|
|
specifier: &str,
|
|
|
|
referrer: &ModuleSpecifier,
|
2020-12-21 08:44:26 -05:00
|
|
|
maybe_import_map: &Option<ImportMap>,
|
2020-12-24 05:53:03 -05:00
|
|
|
) -> ResolvedDependency {
|
2020-12-07 05:46:39 -05:00
|
|
|
let maybe_mapped = if let Some(import_map) = maybe_import_map {
|
2020-12-21 08:44:26 -05:00
|
|
|
if let Ok(maybe_specifier) =
|
|
|
|
import_map.resolve(specifier, referrer.as_str())
|
2020-12-07 05:46:39 -05:00
|
|
|
{
|
|
|
|
maybe_specifier
|
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
None
|
|
|
|
};
|
|
|
|
let remapped = maybe_mapped.is_some();
|
|
|
|
let specifier = if let Some(remapped) = maybe_mapped {
|
|
|
|
remapped
|
|
|
|
} else {
|
|
|
|
match ModuleSpecifier::resolve_import(specifier, referrer.as_str()) {
|
|
|
|
Ok(resolved) => resolved,
|
2020-12-24 05:53:03 -05:00
|
|
|
Err(err) => return ResolvedDependency::Err(err.to_string()),
|
2020-12-07 05:46:39 -05:00
|
|
|
}
|
|
|
|
};
|
|
|
|
let referrer_scheme = referrer.as_url().scheme();
|
|
|
|
let specifier_scheme = specifier.as_url().scheme();
|
|
|
|
if referrer_scheme == "https" && specifier_scheme == "http" {
|
2020-12-24 05:53:03 -05:00
|
|
|
return ResolvedDependency::Err(
|
2020-12-07 05:46:39 -05:00
|
|
|
"Modules imported via https are not allowed to import http modules."
|
|
|
|
.to_string(),
|
|
|
|
);
|
|
|
|
}
|
|
|
|
if (referrer_scheme == "https" || referrer_scheme == "http")
|
|
|
|
&& !(specifier_scheme == "https" || specifier_scheme == "http")
|
|
|
|
&& !remapped
|
|
|
|
{
|
2020-12-24 05:53:03 -05:00
|
|
|
return ResolvedDependency::Err("Remote modules are not allowed to import local modules. Consider using a dynamic import instead.".to_string());
|
2020-12-07 05:46:39 -05:00
|
|
|
}
|
|
|
|
|
2020-12-24 05:53:03 -05:00
|
|
|
ResolvedDependency::Resolved(specifier)
|
2020-12-07 05:46:39 -05:00
|
|
|
}
|
|
|
|
|
|
|
|
// TODO(@kitsonk) a lot of this logic is duplicated in module_graph.rs in
|
|
|
|
// Module::parse() and should be refactored out to a common function.
|
|
|
|
pub fn analyze_dependencies(
|
|
|
|
specifier: &ModuleSpecifier,
|
|
|
|
source: &str,
|
|
|
|
media_type: &MediaType,
|
2020-12-21 08:44:26 -05:00
|
|
|
maybe_import_map: &Option<ImportMap>,
|
2020-12-24 05:53:03 -05:00
|
|
|
) -> Option<(HashMap<String, Dependency>, Option<ResolvedDependency>)> {
|
2020-12-07 05:46:39 -05:00
|
|
|
let specifier_str = specifier.to_string();
|
|
|
|
let source_map = Rc::new(swc_common::SourceMap::default());
|
|
|
|
let mut maybe_type = None;
|
|
|
|
if let Ok(parsed_module) =
|
|
|
|
ast::parse_with_source_map(&specifier_str, source, &media_type, source_map)
|
|
|
|
{
|
|
|
|
let mut dependencies = HashMap::<String, Dependency>::new();
|
|
|
|
|
|
|
|
// Parse leading comments for supported triple slash references.
|
|
|
|
for comment in parsed_module.get_leading_comments().iter() {
|
|
|
|
if let Some(ts_reference) = parse_ts_reference(&comment.text) {
|
|
|
|
match ts_reference {
|
|
|
|
TypeScriptReference::Path(import) => {
|
|
|
|
let dep = dependencies.entry(import.clone()).or_default();
|
|
|
|
let resolved_import =
|
2020-12-21 08:44:26 -05:00
|
|
|
resolve_import(&import, specifier, maybe_import_map);
|
2020-12-07 05:46:39 -05:00
|
|
|
dep.maybe_code = Some(resolved_import);
|
|
|
|
}
|
|
|
|
TypeScriptReference::Types(import) => {
|
|
|
|
let resolved_import =
|
2020-12-21 08:44:26 -05:00
|
|
|
resolve_import(&import, specifier, maybe_import_map);
|
2020-12-07 05:46:39 -05:00
|
|
|
if media_type == &MediaType::JavaScript
|
|
|
|
|| media_type == &MediaType::JSX
|
|
|
|
{
|
|
|
|
maybe_type = Some(resolved_import)
|
|
|
|
} else {
|
|
|
|
let dep = dependencies.entry(import).or_default();
|
|
|
|
dep.maybe_type = Some(resolved_import);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// Parse ES and type only imports
|
|
|
|
let descriptors = parsed_module.analyze_dependencies();
|
|
|
|
for desc in descriptors.into_iter().filter(|desc| {
|
|
|
|
desc.kind != swc_ecmascript::dep_graph::DependencyKind::Require
|
|
|
|
}) {
|
|
|
|
let resolved_import =
|
2020-12-21 08:44:26 -05:00
|
|
|
resolve_import(&desc.specifier, specifier, maybe_import_map);
|
2020-12-07 05:46:39 -05:00
|
|
|
|
|
|
|
// Check for `@deno-types` pragmas that effect the import
|
|
|
|
let maybe_resolved_type_import =
|
|
|
|
if let Some(comment) = desc.leading_comments.last() {
|
|
|
|
if let Some(deno_types) = parse_deno_types(&comment.text).as_ref() {
|
2020-12-21 08:44:26 -05:00
|
|
|
Some(resolve_import(deno_types, specifier, maybe_import_map))
|
2020-12-07 05:46:39 -05:00
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
None
|
|
|
|
};
|
|
|
|
|
|
|
|
let dep = dependencies.entry(desc.specifier.to_string()).or_default();
|
|
|
|
dep.is_dynamic = desc.is_dynamic;
|
|
|
|
match desc.kind {
|
|
|
|
swc_ecmascript::dep_graph::DependencyKind::ExportType
|
|
|
|
| swc_ecmascript::dep_graph::DependencyKind::ImportType => {
|
|
|
|
dep.maybe_type = Some(resolved_import)
|
|
|
|
}
|
2020-12-24 05:53:03 -05:00
|
|
|
_ => {
|
|
|
|
dep.maybe_code_specifier_range = Some(Range {
|
|
|
|
start: Position {
|
|
|
|
line: (desc.specifier_line - 1) as u32,
|
|
|
|
character: desc.specifier_col as u32,
|
|
|
|
},
|
|
|
|
end: Position {
|
|
|
|
line: (desc.specifier_line - 1) as u32,
|
|
|
|
character: (desc.specifier_col
|
|
|
|
+ desc.specifier.chars().count()
|
|
|
|
+ 2) as u32,
|
|
|
|
},
|
|
|
|
});
|
|
|
|
dep.maybe_code = Some(resolved_import);
|
|
|
|
}
|
2020-12-07 05:46:39 -05:00
|
|
|
}
|
|
|
|
if maybe_resolved_type_import.is_some() && dep.maybe_type.is_none() {
|
|
|
|
dep.maybe_type = maybe_resolved_type_import;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
Some((dependencies, maybe_type))
|
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-01-31 22:30:41 -05:00
|
|
|
#[derive(Debug, Deserialize, Serialize)]
|
|
|
|
pub enum CodeLensSource {
|
|
|
|
#[serde(rename = "references")]
|
|
|
|
References,
|
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Debug, Deserialize)]
|
|
|
|
#[serde(rename_all = "camelCase")]
|
|
|
|
pub struct CodeLensData {
|
|
|
|
pub source: CodeLensSource,
|
|
|
|
pub specifier: ModuleSpecifier,
|
|
|
|
}
|
|
|
|
|
2021-02-04 13:53:02 -05:00
|
|
|
fn code_as_string(code: &Option<lsp::NumberOrString>) -> String {
|
|
|
|
match code {
|
|
|
|
Some(lsp::NumberOrString::String(str)) => str.clone(),
|
|
|
|
Some(lsp::NumberOrString::Number(num)) => num.to_string(),
|
|
|
|
_ => "".to_string(),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Determines if two TypeScript diagnostic codes are effectively equivalent.
|
|
|
|
fn is_equivalent_code(
|
|
|
|
a: &Option<lsp::NumberOrString>,
|
|
|
|
b: &Option<lsp::NumberOrString>,
|
|
|
|
) -> bool {
|
|
|
|
let a_code = code_as_string(a);
|
|
|
|
let b_code = code_as_string(b);
|
|
|
|
FIX_ALL_ERROR_CODES.get(a_code.as_str())
|
|
|
|
== FIX_ALL_ERROR_CODES.get(b_code.as_str())
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Return a boolean flag to indicate if the specified action is the preferred
|
|
|
|
/// action for a given set of actions.
|
|
|
|
fn is_preferred(
|
|
|
|
action: &tsc::CodeFixAction,
|
|
|
|
actions: &[(lsp::CodeAction, tsc::CodeFixAction)],
|
|
|
|
fix_priority: u32,
|
|
|
|
only_one: bool,
|
|
|
|
) -> bool {
|
|
|
|
actions.iter().all(|(_, a)| {
|
|
|
|
if action == a {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
if a.fix_id.is_some() {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
if let Some((other_fix_priority, _)) =
|
|
|
|
PREFERRED_FIXES.get(a.fix_name.as_str())
|
|
|
|
{
|
|
|
|
match other_fix_priority.cmp(&fix_priority) {
|
|
|
|
Ordering::Less => return true,
|
|
|
|
Ordering::Greater => return false,
|
|
|
|
Ordering::Equal => (),
|
|
|
|
}
|
|
|
|
if only_one && action.fix_name == a.fix_name {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
true
|
|
|
|
})
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Convert changes returned from a TypeScript quick fix action into edits
|
|
|
|
/// for an LSP CodeAction.
|
|
|
|
async fn ts_changes_to_edit<F, Fut, V>(
|
|
|
|
changes: &[tsc::FileTextChanges],
|
|
|
|
index_provider: &F,
|
|
|
|
version_provider: &V,
|
|
|
|
) -> Result<Option<lsp::WorkspaceEdit>, AnyError>
|
|
|
|
where
|
|
|
|
F: Fn(ModuleSpecifier) -> Fut + Clone,
|
|
|
|
Fut: Future<Output = Result<LineIndex, AnyError>>,
|
|
|
|
V: Fn(ModuleSpecifier) -> Option<i32>,
|
|
|
|
{
|
|
|
|
let mut text_document_edits = Vec::new();
|
|
|
|
for change in changes {
|
|
|
|
let text_document_edit = change
|
|
|
|
.to_text_document_edit(index_provider, version_provider)
|
|
|
|
.await?;
|
|
|
|
text_document_edits.push(text_document_edit);
|
|
|
|
}
|
|
|
|
Ok(Some(lsp::WorkspaceEdit {
|
|
|
|
changes: None,
|
|
|
|
document_changes: Some(lsp::DocumentChanges::Edits(text_document_edits)),
|
|
|
|
change_annotations: None,
|
|
|
|
}))
|
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Debug, Default)]
|
|
|
|
pub struct CodeActionCollection {
|
|
|
|
actions: Vec<(lsp::CodeAction, tsc::CodeFixAction)>,
|
|
|
|
fix_all_actions: HashMap<String, (lsp::CodeAction, tsc::CodeFixAction)>,
|
|
|
|
}
|
|
|
|
|
|
|
|
impl CodeActionCollection {
|
|
|
|
/// Add a TypeScript code fix action to the code actions collection.
|
|
|
|
pub async fn add_ts_fix_action<F, Fut, V>(
|
|
|
|
&mut self,
|
|
|
|
action: &tsc::CodeFixAction,
|
|
|
|
diagnostic: &lsp::Diagnostic,
|
|
|
|
index_provider: &F,
|
|
|
|
version_provider: &V,
|
|
|
|
) -> Result<(), AnyError>
|
|
|
|
where
|
|
|
|
F: Fn(ModuleSpecifier) -> Fut + Clone,
|
|
|
|
Fut: Future<Output = Result<LineIndex, AnyError>>,
|
|
|
|
V: Fn(ModuleSpecifier) -> Option<i32>,
|
|
|
|
{
|
|
|
|
if action.commands.is_some() {
|
|
|
|
// In theory, tsc can return actions that require "commands" to be applied
|
|
|
|
// back into TypeScript. Currently there is only one command, `install
|
|
|
|
// package` but Deno doesn't support that. The problem is that the
|
|
|
|
// `.applyCodeActionCommand()` returns a promise, and with the current way
|
|
|
|
// we wrap tsc, we can't handle the asynchronous response, so it is
|
|
|
|
// actually easier to return errors if we ever encounter one of these,
|
|
|
|
// which we really wouldn't expect from the Deno lsp.
|
|
|
|
return Err(custom_error(
|
|
|
|
"UnsupportedFix",
|
|
|
|
"The action returned from TypeScript is unsupported.",
|
|
|
|
));
|
|
|
|
}
|
|
|
|
let edit =
|
|
|
|
ts_changes_to_edit(&action.changes, index_provider, version_provider)
|
|
|
|
.await?;
|
|
|
|
let code_action = lsp::CodeAction {
|
|
|
|
title: action.description.clone(),
|
|
|
|
kind: Some(lsp::CodeActionKind::QUICKFIX),
|
|
|
|
diagnostics: Some(vec![diagnostic.clone()]),
|
|
|
|
edit,
|
|
|
|
command: None,
|
|
|
|
is_preferred: None,
|
|
|
|
disabled: None,
|
|
|
|
data: None,
|
|
|
|
};
|
|
|
|
self.actions.retain(|(c, a)| {
|
|
|
|
!(action.fix_name == a.fix_name && code_action.edit == c.edit)
|
|
|
|
});
|
|
|
|
self.actions.push((code_action, action.clone()));
|
|
|
|
|
|
|
|
if let Some(fix_id) = &action.fix_id {
|
|
|
|
if let Some((existing_fix_all, existing_action)) =
|
|
|
|
self.fix_all_actions.get(fix_id)
|
|
|
|
{
|
|
|
|
self.actions.retain(|(c, _)| c != existing_fix_all);
|
|
|
|
self
|
|
|
|
.actions
|
|
|
|
.push((existing_fix_all.clone(), existing_action.clone()));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
Ok(())
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Add a TypeScript action to the actions as a "fix all" action, where it
|
|
|
|
/// will fix all occurrences of the diagnostic in the file.
|
|
|
|
pub async fn add_ts_fix_all_action<F, Fut, V>(
|
|
|
|
&mut self,
|
|
|
|
action: &tsc::CodeFixAction,
|
|
|
|
diagnostic: &lsp::Diagnostic,
|
|
|
|
combined_code_actions: &tsc::CombinedCodeActions,
|
|
|
|
index_provider: &F,
|
|
|
|
version_provider: &V,
|
|
|
|
) -> Result<(), AnyError>
|
|
|
|
where
|
|
|
|
F: Fn(ModuleSpecifier) -> Fut + Clone,
|
|
|
|
Fut: Future<Output = Result<LineIndex, AnyError>>,
|
|
|
|
V: Fn(ModuleSpecifier) -> Option<i32>,
|
|
|
|
{
|
|
|
|
if combined_code_actions.commands.is_some() {
|
|
|
|
return Err(custom_error(
|
|
|
|
"UnsupportedFix",
|
|
|
|
"The action returned from TypeScript is unsupported.",
|
|
|
|
));
|
|
|
|
}
|
|
|
|
let edit = ts_changes_to_edit(
|
|
|
|
&combined_code_actions.changes,
|
|
|
|
index_provider,
|
|
|
|
version_provider,
|
|
|
|
)
|
|
|
|
.await?;
|
|
|
|
let title = if let Some(description) = &action.fix_all_description {
|
|
|
|
description.clone()
|
|
|
|
} else {
|
|
|
|
format!("{} (Fix all in file)", action.description)
|
|
|
|
};
|
|
|
|
|
|
|
|
let code_action = lsp::CodeAction {
|
|
|
|
title,
|
|
|
|
kind: Some(lsp::CodeActionKind::QUICKFIX),
|
|
|
|
diagnostics: Some(vec![diagnostic.clone()]),
|
|
|
|
edit,
|
|
|
|
command: None,
|
|
|
|
is_preferred: None,
|
|
|
|
disabled: None,
|
|
|
|
data: None,
|
|
|
|
};
|
|
|
|
if let Some((existing, _)) =
|
|
|
|
self.fix_all_actions.get(&action.fix_id.clone().unwrap())
|
|
|
|
{
|
|
|
|
self.actions.retain(|(c, _)| c != existing);
|
|
|
|
}
|
|
|
|
self.actions.push((code_action.clone(), action.clone()));
|
|
|
|
self.fix_all_actions.insert(
|
|
|
|
action.fix_id.clone().unwrap(),
|
|
|
|
(code_action, action.clone()),
|
|
|
|
);
|
|
|
|
Ok(())
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Move out the code actions and return them as a `CodeActionResponse`.
|
|
|
|
pub fn get_response(self) -> lsp::CodeActionResponse {
|
|
|
|
self
|
|
|
|
.actions
|
|
|
|
.into_iter()
|
|
|
|
.map(|(c, _)| lsp::CodeActionOrCommand::CodeAction(c))
|
|
|
|
.collect()
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Determine if a action can be converted into a "fix all" action.
|
|
|
|
pub fn is_fix_all_action(
|
|
|
|
&self,
|
|
|
|
action: &tsc::CodeFixAction,
|
|
|
|
diagnostic: &lsp::Diagnostic,
|
|
|
|
file_diagnostics: &[&lsp::Diagnostic],
|
|
|
|
) -> bool {
|
|
|
|
// If the action does not have a fix id (indicating it can be "bundled up")
|
|
|
|
// or if the collection already contains a "bundled" action return false
|
|
|
|
if action.fix_id.is_none()
|
|
|
|
|| self
|
|
|
|
.fix_all_actions
|
|
|
|
.contains_key(&action.fix_id.clone().unwrap())
|
|
|
|
{
|
|
|
|
false
|
|
|
|
} else {
|
|
|
|
// else iterate over the diagnostic in the file and see if there are any
|
|
|
|
// other diagnostics that could be bundled together in a "fix all" code
|
|
|
|
// action
|
|
|
|
file_diagnostics.iter().any(|d| {
|
|
|
|
if d == &diagnostic || d.code.is_none() || diagnostic.code.is_none() {
|
|
|
|
false
|
|
|
|
} else {
|
|
|
|
d.code == diagnostic.code
|
|
|
|
|| is_equivalent_code(&d.code, &diagnostic.code)
|
|
|
|
}
|
|
|
|
})
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Set the `.is_preferred` flag on code actions, this should be only executed
|
|
|
|
/// when all actions are added to the collection.
|
|
|
|
pub fn set_preferred_fixes(&mut self) {
|
|
|
|
let actions = self.actions.clone();
|
|
|
|
for (code_action, action) in self.actions.iter_mut() {
|
|
|
|
if action.fix_id.is_some() {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if let Some((fix_priority, only_one)) =
|
|
|
|
PREFERRED_FIXES.get(action.fix_name.as_str())
|
|
|
|
{
|
|
|
|
code_action.is_preferred =
|
|
|
|
Some(is_preferred(action, &actions, *fix_priority, *only_one));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-12-07 05:46:39 -05:00
|
|
|
#[cfg(test)]
|
|
|
|
mod tests {
|
|
|
|
use super::*;
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_as_lsp_range() {
|
|
|
|
let fixture = deno_lint::diagnostic::Range {
|
|
|
|
start: deno_lint::diagnostic::Position {
|
|
|
|
line: 1,
|
|
|
|
col: 2,
|
|
|
|
byte_pos: 23,
|
|
|
|
},
|
|
|
|
end: deno_lint::diagnostic::Position {
|
|
|
|
line: 2,
|
|
|
|
col: 0,
|
|
|
|
byte_pos: 33,
|
|
|
|
},
|
|
|
|
};
|
|
|
|
let actual = as_lsp_range(&fixture);
|
|
|
|
assert_eq!(
|
|
|
|
actual,
|
2021-01-29 14:34:33 -05:00
|
|
|
lsp::Range {
|
|
|
|
start: lsp::Position {
|
2020-12-07 05:46:39 -05:00
|
|
|
line: 0,
|
|
|
|
character: 2,
|
|
|
|
},
|
2021-01-29 14:34:33 -05:00
|
|
|
end: lsp::Position {
|
2020-12-07 05:46:39 -05:00
|
|
|
line: 1,
|
|
|
|
character: 0,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_analyze_dependencies() {
|
|
|
|
let specifier =
|
|
|
|
ModuleSpecifier::resolve_url("file:///a.ts").expect("bad specifier");
|
|
|
|
let source = r#"import {
|
|
|
|
Application,
|
|
|
|
Context,
|
|
|
|
Router,
|
|
|
|
Status,
|
|
|
|
} from "https://deno.land/x/oak@v6.3.2/mod.ts";
|
2021-01-29 14:34:33 -05:00
|
|
|
|
2020-12-07 05:46:39 -05:00
|
|
|
// @deno-types="https://deno.land/x/types/react/index.d.ts";
|
|
|
|
import * as React from "https://cdn.skypack.dev/react";
|
|
|
|
"#;
|
|
|
|
let actual =
|
2020-12-21 08:44:26 -05:00
|
|
|
analyze_dependencies(&specifier, source, &MediaType::TypeScript, &None);
|
2020-12-07 05:46:39 -05:00
|
|
|
assert!(actual.is_some());
|
|
|
|
let (actual, maybe_type) = actual.unwrap();
|
|
|
|
assert!(maybe_type.is_none());
|
|
|
|
assert_eq!(actual.len(), 2);
|
|
|
|
assert_eq!(
|
|
|
|
actual.get("https://cdn.skypack.dev/react").cloned(),
|
|
|
|
Some(Dependency {
|
|
|
|
is_dynamic: false,
|
2020-12-24 05:53:03 -05:00
|
|
|
maybe_code: Some(ResolvedDependency::Resolved(
|
2020-12-07 05:46:39 -05:00
|
|
|
ModuleSpecifier::resolve_url("https://cdn.skypack.dev/react")
|
|
|
|
.unwrap()
|
|
|
|
)),
|
2020-12-24 05:53:03 -05:00
|
|
|
maybe_type: Some(ResolvedDependency::Resolved(
|
2020-12-07 05:46:39 -05:00
|
|
|
ModuleSpecifier::resolve_url(
|
|
|
|
"https://deno.land/x/types/react/index.d.ts"
|
|
|
|
)
|
|
|
|
.unwrap()
|
|
|
|
)),
|
2020-12-24 05:53:03 -05:00
|
|
|
maybe_code_specifier_range: Some(Range {
|
|
|
|
start: Position {
|
|
|
|
line: 8,
|
|
|
|
character: 27,
|
|
|
|
},
|
|
|
|
end: Position {
|
|
|
|
line: 8,
|
|
|
|
character: 58,
|
|
|
|
}
|
|
|
|
}),
|
2020-12-07 05:46:39 -05:00
|
|
|
})
|
|
|
|
);
|
|
|
|
assert_eq!(
|
|
|
|
actual.get("https://deno.land/x/oak@v6.3.2/mod.ts").cloned(),
|
|
|
|
Some(Dependency {
|
|
|
|
is_dynamic: false,
|
2020-12-24 05:53:03 -05:00
|
|
|
maybe_code: Some(ResolvedDependency::Resolved(
|
2020-12-07 05:46:39 -05:00
|
|
|
ModuleSpecifier::resolve_url("https://deno.land/x/oak@v6.3.2/mod.ts")
|
|
|
|
.unwrap()
|
|
|
|
)),
|
|
|
|
maybe_type: None,
|
2020-12-24 05:53:03 -05:00
|
|
|
maybe_code_specifier_range: Some(Range {
|
|
|
|
start: Position {
|
|
|
|
line: 5,
|
|
|
|
character: 11,
|
|
|
|
},
|
|
|
|
end: Position {
|
|
|
|
line: 5,
|
|
|
|
character: 50,
|
|
|
|
}
|
|
|
|
}),
|
2020-12-07 05:46:39 -05:00
|
|
|
})
|
|
|
|
);
|
|
|
|
}
|
|
|
|
}
|