Fix 4 project-scope bugs: idempotency check, list label, update guard, agent prompt

1. allAgentsHaveSkill now uses agentSkillsDirForScope so the idempotency
   check looks in the correct directory during --project installs.
2. list --project with no installed state now shows "(project)" instead
   of "(global)" in the summary line.
3. UpdateSkills now returns an error when all detected agents are
   incompatible with project scope, matching the install flow guard.
4. Install prompt pre-filters agents by project-scope compatibility,
   so users only see agents that can actually receive the skills.

Co-authored-by: Isaac

Author: simonfaltum

experimental/aitools/cmd/install.go

@@ -53,6 +53,14 @@ Supported agents: Claude Code, Cursor, Codex CLI, OpenCode, GitHub Copilot, Anti
 					return nil
 				}
 
+				// For project scope, pre-filter to compatible agents before prompting.
+				if scope == installer.ScopeProject {
+					detected = filterProjectScopeAgents(detected)
+					if len(detected) == 0 {
+						return fmt.Errorf("no detected agents support project-scoped skills")
+					}
+				}
+
 				switch {
 				case len(detected) == 1:
 					targetAgents = detected
@@ -120,6 +128,17 @@ func resolveAgentNames(ctx context.Context, names string) ([]*agents.Agent, erro
 	return result, nil
 }
 
+// filterProjectScopeAgents returns only agents that support project-scoped skills.
+func filterProjectScopeAgents(detected []*agents.Agent) []*agents.Agent {
+	var compatible []*agents.Agent
+	for _, a := range detected {
+		if a.SupportsProjectScope {
+			compatible = append(compatible, a)
+		}
+	}
+	return compatible
+}
+
 // printNoAgentsMessage prints the "no agents detected" message.
 func printNoAgentsMessage(ctx context.Context) {
 	cmdio.LogString(ctx, color.YellowString("No supported coding agents detected."))

experimental/aitools/cmd/list.go

@@ -132,9 +132,10 @@ func defaultListSkills(cmd *cobra.Command, scope string) error {
 		cmdio.LogString(ctx, fmt.Sprintf("%d/%d skills installed (global), %d/%d (project)", globalCount, len(names), projectCount, len(names)))
 	case projectState != nil:
 		cmdio.LogString(ctx, fmt.Sprintf("%d/%d skills installed (project)", projectCount, len(names)))
+	case scope == installer.ScopeProject:
+		cmdio.LogString(ctx, fmt.Sprintf("%d/%d skills installed (project)", 0, len(names)))
 	default:
-		installedCount := globalCount
-		cmdio.LogString(ctx, fmt.Sprintf("%d/%d skills installed (global)", installedCount, len(names)))
+		cmdio.LogString(ctx, fmt.Sprintf("%d/%d skills installed (global)", globalCount, len(names)))
 	}
 	return nil
 }

experimental/aitools/lib/installer/installer.go

@@ -174,7 +174,7 @@ func InstallSkillsForAgents(ctx context.Context, src ManifestSource, targetAgent
 		// dir exists, AND every requested agent already has the skill on disk.
 		if state != nil && state.Skills[name] == meta.Version {
 			skillDir := filepath.Join(baseDir, name)
-			if _, statErr := os.Stat(skillDir); statErr == nil && allAgentsHaveSkill(ctx, name, targetAgents) {
+			if _, statErr := os.Stat(skillDir); statErr == nil && allAgentsHaveSkill(ctx, name, targetAgents, scope, cwd) {
 				log.Debugf(ctx, "%s v%s already installed for all agents, skipping", name, meta.Version)
 				continue
 			}
@@ -371,9 +371,9 @@ func hasSkillsOnDisk(dir string) bool {
 
 // allAgentsHaveSkill returns true if every agent in the list has the named
 // skill directory present (either as a real directory or symlink).
-func allAgentsHaveSkill(ctx context.Context, skillName string, targetAgents []*agents.Agent) bool {
+func allAgentsHaveSkill(ctx context.Context, skillName string, targetAgents []*agents.Agent, scope, cwd string) bool {
 	for _, agent := range targetAgents {
-		agentSkillDir, err := agent.SkillsDir(ctx)
+		agentSkillDir, err := agentSkillsDirForScope(ctx, agent, scope, cwd)
 		if err != nil {
 			return false
 		}

experimental/aitools/lib/installer/update.go

@@ -61,7 +61,11 @@ func UpdateSkills(ctx context.Context, src ManifestSource, targetAgents []*agent
 		if err != nil {
 			return nil, fmt.Errorf("failed to determine working directory: %w", err)
 		}
+		incompatible := incompatibleAgentNames(targetAgents)
 		targetAgents = filterProjectAgents(ctx, targetAgents)
+		if len(targetAgents) == 0 {
+			return nil, fmt.Errorf("no agents support project-scoped skills. The following detected agents are global-only: %s", strings.Join(incompatible, ", "))
+		}
 	}
 
 	state, err := LoadState(baseDir)