aitools install: drop positional [skill-name] form

Per Simon's review: avoid locking the top-level `databricks aitools install`
into a positional that means "skill", since the aitools surface may grow
commands for things beyond skills. `--skills name1,name2` remains the
documented way to scope an install.

The legacy alias `experimental aitools skills install <name>` keeps its
positional form by translating to `--skills` before delegating.

Co-authored-by: Isaac

Author: jamesbroadhead

aitools/cmd/install.go

@@ -57,7 +57,7 @@ func NewInstallCmd() *cobra.Command {
 	var projectFlag, globalFlag bool
 
 	cmd := &cobra.Command{
-		Use:   "install [skill-name]",
+		Use:   "install",
 		Short: "Install AI skills for coding agents",
 		Long: `Install Databricks AI skills for detected coding agents.
 
@@ -66,21 +66,13 @@ Use --project to install to the current project directory instead.
 When multiple agents are detected, skills are stored in a canonical location
 and symlinked to each agent to avoid duplication.
 
-Pass a single skill name as a positional argument to install just that skill,
-or use --skills name1,name2 for multiple. The two forms are mutually exclusive.
+Use --skills name1,name2 to install specific skills.
 
 Supported agents: Claude Code, Cursor, Codex CLI, OpenCode, GitHub Copilot, Antigravity`,
-		Args: cobra.MaximumNArgs(1),
+		Args: cobra.NoArgs,
 		RunE: func(cmd *cobra.Command, args []string) error {
 			ctx := cmd.Context()
 
-			if len(args) == 1 {
-				if skillsFlag != "" {
-					return errors.New("cannot use positional [skill-name] together with --skills; pick one")
-				}
-				skillsFlag = args[0]
-			}
-
 			// Resolve scope.
 			scope, err := resolveScopeWithPrompt(ctx, projectFlag, globalFlag)
 			if err != nil {

aitools/cmd/install_test.go

@@ -262,45 +262,17 @@ func TestInstallAgentsFlagSkipsPrompt(t *testing.T) {
 	assert.Equal(t, []string{"claude-code", "cursor"}, (*calls)[0].agents)
 }
 
-func TestInstallPositionalSkillName(t *testing.T) {
-	setupTestAgents(t)
-	calls := setupInstallMock(t)
-
+func TestInstallRejectsPositionalArgs(t *testing.T) {
 	ctx := cmdio.MockDiscard(t.Context())
 	cmd := NewInstallCmd()
 	cmd.SetContext(ctx)
 	cmd.SetArgs([]string{"databricks-jobs"})
-
-	require.NoError(t, cmd.Execute())
-	require.Len(t, *calls, 1)
-	assert.Equal(t, []string{"databricks-jobs"}, (*calls)[0].opts.SpecificSkills)
-}
-
-func TestInstallPositionalAndSkillsFlagConflict(t *testing.T) {
-	setupTestAgents(t)
-	setupInstallMock(t)
-
-	ctx := cmdio.MockDiscard(t.Context())
-	cmd := NewInstallCmd()
-	cmd.SetContext(ctx)
-	cmd.SetArgs([]string{"databricks-jobs", "--skills", "databricks-pipelines"})
 	cmd.SilenceErrors = true
 	cmd.SilenceUsage = true
 
 	err := cmd.Execute()
 	require.Error(t, err)
-	assert.Contains(t, err.Error(), "cannot use positional")
-}
-
-func TestInstallRejectsTwoPositionalArgs(t *testing.T) {
-	ctx := cmdio.MockDiscard(t.Context())
-	cmd := NewInstallCmd()
-	cmd.SetContext(ctx)
-	cmd.SetArgs([]string{"databricks-jobs", "databricks-pipelines"})
-	cmd.SilenceErrors = true
-	cmd.SilenceUsage = true
-
-	require.Error(t, cmd.Execute())
+	assert.Contains(t, err.Error(), "unknown command")
 }
 
 func TestUpdateRejectsPositionalArgs(t *testing.T) {