Fix <?php tag (microsoft#212631)

Fix microsoft/vscode-copilot-release#1107

Author: roblourens

src/vs/workbench/contrib/chat/common/chatViewModel.ts

@@ -274,24 +274,13 @@ export class ChatViewModel extends Disposable implements IChatViewModel {
 		const renderer = new marked.Renderer();
 		renderer.code = (value, languageId) => {
 			languageId ??= '';
-			const newText = this.fixCodeText(value, languageId);
-			this.codeBlockModelCollection.update(this._model.sessionId, model, codeBlockIndex++, { text: newText, languageId });
+			this.codeBlockModelCollection.update(this._model.sessionId, model, codeBlockIndex++, { text: value, languageId });
 			return '';
 		};
 
 		marked.parse(this.ensureFencedCodeBlocksTerminated(content), { renderer });
 	}
 
-	private fixCodeText(text: string, languageId: string): string {
-		if (languageId === 'php') {
-			if (!text.trim().startsWith('<')) {
-				return `<?php\n${text}\n?>`;
-			}
-		}
-
-		return text;
-	}
-
 	/**
 	 * Marked doesn't consistently render fenced code blocks that aren't terminated.
 	 *

src/vs/workbench/contrib/chat/common/codeBlockModelCollection.ts

@@ -64,7 +64,7 @@ export class CodeBlockModelCollection extends Disposable {
 		const entry = this.getOrCreate(sessionId, chat, codeBlockIndex);
 
 		const extractedVulns = extractVulnerabilitiesFromText(content.text);
-		const newText = extractedVulns.newText;
+		const newText = fixCodeText(extractedVulns.newText, content.languageId);
 		this.setVulns(sessionId, chat, codeBlockIndex, extractedVulns.vulnerabilities);
 
 		const textModel = (await entry.model).textEditorModel;
@@ -137,3 +137,13 @@ export class CodeBlockModelCollection extends Disposable {
 		};
 	}
 }
+
+function fixCodeText(text: string, languageId: string | undefined): string {
+	if (languageId === 'php') {
+		if (!text.trim().startsWith('<')) {
+			return `<?php\n${text}`;
+		}
+	}
+
+	return text;
+}