refactor: simplify loop syntax in combineApiRequests and XmlMatcher (RooCodeInc#3776)

Author: noritaka1166

src/shared/combineApiRequests.ts

@@ -43,9 +43,7 @@ export function combineApiRequests(messages: ClineMessage[]): ClineMessage[] {
 	const result: ClineMessage[] = []
 	const startedIndices: number[] = []
 
-	for (let i = 0; i < messages.length; i++) {
-		const message = messages[i]
-
+	for (const message of messages) {
 		if (message.type !== "say" || (message.say !== "api_req_started" && message.say !== "api_req_finished")) {
 			result.push(message)
 			continue

src/utils/xml-matcher.ts

@@ -42,8 +42,7 @@ export class XmlMatcher<Result = XmlMatcherResult> {
 	}
 
 	private _update(chunk: string) {
-		for (let i = 0; i < chunk.length; i++) {
-			const char = chunk[i]
+		for (const char of chunk) {
 			this.cached.push(char)
 			this.pointer++