prjct-cli 0.31.0 → 0.34.0

package/core/agentic/ground-truth.ts

@@ -17,6 +17,7 @@ import os from 'os'
 import { execSync } from 'child_process'
 
 import type { GroundTruthContext, VerificationResult, Verifier } from '../types'
+import { isNotFoundError } from '../types/fs'
 
 // =============================================================================
 // Utilities
@@ -103,10 +104,14 @@ export async function verifyDone(context: GroundTruthContext): Promise<Verificat
         actual.durationFormatted = formatDuration(actual.durationMs as number)
       }
     }
-  } catch {
-    actual.nowExists = false
-    warnings.push('now.md does not exist')
-    recommendations.push('Create a task with /p:now "task"')
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.nowExists = false
+      warnings.push('now.md does not exist')
+      recommendations.push('Create a task with /p:now "task"')
+    } else {
+      throw error
+    }
   }
 
   // 2. Verify next.md for auto-start
@@ -116,9 +121,13 @@ export async function verifyDone(context: GroundTruthContext): Promise<Verificat
     actual.nextExists = true
     const tasks = nextContent.match(/- \[ \]/g) || []
     actual.pendingTasks = tasks.length
-  } catch {
-    actual.nextExists = false
-    actual.pendingTasks = 0
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.nextExists = false
+      actual.pendingTasks = 0
+    } else {
+      throw error
+    }
   }
 
   // 3. Verify metrics.md is writable
@@ -126,9 +135,13 @@ export async function verifyDone(context: GroundTruthContext): Promise<Verificat
   try {
     await fs.access(path.dirname(metricsPath), fs.constants.W_OK)
     actual.metricsWritable = true
-  } catch {
-    actual.metricsWritable = false
-    warnings.push('Cannot write to metrics directory')
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.metricsWritable = false
+      warnings.push('Cannot write to metrics directory')
+    } else {
+      throw error
+    }
   }
 
   return {
@@ -161,9 +174,9 @@ export async function verifyShip(context: GroundTruthContext): Promise<Verificat
       warnings.push(`${actual.uncommittedFiles} uncommitted file(s)`)
       recommendations.push('Commit changes before shipping')
     }
-  } catch {
+  } catch (error) {
+    // Git errors (not a repo, git not installed) are not blockers
     actual.gitAvailable = false
-    // Not a git repo or git not available - not a blocker
   }
 
   // 2. Check for package.json version (if exists)
@@ -173,8 +186,15 @@ export async function verifyShip(context: GroundTruthContext): Promise<Verificat
     const pkg = JSON.parse(pkgContent)
     actual.currentVersion = pkg.version
     actual.hasPackageJson = true
-  } catch {
-    actual.hasPackageJson = false
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.hasPackageJson = false
+    } else if (error instanceof SyntaxError) {
+      actual.hasPackageJson = false
+      warnings.push('package.json has invalid JSON')
+    } else {
+      throw error
+    }
   }
 
   // 3. Check shipped.md for duplicate feature names
@@ -193,8 +213,12 @@ export async function verifyShip(context: GroundTruthContext): Promise<Verificat
         recommendations.push('Use a different feature name or skip /p:ship')
       }
     }
-  } catch {
-    actual.shippedExists = false
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.shippedExists = false
+    } else {
+      throw error
+    }
   }
 
   // 4. Check for test failures (if test script exists)
@@ -205,8 +229,12 @@ export async function verifyShip(context: GroundTruthContext): Promise<Verificat
       actual.hasTestScript = !!pkg.scripts?.test
       // Note: We don't run tests here, just check if they exist
       // Running tests is the user's responsibility
-    } catch {
-      actual.hasTestScript = false
+    } catch (error) {
+      if (isNotFoundError(error) || error instanceof SyntaxError) {
+        actual.hasTestScript = false
+      } else {
+        throw error
+      }
     }
   }
 
@@ -240,9 +268,13 @@ export async function verifyFeature(context: GroundTruthContext): Promise<Verifi
       warnings.push(`Queue nearly full (${actual.taskCount}/100 tasks)`)
       recommendations.push('Complete some tasks before adding more')
     }
-  } catch {
-    actual.nextExists = false
-    actual.taskCount = 0
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.nextExists = false
+      actual.taskCount = 0
+    } else {
+      throw error
+    }
   }
 
   // 2. Check roadmap.md for duplicate features
@@ -259,8 +291,12 @@ export async function verifyFeature(context: GroundTruthContext): Promise<Verifi
         recommendations.push('Check roadmap for duplicates with /p:roadmap')
       }
     }
-  } catch {
-    actual.roadmapExists = false
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.roadmapExists = false
+    } else {
+      throw error
+    }
   }
 
   // 3. Check if there's an active task (should complete first?)
@@ -272,8 +308,12 @@ export async function verifyFeature(context: GroundTruthContext): Promise<Verifi
     if (actual.hasActiveTask) {
       recommendations.push('Consider completing current task first with /p:done')
     }
-  } catch {
-    actual.hasActiveTask = false
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.hasActiveTask = false
+    } else {
+      throw error
+    }
   }
 
   return {
@@ -311,9 +351,13 @@ export async function verifyNow(context: GroundTruthContext): Promise<Verificati
       warnings.push(`Replacing existing task: "${taskPreview}..."`)
       recommendations.push('Use /p:done first to track completion')
     }
-  } catch {
-    actual.nowExists = false
-    actual.hasActiveTask = false
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.nowExists = false
+      actual.hasActiveTask = false
+    } else {
+      throw error
+    }
   }
 
   // 2. Check next.md for available tasks
@@ -326,8 +370,12 @@ export async function verifyNow(context: GroundTruthContext): Promise<Verificati
     if (!context.params.task && pendingTasks > 0) {
       recommendations.push(`${pendingTasks} tasks available in queue`)
     }
-  } catch {
-    actual.pendingTasks = 0
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.pendingTasks = 0
+    } else {
+      throw error
+    }
   }
 
   return {
@@ -355,8 +403,15 @@ export async function verifyInit(context: GroundTruthContext): Promise<Verificat
     actual.existingConfig = JSON.parse(configContent)
     warnings.push('Project already initialized')
     recommendations.push('Use /p:analyze to refresh analysis or delete .prjct/ to reinitialize')
-  } catch {
-    actual.alreadyInitialized = false
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.alreadyInitialized = false
+    } else if (error instanceof SyntaxError) {
+      actual.alreadyInitialized = false
+      warnings.push('Existing config has invalid JSON')
+    } else {
+      throw error
+    }
   }
 
   // 2. Check if global storage path is writable
@@ -364,13 +419,19 @@ export async function verifyInit(context: GroundTruthContext): Promise<Verificat
   try {
     await fs.access(globalPath, fs.constants.W_OK)
     actual.globalPathWritable = true
-  } catch {
-    try {
-      // Try to create it
-      await fs.mkdir(globalPath, { recursive: true })
-      actual.globalPathWritable = true
-      actual.globalPathCreated = true
-    } catch {
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      try {
+        // Try to create it
+        await fs.mkdir(globalPath, { recursive: true })
+        actual.globalPathWritable = true
+        actual.globalPathCreated = true
+      } catch (mkdirError) {
+        actual.globalPathWritable = false
+        warnings.push('Cannot write to ~/.prjct-cli')
+        recommendations.push('Check directory permissions')
+      }
+    } else {
       actual.globalPathWritable = false
       warnings.push('Cannot write to ~/.prjct-cli')
       recommendations.push('Check directory permissions')
@@ -400,11 +461,20 @@ export async function verifySync(context: GroundTruthContext): Promise<Verificat
     const configContent = await fs.readFile(configPath, 'utf-8')
     actual.hasConfig = true
     actual.config = JSON.parse(configContent)
-  } catch {
-    actual.hasConfig = false
-    warnings.push('Project not initialized')
-    recommendations.push('Run /p:init first')
-    return { verified: false, actual, warnings, recommendations }
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.hasConfig = false
+      warnings.push('Project not initialized')
+      recommendations.push('Run /p:init first')
+      return { verified: false, actual, warnings, recommendations }
+    } else if (error instanceof SyntaxError) {
+      actual.hasConfig = false
+      warnings.push('Config file has invalid JSON')
+      recommendations.push('Delete .prjct/ and run /p:init')
+      return { verified: false, actual, warnings, recommendations }
+    } else {
+      throw error
+    }
   }
 
   // 2. Check if global storage exists
@@ -413,10 +483,14 @@ export async function verifySync(context: GroundTruthContext): Promise<Verificat
   try {
     await fs.access(globalProjectPath)
     actual.globalStorageExists = true
-  } catch {
-    actual.globalStorageExists = false
-    warnings.push('Global storage missing')
-    recommendations.push('Run /p:init to recreate')
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.globalStorageExists = false
+      warnings.push('Global storage missing')
+      recommendations.push('Run /p:init to recreate')
+    } else {
+      throw error
+    }
   }
 
   return {
@@ -444,8 +518,11 @@ export async function verifyAnalyze(context: GroundTruthContext): Promise<Verifi
     try {
       await fs.access(path.join(context.projectPath, file))
       ;(actual.detectedFiles as string[]).push(file)
-    } catch {
-      // File doesn't exist
+    } catch (error) {
+      // ENOENT expected - file doesn't exist
+      if (!isNotFoundError(error)) {
+        throw error
+      }
     }
   }
 
@@ -464,8 +541,11 @@ export async function verifyAnalyze(context: GroundTruthContext): Promise<Verifi
       if (stat.isDirectory()) {
         ;(actual.detectedSrcDirs as string[]).push(dir)
       }
-    } catch {
-      // Directory doesn't exist
+    } catch (error) {
+      // ENOENT expected - directory doesn't exist
+      if (!isNotFoundError(error)) {
+        throw error
+      }
     }
   }
 
@@ -496,9 +576,13 @@ export async function verifySpec(context: GroundTruthContext): Promise<Verificat
     const files = await fs.readdir(specsPath)
     actual.existingSpecs = files.filter((f) => f.endsWith('.md'))
     actual.specCount = (actual.existingSpecs as string[]).length
-  } catch {
-    actual.specsExists = false
-    actual.specCount = 0
+  } catch (error) {
+    if (isNotFoundError(error)) {
+      actual.specsExists = false
+      actual.specCount = 0
+    } else {
+      throw error
+    }
   }
 
   // 2. Check for duplicate spec name

package/core/agentic/prompt-builder.ts

@@ -11,6 +11,7 @@
 
 import fs from 'fs'
 import path from 'path'
+import { isNotFoundError } from '../types/fs'
 import { stateStorage, queueStorage } from '../storage'
 import { outcomeAnalyzer } from '../outcomes'
 import type {
@@ -85,8 +86,11 @@ class PromptBuilder {
           checklists[name] = content
         }
       }
-    } catch {
+    } catch (error) {
       // Silent fail - checklists are optional enhancement
+      if (!isNotFoundError(error)) {
+        console.error(`Checklist loading warning: ${(error as Error).message}`)
+      }
     }
 
     this._checklistsCache = checklists
@@ -118,8 +122,11 @@ class PromptBuilder {
 
       this._stateCache.set(projectId, { state, timestamp: Date.now() })
       return state
-    } catch {
-      return null
+    } catch (error) {
+      if (isNotFoundError(error) || error instanceof SyntaxError) {
+        return null
+      }
+      throw error
     }
   }
 
@@ -175,8 +182,11 @@ class PromptBuilder {
         }
         parts.push('')
       }
-    } catch {
-      // Outcomes not available yet
+    } catch (error) {
+      // Outcomes not available yet - expected for new projects
+      if (!isNotFoundError(error) && !(error instanceof SyntaxError)) {
+        console.error(`Outcome detection warning: ${(error as Error).message}`)
+      }
     }
 
     parts.push('---')
@@ -214,8 +224,11 @@ class PromptBuilder {
       if (fs.existsSync(routingPath)) {
         this._checklistRoutingCache = fs.readFileSync(routingPath, 'utf-8')
       }
-    } catch {
-      // Silent fail
+    } catch (error) {
+      // Silent fail - checklist routing is optional
+      if (!isNotFoundError(error)) {
+        console.error(`Checklist routing warning: ${(error as Error).message}`)
+      }
     }
 
     return this._checklistRoutingCache || null

package/core/agentic/smart-context.ts

@@ -170,7 +170,7 @@ class SmartContext {
           confidence: pattern.confidence,
         })
       }
-    } catch {
+    } catch (_error) {
       // Outcomes not available
     }
 

package/core/agentic/template-loader.ts

@@ -87,7 +87,7 @@ export async function load(commandName: string): Promise<ParsedTemplate> {
     cacheOrder.push(commandName)
 
     return parsed
-  } catch {
+  } catch (_error) {
     throw TemplateError.notFound(commandName)
   }
 }

package/core/agentic/tool-registry.ts

@@ -65,7 +65,8 @@ const toolRegistry: ToolRegistryInterface = {
 toolRegistry.register('Read', async (filePath: unknown): Promise<string | null> => {
   try {
     return await fs.readFile(filePath as string, 'utf-8')
-  } catch {
+  } catch (_error) {
+    // File not found or read error - return null (expected)
     return null
   }
 })
@@ -75,7 +76,8 @@ toolRegistry.register('Write', async (filePath: unknown, content: unknown): Prom
   try {
     await fs.writeFile(filePath as string, content as string, 'utf-8')
     return true
-  } catch {
+  } catch (_error) {
+    // Write error - return false (expected)
     return false
   }
 })

package/core/bus/bus.ts

@@ -188,7 +188,7 @@ class EventBus {
       // Append event
       const line = JSON.stringify(eventData) + '\n'
       await fs.appendFile(eventsPath, line)
-    } catch {
+    } catch (_error) {
       // Silently fail - logging should not break functionality
     }
   }

package/core/commands/cleanup.ts

@@ -6,6 +6,7 @@
 
 import path from 'path'
 
+import { isNotFoundError } from '../types/fs'
 import type { CommandResult, CleanupOptions } from '../types'
 import {
   pathManager,
@@ -44,8 +45,11 @@ export async function cleanupMemory(projectPath: string): Promise<{
           results.freedSpace += sizeMB
         }
       }
-    } catch {
-      // skip
+    } catch (error) {
+      // Skip file if not found, otherwise log unexpected errors
+      if (!isNotFoundError(error)) {
+        console.error(`Cleanup warning for ${filePath}: ${(error as Error).message}`)
+      }
     }
   }
 
@@ -100,8 +104,12 @@ export async function cleanup(
       } else {
         cleaned.push('Memory: No cleanup needed')
       }
-    } catch {
-      cleaned.push('Memory: No file found')
+    } catch (error) {
+      if (isNotFoundError(error)) {
+        cleaned.push('Memory: No file found')
+      } else {
+        cleaned.push(`Memory: Error - ${(error as Error).message}`)
+      }
     }
 
     // Clean ideas using ideasStorage
@@ -112,8 +120,12 @@ export async function cleanup(
       } else {
         cleaned.push('Ideas: No cleanup needed')
       }
-    } catch {
-      cleaned.push('Ideas: No file found')
+    } catch (error) {
+      if (isNotFoundError(error)) {
+        cleaned.push('Ideas: No file found')
+      } else {
+        cleaned.push(`Ideas: Error - ${(error as Error).message}`)
+      }
     }
 
     // Check queue for completed tasks using queueStorage
@@ -128,8 +140,12 @@ export async function cleanup(
       } else {
         cleaned.push('Queue: No completed tasks')
       }
-    } catch {
-      cleaned.push('Queue: No file found')
+    } catch (error) {
+      if (isNotFoundError(error)) {
+        cleaned.push('Queue: No file found')
+      } else {
+        cleaned.push(`Queue: Error - ${(error as Error).message}`)
+      }
     }
 
     await cleanupMemoryInternal(projectPath)

package/core/commands/planning.ts

@@ -281,7 +281,8 @@ export class PlanningCommands extends PrjctCommandsBase {
       let planContent: string
       try {
         planContent = await fileHelper.readFile(planPath)
-      } catch {
+      } catch (_error) {
+        // No plan file - expected for projects without architect mode
         return {
           success: false,
           message:
@@ -465,7 +466,8 @@ Generated: ${new Date().toLocaleString()}
           console.log('═'.repeat(50) + '\n')
 
           return { success: true, specs }
-        } catch {
+        } catch (_error) {
+          // No specs directory - expected for new projects
           out.warn('no specs directory')
           return { success: true, specs: [] }
         }

package/core/commands/setup.ts

@@ -4,10 +4,10 @@
 
 import path from 'path'
 import fs from 'fs'
-import os from 'os'
 import chalk from 'chalk'
 
 import commandInstaller from '../infrastructure/command-installer'
+import pathManager from '../infrastructure/path-manager'
 import type { CommandResult, SetupOptions, MigrateOptions } from '../types'
 import { PrjctCommandsBase } from './base'
 import { VERSION } from '../utils/version'
@@ -126,8 +126,8 @@ export class SetupCommands extends PrjctCommandsBase {
    */
   async installStatusLine(): Promise<{ success: boolean; error?: string }> {
     try {
-      const claudeDir = path.join(os.homedir(), '.claude')
-      const settingsPath = path.join(claudeDir, 'settings.json')
+      const claudeDir = pathManager.getClaudeDir()
+      const settingsPath = pathManager.getClaudeSettingsPath()
       const statusLinePath = path.join(claudeDir, 'prjct-statusline.sh')
 
       // Version is embedded at install time
@@ -194,7 +194,7 @@ echo "⚡ prjct"
       if (fs.existsSync(settingsPath)) {
         try {
           settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'))
-        } catch {
+        } catch (_error) {
           // Invalid JSON, start fresh
         }
       }

package/core/commands/shipping.ts

@@ -5,6 +5,7 @@
 
 import path from 'path'
 
+import { isNotFoundError } from '../types/fs'
 import memorySystem from '../agentic/memory-system'
 import type { CommandResult } from '../types'
 import { detectProjectCommands } from '../utils/project-commands'
@@ -133,7 +134,11 @@ export class ShippingCommands extends PrjctCommandsBase {
 
       const { exitCode } = await this._runWithExitCode(detected.lint.command)
       return { success: exitCode === 0, message: exitCode === 0 ? 'passed' : 'failed' }
-    } catch {
+    } catch (error) {
+      // Lint detection/execution failed - skip gracefully
+      if (isNotFoundError(error)) {
+        return { success: true, message: 'skipped (lint not found)' }
+      }
       return { success: true, message: 'skipped (lint detection failed)' }
     }
   }
@@ -148,7 +153,11 @@ export class ShippingCommands extends PrjctCommandsBase {
 
       const { exitCode } = await this._runWithExitCode(detected.test.command)
       return { success: exitCode === 0, message: exitCode === 0 ? 'passed' : 'failed' }
-    } catch {
+    } catch (error) {
+      // Test detection/execution failed - skip gracefully
+      if (isNotFoundError(error)) {
+        return { success: true, message: 'skipped (tests not found)' }
+      }
       return { success: true, message: 'skipped (test detection failed)' }
     }
   }
@@ -168,8 +177,12 @@ export class ShippingCommands extends PrjctCommandsBase {
         await fileHelper.writeJson(pkgPath, pkg)
       }
       return newVersion
-    } catch {
-      return '0.0.1'
+    } catch (error) {
+      // No package.json or parse error - return default version
+      if (isNotFoundError(error) || error instanceof SyntaxError) {
+        return '0.0.1'
+      }
+      throw error
     }
   }
 
@@ -185,8 +198,13 @@ export class ShippingCommands extends PrjctCommandsBase {
       const updated = changelog.replace('# Changelog\n\n', `# Changelog\n\n${entry}`)
 
       await fileHelper.writeFile(changelogPath, updated)
-    } catch {
-      console.error('   Warning: Could not update CHANGELOG')
+    } catch (error) {
+      // CHANGELOG doesn't exist or can't be written - warn but continue
+      if (isNotFoundError(error)) {
+        console.error('   Warning: CHANGELOG.md not found')
+      } else {
+        console.error('   Warning: Could not update CHANGELOG')
+      }
     }
   }
 
@@ -202,7 +220,11 @@ export class ShippingCommands extends PrjctCommandsBase {
       await toolRegistry.get('Bash')!(`git commit -m "${commitMsg.replace(/"/g, '\\"')}"`)
 
       return { success: true, message: 'Committed' }
-    } catch {
+    } catch (error) {
+      // Git commit failed - likely no changes or not a repo
+      if (isNotFoundError(error)) {
+        return { success: false, message: 'Git not found' }
+      }
       return { success: false, message: 'No changes to commit' }
     }
   }
@@ -214,7 +236,11 @@ export class ShippingCommands extends PrjctCommandsBase {
     try {
       await toolRegistry.get('Bash')!('git push')
       return { success: true, message: 'Pushed to remote' }
-    } catch {
+    } catch (error) {
+      // Git push failed - no remote, auth issue, or git not found
+      if (isNotFoundError(error)) {
+        return { success: false, message: 'Git not found' }
+      }
       return { success: false, message: 'Push failed (no remote or auth issue)' }
     }
   }