fix: do not invoke debug traps during completion funcs (#163)

Author: reubeno

.vscode/settings.json

@@ -11,5 +11,6 @@
     },
     "rust-analyzer.check.command": "clippy",
     "rewrap.wrappingColumn": 90,
-    "rust-analyzer.cargo.features": "all"
+    "rust-analyzer.cargo.features": "all",
+    "rust-analyzer.debug.openDebugPane": true
 }

brush-core/src/completion.rs

@@ -506,8 +506,14 @@ impl Spec {
             args.push(preceding_token);
         }
 
+        // TODO: Find a more appropriate interlock here. For now we use the existing
+        // handler depth count to suppress any debug traps.
+        shell.traps.handler_depth += 1;
+
         let result = shell.invoke_function(function_name, &args).await?;
 
+        shell.traps.handler_depth -= 1;
+
         tracing::debug!("[called completion func '{function_name}' => {result}]");
 
         // When the function returns the special value 124, then it's a request