remove console logs

Author: Andy Klier

auto-commit-approach.js

@@ -0,0 +1,46 @@
+// One-file approach hack to keep the repository clean
+// Instead of using the state management, we'll just hook into the MCP server validation
+// If `--require-clean-branch` is enabled, we'll auto-commit changes before validating
+
+// Update the validateGitStatus function to auto-commit changes
+async function validateGitStatus(filePath, promptId) {
+  if (!gitConfig.requireCleanBranch) {
+    return; // Git validation is disabled
+  }
+  
+  const { isRepo, isClean, repoPath } = await isGitClean(filePath);
+  
+  // When requireCleanBranch is set, we require the file to be in a Git repository
+  if (!isRepo) {
+    throw new Error(
+      "The file " + filePath + " is not in a Git repository. " +
+      "This server is configured to require files to be in Git repositories with clean branches."
+    );
+  }
+  
+  // Auto-commit if repository is not clean
+  if (!isClean) {
+    // Attempt to autocommit
+    try {
+      await autoCommit(repoPath);
+    } catch (error) {
+      // If auto-commit fails, throw the original error
+      throw new Error(
+        "Git repository at " + repoPath + " has uncommitted changes. " + 
+        "This server is configured to require a clean branch before allowing changes."
+      );
+    }
+  }
+}
+
+// Simple auto-commit function
+async function autoCommit(repoPath) {
+  const git = simpleGit(repoPath);
+  
+  // Stage all changes
+  await git.add('.');
+  
+  // Create a commit message with timestamp
+  const timestamp = new Date().toISOString();
+  await git.commit('Auto-commit from MCP server at ' + timestamp);
+}

src/filesystem/index.ts

@@ -130,7 +130,7 @@ async function isGitClean(filePath: string): Promise<{isRepo: boolean, isClean:
 
     return { isRepo: true, isClean, repoPath };
   } catch (error) {
-    console.error('Error checking Git status:', error);
+    // Error checking Git status
     return { isRepo: false, isClean: false, repoPath: null };
   }
 }
@@ -146,7 +146,7 @@ async function validateGitStatus(filePath: string, promptId?: string): Promise<v
   // Skip if we've already checked in this prompt
   const hasValidated = await hasValidatedInPrompt(promptId);
   if (hasValidated) {
-    console.log('Skipping validation for ' + filePath + ' - already validated in this prompt');
+    // Skip validation - already validated
     return;
   }
 
@@ -170,7 +170,6 @@ async function validateGitStatus(filePath: string, promptId?: string): Promise<v
 
   // Mark that we've checked in this prompt
   await markValidatedInPrompt(promptId);
-  console.log('Validation passed for ' + filePath + ' - marked as checked');
 }
 
 // Git validation utilities
@@ -595,7 +594,6 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 
     // Generate a unique prompt ID for this request
     const promptId = 'prompt-' + Date.now() + '-' + Math.floor(Math.random() * 10000);
-    console.log('Processing request ' + name + ' with promptId: ' + promptId);
 
     // Get the response from the appropriate tool handler
     let response;

src/filesystem/state-utils.ts

@@ -32,7 +32,7 @@ export async function resetValidationState(): Promise<void> {
     state.checkedThisPrompt = false;
     state.promptId = null;
     await saveState(state);
-    console.log("State reset for next prompt");
+    // State reset for next prompt
   }
 }
 
@@ -66,5 +66,5 @@ export async function markValidatedInPrompt(promptId?: string): Promise<void> {
   state.promptId = promptId || 'prompt-' + Date.now();
   state.lastCheckTime = Date.now();
   await saveState(state);
-  console.log('Validation marked as complete for prompt: ' + state.promptId);
+  // Validation marked as complete for prompt
 }