fix: wrap bare errors in tool actions for telemetry classification

Test_RunMethodsNoBareErrors requires all action Run() methods to use
sentinel-wrapped errors (fmt.Errorf with %w) for proper telemetry
classification. Fixed bare fmt.Errorf calls in tool upgrade action.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: wbreza

cli/azd/cmd/tool.go

@@ -565,7 +565,7 @@ func (a *toolUpgradeAction) Run(ctx context.Context) (*actions.ActionResult, err
 						return uxlib.Error, r.Error
 					}
 					if !r.Success {
-						return uxlib.Warning, fmt.Errorf("upgrade did not succeed")
+						return uxlib.Warning, fmt.Errorf("upgrade did not succeed: %w", internal.ErrToolUpgradeFailed)
 					}
 					if r.InstalledVersion != "" {
 						setProgress(r.InstalledVersion)

cli/azd/internal/errors.go

@@ -116,3 +116,8 @@ var (
 var (
 	ErrMcpToolsLoadFailed = errors.New("failed to load MCP host tools")
 )
+
+// Tool command errors
+var (
+	ErrToolUpgradeFailed = errors.New("tool upgrade did not succeed")
+)