universal-dev-standards 6.9.0 → 6.11.0

package/src/commands/config.js

@@ -1,6 +1,6 @@
 import chalk from 'chalk';
 import { select, confirm as inquirerConfirm } from '@inquirer/prompts';
-import ora from 'ora';
+import { createSpinner } from '../utils/spinner.js';
 import { unlinkSync, existsSync, writeFileSync, mkdirSync } from 'fs';
 import { join, basename } from 'path';
 import { config } from '../utils/config-manager.js';
@@ -45,11 +45,11 @@ import {
 import { displayLanguageToLocale } from '../utils/locale.js';
 import {
   writeIntegrationFile,
-  getToolFilePath
+  resolveIntegrationTargetFile
 } from '../utils/integration-generator.js';
 import { getMarketplaceSkillsInfo } from '../utils/github.js';
 import { regenerateIntegrations } from './update.js';
-import { mergeInstalledNames } from '../core/manifest.js';
+import { mergeInstalledNames, bumpManifestVersion } from '../core/manifest.js';
 
 /**
  * Get localized message with fallback (for config-specific keys)
@@ -381,12 +381,12 @@ async function handleDisplayLanguageChange() {
     if (confirmReinstall) {
       const cmdLocale = displayLanguageToLocale(newLang);
       if (hasSkillInstalls) {
-        const spinner = ora(t('config.reinstallingSkills', 'Reinstalling Skills...')).start();
+        const spinner = createSpinner(t('config.reinstallingSkills', 'Reinstalling Skills...')).start();
         await installSkillsToMultipleAgents(manifest.skills.installations, null, projectPath, cmdLocale);
         spinner.succeed(t('config.skillsReinstalled', 'Skills reinstalled'));
       }
       if (hasCommandInstalls) {
-        const spinner = ora(t('config.reinstallingCommands', 'Reinstalling Commands...')).start();
+        const spinner = createSpinner(t('config.reinstallingCommands', 'Reinstalling Commands...')).start();
         await installCommandsToMultipleAgents(manifest.commands.installations, null, projectPath, cmdLocale);
         spinner.succeed(t('config.commandsReinstalled', 'Commands reinstalled'));
       }
@@ -401,7 +401,7 @@ async function handleDisplayLanguageChange() {
     });
 
     if (confirmRegen) {
-      const spinner = ora(t('config.applyingPreset', 'Applying...')).start();
+      const spinner = createSpinner(t('config.applyingPreset', 'Applying...')).start();
       regenerateIntegrations(projectPath, manifest);
       spinner.succeed(t('config.integrationsRegenerated', 'Integrations regenerated'));
     }
@@ -665,15 +665,18 @@ export async function runProjectConfiguration(options) {
       newAITools = newAITools.filter(tool => !result.tools.includes(tool));
 
       // Remove integration files for removed tools
-      const spinner = ora(msgObj.removingIntegrations).start();
+      const spinner = createSpinner(msgObj.removingIntegrations).start();
       for (const tool of result.tools) {
-        const filePath = join(projectPath, getToolFilePath(tool));
+        // XSPEC-418 R3: remove the file this tool actually targets (e.g.
+        // CLAUDE.local.md), not always its hardcoded default.
+        const removedToolFile = resolveIntegrationTargetFile(tool, manifest);
+        const filePath = join(projectPath, removedToolFile);
         if (existsSync(filePath)) {
           try {
             unlinkSync(filePath);
-            console.log(chalk.gray(`  ${msgObj.removed}: ${getToolFilePath(tool)}`));
+            console.log(chalk.gray(`  ${msgObj.removed}: ${removedToolFile}`));
           } catch {
-            console.log(chalk.yellow(`  ${msgObj.couldNotRemove}: ${getToolFilePath(tool)}`));
+            console.log(chalk.yellow(`  ${msgObj.couldNotRemove}: ${removedToolFile}`));
           }
         }
 
@@ -711,7 +714,7 @@ export async function runProjectConfiguration(options) {
         }
 
         // Remove from integrationBlockHashes (keyed by tool file path)
-        const toolFileName = getToolFilePath(tool);
+        const toolFileName = removedToolFile;
         if (manifest.integrationBlockHashes?.[toolFileName]) {
           delete manifest.integrationBlockHashes[toolFileName];
         }
@@ -854,7 +857,7 @@ export async function runProjectConfiguration(options) {
   }
 
   // Apply changes
-  const spinner = ora(msgObj.updatingConfig).start();
+  const spinner = createSpinner(msgObj.updatingConfig).start();
 
   const results = {
     copied: [],
@@ -914,7 +917,7 @@ export async function runProjectConfiguration(options) {
 
   // Regenerate integration files if needed
   if (needsIntegrationRegeneration && newAITools.length > 0) {
-    const intSpinner = ora(msgObj.regeneratingIntegrations).start();
+    const intSpinner = createSpinner(msgObj.regeneratingIntegrations).start();
 
     // Build installed standards list
     // Raw, not basename()d. Resolution needs the registry (an ID is not a
@@ -934,7 +937,8 @@ export async function runProjectConfiguration(options) {
     const generatedFiles = new Set();
 
     for (const tool of newAITools) {
-      const targetFile = getToolFilePath(tool);
+      // XSPEC-418 R3
+      const targetFile = resolveIntegrationTargetFile(tool, manifest);
       if (generatedFiles.has(targetFile)) {
         continue; // Skip if already generated (AGENTS.md sharing)
       }
@@ -947,7 +951,9 @@ export async function runProjectConfiguration(options) {
         standardsFormat: manifest.format || 'ai',
         contentMode: newContentMode,
         // Pass output_language for dynamic commit standards generation
-        outputLanguage: newOptions.output_language || 'english'
+        outputLanguage: newOptions.output_language || 'english',
+        // XSPEC-418 R2/R3
+        integrationTargets: manifest.integrationTargets
       };
 
       const result = writeIntegrationFile(tool, toolConfig, projectPath);
@@ -989,7 +995,7 @@ export async function runProjectConfiguration(options) {
 
           if (confirmSkills) {
             const newInstallations = skillCapableTools.map(agent => ({ agent, level }));
-            const spinner = ora(t('config.installingSkillsForNewTools', 'Installing Skills for new tools...')).start();
+            const spinner = createSpinner(t('config.installingSkillsForNewTools', 'Installing Skills for new tools...')).start();
             const skillResult = await installSkillsToMultipleAgents(newInstallations, null, projectPath, cmdLocale);
             spinner.succeed(t('config.skillsInstalledForNewTools', 'Skills installed for new tools'));
 
@@ -1026,7 +1032,7 @@ export async function runProjectConfiguration(options) {
 
           if (confirmCmds) {
             const newCmdInstallations = commandCapableTools.map(agent => ({ agent, level }));
-            const spinner = ora(t('config.installingCommandsForNewTools', 'Installing Commands for new tools...')).start();
+            const spinner = createSpinner(t('config.installingCommandsForNewTools', 'Installing Commands for new tools...')).start();
             const cmdResult = await installCommandsToMultipleAgents(newCmdInstallations, null, projectPath, cmdLocale);
             spinner.succeed(t('config.commandsInstalledForNewTools', 'Commands installed for new tools'));
 
@@ -1052,7 +1058,7 @@ export async function runProjectConfiguration(options) {
   manifest.options = newOptions;
   manifest.contentMode = newContentMode;
   manifest.aiTools = newAITools;
-  manifest.version = '3.2.0';
+  bumpManifestVersion(manifest);
 
   // Update methodology
   if (newMethodology) {
@@ -1124,7 +1130,7 @@ export async function runProjectConfiguration(options) {
 
     if (options.yes) {
       // --yes flag: auto-apply without prompting
-      const applySpinner = ora(msgObj.applyingChanges).start();
+      const applySpinner = createSpinner(msgObj.applyingChanges).start();
       const applyResults = regenerateIntegrations(projectPath, manifest);
       applySpinner.succeed(msgObj.changesApplied || msgObj.regeneratedIntegrations.replace('{count}', applyResults.updated.length));
 
@@ -1146,7 +1152,7 @@ export async function runProjectConfiguration(options) {
       });
 
       if (apply) {
-        const applySpinner = ora(msgObj.applyingChanges).start();
+        const applySpinner = createSpinner(msgObj.applyingChanges).start();
         const applyResults = regenerateIntegrations(projectPath, manifest);
         applySpinner.succeed(msgObj.changesApplied || msgObj.regeneratedIntegrations.replace('{count}', applyResults.updated.length));
 
@@ -1172,12 +1178,12 @@ export async function runProjectConfiguration(options) {
     if (hasSkillInstalls || hasCommandInstalls) {
       const cmdLocale = displayLanguageToLocale(newOptions.display_language);
       if (hasSkillInstalls) {
-        const skillSpinner = ora(t('config.reinstallingSkills', 'Reinstalling Skills...')).start();
+        const skillSpinner = createSpinner(t('config.reinstallingSkills', 'Reinstalling Skills...')).start();
         await installSkillsToMultipleAgents(manifest.skills.installations, null, projectPath, cmdLocale);
         skillSpinner.succeed(t('config.skillsReinstalled', 'Skills reinstalled'));
       }
       if (hasCommandInstalls) {
-        const cmdSpinner = ora(t('config.reinstallingCommands', 'Reinstalling Commands...')).start();
+        const cmdSpinner = createSpinner(t('config.reinstallingCommands', 'Reinstalling Commands...')).start();
         await installCommandsToMultipleAgents(manifest.commands.installations, null, projectPath, cmdLocale);
         cmdSpinner.succeed(t('config.commandsReinstalled', 'Commands reinstalled'));
       }
@@ -1222,7 +1228,7 @@ async function handleSkillsConfiguration(manifest, projectPath, msgObj, common,
 
     // Install to specified level (defaults to project)
     const installations = [{ agent: specificTool, level }];
-    const spinner = ora(`Installing Skills for ${getAgentDisplayName(specificTool)} (${level} level)...`).start();
+    const spinner = createSpinner(`Installing Skills for ${getAgentDisplayName(specificTool)} (${level} level)...`).start();
     const result = await installSkillsToMultipleAgents(installations, null, projectPath);
     spinner.stop();
 
@@ -1359,7 +1365,7 @@ async function handleSkillsConfiguration(manifest, projectPath, msgObj, common,
     }));
 
     // Install Skills
-    const spinner = ora(msgObj.installingSkills || 'Installing Skills...').start();
+    const spinner = createSpinner(msgObj.installingSkills || 'Installing Skills...').start();
     const result = await installSkillsToMultipleAgents(installations, null, projectPath);
     spinner.stop();
 
@@ -1394,7 +1400,7 @@ async function handleSkillsConfiguration(manifest, projectPath, msgObj, common,
   }
 
   // Install Skills
-  const spinner = ora(msgObj.installingSkills || 'Installing Skills...').start();
+  const spinner = createSpinner(msgObj.installingSkills || 'Installing Skills...').start();
   const result = await installSkillsToMultipleAgents(installations, null, projectPath);
   spinner.stop();
 
@@ -1453,7 +1459,7 @@ async function handleCommandsConfiguration(manifest, projectPath, msgObj, common
 
     // Install to selected level
     const installations = [{ agent: specificTool, level: commandsLevel }];
-    const spinner = ora(`Installing Commands for ${getAgentDisplayName(specificTool)} (${commandsLevel} level)...`).start();
+    const spinner = createSpinner(`Installing Commands for ${getAgentDisplayName(specificTool)} (${commandsLevel} level)...`).start();
     const cmdLocale = displayLanguageToLocale(manifest.options?.display_language);
     const result = await installCommandsToMultipleAgents(installations, null, projectPath, cmdLocale);
     spinner.stop();
@@ -1559,7 +1565,7 @@ async function handleCommandsConfiguration(manifest, projectPath, msgObj, common
     }
 
     // Install Commands
-    const spinner = ora(msgObj.installingCommands || 'Installing Commands...').start();
+    const spinner = createSpinner(msgObj.installingCommands || 'Installing Commands...').start();
     const cmdLocale = displayLanguageToLocale(manifest.options?.display_language);
     const result = await installCommandsToMultipleAgents(declinedCommandsWithSupport, null, projectPath, cmdLocale);
     spinner.stop();
@@ -1595,7 +1601,7 @@ async function handleCommandsConfiguration(manifest, projectPath, msgObj, common
   }
 
   // Install Commands
-  const spinner = ora(msgObj.installingCommands || 'Installing Commands...').start();
+  const spinner = createSpinner(msgObj.installingCommands || 'Installing Commands...').start();
   const cmdLocale = displayLanguageToLocale(manifest.options?.display_language);
   const result = await installCommandsToMultipleAgents(selectedAgents, null, projectPath, cmdLocale);
   spinner.stop();

package/src/commands/init.js

@@ -1,8 +1,8 @@
 import chalk from 'chalk';
-import ora from 'ora';
+import { createSpinner } from '../utils/spinner.js';
 import { execSync } from 'child_process';
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'fs';
-import { basename, join } from 'path';
+import { join } from 'path';
 import {
   manifestExists as isInitialized,
   normalizeContentMode,
@@ -26,7 +26,7 @@ import { displayLanguageToLocale } from '../utils/locale.js';
 import { generateReleaseConfig, RELEASE_MODE_LABELS } from '../utils/release-config.js';
 import { guardAgainstSelfAdoption } from '../utils/detect-self-adoption.js';
 import { readInstallYaml } from '../utils/config-manager.js';
-import { getToolFilePath } from '../utils/integration-generator.js';
+import { resolveIntegrationTargetFile } from '../utils/integration-generator.js';
 import { withFileTransaction } from '../utils/transaction.js';
 
 /**
@@ -55,7 +55,7 @@ export async function initCommand(options) {
   }
 
   // STEP 2: Detect project characteristics
-  const spinner = ora(msg.detectingProject).start();
+  const spinner = createSpinner(msg.detectingProject).start();
   const detected = detectAll(projectPath);
   spinner.succeed(msg.analysisComplete);
 
@@ -107,7 +107,10 @@ export async function initCommand(options) {
   // `uds init` re-installs them idempotently.
   const ownedPaths = [join(projectPath, '.standards')];
   for (const tool of (config.integrations || config.aiTools || [])) {
-    const file = getToolFilePath(tool);
+    // XSPEC-418 R2: honors --claude-target so a local-target install's owned
+    // path is CLAUDE.local.md, not CLAUDE.md — rollback must tear down the
+    // file actually written, not the tool's default.
+    const file = resolveIntegrationTargetFile(tool, { integrationTargets: config.integrationTargets });
     if (file) ownedPaths.push(join(projectPath, file));
   }
   if (config.generateAgentsMd) ownedPaths.push(join(projectPath, 'AGENTS.md'));
@@ -123,7 +126,16 @@ export async function initCommand(options) {
         apply: async () => {
           // 1. Install Standards
           const standardsResults = await installStandards(config, projectPath);
-          config.installedStandards = standardsResults.standards.map(s => basename(s));
+          // Paths, not basenames. Everything downstream classifies an option
+          // by an `/options/` segment, so reducing to basenames here is not a
+          // simplification — it deletes the only signal that distinguishes the
+          // two, and `contentLayout: flat` installs option files with no
+          // directory to recover it from. The index block consequently said
+          // "options 0" on every project for two releases, while the manifest
+          // right beside it listed seven. `buildToolIntegrationConfig` (the
+          // update path's equivalent) has carried a comment warning against
+          // exactly this reduction the whole time; only this call site did it.
+          config.installedStandards = standardsResults.standards;
 
           // 1.5. Generate release-config.yaml if non-default mode selected
           if (config.releaseMode && config.releaseMode !== 'ci-cd') {
@@ -649,6 +661,23 @@ function buildNonInteractiveConfig(options, detected, projectPath) {
     ? false // codex/opencode handles AGENTS.md, skip universal output
     : (options.agentsMd !== undefined ? !!options.agentsMd : true);
 
+  // XSPEC-418 R2: --claude-target <project|local>, default 'project'. An
+  // unrecognized value is never substituted quietly (same policy as
+  // --content-mode above) — it falls back to 'project' with a warning rather
+  // than silently writing into CLAUDE.local.md or vice versa.
+  let claudeTargetFlag = options.claudeTarget || 'project';
+  if (claudeTargetFlag !== 'project' && claudeTargetFlag !== 'local') {
+    console.log(chalk.yellow(
+      `⚠ Unknown --claude-target '${claudeTargetFlag}'; using 'project'. Supported: project, local.`
+    ));
+    claudeTargetFlag = 'project';
+  }
+  // Only ever set when 'local' — 'project' or unspecified leaves the manifest
+  // shape completely unchanged (XSPEC-418 AC-5).
+  const integrationTargets = claudeTargetFlag === 'local'
+    ? { 'claude-code': 'CLAUDE.local.md' }
+    : undefined;
+
   return {
     languages: options.lang ? [options.lang] : Object.keys(detected.languages).filter(k => detected.languages[k]),
     frameworks: options.framework ? [options.framework] : Object.keys(detected.frameworks).filter(k => detected.frameworks[k]),
@@ -669,7 +698,8 @@ function buildNonInteractiveConfig(options, detected, projectPath) {
     generateAgentsMd,
     releaseMode: options.releaseMode || 'ci-cd',
     withHooks: !!options.withHooks,
-    contentLayout: options.contentLayout || 'flat'
+    contentLayout: options.contentLayout || 'flat',
+    integrationTargets
   };
 }
 

package/src/commands/spec.js

@@ -9,7 +9,7 @@
  */
 
 import chalk from 'chalk';
-import ora from 'ora';
+import { createSpinner } from '../utils/spinner.js';
 import { select, confirm as inquirerConfirm } from '@inquirer/prompts';
 import { existsSync, readFileSync } from 'node:fs';
 import { join } from 'node:path';
@@ -48,7 +48,7 @@ export async function specCreateCommand(intent, options = {}) {
     output: options.output
   });
 
-  const spinner = ora({
+  const spinner = createSpinner({
     text: t('spec.generating', 'Generating micro-spec...'),
     color: 'cyan'
   }).start();