fix wasm crash

Author: jsumners-nr

Cargo.lock

@@ -27,6 +27,7 @@ wasm-bindgen = { version = "0.2", optional = true }
 tsify = { version='0.5', features = ["js"], optional = true}
 # we need this to enable the js feature
 getrandom = { version = "*", features = ["wasm_js"], optional = true }
+log = "0.4.27"
 
 [dev-dependencies]
 assert_cmd = "2"

Cargo.toml

@@ -79,10 +79,11 @@ pub struct InstrumentationConfig {
     pub channel_name: String,
     pub module: ModuleMatcher,
     pub function_query: FunctionQuery,
+    #[tsify(optional)]
+    #[serde(default = "InstrumentationConfig::empty_callback_config")]
     pub callback_config: CallbackConfig,
 }
 
-#[allow(clippy::needless_return)]
 impl InstrumentationConfig {
     #[must_use]
     pub fn new(channel_name: &str, module: ModuleMatcher, function_query: FunctionQuery) -> Self {
@@ -149,4 +150,9 @@ impl InstrumentationConfig {
     pub fn matches(&self, module_name: &str, version: &str, file_path: &PathBuf) -> bool {
         self.module.matches(module_name, version, file_path)
     }
+
+    #[must_use]
+    pub fn empty_callback_config() -> CallbackConfig {
+        return CallbackConfig{ position: -1 }
+    }
 }

src/config.rs

@@ -395,8 +395,6 @@ pub fn get_script_start_index(script: &Script) -> usize {
 /// ```js
 /// return tr_ch_amp$myChannel.traceSync(__apm$traced, { arguments, self: this, moduleVersion: "1.0.0" })
 /// ```
-#[allow(clippy::bool_comparison)]
-#[allow(clippy::needless_return)]
 #[rustfmt::skip]
 fn construct_trace_statement(config: &InstrumentationConfig, channel_name: &str, mod_version: &str) -> Stmt {
     let mut ctx = "{ arguments, self: this }".to_string();

src/instrumentation.rs

@@ -13,6 +13,8 @@
 #![deny(clippy::complexity)]
 #![deny(clippy::correctness)]
 #![deny(clippy::unwrap_used)]
+#![allow(clippy::bool_comparison)]
+#![allow(clippy::needless_return)]
 
 /**
  * Unless explicitly stated otherwise all files in this repository are licensed under the Apache-2.0 License.