fix: improve insert content feedback and line handling

- Use askApproval helper for consistent feedback handling with applyDiffTool
- Fix line number display to be 1-based in messages
- Remove redundant error message since askApproval handles feedback

Signed-off-by: Eric Wheeler <[email protected]>

Author: Eric Wheeler

src/core/tools/insertContentTool.ts

@@ -204,16 +204,13 @@ export async function insertContentTool(
 		const completeMessage = JSON.stringify({
 			...sharedMessageProps,
 			diff,
-			lineNumber: lineNumber,
+			lineNumber: lineNumber + 1,
 		} satisfies ClineSayTool)
 
-		const didApprove = await cline
-			.ask("tool", completeMessage, false)
-			.then((response) => response.response === "yesButtonClicked")
+		const didApprove = await askApproval("tool", completeMessage)
 
 		if (!didApprove) {
 			await cline.diffViewProvider.revertChanges()
-			pushToolResult("Changes were rejected by the user.")
 			return
 		}