fix: fix napi error

Author: HerringtonDarkholme

Cargo.lock

@@ -87,12 +87,12 @@ test('test file count', async t => {
 test('show good error message for invalid arg', async t => {
   const sg = parse('console.log(123)')
   t.throws(() => sg.root().find({rule: {regex: '('}}), {
-    message: 'Rule contains invalid regex matcher.'
+    message: /Rule contains invalid regex matcher/
   })
   t.throws(() => sg.root().find({
     rule: { all: [{any: [{ kind: '33'}]}]}
   }), {
-    message: 'Rule contains invalid kind matcher.'
+    message: /Rule contains invalid kind matcher/
   })
 })
 

crates/napi/__test__/index.spec.ts

@@ -26,6 +26,7 @@ export interface NapiConfig {
   rule: any
   constraints?: any
   language?: FrontEndLanguage
+  utils?: any
 }
 export function parseFiles(paths: string[], callback: (err: null | Error, result: SgRoot) => void): Promise<number>
 export interface FindConfig {
@@ -35,6 +36,7 @@ export interface FindConfig {
 export class SgNode {
   range(): Range
   isLeaf(): boolean
+  isNamedLeaf(): boolean
   kind(): string
   text(): string
   matches(m: string): boolean

crates/napi/index.d.ts

@@ -63,4 +63,4 @@
     }
   },
   "packageManager": "[email protected]"
-}
+}

crates/napi/package.json

@@ -6,7 +6,7 @@ use ast_grep_core::pinned::{NodeData, PinnedNodeData};
 use ast_grep_core::{matcher::KindMatcher, AstGrep, NodeMatch, Pattern};
 use ignore::types::TypesBuilder;
 use ignore::{WalkBuilder, WalkState};
-use napi::anyhow::{anyhow, Context, Result as Ret};
+use napi::anyhow::{anyhow, Context, Error, Result as Ret};
 use napi::bindgen_prelude::*;
 use napi::threadsafe_function::{ErrorStrategy, ThreadsafeFunction, ThreadsafeFunctionCallMode};
 use napi::{JsNumber, Task};
@@ -196,9 +196,13 @@ fn parse_config(
     constraints: config.constraints.map(serde_json::from_value).transpose()?,
     utils: config.utils.map(serde_json::from_value).transpose()?,
   };
-  rule
-    .get_matcher(&Default::default())
-    .map_err(|e| napi::Error::new(napi::Status::InvalidArg, e.to_string()))
+  rule.get_matcher(&Default::default()).map_err(|e| {
+    let error = Error::from(e)
+      .chain()
+      .map(ToString::to_string)
+      .collect::<Vec<_>>();
+    napi::Error::new(napi::Status::InvalidArg, error.join("\n |->"))
+  })
 }
 
 /// tree traversal API