Remove unnecessary comments referencing #4827

roomote · roomote · commit b7ba298a390f · 2025-06-18T15:45:01.000Z
Addresses comment by @cte in PR #4840 to remove redundant inline comments that reference the issue number, as this information is already captured in the commit history and PR description.
diff --git a/src/core/tools/applyDiffTool.ts b/src/core/tools/applyDiffTool.ts
@@ -165,8 +165,6 @@ export async function applyDiffToolLegacy(
 				return
 			}
 
-			// Checkpoint is now created in askApproval function before this point (fixes #4827)
-
 			// Call saveChanges to update the DiffViewProvider properties
 			await cline.diffViewProvider.saveChanges()
 
diff --git a/src/core/tools/insertContentTool.ts b/src/core/tools/insertContentTool.ts
@@ -140,8 +140,6 @@ export async function insertContentTool(
 			return
 		}
 
-		// Checkpoint is now created in askApproval function before this point (fixes #4827)
-
 		// Call saveChanges to update the DiffViewProvider properties
 		await cline.diffViewProvider.saveChanges()
 
diff --git a/src/core/tools/multiApplyDiffTool.ts b/src/core/tools/multiApplyDiffTool.ts
@@ -302,7 +302,6 @@ Original error: ${errorMessage}`
 					updateOperationResult(opResult.path, { status: "approved" })
 				})
 
-				// Create checkpoint BEFORE processing batch operations (fixes #4827)
 				if (cline.enableCheckpoints) {
 					await cline.checkpointSave()
 				}
@@ -347,7 +346,6 @@ Original error: ${errorMessage}`
 							cline.didRejectTool = true
 						}
 
-						// Create checkpoint BEFORE processing approved operations (fixes #4827)
 						if (hasAnyApproval && cline.enableCheckpoints) {
 							await cline.checkpointSave()
 						}
@@ -377,7 +375,6 @@ Original error: ${errorMessage}`
 							cline.didRejectTool = true
 						}
 
-						// Create checkpoint BEFORE processing approved operations (fixes #4827)
 						if (hasAnyApproval && cline.enableCheckpoints) {
 							await cline.checkpointSave()
 						}
@@ -400,8 +397,6 @@ Original error: ${errorMessage}`
 			updateOperationResult(opResult.path, { status: "approved" })
 		}
 
-		// Checkpoint is now created in askApproval function before this point (fixes #4827)
-
 		// Process approved operations
 		const results: string[] = []
 
diff --git a/src/core/tools/searchAndReplaceTool.ts b/src/core/tools/searchAndReplaceTool.ts
@@ -226,8 +226,6 @@ export async function searchAndReplaceTool(
 			return
 		}
 
-		// Checkpoint is now created in askApproval function before this point (fixes #4827)
-
 		// Call saveChanges to update the DiffViewProvider properties
 		await cline.diffViewProvider.saveChanges()
 
diff --git a/src/core/tools/writeToFileTool.ts b/src/core/tools/writeToFileTool.ts
@@ -212,8 +212,6 @@ export async function writeToFileTool(
 				return
 			}
 
-			// Checkpoint is now created in askApproval function before this point (fixes #4827)
-
 			// Call saveChanges to update the DiffViewProvider properties
 			await cline.diffViewProvider.saveChanges()
 

Original file line number	Diff line number	Diff line change
`@@ -165,8 +165,6 @@ export async function applyDiffToolLegacy(`
`165`	`165`	`return`
`166`	`166`	`}`
`167`	`167`
`168`		`- // Checkpoint is now created in askApproval function before this point (fixes #4827)`
`169`		`-`
`170`	`168`	`// Call saveChanges to update the DiffViewProvider properties`
`171`	`169`	`await cline.diffViewProvider.saveChanges()`
`172`	`170`
Original file line number	Diff line number	Diff line change
`@@ -140,8 +140,6 @@ export async function insertContentTool(`
`140`	`140`	`return`
`141`	`141`	`}`
`142`	`142`
`143`		`- // Checkpoint is now created in askApproval function before this point (fixes #4827)`
`144`		`-`
`145`	`143`	`// Call saveChanges to update the DiffViewProvider properties`
`146`	`144`	`await cline.diffViewProvider.saveChanges()`
`147`	`145`
Original file line number	Diff line number	Diff line change
@@ -302,7 +302,6 @@ Original error: ${errorMessage}`
`302`	`302`	`updateOperationResult(opResult.path, { status: "approved" })`
`303`	`303`	`})`
`304`	`304`
`305`		`- // Create checkpoint BEFORE processing batch operations (fixes #4827)`
`306`	`305`	`if (cline.enableCheckpoints) {`
`307`	`306`	`await cline.checkpointSave()`
`308`	`307`	`}`
@@ -347,7 +346,6 @@ Original error: ${errorMessage}`
`347`	`346`	`cline.didRejectTool = true`
`348`	`347`	`}`
`349`	`348`
`350`		`- // Create checkpoint BEFORE processing approved operations (fixes #4827)`
`351`	`349`	`if (hasAnyApproval && cline.enableCheckpoints) {`
`352`	`350`	`await cline.checkpointSave()`
`353`	`351`	`}`
@@ -377,7 +375,6 @@ Original error: ${errorMessage}`
`377`	`375`	`cline.didRejectTool = true`
`378`	`376`	`}`
`379`	`377`
`380`		`- // Create checkpoint BEFORE processing approved operations (fixes #4827)`
`381`	`378`	`if (hasAnyApproval && cline.enableCheckpoints) {`
`382`	`379`	`await cline.checkpointSave()`
`383`	`380`	`}`
@@ -400,8 +397,6 @@ Original error: ${errorMessage}`
`400`	`397`	`updateOperationResult(opResult.path, { status: "approved" })`
`401`	`398`	`}`
`402`	`399`
`403`		`- // Checkpoint is now created in askApproval function before this point (fixes #4827)`
`404`		`-`
`405`	`400`	`// Process approved operations`
`406`	`401`	`const results: string[] = []`
`407`	`402`
Original file line number	Diff line number	Diff line change
`@@ -226,8 +226,6 @@ export async function searchAndReplaceTool(`
`226`	`226`	`return`
`227`	`227`	`}`
`228`	`228`
`229`		`- // Checkpoint is now created in askApproval function before this point (fixes #4827)`
`230`		`-`
`231`	`229`	`// Call saveChanges to update the DiffViewProvider properties`
`232`	`230`	`await cline.diffViewProvider.saveChanges()`
`233`	`231`
Original file line number	Diff line number	Diff line change
`@@ -212,8 +212,6 @@ export async function writeToFileTool(`
`212`	`212`	`return`
`213`	`213`	`}`
`214`	`214`
`215`		`- // Checkpoint is now created in askApproval function before this point (fixes #4827)`
`216`		`-`
`217`	`215`	`// Call saveChanges to update the DiffViewProvider properties`
`218`	`216`	`await cline.diffViewProvider.saveChanges()`
`219`	`217`