feat(setup:skills): prompt to overwrite existing skills

When running interactively, detect already-installed skills and prompt
the user to overwrite them rather than requiring the --update flag
up front. Non-interactive behavior (--force, --update, --json)
is preserved.

Widen isSkillInstalled options to include the optional directory
override so the target-path check matches the installer's resolution
when --directory is provided.

Author: clavery

.changeset/setup-skills-upgrade-prompt.md

@@ -0,0 +1,6 @@
+---
+'@salesforce/b2c-cli': patch
+'@salesforce/b2c-tooling-sdk': patch
+---
+
+`b2c setup skills` now prompts to overwrite already-installed skills in interactive mode instead of silently skipping them with a "use --update to overwrite" message. The existing `--update` and `--force` flags still work non-interactively.

packages/b2c-cli/src/commands/setup/skills.ts

@@ -16,6 +16,7 @@ import {
   downloadSkillsArtifact,
   scanSkills,
   installSkills,
+  isSkillInstalled,
   getIdeDisplayName,
   getIdeDocsUrl,
   findSkillsByName,
@@ -305,6 +306,34 @@ export default class SetupSkills extends BaseCommand<typeof SetupSkills> {
       }
     }
 
+    // Detect already-installed skills and prompt to upgrade (unless --update or --force set)
+    let update = this.flags.update;
+    if (!update && !this.flags.force) {
+      const existingCount = skillsToInstall.reduce((count, skill) => {
+        return (
+          count +
+          targetIdes.filter((ide) =>
+            isSkillInstalled(skill.name, ide, {
+              global: this.flags.global,
+              projectRoot: process.cwd(),
+              directory,
+            }),
+          ).length
+        );
+      }, 0);
+
+      if (existingCount > 0) {
+        update = await confirm({
+          message: t(
+            'commands.setup.skills.confirmUpgrade',
+            '{{count}} skill(s) are already installed. Overwrite with the new version?',
+            {count: existingCount},
+          ),
+          default: true,
+        });
+      }
+    }
+
     // Install skills for all skillsets in parallel
     const installPromises = skillsets
       .map((skillset) => {
@@ -313,7 +342,7 @@ export default class SetupSkills extends BaseCommand<typeof SetupSkills> {
         return installSkills(skillsForSet, skillsDirs[skillset], {
           ides: targetIdes,
           global: this.flags.global,
-          update: this.flags.update,
+          update,
           projectRoot: process.cwd(),
           directory,
         });

packages/b2c-tooling-sdk/src/skills/installer.ts

@@ -88,7 +88,7 @@ async function copyDirectory(source: string, target: string): Promise<number> {
 export function isSkillInstalled(
   skillName: string,
   ide: IdeType,
-  options: {global: boolean; projectRoot?: string},
+  options: {global: boolean; projectRoot?: string; directory?: string},
 ): boolean {
   const installPath = getSkillInstallPath(ide, skillName, options);
   return fs.existsSync(installPath);