uloop-cli 0.44.2 → 0.45.2

package/src/skills/skills-command.ts

@@ -2,6 +2,9 @@
  * CLI command definitions for skills management.
  */
 
+// CLI commands output to console by design
+/* eslint-disable no-console */
+
 import { Command } from 'commander';
 import {
   getAllSkillStatuses,
@@ -10,53 +13,114 @@ import {
   getInstallDir,
   getTotalSkillCount,
 } from './skills-manager.js';
+import { TargetConfig, ALL_TARGET_IDS, getTargetConfig } from './target-config.js';
+
+interface SkillsOptions {
+  global?: boolean;
+  claude?: boolean;
+  codex?: boolean;
+}
 
 export function registerSkillsCommand(program: Command): void {
-  const skillsCmd = program.command('skills').description('Manage uloop skills for Claude Code');
+  const skillsCmd = program
+    .command('skills')
+    .description('Manage uloop skills for AI coding tools');
 
   skillsCmd
     .command('list')
     .description('List all uloop skills and their installation status')
-    .option('-g, --global', 'Check global installation (~/.claude/skills/)')
-    .action((options: { global?: boolean }) => {
-      listSkills(options.global ?? false);
+    .option('-g, --global', 'Check global installation')
+    .option('--claude', 'Check Claude Code installation')
+    .option('--codex', 'Check Codex CLI installation')
+    .action((options: SkillsOptions) => {
+      const targets = resolveTargets(options);
+      const global = options.global ?? false;
+      listSkills(targets, global);
     });
 
   skillsCmd
     .command('install')
     .description('Install all uloop skills')
-    .option('-g, --global', 'Install to global location (~/.claude/skills/)')
-    .action((options: { global?: boolean }) => {
-      installSkills(options.global ?? false);
+    .option('-g, --global', 'Install to global location')
+    .option('--claude', 'Install to Claude Code')
+    .option('--codex', 'Install to Codex CLI')
+    .action((options: SkillsOptions) => {
+      const targets = resolveTargets(options);
+      if (targets.length === 0) {
+        showTargetGuidance('install');
+        return;
+      }
+      installSkills(targets, options.global ?? false);
     });
 
   skillsCmd
     .command('uninstall')
     .description('Uninstall all uloop skills')
-    .option('-g, --global', 'Uninstall from global location (~/.claude/skills/)')
-    .action((options: { global?: boolean }) => {
-      uninstallSkills(options.global ?? false);
+    .option('-g, --global', 'Uninstall from global location')
+    .option('--claude', 'Uninstall from Claude Code')
+    .option('--codex', 'Uninstall from Codex CLI')
+    .action((options: SkillsOptions) => {
+      const targets = resolveTargets(options);
+      if (targets.length === 0) {
+        showTargetGuidance('uninstall');
+        return;
+      }
+      uninstallSkills(targets, options.global ?? false);
     });
 }
 
-function listSkills(global: boolean): void {
+function resolveTargets(options: SkillsOptions): TargetConfig[] {
+  const targets: TargetConfig[] = [];
+  if (options.claude) {
+    targets.push(getTargetConfig('claude'));
+  }
+  if (options.codex) {
+    targets.push(getTargetConfig('codex'));
+  }
+  return targets;
+}
+
+function showTargetGuidance(command: string): void {
+  console.log(`\nPlease specify at least one target for '${command}':`);
+  console.log('');
+  console.log('Available targets:');
+  console.log('  --claude   Claude Code (.claude/skills/)');
+  console.log('  --codex    Codex CLI (.codex/skills/)');
+  console.log('');
+  console.log('Options:');
+  console.log('  -g, --global   Use global location (~/.claude/ or ~/.codex/)');
+  console.log('');
+  console.log('Examples:');
+  console.log(`  uloop skills ${command} --claude`);
+  console.log(`  uloop skills ${command} --codex --global`);
+  console.log(`  uloop skills ${command} --claude --codex`);
+}
+
+function listSkills(targets: TargetConfig[], global: boolean): void {
   const location = global ? 'Global' : 'Project';
-  const dir = getInstallDir(global);
+  const targetConfigs = targets.length > 0 ? targets : ALL_TARGET_IDS.map(getTargetConfig);
 
-  console.log(`\nuloop Skills Status (${location}):`);
-  console.log(`Location: ${dir}`);
-  console.log('='.repeat(50));
+  console.log(`\nuloop Skills Status:`);
   console.log('');
 
-  const statuses = getAllSkillStatuses(global);
+  for (const target of targetConfigs) {
+    const dir = getInstallDir(target, global);
+
+    console.log(`${target.displayName} (${location}):`);
+    console.log(`Location: ${dir}`);
+    console.log('='.repeat(50));
+
+    const statuses = getAllSkillStatuses(target, global);
+
+    for (const skill of statuses) {
+      const icon = getStatusIcon(skill.status);
+      const statusText = getStatusText(skill.status);
+      console.log(`  ${icon} ${skill.name} (${statusText})`);
+    }
 
-  for (const skill of statuses) {
-    const icon = getStatusIcon(skill.status);
-    const statusText = getStatusText(skill.status);
-    console.log(`  ${icon} ${skill.name} (${statusText})`);
+    console.log('');
   }
 
-  console.log('');
   console.log(`Total: ${getTotalSkillCount()} bundled skills`);
 }
 
@@ -86,33 +150,39 @@ function getStatusText(status: string): string {
   }
 }
 
-function installSkills(global: boolean): void {
+function installSkills(targets: TargetConfig[], global: boolean): void {
   const location = global ? 'global' : 'project';
-  const dir = getInstallDir(global);
 
   console.log(`\nInstalling uloop skills (${location})...`);
   console.log('');
 
-  const result = installAllSkills(global);
+  for (const target of targets) {
+    const dir = getInstallDir(target, global);
+    const result = installAllSkills(target, global);
 
-  console.log(`\x1b[32m✓\x1b[0m Installed: ${result.installed}`);
-  console.log(`\x1b[33m↑\x1b[0m Updated: ${result.updated}`);
-  console.log(`\x1b[90m-\x1b[0m Skipped (up-to-date): ${result.skipped}`);
-  console.log('');
-  console.log(`Skills installed to ${dir}`);
+    console.log(`${target.displayName}:`);
+    console.log(`  \x1b[32m✓\x1b[0m Installed: ${result.installed}`);
+    console.log(`  \x1b[33m↑\x1b[0m Updated: ${result.updated}`);
+    console.log(`  \x1b[90m-\x1b[0m Skipped (up-to-date): ${result.skipped}`);
+    console.log(`  Location: ${dir}`);
+    console.log('');
+  }
 }
 
-function uninstallSkills(global: boolean): void {
+function uninstallSkills(targets: TargetConfig[], global: boolean): void {
   const location = global ? 'global' : 'project';
-  const dir = getInstallDir(global);
 
   console.log(`\nUninstalling uloop skills (${location})...`);
   console.log('');
 
-  const result = uninstallAllSkills(global);
+  for (const target of targets) {
+    const dir = getInstallDir(target, global);
+    const result = uninstallAllSkills(target, global);
 
-  console.log(`\x1b[31m✗\x1b[0m Removed: ${result.removed}`);
-  console.log(`\x1b[90m-\x1b[0m Not found: ${result.notFound}`);
-  console.log('');
-  console.log(`Skills removed from ${dir}`);
+    console.log(`${target.displayName}:`);
+    console.log(`  \x1b[31m✗\x1b[0m Removed: ${result.removed}`);
+    console.log(`  \x1b[90m-\x1b[0m Not found: ${result.notFound}`);
+    console.log(`  Location: ${dir}`);
+    console.log('');
+  }
 }

package/src/skills/skills-manager.ts

@@ -2,10 +2,14 @@
  * Skills manager for installing/uninstalling/listing uloop skills.
  */
 
+// File paths are constructed from home directory and skill names, not from untrusted user input
+/* eslint-disable security/detect-non-literal-fs-filename */
+
 import { existsSync, mkdirSync, readFileSync, writeFileSync, rmSync } from 'fs';
 import { join } from 'path';
 import { homedir } from 'os';
 import { BUNDLED_SKILLS, BundledSkill } from './bundled-skills.js';
+import { TargetConfig } from './target-config.js';
 
 export type SkillStatus = 'installed' | 'not_installed' | 'outdated';
 
@@ -15,26 +19,26 @@ export interface SkillInfo {
   path?: string;
 }
 
-function getGlobalSkillsDir(): string {
-  return join(homedir(), '.claude', 'skills');
+function getGlobalSkillsDir(target: TargetConfig): string {
+  return join(homedir(), target.projectDir, 'skills');
 }
 
-function getProjectSkillsDir(): string {
-  return join(process.cwd(), '.claude', 'skills');
+function getProjectSkillsDir(target: TargetConfig): string {
+  return join(process.cwd(), target.projectDir, 'skills');
 }
 
-function getSkillPath(skillDirName: string, global: boolean): string {
-  const baseDir = global ? getGlobalSkillsDir() : getProjectSkillsDir();
-  return join(baseDir, skillDirName, 'SKILL.md');
+function getSkillPath(skillDirName: string, target: TargetConfig, global: boolean): string {
+  const baseDir = global ? getGlobalSkillsDir(target) : getProjectSkillsDir(target);
+  return join(baseDir, skillDirName, target.skillFileName);
 }
 
-function isSkillInstalled(skill: BundledSkill, global: boolean): boolean {
-  const skillPath = getSkillPath(skill.dirName, global);
+function isSkillInstalled(skill: BundledSkill, target: TargetConfig, global: boolean): boolean {
+  const skillPath = getSkillPath(skill.dirName, target, global);
   return existsSync(skillPath);
 }
 
-function isSkillOutdated(skill: BundledSkill, global: boolean): boolean {
-  const skillPath = getSkillPath(skill.dirName, global);
+function isSkillOutdated(skill: BundledSkill, target: TargetConfig, global: boolean): boolean {
+  const skillPath = getSkillPath(skill.dirName, target, global);
   if (!existsSync(skillPath)) {
     return false;
   }
@@ -42,35 +46,45 @@ function isSkillOutdated(skill: BundledSkill, global: boolean): boolean {
   return installedContent !== skill.content;
 }
 
-export function getSkillStatus(skill: BundledSkill, global: boolean): SkillStatus {
-  if (!isSkillInstalled(skill, global)) {
+export function getSkillStatus(
+  skill: BundledSkill,
+  target: TargetConfig,
+  global: boolean,
+): SkillStatus {
+  if (!isSkillInstalled(skill, target, global)) {
     return 'not_installed';
   }
-  if (isSkillOutdated(skill, global)) {
+  if (isSkillOutdated(skill, target, global)) {
     return 'outdated';
   }
   return 'installed';
 }
 
-export function getAllSkillStatuses(global: boolean): SkillInfo[] {
+export function getAllSkillStatuses(target: TargetConfig, global: boolean): SkillInfo[] {
   return BUNDLED_SKILLS.map((skill) => ({
     name: skill.name,
-    status: getSkillStatus(skill, global),
-    path: isSkillInstalled(skill, global) ? getSkillPath(skill.dirName, global) : undefined,
+    status: getSkillStatus(skill, target, global),
+    path: isSkillInstalled(skill, target, global)
+      ? getSkillPath(skill.dirName, target, global)
+      : undefined,
   }));
 }
 
-export function installSkill(skill: BundledSkill, global: boolean): void {
-  const baseDir = global ? getGlobalSkillsDir() : getProjectSkillsDir();
+export function installSkill(skill: BundledSkill, target: TargetConfig, global: boolean): void {
+  const baseDir = global ? getGlobalSkillsDir(target) : getProjectSkillsDir(target);
   const skillDir = join(baseDir, skill.dirName);
-  const skillPath = join(skillDir, 'SKILL.md');
+  const skillPath = join(skillDir, target.skillFileName);
 
   mkdirSync(skillDir, { recursive: true });
   writeFileSync(skillPath, skill.content, 'utf-8');
 }
 
-export function uninstallSkill(skill: BundledSkill, global: boolean): boolean {
-  const baseDir = global ? getGlobalSkillsDir() : getProjectSkillsDir();
+export function uninstallSkill(
+  skill: BundledSkill,
+  target: TargetConfig,
+  global: boolean,
+): boolean {
+  const baseDir = global ? getGlobalSkillsDir(target) : getProjectSkillsDir(target);
   const skillDir = join(baseDir, skill.dirName);
 
   if (!existsSync(skillDir)) {
@@ -87,17 +101,17 @@ export interface InstallResult {
   skipped: number;
 }
 
-export function installAllSkills(global: boolean): InstallResult {
+export function installAllSkills(target: TargetConfig, global: boolean): InstallResult {
   const result: InstallResult = { installed: 0, updated: 0, skipped: 0 };
 
   for (const skill of BUNDLED_SKILLS) {
-    const status = getSkillStatus(skill, global);
+    const status = getSkillStatus(skill, target, global);
 
     if (status === 'not_installed') {
-      installSkill(skill, global);
+      installSkill(skill, target, global);
       result.installed++;
     } else if (status === 'outdated') {
-      installSkill(skill, global);
+      installSkill(skill, target, global);
       result.updated++;
     } else {
       result.skipped++;
@@ -112,11 +126,11 @@ export interface UninstallResult {
   notFound: number;
 }
 
-export function uninstallAllSkills(global: boolean): UninstallResult {
+export function uninstallAllSkills(target: TargetConfig, global: boolean): UninstallResult {
   const result: UninstallResult = { removed: 0, notFound: 0 };
 
   for (const skill of BUNDLED_SKILLS) {
-    if (uninstallSkill(skill, global)) {
+    if (uninstallSkill(skill, target, global)) {
       result.removed++;
     } else {
       result.notFound++;
@@ -126,8 +140,8 @@ export function uninstallAllSkills(global: boolean): UninstallResult {
   return result;
 }
 
-export function getInstallDir(global: boolean): string {
-  return global ? getGlobalSkillsDir() : getProjectSkillsDir();
+export function getInstallDir(target: TargetConfig, global: boolean): string {
+  return global ? getGlobalSkillsDir(target) : getProjectSkillsDir(target);
 }
 
 export function getTotalSkillCount(): number {

package/src/skills/target-config.ts

@@ -0,0 +1,34 @@
+/**
+ * Target configuration for multi-AI tool support.
+ * Supports Claude Code and Codex CLI, with extensibility for future targets.
+ */
+
+export type TargetId = 'claude' | 'codex';
+
+export interface TargetConfig {
+  id: TargetId;
+  displayName: string;
+  projectDir: string;
+  skillFileName: string;
+}
+
+export const TARGET_CONFIGS: Record<TargetId, TargetConfig> = {
+  claude: {
+    id: 'claude',
+    displayName: 'Claude Code',
+    projectDir: '.claude',
+    skillFileName: 'SKILL.md',
+  },
+  codex: {
+    id: 'codex',
+    displayName: 'Codex CLI',
+    projectDir: '.codex',
+    skillFileName: 'SKILL.md',
+  },
+};
+
+export const ALL_TARGET_IDS: TargetId[] = ['claude', 'codex'];
+
+export function getTargetConfig(id: TargetId): TargetConfig {
+  return TARGET_CONFIGS[id];
+}

package/src/spinner.ts

@@ -0,0 +1,49 @@
+/**
+ * Terminal spinner for showing loading state during async operations.
+ */
+
+// Array index access is safe here (modulo operation keeps it in bounds)
+/* eslint-disable security/detect-object-injection */
+
+const SPINNER_FRAMES = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'] as const;
+const FRAME_INTERVAL_MS = 80;
+
+export interface Spinner {
+  update(message: string): void;
+  stop(): void;
+}
+
+/**
+ * Create a terminal spinner that displays a rotating animation with a message.
+ * Returns a Spinner object with update() and stop() methods.
+ */
+export function createSpinner(initialMessage: string): Spinner {
+  if (!process.stderr.isTTY) {
+    return {
+      update: (): void => {},
+      stop: (): void => {},
+    };
+  }
+
+  let frameIndex = 0;
+  let currentMessage = initialMessage;
+
+  const render = (): void => {
+    const frame = SPINNER_FRAMES[frameIndex];
+    process.stderr.write(`\r\x1b[K${frame} ${currentMessage}`);
+    frameIndex = (frameIndex + 1) % SPINNER_FRAMES.length;
+  };
+
+  render();
+  const intervalId = setInterval(render, FRAME_INTERVAL_MS);
+
+  return {
+    update(message: string): void {
+      currentMessage = message;
+    },
+    stop(): void {
+      clearInterval(intervalId);
+      process.stderr.write('\r\x1b[K');
+    },
+  };
+}

package/src/tool-cache.ts

@@ -3,6 +3,9 @@
  * Handles loading/saving tool definitions from .uloop/tools.json cache.
  */
 
+// File paths are constructed from Unity project root, not from untrusted user input
+/* eslint-disable security/detect-non-literal-fs-filename */
+
 import { existsSync, readFileSync, writeFileSync, mkdirSync } from 'fs';
 import { join } from 'path';
 import defaultToolsData from './default-tools.json';

package/src/version.ts

@@ -4,4 +4,4 @@
  * This file exists to avoid bundling the entire package.json into the CLI bundle.
  * This version is automatically updated by release-please.
  */
-export const VERSION = '0.44.2'; // x-release-please-version
+export const VERSION = '0.45.2'; // x-release-please-version