perf: update deno_doc (#24700)

Ref denoland/deno_doc#616
denoland · Jul 26, 2024 · f7749a9 · f7749a9
1 parent 8924b02
commit f7749a9
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 10 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/cli/Cargo.toml b/cli/Cargo.toml
@@ -67,7 +67,7 @@ deno_ast = { workspace = true, features = ["bundler", "cjs", "codegen", "proposa
 deno_cache_dir = { workspace = true }
 deno_config = { version = "=0.26.1", features = ["workspace", "sync"] }
 deno_core = { workspace = true, features = ["include_js_files_for_snapshotting"] }
-deno_doc = { version = "=0.142.0", features = ["html", "syntect"] }
+deno_doc = { version = "0.144.0", features = ["html", "syntect"] }
 deno_emit = "=0.43.1"
 deno_graph = { version = "=0.80.1", features = ["tokio_executor"] }
 deno_lint = { version = "=0.60.1", features = ["docs"] }

diff --git a/cli/tools/doc.rs b/cli/tools/doc.rs
@@ -178,10 +178,10 @@ pub async fn doc(
           .into_iter()
           .map(|node| deno_doc::html::DocNodeWithContext {
             origin: short_path.clone(),
-            ns_qualifiers: Rc::new(vec![]),
+            ns_qualifiers: Rc::new([]),
             kind_with_drilldown:
-              deno_doc::html::DocNodeKindWithDrilldown::Other(node.kind),
-            inner: std::sync::Arc::new(node),
+              deno_doc::html::DocNodeKindWithDrilldown::Other(node.kind()),
+            inner: Rc::new(node),
             drilldown_parent_kind: None,
             parent: None,
           })
@@ -300,7 +300,7 @@ impl deno_doc::html::HrefResolver for DocResolver {
   }
 
   fn resolve_source(&self, location: &deno_doc::Location) -> Option<String> {
-    Some(location.filename.clone())
+    Some(location.filename.to_string())
   }
 }
 
@@ -492,9 +492,9 @@ fn print_docs_to_stdout(
   doc_flags: DocFlags,
   mut doc_nodes: Vec<deno_doc::DocNode>,
 ) -> Result<(), AnyError> {
-  doc_nodes.retain(|doc_node| doc_node.kind != doc::DocNodeKind::Import);
+  doc_nodes.retain(|doc_node| doc_node.kind() != doc::DocNodeKind::Import);
   let details = if let Some(filter) = doc_flags.filter {
-    let nodes = doc::find_nodes_by_name_recursively(doc_nodes, filter.clone());
+    let nodes = doc::find_nodes_by_name_recursively(doc_nodes, &filter);
     if nodes.is_empty() {
       bail!("Node {} was not found!", filter);
     }

diff --git a/tests/testdata/doc/lint_success_json.out b/tests/testdata/doc/lint_success_json.out
@@ -1,6 +1,5 @@
 [
   {
-    "kind": "class",
     "name": "Test",
     "isDefault": false,
     "location": {
@@ -13,6 +12,7 @@
     "jsDoc": {
       "doc": "My test class."
     },
+    "kind": "class",
     "classDef": {
       "isAbstract": false,
       "constructors": [],