prjct-cli 1.6.8 → 1.6.10

package/core/services/context-generator.ts

@@ -12,50 +12,25 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import pathManager from '../infrastructure/path-manager'
+import type {
+  ContextGeneratorConfig,
+  GitData,
+  ProjectCommands,
+  ProjectStats,
+  SyncAgentInfo,
+} from '../types'
 import { type ContextSources, cite, defaultSources } from '../utils/citations'
 import * as dateHelper from '../utils/date-helper'
 import { mergePreservedSections, validatePreserveBlocks } from '../utils/preserve-sections'
 import { NestedContextResolver } from './nested-context-resolver'
 
-// ============================================================================
-// TYPES
-// ============================================================================
-
-export interface GitData {
-  branch: string
-  commits: number
-}
-
-export interface ProjectStats {
-  name: string
-  version: string
-  ecosystem: string
-  projectType: string
-  fileCount: number
-  languages: string[]
-  frameworks: string[]
-}
-
-export interface Commands {
-  install: string
-  dev: string
-  test: string
-  build: string
-  lint: string
-  format: string
-}
-
-export interface AgentInfo {
-  name: string
-  type: 'workflow' | 'domain'
-  skill?: string
-}
-
-export interface ContextGeneratorConfig {
-  projectId: string
-  projectPath: string
-  globalPath: string
-}
+export type {
+  ContextGeneratorConfig,
+  GitData,
+  ProjectCommands,
+  ProjectStats,
+  SyncAgentInfo,
+} from '../types'
 
 // ============================================================================
 // CONTEXT FILE GENERATOR
@@ -103,8 +78,8 @@ export class ContextFileGenerator {
   async generate(
     git: GitData,
     stats: ProjectStats,
-    commands: Commands,
-    agents: AgentInfo[],
+    commands: ProjectCommands,
+    agents: SyncAgentInfo[],
     sources?: ContextSources
   ): Promise<string[]> {
     const contextPath = path.join(this.config.globalPath, 'context')
@@ -138,8 +113,8 @@ export class ContextFileGenerator {
     contextPath: string,
     git: GitData,
     stats: ProjectStats,
-    commands: Commands,
-    agents: AgentInfo[],
+    commands: ProjectCommands,
+    agents: SyncAgentInfo[],
     sources?: ContextSources
   ): Promise<void> {
     const workflowAgents = agents.filter((a) => a.type === 'workflow').map((a) => a.name)
@@ -348,8 +323,8 @@ ${
   async generateMonorepoContexts(
     git: GitData,
     stats: ProjectStats,
-    commands: Commands,
-    agents: AgentInfo[]
+    commands: ProjectCommands,
+    agents: SyncAgentInfo[]
   ): Promise<string[]> {
     const monoInfo = await pathManager.detectMonorepo(this.config.projectPath)
 
@@ -394,8 +369,8 @@ ${
     resolvedCtx: { content: string; sources: string[]; overrides: string[] },
     git: GitData,
     stats: ProjectStats,
-    commands: Commands,
-    agents: AgentInfo[]
+    commands: ProjectCommands,
+    agents: SyncAgentInfo[]
   ): Promise<string> {
     const workflowAgents = agents.filter((a) => a.type === 'workflow').map((a) => a.name)
     const domainAgents = agents.filter((a) => a.type === 'domain').map((a) => a.name)

package/core/services/diff-generator.ts

@@ -9,39 +9,21 @@
  */
 
 import chalk from 'chalk'
-
-// =============================================================================
-// Types
-// =============================================================================
-
-export interface DiffSection {
-  name: string
-  type: 'added' | 'modified' | 'removed' | 'unchanged'
-  before?: string
-  after?: string
-  lineCount: number
-}
-
-export interface PreservedInfo {
-  name: string
-  lineCount: number
-}
-
-export interface SyncDiff {
-  hasChanges: boolean
-  added: DiffSection[]
-  modified: DiffSection[]
-  removed: DiffSection[]
-  preserved: PreservedInfo[]
-  tokensBefore: number
-  tokensAfter: number
-  tokenDelta: number
-}
-
-export interface DiffOptions {
-  showFullDiff?: boolean
-  colorize?: boolean
-}
+import type {
+  DiffOptions,
+  DiffSection,
+  ParsedMarkdownSection,
+  PreservedInfo,
+  SyncDiff,
+} from '../types'
+
+export type {
+  DiffOptions,
+  DiffSection,
+  ParsedMarkdownSection,
+  PreservedInfo,
+  SyncDiff,
+} from '../types'
 
 // =============================================================================
 // Constants
@@ -64,20 +46,13 @@ export function estimateTokens(content: string): number {
 // Section Parsing
 // =============================================================================
 
-interface ParsedSection {
-  name: string
-  content: string
-  startLine: number
-  endLine: number
-}
-
 /**
  * Parse markdown into sections based on headers
  */
-export function parseMarkdownSections(content: string): ParsedSection[] {
+export function parseMarkdownSections(content: string): ParsedMarkdownSection[] {
   const lines = content.split('\n')
-  const sections: ParsedSection[] = []
-  let currentSection: ParsedSection | null = null
+  const sections: ParsedMarkdownSection[] = []
+  let currentSection: ParsedMarkdownSection | null = null
 
   for (let i = 0; i < lines.length; i++) {
     const line = lines[i]

package/core/services/stack-detector.ts

@@ -11,25 +11,9 @@
 
 import fs from 'node:fs/promises'
 import path from 'node:path'
+import type { StackDetection, StackPackageJson } from '../types'
 
-// ============================================================================
-// TYPES
-// ============================================================================
-
-export interface StackDetection {
-  hasFrontend: boolean
-  hasBackend: boolean
-  hasDatabase: boolean
-  hasDocker: boolean
-  hasTesting: boolean
-  frontendType: 'web' | 'mobile' | 'both' | null
-  frameworks: string[]
-}
-
-interface PackageJson {
-  dependencies?: Record<string, string>
-  devDependencies?: Record<string, string>
-}
+export type { StackDetection, StackPackageJson } from '../types'
 
 // ============================================================================
 // STACK DETECTOR
@@ -148,7 +132,7 @@ export class StackDetector {
    */
   private detectTesting(
     deps: Record<string, string>,
-    pkg: PackageJson,
+    pkg: StackPackageJson,
     stack: StackDetection
   ): void {
     const testingFrameworks = [
@@ -219,7 +203,7 @@ export class StackDetector {
   /**
    * Read and parse package.json
    */
-  private async readPackageJson(): Promise<PackageJson | null> {
+  private async readPackageJson(): Promise<StackPackageJson | null> {
     try {
       const pkgPath = path.join(this.projectPath, 'package.json')
       const content = await fs.readFile(pkgPath, 'utf-8')

package/core/services/sync-service.ts

@@ -32,6 +32,18 @@ import commandInstaller from '../infrastructure/command-installer'
 import configManager from '../infrastructure/config-manager'
 import pathManager from '../infrastructure/path-manager'
 import { metricsStorage } from '../storage/metrics-storage'
+import type {
+  AIToolResult,
+  GitData,
+  ProjectCommands,
+  ProjectStats,
+  ProjectSyncResult,
+  StackDetection,
+  SyncAgentInfo,
+  SyncMetrics,
+  SyncOptions,
+  VerificationReport,
+} from '../types'
 import { type ContextSources, defaultSources, type SourceInfo } from '../utils/citations'
 import * as dateHelper from '../utils/date-helper'
 import log from '../utils/logger'
@@ -39,95 +51,11 @@ import { ContextFileGenerator } from './context-generator'
 import type { SyncDiff } from './diff-generator'
 import { localStateGenerator } from './local-state-generator'
 import { skillInstaller } from './skill-installer'
-import { type StackDetection, StackDetector } from './stack-detector'
-import { syncVerifier, type VerificationReport } from './sync-verifier'
+import { StackDetector } from './stack-detector'
+import { syncVerifier } from './sync-verifier'
 
 const execAsync = promisify(exec)
 
-// ============================================================================
-// TYPES
-// ============================================================================
-
-interface GitData {
-  branch: string
-  commits: number
-  contributors: number
-  hasChanges: boolean
-  stagedFiles: string[]
-  modifiedFiles: string[]
-  untrackedFiles: string[]
-  recentCommits: { hash: string; message: string; date: string }[]
-  weeklyCommits: number
-}
-
-interface ProjectStats {
-  fileCount: number
-  version: string
-  name: string
-  ecosystem: string
-  projectType: string
-  languages: string[]
-  frameworks: string[]
-}
-
-interface Commands {
-  install: string
-  run: string
-  test: string
-  build: string
-  dev: string
-  lint: string
-  format: string
-}
-
-interface AgentInfo {
-  name: string
-  type: 'workflow' | 'domain'
-  skill?: string
-}
-
-interface AIToolResult {
-  toolId: string
-  outputFile: string
-  success: boolean
-}
-
-interface SyncMetrics {
-  duration: number // Sync duration in ms
-  originalSize: number // Estimated tokens before compression
-  filteredSize: number // Actual tokens in context files
-  compressionRate: number // Percentage saved
-}
-
-interface SyncResult {
-  success: boolean
-  projectId: string
-  cliVersion: string
-  git: GitData
-  stats: ProjectStats
-  commands: Commands
-  stack: StackDetection
-  agents: AgentInfo[]
-  skills: { agent: string; skill: string }[]
-  skillsInstalled: { name: string; agent: string; status: 'installed' | 'skipped' | 'error' }[]
-  contextFiles: string[]
-  aiTools: AIToolResult[]
-  syncMetrics?: SyncMetrics
-  verification?: VerificationReport
-  error?: string
-  // Preview mode fields
-  isPreview?: boolean
-  previewDiff?: SyncDiff
-}
-
-interface SyncOptions {
-  aiTools?: string[] // AI tools to generate context for (default: claude, cursor)
-  preview?: boolean // If true, return diff without applying changes
-  skipConfirmation?: boolean // If true, apply changes without confirmation (--yes flag)
-  packagePath?: string // For monorepo: sync only this package path
-  packageName?: string // For monorepo: package name for display
-}
-
 // ============================================================================
 // SYNC SERVICE
 // ============================================================================
@@ -145,7 +73,10 @@ class SyncService {
   /**
    * Main sync method - does everything in one call
    */
-  async sync(projectPath: string = process.cwd(), options: SyncOptions = {}): Promise<SyncResult> {
+  async sync(
+    projectPath: string = process.cwd(),
+    options: SyncOptions = {}
+  ): Promise<ProjectSyncResult> {
     this.projectPath = projectPath
     const startTime = Date.now()
 
@@ -486,8 +417,8 @@ class SyncService {
   // COMMAND DETECTION
   // ==========================================================================
 
-  private async detectCommands(): Promise<Commands> {
-    const commands: Commands = {
+  private async detectCommands(): Promise<ProjectCommands> {
+    const commands: ProjectCommands = {
       install: 'npm install',
       run: 'npm run',
       test: 'npm test',
@@ -552,7 +483,7 @@ class SyncService {
   // SOURCE CITATIONS
   // ==========================================================================
 
-  private buildSources(stats: ProjectStats, commands: Commands): ContextSources {
+  private buildSources(stats: ProjectStats, commands: ProjectCommands): ContextSources {
     const sources = defaultSources()
 
     // Determine ecosystem source file
@@ -609,8 +540,11 @@ class SyncService {
   // AGENT GENERATION
   // ==========================================================================
 
-  private async generateAgents(stack: StackDetection, stats: ProjectStats): Promise<AgentInfo[]> {
-    const agents: AgentInfo[] = []
+  private async generateAgents(
+    stack: StackDetection,
+    stats: ProjectStats
+  ): Promise<SyncAgentInfo[]> {
+    const agents: SyncAgentInfo[] = []
     const agentsPath = path.join(this.globalPath, 'agents')
 
     // Purge old agents
@@ -822,7 +756,7 @@ You are the ${name} expert for this project. Apply best practices for the detect
   // SKILL CONFIGURATION
   // ==========================================================================
 
-  private configureSkills(agents: AgentInfo[]): { agent: string; skill: string }[] {
+  private configureSkills(agents: SyncAgentInfo[]): { agent: string; skill: string }[] {
     const skills: { agent: string; skill: string }[] = []
 
     for (const agent of agents) {
@@ -862,7 +796,7 @@ You are the ${name} expert for this project. Apply best practices for the detect
    * Reads the mapping, checks which packages are needed, and installs missing ones.
    */
   private async autoInstallSkills(
-    agents: AgentInfo[]
+    agents: SyncAgentInfo[]
   ): Promise<{ name: string; agent: string; status: 'installed' | 'skipped' | 'error' }[]> {
     const results: { name: string; agent: string; status: 'installed' | 'skipped' | 'error' }[] = []
 
@@ -963,8 +897,8 @@ You are the ${name} expert for this project. Apply best practices for the detect
   private async generateContextFiles(
     git: GitData,
     stats: ProjectStats,
-    commands: Commands,
-    agents: AgentInfo[],
+    commands: ProjectCommands,
+    agents: SyncAgentInfo[],
     sources?: ContextSources
   ): Promise<string[]> {
     const generator = new ContextFileGenerator({
@@ -973,13 +907,7 @@ You are the ${name} expert for this project. Apply best practices for the detect
       globalPath: this.globalPath,
     })
 
-    return generator.generate(
-      { branch: git.branch, commits: git.commits },
-      stats,
-      commands,
-      agents,
-      sources
-    )
+    return generator.generate(git, stats, commands, agents, sources)
   }
 
   // ==========================================================================
@@ -1112,7 +1040,7 @@ You are the ${name} expert for this project. Apply best practices for the detect
   private async recordSyncMetrics(
     stats: ProjectStats,
     contextFiles: string[],
-    agents: AgentInfo[],
+    agents: SyncAgentInfo[],
     duration: number
   ): Promise<SyncMetrics> {
     const CHARS_PER_TOKEN = 4
@@ -1229,7 +1157,7 @@ You are the ${name} expert for this project. Apply best practices for the detect
     }
   }
 
-  private emptyCommands(): Commands {
+  private emptyCommands(): ProjectCommands {
     return {
       install: 'npm install',
       run: 'npm run',
@@ -1255,4 +1183,5 @@ You are the ${name} expert for this project. Apply best practices for the detect
 }
 
 export const syncService = new SyncService()
-export { SyncService, type SyncResult }
+export { SyncService }
+export type { ProjectSyncResult as SyncResult } from '../types'

package/core/services/sync-verifier.ts

@@ -11,42 +11,22 @@ import { exec } from 'node:child_process'
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { promisify } from 'node:util'
+import type {
+  VerificationCheck,
+  VerificationCheckResult,
+  VerificationConfig,
+  VerificationReport,
+} from '../types'
 import { getErrorMessage, isNotFoundError } from '../types/fs'
 
 const execAsync = promisify(exec)
 
-// =============================================================================
-// TYPES
-// =============================================================================
-
-export interface VerificationCheck {
-  name: string
-  command?: string
-  script?: string
-  enabled?: boolean
-}
-
-export interface VerificationConfig {
-  checks?: VerificationCheck[]
-  failFast?: boolean
-}
-
-export interface CheckResult {
-  name: string
-  passed: boolean
-  output?: string
-  error?: string
-  durationMs: number
-}
-
-export interface VerificationReport {
-  passed: boolean
-  checks: CheckResult[]
-  totalMs: number
-  failedCount: number
-  passedCount: number
-  skippedCount: number
-}
+export type {
+  VerificationCheck,
+  VerificationCheckResult,
+  VerificationConfig,
+  VerificationReport,
+} from '../types'
 
 // =============================================================================
 // BUILT-IN CHECKS
@@ -56,7 +36,7 @@ const BUILTIN_CHECKS = {
   /**
    * Verify all expected context files exist after sync
    */
-  async contextFilesExist(globalPath: string): Promise<CheckResult> {
+  async contextFilesExist(globalPath: string): Promise<VerificationCheckResult> {
     const start = Date.now()
     const expected = ['context/CLAUDE.md']
     const missing: string[] = []
@@ -82,7 +62,7 @@ const BUILTIN_CHECKS = {
   /**
    * Verify generated JSON files are valid
    */
-  async jsonFilesValid(globalPath: string): Promise<CheckResult> {
+  async jsonFilesValid(globalPath: string): Promise<VerificationCheckResult> {
     const start = Date.now()
     const jsonFiles = ['storage/state.json']
     const invalid: string[] = []
@@ -111,7 +91,7 @@ const BUILTIN_CHECKS = {
   /**
    * Verify no sensitive data leaked into context files
    */
-  async noSensitiveData(globalPath: string): Promise<CheckResult> {
+  async noSensitiveData(globalPath: string): Promise<VerificationCheckResult> {
     const start = Date.now()
     const contextDir = path.join(globalPath, 'context')
     const patterns = [
@@ -168,7 +148,7 @@ class SyncVerifier {
     config?: VerificationConfig
   ): Promise<VerificationReport> {
     const totalStart = Date.now()
-    const checks: CheckResult[] = []
+    const checks: VerificationCheckResult[] = []
     const failFast = config?.failFast ?? false
     let skipped = 0
 
@@ -228,7 +208,7 @@ class SyncVerifier {
   private async runCustomCheck(
     check: VerificationCheck,
     projectPath: string
-  ): Promise<CheckResult> {
+  ): Promise<VerificationCheckResult> {
     const start = Date.now()
     const command = check.command || (check.script ? `sh ${check.script}` : null)
 

package/core/services/watch-service.ts

@@ -102,6 +102,8 @@ class WatchService {
   }
   private isRunning: boolean = false
   private syncCount: number = 0
+  private sigintHandler: (() => void) | null = null
+  private sigtermHandler: (() => void) | null = null
 
   /**
    * Start watching for file changes
@@ -151,9 +153,13 @@ class WatchService {
       .on('unlink', (filePath: string) => this.handleChange('unlink', filePath))
       .on('error', (error: unknown) => this.handleError(error as Error))
 
-    // Handle graceful shutdown
-    process.on('SIGINT', () => this.stop())
-    process.on('SIGTERM', () => this.stop())
+    // Handle graceful shutdown — store references for proper removal
+    if (this.sigintHandler) process.off('SIGINT', this.sigintHandler)
+    if (this.sigtermHandler) process.off('SIGTERM', this.sigtermHandler)
+    this.sigintHandler = () => this.stop()
+    this.sigtermHandler = () => this.stop()
+    process.on('SIGINT', this.sigintHandler)
+    process.on('SIGTERM', this.sigtermHandler)
 
     return { success: true }
   }
@@ -177,6 +183,17 @@ class WatchService {
       this.watcher = null
     }
 
+    // Remove signal handlers to prevent accumulation
+    if (this.sigintHandler) {
+      process.off('SIGINT', this.sigintHandler)
+      this.sigintHandler = null
+    }
+    if (this.sigtermHandler) {
+      process.off('SIGTERM', this.sigtermHandler)
+      this.sigtermHandler = null
+    }
+
+    this.pendingChanges.clear()
     this.isRunning = false
     process.exit(0)
   }

package/core/types/citations.ts

@@ -0,0 +1,22 @@
+/**
+ * Citation Types
+ * Types for context source tracking.
+ */
+
+export type SourceType = 'detected' | 'user-defined' | 'inferred'
+
+export interface SourceInfo {
+  file: string
+  type: SourceType
+}
+
+export interface ContextSources {
+  name: SourceInfo
+  version: SourceInfo
+  ecosystem: SourceInfo
+  languages: SourceInfo
+  frameworks: SourceInfo
+  commands: SourceInfo
+  projectType: SourceInfo
+  git: SourceInfo
+}

package/core/types/commands.ts

@@ -157,6 +157,16 @@ export interface AnalyzeOptions {
   [key: string]: unknown
 }
 
+/**
+ * Options for the init (planning) command.
+ */
+export interface InitOptions {
+  /** Skip interactive wizard, use defaults */
+  yes?: boolean
+  /** Initial idea for architect mode */
+  idea?: string | null
+}
+
 // ============================================
 // Migration Types
 // ============================================

package/core/types/diff.ts

@@ -0,0 +1,41 @@
+/**
+ * Diff Types
+ * Types for sync diff and preserved sections.
+ */
+
+export interface DiffSection {
+  name: string
+  type: 'added' | 'modified' | 'removed' | 'unchanged'
+  before?: string
+  after?: string
+  lineCount: number
+}
+
+export interface PreservedInfo {
+  name: string
+  lineCount: number
+}
+
+export interface SyncDiff {
+  hasChanges: boolean
+  added: DiffSection[]
+  modified: DiffSection[]
+  removed: DiffSection[]
+  preserved: PreservedInfo[]
+  tokensBefore: number
+  tokensAfter: number
+  tokenDelta: number
+}
+
+export interface DiffOptions {
+  showFullDiff?: boolean
+  colorize?: boolean
+}
+
+/** Parsed markdown section (header + content range) */
+export interface ParsedMarkdownSection {
+  name: string
+  content: string
+  startLine: number
+  endLine: number
+}