fix: panic in install and missing uninstall alias

Author: yeasy

CHANGELOG.md

@@ -15,6 +15,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Local Install Optimization**: Installing a skill that exists in the local cache now performs a fast file copy instead of a git clone.
 - **Testing**: Added comprehensive unit tests for CLI commands.
 
+### Fixed
+- **Panic on Single-Word Install**: Fixed critical panic when using `ask install <name>` with a single word argument.
+- **Uninstall Alias**: Added missing top-level `ask uninstall` alias (previously only `ask skill uninstall` worked).
+
 ### Changed
 - **Repository Naming**: Local cache directories now use the user-configured repository name (e.g. `anthropics`).
 - **Improved UX**: Reduced verbosity of installation commands.

cmd/install.go

@@ -261,11 +261,12 @@ func installSingleSkill(input string, global bool, agents []string, cfg *config.
 				subDir = strings.Join(parts[2:], "/")
 				skillName = parts[len(parts)-1]
 				repoURL = fmt.Sprintf("https://github.com/%s/%s.git", owner, repo)
-			} else {
+			} else if len(parts) >= 2 {
 				// Potential RepoName/SkillName from cache (e.g. "anthropics-skills/browser-use")
 				// or Standard install: owner/repo (e.g. "browser-use/browser-use")
 
 				foundInCache := false
+
 				repoName := parts[0]
 				skillNamePart := parts[1]
 
@@ -333,6 +334,7 @@ func installSingleSkill(input string, global bool, agents []string, cfg *config.
 							}
 						}
 					}
+
 				}
 
 				if !foundInCache {

cmd/root.go

@@ -75,6 +75,15 @@ var listRootCmd = &cobra.Command{
 	Run:   runList,
 }
 
+var uninstallRootCmd = &cobra.Command{
+	Use:     "uninstall [skill-name]",
+	Short:   "Uninstall a skill (alias for 'skill uninstall')",
+	Long:    "Remove a skill from the skills directory.\nThis is a shortcut for 'ask skill uninstall'.",
+	Example: uninstallCmd.Example,
+	Args:    uninstallCmd.Args,
+	Run:     uninstallCmd.Run,
+}
+
 // Execute adds all child commands to the root command and sets flags appropriately.
 // This is called by main.main(). It only needs to happen once to the rootCmd.
 func Execute() {
@@ -107,6 +116,15 @@ func init() {
 
 	rootCmd.AddCommand(listRootCmd)
 	registerListFlags(listRootCmd)
+
+	rootCmd.AddCommand(uninstallRootCmd)
+	// No specific flags to register for uninstall root shim as it uses uninstallCmd.Run directly?
+	// Actually uninstallCmd.Run uses flags so we should share flags definition or re-register.
+	// Since uninstallCmd is in another file, we can't easily reuse 'registerUninstallFlags' unless we export it.
+	// But uninstallCmd is exported. Let's see how registerListFlags works.
+	// It's likely defined in list.go.
+	// We should probably just copy flags setup here.
+	uninstallRootCmd.Flags().AddFlagSet(uninstallCmd.Flags())
 }
 
 // initConfig reads in config file and ENV variables if set.