fix: Use uniquely named local variables (#13)

Co-authored-by: simon-id <[email protected]>

Author: timfish

src/instrumentation.rs

@@ -115,13 +115,13 @@ impl Instrumentation {
         ));
 
         body.stmts = vec![
-            quote!("const traced = $traced;" as Stmt, traced: Expr = traced_fn.into()),
+            quote!("const __apm$traced = $traced;" as Stmt, traced: Expr = traced_fn.into()),
             quote!(
-                "if (!$ch.hasSubscribers) return traced();" as Stmt,
+                "if (!$ch.hasSubscribers) return __apm$traced();" as Stmt,
                 ch = ch_ident
             ),
             quote!(
-                "return $trace(traced, { arguments, self: this } );" as Stmt,
+                "return $trace(__apm$traced, { arguments, self: this } );" as Stmt,
                 trace = trace_ident
             ),
         ];

tests/wasm/testdata/expected-cjs.js

@@ -28,11 +28,11 @@ module.exports = class Up {
         }
     }
     fetch() {
-        const traced = ()=>{
+        const __apm$traced = ()=>{
             console.log('fetch');
         };
-        if (!tr_ch_apm$up_fetch.hasSubscribers) return traced();
-        return tr_ch_apm$up_fetch.traceSync(traced, {
+        if (!tr_ch_apm$up_fetch.hasSubscribers) return __apm$traced();
+        return tr_ch_apm$up_fetch.traceSync(__apm$traced, {
             arguments,
             self: this
         });

tests/wasm/testdata/expected.mjs

@@ -28,11 +28,11 @@ export class Up {
         }
     }
     fetch() {
-        const traced = ()=>{
+        const __apm$traced = ()=>{
             console.log('fetch');
         };
-        if (!tr_ch_apm$up_fetch.hasSubscribers) return traced();
-        return tr_ch_apm$up_fetch.traceSync(traced, {
+        if (!tr_ch_apm$up_fetch.hasSubscribers) return __apm$traced();
+        return tr_ch_apm$up_fetch.traceSync(__apm$traced, {
             arguments,
             self: this
         });