Enhance LanguageModelToolsService to handle image file widgets in tool invocation messages

[rebornix]

No description provided.

[Copilot]

Pull request overview

Updates the chat LanguageModelToolsService so tool invocations that already surface image outputs via markdown “file pill” links don’t also force-enable toolResultDetails purely due to image data in the tool result. This aims to reduce duplicated UI output when tools provide image files as inline file widgets.

Changes:

• Passes the current ChatToolInvocation into ensureToolDetails so it can inspect the invocation message.
• Adds detection for empty-text markdown links ([](uri) / [ ](uri)) that point to image files.
• Skips auto-populating toolResultDetails for image outputs when the invocation message already contains image file widgets.

Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/chat/browser/tools/languageModelToolsService.ts:989

• This introduces new behavior around suppressing toolResultDetails when the invocation message contains image file widgets, but there are no corresponding tests. There’s already a dedicated languageModelToolsService.test.ts; please add coverage for (1) an image toolResult.content with a toolResultMessage/pastTenseMessage containing [](file:///...png) so toolResultDetails stays unset, and (2) the same with query/fragment in the URI to ensure detection still works.

	private ensureToolDetails(dto: IToolInvocation, toolResult: IToolResult, toolData: IToolData, toolInvocation: ChatToolInvocation | undefined): void {
		if (!toolResult.toolResultDetails && (toolData.alwaysDisplayInputOutput || (this.toolResultHasImages(toolResult) && !this.toolInvocationMessageHasImageFileWidgets(toolInvocation)))) {
			toolResult.toolResultDetails = {