cadr-cli 1.10.0 → 2.0.1

package/dist/presenters/console-presenter.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"console-presenter.js","sourceRoot":"","sources":["../../src/presenters/console-presenter.ts"],"names":[],"mappings":";;;AAiBA,MAAa,gBAAgB;IAC3B,eAAe;QACb,OAAO,CAAC,KAAK,CAAC,yBAAyB,CAAC,CAAC;QACzC,OAAO,CAAC,KAAK,CAAC,0CAA0C,CAAC,CAAC;QAC1D,OAAO,CAAC,KAAK,CAAC,wDAAwD,CAAC,CAAC;IAC1E,CAAC;IAED,YAAY,CAAC,OAAe;QAC1B,OAAO,CAAC,KAAK,CAAC,kBAAkB,OAAO,IAAI,CAAC,CAAC;IAC/C,CAAC;IAED,kBAAkB;QAChB,OAAO,CAAC,KAAK,CAAC,oCAAoC,CAAC,CAAC;IACtD,CAAC;IAED,aAAa,CAAC,OAAoB;QAChC,MAAM,QAAQ,GACZ,OAAO,CAAC,IAAI,KAAK,QAAQ;YACvB,CAAC,CAAC,QAAQ;YACV,CAAC,CAAC,OAAO,CAAC,IAAI,KAAK,aAAa;gBAC9B,CAAC,CAAC,WAAW,OAAO,CAAC,IAAI,IAAI,aAAa,QAAQ,OAAO,CAAC,IAAI,IAAI,MAAM,EAAE;gBAC1E,CAAC,CAAC,aAAa,CAAC;QAEtB,OAAO,CAAC,GAAG,CACT,+BAA+B,OAAO,CAAC,IAAI,KAAK,aAAa,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,QAAQ,GAAG,EAAE,CAC7F,CAAC;QAEF,IAAI,OAAO,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;YAC9B,OAAO,CAAC,GAAG,CAAC,4BAA4B,CAAC,CAAC;YAC1C,OAAO,CAAC,GAAG,CAAC,oBAAoB,CAAC,CAAC;YAClC,OAAO,CAAC,GAAG,CAAC,4BAA4B,CAAC,CAAC;QAC5C,CAAC;aAAM,IAAI,OAAO,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;YAC1C,OAAO,CAAC,GAAG,CAAC,yDAAyD,CAAC,CAAC;QACzE,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,GAAG,CAAC,uCAAuC,CAAC,CAAC;YACrD,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAC;QACnC,CAAC;IACH,CAAC;IAED,kBAAkB,CAAC,KAAe,EAAE,OAAoB;QACtD,MAAM,QAAQ,GACZ,OAAO,CAAC,IAAI,KAAK,QAAQ;YACvB,CAAC,CAAC,QAAQ;YACV,CAAC,CAAC,OAAO,CAAC,IAAI,KAAK,aAAa;gBAC9B,CAAC,CAAC,WAAW,OAAO,CAAC,IAAI,IAAI,aAAa,QAAQ,OAAO,CAAC,IAAI,IAAI,MAAM,EAAE;gBAC1E,CAAC,CAAC,aAAa,CAAC;QAEtB,MAAM,aAAa,GACjB,OAAO,CAAC,IAAI,KAAK,aAAa;YAC5B,CAAC,CAAC,GAAG,KAAK,CAAC,MAAM,QAAQ,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,YAAY,QAAQ,EAAE;YAC5E,CAAC,CAAC,GAAG,KAAK,CAAC,MAAM,IAAI,QAAQ,QAAQ,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,EAAE,CAAC;QAEzE,OAAO,CAAC,GAAG,CAAC,kBAAkB,aAAa,GAAG,CAAC,CAAC;QAChD,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE;YACrB,OAAO,CAAC,GAAG,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC;QAC7B,CAAC,CAAC,CAAC;QACH,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;IAClB,CAAC;IAED,iBAAiB;QACf,OAAO,CAAC,GAAG,CAAC,+BAA+B,CAAC,CAAC;IAC/C,CAAC;IAED,gBAAgB,CAAC,OAAoB,EAAE,QAAgB,EAAE,KAAa;QACpE,MAAM,YAAY,GAChB,OAAO,CAAC,IAAI,KAAK,QAAQ;YACvB,CAAC,CAAC,gBAAgB;YAClB,CAAC,CAAC,OAAO,CAAC,IAAI,KAAK,aAAa;gBAC9B,CAAC,CAAC,SAAS;gBACX,CAAC,CAAC,qBAAqB,CAAC;QAE9B,OAAO,CAAC,GAAG,CAAC,gBAAgB,YAAY,sCAAsC,CAAC,CAAC;QAChF,OAAO,CAAC,GAAG,CAAC,iBAAiB,QAAQ,IAAI,KAAK,OAAO,CAAC,CAAC;IACzD,CAAC;IAED,kBAAkB,CAAC,KAAc;QAC/B,OAAO,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC;QACrC,OAAO,CAAC,KAAK,CAAC,KAAK,KAAK,IAAI,wBAAwB,IAAI,CAAC,CAAC;IAC5D,CAAC;IAED,oBAAoB;QAClB,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC,CAAC;IACvC,CAAC;IAED,qBAAqB,CAAC,OAAwB;QAC5C,OAAO,CAAC,GAAG,CAAC,0CAA0C,CAAC,CAAC;QACxD,OAAO,CAAC,GAAG,CAAC,iBAAiB,OAAO,CAAC,MAAM,IAAI,CAAC,CAAC;QACjD,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC,OAAO,CAAC,UAAU,GAAG,GAAG,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;QAC5E,CAAC;IACH,CAAC;IAED,wBAAwB,CAAC,OAAwB;QAC/C,OAAO,CAAC,GAAG,CAAC,gDAAgD,CAAC,CAAC;QAC9D,OAAO,CAAC,GAAG,CAAC,iBAAiB,OAAO,CAAC,MAAM,IAAI,CAAC,CAAC;QACjD,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC,OAAO,CAAC,UAAU,GAAG,GAAG,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;QAC5E,CAAC;QACD,OAAO,CAAC,GAAG,CAAC,sCAAsC,CAAC,CAAC;IACtD,CAAC;IAED,iBAAiB;QACf,OAAO,CAAC,GAAG,CAAC,gCAAgC,CAAC,CAAC;IAChD,CAAC;IAED,oBAAoB,CAAC,KAAc;QACjC,OAAO,CAAC,KAAK,CAAC,2BAA2B,CAAC,CAAC;QAC3C,OAAO,CAAC,KAAK,CAAC,KAAK,KAAK,IAAI,wBAAwB,IAAI,CAAC,CAAC;IAC5D,CAAC;IAED,cAAc,CAAC,QAAgB;QAC7B,OAAO,CAAC,GAAG,CAAC,gCAAgC,CAAC,CAAC;QAC9C,OAAO,CAAC,GAAG,CAAC,YAAY,QAAQ,IAAI,CAAC,CAAC;QACtC,OAAO,CAAC,GAAG,CAAC,gBAAgB,CAAC,CAAC;QAC9B,OAAO,CAAC,GAAG,CAAC,2CAA2C,CAAC,CAAC;QACzD,OAAO,CAAC,GAAG,CAAC,+CAA+C,CAAC,CAAC;IAC/D,CAAC;IAED,gBAAgB,CAAC,KAAc;QAC7B,OAAO,CAAC,KAAK,CAAC,wBAAwB,CAAC,CAAC;QACxC,OAAO,CAAC,KAAK,CAAC,KAAK,KAAK,IAAI,wBAAwB,IAAI,CAAC,CAAC;IAC5D,CAAC;IAED,sBAAsB;QACpB,OAAO,CAAC,GAAG,CAAC,8BAA8B,CAAC,CAAC;QAC5C,OAAO,CAAC,GAAG,CAAC,mEAAmE,CAAC,CAAC;IACnF,CAAC;IAED,mBAAmB;QACjB,OAAO,CAAC,KAAK,CAAC,kCAAkC,CAAC,CAAC;QAClD,OAAO,CAAC,KAAK,CAAC,2CAA2C,CAAC,CAAC;IAC7D,CAAC;CACF;AApID,4CAoIC;AAEY,QAAA,SAAS,GAAG,IAAI,gBAAgB,EAAE,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cadr-cli",
-  "version": "1.10.0",
+  "version": "2.0.1",
   "description": "Continuous Architectural Decision Records - Automatically capture ADRs as you code",
   "main": "dist/index.js",
   "bin": {

package/src/{adr.test.ts → adr/adr.test.ts}

@@ -1,8 +1,5 @@
 /**
  * ADR File Management Module Tests
- * 
- * Tests for ADR file creation, numbering, and management.
- * Following TDD: These tests are written BEFORE implementation.
  */
 
 import * as fs from 'fs';
@@ -20,14 +17,12 @@ describe('ADR Module', () => {
   const testDir = path.join(__dirname, '../test-adrs');
 
   beforeEach(() => {
-    // Clean up test directory before each test
     if (fs.existsSync(testDir)) {
       fs.rmSync(testDir, { recursive: true, force: true });
     }
   });
 
   afterEach(() => {
-    // Clean up test directory after each test
     if (fs.existsSync(testDir)) {
       fs.rmSync(testDir, { recursive: true, force: true });
     }
@@ -187,14 +182,15 @@ describe('ADR Module', () => {
       expect(result1.filePath).toContain('0001-first.md');
       expect(result2.filePath).toContain('0002-second.md');
       expect(result3.filePath).toContain('0003-third.md');
-      
+
       expect(fs.existsSync(result1.filePath!)).toBe(true);
       expect(fs.existsSync(result2.filePath!)).toBe(true);
       expect(fs.existsSync(result3.filePath!)).toBe(true);
     });
 
     it('saves correct content to file', () => {
-      const content = '# My Decision\n\n* Status: accepted\n* Date: 2025-10-21\n\n## Context\n\nSome context here.';
+      const content =
+        '# My Decision\n\n* Status: accepted\n* Date: 2025-10-21\n\n## Context\n\nSome context here.';
       const result = saveADR(content, 'My Decision', testDir);
 
       const savedContent = fs.readFileSync(result.filePath!, 'utf-8');
@@ -202,7 +198,6 @@ describe('ADR Module', () => {
     });
 
     it('uses default directory when not specified', () => {
-      // Clean up default directory
       const defaultPath = path.join(process.cwd(), DEFAULT_ADR_DIR);
       if (fs.existsSync(defaultPath)) {
         fs.rmSync(defaultPath, { recursive: true, force: true });
@@ -213,44 +208,38 @@ describe('ADR Module', () => {
 
       expect(result.success).toBe(true);
       expect(result.filePath).toContain(DEFAULT_ADR_DIR);
-      
-      // Cleanup
+
       if (fs.existsSync(defaultPath)) {
         fs.rmSync(defaultPath, { recursive: true, force: true });
       }
     });
 
     it('handles file write errors gracefully', () => {
-      // Create a read-only directory to trigger write error
       fs.mkdirSync(testDir, { recursive: true });
-      
-      // Make directory read-only (if not Windows)
+
       if (process.platform !== 'win32') {
         fs.chmodSync(testDir, 0o444);
-        
+
         const content = '# Test\n\nContent';
         const result = saveADR(content, 'Test', testDir);
 
         expect(result.success).toBe(false);
         expect(result.error).toBeDefined();
         expect(result.filePath).toBeUndefined();
-        
-        // Restore permissions for cleanup
+
         fs.chmodSync(testDir, 0o755);
       } else {
-        // On Windows, just verify error handling structure exists
-        // Skip actual permission test as Windows permissions work differently
         const content = '# Test\n\nContent';
         const result = saveADR(content, 'Test', testDir);
-        
-        // Should succeed on Windows since we can't easily simulate permission errors
+
         expect(result).toHaveProperty('success');
         expect(typeof result.success).toBe('boolean');
       }
     });
 
     it('handles long titles correctly', () => {
-      const longTitle = 'This is a very long title that should still work correctly when converted to a filename slug';
+      const longTitle =
+        'This is a very long title that should still work correctly when converted to a filename slug';
       const content = `# ${longTitle}\n\nContent`;
       const result = saveADR(content, longTitle, testDir);
 
@@ -263,7 +252,6 @@ describe('ADR Module', () => {
       const result = saveADR(content, 'Decision with émojis 🚀', testDir);
 
       expect(result.success).toBe(true);
-      // Should strip unicode to safe characters
       expect(result.filePath).toMatch(/0001-.+\.md$/);
     });
   });
@@ -275,4 +263,3 @@ describe('ADR Module', () => {
     });
   });
 });
-

package/src/{adr.ts → adr/adr.ts}

@@ -1,46 +1,32 @@
 /**
  * ADR File Management Module
- * 
+ *
  * Handles creation and management of Architectural Decision Record files.
  * Follows MADR (Markdown Architectural Decision Records) format.
  */
 
 import * as fs from 'fs';
 import * as path from 'path';
-import { loggerInstance as logger } from './logger';
+import { loggerInstance as logger } from '../logger';
 
-/**
- * Default ADR directory relative to project root
- */
 export const DEFAULT_ADR_DIR = 'docs/adr';
 
-/**
- * Convert a title to a filename-safe slug
- * Example: "Use PostgreSQL for Storage" -> "use-postgresql-for-storage"
- */
 export function titleToSlug(title: string): string {
   return title
     .toLowerCase()
-    .replace(/[^a-z0-9]+/g, '-') // Replace non-alphanumeric with hyphens
-    .replace(/^-+|-+$/g, ''); // Remove leading/trailing hyphens
+    .replace(/[^a-z0-9]+/g, '-')
+    .replace(/^-+|-+$/g, '');
 }
 
-/**
- * Get the next ADR number by scanning existing ADR files
- * 
- * @param adrDir - Directory containing ADR files
- * @returns Next available ADR number (e.g., 1, 2, 3...)
- */
 export function getNextADRNumber(adrDir: string): number {
   try {
     if (!fs.existsSync(adrDir)) {
-      return 1; // First ADR
+      return 1;
     }
 
     const files = fs.readdirSync(adrDir);
     const adrNumbers: number[] = [];
 
-    // Extract numbers from existing ADR files (format: 0001-title.md)
     for (const file of files) {
       const match = file.match(/^(\d{4})-/);
       if (match) {
@@ -52,7 +38,6 @@ export function getNextADRNumber(adrDir: string): number {
       return 1;
     }
 
-    // Return next number after the highest existing number
     return Math.max(...adrNumbers) + 1;
   } catch (error) {
     logger.warn('Failed to scan existing ADRs, defaulting to 1', { error, adrDir });
@@ -60,11 +45,6 @@ export function getNextADRNumber(adrDir: string): number {
   }
 }
 
-/**
- * Ensure ADR directory exists, create if it doesn't
- * 
- * @param adrDir - Directory path to ensure exists
- */
 export function ensureADRDirectory(adrDir: string): void {
   if (!fs.existsSync(adrDir)) {
     logger.info('Creating ADR directory', { adrDir });
@@ -72,56 +52,36 @@ export function ensureADRDirectory(adrDir: string): void {
   }
 }
 
-/**
- * Generate ADR filename from number and title
- * 
- * @param number - ADR number (will be zero-padded to 4 digits)
- * @param title - ADR title (will be converted to slug)
- * @returns Filename like "0001-use-postgresql-for-storage.md"
- */
 export function generateADRFilename(number: number, title: string): string {
   const paddedNumber = String(number).padStart(4, '0');
   const slug = titleToSlug(title);
   return `${paddedNumber}-${slug}.md`;
 }
 
-/**
- * Save ADR content to file
- * 
- * @param content - Full markdown content of the ADR
- * @param title - Title extracted from ADR content
- * @param adrDir - Directory to save ADR in (defaults to docs/adr)
- * @returns Object with success status, file path, and any error
- */
 export function saveADR(
   content: string,
   title: string,
   adrDir: string = DEFAULT_ADR_DIR
 ): { success: boolean; filePath?: string; error?: string } {
   try {
-    // Ensure directory exists
     ensureADRDirectory(adrDir);
 
-    // Get next ADR number
     const adrNumber = getNextADRNumber(adrDir);
 
-    // Generate filename
     const filename = generateADRFilename(adrNumber, title);
     const filePath = path.join(adrDir, filename);
 
-    // Check if file already exists (shouldn't happen, but safety check)
     if (fs.existsSync(filePath)) {
       logger.warn('ADR file already exists, using alternative name', { filePath });
       const alternativeFilename = generateADRFilename(adrNumber + 1, title);
       const alternativeFilePath = path.join(adrDir, alternativeFilename);
-      
+
       fs.writeFileSync(alternativeFilePath, content, 'utf-8');
       logger.info('ADR saved successfully', { filePath: alternativeFilePath });
-      
+
       return { success: true, filePath: alternativeFilePath };
     }
 
-    // Write ADR content to file
     fs.writeFileSync(filePath, content, 'utf-8');
 
     logger.info('ADR saved successfully', { filePath, adrNumber });
@@ -133,4 +93,3 @@ export function saveADR(
     return { success: false, error: errorMessage };
   }
 }
-

package/src/adr/index.ts

@@ -0,0 +1 @@
+export * from './adr';

package/src/analysis/analysis.orchestrator.ts

@@ -0,0 +1,175 @@
+/* eslint-disable no-console */
+import { loadConfig, getDefaultConfigPath } from '../config';
+import type { DiffOptions } from '../git/index';
+import {
+  formatPrompt,
+  ANALYSIS_PROMPT_V1,
+  formatGenerationPrompt,
+  promptForGeneration,
+} from '../llm/prompts';
+import { analyzeChanges, generateADRContent } from '../llm/llm';
+import { loggerInstance as logger } from '../logger';
+import { saveADR } from '../adr/adr';
+import * as path from 'path';
+import { createGitStrategy, type GitStrategy } from './strategies/git-strategy';
+import { presenter, type AnalysisSummary } from '../presenters/console-presenter';
+
+export interface AnalysisResult {
+  is_significant: boolean;
+  reason: string;
+  confidence?: number;
+  timestamp: string;
+}
+
+export interface GenerationResult {
+  content: string;
+  title: string;
+  timestamp: string;
+}
+
+async function runAnalysisInternal(diffOptions: DiffOptions): Promise<void> {
+  const configPath = getDefaultConfigPath();
+  const config = await loadConfig(configPath);
+
+  if (!config) {
+    presenter.showConfigError();
+    return;
+  }
+
+  const gitStrategy: GitStrategy = createGitStrategy(diffOptions);
+
+  let changedFiles: string[];
+  try {
+    changedFiles = await gitStrategy.getFiles();
+  } catch (error) {
+    const gitError = error as { name?: string; message?: string };
+    if (gitError.name === 'GitError') {
+      presenter.showGitError(gitError.message || 'Unknown Git error');
+    } else {
+      presenter.showReadFilesError();
+    }
+    logger.error('Failed to get changed files', { error, mode: diffOptions.mode });
+    return;
+  }
+
+  if (changedFiles.length === 0) {
+    presenter.showNoChanges(diffOptions);
+    return;
+  }
+
+  presenter.showAnalyzingFiles(changedFiles, diffOptions);
+
+  let diffContent: string;
+  try {
+    diffContent = await gitStrategy.getDiff();
+  } catch (error) {
+    presenter.showReadFilesError();
+    logger.error('Failed to get diff', { error, mode: diffOptions.mode });
+    return;
+  }
+
+  if (!diffContent || diffContent.trim().length === 0) {
+    presenter.showNoDiffContent();
+    return;
+  }
+
+  const repositoryContext = path.basename(process.cwd());
+  const prompt = formatPrompt(ANALYSIS_PROMPT_V1, {
+    file_paths: changedFiles,
+    diff_content: diffContent,
+  });
+
+  presenter.showSendingToLLM(diffOptions, config.provider, config.analysis_model);
+
+  const response = await analyzeChanges(config, {
+    file_paths: changedFiles,
+    diff_content: diffContent,
+    repository_context: repositoryContext,
+    analysis_prompt: prompt,
+  });
+
+  if (!response.result || response.error) {
+    presenter.showAnalysisFailed(response.error);
+    return;
+  }
+
+  const result = response.result;
+
+  presenter.showAnalysisComplete();
+
+  if (result.is_significant) {
+    const summary: AnalysisSummary = {
+      fileCount: changedFiles.length,
+      mode: diffOptions.mode,
+      isSignificant: true,
+      reason: result.reason,
+      confidence: result.confidence,
+    };
+    presenter.showSignificantResult(summary);
+
+    const shouldGenerate = await promptForGeneration(result.reason);
+
+    if (shouldGenerate) {
+      presenter.showGeneratingADR();
+
+      const generationPrompt = formatGenerationPrompt({
+        file_paths: changedFiles,
+        diff_content: diffContent,
+      });
+
+      const generationResponse = await generateADRContent(config, {
+        file_paths: changedFiles,
+        diff_content: diffContent,
+        reason: result.reason,
+        generation_prompt: generationPrompt,
+      });
+
+      if (!generationResponse.result || generationResponse.error) {
+        presenter.showGenerationFailed(generationResponse.error);
+        logger.error('ADR generation failed', { error: generationResponse.error });
+      } else {
+        const saveResult = saveADR(
+          generationResponse.result.content,
+          generationResponse.result.title
+        );
+
+        if (saveResult.success && saveResult.filePath) {
+          presenter.showADRSuccess(saveResult.filePath);
+          logger.info('ADR generation workflow completed successfully', {
+            filePath: saveResult.filePath,
+            title: generationResponse.result.title,
+          });
+        } else {
+          presenter.showADRSaveError(saveResult.error);
+          logger.error('Failed to save ADR', { error: saveResult.error });
+        }
+      }
+    } else {
+      presenter.showSkippingGeneration();
+    }
+  } else {
+    const summary: AnalysisSummary = {
+      fileCount: changedFiles.length,
+      mode: diffOptions.mode,
+      isSignificant: false,
+      reason: result.reason,
+      confidence: result.confidence,
+    };
+    presenter.showNotSignificantResult(summary);
+  }
+
+  logger.info('Analysis workflow completed successfully', {
+    is_significant: result.is_significant,
+    file_count: changedFiles.length,
+  });
+}
+
+export async function runAnalysis(diffOptions: DiffOptions = { mode: 'all' }): Promise<void> {
+  try {
+    logger.info('Starting analysis workflow');
+    await runAnalysisInternal(diffOptions);
+  } catch (error) {
+    logger.error('Unexpected error in analysis workflow', { error });
+    presenter.showUnexpectedError();
+  }
+}

package/src/analysis/strategies/git-strategy.ts

@@ -0,0 +1,106 @@
+import {
+  getStagedFiles,
+  getStagedDiff,
+  getAllChanges,
+  getAllDiff,
+  type DiffOptions,
+} from '../../git/git.operations';
+import { GitError } from '../../git';
+
+export interface GitStrategy {
+  getFiles(): Promise<string[]>;
+  getDiff(): Promise<string>;
+}
+
+export class StagedChangesStrategy implements GitStrategy {
+  async getFiles(): Promise<string[]> {
+    return getStagedFiles();
+  }
+
+  async getDiff(): Promise<string> {
+    return getStagedDiff();
+  }
+}
+
+export class AllChangesStrategy implements GitStrategy {
+  async getFiles(): Promise<string[]> {
+    return getAllChanges();
+  }
+
+  async getDiff(): Promise<string> {
+    return getAllDiff();
+  }
+}
+
+export class BranchDiffStrategy implements GitStrategy {
+  constructor(
+    private base: string,
+    private head: string
+  ) {}
+
+  async getFiles(): Promise<string[]> {
+    const { exec } = await import('child_process');
+    const { promisify } = await import('util');
+    const execAsync = promisify(exec);
+
+    try {
+      const { stdout } = await execAsync(`git diff --name-only ${this.base}...${this.head}`);
+      return stdout
+        .split('\n')
+        .map((f) => f.trim())
+        .filter((f) => f.length > 0);
+    } catch (error) {
+      const errorWithCode = error as { code?: number };
+      if (errorWithCode.code === 128) {
+        throw new GitError(
+          `Invalid git reference: ${this.base} or ${this.head}. Please ensure both references exist.`,
+          'GIT_ERROR',
+          error instanceof Error ? error : new Error(String(error))
+        );
+      }
+      throw new GitError(
+        'Unable to read Git repository. Please check repository permissions.',
+        'GIT_ERROR',
+        error instanceof Error ? error : new Error(String(error))
+      );
+    }
+  }
+
+  async getDiff(): Promise<string> {
+    const { exec } = await import('child_process');
+    const { promisify } = await import('util');
+    const execAsync = promisify(exec);
+
+    try {
+      const { stdout } = await execAsync(`git diff ${this.base}...${this.head} --unified=1`);
+      return stdout;
+    } catch (error) {
+      const errorWithCode = error as { code?: number };
+      if (errorWithCode.code === 128) {
+        throw new GitError(
+          `Invalid git reference: ${this.base} or ${this.head}. Please ensure both references exist.`,
+          'GIT_ERROR',
+          error instanceof Error ? error : new Error(String(error))
+        );
+      }
+      throw new GitError(
+        'Unable to read Git repository. Please check repository permissions.',
+        'GIT_ERROR',
+        error instanceof Error ? error : new Error(String(error))
+      );
+    }
+  }
+}
+
+export function createGitStrategy(options: DiffOptions): GitStrategy {
+  switch (options.mode) {
+    case 'staged':
+      return new StagedChangesStrategy();
+    case 'all':
+      return new AllChangesStrategy();
+    case 'branch-diff':
+      return new BranchDiffStrategy(options.base || 'origin/main', options.head || 'HEAD');
+    default:
+      return new AllChangesStrategy();
+  }
+}

package/src/commands/analyze.ts

@@ -1,31 +1,31 @@
 /**
  * Analyze Command
- * 
+ *
  * Triggers LLM-powered analysis of code changes.
  * Thin wrapper around analysis orchestration module.
  */
 
-import { runAnalysis } from '../analysis';
+import { runAnalysis } from '../analysis/analysis.orchestrator';
 import { DiffOptions } from '../git';
 import { loggerInstance as logger } from '../logger';
 
 /**
  * Execute the analyze command
  * Analyzes code changes for architectural significance
- * 
+ *
  * @param args - Command line arguments (e.g., ['--staged'], ['--all'])
  */
 export async function analyzeCommand(args: string[] = []): Promise<void> {
   try {
     // Parse command line flags to determine diff options
     const diffOptions: DiffOptions = { mode: 'all' }; // Default to all uncommitted
-    
+
     // Check for --base flag (implies branch-diff mode)
     const baseIndex = args.indexOf('--base');
     if (baseIndex !== -1 && baseIndex + 1 < args.length) {
       diffOptions.mode = 'branch-diff';
       diffOptions.base = args[baseIndex + 1];
-      
+
       // Check for optional --head flag
       const headIndex = args.indexOf('--head');
       if (headIndex !== -1 && headIndex + 1 < args.length) {
@@ -36,11 +36,11 @@ export async function analyzeCommand(args: string[] = []): Promise<void> {
     } else if (args.includes('--all')) {
       diffOptions.mode = 'all';
     }
-    
-    logger.info('Analyze command started', { 
+
+    logger.info('Analyze command started', {
       mode: diffOptions.mode,
       base: diffOptions.base,
-      head: diffOptions.head
+      head: diffOptions.head,
     });
     await runAnalysis(diffOptions);
     logger.info('Analyze command completed');
@@ -53,4 +53,3 @@ export async function analyzeCommand(args: string[] = []): Promise<void> {
     console.error('Please check the logs for more details.\n');
   }
 }
-

package/src/commands/status.test.ts

@@ -0,0 +1,64 @@
+import { statusCommand } from './status';
+import * as config from '../config';
+import * as fs from 'fs';
+
+// Mock dependencies
+jest.mock('../config');
+jest.mock('fs');
+jest.mock('../logger');
+
+describe('Status Command', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  describe('statusCommand', () => {
+    test('displays status when config exists', async () => {
+      (fs.existsSync as jest.Mock).mockReturnValue(true);
+      (config.getDefaultConfigPath as jest.Mock).mockReturnValue('cadr.yaml');
+      (config.loadConfig as jest.Mock).mockResolvedValue({
+        provider: 'openai',
+        analysis_model: 'gpt-4',
+        api_key_env: 'OPENAI_API_KEY'
+      });
+      
+      const consoleSpy = jest.spyOn(console, 'log').mockImplementation();
+      
+      await statusCommand();
+      
+      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('cADR Status'));
+      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Provider: openai'));
+      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Model: gpt-4'));
+      
+      consoleSpy.mockRestore();
+    });
+
+    test('displays warning when config does not exist', async () => {
+      (fs.existsSync as jest.Mock).mockReturnValue(false);
+      (config.getDefaultConfigPath as jest.Mock).mockReturnValue('cadr.yaml');
+      
+      const consoleSpy = jest.spyOn(console, 'log').mockImplementation();
+      
+      await statusCommand();
+      
+      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Not Found'));
+      expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Run `cadr init`'));
+      
+      consoleSpy.mockRestore();
+    });
+
+    test('handles errors gracefully', async () => {
+      (config.getDefaultConfigPath as jest.Mock).mockImplementation(() => {
+        throw new Error('FileSystem error');
+      });
+      
+      const errorSpy = jest.spyOn(console, 'error').mockImplementation();
+      
+      await statusCommand();
+      
+      expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('Failed to retrieve status'));
+      
+      errorSpy.mockRestore();
+    });
+  });
+});