fix: remove bb path override in cli-wallet (#11280)

- Remove config to customize bb path
- Update docs to specify required envs when profiling

closes AztecProtocol/dev-rel#471

aztec-up/bin/aztec-wallet

@@ -3,20 +3,9 @@ set -euo pipefail
 
 export SKIP_PORT_ASSIGNMENT=1
 export WALLET_DATA_DIRECTORY=$(dirname $0)/wallet-data
-export BB_WORKING_DIRECTORY=$(dirname $0)/bb-workdir
-
-if [ -z "${BB_BINARY_PATH:-}" ]; then
-    export BB_BINARY_PATH=$HOME/.bb/bb
-fi
-
-if [ ! -f $BB_BINARY_PATH ]; then
-    echo "bb binary does not exist at $BB_BINARY_PATH. Set BB_BINARY_PATH environment variable to the path of the bb binary."
-    exit 1
-fi
 
 export ENV_VARS_TO_INJECT="WALLET_DATA_DIRECTORY SSH_AUTH_SOCK PXE_PROVER_ENABLED BB_BINARY_PATH BB_WORKING_DIRECTORY"
 
-mkdir -p $BB_WORKING_DIRECTORY
 mkdir -p $WALLET_DATA_DIRECTORY
 
 $(dirname $0)/.aztec-run aztecprotocol/cli-wallet $@

docs/docs/guides/developer_guides/smart_contracts/profiling_transactions.md

@@ -46,8 +46,10 @@ aztec-wallet send transfer -ca token --args accounts:owner 40 -f user
 
 Instead of sending the above transaction, you can simulate it by running the `simulate` command with the same parameters, and then add a `--profile` flag to profile the gate count of each private function in the transaction.
 
+For the time being, you also need to set `BB_BINARY_PATH` and `BB_WORKING_DIRECTORY` environment variables with the below values.
+
 ```bash
-aztec-wallet simulate --profile transfer -ca token --args accounts:owner 40 -f user
+BB_BINARY_PATH=/usr/src/barretenberg/cpp/build/bin/bb BB_WORKING_DIRECTORY=~/.bb/work-dir aztec-wallet simulate --profile transfer -ca token --args accounts:owner 40 -f user
 ```
 
 This will print the following results after some time: