UX: implement new design of azd tool (install/upgrade/list/check) (#9045)

* use agent instead of host, improve multi selector ux

* add tip for grey and multi selector no filter for less than 6 options

* add spinner

* fix spinner bar and host detection

* fix host bug

* update ux message

* azd tool upgrade

* azd tool list, check, upgrade --all

* update azd tool upgrade ux

* add more test coverage

* clean up UX for install

* go fix

* use display name

* make availability check case-insensitive

* default to --all if no prompt or no tty

* only use version comparison for non-skill tools to prevent UX bug

* add mutex

* make sure command is not empty

* make UpdateAvailable  to be true for any agent skill updated

* add version on upgrade to non skill tools

* update comment

* fix the wrapping on narrower terminal widths

* fix ux bug

* fix ux bug

* update ux to include number of skills installed

* address the feedback

* remove all behavior for uninstall/upgrade/install if not specific in no tty

* revert ux for number of skills, need more discussion with designer

* fix bug

* address feedback

* use short name for agent

* address feedback

* add changelog

* rename to agent

* address feedback

* address Kristen's feedback

* use upgrade instead of update, address Hiyo feedback

* update check for multi selector, have stout under spinner grey section

* address all feedback

* address another round of feedback

* address feedback, update comment

Author: hemarina

cli/azd/CHANGELOG.md

@@ -8,6 +8,8 @@
 
 ### Breaking Changes
 
+ - [[#9045]](https://github.com/Azure/azure-dev/pull/9045) The `--host` skill flag on `azd tool install`, `azd tool upgrade`, and `azd tool uninstall` has been renamed to `--agent`. Installed skills in `azd tool list --output json` and `azd tool check --output json` now expand into one row per agent and include the `agent` field. Update scripts and JSON consumers accordingly.
+
 ### Bugs Fixed
 
 ### Other Changes

cli/azd/cmd/extension.go

@@ -507,7 +507,7 @@ func (a *extensionListAction) Run(ctx context.Context) (*actions.ActionResult, e
 // Status indicator constants for extension list display.
 const (
 	statusUpToDate   = "Up to date"
-	statusUpdate     = "Update available"
+	statusUpgrade    = "Upgrade available"
 	statusIncompat   = "Incompatible"
 	statusNotInstall = "Not installed"
 )
@@ -518,7 +518,7 @@ func extensionStatus(installed, updateAvailable, incompatible bool) string {
 	case incompatible:
 		return statusIncompat
 	case updateAvailable:
-		return statusUpdate
+		return statusUpgrade
 	case installed:
 		return statusUpToDate
 	default:
@@ -531,7 +531,7 @@ func extensionStatusColor(s string) string {
 	switch s {
 	case statusUpToDate:
 		return output.WithSuccessFormat(s)
-	case statusUpdate:
+	case statusUpgrade:
 		return output.WithWarningFormat(s)
 	case statusIncompat:
 		return output.WithErrorFormat(s)

cli/azd/cmd/extension_test.go

@@ -659,7 +659,7 @@ func TestExtensionStatus(t *testing.T) {
 	}{
 		{"not installed", false, false, false, statusNotInstall},
 		{"up to date", true, false, false, statusUpToDate},
-		{"update available", true, true, false, statusUpdate},
+		{"update available", true, true, false, statusUpgrade},
 		{"incompatible", true, false, true, statusIncompat},
 	}
 	for _, tt := range tests {
@@ -679,7 +679,7 @@ func TestExtensionStatusColor(t *testing.T) {
 
 	// Verify no panics and non-empty colored output for each status value.
 	for _, s := range []string{
-		statusUpToDate, statusUpdate, statusIncompat, statusNotInstall,
+		statusUpToDate, statusUpgrade, statusIncompat, statusNotInstall,
 	} {
 		result := extensionStatusColor(s)
 		assert.NotEmpty(t, result, "color function should return non-empty for %q", s)

cli/azd/cmd/testdata/TestFigSpec.ts

@@ -6466,12 +6466,22 @@ const completionSpec: Fig.Spec = {
 			subcommands: [
 				{
 					name: ['check'],
-					description: 'Check for tool updates.',
+					description: 'Check for tool upgrades.',
 				},
 				{
 					name: ['install'],
 					description: 'Install specified tools.',
 					options: [
+						{
+							name: ['--agent'],
+							description: 'Install the skill for the specified agent(s): copilot, claude. Use --agent all for every detected agent (skill tools only)',
+							isRepeatable: true,
+							args: [
+								{
+									name: 'agent',
+								},
+							],
+						},
 						{
 							name: ['--all'],
 							description: 'Install all recommended tools',
@@ -6480,16 +6490,6 @@ const completionSpec: Fig.Spec = {
 							name: ['--dry-run'],
 							description: 'Preview what would be installed without making changes',
 						},
-						{
-							name: ['--host'],
-							description: 'Install the skill for the specified agent host(s): copilot, claude. Use --host all for every detected host (skill tools only)',
-							isRepeatable: true,
-							args: [
-								{
-									name: 'host',
-								},
-							],
-						},
 					],
 					args: {
 						name: 'tool-name...',
@@ -6511,6 +6511,16 @@ const completionSpec: Fig.Spec = {
 					name: ['uninstall'],
 					description: 'Uninstall installed tools.',
 					options: [
+						{
+							name: ['--agent'],
+							description: 'Uninstall the skill from the specified agent(s): copilot, claude. Use --agent all (or omit --agent) to remove the skill from every agent it is installed through (skill tools only)',
+							isRepeatable: true,
+							args: [
+								{
+									name: 'agent',
+								},
+							],
+						},
 						{
 							name: ['--all'],
 							description: 'Uninstall all installed tools',
@@ -6519,16 +6529,6 @@ const completionSpec: Fig.Spec = {
 							name: ['--dry-run'],
 							description: 'Preview what would be uninstalled without making changes',
 						},
-						{
-							name: ['--host'],
-							description: 'Uninstall the skill from the specified agent host(s): copilot, claude. Use --host all (or omit --host) to remove the skill from every host it is installed through (skill tools only)',
-							isRepeatable: true,
-							args: [
-								{
-									name: 'host',
-								},
-							],
-						},
 					],
 					args: {
 						name: 'tool-name...',
@@ -6540,19 +6540,23 @@ const completionSpec: Fig.Spec = {
 					description: 'Upgrade installed tools.',
 					options: [
 						{
-							name: ['--dry-run'],
-							description: 'Preview what would be upgraded without making changes',
-						},
-						{
-							name: ['--host'],
-							description: 'Upgrade the skill for the specified agent host(s): copilot, claude. Use --host all for every detected host (skill tools only)',
+							name: ['--agent'],
+							description: 'Upgrade the skill for the specified agent(s): copilot, claude. Use --agent all for every detected agent (skill tools only)',
 							isRepeatable: true,
 							args: [
 								{
-									name: 'host',
+									name: 'agent',
 								},
 							],
 						},
+						{
+							name: ['--all'],
+							description: 'Upgrade all installed tools',
+						},
+						{
+							name: ['--dry-run'],
+							description: 'Preview what would be upgraded without making changes',
+						},
 					],
 					args: {
 						name: 'tool-name...',

cli/azd/cmd/testdata/TestUsage-azd-tool-check.snap

@@ -1,5 +1,5 @@
 
-Check for tool updates.
+Check for tool upgrades.
 
 Usage
   azd tool check [flags]

cli/azd/cmd/testdata/TestUsage-azd-tool-install.snap

@@ -5,9 +5,9 @@ Usage
   azd tool install [tool-name...] [flags]
 
 Flags
-        --all          	: Install all recommended tools
-        --dry-run      	: Preview what would be installed without making changes
-        --host strings 	: Install the skill for the specified agent host(s): copilot, claude. Use --host all for every detected host (skill tools only)
+        --agent strings 	: Install the skill for the specified agent(s): copilot, claude. Use --agent all for every detected agent (skill tools only)
+        --all           	: Install all recommended tools
+        --dry-run       	: Preview what would be installed without making changes
 
 Global Flags
     -C, --cwd string         	: Sets the current working directory.

cli/azd/cmd/testdata/TestUsage-azd-tool-uninstall.snap

@@ -5,9 +5,9 @@ Usage
   azd tool uninstall [tool-name...] [flags]
 
 Flags
-        --all          	: Uninstall all installed tools
-        --dry-run      	: Preview what would be uninstalled without making changes
-        --host strings 	: Uninstall the skill from the specified agent host(s): copilot, claude. Use --host all (or omit --host) to remove the skill from every host it is installed through (skill tools only)
+        --agent strings 	: Uninstall the skill from the specified agent(s): copilot, claude. Use --agent all (or omit --agent) to remove the skill from every agent it is installed through (skill tools only)
+        --all           	: Uninstall all installed tools
+        --dry-run       	: Preview what would be uninstalled without making changes
 
 Global Flags
     -C, --cwd string         	: Sets the current working directory.

cli/azd/cmd/testdata/TestUsage-azd-tool-upgrade.snap

@@ -5,8 +5,9 @@ Usage
   azd tool upgrade [tool-name...] [flags]
 
 Flags
-        --dry-run      	: Preview what would be upgraded without making changes
-        --host strings 	: Upgrade the skill for the specified agent host(s): copilot, claude. Use --host all for every detected host (skill tools only)
+        --agent strings 	: Upgrade the skill for the specified agent(s): copilot, claude. Use --agent all for every detected agent (skill tools only)
+        --all           	: Upgrade all installed tools
+        --dry-run       	: Preview what would be upgraded without making changes
 
 Global Flags
     -C, --cwd string         	: Sets the current working directory.

cli/azd/cmd/testdata/TestUsage-azd-tool.snap

@@ -5,7 +5,7 @@ Usage
   azd tool [command]
 
 Available Commands
-  check    	: Check for tool updates.
+  check    	: Check for tool upgrades.
   install  	: Install specified tools.
   list     	: List all tools with status.
   show     	: Show details for a specific tool.