From f696f8e0f1a49581f598b732ebd7f8a9f54ac0c1 Mon Sep 17 00:00:00 2001 From: Daniel Riccio Date: Tue, 14 Jan 2025 18:03:03 -0500 Subject: [PATCH] refactor: remove debug functionality --- .../strategies/new-unified/edit-strategies.ts | 33 ++++--------------- src/core/diff/strategies/new-unified/index.ts | 2 +- 2 files changed, 7 insertions(+), 28 deletions(-) diff --git a/src/core/diff/strategies/new-unified/edit-strategies.ts b/src/core/diff/strategies/new-unified/edit-strategies.ts index 05f5732..f8dc166 100644 --- a/src/core/diff/strategies/new-unified/edit-strategies.ts +++ b/src/core/diff/strategies/new-unified/edit-strategies.ts @@ -253,12 +253,11 @@ export async function applyEdit( hunk: Hunk, content: string[], matchPosition: number, - confidence: number, - debug: string = '', + confidence: number, minConfidence: number = 0.9 ): Promise { // Don't attempt regular edits if confidence is too low - if (confidence < minConfidence && debug === '') { + if (confidence < minConfidence) { console.log(`Search confidence (${confidence}) below minimum threshold (${minConfidence}), trying git fallback...`); return applyGitFallback(hunk, content); } @@ -270,30 +269,10 @@ export async function applyEdit( { name: 'git-fallback', apply: () => applyGitFallback(hunk, content) } ]; - if (debug !== '') { - // In debug mode, try all strategies including git fallback - const results = await Promise.all([ - ...strategies.map(async strategy => { - console.log(`Attempting edit with ${strategy.name} strategy...`); - const result = await strategy.apply(); - console.log(`Strategy ${strategy.name} succeeded with confidence ${result.confidence}`); - return result; - }) - ]); - - return results.find(result => result.strategy === debug) || { confidence: 0, result: content, strategy: 'none' }; - } else { - // Normal mode - try strategies sequentially until one succeeds - for (const strategy of strategies) { - const result = await strategy.apply(); - if (result.confidence === 1) { - return result; - } - } - // If all strategies fail, try git fallback - - const result = await applyGitFallback(hunk, content); - if(result.confidence === 1) { + // Try strategies sequentially until one succeeds + for (const strategy of strategies) { + const result = await strategy.apply(); + if (result.confidence >= minConfidence) { return result; } } diff --git a/src/core/diff/strategies/new-unified/index.ts b/src/core/diff/strategies/new-unified/index.ts index 7737d90..3e70d98 100644 --- a/src/core/diff/strategies/new-unified/index.ts +++ b/src/core/diff/strategies/new-unified/index.ts @@ -206,7 +206,7 @@ Your diff here const contextStr = prepareSearchString(hunk.changes) const { index: matchPosition, confidence, strategy } = findBestMatch(contextStr, result, 0, this.confidenceThreshold) - const editResult = await applyEdit(hunk, result, matchPosition, confidence, '', this.confidenceThreshold) + const editResult = await applyEdit(hunk, result, matchPosition, confidence, this.confidenceThreshold) if (editResult.confidence >= this.confidenceThreshold) { result = editResult.result } else {