mcp-adr-analysis-server 2.0.21 → 2.1.0

package/dist/src/tools/smart-git-push-tool.js

@@ -31,14 +31,14 @@ import { McpAdrError } from '../types/index.js';
 import { execSync } from 'child_process';
 import { readFileSync, existsSync, statSync } from 'fs';
 import { join, basename } from 'path';
-import { jsonSafeFilePath, jsonSafeMarkdownList, jsonSafeError, jsonSafeUserInput } from '../utils/json-safe.js';
+import { jsonSafeFilePath, jsonSafeMarkdownList, jsonSafeError, jsonSafeUserInput, } from '../utils/json-safe.js';
 import { validateMcpResponse } from '../utils/mcp-response-validator.js';
 import { KnowledgeGraphManager } from '../utils/knowledge-graph-manager.js';
 /**
  * Main smart git push function - Knowledge Graph Enhanced Version
  */
 async function _smartGitPushInternal(args) {
-    const { branch, message, skipValidation = false, allowedArtifacts = [], sensitivityLevel = 'moderate', dryRun = false, projectPath = process.cwd(), checkReleaseReadiness = false, releaseType = 'minor', skipKnowledgeGraphAnalysis = false } = args;
+    const { branch, message, skipValidation = false, allowedArtifacts = [], sensitivityLevel = 'moderate', dryRun = false, projectPath = process.cwd(), checkReleaseReadiness = false, releaseType = 'minor', skipKnowledgeGraphAnalysis = false, } = args;
     try {
         // Step 1: Get staged files using git CLI
         const stagedFiles = await getStagedFiles(projectPath);
@@ -61,26 +61,19 @@ async function _smartGitPushInternal(args) {
                 releaseReadinessResult = await analyzeReleaseReadiness({
                     projectPath,
                     releaseType,
-                    includeAnalysis: true
+                    includeAnalysis: true,
                 });
                 // Update deployment readiness score in health scoring system
                 try {
-                    const { ProjectHealthScoring } = await import('../utils/project-health-scoring.js');
-                    const healthScoring = new ProjectHealthScoring(projectPath);
-                    await healthScoring.updateDeploymentReadinessScore({
-                        releaseScore: releaseReadinessResult.score,
-                        milestoneCompletion: releaseReadinessResult.milestones.length > 0 ?
-                            releaseReadinessResult.milestones.reduce((sum, m) => sum + m.completionRate, 0) / releaseReadinessResult.milestones.length : 0.5,
-                        criticalBlockers: releaseReadinessResult.blockers.filter((b) => b.severity === 'error').length,
-                        warningBlockers: releaseReadinessResult.blockers.filter((b) => b.severity === 'warning').length,
-                        gitHealthScore: releaseReadinessResult.blockers.some((b) => b.type === 'unstable-code') ? 30 : 80
-                    });
+                    // ProjectHealthScoring removed - skip health scoring
+                    console.warn('⚠️ ProjectHealthScoring is deprecated and was removed in memory-centric transformation');
+                    // Skip health scoring update - ProjectHealthScoring removed
                 }
-                catch (healthError) {
+                catch {
                     // Silently handle health scoring errors
                 }
             }
-            catch (error) {
+            catch {
                 // Silently handle release readiness analysis errors
             }
         }
@@ -105,16 +98,23 @@ No staged files found. Use \`git add\` to stage files before pushing.
 - **Goal Progress**: ${kgAnalysis.projectGoalProgress.overallProgress}%
 - **Task Dependencies**: ${kgAnalysis.taskDependencies.completed.length} completed, ${kgAnalysis.taskDependencies.pending.length} pending
 
-${kgAnalysis.activeIntents.length > 0 ? `
+${kgAnalysis.activeIntents.length > 0
+                    ? `
 ### Active Intents
 ${kgAnalysis.activeIntents.map(intent => `- **${intent.currentStatus}**: ${jsonSafeUserInput(intent.humanRequest.substring(0, 80))}...`).join('\n')}
-` : ''}
+`
+                    : ''}
 
-${kgAnalysis.taskDependencies.pending.length > 0 ? `
+${kgAnalysis.taskDependencies.pending.length > 0
+                    ? `
 ### Pending Tasks
 Consider working on these tasks before your next push:
-${kgAnalysis.taskDependencies.pending.slice(0, 5).map(task => `- ${jsonSafeUserInput(task)}`).join('\n')}
-` : ''}
+${kgAnalysis.taskDependencies.pending
+                        .slice(0, 5)
+                        .map(task => `- ${jsonSafeUserInput(task)}`)
+                        .join('\n')}
+`
+                    : ''}
 `;
             }
             // Add release readiness info if checked
@@ -127,16 +127,18 @@ ${jsonSafeUserInput(releaseReadinessResult.summary)}
 ### Recommendations
 ${jsonSafeMarkdownList(releaseReadinessResult.recommendations)}
 
-${releaseReadinessResult.isReady ?
-                    '✅ **Project is ready for release!** Consider creating a release after staging files.' :
-                    '❌ **Project is not ready for release.** Address blockers before proceeding.'}
+${releaseReadinessResult.isReady
+                    ? '✅ **Project is ready for release!** Consider creating a release after staging files.'
+                    : '❌ **Project is not ready for release.** Address blockers before proceeding.'}
 `;
             }
             return {
-                content: [{
+                content: [
+                    {
                         type: 'text',
-                        text: responseText
-                    }]
+                        text: responseText,
+                    },
+                ],
             };
         }
         // Step 3: Analyze staged files if validation is enabled
@@ -145,7 +147,7 @@ ${releaseReadinessResult.isReady ?
             validationResults = await validateStagedFiles(stagedFiles, {
                 sensitivityLevel,
                 allowedArtifacts,
-                projectPath
+                projectPath,
             });
         }
         // Step 4: Check for blocking conditions (Enhanced with Knowledge Graph)
@@ -174,14 +176,16 @@ ${releaseReadinessResult.isReady ?
 Push blocked due to critical issues that must be resolved.
 
 ## Issues Found
-${issues.map(issue => `
+${issues
+                .map(issue => `
 
 ### ${jsonSafeFilePath(issue.file)}
 ${issue.issues.map(i => `- **${i.severity.toUpperCase()}**: ${jsonSafeUserInput(i.message)}`).join('\n')}
 
 **Suggestions:**
 ${jsonSafeMarkdownList(issue.suggestions)}
-`).join('\n')}
+`)
+                .join('\n')}
 `;
             // Add release readiness info if checked and blocked
             if (releaseReadinessResult && releaseReadinessBlocked) {
@@ -191,7 +195,10 @@ ${jsonSafeMarkdownList(issue.suggestions)}
 ${jsonSafeUserInput(releaseReadinessResult.summary)}
 
 ### Critical Blockers
-${releaseReadinessResult.blockers.filter(b => b.severity === 'error').map(b => `- **${b.type}**: ${jsonSafeUserInput(b.message)}`).join('\n')}
+${releaseReadinessResult.blockers
+                    .filter(b => b.severity === 'error')
+                    .map(b => `- **${b.type}**: ${jsonSafeUserInput(b.message)}`)
+                    .join('\n')}
 
 ### Recommendations
 ${jsonSafeMarkdownList(releaseReadinessResult.recommendations)}
@@ -228,14 +235,19 @@ ${kgBlockingConditions.map(bc => `- ${jsonSafeUserInput(bc.recommendation)}`).jo
 - **Goal Progress**: ${kgAnalysis.projectGoalProgress.overallProgress}%
 
 ### Warnings
-${kgAnalysis.blockingConditions.filter(bc => bc.severity === 'warning').map(bc => `- **${bc.type}**: ${jsonSafeUserInput(bc.message)}`).join('\n')}
+${kgAnalysis.blockingConditions
+                    .filter(bc => bc.severity === 'warning')
+                    .map(bc => `- **${bc.type}**: ${jsonSafeUserInput(bc.message)}`)
+                    .join('\n')}
 `;
             }
             return {
-                content: [{
+                content: [
+                    {
                         type: 'text',
-                        text: cancelText
-                    }]
+                        text: cancelText,
+                    },
+                ],
             };
         }
         // Step 5: Execute git push if not dry run
@@ -244,15 +256,15 @@ ${kgAnalysis.blockingConditions.filter(bc => bc.severity === 'warning').map(bc =
             // Update TODO tasks with KG integration
             if (kgAnalysis) {
                 try {
-                    const { TodoJsonManager } = await import('../utils/todo-json-manager.js');
-                    const todoManager = new TodoJsonManager(projectPath);
-                    await updateTodoTasksFromGitPushWithKG(todoManager, stagedFiles, kgAnalysis, releaseReadinessResult);
+                    // TodoJsonManager removed - skipping todo validation
+                    console.warn('⚠️ TodoJsonManager is deprecated and was removed in memory-centric transformation');
+                    // Skip todo task update - TodoJsonManager removed
                 }
                 catch (todoError) {
                     console.error('Error updating TODO tasks with KG:', todoError);
                 }
             }
-            let successText = `# Smart Git Push - Success ✅
+            const successText = `# Smart Git Push - Success ✅
 
 ## Push Details
 - **Branch**: ${branch || 'current'}
@@ -266,7 +278,8 @@ ${kgAnalysis ? `- **Knowledge Graph Analysis**: ✅ Completed` : ''}
 ## Files Pushed
 ${stagedFiles.map(f => `- ${jsonSafeFilePath(f.path)} (${f.status})`).join('\n')}
 
-${kgAnalysis ? `
+${kgAnalysis
+                ? `
 ## Knowledge Graph Analysis
 ### Project Context
 - **Active Intents**: ${kgAnalysis.activeIntents.length}
@@ -277,28 +290,41 @@ ${kgAnalysis ? `
 ### Architectural Insights
 ${kgAnalysis.architecturalAlignment.details.map(detail => `- ${jsonSafeUserInput(detail)}`).join('\n')}
 
-${kgAnalysis.blockingConditions.length > 0 ? `
+${kgAnalysis.blockingConditions.length > 0
+                    ? `
 ### Warnings & Recommendations
-${kgAnalysis.blockingConditions.filter(bc => bc.severity === 'warning').map(bc => `- **${bc.type}**: ${jsonSafeUserInput(bc.message)}`).join('\n')}
-` : ''}
-
-${kgAnalysis.projectGoalProgress.intentProgress.length > 0 ? `
+${kgAnalysis.blockingConditions
+                        .filter(bc => bc.severity === 'warning')
+                        .map(bc => `- **${bc.type}**: ${jsonSafeUserInput(bc.message)}`)
+                        .join('\n')}
+`
+                    : ''}
+
+${kgAnalysis.projectGoalProgress.intentProgress.length > 0
+                    ? `
 ### Intent Progress
 ${kgAnalysis.projectGoalProgress.intentProgress.map(intent => `- **${intent.status}**: ${jsonSafeUserInput(intent.humanRequest.substring(0, 60))}... (${intent.progress}%)`).join('\n')}
-` : ''}
-` : ''}
+`
+                    : ''}
+`
+                : ''}
 
-${issues.length > 0 ? `
+${issues.length > 0
+                ? `
 
 ## Validation Issues (Auto-Approved)
-${issues.map(issue => `
+${issues
+                    .map(issue => `
 
 ### ${jsonSafeFilePath(issue.file)}
 ${issue.issues.map(i => `- **${i.severity.toUpperCase()}**: ${jsonSafeUserInput(i.message)}`).join('\n')}
-`).join('\n')}
-` : ''}
+`)
+                    .join('\n')}
+`
+                : ''}
 
-${releaseReadinessResult ? `
+${releaseReadinessResult
+                ? `
 
 ## Release Readiness Analysis
 ${jsonSafeUserInput(releaseReadinessResult.summary)}
@@ -306,10 +332,11 @@ ${jsonSafeUserInput(releaseReadinessResult.summary)}
 ### Post-Push Recommendations
 ${jsonSafeMarkdownList(releaseReadinessResult.recommendations)}
 
-${releaseReadinessResult.isReady ?
-                '🎉 **Congratulations!** This push completed a release-ready state. Consider creating a release tag.' :
-                '📋 **Next Steps**: Address remaining blockers to achieve release readiness.'}
-` : ''}
+${releaseReadinessResult.isReady
+                    ? '🎉 **Congratulations!** This push completed a release-ready state. Consider creating a release tag.'
+                    : '📋 **Next Steps**: Address remaining blockers to achieve release readiness.'}
+`
+                : ''}
 
 ## Git Output
 \`\`\`
@@ -324,15 +351,17 @@ ${releaseReadinessResult?.isReady ? '- Consider creating a release tag or publis
 ${kgAnalysis ? '- Review knowledge graph insights for follow-up tasks' : ''}
 `;
             return {
-                content: [{
+                content: [
+                    {
                         type: 'text',
-                        text: successText
-                    }]
+                        text: successText,
+                    },
+                ],
             };
         }
         else {
             // Dry run - show what would happen
-            let dryRunText = `# Smart Git Push - Dry Run 🔍
+            const dryRunText = `# Smart Git Push - Dry Run 🔍
 
 ## Analysis Complete
 - **Files to Push**: ${stagedFiles.length}
@@ -345,7 +374,8 @@ ${kgAnalysis ? `- **Knowledge Graph Analysis**: ✅ Completed` : ''}
 ## Staged Files
 ${stagedFiles.map(f => `- ${jsonSafeFilePath(f.path)} (${f.status}) - ${f.size} bytes`).join('\n')}
 
-${kgAnalysis ? `
+${kgAnalysis
+                ? `
 ## Knowledge Graph Analysis Preview
 ### Project Context
 - **Active Intents**: ${kgAnalysis.activeIntents.length}
@@ -356,31 +386,41 @@ ${kgAnalysis ? `
 ### Architectural Assessment
 ${kgAnalysis.architecturalAlignment.details.map(detail => `- ${jsonSafeUserInput(detail)}`).join('\n')}
 
-${kgAnalysis.blockingConditions.length > 0 ? `
+${kgAnalysis.blockingConditions.length > 0
+                    ? `
 ### Potential Issues
 ${kgAnalysis.blockingConditions.map(bc => `- **${bc.type}** (${bc.severity}): ${jsonSafeUserInput(bc.message)}`).join('\n')}
-` : ''}
+`
+                    : ''}
 
-${kgAnalysis.projectGoalProgress.intentProgress.length > 0 ? `
+${kgAnalysis.projectGoalProgress.intentProgress.length > 0
+                    ? `
 ### Intent Progress Impact
 ${kgAnalysis.projectGoalProgress.intentProgress.map(intent => `- **${intent.status}**: ${jsonSafeUserInput(intent.humanRequest.substring(0, 60))}... (${intent.progress}%)`).join('\n')}
-` : ''}
-` : ''}
+`
+                    : ''}
+`
+                : ''}
 
-${issues.length > 0 ? `
+${issues.length > 0
+                ? `
 
 ## Validation Issues Found
-${issues.map(issue => `
+${issues
+                    .map(issue => `
 
 ### ${jsonSafeFilePath(issue.file)}
 ${issue.issues.map(i => `- **${i.severity.toUpperCase()}**: ${jsonSafeUserInput(i.message)}`).join('\n')}
 
 **Suggestions:**
 ${jsonSafeMarkdownList(issue.suggestions)}
-`).join('\n')}
-` : '## ✅ No Validation Issues Found'}
+`)
+                    .join('\n')}
+`
+                : '## ✅ No Validation Issues Found'}
 
-${releaseReadinessResult ? `
+${releaseReadinessResult
+                ? `
 
 ## Release Readiness Analysis
 ${jsonSafeUserInput(releaseReadinessResult.summary)}
@@ -388,10 +428,11 @@ ${jsonSafeUserInput(releaseReadinessResult.summary)}
 ### Pre-Push Recommendations
 ${jsonSafeMarkdownList(releaseReadinessResult.recommendations)}
 
-${releaseReadinessResult.isReady ?
-                '🎉 **Ready for Release!** This push would complete all release requirements.' :
-                '📋 **Release Blockers**: Address these issues before considering this a release.'}
-` : ''}
+${releaseReadinessResult.isReady
+                    ? '🎉 **Ready for Release!** This push would complete all release requirements.'
+                    : '📋 **Release Blockers**: Address these issues before considering this a release.'}
+`
+                : ''}
 
 ## Command to Execute
 \`\`\`bash
@@ -402,10 +443,12 @@ git push${branch ? ` origin ${branch}` : ''}
 **Note:** This was a dry run. No files were actually pushed.
 `;
             return {
-                content: [{
+                content: [
+                    {
                         type: 'text',
-                        text: dryRunText
-                    }]
+                        text: dryRunText,
+                    },
+                ],
             };
         }
     }
@@ -421,7 +464,7 @@ async function getStagedFiles(projectPath) {
         // Get staged files with status
         const gitOutput = execSync('git diff --cached --name-status', {
             cwd: projectPath,
-            encoding: 'utf8'
+            encoding: 'utf8',
         });
         if (!gitOutput.trim()) {
             return [];
@@ -444,7 +487,7 @@ async function getStagedFiles(projectPath) {
                         content = readFileSync(fullPath, 'utf8');
                     }
                 }
-                catch (err) {
+                catch {
                     // Silently handle file read errors
                 }
             }
@@ -452,7 +495,7 @@ async function getStagedFiles(projectPath) {
                 path,
                 status: mapGitStatus(status || 'M'),
                 content: content || '',
-                size
+                size,
             });
         }
         return files;
@@ -466,11 +509,16 @@ async function getStagedFiles(projectPath) {
  */
 function mapGitStatus(status) {
     switch (status) {
-        case 'A': return 'added';
-        case 'M': return 'modified';
-        case 'D': return 'deleted';
-        case 'R': return 'renamed';
-        default: return 'modified';
+        case 'A':
+            return 'added';
+        case 'M':
+            return 'modified';
+        case 'D':
+            return 'deleted';
+        case 'R':
+            return 'renamed';
+        default:
+            return 'modified';
     }
 }
 /**
@@ -504,7 +552,7 @@ async function validateStagedFiles(files, options) {
             file: file.path,
             issues,
             suggestions,
-            approved: issues.length === 0 || issues.every(i => i.severity === 'info')
+            approved: issues.length === 0 || issues.every(i => i.severity === 'info'),
         });
     }
     return results;
@@ -514,25 +562,29 @@ async function validateStagedFiles(files, options) {
  */
 async function checkSensitiveContent(content, filePath) {
     try {
-        // Use enhanced sensitive detector
-        const { analyzeSensitiveContent } = await import('../utils/enhanced-sensitive-detector.js');
+        // Use gitleaks for industry-standard secret detection
+        const { analyzeSensitiveContent } = await import('../utils/gitleaks-detector.js');
         const result = await analyzeSensitiveContent(filePath, content);
         // Convert to ValidationIssue format
         const issues = [];
         for (const match of result.matches) {
             issues.push({
                 type: 'sensitive-content',
-                severity: match.pattern.severity === 'critical' ? 'error' :
-                    match.pattern.severity === 'high' ? 'error' :
-                        match.pattern.severity === 'medium' ? 'warning' : 'info',
+                severity: match.pattern.severity === 'critical'
+                    ? 'error'
+                    : match.pattern.severity === 'high'
+                        ? 'error'
+                        : match.pattern.severity === 'medium'
+                            ? 'warning'
+                            : 'info',
                 message: `${match.pattern.description}: ${match.match}`,
                 pattern: match.pattern.name,
-                line: match.line
+                line: match.line,
             });
         }
         return issues;
     }
-    catch (error) {
+    catch {
         // Silently handle sensitive content check errors
         return [];
     }
@@ -550,10 +602,13 @@ async function checkLLMArtifacts(filePath, content) {
         for (const match of result.matches) {
             const issue = {
                 type: 'llm-artifact',
-                severity: match.pattern.severity === 'error' ? 'error' :
-                    match.pattern.severity === 'warning' ? 'warning' : 'info',
+                severity: match.pattern.severity === 'error'
+                    ? 'error'
+                    : match.pattern.severity === 'warning'
+                        ? 'warning'
+                        : 'info',
                 message: `${match.pattern.description}: ${match.match}`,
-                pattern: match.pattern.name
+                pattern: match.pattern.name,
             };
             if (match.line !== undefined) {
                 issue.line = match.line;
@@ -562,7 +617,7 @@ async function checkLLMArtifacts(filePath, content) {
         }
         return issues;
     }
-    catch (error) {
+    catch {
         // Silently handle LLM artifact check errors
         return [];
     }
@@ -573,24 +628,81 @@ async function checkLLMArtifacts(filePath, content) {
 async function checkLocationRules(filePath, allowedArtifacts) {
     try {
         // Use enhanced location filter
-        const { validateFileLocation } = await import('../utils/location-filter.js');
+        const { validateFileLocationIntelligent } = await import('../utils/location-filter.js');
         // Skip if explicitly allowed
         if (allowedArtifacts.includes(basename(filePath)) || allowedArtifacts.includes(filePath)) {
             return [];
         }
-        const result = validateFileLocation(filePath);
+        // Read file content for intelligent analysis
+        let fileContent;
+        try {
+            fileContent = readFileSync(filePath, 'utf-8');
+        }
+        catch {
+            console.warn(`Could not read file for analysis: ${filePath}`);
+        }
+        const result = await validateFileLocationIntelligent(filePath, fileContent);
         if (!result.isValid) {
-            return [{
+            const issues = [
+                {
                     type: 'wrong-location',
-                    severity: result.severity === 'error' ? 'error' :
-                        result.severity === 'warning' ? 'warning' : 'info',
+                    severity: result.severity === 'error'
+                        ? 'error'
+                        : result.severity === 'warning'
+                            ? 'warning'
+                            : 'info',
                     message: result.message,
-                    pattern: result.rule?.name || 'location-rule'
-                }];
+                    pattern: result.rule?.name || 'location-rule',
+                },
+            ];
+            // Add tree-sitter security insights
+            if (result.treeSitterAnalysis) {
+                const analysis = result.treeSitterAnalysis;
+                // Critical security issues
+                if (analysis.hasSecrets) {
+                    issues.push({
+                        type: 'security-risk',
+                        severity: 'error',
+                        message: `🔐 Security Risk: File contains ${analysis.secrets.length} potential secret(s). Secrets should be in secure locations or environment variables.`,
+                        pattern: 'tree-sitter-secrets',
+                    });
+                }
+                // Dangerous imports
+                const dangerousImports = analysis.imports.filter(i => i.isDangerous);
+                if (dangerousImports.length > 0) {
+                    issues.push({
+                        type: 'security-risk',
+                        severity: 'warning',
+                        message: `⚠️  Dangerous imports detected: ${dangerousImports.map(i => i.module).join(', ')}. Consider safer alternatives.`,
+                        pattern: 'tree-sitter-dangerous-imports',
+                    });
+                }
+                // Infrastructure security risks
+                const riskyInfra = analysis.infraStructure.filter(i => i.securityRisks.length > 0);
+                if (riskyInfra.length > 0) {
+                    const risks = riskyInfra.flatMap(i => i.securityRisks);
+                    issues.push({
+                        type: 'security-risk',
+                        severity: 'error',
+                        message: `🏗️  Infrastructure security risks: ${risks.join(', ')}. Review before deployment.`,
+                        pattern: 'tree-sitter-infra-security',
+                    });
+                }
+                // Architectural violations
+                if (result.intelligentReasons?.some(r => r.includes('violates'))) {
+                    issues.push({
+                        type: 'architecture-violation',
+                        severity: 'error',
+                        message: `🏗️  Architectural violation detected. This may affect system maintainability and scalability.`,
+                        pattern: 'tree-sitter-architecture',
+                    });
+                }
+            }
+            return issues;
         }
         return [];
     }
-    catch (error) {
+    catch {
         // Silently handle location rule check errors
         return [];
     }
@@ -633,7 +745,7 @@ async function executePush(projectPath, branch, message) {
         if (message) {
             const commitOutput = execSync(`git commit -m "${message}"`, {
                 cwd: projectPath,
-                encoding: 'utf8'
+                encoding: 'utf8',
             });
             output += `Commit:\n${commitOutput}\n\n`;
         }
@@ -641,7 +753,7 @@ async function executePush(projectPath, branch, message) {
         const pushCommand = branch ? `git push origin ${branch}` : 'git push';
         const pushOutput = execSync(pushCommand, {
             cwd: projectPath,
-            encoding: 'utf8'
+            encoding: 'utf8',
         });
         output += `Push:\n${pushOutput}`;
         return { output, success: true };
@@ -668,11 +780,13 @@ export async function smartGitPushMcpSafe(args) {
     catch (error) {
         // Always return a safe MCP response, never throw
         const errorResponse = {
-            content: [{
+            content: [
+                {
                     type: 'text',
-                    text: `# Smart Git Push - Error\n\n**Error**: ${jsonSafeError(error)}\n\nPlease check your git configuration and try again.`
-                }],
-            isError: true
+                    text: `# Smart Git Push - Error\n\n**Error**: ${jsonSafeError(error)}\n\nPlease check your git configuration and try again.`,
+                },
+            ],
+            isError: true,
         };
         return validateMcpResponse(errorResponse);
     }
@@ -683,11 +797,12 @@ export async function smartGitPushMcpSafe(args) {
  */
 async function analyzeKnowledgeGraphContext(projectPath, stagedFiles) {
     const kgManager = new KnowledgeGraphManager();
-    const { TodoJsonManager } = await import('../utils/todo-json-manager.js');
-    const todoManager = new TodoJsonManager(projectPath);
+    // TodoJsonManager removed - use mcp-shrimp-task-manager for task management
+    console.warn('⚠️ TodoJsonManager is deprecated and was removed in memory-centric transformation');
     // Load knowledge graph and TODO data
     const kg = await kgManager.loadKnowledgeGraph();
-    const todoData = await todoManager.loadTodoData();
+    // Skip todo data loading - TodoJsonManager removed
+    const todoData = { tasks: {}, metadata: { totalTasks: 0, completedTasks: 0 } };
     // Get active intents
     const activeIntents = kg.intents.filter(i => i.currentStatus === 'executing' || i.currentStatus === 'planning');
     // Analyze file changes against project context
@@ -708,7 +823,7 @@ async function analyzeKnowledgeGraphContext(projectPath, stagedFiles) {
         blockingConditions,
         architecturalAlignment,
         taskDependencies,
-        projectGoalProgress
+        projectGoalProgress,
     };
 }
 /**
@@ -717,7 +832,7 @@ async function analyzeKnowledgeGraphContext(projectPath, stagedFiles) {
 async function analyzeFileChangesContext(stagedFiles, activeIntents, todoData) {
     const fileAnalysis = {
         intentAlignment: [],
-        todoTaskProgress: []
+        todoTaskProgress: [],
     };
     // Check how files relate to active intents
     for (const intent of activeIntents) {
@@ -735,7 +850,7 @@ async function analyzeFileChangesContext(stagedFiles, activeIntents, todoData) {
         fileAnalysis.intentAlignment.push({
             intentId: intent.intentId,
             alignedFiles,
-            conflictingFiles
+            conflictingFiles,
         });
     }
     // Check how files relate to TODO tasks
@@ -743,15 +858,15 @@ async function analyzeFileChangesContext(stagedFiles, activeIntents, todoData) {
     for (const task of tasks) {
         const relatedFiles = stagedFiles.filter(file => {
             const fileName = basename(file.path);
-            return task.title.toLowerCase().includes(fileName.toLowerCase()) ||
+            return (task.title.toLowerCase().includes(fileName.toLowerCase()) ||
                 task.description?.toLowerCase().includes(fileName.toLowerCase()) ||
-                task.title.toLowerCase().includes(file.path.toLowerCase());
+                task.title.toLowerCase().includes(file.path.toLowerCase()));
         });
         if (relatedFiles.length > 0) {
             fileAnalysis.todoTaskProgress.push({
                 taskId: task.id,
                 relatedFiles: relatedFiles.map(f => f.path),
-                progressImpact: 'positive' // Assume file changes indicate progress
+                progressImpact: 'positive', // Assume file changes indicate progress
             });
         }
     }
@@ -800,7 +915,7 @@ async function analyzeArchitecturalAlignment(stagedFiles, activeIntents, _projec
     return {
         score: Math.max(0, Math.min(100, alignmentScore)),
         details,
-        recommendations
+        recommendations,
     };
 }
 /**
@@ -834,7 +949,7 @@ async function analyzeProjectGoalProgress(_kg, _todoData, activeIntents) {
             intentId: intent.intentId,
             humanRequest: intent.humanRequest,
             progress,
-            status: intent.currentStatus
+            status: intent.currentStatus,
         };
     });
     const overallProgress = intentProgress.length > 0
@@ -842,7 +957,7 @@ async function analyzeProjectGoalProgress(_kg, _todoData, activeIntents) {
         : 0;
     return {
         overallProgress,
-        intentProgress
+        intentProgress,
     };
 }
 /**
@@ -857,7 +972,7 @@ async function determineBlockingConditions(_fileAnalysis, architecturalAlignment
             severity: 'error',
             message: `${taskDependencies.blocking.length} critical tasks are blocking this push`,
             recommendation: 'Complete or unblock critical tasks before pushing',
-            affectedFiles: stagedFiles.map(f => f.path)
+            affectedFiles: stagedFiles.map(f => f.path),
         });
     }
     // Check architectural alignment
@@ -867,7 +982,7 @@ async function determineBlockingConditions(_fileAnalysis, architecturalAlignment
             severity: 'warning',
             message: `Low architectural alignment score: ${architecturalAlignment.score}%`,
             recommendation: 'Review architectural compliance of changes',
-            affectedFiles: stagedFiles.map(f => f.path)
+            affectedFiles: stagedFiles.map(f => f.path),
         });
     }
     // Check goal regression
@@ -877,7 +992,7 @@ async function determineBlockingConditions(_fileAnalysis, architecturalAlignment
             severity: 'warning',
             message: `Low project goal progress: ${projectGoalProgress.overallProgress}%`,
             recommendation: 'Focus on completing active project goals',
-            affectedFiles: stagedFiles.map(f => f.path)
+            affectedFiles: stagedFiles.map(f => f.path),
         });
     }
     return blockingConditions;
@@ -893,113 +1008,4 @@ async function findRelevantAdrs(_stagedFiles, _projectPath) {
 /**
  * Update TODO tasks based on successful git push with Knowledge Graph integration
  */
-async function updateTodoTasksFromGitPushWithKG(todoManager, stagedFiles, kgAnalysis, releaseReadinessResult) {
-    try {
-        // Create intent for this git push
-        const kgManager = new KnowledgeGraphManager();
-        const intentId = await kgManager.createIntent(`Git push: ${stagedFiles.length} files`, [`Push ${stagedFiles.map(f => f.path).join(', ')}`], 'medium');
-        // Record tool execution
-        await kgManager.addToolExecution(intentId, 'smart_git_push', {
-            files: stagedFiles.map(f => f.path),
-            branch: 'current',
-            knowledgeGraphAnalysis: true
-        }, {
-            success: true,
-            filesProcessed: stagedFiles.length,
-            architecturalAlignment: kgAnalysis.architecturalAlignment.score
-        }, true, [], // todoTasksCreated
-        kgAnalysis.taskDependencies.completed // todoTasksModified
-        );
-        // Update intent status
-        await kgManager.updateIntentStatus(intentId, 'completed');
-        // Update TODO tasks
-        await updateTodoTasksFromGitPush(todoManager, stagedFiles, releaseReadinessResult);
-    }
-    catch (error) {
-        console.error('Error updating TODO tasks with KG:', error);
-    }
-}
-/**
- * Update TODO tasks based on successful git push
- */
-async function updateTodoTasksFromGitPush(todoManager, stagedFiles, releaseReadinessResult) {
-    try {
-        const data = await todoManager.loadTodoData();
-        const tasks = Object.values(data.tasks);
-        // 1. Auto-update tasks based on file changes
-        for (const file of stagedFiles) {
-            // Look for tasks that mention this file in their title or description
-            const relatedTasks = tasks.filter(task => task.title.toLowerCase().includes(basename(file.path).toLowerCase()) ||
-                task.description?.toLowerCase().includes(basename(file.path).toLowerCase()) ||
-                task.title.toLowerCase().includes(file.path.toLowerCase()));
-            for (const task of relatedTasks) {
-                if (task.status === 'pending' || task.status === 'in_progress') {
-                    // Update to in_progress if pending, or completed if already in_progress
-                    const newStatus = task.status === 'pending' ? 'in_progress' : 'completed';
-                    await todoManager.updateTask({
-                        taskId: task.id,
-                        updates: {
-                            status: newStatus,
-                            notes: `Auto-updated: ${file.path} was ${file.status} in git push`
-                        },
-                        reason: `Git push: ${file.path} ${file.status}`,
-                        triggeredBy: 'tool'
-                    });
-                }
-            }
-        }
-        // 2. Create follow-up tasks based on what was pushed
-        const followUpTasks = [];
-        // If documentation files were changed, create review tasks
-        const docFiles = stagedFiles.filter(f => f.path.match(/\.(md|txt|rst)$/i) &&
-            !f.path.includes('TODO.md'));
-        if (docFiles.length > 0) {
-            followUpTasks.push({
-                title: `Review updated documentation`,
-                description: `Review changes to: ${docFiles.map(f => f.path).join(', ')}`,
-                priority: 'medium',
-                category: 'documentation',
-                tags: ['review', 'documentation']
-            });
-        }
-        // If source code was changed, create testing tasks
-        const codeFiles = stagedFiles.filter(f => f.path.match(/\.(ts|js|py|java|cs|go|rb|php|swift|kt|rs|cpp|c|h)$/i));
-        if (codeFiles.length > 0) {
-            followUpTasks.push({
-                title: `Test changes in ${codeFiles.length} code files`,
-                description: `Verify functionality of: ${codeFiles.map(f => f.path).join(', ')}`,
-                priority: 'high',
-                category: 'testing',
-                tags: ['testing', 'verification']
-            });
-        }
-        // If release readiness improved, create release tasks
-        if (releaseReadinessResult?.isReady) {
-            const existingReleaseTasks = tasks.filter(task => task.title.toLowerCase().includes('release') ||
-                task.tags?.includes('release'));
-            if (existingReleaseTasks.length === 0) {
-                followUpTasks.push({
-                    title: `Prepare release - all requirements met`,
-                    description: `Project is now release-ready. Create release tag and publish.`,
-                    priority: 'critical',
-                    category: 'release',
-                    tags: ['release', 'deployment']
-                });
-            }
-        }
-        // Create follow-up tasks
-        for (const taskData of followUpTasks) {
-            await todoManager.createTask(taskData);
-        }
-        // 3. Update task metadata with git information
-        const now = new Date().toISOString();
-        data.metadata.lastGitPush = now;
-        data.metadata.lastPushFiles = stagedFiles.map(f => f.path);
-        await todoManager.saveTodoData(data);
-    }
-    catch (error) {
-        // Silently handle errors to avoid breaking git push
-        console.error('Error updating TODO tasks from git push:', error);
-    }
-}
 //# sourceMappingURL=smart-git-push-tool.js.map