prjct-cli 0.11.5 → 0.12.0

package/core/infrastructure/{command-installer.js → command-installer/command-installer.ts}

@@ -1,29 +1,31 @@
-const fs = require('fs').promises
-const path = require('path')
-const os = require('os')
-
 /**
- * CommandInstaller - Installs prjct commands to Claude (Code + Desktop)
- *
- * 100% Claude-focused architecture
- * Handles installation and synchronization of /p:* commands
- * to Claude's native slash command system
- *
- * @version 0.5.0
+ * CommandInstaller Class
+ * Installs prjct commands to Claude (Code + Desktop)
  */
-class CommandInstaller {
+
+import fs from 'fs/promises'
+import path from 'path'
+import os from 'os'
+import type { InstallResult, UninstallResult, CheckResult, SyncResult, GlobalConfigResult } from './types'
+import { installGlobalConfig } from './global-config'
+
+export class CommandInstaller {
+  homeDir: string
+  claudeCommandsPath: string
+  claudeConfigPath: string
+  templatesDir: string
+
   constructor() {
     this.homeDir = os.homedir()
     this.claudeCommandsPath = path.join(this.homeDir, '.claude', 'commands', 'p')
     this.claudeConfigPath = path.join(this.homeDir, '.claude')
-    this.templatesDir = path.join(__dirname, '..', '..', 'templates', 'commands')
+    this.templatesDir = path.join(__dirname, '..', '..', '..', 'templates', 'commands')
   }
 
   /**
    * Detect if Claude is installed
-   * @returns {Promise<boolean>} True if Claude directory exists
    */
-  async detectClaude() {
+  async detectClaude(): Promise<boolean> {
     try {
       await fs.access(this.claudeConfigPath)
       return true
@@ -34,13 +36,12 @@ class CommandInstaller {
 
   /**
    * Get list of command files to install
-   * @returns {Promise<string[]>} Array of command filenames
    */
-  async getCommandFiles() {
+  async getCommandFiles(): Promise<string[]> {
     try {
       const files = await fs.readdir(this.templatesDir)
       return files.filter((f) => f.endsWith('.md'))
-    } catch (error) {
+    } catch {
       // Fallback to core commands if template directory not accessible
       return [
         'init.md',
@@ -68,9 +69,8 @@ class CommandInstaller {
 
   /**
    * Install commands to Claude
-   * @returns {Promise<Object>} Installation results
    */
-  async installCommands() {
+  async installCommands(): Promise<InstallResult> {
     const claudeDetected = await this.detectClaude()
 
     if (!claudeDetected) {
@@ -85,8 +85,8 @@ class CommandInstaller {
       await fs.mkdir(this.claudeCommandsPath, { recursive: true })
 
       const commandFiles = await this.getCommandFiles()
-      const installed = []
-      const errors = []
+      const installed: string[] = []
+      const errors: Array<{ file: string; error: string }> = []
 
       for (const file of commandFiles) {
         try {
@@ -98,7 +98,7 @@ class CommandInstaller {
 
           installed.push(file.replace('.md', ''))
         } catch (error) {
-          errors.push({ file, error: error.message })
+          errors.push({ file, error: (error as Error).message })
         }
       }
 
@@ -111,20 +111,19 @@ class CommandInstaller {
     } catch (error) {
       return {
         success: false,
-        error: error.message,
+        error: (error as Error).message,
       }
     }
   }
 
   /**
    * Uninstall commands from Claude
-   * @returns {Promise<Object>} Uninstallation results
    */
-  async uninstallCommands() {
+  async uninstallCommands(): Promise<UninstallResult> {
     try {
       const commandFiles = await this.getCommandFiles()
-      const uninstalled = []
-      const errors = []
+      const uninstalled: string[] = []
+      const errors: Array<{ file: string; error: string }> = []
 
       for (const file of commandFiles) {
         try {
@@ -132,8 +131,8 @@ class CommandInstaller {
           await fs.unlink(filePath)
           uninstalled.push(file.replace('.md', ''))
         } catch (error) {
-          if (error.code !== 'ENOENT') {
-            errors.push({ file, error: error.message })
+          if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
+            errors.push({ file, error: (error as Error).message })
           }
         }
       }
@@ -153,16 +152,15 @@ class CommandInstaller {
     } catch (error) {
       return {
         success: false,
-        error: error.message,
+        error: (error as Error).message,
       }
     }
   }
 
   /**
    * Check if commands are already installed
-   * @returns {Promise<Object>} Installation status
    */
-  async checkInstallation() {
+  async checkInstallation(): Promise<CheckResult> {
     const claudeDetected = await this.detectClaude()
 
     if (!claudeDetected) {
@@ -196,13 +194,12 @@ class CommandInstaller {
 
   /**
    * Update commands (reinstall with latest templates)
-   * @returns {Promise<Object>} Update results
    */
-  async updateCommands() {
+  async updateCommands(): Promise<InstallResult> {
     // Simply reinstall - will overwrite with latest templates
     console.log('🔄 Updating commands with latest templates...')
     const result = await this.installCommands()
-    if (result.success) {
+    if (result.success && result.installed) {
       console.log(`✅ Updated ${result.installed.length} commands`)
     }
     return result
@@ -210,26 +207,22 @@ class CommandInstaller {
 
   /**
    * Install to all detected editors (alias for installCommands)
-   * @returns {Promise<Object>} Installation results
    */
-  async installToAll() {
+  async installToAll(): Promise<InstallResult> {
     return await this.installCommands()
   }
 
   /**
    * Get installation path for Claude commands
-   * @returns {string} Path to Claude commands directory
    */
-  getInstallPath() {
+  getInstallPath(): string {
     return this.claudeCommandsPath
   }
 
   /**
    * Verify command template exists
-   * @param {string} commandName - Command name (without .md extension)
-   * @returns {Promise<boolean>} True if template exists
    */
-  async verifyTemplate(commandName) {
+  async verifyTemplate(commandName: string): Promise<boolean> {
     try {
       const templatePath = path.join(this.templatesDir, `${commandName}.md`)
       await fs.access(templatePath)
@@ -241,9 +234,8 @@ class CommandInstaller {
 
   /**
    * Sync commands - intelligent update that detects and removes orphans
-   * @returns {Promise<Object>} Sync results with added, updated, removed counts
    */
-  async syncCommands() {
+  async syncCommands(): Promise<SyncResult> {
     const claudeDetected = await this.detectClaude()
 
     if (!claudeDetected) {
@@ -262,7 +254,7 @@ class CommandInstaller {
 
       // Get current state
       const templateFiles = await this.getCommandFiles()
-      let installedFiles = []
+      let installedFiles: string[] = []
 
       try {
         installedFiles = await fs.readdir(this.claudeCommandsPath)
@@ -272,7 +264,7 @@ class CommandInstaller {
         installedFiles = []
       }
 
-      const results = {
+      const results: SyncResult = {
         success: true,
         added: 0,
         updated: 0,
@@ -299,7 +291,7 @@ class CommandInstaller {
             results.updated++
           }
         } catch (error) {
-          results.errors.push({ file, error: error.message })
+          results.errors!.push({ file, error: (error as Error).message })
         }
       }
 
@@ -311,7 +303,7 @@ class CommandInstaller {
     } catch (error) {
       return {
         success: false,
-        error: error.message,
+        error: (error as Error).message,
         added: 0,
         updated: 0,
         removed: 0,
@@ -321,101 +313,8 @@ class CommandInstaller {
 
   /**
    * Install or update global CLAUDE.md configuration
-   * @returns {Promise<Object>} Result with success status and action taken
    */
-  async installGlobalConfig() {
-    const claudeDetected = await this.detectClaude()
-
-    if (!claudeDetected) {
-      return {
-        success: false,
-        error: 'Claude not detected',
-        action: 'skipped',
-      }
-    }
-
-    try {
-      // Ensure ~/.claude directory exists
-      const claudeDir = path.join(require('os').homedir(), '.claude')
-      await fs.mkdir(claudeDir, { recursive: true })
-
-      const globalConfigPath = path.join(claudeDir, 'CLAUDE.md')
-      const templatePath = path.join(__dirname, '../../templates/global/CLAUDE.md')
-
-      // Read template content
-      const templateContent = await fs.readFile(templatePath, 'utf-8')
-
-      // Check if global config already exists
-      let existingContent = ''
-      let fileExists = false
-
-      try {
-        existingContent = await fs.readFile(globalConfigPath, 'utf-8')
-        fileExists = true
-      } catch {
-        // File doesn't exist, will create new
-        fileExists = false
-      }
-
-      if (!fileExists) {
-        // Create new file with full template
-        await fs.writeFile(globalConfigPath, templateContent, 'utf-8')
-        return {
-          success: true,
-          action: 'created',
-          path: globalConfigPath,
-        }
-      } else {
-        // File exists - perform intelligent merge
-        const startMarker = '<!-- prjct:start - DO NOT REMOVE THIS MARKER -->'
-        const endMarker = '<!-- prjct:end - DO NOT REMOVE THIS MARKER -->'
-
-        // Check if markers exist in existing file
-        const hasMarkers =
-          existingContent.includes(startMarker) && existingContent.includes(endMarker)
-
-        if (!hasMarkers) {
-          // No markers - append prjct section at the end
-          const updatedContent = existingContent + '\n\n' + templateContent
-          await fs.writeFile(globalConfigPath, updatedContent, 'utf-8')
-          return {
-            success: true,
-            action: 'appended',
-            path: globalConfigPath,
-          }
-        } else {
-          // Markers exist - replace content between markers
-          const beforeMarker = existingContent.substring(
-            0,
-            existingContent.indexOf(startMarker)
-          )
-          const afterMarker = existingContent.substring(
-            existingContent.indexOf(endMarker) + endMarker.length
-          )
-
-          // Extract prjct section from template
-          const prjctSection = templateContent.substring(
-            templateContent.indexOf(startMarker),
-            templateContent.indexOf(endMarker) + endMarker.length
-          )
-
-          const updatedContent = beforeMarker + prjctSection + afterMarker
-          await fs.writeFile(globalConfigPath, updatedContent, 'utf-8')
-          return {
-            success: true,
-            action: 'updated',
-            path: globalConfigPath,
-          }
-        }
-      }
-    } catch (error) {
-      return {
-        success: false,
-        error: error.message,
-        action: 'failed',
-      }
-    }
+  async installGlobalConfig(): Promise<GlobalConfigResult> {
+    return installGlobalConfig(this.claudeConfigPath, () => this.detectClaude())
   }
 }
-
-module.exports = new CommandInstaller()

package/core/infrastructure/command-installer/global-config.ts

@@ -0,0 +1,106 @@
+/**
+ * Global Config Operations
+ * Install/update global CLAUDE.md configuration
+ */
+
+import fs from 'fs/promises'
+import path from 'path'
+import os from 'os'
+import type { GlobalConfigResult } from './types'
+
+/**
+ * Install or update global CLAUDE.md configuration
+ */
+export async function installGlobalConfig(
+  claudeConfigPath: string,
+  detectClaude: () => Promise<boolean>
+): Promise<GlobalConfigResult> {
+  const claudeDetected = await detectClaude()
+
+  if (!claudeDetected) {
+    return {
+      success: false,
+      error: 'Claude not detected',
+      action: 'skipped',
+    }
+  }
+
+  try {
+    // Ensure ~/.claude directory exists
+    const claudeDir = path.join(os.homedir(), '.claude')
+    await fs.mkdir(claudeDir, { recursive: true })
+
+    const globalConfigPath = path.join(claudeDir, 'CLAUDE.md')
+    const templatePath = path.join(__dirname, '../../../templates/global/CLAUDE.md')
+
+    // Read template content
+    const templateContent = await fs.readFile(templatePath, 'utf-8')
+
+    // Check if global config already exists
+    let existingContent = ''
+    let fileExists = false
+
+    try {
+      existingContent = await fs.readFile(globalConfigPath, 'utf-8')
+      fileExists = true
+    } catch {
+      // File doesn't exist, will create new
+      fileExists = false
+    }
+
+    if (!fileExists) {
+      // Create new file with full template
+      await fs.writeFile(globalConfigPath, templateContent, 'utf-8')
+      return {
+        success: true,
+        action: 'created',
+        path: globalConfigPath,
+      }
+    } else {
+      // File exists - perform intelligent merge
+      const startMarker = '<!-- prjct:start - DO NOT REMOVE THIS MARKER -->'
+      const endMarker = '<!-- prjct:end - DO NOT REMOVE THIS MARKER -->'
+
+      // Check if markers exist in existing file
+      const hasMarkers =
+        existingContent.includes(startMarker) && existingContent.includes(endMarker)
+
+      if (!hasMarkers) {
+        // No markers - append prjct section at the end
+        const updatedContent = existingContent + '\n\n' + templateContent
+        await fs.writeFile(globalConfigPath, updatedContent, 'utf-8')
+        return {
+          success: true,
+          action: 'appended',
+          path: globalConfigPath,
+        }
+      } else {
+        // Markers exist - replace content between markers
+        const beforeMarker = existingContent.substring(0, existingContent.indexOf(startMarker))
+        const afterMarker = existingContent.substring(
+          existingContent.indexOf(endMarker) + endMarker.length
+        )
+
+        // Extract prjct section from template
+        const prjctSection = templateContent.substring(
+          templateContent.indexOf(startMarker),
+          templateContent.indexOf(endMarker) + endMarker.length
+        )
+
+        const updatedContent = beforeMarker + prjctSection + afterMarker
+        await fs.writeFile(globalConfigPath, updatedContent, 'utf-8')
+        return {
+          success: true,
+          action: 'updated',
+          path: globalConfigPath,
+        }
+      }
+    }
+  } catch (error) {
+    return {
+      success: false,
+      error: (error as Error).message,
+      action: 'failed',
+    }
+  }
+}

package/core/infrastructure/command-installer/index.ts

@@ -0,0 +1,25 @@
+/**
+ * CommandInstaller - Installs prjct commands to Claude (Code + Desktop)
+ *
+ * 100% Claude-focused architecture
+ * Handles installation and synchronization of /p:* commands
+ * to Claude's native slash command system
+ *
+ * @version 0.5.0
+ */
+
+export type {
+  InstallResult,
+  UninstallResult,
+  CheckResult,
+  SyncResult,
+  GlobalConfigResult,
+} from './types'
+
+export { installGlobalConfig } from './global-config'
+export { CommandInstaller } from './command-installer'
+
+import { CommandInstaller } from './command-installer'
+
+const commandInstaller = new CommandInstaller()
+export default commandInstaller

package/core/infrastructure/command-installer/types.ts

@@ -0,0 +1,41 @@
+/**
+ * Command Installer Types
+ */
+
+export interface InstallResult {
+  success: boolean
+  installed?: string[]
+  errors?: Array<{ file: string; error: string }>
+  path?: string
+  error?: string
+}
+
+export interface UninstallResult {
+  success: boolean
+  uninstalled?: string[]
+  errors?: Array<{ file: string; error: string }>
+  error?: string
+}
+
+export interface CheckResult {
+  installed: boolean
+  claudeDetected: boolean
+  commands?: string[]
+  path?: string
+}
+
+export interface SyncResult {
+  success: boolean
+  added: number
+  updated: number
+  removed: number
+  errors?: Array<{ file: string; error: string }>
+  error?: string
+}
+
+export interface GlobalConfigResult {
+  success: boolean
+  action: string
+  path?: string
+  error?: string
+}

package/core/infrastructure/command-installer.ts

@@ -0,0 +1,8 @@
+/**
+ * CommandInstaller
+ * Re-exports from command-installer/index.ts for backwards compatibility.
+ */
+
+import commandInstaller from './command-installer/index'
+export * from './command-installer/index'
+export default commandInstaller