edsger 0.2.2 → 0.2.4

package/README.md

@@ -12,6 +12,13 @@ AI-powered software development CLI tool with comprehensive feature management,
 - **Hook-Friendly**: Designed for pre-commit hooks and CI/CD automation
 - **Detailed Output**: Clear feedback with blocking/warning/passing results
 
+### 🔧 Code Refactoring
+
+- **Intelligent Analysis**: AI-powered code quality and refactoring analysis
+- **Pattern Detection**: Identifies code smells, duplication, and technical debt
+- **Improvement Suggestions**: Provides prioritized recommendations for code improvement
+- **Interactive Mode**: Ask for confirmation before applying refactoring changes
+
 ### 🎯 Feature Analysis
 
 - **MCP Integration**: Analyze features using Model Context Protocol
@@ -68,6 +75,16 @@ edsger --review --files "**/*.ts" "**/*.js"
 edsger --review --verbose
 ```
 
+### Code Refactoring
+
+```bash
+# Analyze and refactor code in current directory
+edsger --refactor
+
+# Verbose refactoring output
+edsger --refactor --verbose
+```
+
 ### Feature Analysis
 
 ```bash

package/dist/api/features/batch-operations.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Batch update user story statuses
+ */
+export declare function batchUpdateUserStoryStatus(mcpServerUrl: string, mcpToken: string, userStoryIds: string[], status: 'draft' | 'ready' | 'in_progress' | 'done' | 'cancelled', verbose?: boolean): Promise<boolean>;
+/**
+ * Batch update test case statuses
+ */
+export declare function batchUpdateTestCaseStatus(mcpServerUrl: string, mcpToken: string, testCaseIds: string[], status: 'draft' | 'ready' | 'in_progress' | 'done' | 'cancelled', verbose?: boolean): Promise<boolean>;
+/**
+ * Batch delete user stories
+ */
+export declare function batchDeleteUserStories(mcpServerUrl: string, mcpToken: string, userStoryIds: string[], verbose?: boolean): Promise<boolean>;
+/**
+ * Batch delete test cases
+ */
+export declare function batchDeleteTestCases(mcpServerUrl: string, mcpToken: string, testCaseIds: string[], verbose?: boolean): Promise<boolean>;

package/dist/api/features/batch-operations.js

@@ -0,0 +1,100 @@
+import { logInfo, logError } from '../../utils/logger.js';
+import { callMcpEndpoint } from '../mcp-client.js';
+/**
+ * Batch update user story statuses
+ */
+export async function batchUpdateUserStoryStatus(mcpServerUrl, mcpToken, userStoryIds, status, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Batch updating ${userStoryIds.length} user stories to status: ${status}`);
+        }
+        for (const id of userStoryIds) {
+            await callMcpEndpoint(mcpServerUrl, mcpToken, 'user_stories/update_status', {
+                user_story_id: id,
+                status,
+            });
+        }
+        if (verbose) {
+            logInfo('✅ Batch user story status update completed');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to batch update user story statuses: ${errorMessage}`);
+        return false;
+    }
+}
+/**
+ * Batch update test case statuses
+ */
+export async function batchUpdateTestCaseStatus(mcpServerUrl, mcpToken, testCaseIds, status, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Batch updating ${testCaseIds.length} test cases to status: ${status}`);
+        }
+        for (const id of testCaseIds) {
+            await callMcpEndpoint(mcpServerUrl, mcpToken, 'test_cases/update_status', {
+                test_case_id: id,
+                status,
+            });
+        }
+        if (verbose) {
+            logInfo('✅ Batch test case status update completed');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to batch update test case statuses: ${errorMessage}`);
+        return false;
+    }
+}
+/**
+ * Batch delete user stories
+ */
+export async function batchDeleteUserStories(mcpServerUrl, mcpToken, userStoryIds, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Batch deleting ${userStoryIds.length} user stories`);
+        }
+        for (const id of userStoryIds) {
+            await callMcpEndpoint(mcpServerUrl, mcpToken, 'user_stories/delete', {
+                user_story_id: id,
+            });
+        }
+        if (verbose) {
+            logInfo('✅ Batch user story deletion completed');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to batch delete user stories: ${errorMessage}`);
+        return false;
+    }
+}
+/**
+ * Batch delete test cases
+ */
+export async function batchDeleteTestCases(mcpServerUrl, mcpToken, testCaseIds, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Batch deleting ${testCaseIds.length} test cases`);
+        }
+        for (const id of testCaseIds) {
+            await callMcpEndpoint(mcpServerUrl, mcpToken, 'test_cases/delete', {
+                test_case_id: id,
+            });
+        }
+        if (verbose) {
+            logInfo('✅ Batch test case deletion completed');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to batch delete test cases: ${errorMessage}`);
+        return false;
+    }
+}

package/dist/api/features/index.d.ts

@@ -4,4 +4,5 @@ export * from './user-stories.js';
 export * from './test-cases.js';
 export * from './feature-utils.js';
 export * from './status-updater.js';
+export * from './batch-operations.js';
 export * from '../../types/features.js';

package/dist/api/features/index.js

@@ -5,5 +5,6 @@ export * from './user-stories.js';
 export * from './test-cases.js';
 export * from './feature-utils.js';
 export * from './status-updater.js';
+export * from './batch-operations.js';
 // Re-export types
 export * from '../../types/features.js';

package/dist/api/features/test-cases.d.ts

@@ -19,3 +19,11 @@ export declare function createTestCases(mcpServerUrl: string, mcpToken: string,
     description: string;
     is_critical?: boolean;
 }>, verbose?: boolean): Promise<boolean>;
+/**
+ * Delete a test case
+ */
+export declare function deleteTestCase(mcpServerUrl: string, mcpToken: string, testCaseId: string, verbose?: boolean): Promise<boolean>;
+/**
+ * Update test case status
+ */
+export declare function updateTestCaseStatus(mcpServerUrl: string, mcpToken: string, testCaseId: string, status: 'draft' | 'ready' | 'in_progress' | 'done' | 'cancelled', verbose?: boolean): Promise<boolean>;

package/dist/api/features/test-cases.js

@@ -61,3 +61,48 @@ export async function createTestCases(mcpServerUrl, mcpToken, featureId, testCas
         return false;
     }
 }
+/**
+ * Delete a test case
+ */
+export async function deleteTestCase(mcpServerUrl, mcpToken, testCaseId, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Deleting test case: ${testCaseId}`);
+        }
+        await callMcpEndpoint(mcpServerUrl, mcpToken, 'test_cases/delete', {
+            test_case_id: testCaseId,
+        });
+        if (verbose) {
+            logInfo('✅ Test case deleted successfully');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to delete test case: ${errorMessage}`);
+        return false;
+    }
+}
+/**
+ * Update test case status
+ */
+export async function updateTestCaseStatus(mcpServerUrl, mcpToken, testCaseId, status, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Updating test case ${testCaseId} status to: ${status}`);
+        }
+        await callMcpEndpoint(mcpServerUrl, mcpToken, 'test_cases/update_status', {
+            test_case_id: testCaseId,
+            status,
+        });
+        if (verbose) {
+            logInfo('✅ Test case status updated successfully');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to update test case status: ${errorMessage}`);
+        return false;
+    }
+}

package/dist/api/features/user-stories.d.ts

@@ -19,3 +19,11 @@ export declare function createUserStories(mcpServerUrl: string, mcpToken: string
     description: string;
     status?: string;
 }>, verbose?: boolean): Promise<boolean>;
+/**
+ * Delete a user story
+ */
+export declare function deleteUserStory(mcpServerUrl: string, mcpToken: string, userStoryId: string, verbose?: boolean): Promise<boolean>;
+/**
+ * Update user story status
+ */
+export declare function updateUserStoryStatus(mcpServerUrl: string, mcpToken: string, userStoryId: string, status: 'draft' | 'ready' | 'in_progress' | 'done' | 'cancelled', verbose?: boolean): Promise<boolean>;

package/dist/api/features/user-stories.js

@@ -61,3 +61,48 @@ export async function createUserStories(mcpServerUrl, mcpToken, featureId, userS
         return false;
     }
 }
+/**
+ * Delete a user story
+ */
+export async function deleteUserStory(mcpServerUrl, mcpToken, userStoryId, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Deleting user story: ${userStoryId}`);
+        }
+        await callMcpEndpoint(mcpServerUrl, mcpToken, 'user_stories/delete', {
+            user_story_id: userStoryId,
+        });
+        if (verbose) {
+            logInfo('✅ User story deleted successfully');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to delete user story: ${errorMessage}`);
+        return false;
+    }
+}
+/**
+ * Update user story status
+ */
+export async function updateUserStoryStatus(mcpServerUrl, mcpToken, userStoryId, status, verbose) {
+    try {
+        if (verbose) {
+            logInfo(`Updating user story ${userStoryId} status to: ${status}`);
+        }
+        await callMcpEndpoint(mcpServerUrl, mcpToken, 'user_stories/update_status', {
+            user_story_id: userStoryId,
+            status,
+        });
+        if (verbose) {
+            logInfo('✅ User story status updated successfully');
+        }
+        return true;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to update user story status: ${errorMessage}`);
+        return false;
+    }
+}

package/dist/cli/commands/refactor-command.d.ts

@@ -0,0 +1,2 @@
+import { CliOptions } from '../../types/index.js';
+export declare const runRefactor: (options: CliOptions) => Promise<void>;

package/dist/cli/commands/refactor-command.js

@@ -0,0 +1,107 @@
+import { query } from '@anthropic-ai/claude-code';
+import { logInfo, logError, logSuccess } from '../../utils/logger.js';
+import { validateConfiguration } from '../utils/validation.js';
+export const runRefactor = async (options) => {
+    // Load and validate configuration
+    const config = validateConfiguration(options);
+    if (options.verbose) {
+        logInfo('Starting code refactoring for current directory...');
+    }
+    logInfo('Analyzing code for refactoring opportunities...');
+    try {
+        const refactorPrompt = createRefactorPrompt(config);
+        const systemPrompt = createSystemPrompt();
+        let hasCompleted = false;
+        for await (const message of query({
+            prompt: refactorPrompt,
+            options: {
+                appendSystemPrompt: systemPrompt,
+                model: config.claude.model || 'sonnet',
+            },
+        })) {
+            // Stream the refactoring process
+            if (message.type === 'assistant' && message.message?.content) {
+                for (const content of message.message.content) {
+                    if (content.type === 'text') {
+                        console.log(`\n🤖 ${content.text}`);
+                    }
+                    else if (content.type === 'tool_use') {
+                        console.log(`\n🔧 ${content.name}: ${content.input.description || content.input.command || 'Running...'}`);
+                    }
+                }
+            }
+            if (message.type === 'result') {
+                hasCompleted = true;
+                if (message.subtype === 'success') {
+                    logSuccess('Refactoring analysis completed successfully');
+                    process.exit(0);
+                }
+                else {
+                    logError(`Refactoring incomplete: ${message.subtype}`);
+                    if (message.subtype === 'error_max_turns') {
+                        console.log('💡 Try simplifying the codebase or increase timeout');
+                    }
+                    process.exit(1);
+                }
+            }
+        }
+        if (!hasCompleted) {
+            logError('Refactoring process did not complete');
+            process.exit(1);
+        }
+    }
+    catch (error) {
+        logError(`Refactoring failed: ${error instanceof Error ? error.message : String(error)}`);
+        process.exit(1);
+    }
+};
+const createSystemPrompt = () => {
+    return `You are an expert code refactoring assistant using Claude Code. Your task is to analyze code in the current directory and suggest or perform refactoring improvements.
+
+IMPORTANT INSTRUCTIONS:
+1. Use the Bash tool to explore the project structure (ls, find, etc.)
+2. Use the Read tool to examine file contents
+3. Use the Grep tool to search for patterns, code smells, or duplications
+4. Use the Edit tool to apply refactoring changes when appropriate
+5. Analyze code quality, identify technical debt, and suggest improvements
+
+REFACTORING FOCUS AREAS:
+- Code duplication and DRY violations
+- Complex functions that need simplification
+- Poor naming conventions
+- Missing type annotations (TypeScript)
+- Inefficient algorithms or data structures
+- Code organization and structure
+- Missing or inadequate error handling
+- Opportunities for better abstraction
+- Performance improvements
+
+WORKFLOW:
+1. First, explore the project structure to understand the codebase
+2. Identify files that would benefit from refactoring
+3. Analyze specific issues and patterns
+4. Suggest improvements with clear explanations
+5. Ask for user confirmation before making significant changes
+6. Apply changes using the Edit tool if approved
+
+Be thorough but focused. Prioritize high-impact refactorings that improve code maintainability and quality.`;
+};
+const createRefactorPrompt = (config) => {
+    return `Please analyze the code in the current directory for refactoring opportunities.
+
+Steps to follow:
+1. Start by running "ls -la" to see the project structure
+2. Identify key source files (focus on patterns: ${config.patterns.join(', ')})
+3. Read and analyze relevant source files
+4. Look for:
+   - Code duplication
+   - Complex or long functions
+   - Poor naming
+   - Missing types or documentation
+   - Error handling issues
+   - Performance bottlenecks
+5. Provide a summary of findings with prioritized recommendations
+6. Ask if I'd like you to proceed with any specific refactoring
+
+Start by exploring the current directory structure.`;
+};

package/dist/cli/index.js

@@ -11,6 +11,7 @@ import { runTechnicalDesign } from './commands/technical-design-command.js';
 import { runCodeImplementation } from './commands/code-implementation-command.js';
 import { runFunctionalTestingCommand } from './commands/functional-testing-command.js';
 import { runCodeReview } from './commands/code-review-command.js';
+import { runRefactor } from './commands/refactor-command.js';
 // Get package.json version dynamically
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
@@ -28,6 +29,7 @@ program
     .option('-r, --review', 'Review code changes')
     .option('-s, --staged', 'Review only staged changes')
     .option('-f, --files <patterns...>', 'Review specific file patterns')
+    .option('--refactor', 'Refactor code in current directory')
     .option('--feature-analysis <featureId>', 'Analyze feature and generate user stories/test cases')
     .option('--technical-design <featureId>', 'Generate technical design for a feature')
     .option('--implement <featureId>', 'Implement code for a feature based on its specifications')
@@ -70,6 +72,11 @@ export const runEdsger = async (options) => {
         await runFunctionalTestingCommand(options);
         return;
     }
+    // Handle refactor mode
+    if (options.refactor) {
+        await runRefactor(options);
+        return;
+    }
     // Handle code review mode (explicit --review flag or as fallback)
     if (options.review || options.staged || options.files) {
         await runCodeReview(options);

package/dist/cli.d.ts

@@ -1,3 +1,3 @@
 #!/usr/bin/env node
-import { CliOptions } from './types/index.js';
-export declare const runEdsger: (options: CliOptions) => Promise<void>;
+export * from './cli/index.js';
+import './cli/index.js';

package/dist/cli.js

@@ -1,100 +1,5 @@
 #!/usr/bin/env node
-var _a, _b, _c;
-import { Command } from 'commander';
-import { readFileSync } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
-import { config as dotenvConfig } from 'dotenv';
-import { logError } from './utils/logger.js';
-import { runWorkflow } from './cli/commands/workflow-command.js';
-import { runFeatureAnalysis } from './cli/commands/feature-analysis-command.js';
-import { runTechnicalDesign } from './cli/commands/technical-design-command.js';
-import { runCodeImplementation } from './cli/commands/code-implementation-command.js';
-import { runFunctionalTestingCommand } from './cli/commands/functional-testing-command.js';
-import { runCodeReview } from './cli/commands/code-review-command.js';
-// Get package.json version dynamically
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-const packageJson = JSON.parse(readFileSync(join(__dirname, '../package.json'), 'utf-8'));
-const version = packageJson.version;
-// Load environment variables from .env file
-// Load from current working directory (where the command is run)
-dotenvConfig({ path: join(process.cwd(), '.env') });
-const program = new Command();
-program
-    .name('edsger')
-    .description('AI-powered code review CLI tool using Claude Code SDK')
-    .version(version);
-program
-    .option('-r, --review', 'Review code changes')
-    .option('-s, --staged', 'Review only staged changes')
-    .option('-f, --files <patterns...>', 'Review specific file patterns')
-    .option('--feature-analysis <featureId>', 'Analyze feature and generate user stories/test cases')
-    .option('--technical-design <featureId>', 'Generate technical design for a feature')
-    .option('--implement <featureId>', 'Implement code for a feature based on its specifications')
-    .option('--test <featureId>', 'Run functional tests for a feature using Playwright')
-    .option('--workflow', 'Run continuous workflow processor to handle ready_for_dev features')
-    .option('-c, --config <path>', 'Path to config file')
-    .option('-v, --verbose', 'Verbose output');
-program.action(async (options) => {
-    try {
-        await runEdsger(options);
-    }
-    catch (error) {
-        logError(error instanceof Error ? error.message : String(error));
-        process.exit(1);
-    }
-});
-export const runEdsger = async (options) => {
-    // Handle workflow mode
-    if (options.workflow) {
-        await runWorkflow(options);
-        return;
-    }
-    // Handle feature analysis mode
-    if (options.featureAnalysis) {
-        await runFeatureAnalysis(options);
-        return;
-    }
-    // Handle technical design mode
-    if (options.technicalDesign) {
-        await runTechnicalDesign(options);
-        return;
-    }
-    // Handle code implementation mode
-    if (options.implement) {
-        await runCodeImplementation(options);
-        return;
-    }
-    // Handle functional testing mode
-    if (options.test) {
-        await runFunctionalTestingCommand(options);
-        return;
-    }
-    // Handle code review mode (explicit --review flag or as fallback)
-    if (options.review || options.staged || options.files) {
-        await runCodeReview(options);
-        return;
-    }
-    // Default to workflow mode if environment is configured, otherwise code review
-    const productId = process.env.EDSGER_PRODUCT_ID;
-    const mcpToken = process.env.EDSGER_MCP_TOKEN;
-    if (productId && mcpToken) {
-        // Environment is configured for workflow, use that as default
-        options.workflow = true;
-        await runWorkflow(options);
-    }
-    else {
-        // Default to code review mode
-        await runCodeReview(options);
-    }
-};
-// Only parse when this file is run directly (not imported as module)
-// Check if this is the main module being executed
-const isMainModule = import.meta.url === `file://${process.argv[1]}` ||
-    ((_a = process.argv[1]) === null || _a === void 0 ? void 0 : _a.endsWith('/edsger')) ||
-    ((_b = process.argv[1]) === null || _b === void 0 ? void 0 : _b.endsWith('\\edsger')) ||
-    ((_c = process.argv[1]) === null || _c === void 0 ? void 0 : _c.endsWith('cli.js'));
-if (isMainModule) {
-    program.parse();
-}
+// Re-export the CLI module
+export * from './cli/index.js';
+// Import and trigger the CLI parsing if this file is run directly
+import './cli/index.js';

package/dist/phases/code-implementation/analyzer-helpers.d.ts

@@ -0,0 +1,28 @@
+import { EdsgerConfig } from '../../types/index.js';
+import { ChecklistPhaseContext } from '../../services/checklist.js';
+import { ChecklistVerificationResult } from '../code-implementation-verification/index.js';
+import { CodeImplementationResult } from './analyzer.js';
+export interface VerificationCycleResult {
+    passed: boolean;
+    verificationResult: ChecklistVerificationResult | null;
+    nextPrompt: string | null;
+}
+/**
+ * Perform a complete verification cycle:
+ * 1. Verify code implementation against checklist
+ * 2. Log results to audit logs
+ * 3. Create improvement prompt if verification failed
+ */
+export declare function performVerificationCycle(branchName: string, baseBranch: string, checklistContext: ChecklistPhaseContext | null, featureId: string, featureName: string, featureDescription: string | undefined, config: EdsgerConfig, currentIteration: number, maxIterations: number, mcpServerUrl: string, mcpToken: string, verbose?: boolean): Promise<VerificationCycleResult>;
+/**
+ * Build successful implementation result
+ */
+export declare function buildImplementationResult(featureId: string, branchName: string, implementationSummary: string, filesModified: string[], commitHash: string, iterations: number, checklistResults?: any[], checklistItemResults?: any[]): CodeImplementationResult;
+/**
+ * Build verification failure result
+ */
+export declare function buildVerificationFailureResult(featureId: string, branchName: string, implementationSummary: string, filesModified: string[], commitHash: string, verificationResult: ChecklistVerificationResult, iterations: number): CodeImplementationResult;
+/**
+ * Build error result for no implementation
+ */
+export declare function buildNoResultsError(featureId: string, branchName: string): CodeImplementationResult;