feat: support single-quoted JSON strings in StreamingToolCallProcessor

Author: NaccOll

src/core/task/__tests__/tool-call-helper.spec.ts

@@ -310,4 +310,16 @@ describe("handleOpenaiToolCallStreaming", () => {
 		expect(xml).toContain("<echo>")
 		expect(xml).toContain("<msg>hi</msg>")
 	})
+	it("should delegate to processor.processChunk single quote", () => {
+		const processor = new StreamingToolCallProcessor()
+		const chunk = [
+			{
+				index: 0,
+				id: "call_b16cd3f11abe4188b791e50e",
+				function: { name: "read_file", arguments: `{"args": {'file': [{'path': 'a.js'}]}}` },
+			},
+		]
+		const result = handleOpenaiToolCallStreaming(processor, chunk, "openai")
+		expect(result.chunkContent).toContain("a.js")
+	})
 })

src/core/task/tool-call-helper.ts

@@ -48,6 +48,12 @@ class ToolCallProcessingState {
 	isEscaped = false
 	isStreamingStringValue = false
 
+	/**
+	 * Tracks the quote character used for the current string (' or ").
+	 * This enables support for both single-quoted and double-quoted JSON-like strings.
+	 */
+	quoteChar: string | null = null
+
 	// Stack to keep track of JSON objects ({) and arrays ([).
 	bracketStack: ("{" | "[")[] = []
 	// Stack to keep track of XML tags for generating closing tags correctly.
@@ -236,6 +242,11 @@ export class StreamingToolCallProcessor {
 		let xml = ""
 		const args = state.arguments
 
+		// Track which quote type started the string (single or double)
+		if (!state.quoteChar) {
+			state.quoteChar = null
+		}
+
 		while (state.cursor < args.length) {
 			const char = args[state.cursor]
 
@@ -260,10 +271,11 @@ export class StreamingToolCallProcessor {
 							// Stop processing this chunk and wait for the next one.
 							return xml
 						}
-					} else if (char === '"') {
-						// End of string value.
+					} else if (char === state.quoteChar) {
+						// End of string value (support both single and double quote).
 						state.inString = false
 						state.isStreamingStringValue = false
+						state.quoteChar = null
 						const parent = state.bracketStack[state.bracketStack.length - 1]
 						if (parent === "{") {
 							const tag = state.xmlTagStack.pop()!
@@ -283,11 +295,11 @@ export class StreamingToolCallProcessor {
 					if (char === "\\" && !state.isEscaped) {
 						state.currentString += "\\"
 						state.isEscaped = true
-					} else if (char === '"' && !state.isEscaped) {
+					} else if (char === state.quoteChar && !state.isEscaped) {
 						state.inString = false
 						let finalString
 						try {
-							finalString = JSON.parse('"' + state.currentString + '"')
+							finalString = JSON.parse('"' + state.currentString.replace(/"/g, '\\"') + '"')
 						} catch (e) {
 							finalString = state.currentString
 						}
@@ -298,6 +310,7 @@ export class StreamingToolCallProcessor {
 						xml += `${this.getIndent(indentLevel)}${this.onOpenTag(finalString, toolName)}`
 						state.parserState = ParserState.EXPECT_COLON
 						state.currentString = ""
+						state.quoteChar = null
 					} else {
 						state.currentString += char
 						state.isEscaped = false
@@ -395,6 +408,8 @@ export class StreamingToolCallProcessor {
 					}
 					break
 				case '"':
+				case "'":
+					// Support both double and single quote for string start
 					if (state.parserState === ParserState.EXPECT_VALUE) {
 						// We've encountered the start of a string that is a JSON value.
 						state.isStreamingStringValue = true
@@ -403,6 +418,7 @@ export class StreamingToolCallProcessor {
 						state.isStreamingStringValue = false
 					}
 					state.inString = true
+					state.quoteChar = char // Track which quote started the string
 					break
 				case ":":
 					if (state.parserState === ParserState.EXPECT_COLON) {