Conversation
468ca0d to
b8cc450
Compare
PR SummaryMedium Risk Overview Adds a new Written by Cursor Bugbot for commit 6c08cfd. This will update automatically on new commits. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| # Default to yes if user just presses enter | ||
| read -r answer </dev/tty | ||
| case $answer in | ||
| "n" | "N" | "no" | "No" | "NO") |
There was a problem hiding this comment.
Shell script missing case pattern for "nO" input
Low Severity
The case statement handling the telemetry prompt response explicitly enumerates case variations of "no" ("n" | "N" | "no" | "No" | "NO") but is missing "nO". If a user types nO, it falls through to the default * case and enables telemetry, contrary to the user's intent to decline. The Rust implementation handles this correctly by converting input to lowercase before matching.


Add telemetry opt-in for fuelup
This PR implements user consent for telemetry collection across all Fuel toolchain commands. Users are now prompted once to opt-in or opt-out of anonymous usage data collection.
What this enables
fuelup,forc,forc-fmt, etc.)fuelup telemetry enable/disable/statuscommands to change preference anytimeFUELUP_NO_TELEMETRYenvironment variable is set automatically based on user's choice before executing toolchain binariesUser experience
First-time users see:
After opting in, telemetry events from
forcand other tools are sent to InfluxDB (via the fuel-telemetry library). If opted out,FUELUP_NO_TELEMETRY=1is set, disabling all telemetry.