chore: prepare 1.9.0 release

Author: loppety

CHANGELOG.md

@@ -5,6 +5,18 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.9.0] - 2025-11-15
+
+### ⚠️ Breaking Changes / Migrations
+- Standardized flat-provider command filenames to `clavix-<command>` and relocated Cline workflows to `.clinerules/workflows/`; legacy filenames are auto-detected with an opt-out cleanup prompt during `clavix init` and `clavix update`.
+
+### ✨ Enhancements
+- Added adapter filename overrides and shared template loader so all providers respect the new naming scheme while keeping namespaced folders (e.g., `.claude/commands/clavix/`).
+- Implemented legacy command cleanup utilities reused by init/update flows, preserving provider-specific formatting and supporting Gemini/Qwen namespace opt-outs.
+
+### 🧪 Testing
+- `NODE_OPTIONS="--localstorage-file=.jest-localstorage" npm test`
+
 ## [1.8.3] - 2025-11-15
 
 ### 🐛 Fixes

package-lock.json

@@ -1,11 +1,11 @@
 {
   "name": "clavix",
-  "version": "1.8.3",
+  "version": "1.9.0",
   "description": "AI prompt improvement and PRD generation CLI tool for developers",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "bin": {
-    "clavix": "./bin/clavix.js"
+    "clavix": "bin/clavix.js"
   },
   "scripts": {
     "build": "tsc && npm run copy-templates",

package.json

@@ -12,7 +12,8 @@ import { ClavixConfig, DEFAULT_CONFIG } from '../../types/config';
 import { CommandTemplate, AgentAdapter } from '../../types/agent';
 import { GeminiAdapter } from '../../core/adapters/gemini-adapter';
 import { QwenAdapter } from '../../core/adapters/qwen-adapter';
-import { parseTomlSlashCommand } from '../../utils/toml-templates';
+import { loadCommandTemplates } from '../../utils/template-loader';
+import { collectLegacyCommandFiles } from '../../utils/legacy-command-cleanup';
 
 export default class Init extends Command {
   static description = 'Initialize Clavix in the current project';
@@ -60,7 +61,7 @@ export default class Init extends Command {
               value: 'claude-code',
             },
             {
-              name: 'Cline (.cline/workflows/)',
+              name: 'Cline (.clinerules/workflows/)',
               value: 'cline',
             },
             {
@@ -247,21 +248,24 @@ export default class Init extends Command {
           }
         }
 
-        // Migrate from old command structure if needed (Claude Code only)
-        if (providerName === 'claude-code') {
-          await this.migrateOldCommands(adapter);
-        }
-
         // Generate slash commands
         const generatedTemplates = await this.generateSlashCommands(adapter);
 
+        await this.handleLegacyCommands(adapter, generatedTemplates);
+
         if (adapter.name === 'gemini' || adapter.name === 'qwen') {
           const commandPath = adapter.getCommandPath();
           const isNamespaced = commandPath.endsWith(path.join('commands', 'clavix'));
           const namespace = isNamespaced ? path.basename(commandPath) : undefined;
-          const commandNames = generatedTemplates.map((template) =>
-            isNamespaced ? `/${namespace}:${template.name}` : `/${template.name}`
-          );
+          const commandNames = generatedTemplates.map((template) => {
+            if (isNamespaced) {
+              return `/${namespace}:${template.name}`;
+            }
+
+            const filename = adapter.getTargetFilename(template.name);
+            const slashName = filename.slice(0, -adapter.fileExtension.length);
+            return `/${slashName}`;
+          });
 
           console.log(chalk.green(`    → Registered ${commandNames.join(', ')}`));
           console.log(chalk.gray(`    Commands saved to ${commandPath}`));
@@ -371,97 +375,59 @@ See documentation for template format details.
   }
 
   private async generateSlashCommands(adapter: AgentAdapter): Promise<CommandTemplate[]> {
-    const templateDir = path.join(__dirname, '../../templates/slash-commands', adapter.name);
-    const files = await FileSystem.listFiles(templateDir);
-    const extension = adapter.fileExtension;
-    const commandFiles = files.filter((file) => file.endsWith(extension));
-
-    const templates: CommandTemplate[] = [];
-
-    for (const file of commandFiles) {
-      const content = await FileSystem.readFile(path.join(templateDir, file));
-      const name = file.slice(0, -extension.length);
-
-      if (extension === '.toml') {
-        const parsed = parseTomlSlashCommand(content, name, adapter.name);
-        templates.push({
-          name,
-          content: parsed.prompt,
-          description: parsed.description,
-        });
-      } else {
-        templates.push({
-          name,
-          content,
-          description: this.extractDescription(content),
-        });
-      }
-    }
+    const templates = await loadCommandTemplates(adapter);
 
     await adapter.generateCommands(templates);
     return templates;
   }
 
-  private async injectDocumentation(adapter: AgentAdapter): Promise<void> {
-    // Inject AGENTS.md
-    const agentsContent = DocInjector.getDefaultAgentsContent();
-    await DocInjector.injectBlock('AGENTS.md', this.extractClavixBlock(agentsContent));
-
-    // Inject CLAUDE.md if Claude Code selected
-    if (adapter.name === 'claude-code') {
-      const claudeContent = DocInjector.getDefaultClaudeContent();
-      await DocInjector.injectBlock('CLAUDE.md', this.extractClavixBlock(claudeContent));
-    }
-  }
-
-  private async migrateOldCommands(_adapter: AgentAdapter): Promise<void> {
-    // Check for old command structure (.claude/commands/clavix:*.md)
-    const oldCommandsPath = '.claude/commands';
+  private async handleLegacyCommands(adapter: AgentAdapter, templates: CommandTemplate[]): Promise<void> {
+    const commandNames = templates.map((template) => template.name);
+    const legacyFiles = await collectLegacyCommandFiles(adapter, commandNames);
 
-    if (!await FileSystem.exists(oldCommandsPath)) {
+    if (legacyFiles.length === 0) {
       return;
     }
 
-    try {
-      const files = await FileSystem.listFiles(oldCommandsPath, /^clavix:.*\.md$/);
+    const relativePaths = legacyFiles
+      .map((file) => path.relative(process.cwd(), file))
+      .sort((a, b) => a.localeCompare(b));
 
-      if (files.length === 0) {
-        return;
-      }
-
-      console.log(chalk.cyan('🔄 Migrating old command structure...'));
+    console.log(chalk.gray(`    ⚠ Found ${relativePaths.length} deprecated command file(s):`));
+    for (const file of relativePaths) {
+      console.log(chalk.gray(`      • ${file}`));
+    }
 
-      let removed = 0;
-      for (const file of files) {
-        const filePath = path.join(oldCommandsPath, file);
-        if (await FileSystem.exists(filePath)) {
-          await FileSystem.remove(filePath);
-          console.log(chalk.gray(`  ✓ Removed old command: ${file}`));
-          removed++;
-        }
-      }
+    const { removeLegacy } = await inquirer.prompt([
+      {
+        type: 'confirm',
+        name: 'removeLegacy',
+        message: `Remove deprecated files for ${adapter.displayName}? Functionality is unchanged; filenames are being standardized.`,
+        default: true,
+      },
+    ]);
+
+    if (!removeLegacy) {
+      console.log(chalk.gray('    ⊗ Kept legacy files (deprecated naming retained)'));
+      return;
+    }
 
-      if (removed > 0) {
-        console.log(chalk.green(`  ✓ Migration complete: removed ${removed} old command file(s)`));
-      }
-    } catch {
-      // Non-fatal error - log but continue
-      console.log(chalk.yellow('  ⚠ Could not migrate old commands (non-fatal)'));
+    for (const file of legacyFiles) {
+      await FileSystem.remove(file);
+      console.log(chalk.gray(`    ✓ Removed ${path.relative(process.cwd(), file)}`));
     }
   }
 
-  private extractDescription(content: string): string {
-    const yamlMatch = content.match(/description:\s*(.+)/);
-    if (yamlMatch) {
-      return yamlMatch[1].trim().replace(/^['"]|['"]$/g, '');
-    }
+  private async injectDocumentation(adapter: AgentAdapter): Promise<void> {
+    // Inject AGENTS.md
+    const agentsContent = DocInjector.getDefaultAgentsContent();
+    await DocInjector.injectBlock('AGENTS.md', this.extractClavixBlock(agentsContent));
 
-    const tomlMatch = content.match(/description\s*=\s*['"]?(.+?)['"]?(?:\r?\n|$)/);
-    if (tomlMatch) {
-      return tomlMatch[1].trim().replace(/^['"]|['"]$/g, '');
+    // Inject CLAUDE.md if Claude Code selected
+    if (adapter.name === 'claude-code') {
+      const claudeContent = DocInjector.getDefaultClaudeContent();
+      await DocInjector.injectBlock('CLAUDE.md', this.extractClavixBlock(claudeContent));
     }
-
-    return '';
   }
 
 

src/cli/commands/init.ts

@@ -1,5 +1,6 @@
 import { Command, Flags } from '@oclif/core';
 import chalk from 'chalk';
+import inquirer from 'inquirer';
 import * as fs from 'fs-extra';
 import * as path from 'path';
 import JSON5 from 'json5';
@@ -8,6 +9,7 @@ import { AgentManager } from '../../core/agent-manager';
 import { AgentsMdGenerator } from '../../core/adapters/agents-md-generator';
 import { OctoMdGenerator } from '../../core/adapters/octo-md-generator';
 import { AgentAdapter } from '../../types/agent';
+import { collectLegacyCommandFiles } from '../../utils/legacy-command-cleanup';
 
 export default class Update extends Command {
   static description = 'Update managed blocks and slash commands';
@@ -194,7 +196,8 @@ export default class Update extends Command {
     let updated = 0;
 
     for (const command of templateFiles) {
-      const commandFile = path.join(commandsPath, `${command}${extension}`);
+      const filename = adapter.getTargetFilename(command);
+      const commandFile = path.join(commandsPath, filename);
       const templatePath = path.join(templatesDir, `${command}${extension}`);
 
       const newContent = fs.readFileSync(templatePath, 'utf-8');
@@ -204,21 +207,67 @@ export default class Update extends Command {
 
         if (force || currentContent !== newContent) {
           fs.writeFileSync(commandFile, newContent);
-          this.log(chalk.gray(`  ✓ Updated ${command}.md`));
+          this.log(chalk.gray(`  ✓ Updated ${filename}`));
           updated++;
         } else {
-          this.log(chalk.gray(`  • ${command}.md already up to date`));
+          this.log(chalk.gray(`  • ${filename} already up to date`));
         }
       } else {
         fs.writeFileSync(commandFile, newContent);
-        this.log(chalk.gray(`  ✓ Created ${command}.md`));
+        this.log(chalk.gray(`  ✓ Created ${filename}`));
         updated++;
       }
     }
 
+    updated += await this.handleLegacyCommands(adapter, templateFiles);
+
     return updated;
   }
 
+  private async handleLegacyCommands(adapter: AgentAdapter, commandNames: string[]): Promise<number> {
+    if (commandNames.length === 0) {
+      return 0;
+    }
+
+    const legacyFiles = await collectLegacyCommandFiles(adapter, commandNames);
+
+    if (legacyFiles.length === 0) {
+      return 0;
+    }
+
+    const relativePaths = legacyFiles
+      .map((file) => path.relative(process.cwd(), file))
+      .sort((a, b) => a.localeCompare(b));
+
+    this.log(chalk.gray(`  ⚠ Found ${relativePaths.length} deprecated command file(s):`));
+    for (const file of relativePaths) {
+      this.log(chalk.gray(`    • ${file}`));
+    }
+
+    const { removeLegacy } = await inquirer.prompt([
+      {
+        type: 'confirm',
+        name: 'removeLegacy',
+        message: `Remove deprecated files for ${adapter.displayName}? Functionality is unchanged; filenames are being standardized.`,
+        default: true,
+      },
+    ]);
+
+    if (!removeLegacy) {
+      this.log(chalk.gray('  ⊗ Kept legacy files (deprecated naming retained)'));
+      return 0;
+    }
+
+    let removed = 0;
+    for (const file of legacyFiles) {
+      await fs.remove(file);
+      this.log(chalk.gray(`  ✓ Removed ${path.relative(process.cwd(), file)}`));
+      removed++;
+    }
+
+    return removed;
+  }
+
   private getAgentsContent(): string {
     return `## Clavix Integration
 

src/cli/commands/update.ts

@@ -31,5 +31,9 @@ export class AmpAdapter extends BaseAdapter {
     return this.directory;
   }
 
+  getTargetFilename(name: string): string {
+    return `clavix-${name}${this.fileExtension}`;
+  }
+
   // Uses default formatCommand from BaseAdapter (no special formatting)
 }

src/core/adapters/amp-adapter.ts

@@ -23,6 +23,14 @@ export abstract class BaseAdapter implements AgentAdapter {
   abstract detectProject(): Promise<boolean>;
   abstract getCommandPath(): string;
 
+  /**
+   * Determine the target filename for a generated command
+   * Providers can override to customize filename conventions
+   */
+  getTargetFilename(name: string): string {
+    return `${name}${this.fileExtension}`;
+  }
+
   /**
    * Default validation logic - can be overridden
    * Checks if directory can be created and is writable
@@ -91,7 +99,7 @@ export abstract class BaseAdapter implements AgentAdapter {
       // Generate each command file
       for (const template of templates) {
         const content = this.formatCommand(template);
-        const filename = `${template.name}${this.fileExtension}`;
+        const filename = this.getTargetFilename(template.name);
         const filePath = path.join(commandPath, filename);
         await FileSystem.writeFileAtomic(filePath, content);
       }

src/core/adapters/base-adapter.ts

@@ -16,7 +16,7 @@ import { FileSystem } from '../../utils/file-system';
 export class ClineAdapter extends BaseAdapter {
   readonly name = 'cline';
   readonly displayName = 'Cline';
-  readonly directory = '.cline/workflows';
+  readonly directory = '.clinerules/workflows';
   readonly fileExtension = '.md';
   readonly features = {
     supportsSubdirectories: false,
@@ -28,6 +28,10 @@ export class ClineAdapter extends BaseAdapter {
    * Checks for .cline directory
    */
   async detectProject(): Promise<boolean> {
+    if (await FileSystem.exists('.clinerules')) {
+      return true;
+    }
+
     return await FileSystem.exists('.cline');
   }
 
@@ -38,5 +42,9 @@ export class ClineAdapter extends BaseAdapter {
     return this.directory;
   }
 
+  getTargetFilename(name: string): string {
+    return `clavix-${name}${this.fileExtension}`;
+  }
+
   // Uses default formatCommand and generateCommands from BaseAdapter
 }