chore: fix typos in comment-out (#4095)

* chore: fix typos in comment-out

* chore: revert unrelated fixes

src/api/providers/__tests__/bedrock-custom-arn.test.ts

@@ -69,7 +69,7 @@ describe("Bedrock ARN Handling", () => {
 	describe("parseArn direct tests", () => {
 		it("should correctly extract modelType and modelId from foundation-model ARN", () => {
 			const handler = createHandler()
-			//note: properly formated foundation-model ARNs don't have an account id.
+			//note: properly formatted foundation-model ARNs don't have an account id.
 			const arn = "arn:aws:bedrock:us-east-1::foundation-model/anthropic.claude-3-sonnet-20240229-v1:0"
 
 			// Access the private method using type casting
@@ -79,7 +79,7 @@ describe("Bedrock ARN Handling", () => {
 			expect(result.isValid).toBe(true)
 			expect(result.modelType).toBe("foundation-model")
 
-			//verify the id is not the ARN for foudation models, but the ID
+			//verify the id is not the ARN for foundation models, but the ID
 			expect(result.modelId).toBe("anthropic.claude-3-sonnet-20240229-v1:0")
 			expect(result.crossRegionInference).toBe(false)
 		})

src/api/providers/bedrock.ts

@@ -124,7 +124,7 @@ export class AwsBedrockHandler extends BaseProvider implements SingleCompletionH
 		let region = this.options.awsRegion
 
 		// process the various user input options, be opinionated about the intent of the options
-		// and determine the model to use during inference and for cost caclulations
+		// and determine the model to use during inference and for cost calculations
 		// There are variations on ARN strings that can be entered making the conditional logic
 		// more involved than the non-ARN branch of logic
 		if (this.options.awsCustomArn) {

src/core/assistant-message/parseAssistantMessage.ts

@@ -61,7 +61,7 @@ export function parseAssistantMessage(assistantMessage: string): AssistantMessag
 				// contain the closing tag, in which case the param would have
 				// closed and we end up with the rest of the file contents here.
 				// To work around this, we get the string between the starting
-				// ontent tag and the LAST content tag.
+				// content tag and the LAST content tag.
 				const contentParamName: ToolParamName = "content"
 
 				if (currentToolUse.name === "write_to_file" && accumulator.endsWith(`</${contentParamName}>`)) {

src/core/task/Task.ts

@@ -1320,7 +1320,7 @@ export class Task extends EventEmitter<ClineEvents> {
 						// `userContent` has a tool rejection, so interrupt the
 						// assistant's response to present the user's feedback.
 						assistantMessage += "\n\n[Response interrupted by user feedback]"
-						// Instead of setting this premptively, we allow the
+						// Instead of setting this preemptively, we allow the
 						// present iterator to finish and set
 						// userMessageContentReady when its ready.
 						// this.userMessageContentReady = true

src/core/webview/ClineProvider.ts

@@ -369,7 +369,7 @@ export class ClineProvider
 			setPanel(webviewView, "sidebar")
 		}
 
-		// Initialize out-of-scope variables that need to recieve persistent global state values
+		// Initialize out-of-scope variables that need to receive persistent global state values
 		this.getState().then(
 			({
 				terminalShellIntegrationTimeout = Terminal.defaultShellIntegrationTimeout,
@@ -414,7 +414,7 @@ export class ClineProvider
 				: this.getHtmlContent(webviewView.webview)
 
 		// Sets up an event listener to listen for messages passed from the webview view context
-		// and executes code based on the message that is recieved
+		// and executes code based on the message that is received
 		this.setWebviewMessageListener(webviewView.webview)
 
 		// Subscribe to code index status updates if the manager exists
@@ -497,7 +497,7 @@ export class ClineProvider
 	}
 
 	// When initializing a new task, (not from history but from a tool command
-	// new_task) there is no need to remove the previouse task since the new
+	// new_task) there is no need to remove the previous task since the new
 	// task is a subtask of the previous one, and when it finishes it is removed
 	// from the stack and the caller is resumed in this way we can have a chain
 	// of tasks, each one being a sub task of the previous one until the main
@@ -722,7 +722,7 @@ export class ClineProvider
 		/*
 		content security policy of your webview to only allow scripts that have a specific nonce
 		create a content security policy meta tag so that only loading scripts with a nonce is allowed
-		As your extension grows you will likely want to add custom styles, fonts, and/or images to your webview. If you do, you will need to update the content security policy meta tag to explicity allow for these resources. E.g.
+		As your extension grows you will likely want to add custom styles, fonts, and/or images to your webview. If you do, you will need to update the content security policy meta tag to explicitly allow for these resources. E.g.
 				<meta http-equiv="Content-Security-Policy" content="default-src 'none'; style-src ${webview.cspSource}; font-src ${webview.cspSource}; img-src ${webview.cspSource} https:; script-src 'nonce-${nonce}';">
 		- 'unsafe-inline' is required for styles due to vscode-webview-toolkit's dynamic style injection
 		- since we pass base64 images to the webview, we need to specify img-src ${webview.cspSource} data:;
@@ -760,7 +760,7 @@ export class ClineProvider
 
 	/**
 	 * Sets up an event listener to listen for messages passed from the webview context and
-	 * executes code based on the message that is recieved.
+	 * executes code based on the message that is received.
 	 *
 	 * @param webview A reference to the extension webview
 	 */

src/utils/shell.ts

@@ -91,7 +91,7 @@ function getWindowsShellFromVSCode(): string | null {
 
 	// If the profile name indicates PowerShell, do version-based detection.
 	// In testing it was found these typically do not have a path, and this
-	// implementation manages to deductively get the corect version of PowerShell
+	// implementation manages to deductively get the correct version of PowerShell
 	if (defaultProfileName.toLowerCase().includes("powershell")) {
 		if (profile?.path) {
 			// If there's an explicit PowerShell path, return that