Uh oh!
There was an error while loading. Please reload this page.
Conversation
Uh oh!
There was an error while loading. Please reload this page.
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.
| Write-Host "" | ||
| $nsInput = Read-Host " Workspace name [$defaultNamespace]" | ||
| $rawName = if ($nsInput) { $nsInput } else { $defaultNamespace } | ||
| $TB_NAMESPACE = ConvertTo-WorkspaceName -Input_ $rawName |
There was a problem hiding this comment.
PowerShell variable scoping makes workspace name empty in summary
High Severity
$TB_NAMESPACE is assigned inside Install-ClientHelm without the $script: prefix, so it's local to that function. When Print-Summary later reads $TB_NAMESPACE, it gets $null because PowerShell scopes function variables locally by default. The summary's "Workspace" line and log entries referencing $TB_NAMESPACE will be blank. Other cross-function variables in this script correctly use $script: (e.g., $script:LOG_FILE, $script:NVIDIA_DRIVER_OK), so this appears to be an oversight.
Note
Medium Risk
Installer flow and user prompts change (new workspace/namespace sanitization, step sequencing, and redirected command output), which could affect existing automation and troubleshooting if regressions occur. No changes to core backend logic, but failures may now surface primarily via log files.
Overview
Refactors the Windows and macOS/Linux installers to present a new tracebloc client setup UX with a 4-step roadmap, consistent step headers, and clearer guidance for re-runs/reboots and Docker startup delays (including spinners/progress indicators).
Shifts most verbose command output (cluster creation, Helm install, cluster status) into timestamped log files while keeping the console output terse and status-driven, and updates success/error messaging to point users at logs.
Adds a new interactive workspace name prompt (used as the Helm release/namespace) with DNS-1123 sanitization, and refreshes GPU detection/setup messaging across platforms (e.g., Windows AMD unsupported, macOS CPU-only messaging, simplified GPU plugin verification output).
Written by Cursor Bugbot for commit db1cf23. This will update automatically on new commits. Configure here.